From 86c87ded89f03dcc34084398fbe5703853d540fd Mon Sep 17 00:00:00 2001 From: Hare Date: Fri, 19 Jun 2026 20:42:31 +0900 Subject: [PATCH] ticket: dispatch plugin cli schema validation fixes --- .yoi/tickets/00001KVFD3YSV/item.md | 2 +- .yoi/tickets/00001KVFD3YSV/thread.md | 30 ++++++++++++++++++++++++++++ 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/.yoi/tickets/00001KVFD3YSV/item.md b/.yoi/tickets/00001KVFD3YSV/item.md index 58709e28..94e71393 100644 --- a/.yoi/tickets/00001KVFD3YSV/item.md +++ b/.yoi/tickets/00001KVFD3YSV/item.md @@ -2,7 +2,7 @@ title: 'Plugin: add read-only CLI inspection list/show' state: 'inprogress' created_at: '2026-06-19T07:39:23Z' -updated_at: '2026-06-19T11:41:24Z' +updated_at: '2026-06-19T11:42:23Z' assignee: null readiness: 'implementation_ready' risk_flags: ['plugin', 'cli', 'diagnostics', 'read-only', 'json-output', 'no-execution'] diff --git a/.yoi/tickets/00001KVFD3YSV/thread.md b/.yoi/tickets/00001KVFD3YSV/thread.md index 18f63d6d..64928866 100644 --- a/.yoi/tickets/00001KVFD3YSV/thread.md +++ b/.yoi/tickets/00001KVFD3YSV/thread.md @@ -588,3 +588,33 @@ Validation note: - Reviewer performed static code/diff review and relied on Orchestrator-reported cargo/nix validation. No mutations were made by Reviewer. --- + + + +## Plan + +Review follow-up dispatch: + +Reviewer `yoi-reviewer-00001KVFD3YSV-r3` は `request_changes` を返したため、Ticket review として記録し、既存 Coder Pod `yoi-coder-00001KVFD3YSV` に修正依頼を送った。 + +Resolved: +- Status vocabulary blocker は解消済み。 +- Package path/location + API/schema version blocker も解消済み。 + +Remaining blocker: +- Static inspection が Tool definition/schema validity を検証していないため、invalid Tool schema が `active` として報告され得る。 +- Runtime registration path は `validate_tool_name(...)` と `validate_input_schema(...)` を実行しているが、read-only inspection path の `inspect_resolved_plugin_static` は runtime ABI / host API permission / `authorize_plugin_tool(...)` grant check に留まり、同じ static Tool-definition validity を mirror していない。 +- Ticket intent は “tool schema [is] valid” の inspection と `invalid schema` を `rejected` として扱うことを要求している。 + +Required fixes sent to Coder: +- Static inspection に、registration path と同等の static Tool definition validity を追加する。少なくとも Tool name と `input_schema` validation を含める。 +- Invalid tool name / invalid input schema は bounded diagnostics にする。 +- affected tool/package は全 tools invalid/unusable なら `rejected`、一部 usable / 一部 invalid なら `partial` として表現する。 +- invalid Tool schema behavior の focused JSON/human tests を追加する。 +- read-only/no-execution behavior を維持する。 + +Next: +- Coder の修正 commit / validation report を待つ。 +- 修正後、static inspection schema validity / no-execution / typed status semantics を中心に再レビューする。 + +---