Skip to content

review(OPE-7): n-queens adversarial review — APPROVE WITH RESIDUALS - #7

Open
agentforce314 wants to merge 1 commit into
mainfrom
ope/7-nqueens-review
Open

agentforce314 wants to merge 1 commit into
mainfrom
ope/7-nqueens-review

Conversation

@agentforce314

Copy link
Copy Markdown
Owner

Summary

Adversarial Reviewer deliverable for OPE-7 (parent OPE-2). Reviews origin/ope/6-nqueens-proof @ 8ae158a (ProofLab/NQueensTheorem.lean, PR #6). Do not merge — board merges.

Verdict: APPROVE WITH RESIDUALS. No claim (known-classical; deliverable is the machine-checked artifact).

  • Fresh rebuild (module oleans deleted first): ✔ [2240/2240] Built ProofLab.NQueensTheorem, exit 0, Lean 4.10.0.
  • Own #print axioms: n_queens_exists_iff[propext, Classical.choice, Quot.sound]; queens_exists_of_four_le[propext, Quot.sound]. No sorryAx, no ofReduceBool.
  • NonAttacking read cold: rows/cols Fin n, injective columns, both diagonal families over ℕ (not mod n). Negative controls for each clause rejected; weakened predicates accept them (each clause load-bearing). n = 0 convention consistent across STATEMENT.md / header / docstring.
  • Deviation (a) confirmed: LOG's typed ℕ f₂ truncates on one row of every n ≡ 2 (mod 6) board (166/166, Python); Lean form agrees with the signed form everywhere. Erratum for LOG.md — to fix at close-out.
  • Deviation (b) confirmed: e2_no_fix needs 4 ≤ n (n = 2 counterexample); assembly supplies it. Full hypothesis diff table; nothing else changed.
  • Nothing rests on verify_construction.py. Scope (b)–(e) marked out of scope; no novelty language in PRs OPE-3: n-queens fresh Scout dossier + Level B statement pin (board-named) #3formalist(OPE-6): n-queens namesake theorem, zero sorry (supersedes #4) #6.
  • 8 residual risks incl. missing IsNQueensNonAttacking bridge, toolchain trust, LOG erratum.

Files

  • attacks/n-queens-20260912-180329/ADVERSARIAL_REVIEW.md — the verdict
  • review_checks.lean / review_checks.out — scratch checks, lake env lean exit 0
  • review_check_f2.py — deviation (a) check

Verify

git checkout 8ae158a   # artifact
export PATH="$HOME/.elan/bin:$PATH"; cd proofs/lean-project
lake env lean ../../attacks/n-queens-20260912-180329/review_checks.lean   # from this branch's file; exit 0

Paperclip: OPE-7.

🤖 Generated with Claude Code

…LOG.md f₂ erratum

- ADVERSARIAL_REVIEW.md: fresh rebuild (Built, not Replayed), own #print axioms,
  keyword grep, cold read of NonAttacking, negative controls per clause,
  deviations (a)/(b) confirmed, hypothesis diff table, scope/claim posture,
  machine-trust boundary, 8 residual risks. Verdict: APPROVE WITH RESIDUALS.
- review_checks.lean + .out: scratch file run with lake env lean (exit 0).
- review_check_f2.py: LOG typed ℕ-form vs signed vs Lean form, 8<=n<=1000.

No claim. Artifact reviewed: origin/ope/6-nqueens-proof @ 8ae158a.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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