feat(cursor): maintain-herdr fork ops project skill - #110
Conversation
Playbook for PR autopilot, rebase/conflict resolution, draft triage, quality-gate handling and fork hygiene on OnlineChefGroep/herdr. refs #87
|
ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing |
|
Warning Review limit reached
Next review available in: 47 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
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. Comment |
| Always work from a full clone (`git clone https://github.com/OnlineChefGroep/herdr.git`). | ||
| Shallow/partial clones break `git checkout <pr-branch>` and stale rebase state | ||
| (`.git/rebase-merge` leftover) must be aborted before new work: | ||
|
|
||
| ```bash | ||
| git rebase --abort 2>/dev/null || true # clear stale rebase first | ||
| git checkout main && git reset --hard fork/main | ||
| git fetch fork <pr-branch> |
There was a problem hiding this comment.
Fresh clone lacks required remote
The prescribed git clone https://github.com/OnlineChefGroep/herdr.git creates only the origin remote. The next command relies on fork/main, but no fork remote or tracking ref has been configured, so git reset --hard fork/main stops the rebase procedure with an unknown-revision error. Document setup and fetching of fork before its first use, or use the remote established by the clone consistently.
Artifacts
Executable local Git reproduction script
- This script creates local bare Git repositories and runs the documented reset before and after configuring `fork`, proving the behavior without network access—the defect is reproducible.
- This captured execution lists only `origin` and shows `git reset --hard fork/main` failing with exit code 128 and an unknown-revision error—the documented fresh-clone procedure fails.
Configured fork reset success log
- This captured execution adds and fetches `fork`, then shows the same reset completing successfully—the missing fork setup is the required correction.
Ran code and verified through T-Rex
Prompt To Fix With AI
This is a comment left during a code review.
Path: .cursor/skills/maintain-herdr/SKILL.md
Line: 33-40
Comment:
**Fresh clone lacks required remote**
The prescribed `git clone https://github.com/OnlineChefGroep/herdr.git` creates only the `origin` remote. The next command relies on `fork/main`, but no `fork` remote or tracking ref has been configured, so `git reset --hard fork/main` stops the rebase procedure with an unknown-revision error. Document setup and fetching of `fork` before its first use, or use the remote established by the clone consistently.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.refs #87 Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Fork maintenance playbook as a project skill (repo .cursor/skills convention, indexes regenerated):
refs #87
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is enabled.Greptile Summary
This PR adds a Cursor skill for maintaining the Herdr fork and registers it in the Markdown and YAML skill indexes. The fresh-clone rebase instructions have a reproduced failure: a normal clone creates only
origin, but the playbook immediately resets tofork/mainbefore configuring or fetchingfork, so maintainers cannot start the documented rebase flow.Confidence Score: 3/5
Not safe to merge until T-Rex findings are addressed.
An isolated Git reproduction confirmed that the documented command sequence fails with only the remote created by a conventional clone, while adding and fetching
forkmakes the same reset succeed.T-Rex reproduced 2 failing behaviors at runtime in .cursor/skills/maintain-herdr/SKILL.md; the change needs fixes before it is safe to merge.
Files Needing Attention: .cursor/skills/maintain-herdr/SKILL.md
What T-Rex did
Comments Outside Diff (1)
General comment
fork/main, but a conventional clone creates onlyorigin, so the procedure stops before it can rebase.forkremote before its first use.forkremote configuration and a fetch that createsfork/mainbefore line 39, or consistently use the intended configured remote.Prompt To Fix All With AI
Reviews (1): Last reviewed commit: "feat(cursor): add maintain-herdr fork op..." | Re-trigger Greptile