Skip to content

feat: integrate sweep-v2 plan review adapter - #7

Closed
robgfl45 wants to merge 11 commits into
promptadvisers:mainfrom
robgfl45:feat/frozen-sweep-integration-drake
Closed

feat: integrate sweep-v2 plan review adapter#7
robgfl45 wants to merge 11 commits into
promptadvisers:mainfrom
robgfl45:feat/frozen-sweep-integration-drake

Conversation

@robgfl45

Copy link
Copy Markdown

Summary

  • default the headless plan-review adapter to the feature-flagged sweep-v2 engine while retaining explicit --engine legacy compatibility
  • classify sweep results only from terminal state, immutable snapshot/manifest hashes, exact same-snapshot five-persona sidecars/findings, aggregate evidence hashes, and consolidated findings
  • expose generation/hash/persona coverage and copied state/generation evidence paths for cleanup-safe read-back
  • update the staged Hermes project-plan-review skill/runbook for five generations, the 180-second outer timeout reserve, subscription telemetry wording, and mandatory re-review after material Drake normalization
  • add deterministic false-clean regressions for clean convergence, generation-five material max-reached, missing/malformed/mutated/hash-mismatched/degraded/cancelled evidence, timeout reaping, and legacy behavior

Validation

  • python3 -m unittest -v tests/test_adapter.py — 14 passed
  • bash plugins/claudex/tests/sweep-v2-test.sh — 60 passed
  • bash plugins/claudex/tests/platform-validation.sh — 61 passed
  • bash plugins/claudex/tests/smoke-test.sh — 74 passed
  • bash plugins/claudex/tests/synthetic-e2e.sh — 23 passed (supplementary legacy E2E; one real Codex subscription-backed review call, no live sweep)
  • python3 -m py_compile bin/claudex-plan-review tests/test_adapter.py — passed
  • bash -n plugins/claudex/scripts/*.sh plugins/claudex/hooks/*.sh plugins/claudex/tests/*.sh — passed
  • git diff --check / git diff dae1202aaca2ad349d2ec70221d23e59a822142b...HEAD --check — passed
  • gitleaks protect --staged — no leaks found

Scope / rollout

@robgfl45

Copy link
Copy Markdown
Author

Opened against the upstream repository by mistake. Superseded by robgfl45#5.

@robgfl45 robgfl45 closed this Jul 11, 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