style: apply oxlint auto-fixes

pterror · claude · pterror · commit 36a9d1bd6aac · 2026-01-08T14:44:02.000+10:00
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/apps/discord-bot/src/bot.ts b/apps/discord-bot/src/bot.ts
@@ -77,7 +77,7 @@ class DiscordBot {
         // 4. Send Message to Core via sudo
         const parts = message.content.split(" ");
         if (parts[0]) {
-          // execute("sudo", [targetId, verb, ...args])
+          // Execute("sudo", [targetId, verb, ...args])
           socket.execute("sudo", [
             entityId, // Target ID
             parts[0], // Verb
@@ -86,7 +86,7 @@ class DiscordBot {
         }
       } catch (error) {
         console.error("Error handling message:", error);
-        // message.reply("Something went wrong.");
+        // Message.reply("Something went wrong.");
       }
     });
   }
diff --git a/apps/discord-bot/src/session.test.ts b/apps/discord-bot/src/session.test.ts
@@ -55,7 +55,7 @@ describe("Session Manager", () => {
     mockPlayerSocket.on.mockImplementation(
       (event: string, _handler: (...args: any[]) => unknown) => {
         if (event === "message") {
-          // no-op
+          // No-op
         }
       },
     );
diff --git a/apps/discord-bot/src/socket.ts b/apps/discord-bot/src/socket.ts
@@ -70,7 +70,7 @@ export class SocketManager extends EventEmitter {
     this.socket.on("message", (data) => {
       if (data.method === "forward") {
         // Forwarded message from Core
-        // params: { target: entityId, type: string, payload: any }
+        // Params: { target: entityId, type: string, payload: any }
         const { target, type, payload } = data.params;
         this.emit("message", target, { type, ...payload });
       } else {
diff --git a/apps/tui/src/App.tsx b/apps/tui/src/App.tsx
@@ -200,7 +200,7 @@ const App = () => {
     // Use local opcodes if available
     if (clientState.opcodes) {
       const lines = code.split("\n");
-      const line = lines[position.lineNumber - 1] || "";
+      const line = lines[position.lineNumber - 1] ?? "";
       const textBeforeCursor = line.slice(0, position.column - 1);
 
       // Find the word being typed.
@@ -210,7 +210,7 @@ const App = () => {
         const [prefix] = match;
         // Filter opcodes
         // We cast opcodes to any[] because we don't have the type imported,
-        // but we know it has an 'opcode' field.
+        // But we know it has an 'opcode' field.
         const matches = clientState.opcodes.filter((op: any) => op.opcode.startsWith(prefix));
         if (matches.length > 0) {
           // Return the suffix of the first match
@@ -249,7 +249,7 @@ const App = () => {
     }
     return room["contents"]
       .map((id: number) => clientState.entities.get(id))
-      .filter((entity) => !!entity);
+      .filter((entity) => Boolean(entity));
   };
 
   // Handle item inspection
diff --git a/apps/tui/src/components/Compass.tsx b/apps/tui/src/components/Compass.tsx
@@ -53,7 +53,7 @@ export default function Compass({ room, entities }: CompassProps) {
     }
 
     const exit = getExit(dir);
-    const label = DIR_LABELS[dir] || dir;
+    const label = DIR_LABELS[dir] ?? dir;
     const destName = exit ? ((exit["destination_name"] ?? exit["name"]) as string) : "";
 
     return (
diff --git a/apps/tui/src/components/Editor.tsx b/apps/tui/src/components/Editor.tsx
@@ -77,7 +77,7 @@ const Editor: React.FC<EditorProps> = ({
       });
     } else if (key.return) {
       setLines((prev) => {
-        const currentLine = prev[cursor.row] || "";
+        const currentLine = prev[cursor.row] ?? "";
         const before = currentLine.slice(0, cursor.col);
         const after = currentLine.slice(cursor.col);
         const newLines = [...prev];
@@ -89,7 +89,7 @@ const Editor: React.FC<EditorProps> = ({
       if (cursor.col > 0) {
         // Simple backspace within line
         setLines((prev) => {
-          const line = prev[cursor.row] || "";
+          const line = prev[cursor.row] ?? "";
           const newLine = line.slice(0, cursor.col - 1) + line.slice(cursor.col);
           const newLines = [...prev];
           newLines[cursor.row] = newLine;
@@ -99,14 +99,14 @@ const Editor: React.FC<EditorProps> = ({
       } else if (cursor.row > 0) {
         // Merge with previous line
         setLines((prev) => {
-          const currentLine = prev[cursor.row] || "";
-          const prevLine = prev[cursor.row - 1] || "";
+          const currentLine = prev[cursor.row] ?? "";
+          const prevLine = prev[cursor.row - 1] ?? "";
           const newLines = [...prev];
           newLines.splice(cursor.row - 1, 2, prevLine + currentLine);
           return newLines;
         });
         setCursor((prev) => ({
-          col: (lines[prev.row - 1] || "").length,
+          col: (lines[prev.row - 1] ?? "").length,
           row: prev.row - 1,
         }));
       }
@@ -123,7 +123,7 @@ const Editor: React.FC<EditorProps> = ({
         .then((completion) => {
           if (completion) {
             setLines((prev) => {
-              const line = prev[cursor.row] || "";
+              const line = prev[cursor.row] ?? "";
               const newLine = line.slice(0, cursor.col) + completion + line.slice(cursor.col);
               const newLines = [...prev];
               newLines[cursor.row] = newLine;
@@ -149,7 +149,7 @@ const Editor: React.FC<EditorProps> = ({
         .then((completion) => {
           if (completion) {
             setLines((prev) => {
-              const line = prev[cursor.row] || "";
+              const line = prev[cursor.row] ?? "";
               const newLine = line.slice(0, cursor.col) + completion + line.slice(cursor.col);
               const newLines = [...prev];
               newLines[cursor.row] = newLine;
@@ -161,7 +161,7 @@ const Editor: React.FC<EditorProps> = ({
     } else {
       // Regular typing
       setLines((prev) => {
-        const line = prev[cursor.row] || "";
+        const line = prev[cursor.row] ?? "";
         const newLine = line.slice(0, cursor.col) + input + line.slice(cursor.col);
         const newLines = [...prev];
         newLines[cursor.row] = newLine;
@@ -205,7 +205,7 @@ const Editor: React.FC<EditorProps> = ({
                 <Text>
                   {line.slice(0, cursor.col)}
                   <Text inverse color="cyan">
-                    {line[cursor.col] || " "}
+                    {line[cursor.col] ?? " "}
                   </Text>
                   {line.slice(cursor.col + 1)}
                 </Text>
diff --git a/apps/web/src/components/Compass.tsx b/apps/web/src/components/Compass.tsx
@@ -42,14 +42,14 @@ export default function Compass() {
             }}
             classList={{
               compass__cell: true,
-              "compass__cell--active": !!exit(),
+              "compass__cell--active": Boolean(exit()),
             }}
           >
             <div class="compass__cell-label">{props.label}</div>
             <div
               classList={{
                 "compass__cell-dest": true,
-                "compass__cell-dest--active": !!exit(),
+                "compass__cell-dest--active": Boolean(exit()),
               }}
             >
               {exit() ? ((exit()?.["destination_name"] ?? exit()?.["name"]) as string) : "+"}
diff --git a/apps/web/src/components/DigPanel.tsx b/apps/web/src/components/DigPanel.tsx
@@ -9,7 +9,7 @@ interface DigPanelProps {
 }
 
 export default function DigPanel(props: DigPanelProps) {
-  const [direction, setDirection] = createSignal(props.initialDirection || "");
+  const [direction, setDirection] = createSignal(props.initialDirection ?? "");
   const [mode, setMode] = createSignal<"new" | "existing">("new");
   const [roomName, setRoomName] = createSignal("");
   const [targetRoom, setTargetRoom] = createSignal("");
diff --git a/apps/web/src/components/GameLog.tsx b/apps/web/src/components/GameLog.tsx
@@ -18,7 +18,7 @@ export default function GameLog() {
   // Auto-scroll to bottom
   createEffect(() => {
     // oxlint-disable-next-line no-unused-expressions
-    gameStore.state.messages.length; // dependency
+    gameStore.state.messages.length; // Dependency
     if (containerRef) {
       containerRef.scrollTop = containerRef.scrollHeight;
     }
@@ -30,7 +30,7 @@ export default function GameLog() {
         {(msg) => {
           switch (msg.type) {
             case "error": {
-              // fallthrough to message view, which handles error type styling
+              // Fallthrough to message view, which handles error type styling
               return <MessageView text={msg.text} type={msg.type} />;
             }
             case "message": {
diff --git a/apps/web/src/components/RoomPanel.tsx b/apps/web/src/components/RoomPanel.tsx
@@ -46,7 +46,7 @@ export default function RoomPanel() {
     }
     return (roomValue["contents"] as number[])
       .map((id) => gameStore.state.entities.get(id))
-      .filter((entity) => !!entity);
+      .filter((entity) => Boolean(entity));
   };
 
   const exits = () => {
@@ -56,7 +56,7 @@ export default function RoomPanel() {
     }
     return (roomValue["exits"] as number[])
       .map((id) => gameStore.state.entities.get(id))
-      .filter((entity) => !!entity);
+      .filter((entity) => Boolean(entity));
   };
 
   return (
diff --git a/apps/web/src/components/ThemeEditor.tsx b/apps/web/src/components/ThemeEditor.tsx
@@ -19,7 +19,7 @@ export const ThemeEditor: Component<Props> = (props) => {
       "download",
       `${theme.manifest.name.replaceAll(/\s+/g, "_").toLowerCase()}.json`,
     );
-    document.body.append(downloadAnchorNode); // required for firefox
+    document.body.append(downloadAnchorNode); // Required for firefox
     downloadAnchorNode.click();
     downloadAnchorNode.remove();
   };
diff --git a/apps/web/src/store/keybinds.test.ts b/apps/web/src/store/keybinds.test.ts
@@ -9,7 +9,7 @@ const localStorageMock = (() => {
     clear: mock(() => {
       store = {};
     }),
-    getItem: mock((key: string) => store[key] || null),
+    getItem: mock((key: string) => store[key] ?? null),
     removeItem: mock((key: string) => {
       delete store[key];
     }),
@@ -28,7 +28,7 @@ if (!globalThis.localStorage) {
 }
 
 // Import after mock
-// import { keybindsStore } from "./keybinds";
+// Import { keybindsStore } from "./keybinds";
 
 let keybindsStore: any;
 
diff --git a/apps/web/src/store/theme.test.ts b/apps/web/src/store/theme.test.ts
@@ -7,7 +7,7 @@ globalThis.alert = mock(() => {});
 const mockStorage = new Map<string, string>();
 const localStorageMock = {
   clear: mock(() => mockStorage.clear()),
-  getItem: mock((key: string) => mockStorage.get(key) || null),
+  getItem: mock((key: string) => mockStorage.get(key) ?? null),
   key: mock(() => null),
   length: 0,
   removeItem: mock((key: string) => mockStorage.delete(key)),
diff --git a/apps/web/src/store/theme.ts b/apps/web/src/store/theme.ts
@@ -160,7 +160,7 @@ const [state, setState] = createStore<ThemeState>(loadInitialState());
 
 export const themeStore = {
   get activeTheme() {
-    return state.themes.find((theme) => theme.id === state.activeThemeId) || defaultTheme;
+    return state.themes.find((theme) => theme.id === state.activeThemeId) ?? defaultTheme;
   },
 
   createTheme: (name: string) => {
diff --git a/packages/client/src/client.ts b/packages/client/src/client.ts
@@ -88,8 +88,8 @@ export class BloomClient {
 
     this.socket.onerror = (err) => {
       console.error("WebSocket error:", err);
-      // onerror is usually followed by onclose, so we don't need to reconnect here explicitly
-      // unless onclose isn't called, but standard WS behavior says it should be.
+      // Onerror is usually followed by onclose, so we don't need to reconnect here explicitly
+      // Unless onclose isn't called, but standard WS behavior says it should be.
     };
 
     this.socket.onmessage = (event) => {
@@ -355,7 +355,7 @@ export class BloomClient {
         const { params } = notification as StreamChunkNotification;
         // Append to the last message
         // We need to be careful not to mutate the state directly in a way that SolidJS doesn't pick up,
-        // but here we are replacing the messages array.
+        // But here we are replacing the messages array.
         // However, we want to modify the *last* message.
         const messages = [...this.state.messages];
         if (messages.length > 0) {
diff --git a/packages/shared/src/adapters/discord.test.ts b/packages/shared/src/adapters/discord.test.ts
@@ -45,7 +45,7 @@ describe("DiscordAdapter", () => {
 
       const output = discordAdapter.formatRoom(msg);
       const exitField = output.embeds![0].fields![0];
-      expect(exitField.value).toContain("\u{1F512}"); // lock emoji
+      expect(exitField.value).toContain("\u{1F512}"); // Lock emoji
     });
   });
 
diff --git a/packages/shared/src/adapters/discord.ts b/packages/shared/src/adapters/discord.ts
@@ -294,22 +294,30 @@ export class DiscordAdapter implements MessageAdapter<DiscordOutput> {
 
   format(msg: GameMessage): DiscordOutput {
     switch (msg.type) {
-      case "room":
+      case "room": {
         return this.formatRoom(msg);
-      case "inventory":
+      }
+      case "inventory": {
         return this.formatInventory(msg);
-      case "item":
+      }
+      case "item": {
         return this.formatItem(msg);
-      case "status":
+      }
+      case "status": {
         return this.formatStatus(msg);
-      case "text":
+      }
+      case "text": {
         return this.formatText(msg);
-      case "error":
+      }
+      case "error": {
         return this.formatError(msg);
-      case "action":
+      }
+      case "action": {
         return this.formatAction(msg);
-      case "dialogue":
+      }
+      case "dialogue": {
         return this.formatDialogue(msg);
+      }
     }
   }
 
diff --git a/packages/shared/src/adapters/types.ts b/packages/shared/src/adapters/types.ts
@@ -40,17 +40,17 @@ export const MessageColors = {
 
 /** Emoji/icon mapping for consistent visual language */
 export const MessageIcons = {
-  exits: "\u{1F6AA}", // door
-  contents: "\u{1F4E6}", // package
-  inventory: "\u{1F392}", // backpack
-  health: "\u{2764}\u{FE0F}", // red heart
-  mana: "\u{1F535}", // blue circle
-  level: "\u{2B50}", // star
-  locked: "\u{1F512}", // locked
-  unlocked: "\u{1F513}", // unlocked
-  speaker: "\u{1F4AC}", // speech bubble
-  attack: "\u{2694}\u{FE0F}", // crossed swords
-  error: "\u{26A0}\u{FE0F}", // warning
+  exits: "\u{1F6AA}", // Door
+  contents: "\u{1F4E6}", // Package
+  inventory: "\u{1F392}", // Backpack
+  health: "\u{2764}\u{FE0F}", // Red heart
+  mana: "\u{1F535}", // Blue circle
+  level: "\u{2B50}", // Star
+  locked: "\u{1F512}", // Locked
+  unlocked: "\u{1F513}", // Unlocked
+  speaker: "\u{1F4AC}", // Speech bubble
+  attack: "\u{2694}\u{FE0F}", // Crossed swords
+  error: "\u{26A0}\u{FE0F}", // Warning
 } as const;
 
 /**
diff --git a/packages/shared/src/messages.ts b/packages/shared/src/messages.ts
@@ -159,15 +159,18 @@ export function parseGameMessage(data: unknown): GameMessage | null {
     case "text":
     case "error":
     case "action":
-    case "dialogue":
+    case "dialogue": {
       return data as GameMessage;
-    case "message":
+    }
+    case "message": {
       // Legacy format: convert to TextMessage
       return {
         type: "text",
         text: (data as { text?: string }).text ?? "",
       };
-    default:
+    }
+    default: {
       return null;
+    }
   }
 }
diff --git a/scripts/check-opcodes.ts b/scripts/check-opcodes.ts
@@ -107,7 +107,7 @@ function main() {
     }
 
     // Don't fail - just report
-    // process.exit(1);
+    // Process.exit(1);
   } else {
     console.log("\n✅ All opcodes have test coverage!");
   }
diff --git a/scripts/dev-docs.ts b/scripts/dev-docs.ts
@@ -65,7 +65,7 @@ function main() {
     // If docs exit, we exit
     docs.on("exit", (code) => {
       playground.kill();
-      process.exit(code || 0);
+      process.exit(code ?? 0);
     });
   }
 }
diff --git a/seeds/definitions/ChatTree.ts b/seeds/definitions/ChatTree.ts
@@ -29,9 +29,7 @@ export class ChatTree extends EntityBase {
     if (!this.messages) {
       this.messages = {};
     }
-    if (this.next_message_id === undefined) {
-      this.next_message_id = 1;
-    }
+    this.next_message_id ??= 1;
     if (!this.branches) {
       this.branches = { main: null! };
     }
diff --git a/seeds/definitions/Golem.ts b/seeds/definitions/Golem.ts
diff --git a/seeds/definitions/Hotel.ts b/seeds/definitions/Hotel.ts
diff --git a/seeds/definitions/Items.ts b/seeds/definitions/Items.ts
diff --git a/seeds/definitions/Player.ts b/seeds/definitions/Player.ts

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ class DiscordBot {`
`77`	`77`	`// 4. Send Message to Core via sudo`
`78`	`78`	`const parts = message.content.split(" ");`
`79`	`79`	`if (parts[0]) {`
`80`		`- // execute("sudo", [targetId, verb, ...args])`
	`80`	`+ // Execute("sudo", [targetId, verb, ...args])`
`81`	`81`	`socket.execute("sudo", [`
`82`	`82`	`entityId, // Target ID`
`83`	`83`	`parts[0], // Verb`
`@@ -86,7 +86,7 @@ class DiscordBot {`
`86`	`86`	`}`
`87`	`87`	`} catch (error) {`
`88`	`88`	`console.error("Error handling message:", error);`
`89`		`- // message.reply("Something went wrong.");`
	`89`	`+ // Message.reply("Something went wrong.");`
`90`	`90`	`}`
`91`	`91`	`});`
`92`	`92`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ describe("Session Manager", () => {`
`55`	`55`	`mockPlayerSocket.on.mockImplementation(`
`56`	`56`	`(event: string, _handler: (...args: any[]) => unknown) => {`
`57`	`57`	`if (event === "message") {`
`58`		`- // no-op`
	`58`	`+ // No-op`
`59`	`59`	`}`
`60`	`60`	`},`
`61`	`61`	`);`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ export default function Compass({ room, entities }: CompassProps) {`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`const exit = getExit(dir);`
`56`		`- const label = DIR_LABELS[dir] \|\| dir;`
	`56`	`+ const label = DIR_LABELS[dir] ?? dir;`
`57`	`57`	`const destName = exit ? ((exit["destination_name"] ?? exit["name"]) as string) : "";`
`58`	`58`
`59`	`59`	`return (`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ interface DigPanelProps {`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`export default function DigPanel(props: DigPanelProps) {`
`12`		`- const [direction, setDirection] = createSignal(props.initialDirection \|\| "");`
	`12`	`+ const [direction, setDirection] = createSignal(props.initialDirection ?? "");`
`13`	`13`	`const [mode, setMode] = createSignal<"new" \| "existing">("new");`
`14`	`14`	`const [roomName, setRoomName] = createSignal("");`
`15`	`15`	`const [targetRoom, setTargetRoom] = createSignal("");`