Skip to content

Mirror Crucible council delta and termination policy - #773

Merged
mergify[bot] merged 2 commits into
mainfrom
codex/symph-1145-council-policy
Jul 27, 2026
Merged

Mirror Crucible council delta and termination policy#773
mergify[bot] merged 2 commits into
mainfrom
codex/symph-1145-council-policy

Conversation

@ericlitman

@ericlitman ericlitman commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • make round 2+ council prompts review a materialized previousReviewedHead..HEAD patch while retaining the cumulative diff.patch as a path reference
  • preserve full-diff fallbacks for round 1, explicit full review, base movement, rewritten/conflicted history, and unavailable delta materialization
  • thread live journal round/base/head history and prior structured findings through the crabrunner review dispatcher
  • delegate N=2 clean-window, K=3 repeat-fingerprint, and round-15 backstop decisions to the shared Crucible convergence-decision spine while retaining Symphony warning telemetry
  • replace the legacy round-cap telemetry and tests with N/K/backstop semantics

Coalescence decision

Symphony calls the existing Crucible spine through CrabboxSpineClient for convergence authority. Directly importing the Crucible .mjs prompt-scope helper is not deploy-safe because Symphony runs as a standalone dist/ service on pro14 and its typed review-context boundary differs; the local adapter is limited to payload materialization and fallback selection.

Verification

  • pnpm install completed first in the fresh worktree
  • pnpm test — 4,587 passed, 5 skipped
  • pnpm build
  • pnpm typecheck
  • pnpm lint
  • git diff --check

Closes SYMPH-1145


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@ericlitman

Copy link
Copy Markdown
Owner Author

@Mergifyio refresh

@mergify

mergify Bot commented Jul 27, 2026

Copy link
Copy Markdown

refresh

✅ Pull request refreshed

@mergify

mergify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Merge Queue Status

  • Entered queue2026-07-27 03:08 UTC · Rule: default · triggered by merge protections
  • Checks passed · in-place
  • Merged2026-07-27 03:23 UTC · at 2a3cab19fa11980909858c0caa30cb667ee68e8d

This pull request spent 15 minutes 32 seconds in the queue, including 3 minutes 15 seconds running CI.

Required conditions to merge
  • github-review-approved [🛡 GitHub repository ruleset rule Mergify automatic merge queue] (documentation)
  • all of [🛡 Merge Protections rule Queue ready pull requests]:
  • any of [🛡 GitHub branch protection]:
    • check-success = test
    • check-neutral = test
    • check-skipped = test
  • any of [🛡 GitHub branch protection]:
    • check-success = diff-coverage
    • check-neutral = diff-coverage
    • check-skipped = diff-coverage
  • any of [🛡 GitHub repository ruleset rule Mergify automatic merge queue]:
    • check-success = @mergify/Mergify Merge Protections
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections

@mergify mergify Bot added the queued label Jul 27, 2026
@mergify
mergify Bot merged commit 04bcd71 into main Jul 27, 2026
5 of 6 checks passed
@mergify
mergify Bot deleted the codex/symph-1145-council-policy branch July 27, 2026 03:23
@mergify mergify Bot removed the queued label Jul 27, 2026
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