Skip to content

Plugin install path: skills need a full checkout, README has no install line, docs drift #93

Description

@axisrow

⚠️ Выводы не проверены человеком — доверия к ним нет. Ишью заведено по аудиту, который сделал Claude Code. Автор репозитория просил считать эти выводы непроверенными: прежде чем чинить, дважды воспроизведите каждый пункт и отметьте результат в разделе «Перепроверка». Пункт 1 частично — вывод, а не наблюдение (установку через /plugin install Claude не запускал).

Follow-up к #2 / #3: эпик обещал установку «/plugin marketplace add axisrow/claude-devtools → /plugin install session-audit@…».

Наблюдения

  1. Скиллы требуют клон с зависимостями. Оба SKILL.md запускают pnpm analyze:session / pnpm analyze:sessions и сами предупреждают: «if it fails with "no such script", cd to this plugin's checkout first and run pnpm install once». Это ставит все зависимости Electron-приложения. Предположение (не проверено): после /plugin install в каталоге плагина нет node_modules, а pnpm из cwd проекта пользователя скрипта не найдёт.
  2. 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.
  3. В README нет ни слова «plugin» или «marketplace». claude plugin validate . (Claude Code 2.1.296) советует добавить строку установки: /plugin install session-audit --marketplace axisrow/claude-devtools.
  4. Расхождения в документации:

Как воспроизвести

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 — посмотреть, что падает.

Что сделать

  1. Перевести скиллы на бинарники (npx session-audit … / npx session-inventory … или путь к dist-cli внутри плагина) и обновить allowed-tools.
  2. Отделить зависимости CLI от зависимостей приложения, чтобы npx не тянул Electron/React.
  3. Справка CLI — имя бинарника, а не pnpm analyze:*.
  4. Раздел «Plugin» в README с командами установки.
  5. Поправить 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 соответствует коду
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions