holycodex 0.16.3 → 0.16.4-dev.77.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (37) hide show
  1. package/README.md +23 -5
  2. package/dist/agent.js +142 -0
  3. package/dist/assets/plugin/plugin.json +2 -2
  4. package/dist/assets/plugin/skills/code-review/SKILL.md +5 -12
  5. package/dist/assets/plugin/skills/code-review/agents/openai.yaml +1 -1
  6. package/dist/assets/plugin/skills/commit/SKILL.md +23 -7
  7. package/dist/assets/plugin/skills/commit/agents/openai.yaml +1 -1
  8. package/dist/assets/plugin/skills/compress/SKILL.md +4 -6
  9. package/dist/assets/plugin/skills/compress/agents/openai.yaml +1 -1
  10. package/dist/assets/plugin/skills/context7-cli/SKILL.md +4 -10
  11. package/dist/assets/plugin/skills/context7-cli/agents/openai.yaml +1 -1
  12. package/dist/assets/plugin/skills/debugging/SKILL.md +4 -18
  13. package/dist/assets/plugin/skills/debugging/agents/openai.yaml +1 -1
  14. package/dist/assets/plugin/skills/grill-me/SKILL.md +21 -0
  15. package/dist/assets/plugin/skills/grill-me/agents/openai.yaml +6 -0
  16. package/dist/assets/plugin/skills/handoff/SKILL.md +5 -6
  17. package/dist/assets/plugin/skills/handoff/agents/openai.yaml +2 -2
  18. package/dist/assets/plugin/skills/operations/SKILL.md +11 -0
  19. package/dist/assets/plugin/skills/operations/agents/openai.yaml +6 -0
  20. package/dist/assets/plugin/skills/plan/SKILL.md +6 -9
  21. package/dist/assets/plugin/skills/plan/agents/openai.yaml +1 -1
  22. package/dist/assets/plugin/skills/plan-review/SKILL.md +5 -7
  23. package/dist/assets/plugin/skills/plan-review/agents/openai.yaml +1 -1
  24. package/dist/assets/plugin/skills/programming/SKILL.md +4 -7
  25. package/dist/assets/plugin/skills/programming/agents/openai.yaml +1 -1
  26. package/dist/assets/plugin/skills/refactor/SKILL.md +4 -7
  27. package/dist/assets/plugin/skills/refactor/agents/openai.yaml +1 -1
  28. package/dist/assets/plugin/skills/rules/SKILL.md +4 -6
  29. package/dist/assets/plugin/skills/rules/agents/openai.yaml +1 -1
  30. package/dist/assets/plugin/skills/stop-slop/SKILL.md +5 -34
  31. package/dist/assets/plugin/skills/stop-slop/agents/openai.yaml +1 -1
  32. package/dist/assets/plugin/skills/testing-quality.md +10 -26
  33. package/dist/assets/plugin/skills/writing-for-agents/SKILL-MECHANICS.md +6 -16
  34. package/dist/assets/plugin/skills/writing-for-agents/SKILL.md +29 -40
  35. package/dist/assets/plugin/skills/writing-for-agents/agents/openai.yaml +1 -1
  36. package/dist/index.js +103 -57
  37. package/package.json +8 -5
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "holycodex",
3
- "version": "0.16.3",
3
+ "version": "0.16.4",
4
4
  "description": "Root-directed capabilities and deterministic installed assets for HolyCodex.",
5
5
  "author": {
6
6
  "name": "David Basile Filho",
@@ -26,7 +26,7 @@
26
26
  "Code Intelligence"
27
27
  ],
28
28
  "defaultPrompt": [
29
- "Use HolyCodex to implement and verify this task."
29
+ "Activate HolyCodex capabilities for this task."
30
30
  ]
31
31
  }
32
32
  }
@@ -1,16 +1,9 @@
1
1
  ---
2
2
  name: code-review
3
- description: Use once after code or manifest implementation when Root needs adversarial review and bounded repair to a fixed point.
3
+ description: Use after implementation when Root needs adversarial code review and bounded repair to a fixed point.
4
4
  ---
