@graph-knowledge/api 0.3.2 → 0.9.0

package/src/lib/operations/batch-operations.js

@@ -1,62 +1,33 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BatchOperations = void 0;
-const uuid_1 = require("../utils/uuid");
 const api_errors_1 = require("../errors/api-errors");
-const element_defaults_1 = require("../constants/element-defaults");
+const rotation_1 = require("../utils/rotation");
 /**
  * Implementation of batch element operations.
  */
 class BatchOperations {
     firestore;
     auth;
-    validatorRegistry;
-    constructor(firestore, auth, validatorRegistry) {
+    elementBuilder;
+    constructor(firestore, auth, elementBuilder) {
         this.firestore = firestore;
         this.auth = auth;
-        this.validatorRegistry = validatorRegistry;
+        this.elementBuilder = elementBuilder;
     }
     async createElements(documentId, nodeId, inputs) {
         this.auth.requireAuth();
         if (inputs.length === 0) {
             return [];
         }
-        // Validate ALL inputs first (before creating any)
-        for (const input of inputs) {
-            // Custom shapes require premium
-            if (input.type.startsWith("custom:")) {
-                await this.auth.requirePremium();
-            }
-            this.validatorRegistry.validate(input);
-        }
+        // Build all elements first (validates all before writing any)
+        // Uses buildMany for deduped custom shape fetching
+        const newElements = await this.elementBuilder.buildMany(inputs);
         const nodePath = `documents/${documentId}/nodes/${nodeId}`;
         const node = await this.firestore.getDocument(nodePath);
         if (!node) {
             throw new api_errors_1.NotFoundError(`Node ${nodeId} not found`);
         }
-        // Build all elements
-        const newElements = inputs.map((input) => {
-            const defaults = this.getDefaultDimensions(input.type);
-            const x = input.x ?? 0;
-            const y = input.y ?? 0;
-            const element = {
-                id: (0, uuid_1.generateUuid)(),
-                elementType: input.type,
-                x,
-                y,
-                width: input.width ?? defaults.width,
-                height: input.height ?? defaults.height,
-                rotation: this.normalizeRotation(input.rotation ?? 0),
-                properties: this.buildProperties(input)
-            };
-            if (input.isLink !== undefined) {
-                element.isLink = input.isLink;
-            }
-            if (input.linkTarget !== undefined) {
-                element.linkTarget = input.linkTarget;
-            }
-            return element;
-        });
         // Update node with all new elements atomically
         const updatedElements = [...(node.elements || []), ...newElements];
         await this.firestore.updateDocument(nodePath, {
@@ -96,7 +67,7 @@ class BatchOperations {
                 width: update.width ?? element.width,
                 height: update.height ?? element.height,
                 rotation: update.rotation !== undefined
-                    ? this.normalizeRotation(update.rotation)
+                    ? (0, rotation_1.normalizeRotation)(update.rotation)
                     : element.rotation,
                 properties: {
                     ...element.properties,
@@ -138,48 +109,5 @@ class BatchOperations {
             elements: updatedElements
         });
     }
-    /**
-     * Gets default dimensions for an element type.
-     */
-    getDefaultDimensions(type) {
-        if (type.startsWith("custom:")) {
-            return element_defaults_1.DEFAULT_CUSTOM_SHAPE_DIMENSIONS;
-        }
-        return (element_defaults_1.DEFAULT_ELEMENT_DIMENSIONS[type] ?? element_defaults_1.DEFAULT_CUSTOM_SHAPE_DIMENSIONS);
-    }
-    /**
-     * Normalizes rotation to integer in range [0, 360).
-     */
-    normalizeRotation(rotation) {
-        let normalized = rotation % 360;
-        if (normalized < 0) {
-            normalized += 360;
-        }
-        return Math.round(normalized);
-    }
-    /**
-     * Builds properties object from element input.
-     */
-    buildProperties(input) {
-        const props = {
-            __schemaVersion: 1
-        };
-        const commonFields = new Set([
-            "type",
-            "x",
-            "y",
-            "width",
-            "height",
-            "rotation",
-            "isLink",
-            "linkTarget"
-        ]);
-        for (const [key, value] of Object.entries(input)) {
-            if (!commonFields.has(key) && value !== undefined) {
-                props[key] = value;
-            }
-        }
-        return props;
-    }
 }
 exports.BatchOperations = BatchOperations;

package/src/lib/operations/custom-shape-operations.d.ts

@@ -0,0 +1,27 @@
+import { ICustomShapeOperations } from "../interfaces/custom-shape-operations.interface";
+import { IFirestoreClient } from "../interfaces/firestore-client.interface";
+import { IAuthClient } from "../interfaces/auth-client.interface";
+import { CustomShapeDefinitionValidator } from "../validators/custom-shape-definition-validator";
+import { CreateCustomShapeInput, UpdateCustomShapeInput, CustomShapeResult } from "../types/api-types";
+/**
+ * Implementation of custom shape operations.
+ *
+ * Custom shapes are stored in:
+ * - `/users/{uid}/customShapes/{shapeId}` (user drafts)
+ * - `/shapeStore/{shapeId}` (published shapes)
+ */
+export declare class CustomShapeOperations implements ICustomShapeOperations {
+    private readonly firestore;
+    private readonly auth;
+    private readonly validator;
+    constructor(firestore: IFirestoreClient, auth: IAuthClient, validator: CustomShapeDefinitionValidator);
+    create(input: CreateCustomShapeInput): Promise<CustomShapeResult>;
+    list(): Promise<CustomShapeResult[]>;
+    get(shapeId: string): Promise<CustomShapeResult>;
+    update(shapeId: string, input: UpdateCustomShapeInput): Promise<void>;
+    delete(shapeId: string): Promise<void>;
+    publish(shapeId: string): Promise<void>;
+    unpublish(shapeId: string): Promise<void>;
+    listPublished(): Promise<CustomShapeResult[]>;
+    getPublished(shapeId: string): Promise<CustomShapeResult>;
+}

package/src/lib/operations/custom-shape-operations.js

@@ -0,0 +1,175 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CustomShapeOperations = void 0;
+const api_errors_1 = require("../errors/api-errors");
+const uuid_1 = require("../utils/uuid");
+// Default values
+const DEFAULT_ICON = "category";
+const DEFAULT_CATEGORY = "Custom";
+const DEFAULT_WIDTH = 120;
+const DEFAULT_HEIGHT = 120;
+const CURRENT_SHAPE_SCHEMA_VERSION = 1;
+/**
+ * Implementation of custom shape operations.
+ *
+ * Custom shapes are stored in:
+ * - `/users/{uid}/customShapes/{shapeId}` (user drafts)
+ * - `/shapeStore/{shapeId}` (published shapes)
+ */
+class CustomShapeOperations {
+    firestore;
+    auth;
+    validator;
+    constructor(firestore, auth, validator) {
+        this.firestore = firestore;
+        this.auth = auth;
+        this.validator = validator;
+    }
+    async create(input) {
+        const userId = this.auth.requireAuth();
+        await this.auth.requirePremium();
+        this.validator.validateCreate(input);
+        const shapeId = (0, uuid_1.generateUuid)();
+        const shapeData = {
+            name: input.name,
+            icon: input.icon ?? DEFAULT_ICON,
+            category: input.category ?? DEFAULT_CATEGORY,
+            authorId: userId,
+            status: "draft",
+            createdAt: new Date(),
+            updatedAt: new Date(),
+            svgPathData: input.svgPathData,
+            viewBox: input.viewBox,
+            defaultWidth: input.defaultWidth ?? DEFAULT_WIDTH,
+            defaultHeight: input.defaultHeight ?? DEFAULT_HEIGHT,
+            defaultProperties: {
+                fillColor: input.defaultProperties?.fillColor ?? "#4A90D9",
+                strokeColor: input.defaultProperties?.strokeColor ?? "#2C3E50",
+                strokeWidth: input.defaultProperties?.strokeWidth ?? 2
+            },
+            schemaVersion: CURRENT_SHAPE_SCHEMA_VERSION
+        };
+        if (input.description !== undefined) {
+            shapeData.description = input.description;
+        }
+        if (input.aspectRatioLocked !== undefined) {
+            shapeData.aspectRatioLocked = input.aspectRatioLocked;
+        }
+        await this.firestore.setDocument(`users/${userId}/customShapes/${shapeId}`, shapeData);
+        return { id: shapeId, ...shapeData };
+    }
+    async list() {
+        const userId = this.auth.requireAuth();
+        const shapes = await this.firestore.getCollection(`users/${userId}/customShapes`);
+        return shapes;
+    }
+    async get(shapeId) {
+        const userId = this.auth.requireAuth();
+        const shape = await this.firestore.getDocument(`users/${userId}/customShapes/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Custom shape ${shapeId} not found`);
+        }
+        return shape;
+    }
+    async update(shapeId, input) {
+        const userId = this.auth.requireAuth();
+        this.validator.validateUpdate(input);
+        const shape = await this.firestore.getDocument(`users/${userId}/customShapes/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Custom shape ${shapeId} not found`);
+        }
+        const updates = {};
+        if (input.name !== undefined)
+            updates["name"] = input.name;
+        if (input.description !== undefined)
+            updates["description"] = input.description;
+        if (input.icon !== undefined)
+            updates["icon"] = input.icon;
+        if (input.category !== undefined)
+            updates["category"] = input.category;
+        if (input.svgPathData !== undefined)
+            updates["svgPathData"] = input.svgPathData;
+        if (input.viewBox !== undefined)
+            updates["viewBox"] = input.viewBox;
+        if (input.defaultWidth !== undefined)
+            updates["defaultWidth"] = input.defaultWidth;
+        if (input.defaultHeight !== undefined)
+            updates["defaultHeight"] = input.defaultHeight;
+        if (input.aspectRatioLocked !== undefined)
+            updates["aspectRatioLocked"] = input.aspectRatioLocked;
+        if (input.defaultProperties !== undefined) {
+            updates["defaultProperties"] = {
+                ...shape.defaultProperties,
+                ...input.defaultProperties
+            };
+        }
+        updates["updatedAt"] = new Date();
+        if (Object.keys(updates).length > 1) {
+            await this.firestore.updateDocument(`users/${userId}/customShapes/${shapeId}`, updates);
+        }
+    }
+    async delete(shapeId) {
+        const userId = this.auth.requireAuth();
+        const shape = await this.firestore.getDocument(`users/${userId}/customShapes/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Custom shape ${shapeId} not found`);
+        }
+        await this.firestore.deleteDocument(`users/${userId}/customShapes/${shapeId}`);
+    }
+    async publish(shapeId) {
+        const userId = this.auth.requireAuth();
+        await this.auth.requirePremium();
+        const shape = await this.firestore.getDocument(`users/${userId}/customShapes/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Custom shape ${shapeId} not found`);
+        }
+        const now = new Date();
+        // Copy to public store
+        const publishedData = {
+            ...shape,
+            status: "published",
+            publishedAt: now,
+            updatedAt: now
+        };
+        delete publishedData.id;
+        // Atomic: write to store + update draft status
+        const batch = this.firestore.batch();
+        batch.set(`shapeStore/${shapeId}`, publishedData);
+        batch.update(`users/${userId}/customShapes/${shapeId}`, {
+            status: "published",
+            publishedAt: now,
+            updatedAt: now
+        });
+        await batch.commit();
+    }
+    async unpublish(shapeId) {
+        const userId = this.auth.requireAuth();
+        const shape = await this.firestore.getDocument(`users/${userId}/customShapes/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Custom shape ${shapeId} not found`);
+        }
+        // Atomic: delete from store + revert draft status
+        const batch = this.firestore.batch();
+        batch.delete(`shapeStore/${shapeId}`);
+        batch.update(`users/${userId}/customShapes/${shapeId}`, {
+            status: "draft",
+            publishedAt: null,
+            updatedAt: new Date()
+        });
+        await batch.commit();
+    }
+    async listPublished() {
+        this.auth.requireAuth();
+        const shapes = await this.firestore.queryCollection("shapeStore", "status", "published");
+        return shapes;
+    }
+    async getPublished(shapeId) {
+        this.auth.requireAuth();
+        const shape = await this.firestore.getDocument(`shapeStore/${shapeId}`);
+        if (!shape) {
+            throw new api_errors_1.NotFoundError(`Published shape ${shapeId} not found in store`);
+        }
+        return shape;
+    }
+}
+exports.CustomShapeOperations = CustomShapeOperations;

