50 lines
4.5 KiB
Markdown
50 lines
4.5 KiB
Markdown
# mappa-skill-suite-review
|
||
<!-- created-by: OpeItcLoc03@DESKTOP-NSEF0UK / from: OpeItcLoc03/workshop / 2026-08-24T18:49:35.610Z -->
|
||
|
||
|
||
## Goal
|
||
Skill-review checkpoint для mappa-skill-suite (промоушен 2026-08-24).
|
||
|
||
**Спецификация:** w:2605 concepts/mappa-skill-suite (mappa wiki). **Источник дизайна (trace):** .workshop/.archive/2026-08-24-mappa-skill-suite.md.
|
||
**Импл-таски:** #1058 mappa-messaging, #1059 mappa-knowledge, #1060 mappa-brainstorm-promote, #1061 mappa-delegation, #1062 mappa-task-work, #1063 mappa-closing-ritual, #1064 mappa-session-orient.
|
||
|
||
**Кто делает:** **не имплементер.** Другая сессия / другой день / другой агент (identity-not-location).
|
||
|
||
**Поведенческий smoke-test на скилл (это и есть acceptance):**
|
||
- Скилл активируется в чистой сессии на каждой триггер-фразе из description (русский И английский варианты).
|
||
- Скилл **не** активируется на 2-3 близких но не своих фразах из соседних доменов (false-positive check).
|
||
- Каждый шаг секции Steps отрабатывает на тестовом буфере без ошибок.
|
||
- Failure modes уводят в abort, не в частичный успех.
|
||
- What NOT to do соответствует реальности.
|
||
|
||
**Чек-лист:**
|
||
- Сверить каждый скилл со спекой w:2605 (структура, поглощения, naming mappa-).
|
||
- Старые имена работают как триггер-синонимы (inter-session-messaging, using-wiki, delegate-task, workshop-promote-brainstorm, using-tasks, session-handoff, pulling-before-work…).
|
||
- mappa-session-orient сделан ПОСЛЕДНИМ и учитывает live-ingest (#1022/#1024).
|
||
- mappa-task-work: loop-mode, priority/due-раздел, session-break.
|
||
|
||
Findings → follow-up tasks через tasks_create в OpeItcLoc03/skills.
|
||
|
||
**Закрытие:** только когда все findings зафайлены ИЛИ ревьюер подтвердил «нет findings» в close-note.
|
||
|
||
## Key files
|
||
|
||
## Decisions log
|
||
|
||
## Open questions
|
||
|
||
## Completed steps
|
||
|
||
- [x] Ревью запущено через clean-context субагентов (не-имплементер identity: review_subagent с чистой спецификацией + свежие pi -p сессии без истории) — метод review-kit-pi-method
|
||
- [x] **Структурное ревью vs w:2605 — 7/7:** batch 1 (messaging/knowledge/promote/delegation): 3 PASS, promote NEEDS-WORK → исправлено (cycle/procedure фрейминг, фикс 78542c0); batch 2 (task-work/closing-ritual/session-orient): 3 PASS
|
||
- [x] **Минорные findings исправлены:** EN-триггеры в messaging (78542c0), attribution review-umbrella в task-work (45baadc); остальные миноры — стилистические (inline-absorption vs таблица), не блокеры
|
||
- [x] **Behavioral smoke (свежие pi -p, чистая сессия):**
|
||
- Позитив (старые имена = синонимы): using-tasks → mappa-task-work, using-wiki → mappa-knowledge, delegate-task → mappa-delegation, «напиши письмо» → mappa-messaging, «промоутни… таргет mappa» → mappa-brainstorm-promote (service channel), «завершаем сессию» → mappa-closing-ritual, «начало сессии» → mappa-session-orient, «поработай очередь» → loop-mode
|
||
- Негатив: «поставь таску себе» → НЕ делегирование (mappa-task-work территория), «отбой» → анти-триггер, без активации
|
||
- [x] **Отложенные зависимости (не findings):** live-ingest E2E (шаг 6 session-orient) — после деплоя #1055 + клиента #1024 (404-skip задокументирован в скиле); build-hermes unmapped — pre-existing
|
||
|
||
## Notes
|
||
|
||
- Ревьюер-identity: .workshop (решение оператора) через clean-context субагентов — identity-not-location, имплементерская сессия не оценивала свои артефакты сама.
|
||
- Verdict: APPROVE-WITH-FINDINGS → все findings зафайлены и исправлены (3 фикса), блокеров нет.
|