fix(sweep): mine the Greptile summary body for findings, not just inline comments#1707
Conversation
| gh api repos/optave/ops-codegraph-tool/issues/<number>/comments --paginate \ | ||
| --jq '.[] | select(.user.login|test("greptile";"i")) | .body' | ||
| # …and as a review body (Greptile sometimes posts the summary here instead): | ||
| gh api repos/optave/ops-codegraph-tool/pulls/<number>/reviews --paginate \ | ||
| --jq '.[] | select(.user.login|test("greptile";"i")) | .body' | ||
| ``` |
There was a problem hiding this comment.
Wrong repository in step 2d.1
gh api commands
The new bash block references repos/optave/ops-codegraph-tool (this file's home repo), but every other gh api call in the skill targets repos/optave/codegraph — the repo whose PRs are actually swept (Step 1 uses --repo optave/codegraph). When a subagent executes step 2d.1, it queries the wrong repo and silently returns no Greptile comments, so the entire new mining logic becomes a no-op. Both occurrences of ops-codegraph-tool should be replaced with codegraph.
There was a problem hiding this comment.
Fixed in commit ad63cd2 — both gh api calls in step 2d.1 now reference repos/optave/codegraph, matching the file's convention for all other API calls in the skill. Verified there are no remaining ops-codegraph-tool references in the step 2d.1 bash block; the one remaining occurrence (in the follow-up-issue permalink template) is pre-existing and outside this PR's diff. Your own summary review of ad63cd2 confirms the calls now correctly target codegraph.
Problem
The
/sweepskill's addressing logic is inline-comment-centric. Greptile often raises actionable findings in its top-level summary — the### Greptile Summaryissue comment carrying aConfidence Score: N/5and prose review — that are not mirrored as inline review comments. A sweep that only iterates the inline comments gathered in step 2d therefore silently misses these summary-only findings. This is the single most common way a Greptile concern survives a sweep unaddressed.Fix
follow-upissue).Docs-only change to
.claude/skills/sweep/SKILL.md. Part of a cross-repo sweep-skill fix applying the same "mine the summary" step across repos that use the/sweepskill.