@powerhousedao/vetra 4.1.0-dev.82 → 4.1.0-dev.83

package/dist/document-models/document-editor/src/tests/document-model.test.js

@@ -2,17 +2,100 @@
  * This is a scaffold file meant for customization:
  * - change it by adding new tests or modifying the existing ones
  */
-import { describe, it, expect, beforeEach } from "vitest";
-import utils, { initialGlobalState, initialLocalState, } from "../../gen/utils.js";
+import { describe, expect, it } from "vitest";
+import * as creators from "../../gen/base-operations/creators.js";
+import { reducer } from "../../gen/reducer.js";
+import utils from "../../gen/utils.js";
 describe("Document Editor Document Model", () => {
-    it("should create a new Document Editor document", () => {
+    it("should have correct initial values", () => {
         const document = utils.createDocument();
-        expect(document).toBeDefined();
-        expect(document.header.documentType).toBe("powerhouse/document-editor");
+        expect(document.state.global.name).toBe("");
+        expect(document.state.global.documentTypes).toEqual([]);
+        expect(document.state.global.status).toBe("DRAFT");
     });
-    it("should create a new Document Editor document with a valid initial state", () => {
+    it("should handle multiple operations and maintain consistency", () => {
         const document = utils.createDocument();
-        expect(document.state.global).toStrictEqual(initialGlobalState);
-        expect(document.state.local).toStrictEqual(initialLocalState);
+        let updatedDoc = reducer(document, creators.setEditorName({ name: "Test Editor" }));
+        updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+            id: "test-type",
+            documentType: "powerhouse/test",
+        }));
+        updatedDoc = reducer(updatedDoc, creators.setEditorStatus({ status: "CONFIRMED" }));
+        // Verify state consistency
+        expect(updatedDoc.state.global.name).toBe("Test Editor");
+        expect(updatedDoc.state.global.documentTypes).toHaveLength(1);
+        expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+    });
+    describe("Complex Scenarios", () => {
+        it("should handle workflow: set name, add types, confirm status", () => {
+            const document = utils.createDocument();
+            // Step 1: Set editor name
+            let updatedDoc = reducer(document, creators.setEditorName({ name: "Production Editor" }));
+            expect(updatedDoc.state.global.name).toBe("Production Editor");
+            // Step 2: Add document types
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "budget-docs",
+                documentType: "powerhouse/budget",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "invoice-docs",
+                documentType: "powerhouse/invoice",
+            }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Step 3: Confirm editor status
+            updatedDoc = reducer(updatedDoc, creators.setEditorStatus({ status: "CONFIRMED" }));
+            expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+        });
+        it("should handle reset scenario: remove all types and add new ones", () => {
+            const document = utils.createDocument();
+            // Add initial types
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "old-type-1",
+                documentType: "powerhouse/old1",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "old-type-2",
+                documentType: "powerhouse/old2",
+            }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Remove all types
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "old-type-1" }));
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "old-type-2" }));
+            expect(updatedDoc.state.global.documentTypes).toEqual([]);
+            // Add new types from scratch
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-1",
+                documentType: "powerhouse/new1",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-2",
+                documentType: "powerhouse/new2",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-3",
+                documentType: "powerhouse/new3",
+            }));
+            // Verify reset completed
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "old-type-1")).toBeUndefined();
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "new-type-1")).toBeDefined();
+        });
+        it("should maintain state integrity with complex add/remove sequences", () => {
+            const document = utils.createDocument();
+            // Add multiple types
+            let updatedDoc = reducer(document, creators.addDocumentType({ id: "type-a", documentType: "a" }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-b", documentType: "b" }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-c", documentType: "c" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            // Remove middle one
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "type-b" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Add it back with new documentType
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-b", documentType: "b-new" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            // Verify order and content
+            const typeB = updatedDoc.state.global.documentTypes.find((dt) => dt.id === "type-b");
+            expect(typeB?.documentType).toBe("b-new");
+        });
     });
 });

