Skip to content

Reduce first-run friction: init, index honesty, PR-shaped review - #3

Merged
cursor[bot] merged 2 commits into
mainfrom
cursor/friction-review-ccb4
Aug 14, 2026
Merged

Reduce first-run friction: init, index honesty, PR-shaped review#3
cursor[bot] merged 2 commits into
mainfrom
cursor/friction-review-ccb4

Conversation

@Modsofthenation

@Modsofthenation Modsofthenation commented Aug 14, 2026

Copy link
Copy Markdown
Owner

Why

Using Loadpath still asked people to write loadpath.yml, type git SHAs, guess whether the index was current, and leave the brief on their laptop. This PR cuts that loop without turning the tool into a hunk bot.

What changed

  • loadpath init (and Index) detect Django/React roots and draft loadpath.yml in the repo being indexed. Existing manifests are never overwritten; a parent loadpath.yml cannot suppress a child draft.
  • Incremental index skips extract when source, loadpath.yml, and OpenAPI sidecars are unchanged. Stale/boot status is first-class on architecture, review, CLI, and the rail.
  • Review uses a three-dot / merge-base range by default (CLI --base now matches the UI: HEAD~1). Dirty files that overlap the range are called out. “Review this PR” prefers SHAs when the API provides them.
  • Copy the markdown brief; post one PR comment and update it in place (<!-- loadpath-review -->). Empty bodies are rejected; GitHub comment listing is paged so a busy PR does not grow a second Loadpath comment.
  • Empty Review tab explains the product: graph is architecture, brief is the force of this diff.

Adversarial review

Valid findings addressed in 5370b62:

  • Drafting used find_config(), which walks parents — a parent manifest blocked a child loadpath.yml.
  • Status banners never cleared.
  • PR review filled branch names only, ignoring head_sha / base_sha.
  • A second Loadpath comment could be posted if the first sat past page 1.

Tests

  • Detect/init (no overwrite, child draft despite parent yml)
  • Hash-skip reindex
  • Three-dot range + dirty tree
  • GitHub comment upsert (update-in-place)
  • Existing vertical slice, CLI, API e2e
Open in Web Open in Cursor 

Detect Django/React roots and draft loadpath.yml without overwriting.
Skip extract when file and OpenAPI hashes match, surface index/boot
status, review three-dot ranges, warn on a dirty tree, and copy or
upsert a single PR comment for the markdown brief.

Co-authored-by: Damon  <Modsofthenation@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

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: fab50450-70c1-4824-bb16-63edd5626b0b

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

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

Draft loadpath.yml only in the indexed repo root so a parent manifest
cannot suppress it. Prefer PR SHAs for the three-dot range, reject empty
comment bodies, page GitHub comments before inserting a second brief,
and clear stale UI status banners.

Co-authored-by: Damon  <Modsofthenation@users.noreply.github.com>
@Modsofthenation
Modsofthenation marked this pull request as ready for review August 14, 2026 23:20
@cursor
cursor Bot merged commit c5d903b into main Aug 14, 2026
2 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.

2 participants