Enable plugin hooks by default (#22549)

# Why

Plugin-bundled hooks are already wired through the plugin manager,
session setup, and app-server hook listing paths. Keeping `plugin_hooks`
disabled by default means users still need an explicit feature opt-in
before that existing behavior participates in normal plugin loading.

# What

- mark `plugin_hooks` as stable and enable it by default
- add feature-registry test coverage for the new default/stage pairing

Validation:

- `cargo test -p codex-features`
- `just fmt`
This commit is contained in:
Abhinav
2026-05-13 14:10:28 -07:00
committed by GitHub
Unverified
parent 0a2d751fc2
commit 14473c216f
2 changed files with 8 additions and 2 deletions
+2 -2
View File
@@ -975,8 +975,8 @@ pub const FEATURES: &[FeatureSpec] = &[
FeatureSpec {
id: Feature::PluginHooks,
key: "plugin_hooks",
stage: Stage::UnderDevelopment,
default_enabled: false,
stage: Stage::Stable,
default_enabled: true,
},
FeatureSpec {
id: Feature::InAppBrowser,
+6
View File
@@ -183,6 +183,12 @@ fn tool_search_is_stable_and_enabled_by_default() {
assert_eq!(Feature::ToolSearch.default_enabled(), true);
}
#[test]
fn plugin_hooks_are_stable_and_enabled_by_default() {
assert_eq!(Feature::PluginHooks.stage(), Stage::Stable);
assert_eq!(Feature::PluginHooks.default_enabled(), true);
}
#[test]
fn browser_controls_are_stable_and_enabled_by_default() {
assert_eq!(Feature::InAppBrowser.stage(), Stage::Stable);