Repository navigation
Dogfood a source-traced SpecDraft live preview - #804
Conversation
Capture the operator idea through existing notes tools and prepare proposal 0228 plus a review-pending specification draft. Serve the saved sketch through a loopback-only viewer, with named display policies, focused tests and bounded browser observations. Spec-ID: SG-SPEC-0007, SG-SPEC-0050, SG-SPEC-0051 Proposal-ID: 0045, 0228 Spec-Impact: tooling Spec-Rationale: Dogfood source capture and bounded visualization without canonical promotion or hidden note mutations.
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2a9ac4ce09
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
HyperReviewЭскиз по исходникам · Состав · изменения в проекции --- before.hc
+++ after.hc
@@ -0,0 +1,5 @@
+SpecDraftLivePreview
+ NotesSnapshotAdapter
+ DisplayStatePolicy
+ LocalPreviewBoundary
+ NotesMindMapViewВыбранные связи · после Пояснения, источники и ограниченияСуть измененияВ PR добавлены локальный просмотр сохранённых заметок, адаптер снимков, классификация состояний, HTTP-граница и браузерная mind map; в `SpecGraph.md` добавлена ссылка на новую DocC-страницу. Код реализует чтение текущей revision, отображение свежего/пустого/устаревшего/недоступного состояния и сохранение последнего успешного снимка при ошибках. Документы уточняют статус как технический прототип при ожидающем рассмотрения SpecDraft, сохраняют границу существующего NotesStore и исключают редактирование, историю, продвижение и каноническую публикацию. Указанные в документах тестовые и браузерные наблюдения — сообщения источников; здесь выполнение не проверялось. Интерпретация выбранных исходников; порядок узлов не означает порядок выполнения.
Связи и основанияСвязи описаны отдельно от вложенности
Связи в Hypercode (.hcs)Это неполные выборки связей из каждой версии. Отсутствие связи в одной из выборок не устанавливает её добавление или удаление. Пустой лист означает, что связи для этой версии не выбраны. before.hcsafter.hcsОтветственности и источники
Границы анализаPR #804: Исходников в анализе: 30; пропущено: 23. Компилятор проверил структуру Hypercode и ссылки. Соответствие интерпретации коду и поведение программы этим не доказаны. Анализ ограничен предоставленными исходниками и фрагментами контекста; выполнение программы не наблюдалось.
Неизменённых файлов контекста по Git blob: 5. Их фрагменты до и после могут различаться охватом.
Точные диапазоны источников по узламSpecDraftLivePreview
NotesSnapshotAdapter DisplayStatePolicy
LocalPreviewBoundary
NotesMindMapView Ок или не ок? 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 |
…preview-proposal # Conflicts: # tools/docc_sync_contract.json # tools/proposal_promotion_registry.json # tools/proposal_runtime_registry.json
Summary
Motivation
Goals
Changes
read_notes; no second store or filesystem-watcher dependency.make specdraft-preview, focused tests, DocC and runtime/promotion tracking. Retain bounded technical observations with explicit gaps; screenshots stay local.Spec Evidence
Spec-ID: SG-SPEC-0007, SG-SPEC-0050, SG-SPEC-0051
Proposal-ID: 0045, 0228
Spec-Impact: tooling
Spec-Rationale: Implement a bounded visualization of existing pre-canonical notes, preserving source attribution and existing promotion authority. Core implementation-agnostic specification nodes are unchanged.
Intent-Decision-Review: tools/intent_decision_reviews/0228_specdraft_live_preview.json
Validation
Commands run:
.venv/bin/python -B -m pytest -q tests/test_specdraft_preview.py tests/test_conversation_notes.py .venv/bin/python -B tools/python_quality.py node --check tools/specdraft_preview/static/app.js node --test tests/specdraft_preview_inspector.cjs make docc-sync proposal-tracking-gate materialization-packet-check proposal-spec-trace pre-spec-semantics make spec-evidence-gate SPEC_EVIDENCE_BASE_REF=origin/main make intent-decision-review SPEC_EVIDENCE_BASE_REF=origin/main INTENT_DECISION_REVIEW_ARGS='--pr-body-file "$PR_BODY_FILE"'Results:
e255d0f2dce94685a353da78669b0886; proposal handoffb7244c6348c64bfb8ec08f3ba630d9d3; measured live revision002d3ceea1cc439489546a7deb13afa5.Risks / Notes
make specdraft-preview SPECDRAFT_PREVIEW_WORKSPACE=/path/to/sketch; optionalSPECDRAFT_PREVIEW_ARGS='--node-id existing_note'selects a branch. Port 0 prints a free local URL.review_pending; no human acceptance, canonical allocation/materialization or evidence admission is manufactured. Prototype execution is technical evidence attached to candidates.Checklist