[sfaqer]

Closes #33

Что было не так

• Изолированный прогон дописывал свои детальки (OneUnit.ФайлыТестов, ТестовыеМетодыВключаемые, Таймаут и другие) в совет устаревшим СоветДругогоМастера.ЗначенияДеталек. В autumn 5.0.0 сеттер терялся, если лямбда совета уже получила строитель (ПолучитьСтроительМенеджераПараметров()), и дочерняя поделка не находила набор (&Изолированный: совет, настроенный через строитель, теряет детальки OneUnit, и тест не выполняется #33). В autumn 5.0.1 (Строитель настроек 5.0.1: поделка отдаёт менеджер, сеттеры совета работают в любом порядке autumn-library/autumn#150) сеттер снова доходит до строителя, но остается устаревшим.
• Каждый изолированный тест печатал предупреждение об устаревшем методе, CLI - еще два на старте прогона. В MCP-сервере предупреждение уходило в stdout протокола.
• Прямой запуск oscript src/mcp/main.os падал: «Неизвестный тип прилепляемой частицы ... Лог» - не был подключен autumn-logos (через oneunit mcp его подгружал CLI).

Что сделано

• packagedef: autumn 5.0.1, с ним decorator 3.0.0 и collectionos 0.8.4.
• Детальки прогона - отдельный провайдер oneunit строителя совета с приоритетом 0: в поделке (ИсполнительИзоляции) соответствием, в дочернем процессе (ПостроительТочкиВхода) файлом настроек вместо дополнительного каталога поиска.
• Приоритеты configor неочевидны: провайдеры читаются по возрастанию приоритета и прочитанное раньше не перекрывается, а плоский ключ OneUnit.Имя и секция OneUnit - разные ключи индекса, между которыми побеждает прочитанный позже. Поэтому детальки задаются в обеих формах (ПостроительТочкиВхода.НастройкиДеталекПрогона) - их не перекрывают ни совет, ни переменные среды OneUnit_Имя, ни секция OneUnit в autumn-properties.json проекта. Раньше в поделке их перекрывала переменная среды, в процессе - плоский ключ совета.
• src/cli/main.os, src/mcp/main.os собирают совет строителем; src/mcp/main.os подключает autumn-logos.
• Фикстуры советов и тестовые помощники переведены на строитель; README - абзац о том, что совет собирается любым API, а служебные детальки прогона не перекрываются.

Ломающие изменения

• Требуется autumn 5.0.1.
• Совет с фиксированным соответствием деталек (ЗначенияДеталек(Соответствия.ИзЭлементов(...))) больше не работает в изоляции - как не работает и в самом autumn (4.3.13 и 5.x): провайдер configor принимает только Соответствие. Раньше это маскировало копирование деталек в OneUnit. Исправляется оберткой: Новый Соответствие(Соответствия.ИзЭлементов(...)).

Проверка

• Полный прогон локально на autumn 5.0.1: 167/167, предупреждений об устаревших методах в выводе нет (до изменений - 2); после финальных правок фикстур - Изоляция и MCPИнструменты 65/65.
• Новые тесты: совет на устаревшем ЗначенияДеталек работает в изоляции; совет, подменяющий детальки прогона секцией OneUnit и плоским ключом, не перекрывает их на обоих уровнях. Проверено мутациями: без приоритета 0 и без плоских ключей тест падает.
• Сценарий &Изолированный: совет, настроенный через строитель, теряет детальки OneUnit, и тест не выполняется #33 (совет через строитель) покрывают фикстуры ИзолированныйСоветныйНабор и ИзолированныйПроцессныйСоветныйНабор, переведенные на строитель.
• MCP: и oneunit mcp, и oscript src/mcp/main.os отдают в stdout только JSON-RPC.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Улучшения
  • Настройки изолированного прогона теперь имеют приоритет над настройками OneUnit, заданными в совете, для запусков в поделке и отдельном процессе.
  • Поддержаны настройки из устаревших советов изоляции.
  • Обновлена документация по передаче настроек при изолированном запуске.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: b4ec038d-71c1-4623-975d-e775a1c85fc4

📥 Commits

Reviewing files that changed from the base of the PR and between 73102ff and 3e2f57a.

📒 Files selected for processing (13)

• README.md
• packagedef
• src/cli/main.os
• src/core/internal/Классы/ИсполнительИзоляции.os
• src/mcp/main.os
• src/shared/Классы/ПостроительТочкиВхода.os
• tests/fixtures/isolation/ИзолированныйНаборССоветомИзМетода.os
• tests/fixtures/isolation/ИзолированныйНаборСУстаревшимСоветом.os
• tests/fixtures/isolation/Модули/ГенераторИзоляции.os
• tests/fixtures/isolation/НаборСНастройкамиOneUnitВСовете.os
• tests/support/Модули/ПрогоныТестов.os
• tests/Изоляция.os
• tests/Обнаружитель.os

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

Настройка советов переведена на провайдеры параметров autumn 5. Изолированный прогон формирует настройки OneUnit в секционной и плоской формах и передаёт их отдельным провайдером. Добавлены проверки работы с устаревшим советом и приоритета параметров прогона.

Changes

Настройки советов через провайдеры

Layer / File(s)	Summary
Обновление зависимостей и создание советов packagedef, src/cli/main.os, src/mcp/main.os, tests/support/Модули/ПрогоныТестов.os, tests/Обнаружитель.os, tests/fixtures/isolation/ИзолированныйНаборССоветомИзМетода.os	Минимальные версии autumn, collectionos и decorator обновлены. CLI, MCP и тестовая инфраструктура добавляют соответствия через провайдеры параметров.
Формирование и передача настроек изоляции src/shared/Классы/ПостроительТочкиВхода.os, src/core/internal/Классы/ИсполнительИзоляции.os, README.md	Построитель формирует секционные и плоские настройки OneUnit и добавляет провайдер в сценарий. Исполнитель передаёт параметры прогона отдельным провайдером. Документация описывает передачу деталек и их приоритет.
Проверки изолированных советов tests/fixtures/isolation/*, tests/Изоляция.os	Добавлены проверки устаревшего совета и сохранения настроек изолированного прогона при наличии параметров OneUnit в совете. Проверяется обнаружение и выполнение тестов.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Bug fix · Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to 3e2f5

No actionable issue is established; the change appears mergeable after normal checks.

Architecture Summary

Architecture risk: 🔵 Low · up to 3e2f5

The change affects 4 systems.

Changed systems: tests, src, packagedef, README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — tests (service) was modified; 7 changed files map to changed impact.
• observed — src (service) was modified; 4 changed files map to changed impact.
• observed — packagedef (service) was modified; 1 changed file maps to changed impact.
• observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in README.md: Добавлено описание сборки совета строителем autumn 5 или через ЗначенияДеталек, а также передачи служебных деталек изолированного прогона отдельным провайдером с наивысшим приоритетом. Эти детальки нельзя перекрыть советом или секцией OneUnit; прочие детальки передаются изолированной поделке.
• observed — Modified behavior in packagedef: В зависимостях манифеста минимальная версия autumn изменена с 4.3.13 на 5.0.1, collectionos — с 0.8.3 на 0.8.4, decorator — с 2.0.8 на 3.0.0; версии остальных перечисленных зависимостей сохранены.
• observed — Modified behavior in src/cli/main.os: Создание СоветДругогоМастера отделено от настройки параметров: вместо передачи соответствия через ЗначенияДеталек код добавляет это соответствие как провайдер через построитель менеджера параметров.
• observed — Modified behavior in src/core/internal/Классы/ИсполнительИзоляции.os: Добавлена переменная для хранения построителя точки входа.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Заголовок точно описывает основные изменения: обновление до autumn 5.0.1 и перенос деталек прогона на провайдер строителя. Формулировка краткая и понятная.
Linked Issues check	✅ Passed	Требования #33 выполнены. ИсполнительИзоляции больше не вызывает ЗначенияДеталек для служебных параметров. Он добавляет провайдер через `ПолучитьСтроительМенеджераПараметров().ДобавитьПровайдерСоо…
Out of Scope Changes check	✅ Passed	Изменения соответствуют #33 и переходу на autumn 5.0.1. Обновления packagedef, CLI и MCP удаляют устаревший способ передачи деталек и обеспечивают работу новых API. Изменения в `ПостроительТочкиВход…
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Кролик в провайдер добавил деталь,
Чтоб совет в изоляции не терял.
Секция и ключи собрались в JSON,
Тесты проверили каждый закон.
И скачет ушастый: прогон завершён!}

_{Comment @coderabbitai help to get the list of available commands.}