package/dist/document-models/processor-module/src/reducers/base-operations.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/processor-module/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uCAAuC,EAAE,MAAM,yCAAyC,CAAC;AAEvG,eAAO,MAAM,OAAO,EAAE,uCAqBrB,CAAC"}
+{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/processor-module/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uCAAuC,EAAE,MAAM,yCAAyC,CAAC;AAEvG,eAAO,MAAM,OAAO,EAAE,uCAuCrB,CAAC"}

package/dist/document-models/processor-module/src/reducers/base-operations.js

@@ -1,11 +1,24 @@
 export const reducer = {
     setProcessorNameOperation(state, action, dispatch) {
-        state.name = action.input.name;
+        const trimmedName = action.input.name.trim();
+        if (trimmedName === "") {
+            throw new Error("Processor name cannot be empty");
+        }
+        state.name = trimmedName;
     },
     setProcessorTypeOperation(state, action, dispatch) {
-        state.type = action.input.type;
+        const trimmedType = action.input.type.trim();
+        if (trimmedType === "") {
+            throw new Error("Processor type cannot be empty");
+        }
+        state.type = trimmedType;
     },
     addDocumentTypeOperation(state, action, dispatch) {
+        // Check for duplicate ID
+        const existingId = state.documentTypes.find((dt) => dt.id === action.input.id);
+        if (existingId) {
+            throw new Error(`Document type with id "${action.input.id}" already exists`);
+        }
         state.documentTypes.push({
             id: action.input.id,
             documentType: action.input.documentType,

package/dist/document-models/processor-module/src/tests/base-operations.test.js

@@ -2,47 +2,167 @@
  * This is a scaffold file meant for customization:
  * - change it by adding new tests or modifying the existing ones
  */
-import { describe, it, expect, beforeEach } from "vitest";
-import { generateMock } from "@powerhousedao/codegen";
-import utils from "../../gen/utils.js";
-import { z } from "../../gen/schema/index.js";
-import { reducer } from "../../gen/reducer.js";
+import { beforeEach, describe, expect, it } from "vitest";
 import * as creators from "../../gen/base-operations/creators.js";
+import { reducer } from "../../gen/reducer.js";
+import utils from "../../gen/utils.js";
 describe("BaseOperations Operations", () => {
     let document;
     beforeEach(() => {
         document = utils.createDocument();
     });
-    it("should handle setProcessorName operation", () => {
-        const input = generateMock(z.SetProcessorNameInputSchema());
-        const updatedDocument = reducer(document, creators.setProcessorName(input));
-        expect(updatedDocument.operations.global).toHaveLength(1);
-        expect(updatedDocument.operations.global[0].action.type).toBe("SET_PROCESSOR_NAME");
-        expect(updatedDocument.operations.global[0].action.input).toStrictEqual(input);
-        expect(updatedDocument.operations.global[0].index).toEqual(0);
+    describe("setProcessorName", () => {
+        it("should mutate state with new name", () => {
+            const input = { name: "My Processor" };
+            const updatedDocument = reducer(document, creators.setProcessorName(input));
+            expect(updatedDocument.state.global.name).toBe("My Processor");
+        });
+        it("should reject empty string and store error in operation", () => {
+            const input = { name: "" };
+            const updatedDocument = reducer(document, creators.setProcessorName(input));
+            expect(updatedDocument.operations.global).toHaveLength(1);
+            expect(updatedDocument.operations.global[0].error).toBe("Processor name cannot be empty");
+            expect(updatedDocument.state.global.name).toBe("");
+        });
+    });
+    describe("setProcessorType", () => {
+        it("should mutate state with new type", () => {
+            const input = { type: "analytics" };
+            const updatedDocument = reducer(document, creators.setProcessorType(input));
+            expect(updatedDocument.state.global.type).toBe("analytics");
+        });
+        it("should reject empty string and store error in operation", () => {
+            const input = { type: "" };
+            const updatedDocument = reducer(document, creators.setProcessorType(input));
+            expect(updatedDocument.operations.global).toHaveLength(1);
+            expect(updatedDocument.operations.global[0].error).toBe("Processor type cannot be empty");
+            expect(updatedDocument.state.global.type).toBe("");
+        });
     });
-    it("should handle setProcessorType operation", () => {
-        const input = generateMock(z.SetProcessorTypeInputSchema());
-        const updatedDocument = reducer(document, creators.setProcessorType(input));
-        expect(updatedDocument.operations.global).toHaveLength(1);
-        expect(updatedDocument.operations.global[0].action.type).toBe("SET_PROCESSOR_TYPE");
-        expect(updatedDocument.operations.global[0].action.input).toStrictEqual(input);
-        expect(updatedDocument.operations.global[0].index).toEqual(0);
+    describe("setProcessorStatus", () => {
+        it("should mutate state with new status", () => {
+            const input = { status: "CONFIRMED" };
+            const updatedDocument = reducer(document, creators.setProcessorStatus(input));
+            expect(updatedDocument.state.global.status).toBe("CONFIRMED");
+        });
+        it("should transition from DRAFT to CONFIRMED", () => {
+            expect(document.state.global.status).toBe("DRAFT");
+            const updatedDocument = reducer(document, creators.setProcessorStatus({ status: "CONFIRMED" }));
+            expect(updatedDocument.state.global.status).toBe("CONFIRMED");
+        });
+        it("should transition from CONFIRMED to DRAFT", () => {
+            const confirmedDoc = reducer(document, creators.setProcessorStatus({ status: "CONFIRMED" }));
+            const updatedDocument = reducer(confirmedDoc, creators.setProcessorStatus({ status: "DRAFT" }));
+            expect(updatedDocument.state.global.status).toBe("DRAFT");
+        });
     });
-    it("should handle addDocumentType operation", () => {
-        const input = generateMock(z.AddDocumentTypeInputSchema());
-        const updatedDocument = reducer(document, creators.addDocumentType(input));
-        expect(updatedDocument.operations.global).toHaveLength(1);
-        expect(updatedDocument.operations.global[0].action.type).toBe("ADD_DOCUMENT_TYPE");
-        expect(updatedDocument.operations.global[0].action.input).toStrictEqual(input);
-        expect(updatedDocument.operations.global[0].index).toEqual(0);
+    describe("addDocumentType", () => {
+        it("should mutate state by adding document type to array", () => {
+            const input = {
+                id: "test-type-1",
+                documentType: "powerhouse/test",
+            };
+            const updatedDocument = reducer(document, creators.addDocumentType(input));
+            expect(updatedDocument.state.global.documentTypes).toContainEqual({
+                id: "test-type-1",
+                documentType: "powerhouse/test",
+            });
+        });
+        it("should add to empty array from initial state", () => {
+            expect(document.state.global.documentTypes).toEqual([]);
+            const input = {
+                id: "first-type",
+                documentType: "powerhouse/first",
+            };
+            const updatedDocument = reducer(document, creators.addDocumentType(input));
+            expect(updatedDocument.state.global.documentTypes).toEqual([
+                { id: "first-type", documentType: "powerhouse/first" },
+            ]);
+        });
+        it("should add multiple document types sequentially", () => {
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "type-1",
+                documentType: "powerhouse/a",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "type-2",
+                documentType: "powerhouse/b",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "type-3",
+                documentType: "powerhouse/c",
+            }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            expect(updatedDoc.state.global.documentTypes[0]).toEqual({
+                id: "type-1",
+                documentType: "powerhouse/a",
+            });
+            expect(updatedDoc.state.global.documentTypes[1]).toEqual({
+                id: "type-2",
+                documentType: "powerhouse/b",
+            });
+            expect(updatedDoc.state.global.documentTypes[2]).toEqual({
+                id: "type-3",
+                documentType: "powerhouse/c",
+            });
+        });
+        it("should reject duplicate IDs and store error in operation", () => {
+            const input = {
+                id: "duplicate",
+                documentType: "powerhouse/first",
+            };
+            let updatedDoc = reducer(document, creators.addDocumentType(input));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(1);
+            expect(updatedDoc.operations.global[0].error).toBeUndefined();
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "duplicate",
+                documentType: "powerhouse/second",
+            }));
+            expect(updatedDoc.operations.global).toHaveLength(2);
+            expect(updatedDoc.operations.global[1].error).toBe('Document type with id "duplicate" already exists');
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(1);
+        });
     });
-    it("should handle removeDocumentType operation", () => {
-        const input = generateMock(z.RemoveDocumentTypeInputSchema());
-        const updatedDocument = reducer(document, creators.removeDocumentType(input));
-        expect(updatedDocument.operations.global).toHaveLength(1);
-        expect(updatedDocument.operations.global[0].action.type).toBe("REMOVE_DOCUMENT_TYPE");
-        expect(updatedDocument.operations.global[0].action.input).toStrictEqual(input);
-        expect(updatedDocument.operations.global[0].index).toEqual(0);
+    describe("removeDocumentType", () => {
+        it("should mutate state by removing document type from array", () => {
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "to-remove",
+                documentType: "powerhouse/test",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "to-remove" }));
+            expect(updatedDoc.state.global.documentTypes).not.toContainEqual({
+                id: "to-remove",
+                documentType: "powerhouse/test",
+            });
+        });
+        it("should remove existing ID", () => {
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "existing",
+                documentType: "powerhouse/test",
+            }));
+            const lengthBefore = updatedDoc.state.global.documentTypes.length;
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "existing" }));
+            expect(updatedDoc.state.global.documentTypes.length).toBe(lengthBefore - 1);
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "existing")).toBeUndefined();
+        });
+        it("should gracefully handle non-existent ID", () => {
+            const initialState = document.state.global.documentTypes;
+            const updatedDocument = reducer(document, creators.removeDocumentType({ id: "non-existent-id" }));
+            expect(updatedDocument.state.global.documentTypes).toEqual(initialState);
+        });
+        it("should handle empty array gracefully", () => {
+            expect(document.state.global.documentTypes).toEqual([]);
+            const updatedDocument = reducer(document, creators.removeDocumentType({ id: "any-id" }));
+            expect(updatedDocument.state.global.documentTypes).toEqual([]);
+        });
+        it("should add then immediately remove item", () => {
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "temp-type",
+                documentType: "powerhouse/temp",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "temp-type" }));
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "temp-type")).toBeUndefined();
+            expect(updatedDoc.operations.global).toHaveLength(2);
+        });
     });
 });

