Skip to content

feat: add P1 PR consolidation analysis - #285

Merged
Wibias merged 1 commit into
feat/p1-work-item-deliveryfrom
feat/p1-pr-consolidation
Aug 18, 2026
Merged

feat: add P1 PR consolidation analysis#285
Wibias merged 1 commit into
feat/p1-work-item-deliveryfrom
feat/p1-pr-consolidation

Conversation

@Wibias

@Wibias Wibias commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • add deterministic clustering for potentially competing PR implementations
  • treat a shared durable work-item identity as related-work evidence, not automatic replacement proof
  • promote same-work-item pairs to high confidence only when they also have substantial non-noise implementation overlap
  • allow substantial non-noise changed-file overlap without a shared work item as medium-confidence evidence
  • deliberately leave canonicalPr unset during analysis
  • keep analysis read-only and reuse the existing supersede-pr workflow for any later closing/comment mutations
  • expose competing-PR analysis in SKILL.md discovery metadata

Hardening found during self-review

  • a selected canonical PR must have direct supersede-grade implementation-overlap evidence to every PR it would supersede
  • transitive cluster chains such as A-B-C cannot let A supersede C without direct A-C evidence
  • a shared tracker/work-item key alone can form a candidate cluster but cannot produce a supersede plan, because one ticket may intentionally use complementary PRs
  • child routing keeps feat: add P1 work-item delivery orchestration #284's work-item publication and tracker-status fixes instead of reverting them during stack updates

Stack

Validation

  • focused contracts cover related-work clusters, supersede-grade overlap, file-overlap thresholds, cross-base isolation, noise-only overlap, canonical selection, transitive-cluster rejection, same-key-without-overlap rejection, manufactured-cluster rejection, routing, mode, and controller phases
  • repository-wide local suite is intentionally left to the user's local run; GitHub Actions is not used as the current gate

Provenance

The clustering direction was informed by OutThisLife/brooklyn-skills pr-triage and redesigned to compose with GitHub Delivery's existing supersede machinery.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 7f87dfd4-17c4-4e26-836d-ac14e10de1e8

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.

@Wibias
Wibias force-pushed the feat/p1-pr-consolidation branch from 854e4e7 to 7ed139e Compare August 18, 2026 05:51
@Wibias
Wibias marked this pull request as ready for review August 18, 2026 07:46
@Wibias
Wibias force-pushed the feat/p1-pr-consolidation branch from 7ed139e to d05eaca Compare August 18, 2026 07:57
@Wibias
Wibias force-pushed the feat/p1-pr-consolidation branch from d05eaca to 8cb3d70 Compare August 18, 2026 08:03
@Wibias
Wibias merged commit 165c5a7 into main Aug 18, 2026
10 checks passed
@Wibias
Wibias deleted the feat/p1-pr-consolidation branch August 18, 2026 08:12
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