The assigner's wants step proves a restatement heard while an ask runs is asked once more (issue 1966 note 170; group with mesh-contracts) #201

Merged
mesh-tools merged 1 commits from fix/1966-engine-reads-asked-again into main 2026-11-10 10:45:53 +00:00
Member

Issue 1966, note 170 (ruling and criteria: notes 172, 176). Group fix/1966-engine-reads-asked-again with mesh-contracts (the node-engine seat's client, where the fix is); one check.

What changes: tests only, modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go. 1966/AC170-4: a step in TestWantsAreWrittenAndNoDifferenceIs holds the engine's responder, restates with a new reader word and says manifest-reads-changed again while the first ask runs, waits for the client's journal line (no sleep), releases the responder, and checks that the assigner's state ends at the restatement with event-at and that the held want moves. Fails against the trunk client after 16.7 s; passes at the group's head in 1.8 s. No test function added (665 unit / 68 integration, unchanged); no manifest touched.

Checked beside mesh-sdk 3176059, mesh-catalog bfa4a085 and mesh-contracts at the group's head: go vet both ways clean in the assigner, builder, indexer and view; their unit suites pass; the assigner's integration package passes (14.4 s).

Reviewed-by: mesh-code-reviewer@shanks (for lead-47@shanks) at 1a6144aa35 (round 1: finding in the other member, fixed; round 2: approve)

https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ

Issue 1966, note 170 (ruling and criteria: notes 172, 176). Group fix/1966-engine-reads-asked-again with mesh-contracts (the node-engine seat's client, where the fix is); one check. What changes: tests only, modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go. 1966/AC170-4: a step in TestWantsAreWrittenAndNoDifferenceIs holds the engine's responder, restates with a new reader word and says manifest-reads-changed again while the first ask runs, waits for the client's journal line (no sleep), releases the responder, and checks that the assigner's state ends at the restatement with event-at and that the held want moves. Fails against the trunk client after 16.7 s; passes at the group's head in 1.8 s. No test function added (665 unit / 68 integration, unchanged); no manifest touched. Checked beside mesh-sdk 3176059, mesh-catalog bfa4a085 and mesh-contracts at the group's head: go vet both ways clean in the assigner, builder, indexer and view; their unit suites pass; the assigner's integration package passes (14.4 s). Reviewed-by: mesh-code-reviewer@shanks (for lead-47@shanks) at 1a6144aa352b1bfeffcea6aa671e67807f67b499 (round 1: finding in the other member, fixed; round 2: approve) https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ
mesh-tools added 1 commit 2026-11-10 10:43:47 +00:00
S4: the assigner's wants step proves a restatement heard while ace's ask runs is asked once more (issue 1966 note 170)
mesh/check checking 1a6144aa352b on a6bba2af808c | novox/mesh-delivery#201 in group fix/1966-engine-reads-asked-again
mesh/delivery merged: source.main moved to novox/mesh-delivery@a6bba2af808c
mesh/delivery-group group fix/1966-engine-reads-asked-again merged: every member is merged
1a6144aa35
1966/AC170-4: in TestWantsAreWrittenAndNoDifferenceIs, ace's engine's answer is held
while it restates with a new form and says manifest-reads-changed again; the
restatement's event is asked once more after the held ask (the journal line), the
state engine-reads ends at the restatement with its event-at, and the held want moves.
Against the node-engine seat's client on trunk the step fails: the event is dropped.
A step in the existing test, no new function (the repository is over its test budget);
the bed hands the program a journal the test awaits a line in, no sleep.

Claude-Session: https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ
Author
Member

Delivery novox/mesh-delivery@1a6144aa352b — merged since 2026-11-10T10:45:53Z

Source — source.main moved to novox/mesh-delivery@a6bba2af80. Whether it reached the machines is mesh-delivery-view's, derived from source.main, the wants and the haves.

Group fix/1966-engine-reads-asked-again: novox/mesh-contracts@c9ffed7042, novox/mesh-delivery@1a6144aa35

Transitions

  • 2026-11-10 10:43 (new) → proposed (announced): novox/mesh-delivery#201's head announced
  • 2026-11-10 10:43 proposed → ready (accepted): a verdict names this commit and the trunk's head as its trunk commit, the gate passed or warned, and the repository's own check did not fail; or the check gates no merge (check-gates-merge false)
  • 2026-11-10 10:45 ready → merged (merged): on the trunk its modules follow: merged there

The commit's note under refs/notes/mesh-plan keeps every transition of this change up to its merge: git log --notes=mesh-plan.

<!-- mesh-delivery:view --> **Delivery** `novox/mesh-delivery@1a6144aa352b` — **merged** since 2026-11-10T10:45:53Z **Source** — source.main moved to novox/mesh-delivery@a6bba2af808c. Whether it reached the machines is `mesh-delivery-view`'s, derived from `source.main`, the wants and the haves. **Group** `fix/1966-engine-reads-asked-again`: novox/mesh-contracts@c9ffed70421a, novox/mesh-delivery@1a6144aa352b **Transitions** - 2026-11-10 10:43 (new) → proposed (announced): novox/mesh-delivery#201's head announced - 2026-11-10 10:43 proposed → ready (accepted): a verdict names this commit and the trunk's head as its trunk commit, the gate passed or warned, and the repository's own check did not fail; or the check gates no merge (check-gates-merge false) - 2026-11-10 10:45 ready → merged (merged): on the trunk its modules follow: merged there The commit's note under `refs/notes/mesh-plan` keeps every transition of this change up to its merge: `git log --notes=mesh-plan`.
mesh-tools merged commit a6bba2af80 into main 2026-11-10 10:45:53 +00:00
mesh-tools deleted branch fix/1966-engine-reads-asked-again 2026-11-10 10:45:53 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/mesh-delivery#201