5
5
 
6
- Use this skill once after implementation or manifest work. Inspect the actual
7
- integrated result to a fixed point before Root judges readiness. Apply
8
- [testing quality](../testing-quality.md) when judging new or modified tests.
9
-
10
- Inspect the assigned diff, callers, contracts, tests, compatibility, and
11
- artifact. Repair defects inside the review surface, verify each repair, and
12
- repeat until the review reaches a fixed point.
13
-
14
- Flag speculative abstractions, pointless file splits, oversized changes,
15
- unclear dependency direction, and tests that protect implementation detail.
16
- Confirm incidental generated output is absent before stopping.
6
+ Inspect the integrated implementation for correctness, safety, maintainability,
7
+ compatibility, and mergeability. Check callers, contracts, tests, and generated
8
+ artifacts. Repair defects inside the review surface. Flag implementation-detail
9
+ tests and speculative abstractions when they affect the assigned change.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Code review"
3
3
  short_description: "Review one implementation result to a fixed point."
4
- default_prompt: "After implementation, inspect the assigned result once, repair reviewer-owned defects to a fixed point, and return evidence for Root's readiness judgment."
4
+ default_prompt: "Open the code review workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -3,13 +3,29 @@ name: commit
3
3
  description: Use when Root owns a local commit after exact scope and proof are settled; verify scope, create the minimal commit, and report identity.
4
4
  ---
5
5
 
6
- Owner: Root. Boundary: perform all Git/VCS inspection and mutation. Verify the
7
- exact diff scope, required local proof, temporary/generated-artifact cleanup,
8
- and ignore coverage before staging. Preserve unrelated work and ensure
9
- environment secrets, credentials, and private release material are absent from
10
- the staged file list. Local commits need no user approval; every exact push,
11
- tag, merge, CI trigger, publication, or other remote mutation requires fresh
12
- user approval through native `request_user_input` immediately beforehand.
6
+ Root uses this workflow after exact scope and local proof are settled. Verify
7
+ the diff, generated-artifact cleanup, ignore coverage, and staged secret
8
+ exclusions before creating the local commit.
9
+
10
+ Require a passing Reviewer.code fixed-point result after implementation or a
11
+ major codebase change before this VCS exception is used.
12
+
13
+ After integration, Root commits the exact approved ref and gives its exact ref
14
+ and SHA to `Worker.operations` for terminal CI observation.
15
+ Discover the repository's actual development/release topology first; never
16
+ assume GitHub, branch names, or that pending is success. Delegate fixes for
17
+ failures, then repeat integration, commit, push, and observation until the
18
+ repository gate is terminal green. If release is authorized, Root performs the
19
+ repository's own release mechanism only after the development gate is terminal
20
+ green, then delegates terminal release verification. If development and release
21
+ share one pipeline, or the repository has no distinct release gate, record that
22
+ topology and apply the repository's one available terminal gate. Fix failures
23
+ through a new bounded Assignment and repeat the same review/VCS/observation
24
+ cycle.
25
+
26
+ Local commits need no user approval; every exact push, tag, merge, CI trigger,
27
+ publication, or other remote mutation requires fresh user approval through
28
+ native `request_user_input` immediately beforehand.
13
29
 
14
30
  Completion: Root reports the commit identity and post-commit status with
15
31
  redacted evidence, or returns an exact reproducible blocker. Never print secret
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Commit"
3
3
  short_description: "Create one local commit with exact scope proof."
4
- default_prompt: "When Root owns a local commit, verify exact scope, required proof, ignore coverage, and secret exclusions, create only that commit, and report redacted identity and status."
4
+ default_prompt: "Open the commit workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: false
@@ -1,10 +1,8 @@
1
1
  ---
2
2
  name: compress
3
- description: Use when Root assigns text that must be shortened; preserve meaning, constraints, technical terms, and voice.
3
+ description: Use when assigned text must be shortened without changing its contract or voice.
4
4
  ---
