Skip to content

Make review pipeline stages explicit#591

Merged
w1ne merged 2 commits into
developfrom
codex/review-pipeline-stages
Jul 8, 2026
Merged

Make review pipeline stages explicit#591
w1ne merged 2 commits into
developfrom
codex/review-pipeline-stages

Conversation

@w1ne

@w1ne w1ne commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • export an explicit ordered review pipeline stage list
  • split runReviewPipeline into named internal stage helpers
  • add a unit guard for deterministic stage order
  • stabilize the existing carousel DFM timing assertion on GitHub-hosted CI while preserving the strict calibrated budget locally

Verification

  • npm test -- tests/unit/review/reviewPipelineStages.test.ts
  • npm run typecheck
  • npm test -- tests/unit/review/reviewPipelineStages.test.ts tests/unit/mcp/reviewCadAdapter.test.ts tests/unit/mcp/reviewCadEnvelopeOptions.test.ts tests/unit/mcp/reviewCadRepairContext.test.ts tests/integration/mcp/reviewCad.test.ts tests/integration/mcp/designLoop.test.ts tests/integration/mcp/designLoop-interference.test.ts
  • npm test -- tests/integration/dfm/carouselTruthSet.test.ts
  • CI=1 npm test -- tests/integration/dfm/carouselTruthSet.test.ts

Review

  • independent code-review subagent for review-pipeline refactor: no findings

w1ne added 2 commits July 8, 2026 19:57
Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
@w1ne w1ne merged commit c3f14c0 into develop Jul 8, 2026
13 checks passed
@w1ne w1ne deleted the codex/review-pipeline-stages branch July 8, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant