[hivecommons-hive]

Coverage annotation

src/utils/auth.ts getErrorDetails ends in a fallback that stringifies a non-object rejection. All three callers (checkOrgMember, checkCollaborator, the issue-comments read) pass it the rejection of an Octokit call, and @octokit/request wraps every failed fetch and every non-2xx response in a RequestError object, so no response or network fault the bundle can see reaches that return; only __tests__/utils/auth.test.ts:93 drives it, via mockRejectedValueOnce('socket hang up'). It was the single uncovered line left in npm run test:coverage:e2e on main.

This wraps the return in /* v8 ignore start */ … /* v8 ignore stop */, the same treatment 7a1df7a (#483) applied to the reset*() hooks. The defensive arm and its unit test stay in place; no behaviour changes. dist/ repacked (two comment lines only).

Files: src/utils/auth.ts (getErrorDetails), dist/index.js.

Verification (this branch, local)

• npm run build && npm run lint && npm run pack: clean, dist/index.js diff is the two comment lines
• npm run test:coverage:e2e: 83 files / 502 tests pass; All files 100 % lines, auth.ts 100 % lines — line 24 no longer listed (the column now shows only the branch-only folds at 14-17 already inventoried on [quality] coverage-e2e names auth.ts:24 as the only uncovered line on main; getErrorDetails' non-object arm is unreachable through dist/ (Octokit always rejects a RequestError) #507 and the negative-count BRDA artifact at 222)
• npm run test:coverage: 163 files / 2206 tests pass, 100 % lines / 99.75 % branches, thresholds met

Related Issue

Closes #507

---

Filed by quality agent (hold-gated mode). Human review required.

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

[hivecommons-hive]

Important

Held for human review by the hive's ACMM level gate.

This PR was opened by the "quality" agent while Hive policy required a human checkpoint for that agent. Agent PRs are held at ACMM L3–L5; an outreach PR that targets a repository this hive does not manage is held at every level because it publishes project-facing communication into someone else's project.

Hive will keep the hold label until a human removes it. Operators can make a deliberate one-off release during an ACMM level change with release_level_holds=true, but level changes never release this hold automatically.

[github-actions]

Please add a kind label with /kind feature, /kind failing-test or /kind cleanup.

[hivecommons-hive]

Review note from the quality lane, verified against main @ 88241f4 with the committed dist/index.js — not a request to change this PR's outcome, but its stated rationale is no longer accurate once #528 lands.

The ignore comment says "@octokit/request always wraps in a RequestError object, so no bundle input reaches this arm". That holds for every status the fake GitHub can return, but not for every fetch result: fetchWrapper reads fetchResponse.status after its try/catch (dist/index.js:33826), so a Response whose status getter throws propagates as-is. #528 adds __tests__/bundle/nonErrorRejectionPreload.cjs, which does exactly that for one named request. Pointed at /retitle with NON_ERROR_REJECT="GET /repos/Codertocat/Hello-World/collaborators/Codertocat", the bundle logs

::warning::encountered unexpected error checking collaborator status: status=unknown, message=GET /repos/Codertocat/Hello-World/collaborators/Codertocat rejected with a string, not an Error

— that status=unknown is getErrorDetails' non-object arm (auth.ts:14-17), reached through the real client.

Two consistent ways to resolve it; either is fine, the choice is the reviewer's:

1. keep the ignore but reword its reason to "no fake-GitHub status reaches this arm; see nonErrorRejectionPreload.cjs for the only input that does", or
2. drop the ignore and add a fourth case to __tests__/bundle/nonErrorRejectionArms.test.ts (after test(bundle): cover non-Error rejection paths including auth #528) with the /retitle run above, which drives the arm for real and keeps auth.ts at 100 % without a marker.

Nothing here touches this branch; the quality lane will not push to it.

---

🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 8404338

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

[mrbobbytables]

Superseded by merged #528. Its /retitle collaborator-rejection scenario reaches auth.ts:24 through the bundle, verifies the warning and denied write, and restores 100% auth line coverage without an ignore annotation. Defensive production code is retained. Other historical #507 findings remain tracked at their canonical destinations, including #550.