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 state only when to use the skill #129

Merged
jercik merged 1 commit from fix/doc-drift-description-routing into main 2026-10-08 18:08:20 +00:00
Conversation 2 Commits 1 Files changed 1 +1 -1
jercik commented 2026-10-08 14:21:30 +00:00
Owner
Copy link

The verify-doc-drift description opens with a task summary and then states the audit intent twice more, under "Use when" and "Triggers on". It now follows the "Use when" form most skills here use and keeps the intent once.

Follow-up to a review finding on #122, which did not touch the description. I did not run a routing experiment to check that the shorter text selects the skill as often.

🤖 Generated with Claude Code

The `verify-doc-drift` description opens with a task summary and then states the audit intent twice more, under "Use when" and "Triggers on". It now follows the "Use when" form most skills here use and keeps the intent once. Follow-up to a review finding on [#122](https://code.j4k.dev/j4k-oss/agent-skills/pulls/122#issuecomment-138703), which did not touch the description. I did not run a routing experiment to check that the shorter text selects the skill as often. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix: doc drift description should state only when to use the skill
All checks were successful
commit-msg / commitlint (pull_request) Successful in 24s
Details
Node tests / node:test (pull_request) Successful in 3m11s
Details
Review / Review (pull_request_target) Successful in 3m17s
Details
ac953ed968
 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
forgejo-actions commented 2026-10-08 14:24:46 +00:00
Copy link

Review 01M4DY84PVE8Z4J4FMTKMXR021 — head ac953ed968a0e672409b735111c254294b9e667e

Review — j4k-oss/agent-skills @ 93212450b5

Scope: diff against base tree 8e004ccf90c2
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): claims-emitted
  • project-docs (whole/default): no-claims
  • test-trimming: skipped-by-dispatch — no tests changed
  • restated-sets: skipped-by-dispatch — single-line frontmatter description change with no restated sets

Other claims

  • grounding-pending (0)
  • ungrounded (0)
  • rejected (0)
  • duplicate-of (0)
  • unadjudicated (1)
    • 01M4DYC2KJPNF45VMV9S99H8NV low — The fix-direction rule repeats the opening contract and category definition

Coverage

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

lens part arm unit status runs loss
general-bug whole default no-claims 1 no
writing-quality whole default claims-emitted 1 no
project-docs whole default no-claims 1 no
  • test-trimming — skipped-by-dispatch: no tests changed
  • restated-sets — skipped-by-dispatch: single-line frontmatter description change with no restated sets
<!-- review:summary --> **Review** `01M4DY84PVE8Z4J4FMTKMXR021` — head `ac953ed968a0e672409b735111c254294b9e667e` # Review — j4k-oss/agent-skills @ 93212450b52c Scope: diff against base tree `8e004ccf90c2` 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): claims-emitted - project-docs (whole/default): no-claims - test-trimming: skipped-by-dispatch — no tests changed - restated-sets: skipped-by-dispatch — single-line frontmatter description change with no restated sets ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (1) - `01M4DYC2KJPNF45VMV9S99H8NV` low — The fix-direction rule repeats the opening contract and category definition ## Coverage Coverage pass: 01M4DY8FYCKHMJS4AGN4T4HB2R Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | claims-emitted | 1 | no | | project-docs | whole | default | no-claims | 1 | no | - test-trimming — skipped-by-dispatch: no tests changed - restated-sets — skipped-by-dispatch: single-line frontmatter description change with no restated sets
🚀 1
jercik commented 2026-10-08 18:07:56 +00:00
Author
Owner
Copy link

Unadjudicated claim 01M4DYC2KJPNF45VMV9S99H8NV (low, "The fix-direction rule repeats the opening contract and category definition"): checked against skills/verify-doc-drift/SKILL.md. The first two sentences of "Fix direction and missing features" restate line 8 and the code-drift definition, so the duplication is real. It predates this PR, which changes only the frontmatter description, so it does not hold the merge. Tracked in #131.

Unadjudicated claim `01M4DYC2KJPNF45VMV9S99H8NV` (low, "The fix-direction rule repeats the opening contract and category definition"): checked against `skills/verify-doc-drift/SKILL.md`. The first two sentences of "Fix direction and missing features" restate line 8 and the `code-drift` definition, so the duplication is real. It predates this PR, which changes only the frontmatter description, so it does not hold the merge. Tracked in https://code.j4k.dev/j4k-oss/agent-skills/pulls/131.
🚀 1
jercik merged commit 5decb83d5f into main 2026-10-08 18:08:20 +00:00
jercik deleted branch fix/doc-drift-description-routing 2026-10-08 18:08:20 +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!129
No description provided.