novox/mesh-sdk
9
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Hold a holder's caller list from its config watch, so a timed-out ask refuses no verb (issue 1966; group with mesh-catalog, merge first) #182

Merged
mesh-tools merged 1 commits from fix/1966-callers-config-read into main 2026-11-09 22:00:45 +00:00
Conversation 2 Commits 1 Files Changed 2
+103
mesh-tools commented 2026-11-09 21:56:57 +00:00
Member
Copy Link
Copy Source

On 2026-11-09 21:25 UTC the builds of mesh-delivery-view and mesh-delivery-checker were not stored: the seat oci-artifact-registry's publish asked the provisioner for its callers setting on every call, the ask timed out, and the whole verb was refused. HoldVerbCallers reads the list once through WatchConfig and replaces it on each change; a later call never asks; a change that cannot be read keeps the last list and is said, never widened to the defaults.

Quick fix: keep the last value in one holder. Thorough fix (chosen): the shared reader holds it, so every holder of callers gets it.

Group fix/1966-callers-config-read: this pull request, then mesh-catalog's.

issue 1966

Reviewed-by: review-1966@shanks at c9fc6c5227
Pushed-by: lead-47@shanks

https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ

On 2026-11-09 21:25 UTC the builds of mesh-delivery-view and mesh-delivery-checker were not stored: the seat oci-artifact-registry's publish asked the provisioner for its `callers` setting on every call, the ask timed out, and the whole verb was refused. `HoldVerbCallers` reads the list once through `WatchConfig` and replaces it on each change; a later call never asks; a change that cannot be read keeps the last list and is said, never widened to the defaults. Quick fix: keep the last value in one holder. Thorough fix (chosen): the shared reader holds it, so every holder of `callers` gets it. Group fix/1966-callers-config-read: this pull request, then mesh-catalog's. issue 1966 Reviewed-by: review-1966@shanks at c9fc6c52277dd632e92c9a0a418d99abc873fec6 Pushed-by: lead-47@shanks https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ
mesh-tools added 1 commit 2026-11-09 21:56:57 +00:00
Hold a holder's caller list from its config watch, asked once, so a timed-out ask refuses no verb (issue 1966)
mesh/delivery merged: source.main moved to novox/mesh-sdk@7d0a89aa3f98
Details
mesh/delivery-group group fix/1966-callers-config-read merged: every member is merged
Details
mesh/check checking c9fc6c52277d on 7d0a89aa3f98 | novox/mesh-sdk#182 in group fix/1966-callers-config-read
c9fc6c5227 
Claude-Session: https://claude.ai/code/session_01JG73KEq4zoDfM9WtYpcQgZ
mesh-tools commented 2026-11-09 21:57:00 +00:00
Author
Member
Copy Link
Copy Source

Delivery novox/mesh-sdk@c9fc6c52277d — merged since 2026-11-09T22:00:47Z

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

Group fix/1966-callers-config-read: novox/mesh-catalog@24bf59219e, novox/mesh-sdk@c9fc6c5227

Transitions

  • 2026-11-09 21:56 (new) → proposed (announced): novox/mesh-sdk#182's head announced
  • 2026-11-09 21:57 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-09 22:00 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-sdk@c9fc6c52277d` — **merged** since 2026-11-09T22:00:47Z **Source** — source.main moved to novox/mesh-sdk@7d0a89aa3f98. Whether it reached the machines is `mesh-delivery-view`'s, derived from `source.main`, the wants and the haves. **Group** `fix/1966-callers-config-read`: novox/mesh-catalog@24bf59219ee5, novox/mesh-sdk@c9fc6c52277d **Transitions** - 2026-11-09 21:56 (new) → proposed (announced): novox/mesh-sdk#182's head announced - 2026-11-09 21:57 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-09 22:00 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 commented 2026-11-09 21:57:05 +00:00
Author
Member
Copy Link
Copy Source

mesh-delivery: the operator's Approve of this pull request's merge into main is ended: a head it covers moved on: the heads it covers are no longer the group's heads. Nothing is merged on it. The operator is asked again once the checks pass at the new head.

mesh-delivery: the operator's Approve of this pull request's merge into main is ended: a head it covers moved on: the heads it covers are no longer the group's heads. Nothing is merged on it. The operator is asked again once the checks pass at the new head.
mesh-tools merged commit 7d0a89aa3f into main 2026-11-09 22:00:45 +00:00
mesh-tools deleted branch fix/1966-callers-config-read 2026-11-09 22:00:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
mesh/ready
Reviewed and ready: mesh-delivery merges at the marked head once its checks pass (hq ADR 0364)
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
agent-ace agent-g14 agent-novox agent-shanks jschoubben mesh-tools miel.truyen nox-agent
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/mesh-sdk#182