Fix review-agent findings

Author: roodboi

apps/macos/Packages/Features/DashboardFeature/Sources/DashboardFeature/ProjectDetailView.swift

@@ -446,7 +446,10 @@ struct ProjectDetailView: View {
 
   private var runtimeServicesByName: [String: RuntimeService] {
     guard let runtime = project.runtime else { return [:] }
-    return Dictionary(uniqueKeysWithValues: runtime.services.map { ($0.service, $0) })
+    return Dictionary(
+      runtime.services.map { ($0.service, $0) },
+      uniquingKeysWith: { first, _ in first }
+    )
   }
 
   private var serviceHostsByName: [String: [String]] {

apps/macos/Packages/Features/DashboardFeature/Sources/DashboardFeature/ProjectInspectorColumn.swift

@@ -783,7 +783,10 @@ struct ProjectInspectorColumn: View {
 
   private var runtimeServicesByName: [String: RuntimeService] {
     guard let runtime = project.runtime else { return [:] }
-    return Dictionary(uniqueKeysWithValues: runtime.services.map { ($0.service, $0) })
+    return Dictionary(
+      runtime.services.map { ($0.service, $0) },
+      uniquingKeysWith: { first, _ in first }
+    )
   }
 
   private var serviceHostsByName: [String: [String]] {

docs/gateway-api.md

@@ -548,6 +548,10 @@ Non-JSON text frames are treated as raw input.
 | `status` | string | Human-readable status |
 | `name` | string | Container name |
 | `ports` | string | Port mapping string |
+| `image` | string or null | Container image |
+| `ip` | string or null | Container IP address |
+| `mounts` | RuntimeMount[] | Volume/bind mounts |
+| `labels` | object | Container labels |
 | `working_dir` | string or null | Compose working directory |
 
 ### PsItem

src/commands/global.ts

@@ -1956,7 +1956,7 @@ function removeLegacyDnsmasqLines(opts: {
     if (!updated.includes(legacyLine)) {
       continue;
     }
-    updated = updated.replace(legacyLine, "");
+    updated = updated.replaceAll(legacyLine, "");
     changed = true;
   }
 

src/commands/ssh.ts

@@ -147,6 +147,12 @@ async function handleSsh(opts: {
   const sessionArg = args.positionals.session;
 
   if (sessionArg) {
+    if (!SESSION_NAME_PATTERN.test(sessionArg)) {
+      p.log.error(
+        "Invalid session name (only letters, numbers, dashes, underscores, or dots)"
+      );
+      return 1;
+    }
     // Direct connect to specified session
     return await connectToSession({
       hostname,

src/commands/usage.ts

@@ -540,7 +540,7 @@ function resolveHostProcessKind(opts: {
     return "hack tui";
   }
   if (normalized.includes(" hack remote")) {
-    return "hack tui";
+    return "hack remote";
   }
   if (normalized.includes(" hack logs")) {
     return "log-stream";

src/control-plane/extensions/supervisor/shell-service.ts

@@ -368,7 +368,7 @@ function buildShellEnv(opts: {
       env[key] = value;
     }
   }
-  env.TERM = process.env.TERM ?? DEFAULT_TERM;
+  env.TERM = env.TERM || process.env.TERM || DEFAULT_TERM;
   return env;
 }
 

src/daemon/routes/sessions.ts

@@ -389,7 +389,7 @@ function parseSessionCreateInput(
     return { ok: false, error: "missing_name" };
   }
 
-  // Validate session name (tmux restrictions)
+  // Validate session name (alphanumeric, dash, underscore, dot)
   const trimmedName = name.trim();
   if (!SESSION_NAME_PATTERN.test(trimmedName)) {
     return {

src/daemon/runtime-cache.ts

@@ -223,12 +223,16 @@ export function createRuntimeCache(opts: {
             if (!reg) {
               return null;
             }
-            return await resolveProjectMeta({
-              projectName: reg.name,
-              repoRoot: reg.repoRoot,
-              projectDir: reg.projectDir,
-              composeFile: resolve(reg.projectDir, PROJECT_COMPOSE_FILENAME),
-            });
+            try {
+              return await resolveProjectMeta({
+                projectName: reg.name,
+                repoRoot: reg.repoRoot,
+                projectDir: reg.projectDir,
+                composeFile: resolve(reg.projectDir, PROJECT_COMPOSE_FILENAME),
+              });
+            } catch {
+              return null;
+            }
           })
         )
       : [];

tests/daemon-env.test.ts

@@ -92,7 +92,7 @@ describe("handleEnvRoutes", () => {
     if (originalConfigPath !== undefined) {
       process.env.HACK_GLOBAL_CONFIG_PATH = originalConfigPath;
     } else {
-      process.env.HACK_GLOBAL_CONFIG_PATH = undefined;
+      Reflect.deleteProperty(process.env, "HACK_GLOBAL_CONFIG_PATH");
     }
     await rm(tempDir, { recursive: true, force: true });
   });