Skip to content

Add depth repair effect visibility#676

Merged
SoundBlaster merged 2 commits into
mainfrom
codex/0209-depth-repair-effect-visibility
Jul 9, 2026
Merged

Add depth repair effect visibility#676
SoundBlaster merged 2 commits into
mainfrom
codex/0209-depth-repair-effect-visibility

Conversation

@SoundBlaster

Copy link
Copy Markdown
Member

Summary

  • Adds proposal 0209 for review-only structural depth repair effect visibility.
  • Emits structural_depth_delta from rerun preview/materialization and lets Idea Maturity explainers cite the delta evidence.
  • Link related issue/task: Idea-to-Spec depth repair visibility follow-up.

Motivation

  • Depth-driven repair can add event-storming entries and workflow relations, but downstream surfaces need durable evidence of what changed after rerun.
  • This addresses the gap between raw candidate_structure_depth counts and user-facing repair impact explanation.

Goals

  • Emit before/after counts, added refs/relations, remaining shallow dimensions, and effect status.
  • Keep the change report-only/review-only, out of Metrics schema/gate/score semantics.
  • Intentionally out of scope: promotion gating, canonical spec mutation, Ontology writes, Git operations, and read-model publication.

Changes

  • Added proposal/archive source 0209_depth_repair_effect_visibility plus roadmap/DocC/tools docs and proposal registries.
  • Added structural_depth_delta to rerun preview and rerun materialization summaries/deltas.
  • Added Idea Maturity readiness-explainer evidence for structural depth repair effect.
  • Added regression coverage for event-storming entries, workflow topology deltas, materialization deltas, and maturity explainer evidence.

Spec Evidence

For SpecGraph logic/tooling changes, cite at least one existing SG-SPEC-####
or proposal ####. PR body is preferred; commit trailers with the same fields
are accepted. For a true no-impact change, use Spec-ID: NO-SPEC-IMPACT and
provide a rationale.

Spec-ID: NO-SPEC-IMPACT
Proposal-ID: 0209
Spec-Impact: lifecycle
Spec-Rationale: Adds review-only lifecycle visibility for depth repair effects in idea-to-spec rerun artifacts without changing canonical specs or authority gates.

Validation

  • Tests added/updated for changed behavior
  • Local checks passed

Commands run:

.venv/bin/python -m pytest tests/test_idea_to_spec_rerun_preview.py tests/test_idea_to_spec_rerun_materialization.py tests/test_idea_maturity_metrics_report.py -q
make proposal-tracking-gate
make docc-sync
git diff --check
.venv/bin/python -m ruff check tools/idea_to_spec_rerun_preview.py tools/idea_to_spec_rerun_materialization.py tools/idea_maturity_metrics_report.py tests/test_idea_to_spec_rerun_preview.py tests/test_idea_to_spec_rerun_materialization.py tests/test_idea_maturity_metrics_report.py

Results:

  • 72 passed
  • proposal tracking gate passed
  • DocC sync validation passed
  • ruff check passed

Risks / Notes

  • Backward compatibility impact: additive fields in existing review-only artifacts; old consumers can ignore them.
  • Migration/config changes required: none.
  • Known limitations: this does not change Metrics schema/status/gates and does not make depth a score.

Checklist

  • PR title clearly describes the change
  • Scope is focused and minimal
  • Documentation updated (or N/A)
  • Workflow/artifact changes distinguish presence from readiness/status
  • Consumers prefer authoritative repaired/promoted/latest artifacts where relevant
  • Temporary or intermediate paths are scoped to the current run/task where relevant
  • No secrets or sensitive data added

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5bf135e213

ℹ️ 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".

Comment thread tools/idea_to_spec_rerun_preview.py
Comment thread tools/idea_to_spec_rerun_preview.py
Comment thread tools/idea_to_spec_rerun_materialization.py Outdated
Comment thread tools/idea_maturity_metrics_report.py
Comment thread tools/idea_to_spec_rerun_materialization.py
Comment thread tools/idea_to_spec_rerun_materialization.py Outdated
Comment thread tools/idea_to_spec_rerun_materialization.py
@SoundBlaster SoundBlaster merged commit 85bd85a into main Jul 9, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant