Skip to content

maintenance(coder-labs/modules/codex): skip migration notice and add agentapi type flag#781

Merged
35C4n0r merged 9 commits intomainfrom
35C4n0r/chore-codex-maintenance-1
Mar 3, 2026
Merged

maintenance(coder-labs/modules/codex): skip migration notice and add agentapi type flag#781
35C4n0r merged 9 commits intomainfrom
35C4n0r/chore-codex-maintenance-1

Conversation

@35C4n0r
Copy link
Collaborator

@35C4n0r 35C4n0r commented Mar 3, 2026

This PR introduces:

  1. Adding --type flag to agentapi command
  2. Introduce [notice.model_migrations] to skip migration notice, improves tasks UX
  3. Set profile = "aibridge" rather than passing it using --profile flag

Description

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder-labs/modules/codex
New version: v4.1.2
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

Closes: #740

@35C4n0r 35C4n0r changed the title feat(coder-labs/modules/codex): skip migration notice and add agentapi type flag maintenance(coder-labs/modules/codex): skip migration notice and add agentapi type flag Mar 3, 2026
@35C4n0r 35C4n0r self-assigned this Mar 3, 2026
@35C4n0r 35C4n0r added the version:patch Add to PRs requiring a patch version upgrade label Mar 3, 2026
@35C4n0r 35C4n0r merged commit e3abbb9 into main Mar 3, 2026
5 checks passed
@35C4n0r 35C4n0r deleted the 35C4n0r/chore-codex-maintenance-1 branch March 3, 2026 05:05
Comment on lines 65 to 78
```toml
[model_providers.aibridge]
name = "AI Bridge"
base_url = "https://example.coder.com/api/v2/aibridge/openai/v1"
env_key = "CODER_AIBRIDGE_SESSION_TOKEN"
wire_api = "responses"

[profiles.aibridge]
model_provider = "aibridge"
model = "<model>" # as configured in the module input
model_reasoning_effort = "<model_reasoning_effort>" # as configured in the module input
```

Codex then runs with `--profile aibridge`
Copy link
Member

Choose a reason for hiding this comment

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

@35C4n0r we forgot to update this section.

Copy link
Member

Choose a reason for hiding this comment

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

I think we no longer need --profile with the updated config.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

will update this in a new pr.

Copy link
Member

Choose a reason for hiding this comment

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

Please doublecheck its not not needed. I am assuming you have tested it already.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes it's not needed since we set the default in the config.toml now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:patch Add to PRs requiring a patch version upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

codex: add --type codex to agentapi server

2 participants