diff --git a/.yoi/tickets/00001KTR81P9X/item.md b/.yoi/tickets/00001KTR81P9X/item.md index e62996c7..0918b4d5 100644 --- a/.yoi/tickets/00001KTR81P9X/item.md +++ b/.yoi/tickets/00001KTR81P9X/item.md @@ -1,8 +1,8 @@ --- title: 'Extend pod::feature API for external protocol-backed capability providers' -state: 'inprogress' +state: 'done' created_at: '2026-06-10T07:48:14Z' -updated_at: '2026-06-14T06:36:23Z' +updated_at: '2026-06-14T06:39:48Z' assignee: null readiness: 'implementation_ready' risk_flags: ['feature-api', 'tool-registry', 'permission-scope', 'prompt-context', 'dynamic-registry', 'service-lifecycle'] diff --git a/.yoi/tickets/00001KTR81P9X/thread.md b/.yoi/tickets/00001KTR81P9X/thread.md index f6781cde..86d7b323 100644 --- a/.yoi/tickets/00001KTR81P9X/thread.md +++ b/.yoi/tickets/00001KTR81P9X/thread.md @@ -139,4 +139,43 @@ Risks / notes: - Permission denial and bounded-result behavior are not reimplemented in this diff; they are intentionally inherited because provider tools are installed as ordinary Worker tools. +--- + + + +## Implementation report + +Integration outcome: merged and validated. + +- Implementation branch: `ticket-00001KTR81P9X-feature-provider-api` +- Implementation commit: `053a4f90 feat: add protocol provider feature contributions` +- Review commit: `d0e8d791 ticket: approve feature provider api` +- Orchestrator merge commit: `3d140dbb merge: feature provider api` + +Reviewer result: +- `approve`。Provider API extension stays in the contribution/reporting substrate, does not reintroduce HostAuthority/grants, does not implement MCP trust/plugin permission policy, and uses ordinary ToolRegistry/Worker/history/permission paths. + +Orchestrator validation after merge: +- `cargo fmt --check`: PASS +- `git diff --check`: PASS +- `cargo test -p pod feature::tests`: PASS +- `cargo check --workspace --all-targets`: PASS + +Not run: +- `nix build .#yoi`: skipped at integration because prior Coder run hit host disk pressure during final link; this change did not modify dependencies, Cargo.lock, packaging, or source filters. Recent package validation was performed on adjacent E2E/Profile work. + +Residual notes: +- Full `cargo test -p pod` is known to fail unrelated prompt assertion tests expecting old guidance text. Reviewer classified those as unrelated/pre-existing to this feature API diff. + +Next: +- Mark Ticket done and clean up child coder/reviewer Pods plus implementation worktree/branch. + +--- + + + +## State changed + +Implementation branch was reviewed, approved, merged into the Orchestrator branch as `3d140dbb`, and validated in the Orchestrator worktree. Feature-provider focused tests, formatting, diff check, and workspace check passed. Ticket implementation work is done; closure remains separate. + ---