Skip to content

docs: make PROOF-PROGRESS true, amend ADR-0002 OPAQUE triggers, record D4 - #44

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/plan-truth-adr0002-d4
Sep 30, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
docs/plan-truth-adr0002-d4

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Docs-only. Carries out three owner rulings (selector R8/R9).

PROOF-PROGRESS.adoc: checked against PLAN and corrected

The old file said "implementation not started". It also used an M0–M4 scheme that PLAN.adoc never had. It is now rewritten against main at 7ec7e08:

ADR-0002 amendment

  • Trigger 1, except* (PEP 654). Measured against the pinned tree-sitter-python 0.25.0, except* parses as an ordinary except_clause with an anonymous * child. There is no except_group_clause and has_error() is false, so a structural detector is required.
  • Trigger 2, a parse error in the unit.
  • Binding path correction. The except E as e binding lives at value: (as_pattern … alias: (as_pattern_target …)), not at except_clause.alias.

OWNER-DECISIONS: D4 ratified

The RSR Rust template is adopted now. The adoption lands as its own PR, with each workflow shown to spawn jobs under sha_pinning_required.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WRvDivYwLSeVCJUrfjic3f

…d D4

- PROOF-PROGRESS.adoc: rewritten against the code at 7ec7e08. The old
  version said "implementation not started" and invented an M0-M4
  milestone scheme that PLAN.adoc never had. It now maps to PLAN's M0-M8:
  M0-M3 landed, M4 in progress, M5-M8 not started. Also records the
  finding.rs invariants (verified) and the known gaps, including #43.
- ADR-0002 amendment (owner-ratified, selector R8): adds two OPAQUE
  triggers. `except*` parses as an ordinary except_clause with an
  anonymous `*` child, and has_error() is false, so it needs a
  structural detector. The second is a parse error in the unit. Also
  corrects the `except E as e` binding path: it is value: as_pattern ->
  alias: as_pattern_target, not except_clause.alias.
- OWNER-DECISIONS: D4 ratified. Adopt the RSR Rust template now; the
  adoption lands as its own PR.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WRvDivYwLSeVCJUrfjic3f
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 32 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 5d8b987d-3d0b-4254-a885-dc991d4e785e

📥 Commits

Reviewing files that changed from the base of the PR and between 7ec7e08 and 2709497.

📒 Files selected for processing (3)
  • PROOF-PROGRESS.adoc
  • docs/OWNER-DECISIONS.adoc
  • docs/adr/0002-t1-language-and-cfg.adoc

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit 620d99b into main Sep 30, 2026
13 checks passed
@hyperpolymath
hyperpolymath deleted the docs/plan-truth-adr0002-d4 branch September 30, 2026 09:57
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