fix(lint): remove eslint-disable and forbidden cast from unicorn fixes

- auto-agent-flags: replace ||=== chain with agentModes.find() so TypeScript
  narrows AgentMode without a type cast and no eslint-disable comment is needed
- app-ready-terminal-storage: guard with value !== null before .includes() so
  the string|null parameter works without an `as string` cast

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: skulidropek

packages/app/src/docker-git/frontend-lib/core/auto-agent-flags.ts

@@ -14,9 +14,10 @@ export const resolveAutoAgentFlags = (
   if (requested === "auto") {
     return Either.right({ agentMode: undefined, agentAuto: true })
   }
-  // eslint-disable-next-line unicorn/prefer-includes-over-repeated-comparisons
-  if (requested === "claude" || requested === "codex" || requested === "gemini" || requested === "grok") {
-    return Either.right({ agentMode: requested, agentAuto: true })
+  const agentModes: readonly AgentMode[] = ["claude", "codex", "gemini", "grok"]
+  const matchedMode = agentModes.find((mode) => mode === requested)
+  if (matchedMode !== undefined) {
+    return Either.right({ agentMode: matchedMode, agentAuto: true })
   }
   return Either.left({
     _tag: "InvalidOption",

packages/app/src/web/app-ready-terminal-storage.ts

@@ -43,7 +43,7 @@ const readJsonArray = (value: JsonValue | undefined): ReadonlyArray<JsonValue> |
 const isStoredTerminalStatus = (
   value: string | null
 ): value is ActiveTerminalSession["session"]["status"] =>
-  ["ready", "attached", "exited", "failed"].includes(value as string)
+  value !== null && ["ready", "attached", "exited", "failed"].includes(value)
 
 type StoredTerminalSessionFields = {
   readonly createdAt: string | null