You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
docs(arrival-pack): sync rules to 16 and regenerate drifted CLAUDE.md - #3
Drift sync of the generated ARRIVAL-PACK region in CLAUDE.md. The ncl and the generated region differed only by the missing rule 16.
Added rule 16 to the doctrine list in the ncl, taken verbatim from rsr-template-repo.
The generated ARRIVAL-PACK region in CLAUDE.md had drifted from the ncl. The ncl and the generated region differed only by the missing rule 16.
Added rule 16 from rsr-template-repo to the ncl doctrine list, then regenerated.
just claude-md and just validate-claude-md both pass.
Co-Authored-By: Claude <noreply@anthropic.com>
No actionable comments were generated in the recent review.
Walkthrough
Doctrine rule 16 adds model-selection guidance for CI/CD, workflow, template-drift, and status-file work. Both CLAUDE.md and the generated arrival-pack doctrine include the rule.
Both doctrine sources direct upkeep work to use cheaper models. If the work needs substantial effort, the rule says to stop and open a thread about using Sonnet or Haiku. It reserves top-tier models for proof work and hard design.
This adds a model-selection guidance rule to documentation and generated text only. It does not change runtime behavior. The CI failures shown appear unrelated to these changes.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title names the two main parts of the change: the arrival-pack rules are synced to 16 and the drifted CLAUDE.md is regenerated. It is specific and matches the summary, objectives, and description.…
Description check
The description follows the repository template. It has Summary, Changes, the RSR Quality Checklist, and Testing sections. It explains the drift cause and the scope of the change, and it lists the com…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Title check
Explanation
The title names the two main parts of the change: the arrival-pack rules are synced to 16 and the drifted CLAUDE.md is regenerated. It is specific and matches the summary, objectives, and description. It follows the conventional docs(scope): format and is slightly long at 69 characters.
Full details: Description check
Explanation
The description follows the repository template. It has Summary, Changes, the RSR Quality Checklist, and Testing sections. It explains the drift cause and the scope of the change, and it lists the commands run. The Screenshots section is omitted, which is acceptable because it applies only when relevant. The Required checklist leaves the test, formatter, and linter items unchecked with a reason. The Testing section names commands but gives no output. The Summary does not link an issue, and the template does not require one.
Finishing Touches
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Reviewing files that changed from the base of the PR and between 87c0d87 and 91f79d7.
Files selected for processing: 2
.machine_readable/arrival-pack/arrival-pack.ncl
CLAUDE.md
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
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
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.
Summary
Drift sync of the generated ARRIVAL-PACK region in CLAUDE.md. The ncl and the generated region differed only by the missing rule 16.
Added rule 16 to the doctrine list in the ncl, taken verbatim from rsr-template-repo.
Changes
RSR Quality Checklist
Required
As Applicable
Testing
Commands run:
just claude-mdjust validate-claude-mdBoth pass. The owner merges. This is a draft.
Generated by Claude Code
Summary by CodeRabbit