Skip to content

Promote PIT contract candidate#1290

Open
2233admin wants to merge 2 commits into
Fission-AI:mainfrom
2233admin:codex/pit-contract-promotion
Open

Promote PIT contract candidate#1290
2233admin wants to merge 2 commits into
Fission-AI:mainfrom
2233admin:codex/pit-contract-promotion

Conversation

@2233admin

@2233admin 2233admin commented Jul 1, 2026

Copy link
Copy Markdown

Summary

This PR adds a small OpenSpec change proposal for promoting PIT planning notes into a reviewable contract candidate.

It also tightens the related change-stacking wording so archived providers only suppress missing-provider warnings when there is explicit change-level evidence.

Why

The goal is to test the governance workflow on a real but small case: take an existing research/spec insight, turn it into a contract candidate, and require a clear test or evidence gate before it becomes implementation-ready.

Validation

  • node bin\openspec.js validate promote-pit-contract-candidate --json
  • node bin\openspec.js validate add-change-stacking-awareness --json

Generated with Codex using GPT-5.5.

Summary by CodeRabbit

  • New Features

    • Added clearer validation behavior for stacked changes, including warnings when required capabilities lack evidence and stricter rules for suppressing archived warnings.
    • Introduced a new planning workflow for promoting research notes into reviewable contract candidates, with evidence and ownership requirements.
  • Documentation

    • Added guidance for the new promotion workflow and updated the pull request template with summary, rationale, and validation sections.

@2233admin 2233admin requested a review from TabishB as a code owner July 1, 2026 17:16
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bcb44b56-8fd5-4864-88e1-27e326b3ac85

📥 Commits

Reviewing files that changed from the base of the PR and between acc4821 and 194de25.

📒 Files selected for processing (1)
  • .github/PULL_REQUEST_TEMPLATE.md
✅ Files skipped from review due to trivial changes (1)
  • .github/PULL_REQUEST_TEMPLATE.md

📝 Walkthrough

Walkthrough

This PR refines stack-aware validation docs around archived provider evidence and adds a new promote-pit-contract-candidate OpenSpec change with config, README, proposal, spec, and tasks.

Changes

Stack-Aware Validation Refinement

Layer / File(s) Summary
Archived provider evidence rule update
openspec/changes/add-change-stacking-awareness/proposal.md, openspec/changes/add-change-stacking-awareness/specs/change-stacking-workflow/spec.md, openspec/changes/add-change-stacking-awareness/tasks.md
Proposal, spec scenarios, and task 2.4 now require explicit change-level provider evidence to suppress unmatched requires marker warnings; otherwise a non-blocking warning is emitted.

PIT Contract Promotion Trial

Layer / File(s) Summary
Change config and README
openspec/changes/promote-pit-contract-candidate/.openspec.yaml, openspec/changes/promote-pit-contract-candidate/README.md
Adds the change metadata config and a README for the promotion trial.
Proposal and capability spec
openspec/changes/promote-pit-contract-candidate/proposal.md, openspec/changes/promote-pit-contract-candidate/specs/planning-contract-promotion/spec.md
Defines the planning-contract-promotion capability, its citation and ownership requirements, and the evidence-gating rules for promoted contract candidates.
Implementation task checklist
openspec/changes/promote-pit-contract-candidate/tasks.md
Adds the task checklist for extracting the source claim, defining the contract candidate, enforcing evidence gating, and validating the change.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: TabishB

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main addition: promoting a PIT contract candidate.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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