Skip to content

test(conformance): align mutation fixture with runtime provenance - #1099

Closed
seonghobae wants to merge 1 commit into
feat/conformance-runtime-provenance-1094from
fix/conformance-mutation-fixture-provenance-1095
Closed

seonghobae wants to merge 1 commit into
feat/conformance-runtime-provenance-1094from
fix/conformance-mutation-fixture-provenance-1095

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Fixes the inherited cross-engine mutation fixture so its executed evidence includes the required run provenance introduced by #1095.

This is a compatibility-only test correction; production validation and statistical arithmetic are unchanged. It keeps the hostile-container assertion focused on replay admission instead of failing during fixture construction.

Stacks on Draft #1095 and unblocks child Draft #1097.

Copy link
Copy Markdown
Contributor Author

Current exact HEAD is ef3cb445a1e923fc7a3753b06adf523f0ef32f05, based on #1095 c233b236f6dd6be48cc8ab7bd5c8f7fc4c9c247c.

Validation:

  • 71 cross-engine/provenance tests passed
  • Ruff passed
  • Interrogate passed at 100%
  • git diff --check passed

The fixture now supplies raw and normalized output provenance required for its executed evidence, so the hostile-container assertion reaches the intended replay boundary.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5157d4c2-2d55-4280-b89e-01900eb8379e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@seonghobae
seonghobae marked this pull request as ready for review August 20, 2026 22:01
@seonghobae
seonghobae enabled auto-merge (squash) August 20, 2026 22:01

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Copy link
Copy Markdown
Contributor Author

Please re-review this exact current head only: ef3cb445a1e923fc7a3753b06adf523f0ef32f05, based on feat/conformance-runtime-provenance-1094@c233b236f6dd6be48cc8ab7bd5c8f7fc4c9c247c. Prior review records are predecessor-head evidence and must not transfer. Review-only dispatch requested; do not update the branch or merge via bypass.

Copy link
Copy Markdown
Contributor Author

@opencode-agent Please perform a review-only formal review of exact current PR head ef3cb445a1e923fc7a3753b06adf523f0ef32f05. Re-check changed-file scope, current-head findings, unresolved threads, mergeability, and every required Check. Do not reuse a stale review, mutate the branch, self-approve, or merge.

@opencode-agent
opencode-agent Bot disabled auto-merge August 21, 2026 00:18
@seonghobae

Copy link
Copy Markdown
Contributor Author

Current-head review record for ef3cb445a1e923fc7a3753b06adf523f0ef32f05: this stacked test-only change supplies the immutable ConformanceRunProvenance required by an executed mutation fixture, preserving the runtime contract instead of weakening validation. CodeGraph impact tracing shows no production change. Conformance provenance/replay suites pass (31 passed), Ruff, compileall, and git diff --check pass. The PR remains correctly stacked on feat/conformance-runtime-provenance-1094; review this exact HEAD only and merge through the normal stack after its base is accepted, without bypass.

@seonghobae
seonghobae enabled auto-merge (squash) August 21, 2026 01:44
@seonghobae

Copy link
Copy Markdown
Contributor Author

The parent stack now contains the exact fixture correction from this PR at parent HEAD 5200c0a6. Verified locally that current parent branch and this PR head have no tree diff, so this child has no remaining product delta to merge. The parent conformance suites pass (71 passed). Closing this now-empty child preserves the fix in #1095 and avoids a duplicate merge path; no bypass or force push used.

@seonghobae seonghobae closed this Aug 21, 2026
auto-merge was automatically disabled August 21, 2026 01:49

Pull request was closed

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