Skip to content

FE-1573: Construct and explain one real net region from a genuine conversation - #9562

Draft
lunelson wants to merge 62 commits into
mainfrom
ln/fe-1573-construct-and-explain
Draft

FE-1573: Construct and explain one real net region from a genuine conversation#9562
lunelson wants to merge 62 commits into
mainfrom
ln/fe-1573-construct-and-explain

Conversation

@lunelson

@lunelson lunelson commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

🌟 What is the purpose of this PR?

Cut Mission 7 Step A authority: establish whether Brunch can elicit a genuinely complex operational account of Vestera's multi-line production eligibility and changeovers, conserve that meaning, construct a meaningful Petrinaut region from it, and explain every ordinary behaviour-affecting element through declared basis and recorded effects.

This PR is the authority cut only. It converts the accepted planning record into live MISSION.md, leaves Step B in a non-authoritative packet, and restacks the branch onto Mission 6b for review. Implementation, paid evidence, and the product demo have not begun.

What it establishes: the Step A contract, execution graph, inventory/explanation standard, probe tables, and an opened foundation over Mission 6b's accepted narrowed claim. What it does not claim: a constructed region, a working why operation, inherited Mission 7 evidence, Step B authorization, or any release.

🔗 Related links

🚫 Blocked by

  • The parent Voice-reconciliation PR, #9564
  • Mission 6b's narrowed foundation was accepted by Lu on 2026-09-07; its three deferred claims remain explicit constraints
  • Separate owner acceptance before any Step B amendment

🔍 What does this change?

  • Cuts live Mission 7 Step A into root MISSION.md and archives Mission 6.
  • Moves remaining closure/demo work into the Step B packet; that packet is not execution authority.
  • Updates the future spine so FE-1573 is the tracker projection for construction and explanation, advancing FE-1478.
  • Records the restack onto accepted Mission 6b and opens shared implementation and paid Step A runs under the existing budget and proof constraints.
  • Changes no product code, tests, or publishable packages.
🏗️ Agent notes

Mission authority is libs/@hashintel/brunch-agent/MISSION.md (live Step A, cut 2026-09-07). The six sections, condensed:

Imperative. Prove the current core elicitation + plugin sdcpn-modelling guidance can elicit, conserve, construct, and explain one accepted Vestera region. Visible goal: talk about multi-line eligibility and changeovers, watch the region take shape, ask why by name, and see workpiece passage, evidential standing, and recorded construction steps. Hand-edited and basis-less controls must refuse honestly. Step A ends at an owner gate, not a release.

Throughline. Genuine conversation → mounted /agents/chat/:instanceId → production ChatAgent → core update_workpiece settles Markdown/revisionId/sha256 → plugin mutation cites settled revision and declared basis → actual browser validates and records independently observed effects → authorized why resolves record → basis → passage → evidence. Parallel A1 carrier, A2 revisions, A3 browser effects, and early A4 history pins join into model-facing why and the genuine tracer; Lu's owner gate precedes any B1/B2/B3 work.

Proof. This PR proves only that the contract is cut and restacked. The authorized Step A proof (adversarial tracer, four probes, two measurements, inventory/utility rubric, behavioral discriminator) has not started. Prospective oracles live in MISSION.md; none are claimed as existing tests.

Constraints. Mission 6b's accepted foundation permits shared implementation and paid Step A calls, but Mission 7 must preserve causal per-step client results, active-submission Stop, and the explicit limits on hydration attribution, post-settlement local withholding, and latency. Preserve Voice marker/lifecycle/cancellation contracts in A2/A3. Mixed update_workpiece + terminating construction batch is forbidden. Petrinaut owns schemas and mutations; no second log, capture ledger, ontology, or projection engine. Budget: US$100, Sonnet-class, conservative 200-call cap.

Fog-line. Compaction survival, genuine reopen/relocation, passage locators, carrier behavior per admitted class, browser pre/post observation, and whether current guidance can conserve Vestera complexity at an affordable cadence. These are probe questions, not permission to shrink the region or skip an owner gate.

