Reduce first-run friction: init, index honesty, PR-shaped review - #3
Merged
Conversation
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>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
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
marked this pull request as ready for review
August 14, 2026 23:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 draftloadpath.ymlin the repo being indexed. Existing manifests are never overwritten; a parentloadpath.ymlcannot suppress a child draft.loadpath.yml, and OpenAPI sidecars are unchanged. Stale/boot status is first-class on architecture, review, CLI, and the rail.--basenow matches the UI:HEAD~1). Dirty files that overlap the range are called out. “Review this PR” prefers SHAs when the API provides them.<!-- loadpath-review -->). Empty bodies are rejected; GitHub comment listing is paged so a busy PR does not grow a second Loadpath comment.Adversarial review
Valid findings addressed in
5370b62:find_config(), which walks parents — a parent manifest blocked a childloadpath.yml.head_sha/base_sha.Tests