From 0d3dbfe3eeaf7b17641d719fc7a4bcdece5e3473 Mon Sep 17 00:00:00 2001 From: vitya Date: Tue, 9 Jun 2026 20:52:00 +0300 Subject: [PATCH] =?UTF-8?q?meta(tasks):=20close=20[using-markitdown-cli-re?= =?UTF-8?q?write-review]=20=E2=80=94=20VERDICT=20PASS?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reviewed the MCP→CLI rewrite of skills/using-markitdown/SKILL.md. Acceptance (3/3) + 2 bonus checks, all green: - No mcp__markitdown__ in SKILL.md (grep 0; only 2 negative "Docker" mentions explaining the old mount caveat no longer applies). - CLI examples correct: markitdown 0.1.6 on PATH; -o/-x/-m/stdin flags match `markitdown --help` verbatim. - Version bumped 1.0.0 -> 1.0.1 (PATCH). - dist/using-markitdown.skill consistent (v1.0.1, no mcp refs). Informational finding (non-blocking): at review time `docker ps` shows a markitdown-mcp:latest container respawned from the out-of-scope mcpServers.markitdown registration in ~/.claude.json. The impl removed the existing containers correctly and flagged this respawn in the concept page. Filed follow-up [using-markitdown-mcp-deregister] (needs-human: keep-or-drop decision on the MCP registration). Co-Authored-By: Claude Opus 4.8 --- .tasks/STATUS.md | 29 ++++++++++++++++++++--------- 1 file changed, 20 insertions(+), 9 deletions(-) diff --git a/.tasks/STATUS.md b/.tasks/STATUS.md index a71f94e..dc68f4f 100644 --- a/.tasks/STATUS.md +++ b/.tasks/STATUS.md @@ -1,5 +1,5 @@ # Task Board -_Updated: 2026-06-09 — skill-using-system-snapshot-review done (VERDICT PASS 3/3; live tool-contract verify + 9-way fresh-context trigger test, 8 clean; 3 informational findings, none blocking — incl. missing deployment scaffold: skill not installed / not in hermes-mapping / no baseline tasks). Ранее: using-tasks-status-read-perf done (using-tasks 1.2.0→1.3.0; done-task archival rule fixes STATUS.md bloat; literal `tasks_get_status`-for-orientation swap NOT done — tool can't enumerate the board, documented). Ранее: session-break-using-tasks done (using-tasks 1.1.0→1.2.0; `session_break` marker — stop after close before claim-next). Ранее: delegate-task-review done (VERDICT PASS; smoke-test 6/6 fresh subagents, no blocking findings, 3 informational notes). Ранее: delegate-task-test-trigger done (pos 5/5, neg 2/3; FP на «создать задачу себе» → follow-up delegate-task-description-fp-fix, shipped v0.2.1). Ранее (2026-06-08): откат churn'а от агент-раннера (always-on dry-run): 5 тасок (using-yt-tools-rate-limit-guard, archive-roundtrip-test, skills-grouping-revisit, hermes-converter-ci, tdd-criteria-precommit-hook) спуриозно claimed/blocked из-за workspace-divergence бага раннера → возвращены в ⚪ ready. yt-tools re-scoped на plugin-репо `OpeItcLoc03/yt-tools` (исходный stub deprecated)._ +_Updated: 2026-06-09 — using-markitdown-cli-rewrite-review done (VERDICT PASS; SKILL.md без `mcp__markitdown__`, CLI-примеры совпадают с `markitdown 0.1.6 --help`, version 1.0.0→1.0.1 PATCH, dist консистентен; 1 informational finding — docker-контейнер респаунится из out-of-scope `~/.claude.json` MCP-регистрации → follow-up `using-markitdown-mcp-deregister` needs-human). Ранее: skill-using-system-snapshot-review done (VERDICT PASS 3/3; live tool-contract verify + 9-way fresh-context trigger test, 8 clean; 3 informational findings, none blocking — incl. missing deployment scaffold: skill not installed / not in hermes-mapping / no baseline tasks). Ранее: using-tasks-status-read-perf done (using-tasks 1.2.0→1.3.0; done-task archival rule fixes STATUS.md bloat; literal `tasks_get_status`-for-orientation swap NOT done — tool can't enumerate the board, documented). Ранее: session-break-using-tasks done (using-tasks 1.1.0→1.2.0; `session_break` marker — stop after close before claim-next). Ранее: delegate-task-review done (VERDICT PASS; smoke-test 6/6 fresh subagents, no blocking findings, 3 informational notes). Ранее: delegate-task-test-trigger done (pos 5/5, neg 2/3; FP на «создать задачу себе» → follow-up delegate-task-description-fp-fix, shipped v0.2.1). Ранее (2026-06-08): откат churn'а от агент-раннера (always-on dry-run): 5 тасок (using-yt-tools-rate-limit-guard, archive-roundtrip-test, skills-grouping-revisit, hermes-converter-ci, tdd-criteria-precommit-hook) спуриозно claimed/blocked из-за workspace-divergence бага раннера → возвращены в ⚪ ready. yt-tools re-scoped на plugin-репо `OpeItcLoc03/yt-tools` (исходный stub deprecated)._ + + +--- + +## ⚪ [using-markitdown-mcp-deregister] — Полный decommission markitdown MCP: удалить `mcpServers.markitdown` из `~/.claude.json`, иначе каждая новая сессия, грузящая MCP, респаунит anonymous-контейнер из `markitdown-mcp:latest` (и `docker ps` снова показывает markitdown — критерий #2 импл-таски не durably выполним). + +**Источник:** finding из `using-markitdown-cli-rewrite-review` (2026-06-09). Импл-таска `using-markitdown-cli-rewrite` удалила существовавшие контейнеры корректно, но оставила MCP-регистрацию (out of scope, явно флагнуто в `.wiki/concepts/using-markitdown-cli-migration.md` §Out of scope). На ревью контейнер `boring_spence` уже респаунился. + +**Weight:** needs-human — правка user-global `~/.claude.json` + keep-or-drop решение: после rewrite скил советует CLI, но MCP-тул `mcp__markitdown__convert_to_markdown` ещё зарегистрирован. Подтвердить с владельцем что MCP больше не нужен, затем удалить запись + `docker rm -f $(docker ps -aq --filter "ancestor=markitdown-mcp:latest")` + (опц.) `docker rmi markitdown-mcp:latest`. +**Status:** ready +**Where I stopped:** (not started) +**Next action:** Спросить владельца: нужен ли ещё markitdown MCP-тул? Если нет — удалить `mcpServers.markitdown` из `~/.claude.json`, снести контейнеры по ancestor, опц. удалить образ. Если да — закрыть как wontfix (критерий #2 импл-таски тогда считается "removed at impl time", respawn — by design). +**Branch:** n/a + ---