Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions config/codex/config.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,12 @@ notify = [
"notify",
]

web_search = "cached"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

For better maintainability, consider adding a comment to explain this new top-level configuration key. Since web_search was moved from the [features] section and its type changed, a brief explanation of its purpose would be helpful for anyone else editing this file in the future.

Suggested change
web_search = "cached"
# Configures the web search mode.
web_search = "cached"


[features]
ghost_commit = false
undo = false
shell_tool = true
web_search = true
web_search_cached = true
unified_exec = true
apply_patch_freeform = true
exec_policy = true
Expand All @@ -22,10 +23,12 @@ child_agents_md = true
powershell_utf8 = false
enable_request_compression = false
collab = true
connectors = false
apps = true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The feature flag connectors has been renamed to apps. To improve clarity and future maintainability, consider adding a comment explaining what this feature flag controls, as 'apps' is a generic term. Mentioning the old name could also be helpful for those familiar with the previous configuration.

Suggested change
apps = true
# Enables integration with external applications (formerly 'connectors').
apps = true

Copilot AI Feb 1, 2026

Copy link

Choose a reason for hiding this comment

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

The feature flag name "apps" is somewhat generic and could be ambiguous. Based on the PR description, this replaces the "connectors" feature flag, but "apps" doesn't clearly convey what functionality it controls. Consider using a more descriptive name like "app_connectors", "external_apps", or "third_party_apps" to make the configuration more self-documenting and maintainable.

Suggested change
apps = true
app_connectors = true

Copilot uses AI. Check for mistakes.
skills = true
steer = true
collaboration_modes = true
responses_websockets = false
personality = true

[model_providers.cliproxyapi]
name = "CLIProxyAPI"
Expand Down
Loading