5
5
 
6
- Owner: Worker. Boundary: remove repetition without changing requirements,
7
- technical terms, product voice, or required evidence.
8
-
9
- Completion: the compressed text is constraint-equivalent to the input and any
10
- unresolvable ambiguity is explicit.
6
+ Shorten only the assigned text. Preserve meaning, constraints, technical terms,
7
+ product voice, and required evidence. The result is complete when it is
8
+ constraint-equivalent and any ambiguity that cannot be preserved is explicit.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Compress"
3
3
  short_description: "Shorten assigned text without changing its contract or voice."
4
- default_prompt: "When Root assigns text compression, produce one compact constraint-equivalent draft and identify preserved technical terms."
4
+ default_prompt: "Open the compression workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,14 +1,8 @@
1
1
  ---
2
2
  name: context7-cli
3
- description: Use first when Root assigns a current library, framework, SDK, or API fact; use a network research route only for releases, dates, gaps, or corroboration.
3
+ description: Use when a current library, framework, SDK, or API fact needs authoritative documentation.
4
4
  ---
5
5
 
6
- Use the current documentation route before relying on memory, because APIs
7
- and tool behavior change and repository evidence cannot establish currency.
8
-
9
- Owner: Librarian for research; Worker may use the assigned result. Boundary:
10
- gather sourced claims, versions, dates, conflicts, and coverage limits without
11
- editing the repository or deciding architecture.
12
-
13
- Completion: every requested current fact has an authoritative locator or an
14
- explicit coverage gap, and no unsupported external claim is used as proof.
6
+ Use current authoritative documentation before relying on memory. Return the
7
+ requested versions, behavior, dates, conflicts, and coverage limits with
8
+ source locators. Do not turn an external fact into an architecture decision.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Context7 CLI"
3
3
  short_description: "Find assigned current library and API documentation first."
4
- default_prompt: "When Root assigns a current library or API fact, use the documentation route first and return sourced evidence with coverage limits."
4
+ default_prompt: "Open the current documentation lookup workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,22 +1,8 @@
1
1
  ---
2
2
  name: debugging
3
- description: Use when Root assigns a reproducible crash, wrong result, regression, hang, race, leak, or slowdown; isolate and prove the narrow repair.
3
+ description: Use for a reproducible crash, wrong result, regression, hang, race, leak, or slowdown.
4
4
  ---
5
5
 
6
- Reproduce the defect before changing code. A stable red case distinguishes
7
- cause from symptom and makes the repair checkable.
8
-
9
- Owner: Worker within the assigned seam. Boundary: capture the smallest
10
- failing input and trace, form an evidence-backed cause, apply the narrow fix,
11
- and preserve unrelated behavior. Escalate competing causes or material
12
- redesign.
13
-
14
- Discover and use the repository-specific formatter, linter, typecheck, and
15
- validation commands before and after the repair. Start with the smallest
16
- relevant local proof and expand only when the defect crosses that boundary or
17
- the release gate requires it; report unavailable commands rather than
18
- inventing replacements.
19
-
20
- Completion: the failure is reproduced, the cause is supported by repository
21
- evidence, the fix and regression proof pass, and remaining uncertainty is
22
- recorded.
6
+ Reproduce the defect before changing code. Capture the smallest failing input
7
+ and trace, identify the evidence-backed cause, make the narrow repair, and prove
8
+ the regression is gone. Escalate competing causes or a material redesign.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Debugging"
3
3
  short_description: "Reproduce and repair one observable defect."