package/dist/document-models/processor-module/src/tests/document-model.test.js

@@ -2,17 +2,105 @@
  * This is a scaffold file meant for customization:
  * - change it by adding new tests or modifying the existing ones
  */
-import { describe, it, expect, beforeEach } from "vitest";
-import utils, { initialGlobalState, initialLocalState, } from "../../gen/utils.js";
+import { describe, expect, it } from "vitest";
+import * as creators from "../../gen/base-operations/creators.js";
+import { reducer } from "../../gen/reducer.js";
+import utils from "../../gen/utils.js";
 describe("Processor Module Document Model", () => {
-    it("should create a new Processor Module document", () => {
+    it("should have correct initial values", () => {
         const document = utils.createDocument();
-        expect(document).toBeDefined();
-        expect(document.header.documentType).toBe("powerhouse/processor");
+        expect(document.state.global.name).toBe("");
+        expect(document.state.global.type).toBe("");
+        expect(document.state.global.documentTypes).toEqual([]);
+        expect(document.state.global.status).toBe("DRAFT");
     });
-    it("should create a new Processor Module document with a valid initial state", () => {
+    it("should handle multiple operations and maintain consistency", () => {
         const document = utils.createDocument();
-        expect(document.state.global).toStrictEqual(initialGlobalState);
-        expect(document.state.local).toStrictEqual(initialLocalState);
+        let updatedDoc = reducer(document, creators.setProcessorName({ name: "Test Processor" }));
+        updatedDoc = reducer(updatedDoc, creators.setProcessorType({ type: "analytics" }));
+        updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+            id: "test-type",
+            documentType: "powerhouse/test",
+        }));
+        updatedDoc = reducer(updatedDoc, creators.setProcessorStatus({ status: "CONFIRMED" }));
+        // Verify state consistency
+        expect(updatedDoc.state.global.name).toBe("Test Processor");
+        expect(updatedDoc.state.global.type).toBe("analytics");
+        expect(updatedDoc.state.global.documentTypes).toHaveLength(1);
+        expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+    });
+    describe("Complex Scenarios", () => {
+        it("should handle workflow: set name/type, add types, confirm status", () => {
+            const document = utils.createDocument();
+            // Step 1: Set processor name and type
+            let updatedDoc = reducer(document, creators.setProcessorName({ name: "Production Processor" }));
+            expect(updatedDoc.state.global.name).toBe("Production Processor");
+            updatedDoc = reducer(updatedDoc, creators.setProcessorType({ type: "data-analytics" }));
+            expect(updatedDoc.state.global.type).toBe("data-analytics");
+            // Step 2: Add document types
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "budget-docs",
+                documentType: "powerhouse/budget",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "invoice-docs",
+                documentType: "powerhouse/invoice",
+            }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Step 3: Confirm processor status
+            updatedDoc = reducer(updatedDoc, creators.setProcessorStatus({ status: "CONFIRMED" }));
+            expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+        });
+        it("should handle reset scenario: remove all types and add new ones", () => {
+            const document = utils.createDocument();
+            // Add initial types
+            let updatedDoc = reducer(document, creators.addDocumentType({
+                id: "old-type-1",
+                documentType: "powerhouse/old1",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "old-type-2",
+                documentType: "powerhouse/old2",
+            }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Remove all types
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "old-type-1" }));
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "old-type-2" }));
+            expect(updatedDoc.state.global.documentTypes).toEqual([]);
+            // Add new types from scratch
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-1",
+                documentType: "powerhouse/new1",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-2",
+                documentType: "powerhouse/new2",
+            }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({
+                id: "new-type-3",
+                documentType: "powerhouse/new3",
+            }));
+            // Verify reset completed
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "old-type-1")).toBeUndefined();
+            expect(updatedDoc.state.global.documentTypes.find((dt) => dt.id === "new-type-1")).toBeDefined();
+        });
+        it("should maintain state integrity with complex add/remove sequences", () => {
+            const document = utils.createDocument();
+            // Add multiple types
+            let updatedDoc = reducer(document, creators.addDocumentType({ id: "type-a", documentType: "a" }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-b", documentType: "b" }));
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-c", documentType: "c" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            // Remove middle one
+            updatedDoc = reducer(updatedDoc, creators.removeDocumentType({ id: "type-b" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(2);
+            // Add it back with new documentType
+            updatedDoc = reducer(updatedDoc, creators.addDocumentType({ id: "type-b", documentType: "b-new" }));
+            expect(updatedDoc.state.global.documentTypes).toHaveLength(3);
+            // Verify order and content
+            const typeB = updatedDoc.state.global.documentTypes.find((dt) => dt.id === "type-b");
+            expect(typeB?.documentType).toBe("b-new");
+        });
     });
 });

package/dist/document-models/subgraph-module/src/reducers/base-operations.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/subgraph-module/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,sCAAsC,EAAE,MAAM,yCAAyC,CAAC;AAEtG,eAAO,MAAM,OAAO,EAAE,sCAOrB,CAAC"}
+{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/subgraph-module/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,sCAAsC,EAAE,MAAM,yCAAyC,CAAC;AAEtG,eAAO,MAAM,OAAO,EAAE,sCAWrB,CAAC"}

package/dist/document-models/subgraph-module/src/reducers/base-operations.js

@@ -1,6 +1,10 @@
 export const reducer = {
     setSubgraphNameOperation(state, action, dispatch) {
-        state.name = action.input.name;
+        const trimmedName = action.input.name.trim();
+        if (trimmedName === "") {
+            throw new Error("Subgraph name cannot be empty");
+        }
+        state.name = trimmedName;
     },
     setSubgraphStatusOperation(state, action, dispatch) {
         state.status = action.input.status;

package/dist/document-models/subgraph-module/src/tests/base-operations.test.js

@@ -2,23 +2,44 @@
  * This is a scaffold file meant for customization:
  * - change it by adding new tests or modifying the existing ones
  */
-import { describe, it, expect, beforeEach } from "vitest";
-import { generateMock } from "@powerhousedao/codegen";
-import utils from "../../gen/utils.js";
-import { z } from "../../gen/schema/index.js";
-import { reducer } from "../../gen/reducer.js";
+import { beforeEach, describe, expect, it } from "vitest";
 import * as creators from "../../gen/base-operations/creators.js";
+import { reducer } from "../../gen/reducer.js";
+import utils from "../../gen/utils.js";
 describe("BaseOperations Operations", () => {
     let document;
     beforeEach(() => {
         document = utils.createDocument();
     });
-    it("should handle setSubgraphName operation", () => {
-        const input = generateMock(z.SetSubgraphNameInputSchema());
-        const updatedDocument = reducer(document, creators.setSubgraphName(input));
-        expect(updatedDocument.operations.global).toHaveLength(1);
-        expect(updatedDocument.operations.global[0].action.type).toBe("SET_SUBGRAPH_NAME");
-        expect(updatedDocument.operations.global[0].action.input).toStrictEqual(input);
-        expect(updatedDocument.operations.global[0].index).toEqual(0);
+    describe("setSubgraphName", () => {
+        it("should mutate state with new name", () => {
+            const input = { name: "My Subgraph" };
+            const updatedDocument = reducer(document, creators.setSubgraphName(input));
+            expect(updatedDocument.state.global.name).toBe("My Subgraph");
+        });
+        it("should reject empty string and store error in operation", () => {
+            const input = { name: "" };
+            const updatedDocument = reducer(document, creators.setSubgraphName(input));
+            expect(updatedDocument.operations.global).toHaveLength(1);
+            expect(updatedDocument.operations.global[0].error).toBe("Subgraph name cannot be empty");
+            expect(updatedDocument.state.global.name).toBe("");
+        });
+    });
+    describe("setSubgraphStatus", () => {
+        it("should mutate state with new status", () => {
+            const input = { status: "CONFIRMED" };
+            const updatedDocument = reducer(document, creators.setSubgraphStatus(input));
+            expect(updatedDocument.state.global.status).toBe("CONFIRMED");
+        });
+        it("should transition from DRAFT to CONFIRMED", () => {
+            expect(document.state.global.status).toBe("DRAFT");
+            const updatedDocument = reducer(document, creators.setSubgraphStatus({ status: "CONFIRMED" }));
+            expect(updatedDocument.state.global.status).toBe("CONFIRMED");
+        });
+        it("should transition from CONFIRMED to DRAFT", () => {
+            const confirmedDoc = reducer(document, creators.setSubgraphStatus({ status: "CONFIRMED" }));
+            const updatedDocument = reducer(confirmedDoc, creators.setSubgraphStatus({ status: "DRAFT" }));
+            expect(updatedDocument.state.global.status).toBe("DRAFT");
+        });
     });
 });

package/dist/document-models/subgraph-module/src/tests/document-model.test.js

@@ -2,17 +2,34 @@
  * This is a scaffold file meant for customization:
  * - change it by adding new tests or modifying the existing ones
  */
-import { describe, it, expect, beforeEach } from "vitest";
-import utils, { initialGlobalState, initialLocalState, } from "../../gen/utils.js";
+import { describe, expect, it } from "vitest";
+import * as creators from "../../gen/base-operations/creators.js";
+import { reducer } from "../../gen/reducer.js";
+import utils from "../../gen/utils.js";
 describe("Subgraph Module Document Model", () => {
-    it("should create a new Subgraph Module document", () => {
+    it("should have correct initial values", () => {
         const document = utils.createDocument();
-        expect(document).toBeDefined();
-        expect(document.header.documentType).toBe("powerhouse/subgraph");
+        expect(document.state.global.name).toBe("");
+        expect(document.state.global.status).toBe("DRAFT");
     });
-    it("should create a new Subgraph Module document with a valid initial state", () => {
+    it("should handle multiple operations and maintain consistency", () => {
         const document = utils.createDocument();
-        expect(document.state.global).toStrictEqual(initialGlobalState);
-        expect(document.state.local).toStrictEqual(initialLocalState);
+        let updatedDoc = reducer(document, creators.setSubgraphName({ name: "Test Subgraph" }));
+        updatedDoc = reducer(updatedDoc, creators.setSubgraphStatus({ status: "CONFIRMED" }));
+        // Verify state consistency
+        expect(updatedDoc.state.global.name).toBe("Test Subgraph");
+        expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+    });
+    it("should handle complete workflow: set name and confirm status", () => {
+        const document = utils.createDocument();
+        // Step 1: Set subgraph name
+        let updatedDoc = reducer(document, creators.setSubgraphName({ name: "Production Subgraph" }));
+        expect(updatedDoc.state.global.name).toBe("Production Subgraph");
+        // Step 2: Confirm subgraph status
+        updatedDoc = reducer(updatedDoc, creators.setSubgraphStatus({ status: "CONFIRMED" }));
+        expect(updatedDoc.state.global.status).toBe("CONFIRMED");
+        // Verify final state
+        expect(updatedDoc.state.global.name).toBe("Production Subgraph");
+        expect(updatedDoc.state.global.status).toBe("CONFIRMED");
     });
 });

package/dist/document-models/vetra-package/src/reducers/base-operations.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/vetra-package/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,oCAAoC,EAAE,MAAM,yCAAyC,CAAC;AAEpG,eAAO,MAAM,OAAO,EAAE,oCAoCrB,CAAC"}
+{"version":3,"file":"base-operations.d.ts","sourceRoot":"","sources":["../../../../../document-models/vetra-package/src/reducers/base-operations.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,oCAAoC,EAAE,MAAM,yCAAyC,CAAC;AAEpG,eAAO,MAAM,OAAO,EAAE,oCA4CrB,CAAC"}

package/dist/document-models/vetra-package/src/reducers/base-operations.js

@@ -21,6 +21,11 @@ export const reducer = {
         state.author.website = action.input.website;
     },
     addPackageKeywordOperation(state, action, dispatch) {
+        // Check for duplicate ID
+        const existingId = state.keywords.find((keyword) => keyword.id === action.input.id);
+        if (existingId) {
+            throw new Error(`Keyword with id "${action.input.id}" already exists`);
+        }
         state.keywords.push(action.input);
     },
     removePackageKeywordOperation(state, action, dispatch) {