Skip to content

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

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

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.

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

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.

Summary

Summary by CodeRabbit

  • Documentation
    • Updated the guidance for substantial upkeep work: it now names Sonnet and Haiku, rather than Opus, when discussing cheaper models. This change is reflected consistently across the project’s guidance materials.

The upkeep doctrine in the machine-readable arrival pack and CLAUDE.md now names Sonnet and Haiku, and no longer names Opus.

Changes: Upkeep doctrine wording

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

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

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 171af

This is a documentation wording change that removes Opus from the cheaper-model list in rule 16. The generated file matches its source, and the same wording was merged upstream. No behavioral risk is identified.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title names the main change: rule 16's cheaper-model list in the arrival-pack document drops Opus. It matches the raw summary and the PR objectives, and it follows the conventional commit format. …
Description check Passed The description gives a clear summary and a list of changes. It explains the reason (the rule 16 parenthetical wrongly listed Opus) and names the owner ruling and the linked upstream PR. It gives test…
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 main change: rule 16's cheaper-model list in the arrival-pack document drops Opus. It matches the raw summary and the PR objectives, and it follows the conventional commit format. It is specific and concise.

Full details: Description check

Explanation

The description gives a clear summary and a list of changes. It explains the reason (the rule 16 parenthetical wrongly listed Opus) and names the owner ruling and the linked upstream PR. It gives testing commands and marks the screenshots section as not applicable. The template's RSR Quality Checklist is absent. The author did not tick any checklist items or state whether the Required items (tests, formatting, linter, SPDX headers, banned patterns, secrets) apply. The Journal, ECOSYSTEM, and CHANGELOG items are also not addressed. These gaps are non-critical for a one-line documentation change, so the description is mostly complete.

  • 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

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: dacc3025-6a8d-4c0d-a44d-e5e8ec7756b7
Commits

Reviewing files that changed from the base of the PR and between c4569db and 171af87.

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.

Recent review details

Context from checks skipped due to timeout: 23

  • GitHub Check: CodeRabbit
  • GitHub Check: scan / gitleaks
  • GitHub Check: estate-rules
  • GitHub Check: Hypatia Neurosymbolic Analysis
  • GitHub Check: Runtime Policy
  • GitHub Check: Empty-linter (invisible characters)
  • GitHub Check: Groove manifest check
  • GitHub Check: Estate Governance Policy
  • GitHub Check: Validate K9 contracts
  • GitHub Check: docs
  • GitHub Check: analyze (actions, none)
  • GitHub Check: Validate DEED manifests
  • GitHub Check: Rust standalone tools and workspace gate
  • GitHub Check: Hypatia neurosymbolic scan
  • GitHub Check: Patch Bridge CVE triage
  • GitHub Check: SonarQube
  • GitHub Check: Validate eclexiaiser manifest
  • GitHub Check: lint
  • GitHub Check: openssf-compliance
  • GitHub Check: check
  • GitHub Check: panic-attack assail
  • GitHub Check: check
  • GitHub Check: semgrep-cloud-platform/scan
Additional context used

Code guidelines: 1

.github/copilot-instructions.md — auto-discovered
Path-based instructions: 1

Source excerpt: Never use AGPL-3.0.

CodeRabbit inference engine (.github/copilot-instructions.md)

Files:

  • .machine_readable/arrival-pack/arrival-pack.ncl
  • CLAUDE.md
Learnings: 1

Common learnings

Learnt from: hyperpolymath
Repo: hyperpolymath/anytype PR: 91
File: .machine_readable/arrival-pack/arrival-pack.ncl:51-51
Timestamp: 2026-11-12T01:53:04.264Z
Learning: Within repository "hyperpolymath/anytype", only for files matching one of [".machine_readable/arrival-pack/arrival-pack.ncl"]: When reviewing a proposed wording change to the estate-common doctrine in `.machine_readable/arrival-pack/arrival-pack.ncl`, check it against the upstream template. If a wording correction is needed, recommend changing the template first and then syncing this repository, rather than introducing repo-only wording that creates drift.
Additional comments: 2

CLAUDE.md: 1

31-31: LGTM!

.machine_readable/arrival-pack/arrival-pack.ncl: 1

51-51: Data Integrity & Integration

The concern is refuted. The upstream template no longer names Opus in rule 16. Merged PR #330 changes the wording to "Sonnet and Haiku", the same wording this PR adds to .machine_readable/arrival-pack/arrival-pack.ncl and CLAUDE.md. Nothing remains to correct upstream or to sync.

@sonarqubecloud

Copy link
Copy Markdown

@linear-code
linear-code Bot marked this pull request as ready for review November 12, 2026 02:57
@hyperpolymath
hyperpolymath merged commit 20856da into main Nov 12, 2026
32 checks passed
@hyperpolymath
hyperpolymath deleted the docs/rule-16-drop-opus branch November 12, 2026 04:07
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