Skip to content

Добавить командный skill foxford-frontend-review - #1

Closed
aralot wants to merge 5 commits into
foxford:mainfrom
aralot:feat/foxford-frontend-review
Closed

aralot wants to merge 5 commits into
foxford:mainfrom
aralot:feat/foxford-frontend-review

Conversation

@aralot

@aralot aralot commented Oct 19, 2026

Copy link
Copy Markdown

Повторяющиеся договорённости из frontend-ревью собраны в foxford-frontend-review. Скилл определяет применимость правил к изменениям задачи и исправляет однозначные локальные нарушения; по явному запросу возвращает отчёт без изменений.

CODESTYLE.md содержит 12 правил со стабильными ссылками, парными примерами и исключениями: CSS, React-компоненты, публичные импорты, локализация, UI-kit и существующий код с @foxford/vm. Документ можно читать и цитировать без установки скилла. Добавлены инструкция установки через npx skills и сохранённые сценарии оценки.

Проверка

  • 19 октября на Go 1.26.1 прошли gofmt, go vet ./..., go test -race ./... и go build -o .build/adc ./bin/adc.
  • По сохранённым результатам оценки 16 октября: изолированная локальная установка для Claude Code/Codex, 12/12 тестов примеров, проверка типов и ограниченная браузерная проверка CSS прошли.
  • Проверены backend-only и смешанный diff, незакоммиченные изменения, режим без правок, проектные исключения и сохранение пользовательских изменений.

Границы первой версии

Существующие implement, code-review и runner не изменены: описана будущая точка подключения после ревью перед коммитом. Независимый запуск в новых пользовательских сессиях Claude/Codex не проводился; оценка инструкций выполнена агентом на подготовленных примерах. VM-правила проверены для версии 2.0.2. Docker runtime smoke с opt-in переменными не запускался; Go/runtime эта ветка не меняет.

Отчёты от 16 октября сохраняют исторический блокер отсутствия Go. Он снят повторным прогоном 19 октября, указанным выше.

Sergey Zhirkov added 5 commits October 16, 2026 22:36
Ticket: /Users/user/Work/stoege/.aralot/tickets/team-frontend-review-skill/issues/01-installable-css-review.md
Ticket: /Users/user/Work/stoege/.aralot/tickets/team-frontend-review-skill/issues/02-react-public-imports-i18n.md
Ticket: /Users/user/Work/stoege/.aralot/tickets/team-frontend-review-skill/issues/03-ui-kit-contract-and-design-decision.md
Ticket: /Users/user/Work/stoege/.aralot/tickets/team-frontend-review-skill/issues/04-vm-api-review.md
Ticket: /Users/user/Work/stoege/.aralot/tickets/team-frontend-review-skill/issues/05-integrated-acceptance-and-usage.md
@aralot aralot closed this Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant