Repository navigation
Package portable SpecDraft CLI, preview resources and matched skill - #812
Conversation
Spec-ID: SG-SPEC-0007 Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Preserve captured intent provenance while installing existing notes and preview interfaces independently of the developer checkout.
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Include only declared portable runtime resources and retain distribution verification guidance.
Spec-ID: SG-SPEC-0007 Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Bind captured intent to the review proposal and preserve actual installed distribution evidence without canonical adoption.
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Keep CN-CAPTURE review inputs unchanged while shipping portable skill resources separately.
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Preserve historical review hashes without freezing current build metadata; continue rejecting changed inputs in a selected review revision.
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Review five draft obligations individually without changing canonical Intent counts or inventing new policy Specifications.
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. |
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Use runner.temp only where GitHub exposes the runner context; static workflow validation prevents a pre-job failure.
Proposal-ID: 0230 Spec-Impact: tooling Spec-Rationale: Keep source inventory integrity current; all five individual decisions and the exact PR description remain unchanged.
Intent Atoms · v1Scope: specification YAML in Status: complete Proposal obligations · draftStatus: complete for declared fields Informational only; this report does not gate merge readiness. |
HyperReviewЭскиз по исходникам · Состав · изменения в проекции --- before.hc
+++ after.hc
@@ -0,0 +1,3 @@
+PortableSpecDraftRuntime
+ InstalledCommandFacade
+ SkillResourceInstallerВыбранные связи · после Пояснения, источники и ограниченияСуть измененияPR добавляет установленный Python-интерфейс SpecDraft: метаданные пакета задают команду `specgraph`, фасад маршрутизирует notes и preview к сохранённым интерфейсам и добавляет явную установку ресурсов skill. Также добавлены руководство, smoke-сценарий и workflow сборки/установки; фрагмент workflow неполный и не подтверждает запуск. Комментарии и документация утверждают сохранение прежних заметок и предпросмотра, а неизменённые исходные API/store и preview подтверждают, что их реализация в этом PR не менялась. Уточнены ограничения: выбранное workspace хранит данные отдельно от установки, preview остаётся loopback/read-only, кандидат не заявлен опубликованным в package index, а полный supervisor, Docker и native Windows находятся вне среза. Интерпретация выбранных исходников; порядок узлов не означает порядок выполнения.
Связи и основанияСвязи описаны отдельно от вложенности
Связи в Hypercode (.hcs)Это неполные выборки связей из каждой версии. Отсутствие связи в одной из выборок не устанавливает её добавление или удаление. Пустой лист означает, что связи для этой версии не выбраны. before.hcsafter.hcsОтветственности и источники
Границы анализаPR #812: Исходников в анализе: 41; пропущено: 14. Компилятор проверил структуру Hypercode и ссылки. Соответствие интерпретации коду и поведение программы этим не доказаны. Анализ ограничен предоставленными исходниками и фрагментами контекста; выполнение программы не наблюдалось.
Неизменённых файлов контекста по Git blob: 8. Их фрагменты до и после могут различаться охватом.
Точные диапазоны источников по узламPortableSpecDraftRuntime
InstalledCommandFacade
SkillResourceInstaller Ок или не ок? CI · результаты проверокСнимок для Результат check-run не доказывает выполнение конкретного теста, полноту покрытия или соответствие архитектуре.
Снимок неполный: показано не более 12 проверок. Отсутствие, ожидание или сбой CI не блокируют архитектурный обзор. Review gate: review thread. |
SpecificationMetrics · diagnostic PR measurement
Collection: base
Scope: files assigned |
Summary
Install SpecDraft notes and preview independently of the SpecGraph developer checkout, with one versioned CLI, packaged assets and self-contained skill. Proposal 0230 records the bounded delivery contract.
Motivation
Consumer projects currently depend on a changing source checkout. A fixed pipx installation lets them keep using SpecDraft while development and worktree cleanup continue separately.
Goals
specgraph notes,specgraph preview,specgraph --versionand explicit skill export.Changes
user_intent::specdraft-portable-runtime-20261027→proposal::0230(under_review). Record five proposal obligations and draft criterion/test bindings. No model refinement or canonical adoption is claimed.pyproject.tomldoes not rewrite or freeze previous reviews; changes within a selected review snapshot still fail.Spec Evidence
Spec-ID: SG-SPEC-0007
Proposal-ID: 0230
Spec-Impact: tooling
Spec-Rationale: Preserve attributed intent and review boundaries while distributing the existing note/preview implementation independently. Governance specs remain unchanged; the draft target has no canonical Requirement binding.
Intent-Decision-Review: tools/intent_decision_reviews/0230_portable_specdraft.json
CLI dispatch, package-resource copying and destination byte checks are adapter/filesystem mechanics. Preview continues to reuse the existing immutable display-state Specifications; this packaging slice adds no domain policy classes merely to increase S.
Validation
Commands run:
Results:
Risks / Notes
python -m specgraphnow exposes the real CLI instead of the scaffold greeting.Checklist