Skip to content

Cover proposal 0036 topology observation#388

Merged
SoundBlaster merged 1 commit into
mainfrom
codex/proposal-0036-reobservation
May 16, 2026
Merged

Cover proposal 0036 topology observation#388
SoundBlaster merged 1 commit into
mainfrom
codex/proposal-0036-reobservation

Conversation

@SoundBlaster
Copy link
Copy Markdown
Member

Summary

  • What changed? Updated Proposal 0036 observation coverage to use a markdown-wrapping-safe marker.
  • Link related issue/task (if any): graph next-move reobservation follow-up.

Motivation

  • Why is this change needed? Proposal 0036 was implemented, but its observation marker expected a single wrapped sentence and kept surfacing as a stale reobservation gap.
  • What problem, gap, or user need does it address? Lets the graph advance from stale proposal observation cleanup to actual runtime realization backlog work.

Goals

  • What should this PR achieve? Mark Proposal 0036 observation coverage as covered and guard it with a focused regression test.
  • What is intentionally out of scope? No runtime behavior changes and no canonical spec changes.

Changes

  • Replaced the brittle full-sentence Proposal 0036 observation marker with a stable substring present in the proposal markdown.
  • Added a regression asserting Proposal 0036 has implemented runtime, covered validation, covered observation, and no next gap.

Validation

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

Commands run:

python3 -m pytest -q tests/test_supervisor.py -k 'proposal_0036 or proposal_0021 or proposal_runtime_index'
git diff --check
make backlog
make next-move
ruff check tests/test_supervisor.py
python3 -m pytest -q tests/test_supervisor.py

Results:

  • Focused proposal runtime tests: 4 passed, 605 deselected.
  • Backlog summary after both stacked fixes: entry_count = 29, runtime_realization = 20, no reobservation bucket.
  • Next move now points to Proposal 0009 / runtime_realization.
  • Full supervisor tests: 609 passed in 288.76s.

Risks / Notes

Checklist

  • PR title clearly describes the change
  • Scope is focused and minimal
  • Documentation updated (or N/A)
  • No secrets or sensitive data added

Copy link
Copy Markdown
Member Author

@SoundBlaster SoundBlaster left a comment

Choose a reason for hiding this comment

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

Reviewed the PR-local diff for Proposal 0036. I did not find a blocking issue here: narrowing the marker to reflective/proposal branch starts moving toward domain-first matches the wrapped markdown text while staying specific to the Proposal 0036 document.

Residual dependency: this PR is stacked on #387, where I left a blocking review comment about the Proposal 0021 marker being too broad. This PR should be fine once the parent stack issue is resolved.

@SoundBlaster SoundBlaster changed the base branch from codex/proposal-0021-reobservation to main May 16, 2026 19:40
@SoundBlaster SoundBlaster merged commit 67dcef7 into main May 16, 2026
5 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