0208: Add safe workflow topology repair#675
Merged
Merged
Conversation
Proposal-ID: 0208 Spec-Impact: behavior | schema | lifecycle | tooling
fc1a8c1 to
f1c7ff3
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fc1a8c1c25
ℹ️ 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".
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
0208for safe workflow-topology repair from structural-depth findings.workflow_topology_gapclarification requests when a candidate has commands plus event-storming context but no workflow edges.event_storming_hints.workflow_relationsthrough rerun input, preview, and review-only materialization.event_storming_relation[]answers.Motivation
Goals
Changes
0208_safe_workflow_topology_repairproposal, archive source, registries, README, roadmap, and DocC updates.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 fieldsare accepted. For a true no-impact change, use
Spec-ID: NO-SPEC-IMPACTandprovide a rationale.
Spec-ID: N/A
Proposal-ID: 0208
Spec-Impact: behavior | schema | lifecycle | tooling
Spec-Rationale: Adds typed review-only workflow topology repair semantics to the existing idea-to-spec clarification/rerun pipeline.
Validation
Commands run:
.venv/bin/python -m py_compile tools/idea_to_spec_clarification_requests.py tools/idea_to_spec_answer_rerun_input.py tools/idea_to_spec_rerun_preview.py tools/idea_to_spec_rerun_materialization.py .venv/bin/python -m pytest tests/test_idea_to_spec_clarification_requests.py tests/test_idea_to_spec_answer_rerun_input.py tests/test_idea_to_spec_rerun_preview.py tests/test_idea_to_spec_rerun_materialization.py tests/test_real_idea_answer_authoring.py -q make proposal-tracking-gate docc-sync PYTHON=.venv/bin/python git diff --check make test PYTHON=.venv/bin/python make publish-bundle PYTHON=.venv/bin/python make spec-evidence-gate SPEC_EVIDENCE_BASE_REF=27df57a9610a1463084e782afe62376dfcbdcf5c SPEC_EVIDENCE_HEAD_REF=f1c7ff30659a3c9feb554f8f9dbb70b8fcd62fa8Results:
76 passed1822 passed1316files, safety gatepassedRisks / Notes
Checklist