@grackle-ai/web-components 0.115.1 → 0.116.0

package/src/index.ts

@@ -55,7 +55,7 @@ export { StatusBar, AppNav, Sidebar, BottomStatusBar, TABS } from "./components/
 export type { AppTab } from "./components/layout/index.js";
 
 // Lists
-export { EnvironmentNav, FindingsNav } from "./components/lists/index.js";
+export { EnvironmentNav } from "./components/lists/index.js";
 export { TaskList } from "./components/lists/TaskList.js";
 export { HighlightedText, buildTaskTree, groupTasksByStatus } from "./components/lists/listHelpers.js";
 
@@ -65,7 +65,7 @@ export type { CalloutVariant } from "./components/notifications/index.js";
 export { UpdateBanner } from "./components/notifications/UpdateBanner.js";
 
 // Panels
-export { FindingsPanel, TokensPanel, AppearancePanel, AboutPanel, TaskEditPanel, TaskActionButtons, TaskOverviewPanel, PluginsPanel, GitHubAccountsPanel } from "./components/panels/index.js";
+export { TokensPanel, AppearancePanel, AboutPanel, TaskEditPanel, TaskActionButtons, TaskOverviewPanel, PluginsPanel, GitHubAccountsPanel } from "./components/panels/index.js";
 export type { PluginsPanelProps, GitHubAccountsPanelProps } from "./components/panels/index.js";
 export type { TaskActionButtonsProps } from "./components/panels/TaskActionButtons.js";
 export type { TaskOverviewPanelProps } from "./components/panels/TaskOverviewPanel.js";
