chore: merge review-fix packet (demo line-0 guard)

Author: kathiekiwi

apps/webapp/app/components/dashboard-agent/demo/demo.test.ts

@@ -16,7 +16,7 @@ import { join } from "node:path";
 import { describe, expect, it } from "vitest";
 import { resolveSuggestedPrompts } from "../suggested-prompts";
 import * as fixtures from "./fixtures";
-import { DEMO_ID_PREFIX, DEMO_MARKER } from "./ids";
+import { DEMO_ID_PREFIX, DEMO_MARKER, demoSourceUri } from "./ids";
 
 const DEMO_DIR = __dirname;
 
@@ -70,6 +70,12 @@ describe("demo ids", () => {
       expect(id).toContain(DEMO_MARKER);
     }
   });
+
+  it("rejects a zero source line instead of dropping it", () => {
+    expect(() => demoSourceUri("abc", "src/a.ts", 0)).toThrow(/positive integer/);
+    expect(demoSourceUri("abc", "src/a.ts", 42)).toContain("?line=42");
+    expect(demoSourceUri("abc", "src/a.ts")).not.toContain("line=");
+  });
 });
 
 describe("view block fixtures", () => {

apps/webapp/app/components/dashboard-agent/demo/ids.ts

@@ -44,7 +44,13 @@ export function demoReportUri(key: string): TriggerUri {
 }
 
 export function demoSourceUri(sha: string, path: string, line?: number): TriggerUri {
-  return formatTriggerUri({ kind: "source", ...scope, sha, path, ...(line ? { line } : {}) });
+  return formatTriggerUri({
+    kind: "source",
+    ...scope,
+    sha,
+    path,
+    ...(line !== undefined ? { line } : {}),
+  });
 }
 
 export function demoInvestigationUri(investigationId: string): TriggerUri {