Skip to content

docs: add OpenAI model tier mapping alongside Anthropic tiers#5

Merged
cristim merged 3 commits into
mainfrom
docs/openai-model-mapping
Apr 30, 2026
Merged

docs: add OpenAI model tier mapping alongside Anthropic tiers#5
cristim merged 3 commits into
mainfrom
docs/openai-model-mapping

Conversation

@cristim
Copy link
Copy Markdown
Member

@cristim cristim commented Apr 30, 2026

Summary

CLAUDE.md may also be loaded by OpenAI tooling. Adds an explicit Anthropic→OpenAI tier mapping near the Core Tenets so an OpenAI-driven session resolves Anthropic tier names (Haiku / Sonnet / Opus) to the equivalent OpenAI tiers without having to guess:

Anthropic tier OpenAI tier
Haiku GPT-5.4-mini
Sonnet GPT-5.4
Opus GPT-5.5

The mapping note also calls out that the cheapest/mid/top-tier alignment should be kept in sync if local OpenAI model names change.

§2 "Subagent Strategy" additionally gets three parallel OpenAI mini-tier / mid-tier / top-tier bullets in the model rubric so an OpenAI-side reader sees the same task examples directly, and the existing "Default: delegate to the cheapest sufficient tier" + "When in doubt, go one tier cheaper" rules are generalised to cover both model families instead of naming Haiku/Sonnet specifically.

Test plan

  • Read CLAUDE.md lines 14–22 standalone — the OpenAI mapping is unambiguous and locatable from the Core Tenets
  • Read CLAUDE.md §2 "Subagent Strategy" — the three new OpenAI bullets sit alongside the Haiku/Sonnet/Opus ones, and the "When in doubt" + "Default: delegate down" rules read consistently for both families
  • git diff --stat origin/main shows exactly one file changed (CLAUDE.md), 12 insertions / 2 deletions

Summary by CodeRabbit

  • Documentation
    • Updated delegation guidance: added explicit Anthropic↔OpenAI tier mappings, adjusted delegate-down behavior to consider cheaper Claude or OpenAI subagents, expanded the tier-to-task complexity rubric with OpenAI mini/mid/top categories, and generalized the “when in doubt” cheaper-tier fallback across model families.

CLAUDE.md may also be loaded by OpenAI tooling. Adds a mapping note
near the Core Tenets so an OpenAI-driven session reads the same
rules and resolves Anthropic tier names to the equivalent OpenAI
tiers (Haiku → GPT-5.4-mini, Sonnet → GPT-5.4, Opus → GPT-5.5).

Also extends the §2 model-rubric block with three parallel
"OpenAI mini-tier / mid-tier / top-tier" bullets so an OpenAI-side
reader sees the same task examples without having to mentally map
the Anthropic ones, and generalises the "Default: delegate to the
cheapest sufficient tier" + "When in doubt, go one tier cheaper"
rules to cover both model families.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f6467c3b-0db5-440f-b6fa-15410dbc9430

📥 Commits

Reviewing files that changed from the base of the PR and between 332c830 and 44e4314.

📒 Files selected for processing (1)
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Updates to CLAUDE.md: adds explicit OpenAI↔Anthropic tier mappings, modifies delegation rules to allow cheaper OpenAI subagents as fallback, expands the tier-to-task-complexity rubric with OpenAI mini/mid/top categories, and generalizes the cheaper-tier fallback across both model families.

Changes

Cohort / File(s) Summary
Documentation Update
CLAUDE.md
Added OpenAI-to-Anthropic tier mappings; changed delegation rule to consider cheaper OpenAI subagents when spawning lower-tier agents; extended tier-to-task-complexity rubric with OpenAI mini/mid/top tiers; generalized "when in doubt" cheaper-tier fallback to include both Claude and OpenAI.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 I hopped through lines both neat and spry,

Mapped models so cheaper ones might try,
Claude and OpenAI now share the trail,
A rubric updated, tidy and frail.
🥕✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarizes the main change: adding OpenAI model tier mapping alongside existing Anthropic tiers in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/openai-model-mapping