4
- default_prompt: "When Root assigns a reproducible defect, reproduce it, isolate its cause, make the narrow repair, and prove the regression is gone."
4
+ default_prompt: "Open the debugging workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -0,0 +1,21 @@
1
+ ---
2
+ name: grill-me
3
+ description: Use when an unspecified choice has multiple reasonable outcomes that could materially change the work.
4
+ ---
5
+
6
+ # Grill me
7
+
8
+ Use this skill only for a material unresolved choice involving architecture,
9
+ behavior, UX, compatibility, dependencies, cost, security, external effects,
10
+ or another meaningful user-visible result. Infer obvious safe defaults and
11
+ continue without questioning routine omissions.
12
+
13
+ Before asking, finish authorized discovery and reversible preparation that can
14
+ narrow the choice without deciding it. Ask only for the earliest unresolved
15
+ choice required by the next executable phase. After the answer, continue the
16
+ discovery and ask the next dependent question only if it remains material.
17
+ Avoid questionnaires about later phases whose relevance may change.
18
+
19
+ Persist the answer through the current Intent or Plan state model. Return to
20
+ Root with the selected decision, its scope, and the evidence or dependency it
21
+ unblocked; use `needs_root_input` when the choice remains unresolved.
@@ -0,0 +1,6 @@
1
+ interface:
2
+ display_name: "Grill me"
3
+ short_description: "Ask only material, dependency-ordered clarification questions."
4
+ default_prompt: "Open the material-choice clarification workflow for this invocation."
5
+ policy:
6
+ allow_implicit_invocation: true
@@ -1,10 +1,9 @@
1
1
  ---
2
2
  name: handoff
3
- description: Use when Root pauses or transfers work; produce one redacted resumable handoff.
3
+ description: Use when a caller needs a redacted projection of current Intent state for resume or export.
4
4
  ---
5
5
 
6
- Owner: Worker. Boundary: retain scope, owner, permissions, decisions,
7
- evidence, blockers, remaining risk, and the exact next action; omit secrets
8
- and unchanged narration.
9
-
10
- Completion: one bounded, redacted handoff is ready for the receiving owner.
6
+ Produce one compact projection of current Intent, Plan, and Assignment status,
7
+ evidence, blockers, remaining risk, and the exact next action. Omit secrets,
8
+ raw prompts, transcripts, and unchanged narration. Handoff is an export view;
9
+ the semantic state remains authoritative.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Handoff"
3
- short_description: "Produce one redacted handoff when assigned work pauses or transfers."
4
- default_prompt: "When Root pauses or transfers work, write one redacted handoff containing scope, constraints, evidence, risks, and the next action."
3
+ short_description: "Export a redacted projection of current Intent state."
4
+ default_prompt: "Open the handoff workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: false
@@ -0,0 +1,11 @@
1
+ ---
2
+ name: operations
3
+ description: Use for exact-ref CI or release observation through terminal evidence.
4
+ ---
5
+
6
+ Observe the exact approved ref and SHA after Root's VCS action. Discover
7
+ whether the repository has separate development and release gates, one combined
8
+ pipeline, or no formal separation. Report terminal green, terminal failure, or
9
+ an exact unavailable or ambiguous blocker. Pending and running are not success.
10
+ Do not rerun, cancel, approve, merge, push, tag, publish, deploy, or otherwise
11
+ mutate external state.
@@ -0,0 +1,6 @@
1
+ interface:
2
+ display_name: "Operations"
3
+ short_description: "Observe exact-ref CI or release state to terminal evidence."
4
+ default_prompt: "Open the terminal operations observation workflow for this invocation."
5
+ policy:
6
+ allow_implicit_invocation: false
@@ -1,13 +1,10 @@
1
1
  ---
2
2
  name: plan
3
- description: Use when Root faces unresolved architecture, scope, coordination, or material risk before work begins; produce the implementation-ready decision record.
3
+ description: Use when unresolved architecture, scope, coordination, or material risk requires an implementation-ready Plan.
4
4
  ---
5
5
 
6
- Owner: Root. Boundary: identify alternatives, constraints, risks, route,
7
- acceptance evidence, exclusions, and recovery without dispatching
8
- implementation work before the plan gate opens. Specialists may supply facts
9
- but may not decide the outcome.
10
-
11
- Completion: Root has an implementation-ready plan naming scope, owners and
12
- seams, data and control flow, policy, recovery, tests, compatibility, the
13
- preferred shape, and a bounded proof route.
6
+ Resolve the current material choices into an implementation-ready Plan with
7
+ scope, owners, seams, data and control flow, policy, recovery, compatibility,
8
+ acceptance evidence, and a bounded proof route. Keep planning proportional:
9
+ trivial work can proceed without a Plan. Persist the canonical revision through
10
+ the semantic Plan operation.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Plan"
3
3
  short_description: "Resolve material scope and architecture choices before work."
