Skip to content

proof(flow): item 497 — the strict-lead carry: FlowArm.openNest's ε lead carries trailingPropertyRun = [], accum_step_flow_trace's held-run nested-open disjunct folds into the first - #318

Open
devin-ai-integration[bot] wants to merge 4 commits into
fix-a-grammar-completenessfrom
proof/s3-strict-lead

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This is item 497. It pays S3's item 474 NEXT (a), the strict-lead carry that S3 handed to Row 19. It works on the flow side only and touches none of R12's producers.

FlowArm.openNest.h_lead's ε disjunct now also carries trailingPropertyRun sc.tokens = []. accum_step_flow's h_tuple open-lead conjunct carries the same fact.

-- before
(sp_br = sp_flow₂ ∧ (gapTail sc).leadFree = true)
-- after
(sp_br = sp_flow₂ ∧ (gapTail sc).leadFree = true ∧ trailingPropertyRun sc.tokens = [])

Effect on accum_step_flow_trace:

  • A held run (GapRead.props' h_held : trailingPropertyRun sc.tokens ≠ []) refutes the ε disjunct.
  • The lead is then the strict separation SSeparate n .flowIn sp_scan sp_br that openNestSeqProps / openNestMapProps ask for.
  • So the old fourth disjunct, the held-run nested open [&a [b]], is deleted.
  • That case now lands in the first disjunct with trace ev ++ ev_p. The conclusion has three disjuncts.

Size:

  • StreamAccum.lean: 5 lines. Only the white arm of h_tuple needed a new term, trailingPropertyRun_eq_nil_of_sync h_sync. The props arm already returned Or.inl, and the two openNest producer sites build unchanged.
  • TraceOf/FlowWalk.lean: about 18 lines.
  • accum_step_flow_trace has no other consumer.

Rule 2: the existing P109 (iii) row holds on this tree: the run is never glued to the bracket (glued 0, spanning a line break 1).

Pins moved:

  • GhostIndexCensus: the FlowArm.openNest[14].h_lead ε row text gains the new conjunct. The count stays 123.
  • ValueLineOrigins: liteNodes 673507 → 674092. This is term size only.

Closure log: items 491–496 are declared as gaps, each claimed by an open PR (#311, #313, #312, #316, #315, #317). check-closure-log is green for 11..497.

Local battery:

  • lake build: 1564 jobs, exit 0.
  • All gates are green.
  • hygiene: 11565 / 0 / 0.
  • run-all-tests: 0.
  • suiterunner: 869 / 0 / 151.
  • eventscore: 347 / 358.
  • adversarial: 62181 / 62181.
  • Axioms: the standard three on accum_step_flow and accum_step_flow_trace, and propext on FlowArm.openNest.

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/12a32597e3554be1aeeb264f49263290
Open in Devin Desktop: https://nasa-jpl-demo.devinenterprise.com/desktop/session/12a32597e3554be1aeeb264f49263290?variant=devin
Requested by: @NicolasRouquette


🟢 Devin Review found nothing to fix · View findings

Devin Review

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot changed the title proof(flow): item 496 — the strict-lead carry: FlowArm.openNest's ε lead carries trailingPropertyRun = [], accum_step_flow_trace's held-run nested-open disjunct folds into the first proof(flow): item 497 — the strict-lead carry: FlowArm.openNest's ε lead carries trailingPropertyRun = [], accum_step_flow_trace's held-run nested-open disjunct folds into the first Nov 14, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

devin-ai-integration Bot and others added 4 commits November 15, 2026 00:23
…ead carries trailingPropertyRun = [], accum_step_flow_trace's held-run nested-open disjunct folds into the first

Co-Authored-By: Nicolas Rouquette <nicolas.f.rouquette@jpl.nasa.gov>
…317)

Co-Authored-By: Nicolas Rouquette <nicolas.f.rouquette@jpl.nasa.gov>
…ws, merged without the re-pin)

Co-Authored-By: Nicolas Rouquette <nicolas.f.rouquette@jpl.nasa.gov>
…em 498

Co-Authored-By: Nicolas Rouquette <nicolas.f.rouquette@jpl.nasa.gov>

This branch has not been deployed

No deployments
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