[Vivswan]

Before

src/github/api.ts, three request sites, each with its own header literal:
  tryList()     headers: { accept: "application/vnd.github+json", "x-github-api-version": this.apiVersion }
  request()     headers: { accept: options?.accept ?? "application/vnd.github+json", "x-github-api-version": this.apiVersion }
  tryGraphql()  headers: { accept: "application/vnd.github+json", "x-github-api-version": this.apiVersion }

After

constructor   this.octokit.request = this.octokit.request.defaults({ headers: { accept, "x-github-api-version" } })
  tryList()     (no headers)
  request()     ...(options?.accept === undefined ? {} : { headers: { accept: options.accept } })
  tryGraphql()  (no headers)

How

• Octokit's request defaults carry the two headers every request sends, set once after construction.
• The raw media-type read still passes its own accept, which the defaults merge lets override.
• The plugins' hooks stay: defaults() keeps the wrapped request hook, so retry, throttling, and the request log see every call.
• Nothing on the wire changes: the same two headers with the same values leave each path.

Proof

• Census: grep -c '"x-github-api-version"' src/github/api.ts, 3 before, 1 after.
• E2e: bun test/e2e/run.ts --sections labels,branches, 98/98; the mock refuses any request without the version header, on the list, request, and GraphQL paths alike.
• Tests: bun run test test/github, 123 pass, none added.
• Gates: typecheck and biome clean.

Technical details

• Library: @octokit/core 7.0.8 (already a dependency), request.defaults({ headers }); the constructor takes no header option beyond userAgent and timeZone, so the defaults are set right after construction.
• Accounting: 1 file, +8 -14.
• The paginate iterator's first page inherits the defaults too: the walk's request wrapper reuses the reassigned request's endpoint and defaults.
• No new test: a pin of the defaults would restate the source; the e2e mock's refusal is the wire-level pin.

[Copilot]

Copilot review overview

🟢 Approval recommended

The small extraction preserves supported request behavior, and no blocking issues remain.

Review effort: Balanced
Findings: None

What changed in this PR

Centralizes request-header construction in GitHubApi without changing supported header values.

Changes:

• Adds a private builder for the accept and API-version headers.
• Uses it for REST requests, pagination, and GraphQL, preserving custom accept values.

File	Description
src/​github/​api.ts	Replaces three inline header objects with one shared builder.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟢 Approval recommended

The focused refactor preserves header values and request hooks, with no unresolved findings.

Review effort: Balanced
Findings: None