Repository navigation
Plugin install path: skills need a full checkout, README has no install line, docs drift #93
Copy link
Copy link
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
⚠️ Выводы не проверены человеком — доверия к ним нет. Ишью заведено по аудиту, который сделал Claude Code. Автор репозитория просил считать эти выводы непроверенными: прежде чем чинить, дважды воспроизведите каждый пункт и отметьте результат в разделе «Перепроверка». Пункт 1 частично — вывод, а не наблюдение (установку через/plugin installClaude не запускал).
Follow-up к #2 / #3: эпик обещал установку «/plugin marketplace add axisrow/claude-devtools → /plugin install session-audit@…».
Наблюдения
- Скиллы требуют клон с зависимостями. Оба
SKILL.mdзапускаютpnpm analyze:session/pnpm analyze:sessionsи сами предупреждают: «if it fails with "no such script", cd to this plugin's checkout first and runpnpm installonce». Это ставит все зависимости Electron-приложения. Предположение (не проверено): после/plugin installв каталоге плагина нетnode_modules, аpnpmиз cwd проекта пользователя скрипта не найдёт. - npx-бинарники уже есть (CLI: expose session-audit as npx-runnable bin (stretch from #8) #22). Проверено:
pnpm build:cli→pnpm pack→npm install <tgz>в пустой папке →session-auditиsession-inventoryработают. Но установка тянет 324 пакета (всеdependenciesприложения), а--helpпечатаетusage: pnpm analyze:session. - В README нет ни слова «plugin» или «marketplace».
claude plugin validate .(Claude Code 2.1.296) советует добавить строку установки:/plugin install session-audit --marketplace axisrow/claude-devtools. - Расхождения в документации:
- CHANGELOG (
0.1.0-fork.1) пишет «removed thepriceFamilystopgap», а коммит8485b57(fix(cli): data quality — ghost rounds excluded, router-retry copies flagged (#14, #15) #18) вернулpriceFamily()для не-Claude моделей; - после
fork.1около 50 изменений форка (False context spikes from zero-usage provider rounds #14–fix(turn): task-notification opens no user turn #70) в CHANGELOG не попали; секция[Unreleased]ниже относится к апстриму.
- CHANGELOG (
Как воспроизвести
grep -n -i -E "plugin|marketplace" README.md # пусто
claude plugin validate . # совет про строку установки
pnpm build:cli && pnpm pack --pack-destination /tmp/p
mkdir /tmp/t && cd /tmp/t && npm init -y && npm install --ignore-scripts /tmp/p/*.tgz
npx --no-install session-audit --help # usage: pnpm ...
git log --format='%h %s' -S priceFamily -- src/cli/analyzeSession.tsПлюс вручную: в чистой сессии Claude Code /plugin marketplace add axisrow/claude-devtools, /plugin install session-audit, вызвать /audit-session — посмотреть, что падает.
Что сделать
- Перевести скиллы на бинарники (
npx session-audit …/npx session-inventory …или путь кdist-cliвнутри плагина) и обновитьallowed-tools. - Отделить зависимости CLI от зависимостей приложения, чтобы
npxне тянул Electron/React. - Справка CLI — имя бинарника, а не
pnpm analyze:*. - Раздел «Plugin» в README с командами установки.
- Поправить CHANGELOG:
priceFamily, записи о изменениях послеfork.1, порядок секций.
Перепроверка (каждый пункт — дважды, независимо)
-
/plugin install+/audit-sessionв чистой сессии не работает без ручногоpnpm install— проверка №1 (не проверялось ни разу) - то же — проверка №2
- npx-путь работает из пустой папки — проверка №1
- то же — проверка №2
- README без инструкции установки плагина; CHANGELOG расходится с кодом — проверка №1 и №2
Acceptance
-
/plugin install→/audit-sessionработает в чистой сессии без ручных шагов -
npx session-audit --helpпоказывает правильное имя; установка не тянет зависимости приложения - README содержит установку плагина; CHANGELOG соответствует коду
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request
Activity
feat(plugin): self-contained install — committed dist-cli, plugin REA…
axisrow commentedon Oct 12, 2026
Закрыто PR #129 (смержено 2026-10-12): закоммиченный dist-cli, скиллы дергают бинарь через CLAUDE_PLUGIN_ROOT без pnpm install, раздел Claude Code Plugin в README, CHANGELOG backfill, CI-гард git diff --exit-code dist-cli.
axisrow commentedon Oct 12, 2026
Переоткрываю: PR #129 был смержен в base-ветку ao/claude-devtools-49/issue-92-skill-docs (ветка уже смерженного #128), а не в main — автозакрытие не сработало, и изменений dist-cli на main нет. Готовлю отдельный PR из cherry-pick 129 в main.