4
- default_prompt: "When Root faces an unresolved material choice, produce an implementation-ready plan covering scope, route, risk, and proof."
4
+ default_prompt: "Open the planning workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,11 +1,9 @@
1
1
  ---
2
2
  name: plan-review
3
- description: Use when Root has a complete plan that needs adversarial review; check feasibility, order, risk, and proof to a fixed point.
3
+ description: Use when a complete Plan needs adversarial feasibility, order, risk, and proof review.
4
4
  ---
5
5
 
6
- Owner: Reviewer. Boundary: inspect the complete plan once to a fixed point;
7
- repair only reviewer-owned plan defects. Return material product or
8
- architecture choices to Root.
9
-
10
- Completion: the plan is complete with proof, blocked with evidence, or
11
- explicitly needs a Root decision.
6
+ Inspect the complete Plan for feasibility, ordering, risk, and proof. Repair
7
+ only reviewer-owned plan defects; return material product or architecture
8
+ choices to Root. A Plan revision replaces the canonical revision through the
9
+ semantic plan operation.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Plan review"
3
3
  short_description: "Review one complete plan to a fixed point."
4
- default_prompt: "When Root has a complete plan, inspect it once, repair reviewer-owned defects to a fixed point, and return structured proof."
4
+ default_prompt: "Open the plan review workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,11 +1,8 @@
1
1
  ---
2
2
  name: programming
3
- description: Use when Root has decided a seam and assigns bounded implementation, fixes, tests, or manifests with known acceptance behavior.
3
+ description: Use when Root has decided a bounded implementation seam with known acceptance behavior.
4
4
  ---
5
5
 
6
- Implement the decided seam inside the assigned files. Inspect the callers and
7
- tests needed to preserve its behavior and typed boundaries, then add focused
8
- proof at the least brittle observable boundary. Discover the repository's own
9
- validation commands and run the smallest relevant check first; broaden when
10
- the changed boundary or release gate requires it. Inspect changed files and
11
- remove incidental generated output before stopping.
6
+ Implement only the decided seam. Inspect its callers and typed boundaries
7
+ before implementation, and add proof at the least brittle observable boundary
8
+ when the acceptance behavior requires it.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Programming"
3
3
  short_description: "Implement one decided seam with proportional proof."
4
- default_prompt: "When Root assigns a decided seam, implement it, add focused proof, inspect changed files, and return structured evidence."
4
+ default_prompt: "Open the bounded programming workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,11 +1,8 @@
1
1
  ---
2
2
  name: refactor
3
- description: Use when Root has decided one seam needs restructuring; preserve behavior, ownership, and interfaces.
3
+ description: Use when one decided seam needs restructuring while preserving behavior and interfaces.
4
4
  ---
5
5
 
6
- Restructure only the decided seam. Trace its callers before changing a shared
7
- root cause, preserve public behavior and interfaces, and keep the smallest
8
- cohesive change. Discover the repository's own validation commands and run the
9
- smallest relevant behavior-locked check first; broaden when the changed
10
- boundary or release gate requires it. Inspect changed files and remove
11
- incidental generated output before stopping.
6
+ Restructure only the decided seam. Trace callers before changing a shared root
7
+ cause and preserve public behavior and interfaces. Verify the behavior boundary
8
+ after the refactor.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Refactor"
3
3
  short_description: "Restructure one decided seam while preserving its contract."
4
- default_prompt: "When Root assigns a seam refactor, preserve behavior and interfaces, inspect the changed files, and return focused proof."
4
+ default_prompt: "Open the bounded refactor workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,10 +1,8 @@
1
1
  ---
