Skip to content

fix(pre-open): explain unknown review-scope decisions - #464

Merged
Wibias merged 3 commits into
mainfrom
fix/preopen-unknown-next-action
Sep 20, 2026
Merged

Wibias merged 3 commits into
mainfrom
fix/preopen-unknown-next-action

Conversation

@Wibias

@Wibias Wibias commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Summary

Make compact pre-open output explain why a decision is unknown instead of mapping every incomplete review scope to restore_branch_evidence.

Trace evidence

The newest Grok trace inherited an old large-diff artifact with decision: unknown. The compact output said restore_branch_evidence, which sent the agent down a branch/head investigation even though the actual cause was review-scope incompleteness.

Fix

  • expose blocking review-scope uncertainty as incompleteReasons
  • map missing patch evidence to inspect_missing_patch_evidence
  • map invalid probe-registry state to repair_probe_registry
  • use resolve_incomplete_review_scope for other incomplete review-scope causes
  • retain existing complete_evidence and proceed_to_publication dispositions

This keeps the compact result deterministic while removing a misleading recovery instruction.

No merge or release publication is authorized by this PR.

@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 880b69ba-e199-44dd-8d44-9f4cea0ec337


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.

@Wibias
Wibias marked this pull request as ready for review September 20, 2026 22:50
@Wibias
Wibias merged commit c35e22b into main Sep 20, 2026
13 checks passed
@Wibias
Wibias deleted the fix/preopen-unknown-next-action branch September 20, 2026 23:03
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