Merge pull request #5438 from nodetool-ai/claude/remove-dead-code-1788169398

Author: georgi

web/src/components/appbuilder/merge.ts

@@ -19,7 +19,7 @@ import type {
 import { mergeByUnits } from "../../stores/documentMerge";
 
 /** One flattened component: the Puck node plus where it hangs. */
-export interface FlatComponent {
+interface FlatComponent {
   node: { type: string; props: Record<string, unknown> & { id: string } };
   parentId: string | null;
   slot: string | null;
@@ -31,7 +31,7 @@ export interface FlatComponent {
  * Puck tree on the way out (the adapter's write rebuilds it), so a merged
  * result drops straight back into `ui.content`.
  */
-export interface AppMergeDoc {
+interface AppMergeDoc {
   content: unknown[];
   operations: unknown[];
   variables: unknown[];
@@ -67,7 +67,7 @@ type ComponentNodeLike = {
 };
 
 /** Flatten a Puck content tree into per-component merge units. */
-export function flattenAppComponents(content: unknown[]): FlatComponent[] {
+function flattenAppComponents(content: unknown[]): FlatComponent[] {
   const out: FlatComponent[] = [];
   const walk = (
     items: AnyNode[],
@@ -107,7 +107,7 @@ export function flattenAppComponents(content: unknown[]): FlatComponent[] {
  * Rebuild the Puck content tree from merged flat units. Roots keep their
  * merged order; children hang off their parent's slot in merged order.
  */
-export function rebuildAppComponents(flat: FlatComponent[]): unknown[] {
+function rebuildAppComponents(flat: FlatComponent[]): unknown[] {
   // Drop units whose parent is gone (recursively: a dropped child can
   // orphan its own children).
   let kept = [...flat];
@@ -170,7 +170,7 @@ const named = (unit: unknown): string => {
   return String(unitRecord.name ?? byId(unit));
 };
 
-export const appMergeAdapter: DocumentMergeAdapter<AppMergeDoc> = {
+const appMergeAdapter: DocumentMergeAdapter<AppMergeDoc> = {
   collections: [
     {
       kind: "component",

web/src/components/chat/message/toolCallPhrase.ts

@@ -12,7 +12,7 @@ import { formatToolName } from "../../../utils/formatUtils";
 import { isObjectLike, isString } from "../../../utils/typePredicates";
 
 /** What a tool does, as far as the row phrasing is concerned. */
-export type ToolPhraseKind =
+type ToolPhraseKind =
   | "search"
   | "page"
   | "read"
@@ -74,7 +74,7 @@ const DETAIL_KEYS = [
 ] as const;
 
 /** A single row's label plus the mono-rendered thing it acted on. */
-export interface ToolRowPhrase {
+interface ToolRowPhrase {
   label: string;
   detail: string | null;
 }
@@ -114,7 +114,7 @@ export function toolCallDetail(
 const LOCATION_KEYS = ["url", "uri", "path", "file", "filename"] as const;
 
 /** Whether a call's detail identifies a location rather than free text. */
-export function hasLocationDetail(call: ToolCall): boolean {
+function hasLocationDetail(call: ToolCall): boolean {
   const args = call.args;
   if (!isObjectLike(args)) {
     return false;
@@ -144,7 +144,7 @@ function compactUrl(value: string): string {
 }
 
 /** How a run of same-tool calls renders: one row each, or one counted row. */
-export type RunDisplay = "list" | "count";
+type RunDisplay = "list" | "count";
 
 /** Above this, a run is always counted — a wall of rows is not a timeline. */
 const MAX_LISTED_RUN = 4;

web/src/components/projects/projectStatus.ts

@@ -11,7 +11,7 @@ import type { RouterOutputs } from "../../trpc/client";
 
 export type ProjectDetail = RouterOutputs["projects"]["summaries"][number];
 export type ProjectDocument = ProjectDetail["documents"][number];
-export type ProjectDocumentStatus = NonNullable<ProjectDocument["status"]>;
+type ProjectDocumentStatus = NonNullable<ProjectDocument["status"]>;
 
 const statusOfKind = <K extends ProjectDocumentStatus["kind"]>(
   documents: readonly ProjectDocument[],
@@ -78,7 +78,7 @@ export const projectStatusLine = (
   return parts.join(" · ") + partialSuffix;
 };
 
-export interface ProjectProgress {
+interface ProjectProgress {
   label: string;
   /** Everything the board set out to render exists. */
   done: boolean;
@@ -132,7 +132,7 @@ export const documentStatusLine = (document: ProjectDocument): string => {
   }
 };
 
-export interface DocumentProgress {
+interface DocumentProgress {
   label: string;
   tone: "done" | "neutral" | "rendering";
 }
@@ -183,7 +183,7 @@ export const formatDocumentSpend = (document: ProjectDocument): string => {
 };
 
 /** The step a project is waiting on, and the document that performs it. */
-export interface ProjectNextStep {
+interface ProjectNextStep {
   label: string;
   document: ProjectDocument;
 }

web/src/components/sketch/sam/SamServiceNode.ts

@@ -55,7 +55,7 @@ const ACTIVE_DOWNLOAD_STATUSES = new Set([
   "progress"
 ]);
 
-export interface SamNodeConfig {
+interface SamNodeConfig {
   backendId: SegmentBackend;
   nodeType: string;
   displayName: string;
@@ -65,7 +65,7 @@ export interface SamNodeConfig {
   requiredSecret: string | null;
 }
 
-export const SAM_NODE_CONFIGS: Record<string, SamNodeConfig> = {
+const SAM_NODE_CONFIGS: Record<string, SamNodeConfig> = {
   "local-sam3": {
     backendId: "local-sam3",
     nodeType: LOCAL_SAM3_NODE_TYPE,
@@ -86,7 +86,7 @@ export const SAM_NODE_CONFIGS: Record<string, SamNodeConfig> = {
   }
 };
 
-export const DEFAULT_SAM_NODE_BACKEND = "local-sam3";
+const DEFAULT_SAM_NODE_BACKEND = "local-sam3";
 
 interface LocalSam3PromptMetadata {
   capabilities: SamBackendCapabilities;

web/src/components/sketch/sketchCanvasHooks/DisplayFrameCoordinator.ts

@@ -85,12 +85,12 @@ export type RedrawReason =
  * - `immediate`: composite synchronously (low-latency path for direct drawing)
  * - `raf`: schedule via requestAnimationFrame (batched, coalesced)
  */
-export type RedrawUrgency = "immediate" | "raf";
+type RedrawUrgency = "immediate" | "raf";
 
 /**
  * A typed redraw request that records why and how urgently a frame is needed.
  */
-export interface RedrawRequest {
+interface RedrawRequest {
   reason: RedrawReason;
   urgency: RedrawUrgency;
   /** Optional dirty rect for partial compositing. */
@@ -106,12 +106,12 @@ export interface RedrawRequest {
  * - `bootstrap`: Canvas2D temp surface while WebGPU initializes
  * - `display`: real display canvas (WebGPU or Canvas2D final)
  */
-export type DisplayTarget = "bootstrap" | "display";
+type DisplayTarget = "bootstrap" | "display";
 
 /**
  * Which rendering backend is active.
  */
-export type DisplayBackend = "webgpu" | "canvas2d";
+type DisplayBackend = "webgpu" | "canvas2d";
 
 // ─── Interaction readiness ───────────────────────────────────────────────────
 
@@ -121,7 +121,7 @@ export type DisplayBackend = "webgpu" | "canvas2d";
  * When all flags are true, the display pipeline is ready for preview-only
  * and click-only interactions without requiring a prior stroke.
  */
-export interface InteractionReadiness {
+interface InteractionReadiness {
   /** The runtime (Canvas2D or WebGPU) has been initialized. */
   runtimeReady: boolean;
   /** A hydration cycle has been scheduled and is still pending decode/upload. */
@@ -135,7 +135,7 @@ export interface InteractionReadiness {
 /**
  * Returns true when all readiness conditions are met.
  */
-export function isInteractionReady(state: InteractionReadiness): boolean {
+function isInteractionReady(state: InteractionReadiness): boolean {
   return (
     state.runtimeReady &&
     !state.hydrationPending &&
@@ -147,7 +147,7 @@ export function isInteractionReady(state: InteractionReadiness): boolean {
 /**
  * Create a fresh readiness state (nothing ready yet).
  */
-export function createInitialReadiness(): InteractionReadiness {
+function createInitialReadiness(): InteractionReadiness {
   return {
     runtimeReady: false,
     hydrationPending: false,
@@ -189,7 +189,7 @@ const MAX_TRACE_EVENTS = 200;
  * Stores the last N events so temporal startup bugs can be debugged
  * without scattering temporary logs across tools and runtimes.
  */
-export class DisplayTracer {
+class DisplayTracer {
   private events: TraceEvent[] = [];
   private enabled: boolean;
 
@@ -234,7 +234,7 @@ export class DisplayTracer {
 
 // ─── Frame coordinator ───────────────────────────────────────────────────────
 
-export interface FrameCoordinatorCallbacks {
+interface FrameCoordinatorCallbacks {
   /** Execute pending stroke buffer merge. */
   drainPendingStroke: () => void;
   /** Run the composite pipeline immediately. */

web/src/components/sketch/sketchClipboard.ts

@@ -15,7 +15,7 @@ import type { Layer, Point, Selection } from "./types";
 /**
  * For copy/export: zero or scale RGBA alpha by the document-space mask inside `bounds`.
  */
-export function multiplyImageDataAlphaBySelectionMask(
+function multiplyImageDataAlphaBySelectionMask(
   imageData: ImageData,
   bounds: { x: number; y: number; width: number; height: number },
   sel: Selection
@@ -35,7 +35,7 @@ export function multiplyImageDataAlphaBySelectionMask(
 }
 
 /** Best-effort: decode the first image on the system clipboard into a canvas. */
-export async function readSystemClipboardImageCanvas(): Promise<HTMLCanvasElement | null> {
+async function readSystemClipboardImageCanvas(): Promise<HTMLCanvasElement | null> {
   try {
     const items = await navigator.clipboard.read();
 
@@ -90,7 +90,7 @@ export function writeImageCanvasToSystemClipboardPng(canvas: HTMLCanvasElement):
   }
 }
 
-export interface ResolveSketchPasteImageOptions {
+interface ResolveSketchPasteImageOptions {
   internalBuffer: HTMLCanvasElement | null;
   /**
    * When true (e.g. Ctrl+Shift+V), read the in-app buffer before the OS clipboard
@@ -122,7 +122,7 @@ export async function resolveSketchPasteImageCanvas(
   return image;
 }
 
-export interface BuildSketchInternalClipboardParams {
+interface BuildSketchInternalClipboardParams {
   snapshot: HTMLCanvasElement;
   layer: Layer;
   documentCanvasWidth: number;
@@ -192,7 +192,7 @@ export function buildSketchInternalClipboardCanvas(
   return tmp;
 }
 
-export interface SketchPasteDrawParams {
+interface SketchPasteDrawParams {
   /** Layer composite offset in document space (from {@link getLayerGeometry}). */
   offset: Point;
   /** Document-space top-left of pixel under cursor, if known. */

web/src/components/sketch/types/document.ts

@@ -156,18 +156,6 @@ export interface LayerImageReference {
   objectFit: LayerImageObjectFit;
 }
 
-// ─── Layer Effects ────────────────────────────────────────────────────────────
-
-/**
- * Supported per-layer effect types.
- * Each effect is evaluated between "draw raster" and "blend into composite"
- * via the runtime's `evaluateLayerEffects` method.
- */
-export type LayerEffectType =
-  | "brightness_contrast"
-  | "hue_saturation"
-  | "exposure";
-
 // ─── Per-effect typed interfaces ──────────────────────────────────────────────
 
 interface BrightnessContrastEffect {

web/src/components/timeline/timelineAgentBridge.ts

@@ -174,7 +174,7 @@ export interface TimelineAddShapeClipOptions {
 }
 
 /** Render/audio params the agent can patch on any clip. */
-export interface TimelineClipParamsPatch {
+interface TimelineClipParamsPatch {
   name?: string;
   opacity?: number;
   speedMultiplier?: number;
@@ -207,7 +207,7 @@ interface TimelineClipBindingPatch {
   regenerate?: boolean;
 }
 
-export interface TimelineTrimPatch {
+interface TimelineTrimPatch {
   /** New clip duration on the timeline (ms). */
   durationMs?: number;
   /** Source-time trim start (ms). */
@@ -216,7 +216,7 @@ export interface TimelineTrimPatch {
   outPointMs?: number;
 }
 
-export interface TimelineMovePatch {
+interface TimelineMovePatch {
   /** New absolute start on the timeline (ms). */
   startMs?: number;
   /** Reassign the clip to a different track. */
@@ -285,7 +285,7 @@ export interface TimelineAddMediaClipOptions {
 }
 
 /** How {@link TimelineAgentHandler.setClipAnimations} applies its inputs. */
-export type ClipAnimationMode = "add" | "replace";
+type ClipAnimationMode = "add" | "replace";
 
 /**
  * Operations the live {@link TimelineEditor} exposes to the agent tooling

web/src/core/chat/chatProtocol.ts

@@ -77,7 +77,7 @@ export interface WorkflowUpdatedUpdate {
   graph: Graph;
 }
 
-export interface GenerationStoppedUpdate {
+interface GenerationStoppedUpdate {
   type: "generation_stopped";
   message: string;
 }
@@ -131,7 +131,7 @@ interface SecretRequestMessage {
   help_url: string | null;
 }
 
-export interface ToolCallMessage {
+interface ToolCallMessage {
   type: "tool_call";
   tool_call_id: string;
   name: string;
@@ -164,7 +164,7 @@ interface ChatTurnActiveUpdate {
   last_seq: number;
 }
 
-export type MsgpackData =
+type MsgpackData =
   | JobUpdate
   | Chunk
   | Prediction
@@ -191,7 +191,7 @@ export type MsgpackData =
   | ChatTurnActiveUpdate
   | ErrorMessage;
 
-export interface ToolResultMessage {
+interface ToolResultMessage {
   type: "tool_result";
   tool_call_id: string;
   result: unknown;