2
2
  name: rules
3
- description: Use when Root assigns a repository rule discovery, validation, or application failure; trace the owning rule path.
3
+ description: Use when a repository rule needs discovery, validation, or diagnosis of an application failure.
4
4
  ---
5
5
 
6
- Owner: Worker. Boundary: inspect the owning rule loader and preserve
7
- deduplication, limits, caching, trust boundaries, and rule ownership.
8
-
9
- Completion: the rule path, policy, observed failure, and verification evidence
10
- are explicit, or the exact blocker is returned.
6
+ Trace the owning rule loader and its limits, caching, trust boundary, and
7
+ deduplication. State the observed failure, the owning policy, and the evidence
8
+ that proves the rule path or exact blocker.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Rules"
3
3
  short_description: "Trace one repository rule discovery or application failure."
4
- default_prompt: "When Root assigns a rule failure, inspect the owning rule seam and return exact behavior, limits, and failure evidence."
4
+ default_prompt: "Open the repository rules workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,38 +1,9 @@
1
1
  ---
2
2
  name: stop-slop
3
- description: Use when Root assigns prose to draft, edit, or review; remove predictable AI writing patterns while preserving the requested voice.
3
+ description: Use when assigned prose should lose predictable AI writing patterns while preserving voice.
4
4
  ---
5
5
 
6
- Owner: Worker or Reviewer within the assigned prose surface. Boundary: edit
7
- only the assigned text, preserve meaning and product voice, and use the linked
8
- references for phrase, structure, and example branches when needed.
9
-
10
- # Stop Slop
11
-
12
- Remove predictable AI writing patterns from prose.
13
-
14
- ## Core rules
15
-
16
- 1. Cut filler phrases, emphasis crutches, and adverbs. See
17
- [phrases](references/phrases.md).
18
- 2. Break formulaic structures, binary contrasts, negative listings, dramatic
19
- fragments, rhetorical setups, and false agency. See
20
- [structures](references/structures.md).
21
- 3. Use active voice and name the person doing the work.
22
- 4. Replace vague declarations and lazy extremes with specific facts.
23
- 5. Prefer "you" and concrete details over distant abstractions.
24
- 6. Vary sentence length and paragraph rhythm. Avoid em dashes.
25
- 7. State facts directly without softening or hand-holding.
26
- 8. Rewrite pull-quote language as plain language.
27
-
28
- Before delivery, check for adverbs, passive voice, inanimate actors, vague
29
- claims, throat-clearing, binary contrasts, meta-joiners, em dashes, and repeated
30
- rhythm. Use [examples](references/examples.md) to calibrate edits.
31
-
32
- Completion: the assigned prose is direct, specific, constraint-equivalent,
33
- and reviewed against the relevant references; any deliberate voice tradeoff
34
- is recorded.
35
-
36
- ## License
37
-
38
- MIT
6
+ Edit only the assigned prose. Remove filler, formulaic structure, vague claims,
7
+ passive voice, and repeated rhythm while preserving meaning and voice. Use the
8
+ linked [phrase](references/phrases.md), [structure](references/structures.md),
9
+ and [example](references/examples.md) references when needed.
@@ -1,6 +1,6 @@
1
1
  interface:
2
2
  display_name: "Stop Slop"
3
3
  short_description: "Edit assigned prose to remove predictable AI patterns."
4
- default_prompt: "When Root assigns prose editing, apply the concise rules and disclosed references while preserving meaning and voice."
4
+ default_prompt: "Open the assigned prose editing workflow for this invocation."
5
5
  policy:
6
6
  allow_implicit_invocation: true
@@ -1,28 +1,12 @@
1
1
  # Testing quality
2
2
 
