disabled by default

Author: kommander

packages/opencode/specs/tui-plugins.md

@@ -55,7 +55,7 @@ Example:
 - Internal plugins can declare `enabled: false` to be registered but inactive by default; `plugin_enabled` and runtime KV can still enable them by id.
 - `plugin_enabled` is merged across config layers.
 - Runtime enable/disable state is also stored in KV under `plugin_enabled`; that KV state overrides config on startup.
-- `attention.enabled` disables all `api.attention.notify(...)` delivery when set to `false`.
+- `attention.enabled` defaults to `false`; when `false`, it disables all `api.attention.notify(...)` delivery.
 - `attention.notifications` and `attention.sound` independently control terminal-mediated desktop notifications and built-in sounds.
 - `attention.volume` sets the default built-in sound volume from `0` to `1`.
 - `attention.sound_pack` selects the initial semantic sound pack. Persisted runtime selection in KV can override it.

packages/opencode/specs/v2/notifications.md

@@ -0,0 +1,13 @@
+# TUI Notifications Default
+
+Problem:
+
+- v1 defaults `attention.enabled` to `false`
+- users can opt in with `attention.enabled = true`
+- v2 should make core TUI notifications a default behavior
+
+## v2 Target
+
+Flip `attention.enabled` to `true` by default in v2.
+
+Keep `attention.enabled = false` as the explicit opt-out.

packages/opencode/src/cli/cmd/tui/config/tui.ts

@@ -231,7 +231,7 @@ const loadState = Effect.fn("TuiConfig.loadState")(function* (ctx: { directory:
   const result: Resolved = {
     ...acc.result,
     attention: {
-      enabled: acc.result.attention?.enabled ?? true,
+      enabled: acc.result.attention?.enabled ?? false,
       notifications: acc.result.attention?.notifications ?? true,
       sound: acc.result.attention?.sound ?? true,
       volume: acc.result.attention?.volume ?? 0.4,

packages/opencode/test/cli/run/runtime.boot.test.ts

@@ -94,7 +94,7 @@ function config(input?: {
   })
   return {
     attention: {
-      enabled: true,
+      enabled: false,
       notifications: true,
       sound: true,
       volume: 0.4,

packages/opencode/test/config/tui.test.ts

@@ -145,7 +145,7 @@ test("loads tui config with the same precedence order as server config paths", a
 test("resolves attention config defaults and overrides", async () => {
   await using defaults = await tmpdir()
   expect((await getTuiConfig(defaults.path)).attention).toEqual({
-    enabled: true,
+    enabled: false,
     notifications: true,
     sound: true,
     volume: 0.4,

packages/opencode/test/fixture/tui-runtime.ts

@@ -24,7 +24,7 @@ export function createTuiResolvedConfig(input: ResolvedInput = {}): TuiConfig.Re
   return {
     ...input,
     attention: {
-      enabled: true,
+      enabled: false,
       notifications: true,
       sound: true,
       volume: 0.4,