docs(skills): harness skill pack for Claude Code / Pi - #11
Merged
Conversation
Add verb-first skills under skills/tracegrad-harness/ so a harness can drive import → estimate → propose → review → export without bloating core. Unattended apply stays off unless a policy file permits it. Adapters remain sidecar examples, not package code. Co-authored-by: Dickson Neoh <dickson.neoh@gmail.com>
Forbid bare interactive apply on the harness path; require --accept with human- or policy-named indices. Distinguish unattended policy apply from attended human accepts. Drop run from import-traces Exact CLI and omit inventable apply indices from next-batch Exact CLI. Co-authored-by: Dickson Neoh <dickson.neoh@gmail.com>
Drop inventable apply indices from export-prompt Exact CLI. Correct sidecar FIELD_MAP comment to Tracegrad field → foreign path. Forbid --all on the harness path; describe policy accept as a TOML integer array. Co-authored-by: Dickson Neoh <dickson.neoh@gmail.com>
Front-load trigger words, ordered steps with checkable done-conditions, and a single apply-gate in review-edits. Disclose the JSONL contract beside import-traces. Captain locks unchanged: --accept only, unattended apply off by default, no Kitaru, adapters stay sidecars. Co-authored-by: Dickson Neoh <dickson.neoh@gmail.com>
Flip import-traces gloss so map examples match sidecar keys (trace_id → vendor id path), not the inverted foreign→Tracegrad order. Co-authored-by: Dickson Neoh <dickson.neoh@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/tracegrad-harness/with five verb-first skills: import-traces, propose-edits, review-edits, export-prompt, next-batch[y/N]or uses--all; only--acceptwith human- or policy-named indicesCaptain locks
tracegrad runstill does not write the promptTest plan
Do not merge until captain word.