Stop or reorient. Stop if a join drops Mission 6b's causal-result, marker, lifecycle, attribution or cancellation constraints or claims its deferred properties as inherited proof; if the tracer needs an answer key or a simpler substitute region; if mixed-batch or uncitable revisions are required; if effects cannot be independently checked; or if work broadens into Mission 9/10/11 scope.

Deferred. Step B packet owns demo/closure. Missions 9/10 own repeat/change/revision breadth. Mission 6b's direct-user hydration attribution, post-settlement durable withholding, and comparative latency remain explicit deferred limitations rather than Mission 7 obligations or inherited passes.

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • require changes to docs which are made as part of this PR (live MISSION.md, MISSION.next.md, Step B packet, and the restack evidence note)

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

⚠️ Known issues

  • No implementation yet. Reviewers should treat this as a contract review, not a product change.
  • Mission 6b is accepted narrowly. Its owner witness passes the local microphone/mutation/resume/active-Stop path while explicitly deferring direct-user hydration attribution, post-settlement durable withholding, and comparative latency. Mission 7 must not claim those properties or reuse the prepared-fixture witness as its own proof.
  • Step B is not authorized. The demo script, revision list/diff, and full-region release gate live only in the amendment packet.

🐾 Next steps

  • Review and accept or amend this Step A contract.
  • Re-pin the complete prompt/tool baseline from accepted #9564, including causal per-step client results, before instrument freeze or paid runs.
  • Shared implementation and paid Step A work may proceed within the accepted authority and budget; Mission 7 still owes every named oracle.
  • Step B requires a separate owner acceptance and an authority-only amendment commit.

🛡 What tests cover this?

None added. This PR changes mission documents only. Existing parent suites are regression priors, not Step A evidence.

❓ How to test this?

  1. Read MISSION.md Status, Imperative, Throughline, Proof, Constraints, Fog-line, and Stop or reorient.
  2. Confirm the Step B packet is marked non-authoritative and is not treated as a second live mission.
  3. Confirm MISSION.next.md points FE-1573 / FE-1478 at this cut and does not authorize implementation.
  4. Confirm no product, test, or changeset files are in the unique diff against #9564.

📹 Demo

None. The product demo is a Step B obligation after the owner gate.

@vercel

vercel Bot commented Sep 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
hash Ready Ready Preview Sep 8, 2026 6:26pm UTC
hashdotdesign-tokens Ready Ready Preview Sep 8, 2026 6:26pm UTC
petrinaut Ready Ready Preview Sep 8, 2026 6:26pm UTC
petrinaut-docs Ready Ready Preview Sep 8, 2026 6:26pm UTC

Request Review

@github-actions github-actions Bot added area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team labels Sep 7, 2026

lunelson commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

@lunelson
lunelson changed the base branch from ln/fe-1575-resumable-workpiece-petrinaut to graphite-base/9562 September 7, 2026 11:15
@lunelson
lunelson force-pushed the ln/fe-1573-construct-and-explain branch from 2506ec5 to 03bbac5 Compare September 7, 2026 12:56
@lunelson
lunelson changed the base branch from graphite-base/9562 to ln/fe-1580-reconcile-voice-resumable-workpiece September 7, 2026 12:57
@lunelson lunelson changed the title Prepare Mission 7 cut with Vestera and parallel execution structure FE-1573: Construct and explain one real net region from a genuine conversation Sep 7, 2026
@lunelson
lunelson force-pushed the ln/fe-1573-construct-and-explain branch from 03bbac5 to 7fed841 Compare September 7, 2026 16:10
@lunelson
lunelson force-pushed the ln/fe-1573-construct-and-explain branch from 7fed841 to bba337a Compare September 7, 2026 17:19
@lunelson
lunelson changed the base branch from ln/fe-1580-reconcile-voice-resumable-workpiece to graphite-base/9562 September 7, 2026 18:03
Co-authored-by: Cursor <cursoragent@cursor.com>
@lunelson
lunelson deployed to pull-request September 8, 2026 18:09 — with GitHub Actions Active
@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.31%. Comparing base (94dff8e) to head (e38a6a7).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9562      +/-   ##
==========================================
+ Coverage   65.93%   66.31%   +0.38%     
==========================================
  Files        1886     1774     -112     
  Lines      198341   191759    -6582     
  Branches     8236     7841     -395     
