Skip to content

refactor(aidd-context): simplify skill generation rules - #1028

Draft
alexsoyes wants to merge 2 commits into
nextfrom
codex/skill-generation-rules
Draft

alexsoyes wants to merge 2 commits into
nextfrom
codex/skill-generation-rules

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

🎯 What & why

Keep skill authoring concise: integrate observable checks into the process and reserve optional rules for constraints. Extracted from #1027 without changing its agent workflows.

🛠️ How it works

  • Contract: replace separate Test sections with Process checks and optional Rules.
  • Templates: align the generator actions and scaffolds; retain confirmation, path boundaries, read-back, and repair checks.
  • Wording: bold important opening words in bullets without procedural labels; keep one canonical home per rule and prefer positive instructions to never.

🧪 How to verify

  • Static checks: node scripts/check-architecture-rules.js passes; commit hooks pass 681 script tests. The CLI suite passes 7,025 tests, one skipped.
  • Scope: nine files under 04-skill-generate plus its architecture sentence. No agent, implementation, or reload changes.
  • Runtime: a new generated skill was not exercised for this extraction.

🔗 Linked issue

Refs #983. Split from #1027; both target next independently.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant