-
-
Notifications
You must be signed in to change notification settings - Fork 89
feat(github-app): per-file/line check-run annotations #537
Copy link
Copy link
Closed as not planned
3 / 33 of 3 issues completedClosed as not planned
3 / 33 of 3 issues completed
Copy link
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.roadmapOn the Wave-2 agent-layer roadmap board (project 9)On the Wave-2 agent-layer roadmap board (project 9)
Milestone
Description
Metadata
Metadata
Assignees
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.roadmapOn the Wave-2 agent-layer roadmap board (project 9)On the Wave-2 agent-layer roadmap board (project 9)
Parent roadmap: #525
Parent phase: #528
Background
The check-run surface exists (#272, closed) but the code only sends
{title, summary, text}— the GitHub Checksoutput.annotations[]array is unused, so maintainers can't see per-file/line hotspots.Goal
Add per-file/line annotations to the
Gittensory Contextcheck for duplicate-overlap, missing-test, and slop hotspots.Current Behavior
createOrUpdateNamedCheckRun(src/github/app.ts) passes only title/summary/text;grep annotationsover the check path is empty.Desired Behavior
output.annotations[](path + line + level + message), cap 50 with a "…N more" rollup.buildCollisionReport+PullRequestFileRecord. Annotate the Context check only; keep Gate a clean binary for branch protection.checkRunDetailLevel.Implementation Requirements
Public/Private Output Boundaries
sanitizeForCheckRun. No raw trust/credibility numbers or reward language.Acceptance Criteria
Testing Requirements
npm run test:ci, 97%+ coverage. Unit tests for annotation building + the 50-cap; sanitizer tests for messages.Sub-issues