Skip to content

fix(misc): remove duplicate docs page, update clopus 4.7#4200

Merged
waleedlatif1 merged 4 commits intostagingfrom
fix/misc
Apr 16, 2026
Merged

fix(misc): remove duplicate docs page, update clopus 4.7#4200
waleedlatif1 merged 4 commits intostagingfrom
fix/misc

Conversation

@waleedlatif1
Copy link
Copy Markdown
Collaborator

Summary

  • remove duplicate docs page, update clopus 4.7

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 16, 2026 7:35pm

Request Review

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 16, 2026

PR Summary

Medium Risk
Changes LLM request payload composition and model capability metadata, which can alter Claude Opus 4.7 behavior (temperature/thinking options) at runtime. Docs-only changes are low risk.

Overview
Removes the duplicate custom-tools docs page from the top-level docs nav and consolidates content under tools/custom-tools.mdx, adding a Custom Tools vs MCP Tools comparison and updating the docs to state custom tool deletion requires Admin.

Updates Anthropic request construction to only send temperature when the selected model supports it, and revises the claude-opus-4-7 model definition by removing temperature capability and adding the xhigh thinking level.

Reviewed by Cursor Bugbot for commit 7ab2e69. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 16, 2026

Greptile Summary

This PR does two things: removes a duplicate custom-tools docs page (the content already lives at tools/custom-tools.mdx and is registered in tools/meta.json) and adds claude-opus-4-7 to the model registry with adaptive-thinking-only support and a new xhigh effort level.

  • claude-opus-4-7 is correctly wired through supportsAdaptiveThinking() in core.ts, which already checks for opus-4-7 strings, so adaptive thinking flows end-to-end.
  • The new xhigh effort level is unique to Opus 4.7 among Anthropic models (Opus 4.6 / Sonnet 4.6 only go up to max). If xhigh is not a recognized value in Anthropic's output_config.effort API, any user who selects it will receive an API error.

Confidence Score: 4/5

Safe to merge after confirming 'xhigh' is a valid Anthropic API effort value for Opus 4.7.

All changes are straightforward — docs deduplication is clean and the model wiring through supportsAdaptiveThinking is correct. The only open question is whether 'xhigh' is a valid Anthropic API value for output_config.effort; if it is not, users selecting that thinking level will hit a hard API error in production.

apps/sim/providers/models.ts — verify the 'xhigh' thinking level is a recognized Anthropic API effort value.

Important Files Changed

Filename Overview
apps/sim/providers/models.ts Adds claude-opus-4-7 with contextWindow 1M, maxOutputTokens 128k, nativeStructuredOutputs, and thinking levels ['low','medium','high','xhigh','max'] — the 'xhigh' level is new and not present in any other Anthropic model's thinking levels.
apps/sim/providers/anthropic/core.ts supportsAdaptiveThinking() already includes 'opus-4-7' check; buildThinkingConfig correctly routes Opus 4.7 to the adaptive path; no logic changes required beyond the models.ts addition.
apps/docs/content/docs/en/meta.json Removes the top-level 'custom-tools' page reference; content remains accessible via the 'tools' section entry which already lists custom-tools in its own meta.json.
apps/docs/content/docs/en/tools/custom-tools.mdx Authoritative custom tools documentation page; unchanged, confirming the deleted custom-tools/index.mdx was indeed a duplicate.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Request with thinkingLevel] --> B{buildThinkingConfig}
    B --> C{supportsAdaptiveThinking?}
    C -- "opus-4-7 ✓" --> D["thinking: { type: 'adaptive' }\noutputConfig: { effort: thinkingLevel }"]
    C -- "No (e.g. opus-4-5)" --> E["thinking: { type: 'enabled', budget_tokens: N }"]
    D --> F["Anthropic API: output_config.effort = 'low'|'medium'|'high'|'xhigh'|'max'"]
    E --> G["Anthropic API: budget_tokens = 2048/8192/32768"]
    F --> H{xhigh valid API value?}
    H -- Yes --> I[Success]
    H -- No --> J[API Error returned to user]
Loading

Reviews (1): Last reviewed commit: "fix(misc): remove duplicate docs page, u..." | Re-trigger Greptile

Comment thread apps/sim/providers/models.ts
- Remove duplicate custom-tools page (custom-tools/index.mdx → tools/custom-tools.mdx is canonical)
- Remove comparison table from custom-tools per product preference
- Fix permissions inconsistency: delete now requires Admin across all docs
- Consolidate sdks/ into api-reference/ (sdks/ directory deleted)
- Fix Python SDK docs: correct param is `input`, not `input_data`
- Fix TypeScript SDK docs: correct signature is executeWorkflow(id, input, options) not options-object form
- Add FAQ sections to both SDK reference pages
@waleedlatif1
Copy link
Copy Markdown
Collaborator Author

Re: Greptile reviewxhigh is confirmed valid per Anthropic's official migration guide and adaptive thinking docs. It's documented as an exclusively Opus 4.7 effort level (output_config.effort) that maps 1:1 to the API — no translation layer needed. Inline thread resolved.

@waleedlatif1
Copy link
Copy Markdown
Collaborator Author

Re: Cursor review — The medium-risk assessment is correct. Both the temperature guard (supportsTemperature check before setting payload temperature) and the xhigh effort level have been verified against Anthropic's official docs. The xhigh value is a documented Anthropic API value for output_config.effort on Opus 4.7, and the temperature deprecation is a confirmed breaking change per the migration guide. All good to ship.

@waleedlatif1 waleedlatif1 merged commit 9f41736 into staging Apr 16, 2026
8 of 9 checks passed
@waleedlatif1 waleedlatif1 deleted the fix/misc branch April 16, 2026 20:05
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.

1 participant