From ebf6beaaf12e19ba3a6d264f6fb6166ad7fbd22b Mon Sep 17 00:00:00 2001 From: Hare Date: Sat, 20 Jun 2026 22:48:46 +0900 Subject: [PATCH] ticket: record mcp cli inspection implementation report --- .yoi/tickets/00001KVJKHAFE/item.md | 2 +- .yoi/tickets/00001KVJKHAFE/thread.md | 49 ++++++++++++++++++++++++++++ 2 files changed, 50 insertions(+), 1 deletion(-) diff --git a/.yoi/tickets/00001KVJKHAFE/item.md b/.yoi/tickets/00001KVJKHAFE/item.md index ab949ac8..716c69e0 100644 --- a/.yoi/tickets/00001KVJKHAFE/item.md +++ b/.yoi/tickets/00001KVJKHAFE/item.md @@ -2,7 +2,7 @@ title: 'MCP: add yoi CLI inspection commands' state: 'inprogress' created_at: '2026-06-20T13:29:16Z' -updated_at: '2026-06-20T13:33:06Z' +updated_at: '2026-06-20T13:48:38Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-06-20T13:31:00Z' diff --git a/.yoi/tickets/00001KVJKHAFE/thread.md b/.yoi/tickets/00001KVJKHAFE/thread.md index 90aad7b2..45e2ba35 100644 --- a/.yoi/tickets/00001KVJKHAFE/thread.md +++ b/.yoi/tickets/00001KVJKHAFE/thread.md @@ -110,3 +110,52 @@ Next action: - Wait for Coder implementation report, then inspect branch diff/validation evidence and route to Reviewer。 --- + + + +## Implementation report + +Coder implementation report received from `yoi-coder-00001KVJKHAFE`. + +Implementation commit: +- `c91f5fc9 mcp: add cli inspection` + +Changed areas reported: +- Added `crates/yoi/src/mcp_cli.rs`: + - Read-only `yoi mcp` inspection reports。 + - Commands: `list`, `show`, `tools`, `resources`, `prompts`。 + - Human and `--json` output。 + - Static resolved config inspection only; no MCP server processes are started。 + - Live/provider-discovered state is explicitly reported as `not_live` / `unavailable`。 + - Env values, secret refs, env refs, args, resource content, and prompt content are redacted/omitted。 + - Resource/prompt operation eligibility is reported without content fetch。 +- Updated `crates/yoi/src/main.rs`: + - Added `mcp` namespace to parser, dispatcher, help output, and parser tests。 + +Coder validation reported: +- `cargo test -p yoi mcp`: passed。 +- `cargo check -p yoi`: passed。 +- `cargo fmt --check`: passed。 +- `git diff --check`: passed。 +- `cargo run -p yoi -- --help | grep -E 'yoi mcp (list|show|tools)'`: passed; help shows MCP namespace。 +- `yoi ticket doctor`: passed。 + +Package impact: +- No dependency, lockfile, source-filter, or `package.nix` changes。 +- Nix not run because packaging inputs were not changed。 + +Known risks / deferrals: +- Live Pod MCP state is not integrated; provider-discovered tools/resources/prompts are therefore reported as `not_live` / `unavailable` rather than stale or fabricated。 +- `tools` cannot list Yoi stable provider tool names without live discovery; JSON schema includes future item shape, but item list is empty in static mode。 +- Touched `crates/yoi/src/main.rs` CLI parser/help, so there is possible narrow conflict risk with parallel Plugin Service/Ingress work. No Plugin Service/Ingress files were touched。 + +Orchestrator evidence checked before review dispatch: +- Implementation worktree is clean。 +- HEAD is `c91f5fc9`。 +- Diff from acceptance `12d7e69f..HEAD` is one implementation commit touching 2 files, about 1438 insertions / 1 deletion。 +- `git diff --check 12d7e69f..HEAD` produced no diagnostics。 + +Next action: +- Dispatch Reviewer for r1 review against Ticket requirements, with focus on read-only inspection boundary, no process start/no tools-call/no resource-prompt content fetch, static vs live unavailable honesty, redaction, JSON/human shape, help/tests, and conflict risk。 + +---