3
- Use this reference when Root, Worker, or Reviewer evaluates a test for a
4
- requested change.
5
-
6
- Owner: Worker or Reviewer within the assigned test surface. Boundary: protect
7
- stable behavior, contracts, invariants, safety properties, compatibility
8
- guarantees, and meaningful failure modes. Prefer observable and typed or
9
- public boundaries while leaving implementation choices free.
10
-
11
- Before adding a regression or package-verification test, identify:
12
-
13
- 1. the stable behavior it protects;
14
- 2. the real regression it catches;
15
- 3. why a less brittle boundary cannot protect it.
16
-
17
- Remove or rewrite tests that freeze incidental counts, inventories, internal
18
- paths, prompt or skill sizes, exact non-contract wording, irrelevant call order,
19
- deleted names, broad repository shape, or one implementation strategy. Avoid
20
- large internal snapshots and static source inspection when a behavior check
21
- exists. Package-verification tests should consume the shipped artifact through
22
- one supported outer boundary with minimal realistic setup, proving a meaningful
23
- artifact behavior without duplicating the suite or crawling inventories.
24
-
25
- Use proportional proof. A test that constrains more design than meaningful regression risk should not exist.
26
-
27
- Completion: each retained test names the stable behavior and regression it
28
- protects, and the chosen boundary is no more brittle than necessary.
3
+ Use this reference when assessing a test for a requested change. Identify the
4
+ stable behavior, contract, invariant, compatibility guarantee, safety property,
5
+ or meaningful failure mode it protects; the regression it catches; and why a
6
+ less brittle boundary cannot protect it.
7
+
8
+ Do not freeze incidental counts, inventories, internal paths, prompt or skill
9
+ wording, irrelevant call order, broad repository shape, or one implementation
10
+ strategy. Prefer observable typed or public boundaries. Package-verification
11
+ tests should consume the shipped artifact through one supported outer boundary
12
+ with minimal realistic setup, proving behavior without duplicating the suite.
@@ -1,19 +1,9 @@
1
1
  # Skill mechanics
2
2
 
3
- This is the skill-specific branch of [writing-for-agents](SKILL.md): frontmatter, invocation choice, and router skills.
3
+ Use a model-invoked skill when its description names a distinct model-facing
4
+ branch. Keep the description trigger-first and short. Keep routing and
5
+ invocation metadata in `agents/openai.yaml`; the body owns the behavior.
4
6
 
5
- ## Invocation
6
-
7
- A **model-invoked** skill keeps a trigger-first `description` so an agent can select it and another skill can reach its shared reference. The always-loaded pointer spends context on every turn, so use one compact trigger per real branch and remove routing prose from the body.
8
-
9
- A **user-invoked** skill sets `disable-model-invocation: true`. Its description becomes a human-facing one-line summary with trigger lists removed. Use this when only a person should select the skill.
10
-
11
- Shared reference required by user-invoked skills belongs in a plain disclosed file that each can point to.
12
-
13
- ## Splitting by invocation
14
-
15
- Split a model-invoked skill only when a distinct leading word should trigger it independently or another skill must reach it. The independent route must justify its permanent context pointer.
16
-
17
- ## Router skills
18
-
19
- When many user-invoked skills exceed human cognitive load, add one user-invoked router that names them and their branches. A router can guide the person; it cannot autonomously invoke skills whose model invocation is disabled.
7
+ Put uncommon mechanics in a linked reference only when a caller needs them.
8
+ Keep each instruction in one source of truth and avoid copying body text into
9
+ metadata or neighboring skills.
@@ -1,46 +1,35 @@
1
1
  ---
2
2
  name: writing-for-agents
3
- description: Use when Root prepares or reviews a subagent delegation, handoff, profile, skill, prompt, or agent-facing instruction; write a low-context independent contract. Before each dispatch ensure this skill is loaded and applied; reuse it until compaction, a new context, or an incomplete/unavailable load.
3
+ description: Use when GPT-6 Astra Root writes or reviews a GPT-5.6 Luna specialist contract.
4
4
  ---
5
5
 
6
6
  # Writing for agents
7
7
 
