meta(tasks): close [using-markitdown-cli-rewrite-review] — VERDICT PASS

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 <noreply@anthropic.com>
This commit is contained in:
2026-06-09 20:52:00 +03:00
parent efd21fba5e
commit 0d3dbfe3ee

View File

@@ -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)._
<!--
Canonical layout. One block per task. Per-task deep context lives in
@@ -923,20 +923,31 @@ No `consult` was needed; nothing irreversible or cross-cutting. Files changed: `
---
## 🔴 [using-markitdown-cli-rewrite-review] — Review результата `using-markitdown-cli-rewrite`: проверить что SKILL.md не содержит mcp__markitdown__ вызовов, CLI-примеры корректны, docker контейнер удалён, version bumped.
## 🟢 [using-markitdown-cli-rewrite-review] — Review результата `using-markitdown-cli-rewrite`: проверить что SKILL.md не содержит mcp__markitdown__ вызовов, CLI-примеры корректны, docker контейнер удалён, version bumped.
**Weight:** needs-claude
**notify:** OpeItcLoc03/workshop
**Status:** active
**Where I stopped:** (not started)
**Next action:** Прочитать обновлённый SKILL.md, проверить acceptance criteria, вынести вердикт PASS/FAIL.
**Blocker:** using-markitdown-cli-rewrite
**Status:** done
**Verdict:** **PASS** (deliverable correct; 1 informational finding на docker-respawn — out-of-scope MCP registration).
**Where I stopped:** Review прогнан 2026-06-09. Acceptance: (1) SKILL.md `mcp__markitdown__` — grep 0 совпадений, остались только 2 *негативных* упоминания "Docker"/"no longer apply" (объясняют, что старый mount-caveat больше не действует) ✅; (2) CLI-примеры корректны — `markitdown 0.1.6` на PATH, флаги `-o`/`-x`/`-m`/stdin-via-`cat` совпадают с `--help` дословно ✅; (3) version 1.0.0→1.0.1 PATCH (git-confirmed) ✅; bonus — `dist/using-markitdown.skill` консистентен (v1.0.1, no mcp refs) ✅. **Finding (informational, не блокирует):** на момент ревью `docker ps` показывает 1 контейнер `markitdown-mcp:latest` (`boring_spence`, up ~1 min) — это задокументированный respawn от out-of-scope `mcpServers.markitdown` в `~/.claude.json` (импл удалил существовавшие контейнеры корректно, флагнул respawn в concept-page). Критерий #2 не durably выполним без deregister MCP → follow-up `using-markitdown-mcp-deregister` (needs-human: keep-or-drop решение по user-global config).
**Next action:** (none — kept until merged)
**Branch:** n/a
**Notify:** OpeItcLoc03/workshop
**Owner:** DESKTOP-NSEF0UK:claude-opus:11584
**Claim token:** 232ba323-c664-437e-9eca-fe05caed379f
**Claim expires at:** 2026-06-09T17:58:51.892Z
<!-- created-by: OpeItcLoc03@DESKTOP-NSEF0UK / from: OpeItcLoc03/workshop / 2026-06-09T17:44:01.459Z -->
<!-- closed-by: DESKTOP-NSEF0UK:claude-opus:11584 / 2026-06-09 / VERDICT PASS; acceptance 3/3 + 2 bonus checks green; 1 informational finding (docker respawn from out-of-scope ~/.claude.json MCP registration) → follow-up using-markitdown-mcp-deregister -->
---
## ⚪ [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
<!-- created-by: DESKTOP-NSEF0UK:claude-opus:11584 / from: using-markitdown-cli-rewrite-review finding / 2026-06-09 -->
---