package/src/lib/operations/document-operations.js

@@ -62,18 +62,25 @@ class DocumentOperations {
             throw new api_errors_1.NotFoundError(`Document ${documentId} not found`);
         }
         const nodes = await this.firestore.getCollection(`documents/${documentId}/nodes`);
+        // Ensure rootNodeId is present (legacy documents may lack it)
+        const rootNodeId = docData.rootNodeId ??
+            nodes.find((n) => n.level === 0)?.id ??
+            "";
         return {
             ...docData,
-            nodes
+            nodes,
+            rootNodeId
         };
     }
     async list() {
         const userId = this.auth.requireAuth();
         const documents = await this.firestore.queryCollection("documents", "owner", userId);
         // Return shallow documents without nodes
+        // Ensure rootNodeId is present (legacy documents may lack it)
         return documents.map((doc) => ({
             ...doc,
-            nodes: []
+            nodes: [],
+            rootNodeId: doc.rootNodeId ?? ""
         }));
     }
     async update(documentId, input) {

package/src/lib/operations/element-operations.d.ts

@@ -2,38 +2,19 @@ import { GraphElement } from "../models";
 import { IElementOperations } from "../interfaces/element-operations.interface";
 import { IFirestoreClient } from "../interfaces/firestore-client.interface";
 import { IAuthClient } from "../interfaces/auth-client.interface";
-import { IElementValidatorRegistry } from "../interfaces/element-validator.interface";
 import { AnyElementInput, UpdateElementInput } from "../types/element-input-types";
+import { ElementBuilder } from "../utils/element-builder";
 /**
  * Implementation of element CRUD operations.
  */
 export declare class ElementOperations implements IElementOperations {
     private readonly firestore;
     private readonly auth;
-    private readonly validatorRegistry;
-    constructor(firestore: IFirestoreClient, auth: IAuthClient, validatorRegistry: IElementValidatorRegistry);
+    private readonly elementBuilder;
+    constructor(firestore: IFirestoreClient, auth: IAuthClient, elementBuilder: ElementBuilder);
     get(documentId: string, nodeId: string, elementId: string): Promise<GraphElement>;
     list(documentId: string, nodeId: string): Promise<GraphElement[]>;
     create(documentId: string, nodeId: string, input: AnyElementInput): Promise<GraphElement>;
     update(documentId: string, nodeId: string, elementId: string, input: UpdateElementInput): Promise<void>;
     delete(documentId: string, nodeId: string, elementId: string): Promise<void>;
-    /**
-     * Gets default dimensions for an element type.
-     */
-    private getDefaultDimensions;
-    /**
-     * Normalizes rotation to integer in range [0, 360).
-     */
-    private normalizeRotation;
-    /**
-     * Auto-measures text width for text elements created without explicit width/maxWidth.
-     * Uses a lazily-initialized TextMeasurementService singleton.
-     */
-    private measureTextWidth;
-    private static _textMeasurementService;
-    /**
-     * Builds properties object from element input.
-     * Extracts type-specific properties, excluding common fields.
-     */
-    private buildProperties;
 }

package/src/lib/operations/element-operations.js

@@ -1,21 +1,19 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ElementOperations = void 0;
-const uuid_1 = require("../utils/uuid");
 const api_errors_1 = require("../errors/api-errors");
-const element_defaults_1 = require("../constants/element-defaults");
-const text_measurement_service_1 = require("../utils/text-measurement/text-measurement-service");
+const rotation_1 = require("../utils/rotation");
 /**
  * Implementation of element CRUD operations.
  */
 class ElementOperations {
     firestore;
     auth;
-    validatorRegistry;
-    constructor(firestore, auth, validatorRegistry) {
+    elementBuilder;
+    constructor(firestore, auth, elementBuilder) {
         this.firestore = firestore;
         this.auth = auth;
-        this.validatorRegistry = validatorRegistry;
+        this.elementBuilder = elementBuilder;
     }
     async get(documentId, nodeId, elementId) {
         this.auth.requireAuth();
@@ -41,50 +39,12 @@ class ElementOperations {
     }
     async create(documentId, nodeId, input) {
         this.auth.requireAuth();
-        // Custom shapes require premium
-        if (input.type.startsWith("custom:")) {
-            await this.auth.requirePremium();
-        }
-        this.validatorRegistry.validate(input);
         const nodePath = `documents/${documentId}/nodes/${nodeId}`;
         const node = await this.firestore.getDocument(nodePath);
         if (!node) {
             throw new api_errors_1.NotFoundError(`Node ${nodeId} not found`);
         }
-        const defaults = this.getDefaultDimensions(input.type);
-        // Connectors have optional x/y since their position is derived from connected elements
-        const x = input.x ?? 0;
-        const y = input.y ?? 0;
-        // For text elements, maxWidth is a convenience alias for width (the wrap boundary)
-        let resolvedWidth = input.width;
-        if (input.type === "text" && resolvedWidth === undefined) {
-            const textInput = input;
-            if (textInput.maxWidth !== undefined) {
-                resolvedWidth = textInput.maxWidth;
-            }
-            else {
-                // Auto-measure text width to avoid the 100px default which causes
-                // unexpected wrapping in the editor.
-                resolvedWidth = this.measureTextWidth(textInput);
-            }
-        }
-        const element = {
-            id: (0, uuid_1.generateUuid)(),
-            elementType: input.type,
-            x,
-            y,
-            width: resolvedWidth ?? defaults.width,
-            height: input.height ?? defaults.height,
-            rotation: this.normalizeRotation(input.rotation ?? 0),
-            properties: this.buildProperties(input)
-        };
-        // Only add link properties if they have values (Firestore rejects undefined)
-        if (input.isLink !== undefined) {
-            element.isLink = input.isLink;
-        }
-        if (input.linkTarget !== undefined) {
-            element.linkTarget = input.linkTarget;
-        }
+        const element = await this.elementBuilder.build(input);
         const updatedElements = [...(node.elements || []), element];
         await this.firestore.updateDocument(nodePath, {
             elements: updatedElements
@@ -110,7 +70,7 @@ class ElementOperations {
             width: input.width ?? existingElement.width,
             height: input.height ?? existingElement.height,
             rotation: input.rotation !== undefined
-                ? this.normalizeRotation(input.rotation)
+                ? (0, rotation_1.normalizeRotation)(input.rotation)
                 : existingElement.rotation,
             properties: {
                 ...existingElement.properties,
@@ -144,78 +104,5 @@ class ElementOperations {
             elements: updatedElements
         });
     }
-    /**
-     * Gets default dimensions for an element type.
-     */
-    getDefaultDimensions(type) {
-        if (type.startsWith("custom:")) {
-            return element_defaults_1.DEFAULT_CUSTOM_SHAPE_DIMENSIONS;
-        }
-        return (element_defaults_1.DEFAULT_ELEMENT_DIMENSIONS[type] ?? element_defaults_1.DEFAULT_CUSTOM_SHAPE_DIMENSIONS);
-    }
-    /**
-     * Normalizes rotation to integer in range [0, 360).
-     */
-    normalizeRotation(rotation) {
-        let normalized = rotation % 360;
-        if (normalized < 0) {
-            normalized += 360;
-        }
-        return Math.round(normalized);
-    }
-    /**
-     * Auto-measures text width for text elements created without explicit width/maxWidth.
-     * Uses a lazily-initialized TextMeasurementService singleton.
-     */
-    measureTextWidth(input) {
-        if (!ElementOperations._textMeasurementService) {
-            ElementOperations._textMeasurementService = new text_measurement_service_1.TextMeasurementService();
-        }
-        const metrics = ElementOperations._textMeasurementService.measureText(input.text, {
-            fontSize: input.fontSize,
-            fontFamily: input.fontFamily,
-            fontWeight: input.fontWeight,
-            textAlign: input.textAlign,
-            lineHeight: input.lineHeight
-        });
-        return Math.ceil(metrics.width);
-    }
-    static _textMeasurementService = null;
-    /**
-     * Builds properties object from element input.
-     * Extracts type-specific properties, excluding common fields.
-     */
-    buildProperties(input) {
-        const props = {
-            __schemaVersion: 1
-        };
-        // Exclude common fields and convenience aliases, copy the rest as properties
-        const commonFields = new Set([
-            "type",
-            "x",
-            "y",
-            "width",
-            "height",
-            "rotation",
-            "isLink",
-            "linkTarget",
-            "maxWidth"
-        ]);
-        for (const [key, value] of Object.entries(input)) {
-            if (!commonFields.has(key) && value !== undefined) {
-                props[key] = value;
-            }
-        }
-        // For text elements, set __usesTopLeft: true so the app's normalization
-        // knows that x/y is the bounding box top-left (not the text anchor point).
-        // Without this, the app would incorrectly adjust x/y based on textAlign,
-        // causing centered text to be positioned in negative coordinates.
-        // Since we now always resolve a proper width for text (via explicit width,
-        // maxWidth, or auto-measurement), we always set this flag.
-        if (input.type === "text") {
-            props["__usesTopLeft"] = true;
-        }
-        return props;
-    }
 }
 exports.ElementOperations = ElementOperations;

package/src/lib/operations/node-operations.d.ts

@@ -3,7 +3,8 @@ import { INodeOperations } from "../interfaces/node-operations.interface";
 import { IFirestoreClient } from "../interfaces/firestore-client.interface";
 import { IAuthClient } from "../interfaces/auth-client.interface";
 import { NodeValidator } from "../validators/node-validator";
-import { CreateNodeInput, UpdateNodeInput } from "../types/api-types";
+import { CreateNodeInput, UpdateNodeInput, CreateNodeWithElementsInput, CreateNodeWithElementsResult } from "../types/api-types";
+import { ElementBuilder } from "../utils/element-builder";
 /**
  * Implementation of node CRUD operations.
  */
@@ -11,10 +12,15 @@ export declare class NodeOperations implements INodeOperations {
     private readonly firestore;
     private readonly auth;
     private readonly validator;
-    constructor(firestore: IFirestoreClient, auth: IAuthClient, validator: NodeValidator);
+    private readonly elementBuilder;
+    constructor(firestore: IFirestoreClient, auth: IAuthClient, validator: NodeValidator, elementBuilder: ElementBuilder);
     create(documentId: string, input: CreateNodeInput): Promise<GraphNode>;
     createWithId(documentId: string, nodeId: string, input: CreateNodeInput): Promise<GraphNode>;
-    private createNodeWithId;
+    createWithElements(documentId: string, input: CreateNodeWithElementsInput): Promise<CreateNodeWithElementsResult>;
+    /**
+     * Shared node building and persistence logic.
+     */
+    private _buildAndWrite;
     get(documentId: string, nodeId: string): Promise<GraphNode>;
     list(documentId: string): Promise<GraphNode[]>;
     update(documentId: string, nodeId: string, input: UpdateNodeInput): Promise<void>;

package/src/lib/operations/node-operations.js

@@ -13,19 +13,32 @@ class NodeOperations {
     firestore;
     auth;
     validator;
-    constructor(firestore, auth, validator) {
+    elementBuilder;
+    constructor(firestore, auth, validator, elementBuilder) {
         this.firestore = firestore;
         this.auth = auth;
         this.validator = validator;
+        this.elementBuilder = elementBuilder;
     }
     async create(documentId, input) {
         const nodeId = (0, uuid_1.generateUuid)();
-        return this.createNodeWithId(documentId, nodeId, input);
+        return this._buildAndWrite(documentId, nodeId, input, []);
     }
     async createWithId(documentId, nodeId, input) {
-        return this.createNodeWithId(documentId, nodeId, input);
+        return this._buildAndWrite(documentId, nodeId, input, []);
     }
-    async createNodeWithId(documentId, nodeId, input) {
+    async createWithElements(documentId, input) {
+        // Build all elements first (validates all before creating anything)
+        // Uses buildMany for deduped custom shape fetching
+        const elements = await this.elementBuilder.buildMany(input.elements);
+        const nodeId = (0, uuid_1.generateUuid)();
+        const node = await this._buildAndWrite(documentId, nodeId, input, elements);
+        return { node, elements };
+    }
+    /**
+     * Shared node building and persistence logic.
+     */
+    async _buildAndWrite(documentId, nodeId, input, elements) {
         const userId = this.auth.requireAuth();
         this.validator.validateCreate(input);
         // Determine level from parent if provided
@@ -40,7 +53,7 @@ class NodeOperations {
             id: nodeId,
             title: input.title,
             content: input.content ?? "",
-            elements: [],
+            elements,
             canvasWidth: input.canvasWidth ?? DEFAULT_CANVAS_WIDTH,
             canvasHeight: input.canvasHeight ?? DEFAULT_CANVAS_HEIGHT,
             level,

package/src/lib/operations/template-operations.d.ts

@@ -1,7 +1,7 @@
 import { ITemplateOperations } from "../interfaces/template-operations.interface";
 import { IFirestoreClient } from "../interfaces/firestore-client.interface";
 import { IAuthClient } from "../interfaces/auth-client.interface";
-import { CreateTemplateInput, DocumentResult } from "../types/api-types";
+import { CreateTemplateInput, UpdateTemplateInput, DocumentResult } from "../types/api-types";
 import { TemplateValidator } from "../validators/template-validator";
 /**
  * Implementation of template operations.
@@ -17,6 +17,8 @@ export declare class TemplateOperations implements ITemplateOperations {
     create(input: CreateTemplateInput): Promise<DocumentResult>;
     list(): Promise<DocumentResult[]>;
     get(templateId: string): Promise<DocumentResult>;
+    update(templateId: string, input: UpdateTemplateInput): Promise<void>;
+    delete(templateId: string): Promise<void>;
     clone(templateId: string, title?: string): Promise<DocumentResult>;
     publish(templateId: string): Promise<void>;
     unpublish(templateId: string): Promise<void>;