codex-orchestrator 2.0.17 → 2.0.19

package/internal-workflow/skills/to-tickets/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: to-tickets
-description: Compile Plan's contextual PRD draft or approved product authority into the smallest executable ticket packet, run fresh semantic review per settled revision, obtain one approval, publish serially, verify tracker read-back, and stop before implementation.
+description: Compile Plan's contextual PRD draft or approved product authority into the smallest executable ticket packet, run complete semantic review with one fresh final review after any consolidated repair, obtain one approval, publish serially, verify tracker read-back, and stop before implementation.
 ---
 
 # To Tickets
@@ -15,8 +15,9 @@ ticket contains only its local observable outcome, scope, blockers, proof, and
 Parent link. The approved ticket body plus Parent PRD is the final execution
 authority; no later planning pass is inserted.
 
-`$to-tickets` owns one fresh reviewer per settled revision, one explicit user
-approval, serialized publication, reconciliation, and tracker read-back.
+`$to-tickets` owns complete whole-packet `$plan-review`, one fresh final review
+after any consolidated repair, one explicit user approval, serialized
+publication, reconciliation, and tracker read-back.
 
 The primary route remains `planning-only` while compiling or publishing these
 artifacts. `ticket-graph` begins only after separate delivery authorization;
@@ -45,10 +46,6 @@ outcome and blocking relationship. Risk strengthens proof and review focus; it
 does not create tickets. Merge steps that share one owner, release, and
 validation path.
 
-Do not publish a child whose outcome is only transient evidence or a tracker
-comment. Resolve that discovery before compiling the executable graph, or make
-the required durable result a committed artifact.
-
 Where repository evidence proves a small prefactor is required, "Make the
 change easy, then make the easy change." Keep it inside the same vertical
 outcome unless it is independently verifiable and genuinely blocks later work.
@@ -144,12 +141,12 @@ an explicit blocking discovery outcome. The expected review verdict must be
 
 ### 4. Run one fresh semantic packet review
 
-For each settled revision composed by `$to-tickets` before approval or
-publication, launch exactly one fresh `standards_reviewer` without history
+After the complete packet satisfies the freeze conditions, invoke
+`$plan-review` with exactly one fresh `plan_reviewer` without history
 (`fork_context=false` on V1; `fork_turns="none"` on V2). Begin with
-`Assigned role: standards_reviewer`, provide the full authority and packet, and
-apply all three lenses in the same activation: source fidelity, ticket
-executability, and minimum solution and scope conservation.
+`Assigned role: plan_reviewer`, provide the full authority and packet, and apply
+all three lenses in the same activation: source fidelity, ticket executability,
+and minimum solution and scope conservation.
 
 - **Source fidelity:** every product claim, decision, scope boundary,
   consequence, and blocker is grounded in the source; no behavior or ownership
@@ -185,19 +182,28 @@ Before verdict, traverse each applicable seam:
 Return concise evidence for applicable seams and omit `N/A` seams. This is
 review output, not another packet artifact.
 
-The reviewer is read-only and returns `APPROVE` or `NEEDS_WORK` with exact
-source/artifact evidence. Capture a non-empty fresh child identity and wait for
-that same child. Do not split the lenses across children and do not launch the
-delivery Standards review. After `NEEDS_WORK`, repair the packet and launch a
-new reviewer without prior review history over the complete revised packet. A
-same-context or findings-only recheck cannot approve. Repeat until the latest
-revision receives one full approval. Missing or failed review blocks
-publication; product-changing repairs return to the user.
-
-Before re-review, repair every verified blocker and sweep the complete packet
-for the same defect class. If a blocker exposes unfinished inventory,
+The reviewer is read-only and returns `APPROVE` or `NEEDS_WORK` with every
+material blocker found after completing all applicable lenses and seam
+traversals, plus exact source/artifact evidence. It must complete the
+whole-packet review and must not stop after representative findings. Capture a
+non-empty fresh child identity and wait for that same child. Do not split the
+lenses across children and do not launch the delivery Standards review.
+
+When review returns a plan-changing `QUESTION`, return it through Plan to the
+user and stop without revising or publishing the packet. Resume only after the
+answer becomes authority and the complete packet is settled again.
+
+After `NEEDS_WORK` with verified blockers, repair all blockers together and
+sweep the complete packet for every defect class they expose. Do not review
+intermediate repair edits. If a blocker exposes unfinished inventory,
 authority, or composition, return to drafting and pass the freeze conditions
