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): drop Opus from rule 16's cheaper-model list - #38
Follows hyperpolymath/rsr-template-repo#330 (merged). Rule 16 bars top-tier models from upkeep work, but its parenthetical listed Opus among the cheaper models. Owner ruling: drop Opus. The list now reads "(Sonnet and Haiku)".
Changes
.machine_readable/arrival-pack/arrival-pack.ncl: rule 16 wording, matching the template.
CLAUDE.md: regenerated. The generated region was not hand-edited.
No licence files touched.
RSR Quality Checklist
Required
Tests pass: not run. Only generated documentation text changed, and just validate-claude-md passes.
Code is formatted: not applicable. No source code changed.
Linter is clean: not applicable. No source code changed.
No banned language patterns
No unsafe blocks without // SAFETY: comments (none added)
No banned functions (none added)
SPDX license headers present on all new/modified source files: not applicable. No source files changed, and licence headers were not touched.
No secrets, credentials, or .env files included
As Applicable
Not applicable. This changes doctrine wording only: no project state, integrations, architecture, roadmap, dependencies or ABI/FFI change, and no CHANGELOG entry is needed.
Follow rsr-template-repo#330: rule 16 now lists only Sonnet and Haiku.
Regenerated CLAUDE.md with just claude-md; just validate-claude-md passes.
Co-Authored-By: Claude <noreply@anthropic.com>
No actionable comments were generated in the recent review.
Walkthrough
The model-selection guidance in Doctrine item 16 removes Opus from the parenthetical list. Sonnet and Haiku remain listed in both the arrival-pack and CLAUDE.md.
hyperpolymath/secret-types#37: Adds Doctrine item 16 to the same arrival-pack and CLAUDE.md guidance that this change edits.
Merge Risk: Minimal · up to 418fe
This is a small documentation wording change that keeps the guidance and its generated copy consistent. It poses no meaningful merge risk.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title names the main change: rule 16 in the arrival-pack doctrine no longer lists Opus among the cheaper models. It also names the affected area (arrival-pack). The title is specific and matches t…
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.
Description check
The description follows the repository template's main sections (Summary, Changes, RSR Quality Checklist, Testing, Screenshots) and clearly states the purpose: rule 16 listed Opus among the cheaper mo…
Full details: Title check
Explanation
The title names the main change: rule 16 in the arrival-pack doctrine no longer lists Opus among the cheaper models. It also names the affected area (arrival-pack). The title is specific and matches the raw summary and PR objectives. The 'docs(arrival-pack)' prefix follows a conventional-commit style and is accurate because the change edits doctrine wording.
Full details: Description check
Explanation
The description follows the repository template's main sections (Summary, Changes, RSR Quality Checklist, Testing, Screenshots) and clearly states the purpose: rule 16 listed Opus among the cheaper models, and the owner ruling drops it, leaving Sonnet and Haiku. It links the related upstream PR (hyperpolymath/rsr-template-repo#330) and lists the two changed files. Testing is described: just claude-md was run and just validate-claude-md passes. Checklist items are marked with reasons where not applicable. The main gaps are minor. The 'Tests pass' box is unchecked with a note that tests were not run, and the SPDX item is marked not applicable. The template's SPDX header comment is not mentioned, but no source files changed, so this is acceptable. The description is substantive and mostly complete.
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 ee71876 and 418fe68.
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.
infoMACH-007 — ECOSYSTEM.scm not found in .machine_readable/ directory.
infoMACH-007 — META.scm not found in .machine_readable/ directory.
infoMACH-007 — STATE.scm not found in .machine_readable/ directory.
infoMACH-005 — No robots.txt found. Consider adding one for crawler control.
infoVIS-005 — CLAUDE.md:9 — Line 177 characters exceeds max length of 80.
88 further finding(s) in files this change does not touch: reported, not graded (0 critical, 2 high).
rhodibot (RSR compliance): 13 finding(s)
13 further finding(s) in files this change does not touch: reported, not graded (0 critical, 12 high).
accessibilitybot: not_applicable — template default: enrol once the repository serves HTML
⚪ not applicable: template default: enrol once the repository serves HTML
cipherbot: not_applicable — template default: enrol once the repository implements cryptography
⚪ not applicable: template default: enrol once the repository implements cryptography
echidnabot: not_applicable — template default: enrol once the repository carries proofs and an ECHIDNA engine is configured
⚪ not applicable: template default: enrol once the repository carries proofs and an ECHIDNA engine is configured
panicbot: not_applicable — panic-attack is already graded by the verdict grid's own panic-attack row, from static-analysis-gate.yml evidence; enrolling panicbot would report one scanner twice and give two rows that can disagree
⚪ not applicable: panic-attack is already graded by the verdict grid's own panic-attack row, from static-analysis-gate.yml evidence; enrolling panicbot would report one scanner twice and give two rows that can disagree
seambot: not_applicable — template default: enrol once a seam register exists at spec/seams/seam-register.json
⚪ not applicable: template default: enrol once a seam register exists at spec/seams/seam-register.json
⚪ not applicable: reserved fleet slot; no adapter exists (gitbot-fleet bots/sustainabot/README.adoc)
Scope: static findings reported by each row's tool at the named commit. The table grades only findings in files this change touches, so the colour answers "does this change introduce anything?", not "what does the whole repository still carry?". Standing findings are counted in each accordion and in report.json; the baseline filter remains the place where acknowledged debt is recorded. The lane state is the worst GATING row; advisory rows are listed and counted but never set it. Purple, red, yellow and blue grade the worst GRADED finding present; orange is evidence missing, malformed or a scanner that did not run; grey is an explicit scope exclusion with its reason. No row is a security certification. report.json, badge.json and the raw findings are in the verdict-grid artifact on the linked run.
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
Follows hyperpolymath/rsr-template-repo#330 (merged). Rule 16 bars top-tier models from upkeep work, but its parenthetical listed Opus among the cheaper models. Owner ruling: drop Opus. The list now reads "(Sonnet and Haiku)".
Changes
.machine_readable/arrival-pack/arrival-pack.ncl: rule 16 wording, matching the template.CLAUDE.md: regenerated. The generated region was not hand-edited.RSR Quality Checklist
Required
just validate-claude-mdpasses.unsafeblocks without// SAFETY:comments (none added).envfiles includedAs Applicable
Not applicable. This changes doctrine wording only: no project state, integrations, architecture, roadmap, dependencies or ABI/FFI change, and no CHANGELOG entry is needed.
Testing
just claude-mdjust validate-claude-md(passes)Screenshots
Not applicable. One-line text change.
The owner merges. Not merged by the bot.
Generated by Claude Code
Summary by CodeRabbit
Generated by Claude Code