Fix shouldSynchronize CI test and kittyKeyboard enabled mode

- Branch shouldSynchronize test assertion on isInCi so it expects the
  correct value in both CI and non-CI environments
- Restructure initKittyKeyboard so mode 'enabled' bypasses the
  interactive guard while still requiring TTY streams, matching the
  documented behavior

Author: costajohnt

src/ink.tsx

@@ -1012,19 +1012,28 @@ export default class Ink {
 		const opts = this.options.kittyKeyboard;
 		const mode = opts.mode ?? 'auto';
 
-		if (
-			mode === 'disabled' ||
-			!this.interactive ||
-			!this.options.stdin.isTTY ||
-			!this.options.stdout.isTTY
-		) {
+		if (mode === 'disabled') {
 			return;
 		}
 
 		const flags: KittyFlagName[] = opts.flags ?? ['disambiguateEscapeCodes'];
 
+		// 'enabled' force-enables the protocol as long as both streams are TTYs,
+		// regardless of the interactive setting (e.g. even in CI).
 		if (mode === 'enabled') {
-			this.enableKittyProtocol(flags);
+			if (this.options.stdin.isTTY && this.options.stdout.isTTY) {
+				this.enableKittyProtocol(flags);
+			}
+
+			return;
+		}
+
+		// Auto mode: require interactive + TTY
+		if (
+			!this.interactive ||
+			!this.options.stdin.isTTY ||
+			!this.options.stdout.isTTY
+		) {
 			return;
 		}
 

test/write-synchronized.tsx

@@ -1,5 +1,6 @@
 import EventEmitter from 'node:events';
 import test from 'ava';
+import isInCi from 'is-in-ci';
 import {bsu, esu, shouldSynchronize} from '../src/write-synchronized.js';
 
 const createStream = ({tty = false} = {}) => {
@@ -38,8 +39,12 @@ test('shouldSynchronize returns false for non-TTY stream', t => {
 test('shouldSynchronize uses CI detection when interactive is not specified', t => {
 	const ttyStream = createStream({tty: true});
 	// When interactive is omitted, shouldSynchronize falls back to is-in-ci.
-	// In a non-CI environment with a TTY stream, this should return true.
-	t.true(shouldSynchronize(ttyStream));
+	// In CI the result is false (non-interactive by design); outside CI it's true.
+	if (isInCi) {
+		t.false(shouldSynchronize(ttyStream));
+	} else {
+		t.true(shouldSynchronize(ttyStream));
+	}
 });
 
 test('shouldSynchronize returns false for non-TTY stream when interactive is not specified', t => {