Skip to content

Preserve scoped repaired handoff inputs#682

Merged
SoundBlaster merged 3 commits into
mainfrom
codex/dedicated-canary-repaired-handoff-scoping
Jul 11, 2026
Merged

Preserve scoped repaired handoff inputs#682
SoundBlaster merged 3 commits into
mainfrom
codex/dedicated-canary-repaired-handoff-scoping

Conversation

@SoundBlaster

@SoundBlaster SoundBlaster commented Jul 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Preserve caller-supplied scoped repaired-handoff inputs in the product workspace Make flow.
  • Bind Idea Maturity to the same scoped repaired handoff.
  • Cover a non-default workspace invocation.

Motivation

  • Dedicated workspaces must not rebuild repaired artifacts from root/default runs/* inputs.

Goals

  • Keep repaired promotion handoff provenance workspace-scoped.
  • Preserve the existing repaired handoff contract.

Changes

  • Remove target-local variable overrides that discarded caller-supplied repaired input paths.
  • Add a dry-run Make regression for scoped repaired promotion handoff inputs.

Spec Evidence

Spec-ID: N/A
Proposal-ID: 0177
Spec-Impact: lifecycle
Spec-Rationale: This is a repair to the existing 0177 repaired-candidate promotion handoff flow: scoped workspace inputs must remain the authoritative source through handoff and maturity generation.

Validation

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

Commands run:

.venv/bin/python -m pytest tests/test_idea_maturity_metrics_report.py -q -k repaired_promotion_handoff
make spec-evidence-gate SPEC_EVIDENCE_BASE_REF=main SPEC_EVIDENCE_HEAD_REF=HEAD

Results:

  • Focused repaired-handoff tests passed.

Risks / Notes

  • Backward compatibility impact: default root paths remain unchanged when callers do not provide scoped paths.
  • Migration/config changes required: none.
  • Known limitations: output remains review-only and does not promote canonical specs.

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: 8b88856da6

ℹ️ 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 Makefile Outdated
Comment thread Makefile Outdated
Proposal-ID: 0177

Spec-Impact: lifecycle

Spec-Rationale: Scoped repaired-handoff inputs are a lifecycle correction to the existing 0177 handoff contract.
@SoundBlaster SoundBlaster merged commit eb251b0 into main Jul 11, 2026
6 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