Skip to content

test: preserve scalar result across status query - #114

Merged
Th0rgal merged 1 commit into
mainfrom
codex/lsc1-07-status-preserve
Sep 4, 2026
Merged

Th0rgal merged 1 commit into
mainfrom
codex/lsc1-07-status-preserve

Conversation

@Th0rgal

@Th0rgal Th0rgal commented Sep 4, 2026

Copy link
Copy Markdown
Member

Summary

  • add the bounded LSC1-07 full-LSC-1 HostRuntime trace for NEGOTIATE, fixture step-0 SET_CONSTANT, STATUS_QUERY, and the untouched host-generated RETIRE
  • require STATUS to report RESULT_PENDING for transaction 1 while preserving the staged CRC, next PC/FP, proposed m[2]=3 write, and pending state
  • compare executable-model and authored-RTL response frames with only the documented NEGOTIATE feature-mask normalization
  • add a fail-closed temporary RTL mutant changing only the scalar STATUS transaction-ID selection from staged_txn_id to zero

Validation

  • make lsc1-scalar-status-host-boundary
  • make lsc1-scalar-status-host-boundary-mutation
  • python3 test/test_lsc1_scalar_status_workflow.py -v
  • (cd lean && lake build LeanVMBMinCore.HostPreparedBoundary)
  • make consistency interface-check smoke placeholders checksum-check
  • make check
  • git diff --check

Claim boundary

This is finite executable-model observation plus a separate finite authored-RTL differential and targeted mutant. Lean compiles separately and does not prove this RTL trace. Production executable-model, Lean sources, and authored RTL behavior are unchanged. Netlist, P&R, FPGA, hardware, LSC-1µ, universal refinement, and end-to-end claims are excluded.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-04T04:59:00.219145Z e55ff63 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Th0rgal
Th0rgal merged commit 6f38112 into main Sep 4, 2026
48 checks passed
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