Skip to content

fix: move organization model overrides from Models to Coder Agents settings - #28473

Merged
ethanndickson merged 4 commits into
mainfrom
overrides-scoping-0sy9
Aug 24, 2026
Merged

fix: move organization model overrides from Models to Coder Agents settings#28473
ethanndickson merged 4 commits into
mainfrom
overrides-scoping-0sy9

Conversation

@ethanndickson

@ethanndickson ethanndickson commented Aug 24, 2026

Copy link
Copy Markdown
Member

Summary

Relocates the organization-scoped Coder Agents model override controls from the Defaults & overrides tab under AI Settings > Models into an Organization settings section on AI Settings > Coder Agents. Models remains the home for reusable model definitions, provider availability, default designation, and ACLs; Coder Agents now owns all agent-specific model/reasoning override controls. The backend organization scoping from #28442 is unchanged.

Changes

  • Coder Agents page is split into two sections:
    • Organization settings: General subagent, Explore subagent, title generation, compaction, and Advisor (behind the chat-advisor experiment) model/reasoning overrides, with an organization picker when more than one accessible organization exists.
    • Deployment settings: personal override policy, virtual desktop provider (behind its experiment), and Advisor usage limits, visible only with editDeploymentConfig.
  • Permissions: the page and its sidebar entry are accessible with either editDeploymentConfig or organization chat model access. Organization mutations require editChatModelConfigs on the selected organization; otherwise controls render read-only. Organization discovery, selected-organization permissions, and deployment queries fail independently.
  • Models page drops the Defaults & overrides tab and the now-single-item tab UI. The old DefaultsPage components move to CoderAgentsPage/OrganizationAgentSettings*.
  • Compatibility: /ai/settings/models/defaults redirects to /ai/settings/coder-agents, preserving org and other query parameters.
  • Docs: intentionally not updated here; a follow-up PR will update docs/ai-coder/agents/models.md and docs/ai-coder/agents/platform-controls/advisor.md.

Testing

  • Targeted unit tests (ModelDefaultsRedirect, organizationModels): 9 passing.
  • Affected Storybook interaction tests (Coder Agents page/view, organization agent settings, Models layout, AI settings sidebar): 41 passing.
  • pnpm check, full pnpm lint (Biome, tsc, circular deps, React compiler, knip), pnpm format.

@github-actions

Copy link
Copy Markdown
Contributor

Docs preview

Check off each page once it's been reviewed. If a page changes in a later push, its checkbox clears automatically so it gets a fresh look. Pages not yet wired into the docs navigation aren't listed here.

…ttings

The Defaults & overrides tab under AI Settings > Models mixed agent
override controls into the reusable model catalog and gated them behind
the Models layout, while the Coder Agents page was gated wholesale by
editDeploymentConfig and blocked organization model administrators.

Relocate the organization override controls (General, Explore, title
generation, compaction, and Advisor) into an Organization settings
section on the Coder Agents page with an organization picker, and keep
deployment-wide controls in a separate Deployment settings section
visible only with editDeploymentConfig. The page and sidebar entry are
now accessible with either deployment or organization model access.

The legacy /ai/settings/models/defaults route redirects to
/ai/settings/coder-agents preserving query parameters. The Models page
drops its now-single tab UI and the unused defaults path helpers.
@ethanndickson
ethanndickson force-pushed the overrides-scoping-0sy9 branch from 0d75682 to 034a2a2 Compare August 24, 2026 15:57
@coderagents

coderagents Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Documentation Check

This PR removes the Defaults & overrides tab under AI Settings > Models and relocates the organization-scoped admin override controls to an Organization settings section on AI Settings > Coder Agents. Two docs still point at the old location and are now inaccurate. (The PR body notes docs are intentionally deferred to a follow-up; flagging here for tracking.)

Updates Needed

  • docs/ai-coder/agents/models.md - The Admin overrides bullet (~line 305) says to configure them on the "Defaults & overrides tab under AI Settings > Models". Update to the new location: AI Settings > Coder Agents > Organization settings.
  • docs/ai-coder/agents/platform-controls/advisor.md - The advisor model override section (~lines 42-44) also references the "Defaults & overrides tab under AI Settings > Models". Update to AI Settings > Coder Agents > Organization settings.

Automated review via Coder Agents

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 034a2a21ea

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread site/src/pages/AISettingsPage/CoderAgentsPage/CoderAgentsPage.tsx
Comment thread site/src/router.tsx
@ethanndickson

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

Reviewed commit: bd09ccbdae

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ethanndickson
ethanndickson merged commit eab5e28 into main Aug 24, 2026
26 checks passed
@ethanndickson
ethanndickson deleted the overrides-scoping-0sy9 branch August 24, 2026 16:43
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants