fix(kyberforge): trim forge description and body to ADR-0020 budgets #145

Claude · 2026-09-30T07:16:36Z

Claude commented

2026-09-30 07:16:36 +00:00

Summary

Trim the forge skill under the ADR-0020 targets: description 250 characters (target 250), body 595 words (target 600).
The description now ends in three boundary clauses (Not a named skill -> skill-author, and likewise for agent-author and apm-workflow).
The context: fork vs /fork gotcha moved from SKILL.md to references/author-routes.md, the only place the fork-or-inline choice is made. references/sources.md and the source_keys frontmatter follow.
No routing row, gate or behaviour changed. The "matches no row" and chaining paragraphs stay in the body, trimmed in place, because no reference is read in those cases.
Version bumps: forge metadata 1.0.1 -> 1.0.2, kyberforge package 2.0.1 -> 2.0.2, plus the matching executables.allow key in the root apm.yml and the regenerated .claude-plugin/marketplace.json that the pre-push gates require.

Fixes #143

Test plan

factory-audit on plugins/kyberforge/.apm/skills/forge/, run twice in clean contexts: no FAIL or SUGGESTION findings, one provenance INFO that was checked by hand.
Independent clean-context change review: ship, no blockers. Its three nits (marketplace cue dropped from the description, one example phrase dropped, package version bump) are either accepted for the 250-character budget or fixed here.
Pre-push hooks pass, including the test suite, check-executables-allow-sync and apm pack --check-clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KkT7RSDwDbmrM9T34b6sTi

## Summary - Trim the `forge` skill under the ADR-0020 targets: description 250 characters (target 250), body 595 words (target 600). - The description now ends in three boundary clauses (`Not a named skill -> skill-author`, and likewise for `agent-author` and `apm-workflow`). - The `context: fork` vs `/fork` gotcha moved from `SKILL.md` to `references/author-routes.md`, the only place the fork-or-inline choice is made. `references/sources.md` and the `source_keys` frontmatter follow. - No routing row, gate or behaviour changed. The "matches no row" and chaining paragraphs stay in the body, trimmed in place, because no reference is read in those cases. - Version bumps: `forge` metadata 1.0.1 -> 1.0.2, `kyberforge` package 2.0.1 -> 2.0.2, plus the matching `executables.allow` key in the root `apm.yml` and the regenerated `.claude-plugin/marketplace.json` that the pre-push gates require. Fixes #143 ## Test plan - [x] `factory-audit` on `plugins/kyberforge/.apm/skills/forge/`, run twice in clean contexts: no FAIL or SUGGESTION findings, one provenance INFO that was checked by hand. - [x] Independent clean-context change review: ship, no blockers. Its three nits (marketplace cue dropped from the description, one example phrase dropped, package version bump) are either accepted for the 250-character budget or fixed here. - [x] Pre-push hooks pass, including the test suite, `check-executables-allow-sync` and `apm pack --check-clean`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01KkT7RSDwDbmrM9T34b6sTi

Claude added the Kind/Enhancement

Priority

Medium

3

labels 2026-09-30 07:16:36 +00:00

Claude added 2 commits 2026-09-30 07:16:36 +00:00

fix(kyberforge): trim forge description and body to ADR-0020 budgets cacfa1b374

factory-audit flagged forge's description and body as over the ADR-0020
targets (250 chars / 600 words). The description now uses three boundary
clauses and the body is 595 words. The context: fork vs /fork gotcha moved
to references/author-routes.md, the only place the fork-or-inline choice
is made. No routing row or behaviour changed.

Fixes: #143

Co-Authored-By: Claude Code <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KkT7RSDwDbmrM9T34b6sTi

chore(kyberforge): sync executables allow key and marketplace to 2.0.2 8c583b5fd5

The kyberforge version bump needs the matching executables.allow key in
the root apm.yml (ADR-0019) and a regenerated checked-in marketplace.json,
or the pre-push gates check-executables-allow-sync and apm pack
--check-clean fail.

Refs: #143

Co-Authored-By: Claude Code <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KkT7RSDwDbmrM9T34b6sTi

Claude added this to the Skills & Agents milestone 2026-09-30 07:16:41 +00:00

Defame1297 merged commit 3ff0741857 into main

2026-09-30 15:27:41 +00:00

Defame1297 deleted branch fix/forge-adr-0020-budgets

2026-09-30 15:27:41 +00:00

Defame1297 referenced this issue from a commit

2026-09-30 15:27:43 +00:00

Merge pull request 'fix(kyberforge): trim forge description and body to ADR-0020 budgets' (#145) from fix/forge-adr-0020-budgets into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Defame1297/holocron#145