mirror of
https://github.com/EveryInc/compound-engineering-plugin.git
synced 2026-09-19 01:09:55 +08:00
Trevin Chow
7f6cbd4393
The Codex delegation prompt template silently dropped each plan unit's Execution note. The standard inline subagent path treats it as a first-class signal (ce-work-beta/SKILL.md:104, 108, 172, 208, 270), but plans using test-first or characterization-first lost that signal whenever the executor was Codex via ce-work-beta delegate:codex. Add an <execution_note> tag between <approach> and <constraints>, mirroring the standard path. Population covers single- and multi-unit batches, prefixes each entry with U<ID>: when present and Unit <n>: otherwise (no invented IDs), and merges session-level posture requests into units lacking their own note. test-first reproduces the standard guardrails; characterization-first captures existing behavior first; any other non-empty note is binding but subordinate to constraints/testing/ verify/output_contract. A regression assertion in tests/pipeline-review-contract.test.ts locks <execution_note> presence and both posture strings so the field cannot recur silently. Reapplies PR #820 onto the post-#967 root-native skill paths; that PR was anchored to the old plugins/compound-engineering/skills/ tree. Co-authored-by: David A. Lee <davidalee@users.noreply.github.com>