holycodex 0.16.2-dev.64.1 → 0.16.3-dev.66.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.
- package/dist/assets/plugin/plugin.json +1 -1
- package/dist/assets/plugin/skills/code-review/SKILL.md +6 -13
- package/dist/assets/plugin/skills/programming/SKILL.md +6 -30
- package/dist/assets/plugin/skills/refactor/SKILL.md +6 -18
- package/dist/assets/plugin/skills/writing-for-agents/SKILL.md +5 -1
- package/dist/index.js +29 -31
- package/package.json +2 -2
|
@@ -7,17 +7,10 @@ Use this skill once after implementation or manifest work. Inspect the actual
|
|
|
7
7
|
integrated result to a fixed point before Root judges readiness. Apply
|
|
8
8
|
[testing quality](../testing-quality.md) when judging new or modified tests.
|
|
9
9
|
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
retains architecture, product, scope, trust, interface, external-state, and
|
|
14
|
-
final readiness decisions.
|
|
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.
|
|
15
13
|
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
unmergeable diffs, unclear dependency direction, and unnecessary tests.
|
|
20
|
-
Check that temporary/generated output is absent and Git/VCS remains Root-only.
|
|
21
|
-
|
|
22
|
-
Completion: no reviewer-owned defect remains, verification evidence is
|
|
23
|
-
recorded, and every residual risk or Root decision is explicit.
|
|
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.
|
|
@@ -3,33 +3,9 @@ name: programming
|
|
|
3
3
|
description: Use when Root has decided a seam and assigns bounded implementation, fixes, tests, or manifests with known acceptance behavior.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
Implement
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
boundary when it has real callers or clear near-term value; do not add
|
|
13
|
-
speculative abstractions or split a cohesive file without a concrete benefit.
|
|
14
|
-
Keep the smallest correct, independently mergeable diff with deliberate error
|
|
15
|
-
handling and sensible dependency direction. Git/VCS work remains Root-only.
|
|
16
|
-
|
|
17
|
-
Before checking a change, discover the repository's own formatter, linter,
|
|
18
|
-
typecheck, and validation commands from its package scripts, task config, and
|
|
19
|
-
contributor guidance. Run the smallest relevant local command first, then
|
|
20
|
-
broaden only when the changed boundary or release gate requires it; report
|
|
21
|
-
which commands were unavailable instead of inventing replacements.
|
|
22
|
-
|
|
23
|
-
When implementation changes tests, apply [testing quality](../testing-quality.md)
|
|
24
|
-
before adding or retaining them: protect a meaningful stable behavior at the
|
|
25
|
-
least brittle observable boundary, and avoid tests for implementation detail.
|
|
26
|
-
Remove temporary or incidental generated output before handoff. Escalate
|
|
27
|
-
material choices.
|
|
28
|
-
|
|
29
|
-
Owner: Worker. Boundary: preserve repository-native ownership, dependency
|
|
30
|
-
direction, trust checks, and acceptance behavior; do not broaden scope or
|
|
31
|
-
delegate.
|
|
32
|
-
|
|
33
|
-
Completion: the requested behavior and focused proof are present, changed
|
|
34
|
-
files are inspected, diagnostics and proportional checks pass, and structured
|
|
35
|
-
evidence names residual risk or a precise blocker.
|
|
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.
|
|
@@ -3,21 +3,9 @@ name: refactor
|
|
|
3
3
|
description: Use when Root has decided one seam needs restructuring; preserve behavior, ownership, and interfaces.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
Discover the repository-specific formatter, linter, typecheck, and validation
|
|
14
|
-
commands from its package scripts, task config, and contributor guidance. Run
|
|
15
|
-
the smallest relevant local proof first and broaden only when the changed
|
|
16
|
-
boundary or release gate requires it; do not invent unavailable commands.
|
|
17
|
-
|
|
18
|
-
Owner: Worker. Boundary: change only the assigned seam; preserve ownership,
|
|
19
|
-
dependency direction, public behavior, and interfaces.
|
|
20
|
-
|
|
21
|
-
Completion: changed files are inspected, focused behavior-locked checks are
|
|
22
|
-
clean, temporary/generated output is removed, and any residual risk or blocker
|
|
23
|
-
is explicit.
|
|
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.
|
|
@@ -10,8 +10,12 @@ Treat each instruction as an interface with one contract:
|
|
|
10
10
|
- **Owner:** the role permitted to act or decide.
|
|
11
11
|
- **Boundary:** the exact scope, authority, constraints, and escalation point.
|
|
12
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.
|
|
13
15
|
|
|
14
|
-
Write an objective, scope, retained constraints, evidence, exclusions, and the
|
|
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.
|
|
15
19
|
|
|
16
20
|
Before any Root subagent dispatch, ensure this skill is fully loaded and applied
|
|
17
21
|
in the current context. Reuse the loaded instructions for later dispatches
|