Repository navigation
Close worktree lifecycle and retain recovery review points - #807
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
HyperReviewЭскиз по исходникам · Состав · изменения в проекции --- before.hc
+++ after.hc
@@ -1 +1,2 @@
SpecGraphEngineeringPractice
+ WorktreeLifecycleGuidanceВыбранные связи · после Пояснения, источники и ограниченияСуть измененияИзменённый CONTRIBUTING.md добавляет инструкцию применять существующий worktree-lifecycle skill до создания дерева и при его закрытии; добавлены обычная документация и DocC-страница с процедурами closeout и восстановления и с описанием локальных находок. Текст явно сохраняет границу: архивирование и salvage не одобряют канонические изменения и не вводят runtime policy или новый CI gate. Существующая обязанность синхронизировать публичную техническую документацию с DocC стала применима к новому руководству; показанные тесты и metric/style-инструменты имеют одинаковые полные blob на обеих сторонах и не являются кодом этого изменения. Интерпретация выбранных исходников; порядок узлов не означает порядок выполнения.
Связи и основанияСвязи описаны отдельно от вложенности
Связи в Hypercode (.hcs)Это неполные выборки связей из каждой версии. Отсутствие связи в одной из выборок не устанавливает её добавление или удаление. Пустой лист означает, что связи для этой версии не выбраны. before.hcsafter.hcsОтветственности и источники
Границы анализаPR #807: Исходников в анализе: 10; пропущено: 18. Компилятор проверил структуру Hypercode и ссылки. Соответствие интерпретации коду и поведение программы этим не доказаны. Анализ ограничен предоставленными исходниками и фрагментами контекста; выполнение программы не наблюдалось. Неизменённых файлов контекста по Git blob: 3. Их фрагменты до и после могут различаться охватом.
Точные диапазоны источников по узламSpecGraphEngineeringPractice WorktreeLifecycleGuidance Ок или не ок? CI · результаты проверокСнимок для Результат check-run не доказывает выполнение конкретного теста, полноту покрытия или соответствие архитектуре.
Снимок неполный: показано не более 12 проверок. Отсутствие, ожидание или сбой CI не блокируют архитектурный обзор. Review gate: review thread. |
Intent Atoms · v1Scope: specification YAML in Status: complete Proposal obligations · draftStatus: complete for declared fields Informational only; this report does not gate merge readiness. |
SpecificationMetrics · diagnostic PR measurement
Collection: base
Scope: files assigned |
Summary
worktree-lifecycleskill and require its closeout guidance in the repository's agent/contributor instructions.Motivation
Old execution directories outlived their task and review records. A live ignored SpecDraft notebook also depended on a temporary worktree, while the primary checkout stayed on an older branch.
Goals
Changes
Spec Evidence
Spec-ID: NO-SPEC-IMPACT
Proposal-ID: 0048
Spec-Impact: docs-only
Spec-Rationale: Engineering lifecycle guidance preserves the existing salvage review boundary from proposal 0048. Canonical YAML, runtime code, authority and evidence admission remain unchanged.
Validation
Commands run:
Results:
Risks / Notes
Checklist