Skip to content

docs(arrival-pack): drop Opus from rule 16's cheaper-model list - #8

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/rule-16-drop-opus
Nov 12, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
docs/rule-16-drop-opus

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner

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.
  • 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.

Testing

  • just claude-md
  • just 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

  • Documentation
    • Updated model-selection guidance for discussions involving substantial upkeep work. Sonnet and Haiku are now the models listed as lower-cost options; Opus is no longer included in that list.
    • The guidance is consistent across the relevant project documentation.

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>
@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The upkeep doctrine’s cheaper-model list now names Sonnet and Haiku, removing Opus from both references.

Changes: Upkeep doctrine guidance

Layer / File(s) Summary
Update the cheaper-model list
.machine_readable/arrival-pack/arrival-pack.ncl, CLAUDE.md
Both doctrine references now list Sonnet and Haiku instead of Opus, Sonnet, and Haiku.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: {{owner}}

Merge Risk: Minimal · up to 9ab41

This is a documentation-only wording change that removes Opus from the cheaper-model list. It has no runtime impact. The failing checks are unrelated to this change.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title names the change: rule 16 of the arrival-pack doctrine no longer lists Opus among the cheaper models. It matches the raw summary and the objectives, and it names the scope (arrival-pack) and…
Description check Passed The description follows the repository template. It has a Summary that explains the change and why, a Changes list, and the RSR Quality Checklist with each item marked or annotated as not applicable. …
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title names the change: rule 16 of the arrival-pack doctrine no longer lists Opus among the cheaper models. It matches the raw summary and the objectives, and it names the scope (arrival-pack) and the type (docs). It is specific and concise.

Full details: Description check

Explanation

The description follows the repository template. It has a Summary that explains the change and why, a Changes list, and the RSR Quality Checklist with each item marked or annotated as not applicable. It includes a Testing section that lists the commands run. Some Required items are unchecked, but the author explains each one: tests were not run because only generated documentation changed, and the source-code checks do not apply. The Screenshots section is filled in. The description leaves no required section missing.

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.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between b3af3d4 and 9ab416b.

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.

@sonarqubecloud

Copy link
Copy Markdown

hyperpolymath commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner Author

Three checks fail here. None of them is caused by this PR, which changes only the rule 16 line in arrival-pack.ncl and the regenerated CLAUDE.md.

  • openssf-compliance: fails with "89 file(s) contain unfilled {{PLACEHOLDER}} tokens". Those files are already in the repo.
  • Hypatia neurosymbolic scan: the scanner itself crashes with exit 139 (a segfault) before it reports any findings.
  • Deposit findings for gitbot-fleet: fails with "Artifact not found for name: hypatia-findings". It follows from the Hypatia crash above, which produces no artifact.

All three failed the same way on #6 (the earlier rule 16 PR) and on #7. Both PRs were merged. None of them runs on pushes to main, so there is no base run to compare with. This PR has no fix for any of them. The placeholders need a template fill pass, and the Hypatia crash belongs upstream in the scanner.


Generated by Claude Code

@hyperpolymath
hyperpolymath marked this pull request as ready for review November 12, 2026 04:11
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-12T04:11:14.800357Z
We couldn’t complete this review. Comment @codex review to try again.
9ab416b Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@hyperpolymath
hyperpolymath merged commit 8b74040 into main Nov 12, 2026
18 of 21 checks passed
@hyperpolymath
hyperpolymath deleted the docs/rule-16-drop-opus branch November 12, 2026 04:45
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.

3 participants