new config: app:hideaibutton, app:disablectrlshiftarrows, app:disablectrlshiftdisplay

[sawka]

| app:hideaibutton | bool | Set to true to hide the AI button in the tab bar (defaults to false) |
| app:disablectrlshiftarrows | bool | Set to true to disable Ctrl+Shift+Arrow keybindings for block navigation (defaults to false) |
| app:disablectrlshiftdisplay | bool | Set to true to disable the Ctrl+Shift visual indicator display (defaults to false) |

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This PR introduces three new boolean settings: app:hideaibutton, app:disablectrlshiftarrows, and app:disablectrlshiftdisplay. It updates default configuration and docs to v0.14.0, adds corresponding constants and fields in pkg/wconfig, extends the JSON schema and TypeScript settings type, and wires the settings into the frontend so the AI button can be hidden and Ctrl+Shift+Arrow and Ctrl+Shift display-related behaviors can be suppressed when their settings are enabled.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly reflects the main changes: three new configuration keys are added (app:hideaibutton, app:disablectrlshiftarrows, app:disablectrlshiftdisplay) with all changes centered around these features.
Description check	✅ Passed	The description provides specific details about the three new configuration options, their purposes, and default values—directly corresponding to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sawka/config-updates-2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

docs/docs/config.mdx (1)

110-144: ⚠️ Potential issue | 🟡 Minor

The default configuration example block is stale and doesn't include the new keys.

Line 110 references "v0.11.5" but the three new settings (app:hideaibutton, app:disablectrlshiftarrows, app:disablectrlshiftdisplay) aren't listed in this example. Consider updating the version reference and adding the new defaults to keep the docs in sync.

🧹 Nitpick comments (1)

frontend/app/store/keymodel.ts (1)

533-564: Consider extracting the repeated settings check into a helper.

The same globalStore.get(getSettingsKeyAtom("app:disablectrlshiftarrows")) call appears in all four arrow handlers. A small helper or a single check before the switch could reduce duplication.

♻️ Example refactor

+function isCtrlShiftArrowsDisabled(): boolean {
+    return globalStore.get(getSettingsKeyAtom("app:disablectrlshiftarrows"));
+}
+
 globalKeyMap.set("Ctrl:Shift:ArrowUp", () => {
-    const disableCtrlShiftArrows = globalStore.get(getSettingsKeyAtom("app:disablectrlshiftarrows"));
-    if (disableCtrlShiftArrows) {
+    if (isCtrlShiftArrowsDisabled()) {
         return false;
     }
     switchBlockInDirection(NavigateDirection.Up);
     return true;
 });

Apply the same pattern to the other three handlers.

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In `@docs/docs/config.mdx`:
- Line 145: The default in the shown JSON ("term:durable": false) conflicts with
the keys table that states term:durable defaults to true; update the JSON
default to "term:durable": true so the sample config matches the documented
default (or, if you prefer the JSON value, update the keys table text to say
"defaults to false"); locate the term:durable entry in the config block and make
the corresponding change so both the sample config and the keys table are
consistent.
- Line 137: Add a documentation entry for the configuration key
"window:fullscreenonlaunch" to the Configuration Keys table: describe its
purpose (controls whether the app launches in fullscreen), its type (boolean),
default value (false), and any platform/behavior notes; update the keys table in
docs/docs/config.mdx (the Configuration Keys section rows) to include a new row
for "window:fullscreenonlaunch" matching the format used by the other keys so it
appears alongside keys from lines ~35–108.