holocron

Defame1297/holocron

Fork 0

Commit Graph

Author	SHA1	Message	Date
Defame1297	4a5c3c0cff	feat(kyberforge): enforce the ADR-0020 context contract for skills and agents Skill name+description pairs are preloaded into every session, costing ~6,200 tokens across 39 skills before any skill is invoked. The authoring rules mandated that growth: skill-author:104 and description-quality.md:21 both required padding, while skill-author:102 (the deflating rule) had no FAIL condition behind it. Gates (blocking, no baseline file): - description 250 chars SUGGESTION / 400 FAIL, measured on the folded YAML value - body-only 600 words SUGGESTION / 900 FAIL, independent of the unchanged whole-file 2770-word / 500-line spec backstop - every boundary-clause routing target must resolve to a real skill or agent; catches skill-improve, neuledge-context and gitea-labels - agents take the description gates but deliberately no body gate; a test pins that absence Vale: DescriptionOpener widened to ^This\b, new CompositionNote rule banning architecture notes from descriptions. 10 hits, 0 false positives. Kyberforge's own four skills retrofitted: descriptions 3,364 -> 938 chars (-72%), bodies 8,306 -> 2,487 words (-70%), all via the apm-workflow dispatch pattern. Fixes the skill-improve dangling route and the agent-author misroute to manual review. Also fixes a pre-existing false positive where any line-initial 'read ' was flagged as interactive input, which had already caused two scripts to be rewritten around it. Refs: ADR-0020	2026-08-14 21:13:13 +00:00
Defame1297	4ea9e21ead	fix(kyberforge): apply audit findings to agent-author skill - Fix Copilot format conflation: split CLI vs cloud/IDE in Step 3, clean copilot template to CLI-only fields, add body length limit (30k) - Add missing CC fields to Step 2: disallowedTools, skills, color, initialPrompt, background - Fix Gotchas: add WaitForMcpServers to unavailable tools list, add ExitPlanMode plan-mode carve-out - Make 'Use proactively' conditional (was unconditional directive) - Clarify audit invocation to 'Invoke kyberforge:agent-audit skill directly' - Add 'Would the agent get this wrong?' heuristic to Improve Step 4 - Add agent-audit co-install prerequisite check - Add pre-audit manual checklist to validate/close step - Add references/scripts.md for new-agent.sh conventions - Fix sources.md: remove template/script files from Contributing files (templates can't carry YAML provenance without leaking into user files), add 3 missing research slugs with (none) contributing files - Add context7-github-en-copilot to deployment-modes.md source_keys - Update README.md and references/README.md with new scripts.md entry Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0147vXtL5sP6vorDdqXGJJU9	2026-07-04 11:11:11 +00:00
Defame1297	ba68b09c87	feat(kyberforge): add agent-author skill for Claude Code and Copilot CLI agents ## Why skill-author explicitly excludes agent definition files ("Do not use to author agent definition files"). No factory skill existed to create or improve the .md / .agent.md files that define Claude Code subagents and Copilot CLI agents in a plugin, project, or user scope. This fills that gap. ## Implementation Notes Single-root scaffold convention: new-agent.sh <name> <root> derives both provider file paths from the root by convention — plugin scope (plugin.json present) writes both files into <root>/agents/; non-plugin scope writes .claude/agents/<name>.md and .github/agents/<name>.agent.md. This keeps input minimal while always generating both provider files. See ADR-0015. Routing is file-level (not directory-level like skill-author): neither file exists → create flow; at least one exists → improve flow; scaffold is a file-by-file no-op so retries are safe. No companion agent-audit skill — inline validation in the close step covers the simpler agent field contract. agent-audit is tracked as a follow-on. ## Impact Closes the skill-author gap for agent definitions. Follow-ons tracked in Gitea #11: agent-audit skill and --copilot-dest override flag for non-standard Copilot project paths. --- ADR: docs/adr/0015-agent-author-dual-provider-scaffold.md Refs: #10 Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-27 15:07:33 +00:00

Author

SHA1

Message

Date

Defame1297

4a5c3c0cff

feat(kyberforge): enforce the ADR-0020 context contract for skills and agents

Skill name+description pairs are preloaded into every session, costing
~6,200 tokens across 39 skills before any skill is invoked. The authoring
rules mandated that growth: skill-author:104 and description-quality.md:21
both required padding, while skill-author:102 (the deflating rule) had no
FAIL condition behind it.

Gates (blocking, no baseline file):
- description 250 chars SUGGESTION / 400 FAIL, measured on the folded
  YAML value
- body-only 600 words SUGGESTION / 900 FAIL, independent of the unchanged
  whole-file 2770-word / 500-line spec backstop
- every boundary-clause routing target must resolve to a real skill or
  agent; catches skill-improve, neuledge-context and gitea-labels
- agents take the description gates but deliberately no body gate; a test
  pins that absence

Vale: DescriptionOpener widened to ^This\b, new CompositionNote rule
banning architecture notes from descriptions. 10 hits, 0 false positives.

Kyberforge's own four skills retrofitted: descriptions 3,364 -> 938 chars
(-72%), bodies 8,306 -> 2,487 words (-70%), all via the apm-workflow
dispatch pattern. Fixes the skill-improve dangling route and the
agent-author misroute to manual review.

Also fixes a pre-existing false positive where any line-initial 'read '
was flagged as interactive input, which had already caused two scripts to
be rewritten around it.

Refs: ADR-0020

2026-08-14 21:13:13 +00:00

Defame1297

4ea9e21ead

fix(kyberforge): apply audit findings to agent-author skill

- Fix Copilot format conflation: split CLI vs cloud/IDE in Step 3,
  clean copilot template to CLI-only fields, add body length limit (30k)
- Add missing CC fields to Step 2: disallowedTools, skills, color,
  initialPrompt, background
- Fix Gotchas: add WaitForMcpServers to unavailable tools list,
  add ExitPlanMode plan-mode carve-out
- Make 'Use proactively' conditional (was unconditional directive)
- Clarify audit invocation to 'Invoke kyberforge:agent-audit skill directly'
- Add 'Would the agent get this wrong?' heuristic to Improve Step 4
- Add agent-audit co-install prerequisite check
- Add pre-audit manual checklist to validate/close step
- Add references/scripts.md for new-agent.sh conventions
- Fix sources.md: remove template/script files from Contributing files
  (templates can't carry YAML provenance without leaking into user files),
  add 3 missing research slugs with (none) contributing files
- Add context7-github-en-copilot to deployment-modes.md source_keys
- Update README.md and references/README.md with new scripts.md entry

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0147vXtL5sP6vorDdqXGJJU9

2026-07-04 11:11:11 +00:00

Defame1297

ba68b09c87

feat(kyberforge): add agent-author skill for Claude Code and Copilot CLI agents

## Why

skill-author explicitly excludes agent definition files ("Do not use to author
agent definition files"). No factory skill existed to create or improve the
.md / .agent.md files that define Claude Code subagents and Copilot CLI agents
in a plugin, project, or user scope. This fills that gap.

## Implementation Notes

Single-root scaffold convention: new-agent.sh <name> <root> derives both
provider file paths from the root by convention — plugin scope (plugin.json
present) writes both files into <root>/agents/; non-plugin scope writes
.claude/agents/<name>.md and .github/agents/<name>.agent.md. This keeps
input minimal while always generating both provider files. See ADR-0015.

Routing is file-level (not directory-level like skill-author): neither file
exists → create flow; at least one exists → improve flow; scaffold is a
file-by-file no-op so retries are safe.

No companion agent-audit skill — inline validation in the close step covers
the simpler agent field contract. agent-audit is tracked as a follow-on.

## Impact

Closes the skill-author gap for agent definitions. Follow-ons tracked in
Gitea #11: agent-audit skill and --copilot-dest override flag for non-standard
Copilot project paths.

---
ADR: docs/adr/0015-agent-author-dual-provider-scaffold.md
Refs: #10
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-06-27 15:07:33 +00:00

3 Commits