Conversation
The client holds each order lane in a domain tier and a communication tier that plan 691 named, and a view tier it did not: Deferred, derived by toDeferred and read only by the pages, with Provisional carrying a value each lane chooses for itself. The plan names the view tier as a DU per lane whose cases are the states a page can be in, states the five tiers of client state and the rule that a command is built in a Settled arm only, and schedules the Session, Signing and plain-fetch applications as follow-ups. The code waits for plan 646 step 4. Refs informedica#706 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K9K57bmde8G9rfreSMKZ6L
Review on informedica#707: the branch was cut from the fork's master, before plan 646 merged, so its tree lacked the prerequisite; rebased onto upstream master. Step 7 now says that every caller of Deferred.inProgress reads a lane and no plain fetch does, so deleting it leaves nothing to migrate. Refs informedica#706 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K9K57bmde8G9rfreSMKZ6L
Each lane gets the DU the pages will read: the states a page can be in, each with what is valid in it and nothing of the request. The workbench: no patient, evaluating, settled, changing with the context sent shown meanwhile. The plan: no patient, opening, settled and changing with the dialog's selection inside them, so a selection without a plan cannot be written. `view` in both machines projects to them, case for case as `toDeferred` does today; `holds` and `dialog` are the two page rules that are more than a match. Nothing reads them yet. A seed shows as settled for now; the arm goes with the seed state (informedica#646). Refs informedica#706 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K9K57bmde8G9rfreSMKZ6L
halcwb
marked this pull request as ready for review
September 15, 2026 09:33
Contributor
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Step 1 of plan #706 (#707), stacked on it: the view DUs of the two order lanes, next to today's
toDeferred, read by nothing yet.OrderContextView = NoPatient | Evaluating | Settled | Changing,OrderPlanView = NoPatient | Opening | Settled of plan * selected | Changing of plan * selected; a selection without a plan cannot be written.OrderContextState.viewandOrderPlanState.viewproject case for case astoDeferreddoes; the tests mirror the projection tests value for value.OrderPlanView.holds(the prescribe button's rule) andOrderContextView.dialog(the order dialog's context, settled or changing as the plan is) are the two page rules that are more than a match.dialoglives in the context machine file because it compiles after the plan's.Settledfor now; that arm goes withSeededin 646 step 4, after which this branch rebases.Shared tests 513 green (6 new), Fable compile green, dependency rule green, 201 lines.
🤖 Generated with Claude Code
https://claude.ai/code/session_01K9K57bmde8G9rfreSMKZ6L