8
- Treat each instruction as an interface with one contract:
9
-
10
- - **Owner:** the role permitted to act or decide.
11
- - **Boundary:** the exact scope, authority, constraints, and escalation point.
12
- - **Completion:** the observable outcome and proof that permit the agent to stop.
13
- - **Return:** the result Root needs to integrate: outcome or findings, changed
14
- paths, verification, unresolved risk, and any requested decision.
15
-
16
- Write an objective, scope, retained constraints, evidence, exclusions, and the
17
- exact missing fact or material choice that returns to the owner. The receiver
18
- must be able to act, prove, and stop without reconstructing intent.
19
-
20
- Before any Root subagent dispatch, ensure this skill is fully loaded and applied
21
- in the current context. Reuse the loaded instructions for later dispatches
22
- while they remain complete; reload after compaction, a new context, or an
23
- incomplete/unavailable load.
24
-
25
- ## Information design
26
-
27
- A **context pointer** names out-of-context material and front-loads the branches that load it. Keep one trigger per genuine branch; synonyms spend permanent context without improving routing.
28
-
29
- Budget two loads:
30
-
31
- - **Context load:** always-loaded descriptions and repository rules.
32
- - **Cognitive load:** material a person must remember and select.
33
-
34
- Place information by immediacy: in-file steps, in-file reference, then disclosed reference. Inline what every invocation needs. Put branch-specific mechanics behind a clear pointer. Co-locate each concept with its rules and caveats; split real invocation or sequence branches when one file sprawls or later steps cause premature completion.
35
-
36
- Each step needs a clear, demanding completion criterion. Sharpen a vague bound before splitting the sequence. Use leading words with useful pretrained meaning to compress repeated concepts. Prefer positive steering; keep prohibitions for hard guardrails and pair them with the target behavior.
37
-
38
- ## Pruning
39
-
40
- Keep each meaning in one source of truth. Treat code, configuration, directory layout, and `--help` as authoritative environment lookups; document only reasons, gotchas, or contracts the environment cannot reveal. Delete stale caches, irrelevant branches, sediment, and instructions that do not change model behavior.
41
-
42
- Use architectural role names: Root owns material decisions, Workers implement bounded seams, Reviewers repair their assigned findings to fixed point, and other specialists follow their literal authority. Continuations carry changed constraints, evidence, decisions, and semantic delta instead of replaying accepted context.
43
-
44
- When the document is a skill, read [SKILL-MECHANICS.md](SKILL-MECHANICS.md) for invocation and frontmatter mechanics.
45
-
46
- Completion means the instruction routes at the right branch, names authority and exclusions, prevents premature completion, supplies checkable proof, and contains no duplicated or no-op policy.
8
+ This skill helps GPT-6 Astra Root author instructions for GPT-5.6 Luna. First
9
+ identify Luna's effective context: the generated Luna baseline, repository
10
+ `AGENTS.md`, the selected native Role.task, loaded skills, and relevant config
11
+ or metadata. Do not restate a directive already visible to that receiver.
12
+
13
+ Give each Assignment only the delta Luna needs:
14
+
15
+ - objective and exact scope;
16
+ - relevant context and authority;
17
+ - unique hard constraints and exclusions;
18
+ - acceptance criteria and required evidence;
19
+ - the material choice or blocker that must escalate;
20
+ - the return payload and stopping condition.
21
+
22
+ Place new policy at the narrowest owner: Root/session behavior in Root
23
+ instructions, repository conventions in `AGENTS.md`, Luna-global constraints in
24
+ the generated baseline, task-specific behavior in Role.task, and conditional
25
+ workflow in a skill or a disclosed reference. Keep invocation metadata as
26
+ routing metadata.
27
+
28
+ State completion before optional procedure. Use positive, literal boundaries,
29
+ and disclose uncommon branches progressively. Let Astra infer routine safe
30
+ defaults and use its judgment to decide what Luna actually needs; do not add
31
+ competence scaffolding, generic recipes, or a broad questionnaire.
32
+
33
+ When a contract is complete, it is independently actionable for Luna, contains
34
+ no same-receiver semantic duplicate, and identifies the evidence that lets Root
35
+ integrate the result.