chore/feat(wp498): Ф14 — убрать мёртвый текст про триал + одно имя наружу (Наставник) - #307
Open
TserenTserenov wants to merge 3 commits into
Open
chore/feat(wp498): Ф14 — убрать мёртвый текст про триал + одно имя наружу (Наставник)#307TserenTserenov wants to merge 3 commits into
TserenTserenov wants to merge 3 commits into
Conversation
…10 убрал триал из кода ещё раньше) Fable-ревью (Ф13) нашло: WP-210 Ф2a убрал пробный период из кода, но текст остался в 4 местах — обещал функциональность, которой нет. - Удалены неиспользуемые ключи trial_active/trial_expiring/trial_expired/status_trial (schema.yaml + es/fr/zh, ни одного вызова в Python) и day_7_trial (schema.yaml, тоже без вызовов). - mydata.py: убрано «Trial 30 дней» из описания тира T0 на экране /mydata. - paywall.consultation: добавлено упоминание, что Наставник/Навигатор/Диагност теперь бесплатны без подписки (Ф12/Ф13) — раньше текст обещал платный доступ ко всей консультации целиком. Refs: peer-session 2026-09-05-06-wp498-mentor-concept-named-code Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…едёт к Наставнику
Fable-ревью + решение пилота 05.09: участники не различают роли по именам, а
Навигатор/Диагност (DP.SC.197, DP.ROLE.042 §5-6) — компоненты связки Наставника,
не отдельные роли того же уровня. Автодетект по содержанию текста (без явного
имени) теперь всегда ведёт к mentor; отдельные имена navigator/diagnostician
остаются доступны только явным префиксом («Навигатор, …») или командой
(/navigator, /diagnose, WP-156).
- states/common/consultation.py: _detect_role() шаг 2 (content pattern matching)
унифицирован на return "mentor" для обоих списков; явный префикс (шаг 1) и
concept_naming (шаг 1a) не тронуты.
- config/prompts/mentor.md: сценарии 10-11 — алгоритмы Навигатора (SS.1, выбор
старта/мема) и Диагноста (SS.2, формальная диагностика 5 уровней) перенесены
внутрь диспетчер-промпта — холодное ревью нашло, что без этого Наставник
отвечал бы на эти сценарии беднее, чем раньше отвечали отдельные промпты.
- docs/scenarios/02-04-consultation.md: убран устаревший алгоритм «диагност →
навигатор → наставник, первое совпадение побеждает» — теперь всё content-only
ведёт к mentor.
- Тесты: explicit-prefix ("Навигатор, ...") остаются navigator/diagnostician;
content-only ("с чего начать", "протестируй меня" и т.д.) теперь mentor;
+ 2 теста на покрытие mentor.md алгоритмами SS.1/SS.2.
Refs: peer-session 2026-09-05-06-wp498-mentor-concept-named-code
Review iters: 1 cold-context (CONDITIONAL → пробел в mentor.md закрыт, doc drift починен)
Verify: 553 passed / 10 pre-existing fails (same as clean pilot base)
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…naming перед списками, не после) Самопроверка после отправки предыдущего коммита на верификацию: описание шага «Автодетект по содержанию» перепутало реальный порядок проверки (_detect_role: explicit prefix → concept_naming якоря 1a → diagnostician → navigator, все content-совпадения → mentor). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This was referenced Sep 5, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что
Реализация двух из четырёх пунктов, согласованных пилотом после разбора концепции доступа к ролям независимой ревью-моделью (Fable).
П.1 — мёртвая копия про пробный период. WP-210 убрал пробный период из кода ещё раньше, но текст остался в 5 местах и обещал функциональность, которой нет. Удалены неиспользуемые ключи
trial_active/trial_expiring/trial_expired/status_trial/day_7_trial(schema.yaml + es/fr/zh — нигде не вызываются из Python), убрано «Trial 30 дней» из описания тира T0 на/mydata. Текст платного экрана консультации теперь говорит, что Наставник/Навигатор/Диагност бесплатны.П.2 — «одно имя наружу». Участники не различают роли по именам (дословно пилот: «люди не разделяют эти роли у себя»), а по документам платформы Навигатор и Диагност — компоненты связки Наставника, не отдельные роли того же уровня. Автодетект по содержанию текста (без явного имени) теперь всегда ведёт к Наставнику; отдельные имена остаются доступны только явным префиксом или командой (
/navigator,/diagnose).Холодное ревью (Sonnet) на первый проход вернуло CONDITIONAL:
mentor.mdне покрывал алгоритмы, которые раньше давали отдельныеnavigator.md/diagnostician.md(выбор старта, работа с мемами, формальная 5-шаговая диагностика). Добавлены сценарии 10-11 вmentor.mdс полным переносом этих алгоритмов, обновлена документация сценария (убран устаревший порядок «диагност → навигатор → наставник, первое совпадение побеждает»).Проверка
553 passed, 10 падений — та же десятка, что и на чистой базе pilot. Роутинг подтверждён исполнением: явный префикс сохраняет свою роль, содержимое без имени всегда ведёт к mentor.
Осталось (не в этом PR)
concept_namedдля тира T4: требует АрхГейта (архитектурное решение поперёк трёх сервисов), сейчас в работе.Refs: peer-session 2026-09-05-06-wp498-mentor-concept-named-code
🤖 Generated with Claude Code