Started by GitHub push by ti-chi-bot[bot]Started by GitHub push by ti-chi-bot[bot]00159161591644016440161.0223382233806635464206420166383691152ea8fa148a7c0a037fdb2d89e217edbc283691152ea8fa148a7c0a037fdb2d89e217edbc2origin/main83691152ea8fa148a7c0a037fdb2d89e217edbc283691152ea8fa148a7c0a037fdb2d89e217edbc2origin/main83691152ea8fa148a7c0a037fdb2d89e217edbc283691152ea8fa148a7c0a037fdb2d89e217edbc2origin/mainhttps://github.com/PingCAP-QE/ci.githttps://do.pingcap.net/jenkins/blue/organizations/jenkins/seed/detail/seed/1663/artifactshttps://do.pingcap.net/jenkins/blue/organizations/jenkins/seed/detail/seed/1663/changeshttps://do.pingcap.net/jenkins/blue/organizations/jenkins/seed/detail/seed/1663/https://do.pingcap.net/jenkins/blue/organizations/jenkins/seed/detail/seed/1663/testsfalse#16634401647130seed #16631663falsefalse16633676838SUCCESS1777543476054https://do.pingcap.net/jenkins/job/seed/1663/default-41jzvtools/reporters/ci/flaky-tests/core/GithubIssueManager.tstools/reporters/ci/flaky-tests/render/HtmlRenderer.tstools/reporters/ci/flaky-tests/deno.locktools/reporters/ci/flaky-tests/core/GithubIssueManager.test.tstools/reporters/ci/flaky-tests/deno.json83691152ea8fa148a7c0a037fdb2d89e217edbc21777543449000https://do.pingcap.net/jenkins/user/noreplynoreplynoreply@github.comrefactor(flaky-tests): simplify GitHub types and API usage, remove custom wrappers (#4567) ## Summary Simplify GitHub-related code in `flaky-tests`: remove custom wrapper layer and use Octokit types/APIs directly. ### Changes 1. **Remove custom `GitHubClient` wrapper class** (~120 lines) `GitHubClient` wrapped 5 Octokit API methods (`searchIssues`, `createIssue`, `reopenIssue`, `addLabels`, `addComment`), each manually mapping response fields to a custom `GitHubIssueApi` interface. Now calls Octokit's native REST APIs directly. 2. **Remove `GitHubIssueApi` and `OctokitLike` interfaces** No more custom interfaces — use `Octokit` type directly from `@octokit/rest`. 3. **Flatten helper functions** Extract Octokit calls into module-level standalone functions (`searchIssues`, `createIssue`, etc.) instead of class instance methods. 4. **Test at Octokit mock level** Tests no longer mock `GitHubClient`. Instead, inject a mock Octokit instance via the `octokit` constructor option. 5. **Fix GitHub API deprecation warning** Add `X-GitHub-Api-Version: 2026-03-10` header to use the latest API version. ### Files Changed - `tools/reporters/ci/flaky-tests/core/GithubIssueManager.ts` — major simplification - `tools/reporters/ci/flaky-tests/core/GithubIssueManager.test.ts` — updated mocking strategy - `tools/reporters/ci/flaky-tests/deno.json` / `deno.lock` — dependency updates - `tools/reporters/ci/flaky-tests/render/HtmlRenderer.ts` — label text optimization --------- Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io> Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> 2026-04-30 10:04:09 +000083691152ea8fa148a7c0a037fdb2d89e217edbc2refactor(flaky-tests): simplify GitHub types and API usage, removeedittools/reporters/ci/flaky-tests/core/GithubIssueManager.tsedittools/reporters/ci/flaky-tests/render/HtmlRenderer.tsedittools/reporters/ci/flaky-tests/deno.lockedittools/reporters/ci/flaky-tests/core/GithubIssueManager.test.tsedittools/reporters/ci/flaky-tests/deno.jsongithttps://do.pingcap.net/jenkins/user/noreplynoreplynoreply