-again.
+again. Once the complete revised packet satisfies the freeze conditions,
+launch one fresh final whole-packet `plan_reviewer` without prior review
+history. Use one fresh reviewer per settled revision; a same-context recheck
+cannot approve. Further review is justified only by a remaining material
+blocker; optional hardening, broader proof, stylistic refinement, or
+requirements without authority do not reopen the review cycle. Missing or
+failed review blocks publication; product-changing repairs return to the user.
 
 ### 5. Get one explicit approval
 

package/internal-workflow/skills/to-tickets/agents/openai.yaml

@@ -1,6 +1,6 @@
 interface:
   display_name: "To Tickets"
   short_description: "Review and publish executable ticket packets"
-  default_prompt: "Use $to-tickets to compile and fully review each settled packet revision, obtain one approval, publish serially, verify tracker read-back, and stop before implementation."
+  default_prompt: "Use $to-tickets to compile a complete packet, review it fully, consolidate any repairs before one fresh final review, obtain one approval, publish serially, verify tracker read-back, and stop before implementation."
 policy:
   allow_implicit_invocation: false

package/internal-workflow/skills/to-tickets/evals/evals.json

@@ -15,10 +15,10 @@
       "forbidden": ["create duplicate durable planning state", "request another execution artifact"]
     },
     {
-      "id": "complete-packet-gets-one-review-per-revision",
+      "id": "complete-packet-review-after-freeze",
       "prompt": "Prepare a publish-ready PRD and three-ticket graph after Plan passed the combined PRD draft to you.",
-      "expected": ["launch one fresh semantic reviewer over each settled complete revision", "apply source fidelity and ticket executability in that review", "obtain one explicit user approval before publication"],
-      "forbidden": ["review the intermediate PRD", "launch separate workflow owners for each lens", "request multiple approvals"]
+      "expected": ["launch one fresh semantic reviewer only after the complete packet satisfies the freeze conditions", "apply source fidelity and ticket executability in that review", "obtain one explicit user approval before publication"],
+      "forbidden": ["review the intermediate PRD", "review intermediate repair edits", "launch separate workflow owners for each lens", "request multiple approvals"]
     },
     {
       "id": "serialized-publication-recovers-response-loss",
@@ -102,8 +102,8 @@
     {
       "id": "repair-gets-fresh-full-packet-review",
       "prompt": "Semantic review returns NEEDS_WORK and the packet is repaired before user approval.",
-      "expected": ["repair every verified blocker and sweep the complete packet for the same defect class", "pass the settled packet conditions again", "launch one new reviewer without prior review history", "review the complete revised packet", "keep one user approval for the final packet"],
-      "forbidden": ["continue with the same reviewer", "approve from a findings-only recheck", "launch multiple reviewers for one revision"]
+      "expected": ["repair all verified blockers together and sweep the complete packet for every defect class they expose", "pass the freeze conditions again", "launch one fresh final reviewer without prior review history", "review the complete revised packet", "keep one user approval for the final packet"],
+      "forbidden": ["review intermediate repair edits", "continue with the same reviewer", "approve from a findings-only recheck", "reopen review for optional hardening or requirements without authority"]
     },
     {
       "id": "private-design-stays-with-implement",
@@ -128,6 +128,31 @@
       "prompt": "UI acceptance requires dynamic content absent from the proposed API or DTO.",
       "expected": ["reject the packet until an existing or authorized data contract supplies the content"],
       "forbidden": ["let the UI invent the data", "add a new owner solely for proof"]
+    },
+    {
+      "id": "final-packet-uses-plan-review-once",
+      "prompt": "Compile an approved PRD into a complete publish-ready Parent and multi-ticket packet.",
+      "expected": [
+        "do not review the intermediate PRD",
+        "invoke one fresh plan-review over the complete frozen packet",
+        "cover source fidelity, ticket executability, and minimum solution in that review"
+      ],
+      "forbidden": [
+        "invoke standards_reviewer for planning review",
+        "review both the PRD and final packet"
+      ]
+    },
+    {
+      "id": "plan-review-question-returns-to-user",
+      "prompt": "The final packet review raises an unresolved ownership question but finds no verified blocker.",
+      "expected": [
+        "return the question through Plan to the user",
+        "resume only after the answer becomes authority and the packet is settled again"
+      ],
+      "forbidden": [
+        "guess the answer",
+        "repair or publish the packet"
+      ]
     }
   ]
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-orchestrator",
-  "version": "2.0.17",
+  "version": "2.0.19",
   "description": "Reusable GitHub Issues runner for Codex.",
   "type": "module",
   "license": "MIT",