j4k-oss/agent-skills
Watch
1
0
Fork
You've already forked agent-skills
 0
Code Issues 6 Pull requests Releases Packages Activity Actions

fix: doc drift description should not claim every finding is verified against the code #128

Merged
jercik merged 1 commit from fix/doc-drift-description-verified-claim into main 2026-10-08 12:32:45 +00:00
Conversation 1 Commits 1 Files changed 1 +1 -1
jercik commented 2026-10-08 11:18:11 +00:00
Owner
Copy link

Removes the method summary from the verify-doc-drift catalog description, which says every finding is "adversarially verified against the code". That is false for obvious and duplicate findings, which are cited from documentation.

Follow-up to a review finding on #122, which did not introduce the text. This follows the shortening in #125.

🤖 Generated with Claude Code

Removes the method summary from the `verify-doc-drift` catalog description, which says every finding is "adversarially verified against the code". That is false for `obvious` and `duplicate` findings, which are cited from documentation. Follow-up to a review finding on [#122](https://code.j4k.dev/j4k-oss/agent-skills/pulls/122), which did not introduce the text. This follows the shortening in #125. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix: shorten the doc drift catalog description
All checks were successful
commit-msg / commitlint (pull_request) Successful in 19s
Details
Review / Review (pull_request_target) Successful in 1m27s
Details
Node tests / node:test (pull_request) Successful in 2m14s
Details
cd3150db16 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
forgejo-actions commented 2026-10-08 11:19:37 +00:00
Copy link

Review 01M4DKRGYQZS0R68B737NERHEG — head cd3150db16dc1bfab52645cbce834880194109a2

Review — j4k-oss/agent-skills @ 0321335b2c

Scope: diff against base tree 0818591dbd8d
Status: dispatched — coverage complete (3/3 slots terminal)
Facts: current review-wide projection

Computed under:

{
  "abandonment": "abandonment-v1",
  "anchor_recipe": 1,
  "batch_policy": "batch-v1",
  "coverage": "coverage-v3",
  "dispatch_policy": "dispatch-v2",
  "grounder_version": 1,
  "grounding_read_rule": "grounding-read-v1",
  "promotion_policy": "promotion-v1",
  "report": "report-v4",
  "tally": "tally-v1",
  "triage_settle": "triage-settle-v2"
}

Findings (0)

No findings survived.

Reviewed:

  • general-bug (whole/default): no-claims
  • writing-quality (whole/default): no-claims
  • project-docs (whole/default): no-claims
  • test-trimming: skipped-by-dispatch — no tests changed; only a skill description line was edited
  • restated-sets: skipped-by-dispatch — change shortens a single description and adds no enumerated sets or duplicated lists

Other claims

  • grounding-pending (0)
  • ungrounded (0)
  • rejected (0)
  • duplicate-of (0)
  • unadjudicated (0)

Coverage

Coverage pass: 01M4DKS4CDSSFH14JCX87J8TFS
Accounting: complete
Slot health: healthy

lens part arm unit status runs loss
general-bug whole default no-claims 1 no
writing-quality whole default no-claims 1 no
project-docs whole default no-claims 1 no
  • test-trimming — skipped-by-dispatch: no tests changed; only a skill description line was edited
  • restated-sets — skipped-by-dispatch: change shortens a single description and adds no enumerated sets or duplicated lists
<!-- review:summary --> **Review** `01M4DKRGYQZS0R68B737NERHEG` — head `cd3150db16dc1bfab52645cbce834880194109a2` # Review — j4k-oss/agent-skills @ 0321335b2c43 Scope: diff against base tree `0818591dbd8d` Status: dispatched — coverage complete (3/3 slots terminal) Facts: current review-wide projection Computed under: ```json { "abandonment": "abandonment-v1", "anchor_recipe": 1, "batch_policy": "batch-v1", "coverage": "coverage-v3", "dispatch_policy": "dispatch-v2", "grounder_version": 1, "grounding_read_rule": "grounding-read-v1", "promotion_policy": "promotion-v1", "report": "report-v4", "tally": "tally-v1", "triage_settle": "triage-settle-v2" } ``` ## Findings (0) No findings survived. Reviewed: - general-bug (whole/default): no-claims - writing-quality (whole/default): no-claims - project-docs (whole/default): no-claims - test-trimming: skipped-by-dispatch — no tests changed; only a skill description line was edited - restated-sets: skipped-by-dispatch — change shortens a single description and adds no enumerated sets or duplicated lists ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (0) ## Coverage Coverage pass: 01M4DKS4CDSSFH14JCX87J8TFS Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | no-claims | 1 | no | | project-docs | whole | default | no-claims | 1 | no | - test-trimming — skipped-by-dispatch: no tests changed; only a skill description line was edited - restated-sets — skipped-by-dispatch: change shortens a single description and adds no enumerated sets or duplicated lists
🚀 1
jercik merged commit 91b2504261 into main 2026-10-08 12:32:45 +00:00
jercik deleted branch fix/doc-drift-description-verified-claim 2026-10-08 12:32:45 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
j4k-oss/agent-skills!128
No description provided.