==========================================
- Hits       130772   127168    -3604     
+ Misses      66039    63109    -2930     
+ Partials     1530     1482      -48     
Flag Coverage Δ
apps.hash-ai-worker-py ?
apps.hash-ai-worker-ts 1.99% <ø> (ø)
apps.hash-api 15.57% <ø> (ø)
apps.hash-graph 12.54% <ø> (ø)
backend-integration-tests ?
blockprotocol.type-system 38.15% <ø> (ø)
deer ?
error-stack ?
local.claude-hooks 0.00% <ø> (ø)
local.harpc-client 51.49% <ø> (ø)
local.hash-backend-utils 3.27% <ø> (ø)
local.hash-graph-sdk 10.02% <ø> (ø)
local.hash-isomorphic-utils 12.22% <ø> (ø)
local.hash-subgraph ?
rust.antsi 2.36% <ø> (ø)
rust.deer ?
rust.error-stack 90.81% <ø> (ø)
rust.harpc-codec 84.70% <ø> (ø)
rust.harpc-net 96.19% <ø> (-0.02%) ⬇️
rust.harpc-tower 67.03% <ø> (ø)
rust.harpc-types 0.00% <ø> (ø)
rust.harpc-wire-protocol 92.23% <ø> (ø)
rust.hash-codec 72.76% <ø> (ø)
rust.hash-config 81.14% <ø> (ø)
rust.hash-graph-api 19.71% <ø> (ø)
rust.hash-graph-atlas 80.36% <ø> (-0.01%) ⬇️
rust.hash-graph-authentication 96.02% <ø> (ø)
rust.hash-graph-authorization 63.14% <ø> (ø)
rust.hash-graph-embeddings 91.88% <ø> (ø)
rust.hash-graph-postgres-store 32.15% <ø> (ø)
rust.hash-graph-store 48.41% <ø> (ø)
rust.hash-graph-temporal-versioning 50.18% <ø> (ø)
rust.hash-graph-types 0.00% <ø> (ø)
rust.hash-graph-validation 84.71% <ø> (ø)
rust.hash-middleware 90.92% <ø> (ø)
rust.hashql-ast 89.63% <ø> (ø)
rust.hashql-compiletest 28.39% <ø> (ø)
rust.hashql-core 78.92% <ø> (ø)
rust.hashql-diagnostics 72.51% <ø> (ø)
rust.hashql-eval 79.82% <ø> (ø)
rust.hashql-hir 89.09% <ø> (ø)
rust.hashql-mir 87.92% <ø> (ø)
rust.hashql-syntax-jexpr 94.04% <ø> (ø)
rust.sarif ?
sarif ?
tests.hash-backend-integration ?
unit-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq

codspeed-hq Bot commented Sep 8, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ 6 benchmarks measured no execution time

Nothing ran under measurement, usually because the compiler removed the code under test. These results are not comparable, so they count as unchanged.

Preventing compiler optimizations

✅ 98 untouched benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
⚠️ as_constant < 1 ns < 1 ns N/A
⚠️ constant_equal < 1 ns < 1 ns N/A
⚠️ constant_not_equal < 1 ns < 1 ns N/A
⚠️ access < 1 ns < 1 ns N/A
⚠️ runtime_equal < 1 ns < 1 ns N/A
⚠️ runtime_not_equal < 1 ns < 1 ns N/A

Comparing ln/fe-1573-construct-and-explain (e38a6a7) with main (94dff8e)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (e1593ae) during the generation of this report, so 94dff8e was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps area/deps Relates to third-party dependencies (area) area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) area/tests New or updated tests type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

2 participants