Conversation
8726aeb to
3a15b2f
Compare
PR SummaryLow Risk Overview ADR-0028 introduces stations as runtime boundaries with binding-scoped identity, a single local reservation path, layered gateway/station/provider receipts, and explicit Codex ref/selector compatibility rules. A new stations-providers execution packet (goal, plan, refs, retro) sequences DIS-70 through DIS-93, Hermes pickup gates, and verification expectations. Mesh and gateway docs are reconciled with that contract: ADR-0013/0014 shift from machine/Codex federation and SSH/ad-hoc modes toward station targets, Tailscale direct plus authenticated cloud MCP, and immutable operation envelopes. Claude plan stops proposing its own lane-identity migration—DIS-79–82 and DIS-70 own shared schema, routing, reservations, and evidence; DIS-50 consumes them. No runtime or registry behavior changes in this PR. Reviewed by Cursor Bugbot for commit 322d893. Configure here. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_5cb3b35b-332a-4538-8890-3ac46cc37142) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3a15b2f6c0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_065297a2-94eb-4918-8500-cdea334759e4) |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 322d893659
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
322d893 to
c3aa0af
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Hooray! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
## Context `main` at e734dac carries the full stations/providers stack (#111–#122) on top of 0.12.0 with no published release. Releases are cut by CI only when `pyproject.toml` carries an untagged version, so this PR is the bump. ## What changed - `pyproject.toml`: 0.12.0 → 0.13.0 - `uv.lock`: project version line only No code changes. When this merges, main CI tags `v0.13.0`, creates the GitHub Release, and dispatches the PyPI publish workflow. ## Since v0.12.0 Provider-binding identity (schema v24), guarded binding-qualified routing, frozen provider requests (v25), correlated receipt observations (v26), independent provider startup/recovery, the Hermes local adapter with `lane_launches` (v27) and generation quarantine, operator diagnostics, packaged skills/docs, and the installed-wheel smoke. Registry schema moves 23 → 29; the release notes carry the migrate-before-start upgrade step. ## Verification - `just check` on this branch: ruff, format, `mypy --strict`, 1,367 tests (17 live tests deselected), wheel/sdist build reporting 0.13.0, package-content check. - `just release-status` reports `create GitHub Release v0.13.0`. Refs: DIS-96
Context
Dispatch needs one durable vocabulary for provider identity, routing, delivery evidence, and future Hermes work. This slice records the shared contract that lets those surfaces evolve without conflating provider bindings, native session identity, or receipt state.
What changed
Verification
7e5d6bais clean at 5/5 with zero P0–P3 findings; adversarial supervisor coverage exercised all 27 immediate and 12 delayed recovery/closure outcome combinations.Risks and boundaries
This is documentation and contract work. It does not implement provider migration, Hermes runtime support, gateway execution, release, or installed-runtime changes. The stack remains ready for review and unmerged.
Closes: DIS-78