@@ -126,13 +126,13 @@ export type { UseGrackleSocketResult, GrackleContextType } from "./context/Grack
 
 export type {
   Environment, Session, UsageStats, SessionEvent,
-  Workspace, TaskData, FindingData, TokenInfo,
+  Workspace, TaskData, TokenInfo,
   CredentialProviderConfig, Codespace, DockerContainer, PersonaData,
   ScheduleData, ScheduleUpdate, UseSchedulesResult,
   ProvisionStatus, GrackleEvent, WsMessage, SendFunction,
   GraphNode, GraphLink, NodeDetail, UseKnowledgeResult,
   UseEnvironmentsResult, UseSessionsResult, UseWorkspacesResult,
-  UseTasksResult, UseFindingsResult, UseTokensResult,
+  UseTasksResult, UseTokensResult,
   UseCredentialsResult, UseCodespacesResult, UseDockerContainersResult, UsePersonasResult,
   UsePluginsResult, PluginData,
   StreamData, StreamSubscriberData, UseStreamsResult,
@@ -164,7 +164,6 @@ export {
   SCHEDULES_URL, NEW_SCHEDULE_URL, scheduleUrl,
   SETTINGS_APPEARANCE_URL, SETTINGS_ABOUT_URL, SETTINGS_SHORTCUTS_URL,
   PAIR_PATH, NEW_WORKSPACE_URL, KNOWLEDGE_URL, HOME_URL,
-  FINDINGS_URL, findingsUrl, findingUrl,
   CHAT_URL, COORDINATION_URL, TASKS_URL,
 } from "./utils/navigation.js";
 
@@ -177,8 +176,6 @@ export type { TaskStatusKey, VirtualStatus, DisplayStatus, TaskStatusStyle } fro
 
 export { formatTokens, formatCost } from "./utils/format.js";
 export { formatRelativeTime, formatCountdown } from "./utils/time.js";
-export { CATEGORY_COLORS, getCategoryColor } from "./utils/findingCategory.js";
-export type { CategoryColor } from "./utils/findingCategory.js";
 
 export type { BreadcrumbSegment } from "./utils/breadcrumbs.js";
 export {
@@ -186,7 +183,6 @@ export {
   buildEnvironmentsBreadcrumbs, buildNewEnvironmentBreadcrumbs,
   buildNewChatBreadcrumbs, buildSessionBreadcrumbs,
   buildWorkspaceBreadcrumbs, buildTaskBreadcrumbs, buildNewTaskBreadcrumbs,
-  buildFindingsBreadcrumbs, buildFindingBreadcrumbs,
 } from "./utils/breadcrumbs.js";
 
 export { groupConsecutiveTextEvents, pairToolEvents } from "./utils/sessionEvents.js";

package/src/mocks/MockGrackleProvider.tsx

@@ -24,7 +24,6 @@ import type {
   Environment,
   Session,
   SessionEvent,
-  FindingData,
   TaskData,
   Workspace,
   TokenInfo,
@@ -48,7 +47,6 @@ import {
   MOCK_EVENTS,
   MOCK_WORKSPACES,
   MOCK_TASKS,
-  MOCK_FINDINGS,
   MOCK_TOKENS,
   MOCK_PERSONAS,
   MOCK_TASK_SESSIONS,
@@ -97,9 +95,6 @@ export function MockGrackleProvider({ children }: MockGrackleProviderProps): JSX
   const [workspaces, setWorkspaces] = useState<Workspace[]>(MOCK_WORKSPACES);
   const [workspaceLinkError, setWorkspaceLinkError] = useState("");
   const [tasks, setTasks] = useState<TaskData[]>(MOCK_TASKS);
-  const [findings, setFindings] = useState<FindingData[]>(MOCK_FINDINGS);
-  const [selectedFinding, setSelectedFinding] = useState<FindingData | undefined>(undefined);
-  const findingLoading = false;
   const [tokens, setTokens] = useState<TokenInfo[]>(MOCK_TOKENS);
   const [credentialProviders, setCredentialProviders] = useState<CredentialProviderConfig>({
     claude: "off",
@@ -768,59 +763,6 @@ export function MockGrackleProvider({ children }: MockGrackleProviderProps): JSX
     [],
   );
 
-  /** Filters findings by workspaceId. */
-  const loadFindings: UseGrackleSocketResult["findings"]["loadFindings"] = useCallback(
-    async (workspaceId: string) => {
-      console.log("[MockGrackle] loadFindings", workspaceId);
-      setFindings(MOCK_FINDINGS.filter((f) => f.workspaceId === workspaceId));
-    },
-    [],
-  );
-
-  /** Load all findings across all workspaces. */
-  const loadAllFindings: UseGrackleSocketResult["findings"]["loadAllFindings"] = useCallback(async () => {
-    console.log("[MockGrackle] loadAllFindings");
-    setFindings([...MOCK_FINDINGS]);
-  }, []);
-
-  /** Load a single finding by ID. */
-  const loadFinding: UseGrackleSocketResult["findings"]["loadFinding"] = useCallback(
-    async (findingId: string) => {
-      console.log("[MockGrackle] loadFinding", findingId);
-      const found = MOCK_FINDINGS.find((f) => f.id === findingId);
-      setSelectedFinding(found);
-    },
-    [],
-  );
-
-  /** Adds a new finding to state. */
-  const postFinding: UseGrackleSocketResult["findings"]["postFinding"] = useCallback(
-    async (
-      workspaceId: string,
-      title: string,
-      content: string,
-      category?: string,
-      tags?: string[],
-    ) => {
-      console.log("[MockGrackle] postFinding", { workspaceId, title });
-
-      const newFinding: FindingData = {
-        id: nextId("find"),
-        workspaceId,
-        taskId: "",
-        sessionId: "",
-        category: category || "general",
-        title,
-        content,
-        tags: tags || [],
-        createdAt: new Date().toISOString(),
-      };
-
-      setFindings((prev) => [...prev, newFinding]);
-    },
-    [nextId],
-  );
-
   /** No-op in mock mode (environments are pre-seeded). */
   const loadEnvironments: UseGrackleSocketResult["environments"]["loadEnvironments"] = useCallback(async () => {
     console.log("[MockGrackle] loadEnvironments");
@@ -1040,17 +982,6 @@ export function MockGrackleProvider({ children }: MockGrackleProviderProps): JSX
         domainHook: NOOP_DOMAIN_HOOK,
       },
 
-      findings: {
-        findings,
-        selectedFinding,
-        findingLoading,
-        findingsLoading: false,
-        loadFindings,
-        loadAllFindings,
-        loadFinding,
-        postFinding,
-        domainHook: NOOP_DOMAIN_HOOK,
-      },
 
       tokens: {
         tokens,
@@ -1389,8 +1320,6 @@ export function MockGrackleProvider({ children }: MockGrackleProviderProps): JSX
       workspaces,
       workspaceLinkError,
       tasks,
-      findings,
-      selectedFinding,
       tokens,
       credentialProviders,
       personas,
@@ -1416,10 +1345,6 @@ export function MockGrackleProvider({ children }: MockGrackleProviderProps): JSX
       resumeTask,
       updateTask,
       deleteTask,
-      loadFindings,
-      loadAllFindings,
-      loadFinding,
-      postFinding,
       loadEnvironments,
       addEnvironment,
       updateEnvironment,

package/src/mocks/mockData.ts

@@ -2,8 +2,8 @@
  * Static mock data for visual testing (`?mock` mode).
  *
  * Provides realistic sample entities that exercise every UI state:
- * multiple environments, sessions in various statuses, workspaces with
- * tasks at different lifecycle stages, and findings across all categories.
+ * multiple environments, sessions in various statuses, and workspaces with
+ * tasks at different lifecycle stages.
  */
 
 import type {
@@ -12,7 +12,6 @@ import type {
   SessionEvent,
   Workspace,
   TaskData,
-  FindingData,
   TokenInfo,
   PersonaData,
 } from "../hooks/types.js";
@@ -1322,96 +1321,6 @@ export const MOCK_TASKS: TaskData[] = [
   },
 ];
 
-// ─── Findings ───────────────────────────────────────
-
-/** Sample findings across every category to exercise the FindingsPanel styling. */
-export const MOCK_FINDINGS: FindingData[] = [
-  {
-    id: "find-001",
-    workspaceId: "proj-alpha",
-    taskId: "task-001",
-    sessionId: "sess-001",
-    category: "architecture",
-    title: "Auth middleware is tightly coupled to Express",
-    content:
-      "The current auth middleware directly references Express Request/Response types. Consider extracting a framework-agnostic token verification layer so we can reuse it in the WebSocket auth path.",
-    tags: ["auth", "decoupling", "middleware"],
-    createdAt: "2026-02-27T08:16:00Z",
-  },
-  {
-    id: "find-002",
-    workspaceId: "proj-alpha",
-    taskId: "task-003",
-    sessionId: "sess-002",
-    category: "api",
-    title: "Missing pagination on GET /api/users",
-    content:
-      "The users endpoint returns all rows without limit/offset. For datasets over 10k rows this will cause timeouts. Recommend cursor-based pagination with a default page size of 50.",
-    tags: ["api", "pagination", "performance"],
-    createdAt: "2026-02-27T07:31:00Z",
-  },
-  {
-    id: "find-003",
-    workspaceId: "proj-alpha",
-    taskId: "task-005",
-    sessionId: "sess-003",
-    category: "bug",
-    title: "Race condition in session cleanup",
-    content:
-      "When two requests hit /api/logout concurrently, the second call throws a 500 because the session row has already been deleted. Needs an idempotent DELETE or a conditional check.",
-    tags: ["bug", "concurrency", "sessions"],
-    createdAt: "2026-02-26T22:50:00Z",
-  },
-  {
-    id: "find-004",
-    workspaceId: "proj-alpha",
-    taskId: "task-004",
-    sessionId: "",
-    category: "decision",
-    title: "Chose pg-pool over knex connection pool",
-    content:
-      "pg-pool gives us direct control over idle timeout, max connections, and health check queries. Knex wraps pg-pool anyway and adds overhead we don't need since we write raw SQL.",
-    tags: ["database", "decision", "postgres"],
-    createdAt: "2026-02-23T11:30:00Z",
-  },
-  {
-    id: "find-005",
-    workspaceId: "proj-alpha",
-    taskId: "",
-    sessionId: "",
-    category: "dependency",
-    title: "jsonwebtoken has 3 high-severity CVEs",
-    content:
-      "The jsonwebtoken package (v8.x) has known vulnerabilities. Consider migrating to jose which is maintained, supports ESM, and covers the same JWS/JWE surface area with zero dependencies.",
-    tags: ["security", "dependency", "jwt"],
-    createdAt: "2026-02-27T08:20:00Z",
-  },
-  {
-    id: "find-006",
-    workspaceId: "proj-alpha",
-    taskId: "task-001",
-    sessionId: "sess-001",
-    category: "pattern",
-    title: "Consistent error response shape",
-    content:
-      'All error responses should follow the shape `{ error: string, code: string, details?: unknown }`. Currently some routes return `{ message: string }` and others return `{ error: string }`.',
-    tags: ["api", "consistency", "error-handling"],
-    createdAt: "2026-02-27T08:17:00Z",
-  },
-  {
-    id: "find-007",
-    workspaceId: "proj-beta",
-    taskId: "task-007",
-    sessionId: "sess-004",
-    category: "architecture",
-    title: "Watermark storage should be pluggable",
-    content:
-      "The incremental load watermarks are currently stored in a local SQLite file. For production multi-worker scenarios, this needs to be backed by a shared store (Redis or Postgres).",
-    tags: ["architecture", "pipeline", "scalability"],
-    createdAt: "2026-02-27T09:05:00Z",
-  },
-];
-
 // ─── Tokens ──────────────────────────────────────────
 
 /** Sample tokens for the settings panel. */
@@ -1457,7 +1366,7 @@ export const MOCK_PERSONAS: PersonaData[] = [
     updatedAt: "2026-02-20T10:00:00Z",
     type: "agent",
     script: "",
-    allowedMcpTools: ["task_create", "task_update", "task_list", "finding_post", "finding_list", "knowledge_search", "knowledge_create_node", "session_spawn"],
+    allowedMcpTools: ["task_create", "task_update", "task_list", "knowledge_search", "session_spawn"],
   },
   {
     id: "persona-fe",
@@ -1478,8 +1387,8 @@ export const MOCK_PERSONAS: PersonaData[] = [
   {
     id: "persona-reviewer",
     name: "Code Reviewer",
-    description: "Reviews diffs for correctness, security, and style. Posts findings for issues discovered.",
-    systemPrompt: "You are a meticulous code reviewer. Check for security vulnerabilities, performance issues, and style consistency. Post findings for anything noteworthy.",
+    description: "Reviews diffs for correctness, security, and style. Reports issues discovered.",
+    systemPrompt: "You are a meticulous code reviewer. Check for security vulnerabilities, performance issues, and style consistency. Report anything noteworthy.",
     toolConfig: JSON.stringify({ allowedTools: ["Read", "Grep", "Glob"] }),
     runtime: "claude-code",
     model: "claude-sonnet-4-6",
@@ -1489,7 +1398,7 @@ export const MOCK_PERSONAS: PersonaData[] = [
     updatedAt: "2026-02-20T10:10:00Z",
     type: "agent",
     script: "",
-    allowedMcpTools: ["finding_post", "finding_list", "knowledge_search"],
+    allowedMcpTools: ["knowledge_search"],
   },
   {
     id: "persona-tester",
@@ -1510,7 +1419,7 @@ export const MOCK_PERSONAS: PersonaData[] = [
   {
     id: "persona-lint",
     name: "Lint & Format",
-    description: "Scripted persona — runs ESLint and Prettier on changed files, auto-fixes violations, and posts a findings summary.",
+    description: "Scripted persona — runs ESLint and Prettier on changed files, auto-fixes violations, and logs a summary.",
     systemPrompt: "",
     toolConfig: "{}",
     runtime: "genaiscript",
@@ -1520,7 +1429,7 @@ export const MOCK_PERSONAS: PersonaData[] = [
     createdAt: "2026-02-21T09:00:00Z",
     updatedAt: "2026-02-21T09:00:00Z",
     type: "script",
-    script: 'const files = env.files.filter(f => /\\.(ts|tsx|js)$/.test(f.filename));\nfor (const f of files) {\n  const eslintResult = await host.exec("npx", ["eslint", "--fix", f.filename]);\n  if (eslintResult.exitCode !== 0) {\n    env.findings.push({ category: "lint", title: `Lint issues in ${f.filename}`, content: eslintResult.stderr });\n  }\n  const prettierResult = await host.exec("npx", ["prettier", "--write", f.filename]);\n  if (prettierResult.exitCode !== 0) {\n    env.findings.push({ category: "format", title: `Prettier issues in ${f.filename}`, content: prettierResult.stderr });\n  }\n}\nenv.findings.push({ category: "summary", title: "Lint & format pass complete", content: `Ran ESLint and Prettier on ${files.length} files` });',
+    script: 'const files = env.files.filter(f => /\\.(ts|tsx|js)$/.test(f.filename));\nfor (const f of files) {\n  const eslintResult = await host.exec("npx", ["eslint", "--fix", f.filename]);\n  if (eslintResult.exitCode !== 0) {\n    console.log(`Lint issues in ${f.filename}: ${eslintResult.stderr}`);\n  }\n  const prettierResult = await host.exec("npx", ["prettier", "--write", f.filename]);\n  if (prettierResult.exitCode !== 0) {\n    console.log(`Prettier issues in ${f.filename}: ${prettierResult.stderr}`);\n  }\n}\nconsole.log(`Ran ESLint and Prettier on ${files.length} files`);',
     allowedMcpTools: [],
   },
 ];

package/src/test-utils/storybook-helpers.ts

@@ -11,7 +11,6 @@ import type {
   SessionEvent,
   Workspace,
   TaskData,
-  FindingData,
   TokenInfo,
   PersonaData,
   ScheduleData,
@@ -25,7 +24,6 @@ export {
   MOCK_SESSIONS,
   MOCK_WORKSPACES,
   MOCK_TASKS,
-  MOCK_FINDINGS,
   MOCK_TOKENS,
   MOCK_PERSONAS,
 } from "../mocks/mockData.js";
@@ -111,22 +109,6 @@ export function makeTask(overrides: Partial<TaskData> = {}): TaskData {
   };
 }
 
-/** Create a FindingData with sensible defaults. */
-export function makeFinding(overrides: Partial<FindingData> = {}): FindingData {
-  return {
-    id: nextId("finding"),
-    workspaceId: "ws-1",
-    taskId: "task-1",
-    sessionId: "sess-1",
-    category: "general",
-    title: "Test Finding",
-    content: "Finding content",
-    tags: [],
-    createdAt: "2026-01-01T00:00:00Z",
-    ...overrides,
-  };
-}
-
 /** Create a TokenInfo with sensible defaults. */
 export function makeToken(overrides: Partial<TokenInfo> = {}): TokenInfo {
   return {
@@ -253,7 +235,6 @@ export const buildEnvironment: typeof makeEnvironment = makeEnvironment;
 export const buildSession: typeof makeSession = makeSession;
 export const buildWorkspace: typeof makeWorkspace = makeWorkspace;
 export const buildTask: typeof makeTask = makeTask;
-export const buildFinding: typeof makeFinding = makeFinding;
 export const buildToken: typeof makeToken = makeToken;
 export const buildPersona: typeof makePersona = makePersona;
 export const buildEvent: typeof makeEvent = makeEvent;

package/src/utils/breadcrumbs.test.ts

@@ -8,8 +8,6 @@ import {
   buildNewTaskBreadcrumbs,
   buildNewChatBreadcrumbs,
   buildSessionBreadcrumbs,
-  buildFindingsBreadcrumbs,
-  buildFindingBreadcrumbs,
   type BreadcrumbSegment,
 } from "./breadcrumbs.js";
 import type { Environment, TaskData, Workspace } from "../hooks/types.js";
@@ -228,47 +226,6 @@ describe("breadcrumb builders", () => {
   });
 });
 
-describe("findings breadcrumb builders", () => {
-  it("buildFindingsBreadcrumbs returns Home > Findings (non-clickable)", () => {
-    const segments: BreadcrumbSegment[] = buildFindingsBreadcrumbs();
-    expect(segments).toHaveLength(2);
-    expect(segments[0].label).toBe("Home");
-    expect(segments[0].url).toBe("/");
-    expect(segments[1].label).toBe("Findings");
-    expect(segments[1].url).toBeUndefined();
-  });
-
-  it("buildFindingBreadcrumbs without scope returns Home > Findings > Title", () => {
-    const segments: BreadcrumbSegment[] = buildFindingBreadcrumbs("My Finding", undefined, undefined, [], []);
-    expect(segments).toHaveLength(3);
-    expect(segments[0].label).toBe("Home");
-    expect(segments[0].url).toBe("/");
-    expect(segments[1].label).toBe("Findings");
-    expect(segments[1].url).toBe("/findings");
-    expect(segments[2].label).toBe("My Finding");
-    expect(segments[2].url).toBeUndefined();
-  });
-
-  it("buildFindingBreadcrumbs with workspace scope returns full chain", () => {
-    const workspaces: Workspace[] = [makeWorkspace("ws1", "My Workspace", "env1")];
-    const environments: Environment[] = [makeEnvironment("env1", "Local Dev")];
-    const segments: BreadcrumbSegment[] = buildFindingBreadcrumbs("My Finding", "ws1", "env1", workspaces, environments);
-    expect(segments).toHaveLength(6);
-    expect(segments[0].label).toBe("Home");
-    expect(segments[0].url).toBe("/");
-    expect(segments[1].label).toBe("Environments");
-    expect(segments[1].url).toBe("/environments");
-    expect(segments[2].label).toBe("Local Dev");
-    expect(segments[2].url).toBe("/environments/env1");
-    expect(segments[3].label).toBe("My Workspace");
-    expect(segments[3].url).toBe("/environments/env1/workspaces/ws1");
-    expect(segments[4].label).toBe("Findings");
-    expect(segments[4].url).toBe("/environments/env1/workspaces/ws1/findings");
-    expect(segments[5].label).toBe("My Finding");
-    expect(segments[5].url).toBeUndefined();
-  });
-});
-
 describe("task status helpers", () => {
   it("maps legacy task statuses to canonical styles", () => {
     expect(getStatusStyle("pending").label).toBe("Not Started");

package/src/utils/breadcrumbs.ts

@@ -1,5 +1,5 @@
 import type { Environment, TaskData, Workspace } from "../hooks/types.js";
-import { ENVIRONMENTS_URL, environmentUrl, FINDINGS_URL, findingsUrl, HOME_URL, SETTINGS_URL, taskUrl, workspaceUrl } from "./navigation.js";
+import { ENVIRONMENTS_URL, environmentUrl, HOME_URL, SETTINGS_URL, taskUrl, workspaceUrl } from "./navigation.js";
 
 /** A single segment in the breadcrumb trail. */
 export interface BreadcrumbSegment {
@@ -184,39 +184,3 @@ export function buildNewTaskBreadcrumbs(
   return segments;
 }
 
-/** Findings breadcrumb segment. */
-const FINDINGS_SEGMENT: BreadcrumbSegment = { label: "Findings", url: FINDINGS_URL };
-
-/** Build breadcrumbs for the findings landing page. */
-export function buildFindingsBreadcrumbs(): BreadcrumbSegment[] {
-  return [HOME_SEGMENT, { label: "Findings", url: undefined }];
-}
-
-/** Build breadcrumbs for a finding detail page, optionally scoped to a workspace. */
-export function buildFindingBreadcrumbs(
-  findingTitle: string,
-  workspaceId: string | undefined,
-  environmentId: string | undefined,
-  workspaces: Workspace[],
-  environments: Environment[],
-): BreadcrumbSegment[] {
-  const segments: BreadcrumbSegment[] = [HOME_SEGMENT];
-
-  if (workspaceId && environmentId) {
-    const workspace = workspaces.find((p) => p.id === workspaceId);
-    const environment = environments.find((e) => e.id === environmentId);
-    segments.push(ENVIRONMENTS_SEGMENT);
-    if (environment) {
-      segments.push({ label: environment.displayName, url: environmentUrl(environmentId) });
-    }
-    if (workspace) {
-      segments.push({ label: workspace.name, url: workspaceUrl(workspaceId, environmentId) });
-    }
-    segments.push({ label: "Findings", url: findingsUrl(workspaceId, environmentId) });
-  } else {
-    segments.push(FINDINGS_SEGMENT);
-  }
-
-  segments.push({ label: findingTitle, url: undefined });
-  return segments;
-}

package/src/utils/navigation.ts

@@ -45,7 +45,7 @@ export function workspaceUrl(workspaceId: string, environmentId?: string): strin
 }
 
 /** Build URL for a task detail page, optionally targeting a specific tab and workspace/environment scope. */
-export function taskUrl(taskId: string, tab?: "stream" | "findings", workspaceId?: string, environmentId?: string): string {
+export function taskUrl(taskId: string, tab?: "stream", workspaceId?: string, environmentId?: string): string {
   const encodedTaskId = encodeURIComponent(taskId);
   let base: string;
   if (workspaceId && environmentId) {
@@ -180,25 +180,6 @@ export const NEW_WORKSPACE_URL: string = "/workspaces/new";
 /** URL for the knowledge graph explorer page. */
 export const KNOWLEDGE_URL: string = "/knowledge";
 
-/** URL for the findings landing page. */
-export const FINDINGS_URL: string = "/findings";
-
-/** Build URL for the findings list page, optionally scoped to a workspace. */
-export function findingsUrl(workspaceId?: string, environmentId?: string): string {
-  if (workspaceId && environmentId) {
-    return `/environments/${encodeURIComponent(environmentId)}/workspaces/${encodeURIComponent(workspaceId)}/findings`;
-  }
-  return FINDINGS_URL;
-}
-
-/** Build URL for a finding detail page, optionally scoped to a workspace. */
-export function findingUrl(findingId: string, workspaceId?: string, environmentId?: string): string {
-  const encodedFindingId = encodeURIComponent(findingId);
-  if (workspaceId && environmentId) {
-    return `/environments/${encodeURIComponent(environmentId)}/workspaces/${encodeURIComponent(workspaceId)}/findings/${encodedFindingId}`;
-  }
-  return `/findings/${encodedFindingId}`;
-}
 
 /** Build URL for the root-task chat page. */
 export function chatUrl(): string {

package/src/utils/route-config.test.ts

@@ -7,15 +7,12 @@ import {
   newTaskUrl,
   newChatUrl,
   personaUrl,
-  findingUrl,
-  findingsUrl,
   HOME_URL,
   SETTINGS_URL,
   PERSONAS_URL,
   NEW_PERSONA_URL,
   NEW_ENVIRONMENT_URL,
   NEW_WORKSPACE_URL,
-  FINDINGS_URL,
 } from "./navigation.js";
 
 describe("URL builder functions", () => {
@@ -38,10 +35,6 @@ describe("URL builder functions", () => {
     expect(taskUrl("task-1", "stream")).toBe("/tasks/task-1/stream");
   });
 
-  it("taskUrl with findings tab produces correct path", () => {
-    expect(taskUrl("task-1", "findings")).toBe("/tasks/task-1/findings");
-  });
-
   it("taskUrl encodes taskId", () => {
     expect(taskUrl("has space")).toBe("/tasks/has%20space");
     expect(taskUrl("has space", "stream")).toBe("/tasks/has%20space/stream");
@@ -50,7 +43,6 @@ describe("URL builder functions", () => {
   it("taskUrl with workspace and environment produces environment-scoped path", () => {
     expect(taskUrl("task-1", undefined, "ws-1", "env-1")).toBe("/environments/env-1/workspaces/ws-1/tasks/task-1");
     expect(taskUrl("task-1", "stream", "ws-1", "env-1")).toBe("/environments/env-1/workspaces/ws-1/tasks/task-1/stream");
-    expect(taskUrl("task-1", "findings", "ws-1", "env-1")).toBe("/environments/env-1/workspaces/ws-1/tasks/task-1/findings");
   });
 
   it("taskUrl with only workspaceId (no environmentId) falls back to legacy workspace path", () => {
@@ -102,27 +94,4 @@ describe("URL builder functions", () => {
     expect(personaUrl("has space")).toBe("/settings/personas/has%20space");
   });
 
-  it("FINDINGS_URL constant is correct", () => {
-    expect(FINDINGS_URL).toBe("/findings");
-  });
-
-  it("findingUrl produces base path without scope", () => {
-    expect(findingUrl("f1")).toBe("/findings/f1");
-  });
-
-  it("findingUrl with workspace and environment produces scoped path", () => {
-    expect(findingUrl("f1", "ws1", "env1")).toBe("/environments/env1/workspaces/ws1/findings/f1");
-  });
-
-  it("findingUrl encodes special characters", () => {
-    expect(findingUrl("has space")).toBe("/findings/has%20space");
-  });
-
-  it("findingsUrl without scope returns base findings path", () => {
-    expect(findingsUrl()).toBe("/findings");
-  });
-
-  it("findingsUrl with workspace and environment produces scoped path", () => {
-    expect(findingsUrl("ws1", "env1")).toBe("/environments/env1/workspaces/ws1/findings");
-  });
 });