Review rate limit: 2/5 reviews remaining, refill in 33 minutes and 1 second.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CLAUDE.md`:
- Around line 212-220: The "Haiku (default for most delegations)" phrasing can
be misread across model families; update the Haiku bullet and the "OpenAI
mini-tier" bullet so "default" is explicitly scoped per family or removed—e.g.,
change "Haiku (default for most delegations)" to "Haiku (default for Claude
family delegations)" or drop "default" and rely on the "cheapest sufficient
tier" rule, and mirror that clarification in the "OpenAI mini-tier (same role as
Haiku)" line so the default tier language is unambiguous between Claude and
OpenAI.
- Around line 17-23: Update the Anthropic→OpenAI mapping entries so the OpenAI
model identifiers use the lowercase "gpt-" prefix: change "GPT-5.4-mini" to
"gpt-5.4-mini", "GPT-5.4" to "gpt-5.4", and "GPT-5.5" to "gpt-5.5" in the
mapping block (the Haiku, Sonnet, and Opus lines in CLAUDE.md).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 25136b9e-ed83-44e6-87ea-31cd07a686c2

📥 Commits

Reviewing files that changed from the base of the PR and between 5d64f48 and de9fb09.

📒 Files selected for processing (1)
  • CLAUDE.md

Comment thread CLAUDE.md
Comment thread CLAUDE.md
@cristim
Copy link
Copy Markdown
Member Author

cristim commented Apr 30, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
CLAUDE.md (1)

217-220: 💤 Low value

(Optional) Clean up the LanguageTool “repeated word” warning near the OpenAI top-tier bullet.

Static analysis reported three successive sentences beginning with the same word around the OpenAI rubric area (near Line 219). This won’t affect behavior, but if you’re keeping CLAUDE.md lint-clean, consider a small reword to remove the repeated sentence-start pattern.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@CLAUDE.md` around lines 217 - 220, Reword the three successive sentences that
begin the OpenAI rubric bullets to avoid repeating the same sentence-start
pattern near the "OpenAI top-tier" bullet: edit the bullet text for "OpenAI
mini-tier", "OpenAI mid-tier", and/or "OpenAI top-tier" in CLAUDE.md so the
sentences no longer all start with the same word (e.g., change one bullet's
leading clause to "Includes" or "Handles" or rephrase the OpenAI top-tier
sentence), preserving original meaning and examples while keeping the bullets
consistent and lint-clean.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@CLAUDE.md`:
- Around line 217-220: Reword the three successive sentences that begin the
OpenAI rubric bullets to avoid repeating the same sentence-start pattern near
the "OpenAI top-tier" bullet: edit the bullet text for "OpenAI mini-tier",
"OpenAI mid-tier", and/or "OpenAI top-tier" in CLAUDE.md so the sentences no
longer all start with the same word (e.g., change one bullet's leading clause to
"Includes" or "Handles" or rephrase the OpenAI top-tier sentence), preserving
original meaning and examples while keeping the bullets consistent and
lint-clean.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4d99ba43-71e2-42c3-b14c-bca7165b90c7

📥 Commits

Reviewing files that changed from the base of the PR and between de9fb09 and 332c830.

📒 Files selected for processing (1)
  • CLAUDE.md

@cristim cristim added triaged Item has been triaged type/docs Documentation severity/low Minor harm urgency/eventually No deadline impact/internal Team-internal only effort/xs Trivial / one-liner priority/p3 Polish / idea / may never ship labels Apr 30, 2026
@cristim
Copy link
Copy Markdown
Member Author

cristim commented Apr 30, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@cristim cristim merged commit cb63945 into main Apr 30, 2026
1 check passed
@cristim cristim deleted the docs/openai-model-mapping branch May 4, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/xs Trivial / one-liner impact/internal Team-internal only priority/p3 Polish / idea / may never ship severity/low Minor harm triaged Item has been triaged type/docs Documentation urgency/eventually No deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant