mirror of
https://github.com/EveryInc/compound-engineering-plugin.git
synced 2026-10-20 11:18:31 +02:00
Trevin ChowandClaude Opus 4.7
8d1c7a233f
The repo-root AGENTS.md "Scratch Space" rule (in effect since
PR #588 / commit 12aaad31, 2026-04-17) prescribes /tmp as the default
for cross-invocation reusable scratch:
Cross-invocation reusable: stable path
/tmp/compound-engineering/<skill-name>/<run-id>/ -- not mktemp -d --
so later invocations of the same skill can discover sibling run-ids.
Use /tmp directly rather than $TMPDIR so paths stay accessible.
ce-code-review's per-run artifacts match that profile exactly:
per-run-id directory, surfaced in the completion report so the user
can grep across runs, not user-curated state. The .context/ usage
predated the rule shift and was the only remaining drift in the plugin
tree besides ce-optimize. ce-optimize stays under .context/ -- it's
the literal example AGENTS.md cites for the user-curated exception
(per-spec optimization log surviving across sessions on the same
checkout).
Touches every reference to the per-run artifact path:
- ce-code-review SKILL.md (Stage 4 mkdir, dispatch, Stage 5/6
enrichment, Step 4 artifact emit, mode-specific guidance)
- ce-code-review references: subagent-template.md, tracker-defer.md,
walkthrough.md
- Cross-skill copies of tracker-defer.md (ce-work, ce-work-beta, lfg)
that read ce-code-review's artifacts when filing tickets
- Generic phrasing tightened in SKILL.md report-only mode and Stage 4
read-only contract (was ".context artifacts" / ".context/ artifact
path", now "run artifacts" / "run-artifact path under /tmp/...").
- tests/review-skill-contract.test.ts updated for the new path string
and the new generic phrasing.
Tests: 908/908 pass. release:validate clean.
Surfaced as a follow-up during interactive eval of PR #685.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>