@langadventurellc/task-trellis-mcp 1.0.1 → 1.1.0

package/dist/tools/__tests__/updateObjectTool.test.js

@@ -1,16 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const models_1 = require("../../models");
 const updateObjectTool_1 = require("../updateObjectTool");
-// Mock the validateStatusTransition function
-jest.mock("../../validation/validateStatusTransition", () => ({
-    validateStatusTransition: jest.fn(),
-}));
-const validateStatusTransition_1 = require("../../validation/validateStatusTransition");
-const mockValidateStatusTransition = validateStatusTransition_1.validateStatusTransition;
 describe("updateObjectTool", () => {
+    let mockService;
     let mockRepository;
     beforeEach(() => {
+        mockService = {
+            updateObject: jest.fn(),
+        };
         mockRepository = {
             getObjectById: jest.fn(),
             getObjects: jest.fn(),
@@ -20,190 +17,75 @@ describe("updateObjectTool", () => {
         jest.clearAllMocks();
     });
     describe("handleUpdateObject", () => {
-        const mockTrellisObject = {
-            id: "T-test-task",
-            type: models_1.TrellisObjectType.TASK,
-            title: "Test Task",
-            status: models_1.TrellisObjectStatus.OPEN,
-            priority: models_1.TrellisObjectPriority.MEDIUM,
-            parent: "F-test-feature",
-            prerequisites: [],
-            affectedFiles: new Map(),
-            log: [],
-            schema: "1.0",
-            childrenIds: [],
-            body: "This is a test task",
-            created: "2025-01-15T10:00:00Z",
-            updated: "2025-01-15T10:00:00Z",
+        const mockResult = {
+            content: [
+                {
+                    type: "text",
+                    text: "Successfully updated object: {...}",
+                },
+            ],
         };
-        it("should successfully update an object with all properties", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            mockValidateStatusTransition.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
+        it("should call service.updateObject with correct parameters for all properties", async () => {
+            mockService.updateObject.mockResolvedValue(mockResult);
+            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
                 id: "T-test-task",
                 priority: "high",
                 prerequisites: ["T-prereq-1", "T-prereq-2"],
                 body: "Updated body content",
                 status: "draft",
+                force: true,
             });
-            const expectedUpdatedObject = {
-                ...mockTrellisObject,
-                priority: "high",
-                prerequisites: ["T-prereq-1", "T-prereq-2"],
-                body: "Updated body content",
-                status: "draft",
-            };
-            expect(mockRepository.getObjectById).toHaveBeenCalledWith("T-test-task");
-            expect(mockValidateStatusTransition).toHaveBeenCalledWith(expectedUpdatedObject, mockRepository);
-            expect(mockRepository.saveObject).toHaveBeenCalledWith(expectedUpdatedObject);
-            expect(result.content[0].text).toContain("Successfully updated object:");
-            expect(result.content[0].text).toContain("T-test-task");
+            expect(mockService.updateObject).toHaveBeenCalledWith(mockRepository, "T-test-task", "high", ["T-prereq-1", "T-prereq-2"], "Updated body content", "draft", true);
+            expect(result).toBe(mockResult);
         });
-        it("should successfully update only specified properties", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
+        it("should call service.updateObject with only specified properties", async () => {
+            mockService.updateObject.mockResolvedValue(mockResult);
+            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
                 id: "T-test-task",
                 priority: "low",
             });
-            const expectedUpdatedObject = {
-                ...mockTrellisObject,
-                priority: "low",
-            };
-            expect(mockRepository.saveObject).toHaveBeenCalledWith(expectedUpdatedObject);
-            expect(mockValidateStatusTransition).not.toHaveBeenCalled();
-            expect(result.content[0].text).toContain("Successfully updated object:");
+            expect(mockService.updateObject).toHaveBeenCalledWith(mockRepository, "T-test-task", "low", undefined, undefined, undefined, false);
+            expect(result).toBe(mockResult);
         });
-        it("should return error when object is not found", async () => {
-            mockRepository.getObjectById.mockResolvedValue(null);
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-nonexistent",
-                priority: "high",
-            });
-            expect(mockRepository.getObjectById).toHaveBeenCalledWith("T-nonexistent");
-            expect(mockRepository.saveObject).not.toHaveBeenCalled();
-            expect(mockValidateStatusTransition).not.toHaveBeenCalled();
-            expect(result.content[0].text).toBe("Error: Object with ID 'T-nonexistent' not found");
-        });
-        it("should validate status transition when status changes to in-progress", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            mockValidateStatusTransition.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
+        it("should handle missing optional parameters correctly", async () => {
+            mockService.updateObject.mockResolvedValue(mockResult);
+            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
                 id: "T-test-task",
                 status: "in-progress",
             });
-            expect(mockValidateStatusTransition).toHaveBeenCalledWith(expect.objectContaining({
-                ...mockTrellisObject,
-                status: "in-progress",
-            }), mockRepository);
-            expect(mockRepository.saveObject).toHaveBeenCalled();
-            expect(result.content[0].text).toContain("Successfully updated object:");
+            expect(mockService.updateObject).toHaveBeenCalledWith(mockRepository, "T-test-task", undefined, undefined, undefined, "in-progress", false);
+            expect(result).toBe(mockResult);
         });
-        it("should validate status transition when status changes to done", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            mockValidateStatusTransition.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
+        it("should default force to false when not provided", async () => {
+            mockService.updateObject.mockResolvedValue(mockResult);
+            await (0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
                 id: "T-test-task",
                 status: "done",
             });
-            expect(mockValidateStatusTransition).toHaveBeenCalledWith(expect.objectContaining({
-                ...mockTrellisObject,
-                status: "done",
-            }), mockRepository);
-            expect(mockRepository.saveObject).toHaveBeenCalled();
-            expect(result.content[0].text).toContain("Successfully updated object:");
+            expect(mockService.updateObject).toHaveBeenCalledWith(mockRepository, "T-test-task", undefined, undefined, undefined, "done", false);
         });
-        it("should reject status change when validation fails", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            const validationError = new Error("Cannot update status to 'in-progress' - prerequisites are not complete. Use force=true to override.");
-            mockValidateStatusTransition.mockRejectedValue(validationError);
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
-                status: "in-progress",
-            });
-            expect(mockValidateStatusTransition).toHaveBeenCalled();
-            expect(mockRepository.saveObject).not.toHaveBeenCalled();
-            expect(result.content[0].text).toBe("Error updating object: Cannot update status to 'in-progress' - prerequisites are not complete. Use force=true to override.");
-        });
-        it("should bypass validation when force is true", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
-                status: "in-progress",
-                force: true,
-            });
-            expect(mockValidateStatusTransition).not.toHaveBeenCalled();
-            expect(mockRepository.saveObject).toHaveBeenCalled();
-            expect(result.content[0].text).toContain("Successfully updated object:");
-        });
-        it("should not validate status transitions for non-status updates", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
+        it("should pass through service errors", async () => {
+            const errorResult = {
+                content: [
+                    {
+                        type: "text",
+                        text: "Error updating object: Object not found",
+                    },
+                ],
+            };
+            mockService.updateObject.mockResolvedValue(errorResult);
+            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
+                id: "T-nonexistent",
                 priority: "high",
-                body: "Updated body",
             });
-            expect(mockValidateStatusTransition).not.toHaveBeenCalled();
-            expect(mockRepository.saveObject).toHaveBeenCalled();
-            expect(result.content[0].text).toContain("Successfully updated object:");
+            expect(result).toBe(errorResult);
         });
-        it("should handle repository errors gracefully", async () => {
-            mockRepository.getObjectById.mockRejectedValue(new Error("Database connection failed"));
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
+        it("should handle service rejections", async () => {
+            mockService.updateObject.mockRejectedValue(new Error("Service error"));
+            await expect((0, updateObjectTool_1.handleUpdateObject)(mockService, mockRepository, {
                 id: "T-test-task",
                 status: "done",
-            });
-            expect(result.content[0].text).toBe("Error updating object: Database connection failed");
-        });
-        it("should handle saveObject errors gracefully", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockRepository.saveObject.mockRejectedValue(new Error("Failed to save"));
-            mockValidateStatusTransition.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
-                status: "done",
-            });
-            expect(result.content[0].text).toBe("Error updating object: Failed to save");
-        });
-        it("should handle validation errors gracefully", async () => {
-            mockRepository.getObjectById.mockResolvedValue(mockTrellisObject);
-            mockValidateStatusTransition.mockRejectedValue(new Error("Validation failed"));
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
-                status: "in-progress",
-            });
-            expect(result.content[0].text).toBe("Error updating object: Validation failed");
-        });
-        it("should handle complex update with multiple properties and status validation", async () => {
-            const objectWithPrereqs = {
-                ...mockTrellisObject,
-                prerequisites: ["T-prereq-1"],
-            };
-            mockRepository.getObjectById.mockResolvedValue(objectWithPrereqs);
-            mockRepository.saveObject.mockResolvedValue();
-            mockValidateStatusTransition.mockResolvedValue();
-            const result = await (0, updateObjectTool_1.handleUpdateObject)(mockRepository, {
-                id: "T-test-task",
-                priority: "high",
-                prerequisites: ["T-prereq-1", "T-prereq-2", "T-prereq-3"],
-                body: "Updated body with new requirements",
-                status: "in-progress",
-            });
-            const expectedUpdatedObject = {
-                ...objectWithPrereqs,
-                priority: "high",
-                prerequisites: ["T-prereq-1", "T-prereq-2", "T-prereq-3"],
-                body: "Updated body with new requirements",
-                status: "in-progress",
-            };
-            expect(mockValidateStatusTransition).toHaveBeenCalledWith(expectedUpdatedObject, mockRepository);
-            expect(mockRepository.saveObject).toHaveBeenCalledWith(expectedUpdatedObject);
-            expect(result.content[0].text).toContain("Successfully updated object:");
+            })).rejects.toThrow("Service error");
         });
     });
 });

package/dist/tools/__tests__/updateObjectTool.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"updateObjectTool.test.js","sourceRoot":"","sources":["../../../src/tools/__tests__/updateObjectTool.test.ts"],"names":[],"mappings":";;AACA,yCAKsB;AACtB,0DAAyD;AAEzD,6CAA6C;AAC7C,IAAI,CAAC,IAAI,CAAC,2CAA2C,EAAE,GAAG,EAAE,CAAC,CAAC;IAC5D,wBAAwB,EAAE,IAAI,CAAC,EAAE,EAAE;CACpC,CAAC,CAAC,CAAC;AAEJ,wFAAqF;AAErF,MAAM,4BAA4B,GAChC,mDAEC,CAAC;AAEJ,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,IAAI,cAAuC,CAAC;IAE5C,UAAU,CAAC,GAAG,EAAE;QACd,cAAc,GAAG;YACf,aAAa,EAAE,IAAI,CAAC,EAAE,EAAE;YACxB,UAAU,EAAE,IAAI,CAAC,EAAE,EAAE;YACrB,UAAU,EAAE,IAAI,CAAC,EAAE,EAAE;YACrB,YAAY,EAAE,IAAI,CAAC,EAAE,EAAE;SACxB,CAAC;QACF,IAAI,CAAC,aAAa,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,MAAM,iBAAiB,GAAkB;YACvC,EAAE,EAAE,aAAa;YACjB,IAAI,EAAE,0BAAiB,CAAC,IAAI;YAC5B,KAAK,EAAE,WAAW;YAClB,MAAM,EAAE,4BAAmB,CAAC,IAAI;YAChC,QAAQ,EAAE,8BAAqB,CAAC,MAAM;YACtC,MAAM,EAAE,gBAAgB;YACxB,aAAa,EAAE,EAAE;YACjB,aAAa,EAAE,IAAI,GAAG,EAAE;YACxB,GAAG,EAAE,EAAE;YACP,MAAM,EAAE,KAAK;YACb,WAAW,EAAE,EAAE;YACf,IAAI,EAAE,qBAAqB;YAC3B,OAAO,EAAE,sBAAsB;YAC/B,OAAO,EAAE,sBAAsB;SAChC,CAAC;QAEF,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;YACxE,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAC9C,4BAA4B,CAAC,iBAAiB,EAAE,CAAC;YAEjD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,MAAM;gBAChB,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,CAAC;gBAC3C,IAAI,EAAE,sBAAsB;gBAC5B,MAAM,EAAE,OAAO;aAChB,CAAC,CAAC;YAEH,MAAM,qBAAqB,GAAG;gBAC5B,GAAG,iBAAiB;gBACpB,QAAQ,EAAE,MAAM;gBAChB,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,CAAC;gBAC3C,IAAI,EAAE,sBAAsB;gBAC5B,MAAM,EAAE,OAAO;aAChB,CAAC;YAEF,MAAM,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAAC,aAAa,CAAC,CAAC;YACzE,MAAM,CAAC,4BAA4B,CAAC,CAAC,oBAAoB,CACvD,qBAAqB,EACrB,cAAc,CACf,CAAC;YACF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACpD,qBAAqB,CACtB,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;YACzE,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;QAC1D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;YACpE,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAE9C,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,KAAK;aAChB,CAAC,CAAC;YAEH,MAAM,qBAAqB,GAAG;gBAC5B,GAAG,iBAAiB;gBACpB,QAAQ,EAAE,KAAK;aAChB,CAAC;YAEF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACpD,qBAAqB,CACtB,CAAC;YACF,MAAM,CAAC,4BAA4B,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAC5D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;YAC5D,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;YAErD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,eAAe;gBACnB,QAAQ,EAAE,MAAM;aACjB,CAAC,CAAC;YAEH,MAAM,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC,oBAAoB,CACvD,eAAe,CAChB,CAAC;YACF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YACzD,MAAM,CAAC,4BAA4B,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAC5D,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CACjC,iDAAiD,CAClD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;YACpF,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAC9C,4BAA4B,CAAC,iBAAiB,EAAE,CAAC;YAEjD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,aAAa;aACtB,CAAC,CAAC;YAEH,MAAM,CAAC,4BAA4B,CAAC,CAAC,oBAAoB,CACvD,MAAM,CAAC,gBAAgB,CAAC;gBACtB,GAAG,iBAAiB;gBACpB,MAAM,EAAE,aAAa;aACtB,CAAC,EACF,cAAc,CACf,CAAC;YACF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,gBAAgB,EAAE,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;YAC7E,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAC9C,4BAA4B,CAAC,iBAAiB,EAAE,CAAC;YAEjD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,MAAM;aACf,CAAC,CAAC;YAEH,MAAM,CAAC,4BAA4B,CAAC,CAAC,oBAAoB,CACvD,MAAM,CAAC,gBAAgB,CAAC;gBACtB,GAAG,iBAAiB;gBACpB,MAAM,EAAE,MAAM;aACf,CAAC,EACF,cAAc,CACf,CAAC;YACF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,gBAAgB,EAAE,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;YACjE,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,MAAM,eAAe,GAAG,IAAI,KAAK,CAC/B,qGAAqG,CACtG,CAAC;YACF,4BAA4B,CAAC,iBAAiB,CAAC,eAAe,CAAC,CAAC;YAEhE,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,aAAa;aACtB,CAAC,CAAC;YAEH,MAAM,CAAC,4BAA4B,CAAC,CAAC,gBAAgB,EAAE,CAAC;YACxD,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YACzD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CACjC,4HAA4H,CAC7H,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,6CAA6C,EAAE,KAAK,IAAI,EAAE;YAC3D,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAE9C,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,aAAa;gBACrB,KAAK,EAAE,IAAI;aACZ,CAAC,CAAC;YAEH,MAAM,CAAC,4BAA4B,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAC5D,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,gBAAgB,EAAE,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;YAC7E,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAE9C,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,MAAM;gBAChB,IAAI,EAAE,cAAc;aACrB,CAAC,CAAC;YAEH,MAAM,CAAC,4BAA4B,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;YAC5D,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,gBAAgB,EAAE,CAAC;YACrD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;YAC1D,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAC5C,IAAI,KAAK,CAAC,4BAA4B,CAAC,CACxC,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,MAAM;aACf,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CACjC,mDAAmD,CACpD,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;YAC1D,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,gBAAgB,CAAC,CAAC,CAAC;YACzE,4BAA4B,CAAC,iBAAiB,EAAE,CAAC;YAEjD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,MAAM;aACf,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CACjC,uCAAuC,CACxC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;YAC1D,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,4BAA4B,CAAC,iBAAiB,CAC5C,IAAI,KAAK,CAAC,mBAAmB,CAAC,CAC/B,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,aAAa;aACtB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CACjC,0CAA0C,CAC3C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;YAC3F,MAAM,iBAAiB,GAAG;gBACxB,GAAG,iBAAiB;gBACpB,aAAa,EAAE,CAAC,YAAY,CAAC;aAC9B,CAAC;YAEF,cAAc,CAAC,aAAa,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC;YAClE,cAAc,CAAC,UAAU,CAAC,iBAAiB,EAAE,CAAC;YAC9C,4BAA4B,CAAC,iBAAiB,EAAE,CAAC;YAEjD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,cAAc,EAAE;gBACtD,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,MAAM;gBAChB,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,EAAE,YAAY,CAAC;gBACzD,IAAI,EAAE,oCAAoC;gBAC1C,MAAM,EAAE,aAAa;aACtB,CAAC,CAAC;YAEH,MAAM,qBAAqB,GAAG;gBAC5B,GAAG,iBAAiB;gBACpB,QAAQ,EAAE,MAAM;gBAChB,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,EAAE,YAAY,CAAC;gBACzD,IAAI,EAAE,oCAAoC;gBAC1C,MAAM,EAAE,aAAa;aACtB,CAAC;YAEF,MAAM,CAAC,4BAA4B,CAAC,CAAC,oBAAoB,CACvD,qBAAqB,EACrB,cAAc,CACf,CAAC;YACF,MAAM,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACpD,qBAAqB,CACtB,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QAC3E,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"updateObjectTool.test.js","sourceRoot":"","sources":["../../../src/tools/__tests__/updateObjectTool.test.ts"],"names":[],"mappings":";;AAGA,0DAAyD;AAEzD,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,IAAI,WAA4C,CAAC;IACjD,IAAI,cAAuC,CAAC;IAE5C,UAAU,CAAC,GAAG,EAAE;QACd,WAAW,GAAG;YACZ,YAAY,EAAE,IAAI,CAAC,EAAE,EAAE;SACsB,CAAC;QAEhD,cAAc,GAAG;YACf,aAAa,EAAE,IAAI,CAAC,EAAE,EAAE;YACxB,UAAU,EAAE,IAAI,CAAC,EAAE,EAAE;YACrB,UAAU,EAAE,IAAI,CAAC,EAAE,EAAE;YACrB,YAAY,EAAE,IAAI,CAAC,EAAE,EAAE;SACxB,CAAC;QAEF,IAAI,CAAC,aAAa,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,MAAM,UAAU,GAAG;YACjB,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,oCAAoC;iBAC3C;aACF;SACF,CAAC;QAEF,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;YAC3F,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,UAAU,CAAC,CAAC;YAEvD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBACnE,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,MAAM;gBAChB,aAAa,EAAE,CAAC,YAAY,EAAE,YAAY,CAAC;gBAC3C,IAAI,EAAE,sBAAsB;gBAC5B,MAAM,EAAE,OAAO;gBACf,KAAK,EAAE,IAAI;aACZ,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,MAA+B,EAC/B,CAAC,YAAY,EAAE,YAAY,CAAC,EAC5B,sBAAsB,EACtB,OAA8B,EAC9B,IAAI,CACL,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAClC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;YAC/E,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,UAAU,CAAC,CAAC;YAEvD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBACnE,EAAE,EAAE,aAAa;gBACjB,QAAQ,EAAE,KAAK;aAChB,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,KAA8B,EAC9B,SAAS,EACT,SAAS,EACT,SAAS,EACT,KAAK,CACN,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAClC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,UAAU,CAAC,CAAC;YAEvD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBACnE,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,aAAa;aACtB,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,SAAS,EACT,SAAS,EACT,SAAS,EACT,aAAoC,EACpC,KAAK,CACN,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAClC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,UAAU,CAAC,CAAC;YAEvD,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBACpD,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,MAAM;aACf,CAAC,CAAC;YAEH,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,SAAS,EACT,SAAS,EACT,SAAS,EACT,MAA6B,EAC7B,KAAK,CACN,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;YAClD,MAAM,WAAW,GAAG;gBAClB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,yCAAyC;qBAChD;iBACF;aACF,CAAC;YACF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,WAAW,CAAC,CAAC;YAExD,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBACnE,EAAE,EAAE,eAAe;gBACnB,QAAQ,EAAE,MAAM;aACjB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kCAAkC,EAAE,KAAK,IAAI,EAAE;YAChD,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;YAEvE,MAAM,MAAM,CACV,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE;gBAC9C,EAAE,EAAE,aAAa;gBACjB,MAAM,EAAE,MAAM;aACf,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC;QACrC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tools/appendModifiedFilesTool.d.ts

@@ -0,0 +1,30 @@
+import { Repository } from "../repositories";
+import { TaskTrellisService } from "../services/TaskTrellisService";
+export declare const appendModifiedFilesTool: {
+    readonly name: "append_modified_files";
+    readonly description: "Appends modified files information to a trellis object in the task trellis system\n\nUse this tool to record files that have been modified during task execution, along with descriptions of the modifications made. This helps maintain a comprehensive record of changes associated with each work item for tracking and audit purposes.\n\nFile modification tracking purposes:\n- Record which files were changed during task execution\n- Document the nature of changes made to each file\n- Maintain audit trail of file-level modifications\n- Support code review and change management processes\n- Enable impact analysis for future changes\n\nInput requirements:\n- Object ID: The unique identifier of the trellis object to update\n- Files Changed: A record mapping file paths to descriptions of modifications\n\nFile path guidelines:\n- Use relative paths from project root (e.g., \"src/components/Button.tsx\")\n- Include file extensions for clarity\n- Use forward slashes for path separators\n\nDescription guidelines:\n- Provide clear, concise descriptions of what was changed\n- Focus on the purpose and impact of changes rather than implementation details\n- Use consistent terminology across related modifications\n\nThe tool automatically merges descriptions for files that were previously modified, creating a comprehensive change history for each file within the context of the trellis object.";
+    readonly inputSchema: {
+        readonly type: "object";
+        readonly properties: {
+            readonly id: {
+                readonly type: "string";
+                readonly description: "ID of the trellis object to update with modified files information";
+            };
+            readonly filesChanged: {
+                readonly type: "object";
+                readonly description: "Record of file paths to descriptions of modifications made";
+                readonly additionalProperties: {
+                    readonly type: "string";
+                };
+            };
+        };
+        readonly required: readonly ["id", "filesChanged"];
+    };
+};
+export declare function handleAppendModifiedFiles(service: TaskTrellisService, repository: Repository, args: unknown): Promise<{
+    content: Array<{
+        type: string;
+        text: string;
+    }>;
+}>;
+//# sourceMappingURL=appendModifiedFilesTool.d.ts.map

package/dist/tools/appendModifiedFilesTool.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"appendModifiedFilesTool.d.ts","sourceRoot":"","sources":["../../src/tools/appendModifiedFilesTool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,kBAAkB,EAAE,MAAM,gCAAgC,CAAC;AAEpE,eAAO,MAAM,uBAAuB;;;;;;;;;;;;;;;;;;;;CA+C1B,CAAC;AAEX,wBAAsB,yBAAyB,CAC7C,OAAO,EAAE,kBAAkB,EAC3B,UAAU,EAAE,UAAU,EACtB,IAAI,EAAE,OAAO,GACZ,OAAO,CAAC;IAAE,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;CAAE,CAAC,CAO7D"}

package/dist/tools/appendModifiedFilesTool.js

@@ -0,0 +1,55 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.appendModifiedFilesTool = void 0;
+exports.handleAppendModifiedFiles = handleAppendModifiedFiles;
+exports.appendModifiedFilesTool = {
+    name: "append_modified_files",
+    description: `Appends modified files information to a trellis object in the task trellis system
+
+Use this tool to record files that have been modified during task execution, along with descriptions of the modifications made. This helps maintain a comprehensive record of changes associated with each work item for tracking and audit purposes.
+
+File modification tracking purposes:
+- Record which files were changed during task execution
+- Document the nature of changes made to each file
+- Maintain audit trail of file-level modifications
+- Support code review and change management processes
+- Enable impact analysis for future changes
+
+Input requirements:
+- Object ID: The unique identifier of the trellis object to update
+- Files Changed: A record mapping file paths to descriptions of modifications
+
+File path guidelines:
+- Use relative paths from project root (e.g., "src/components/Button.tsx")
+- Include file extensions for clarity
+- Use forward slashes for path separators
+
+Description guidelines:
+- Provide clear, concise descriptions of what was changed
+- Focus on the purpose and impact of changes rather than implementation details
+- Use consistent terminology across related modifications
+
+The tool automatically merges descriptions for files that were previously modified, creating a comprehensive change history for each file within the context of the trellis object.`,
+    inputSchema: {
+        type: "object",
+        properties: {
+            id: {
+                type: "string",
+                description: "ID of the trellis object to update with modified files information",
+            },
+            filesChanged: {
+                type: "object",
+                description: "Record of file paths to descriptions of modifications made",
+                additionalProperties: {
+                    type: "string",
+                },
+            },
+        },
+        required: ["id", "filesChanged"],
+    },
+};
+async function handleAppendModifiedFiles(service, repository, args) {
+    const { id, filesChanged } = args;
+    return await service.appendModifiedFiles(repository, id, filesChanged);
+}
+//# sourceMappingURL=appendModifiedFilesTool.js.map

package/dist/tools/appendModifiedFilesTool.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"appendModifiedFilesTool.js","sourceRoot":"","sources":["../../src/tools/appendModifiedFilesTool.ts"],"names":[],"mappings":";;;AAoDA,8DAWC;AA5DY,QAAA,uBAAuB,GAAG;IACrC,IAAI,EAAE,uBAAuB;IAC7B,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;oLAyBqK;IAClL,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,EAAE,EAAE;gBACF,IAAI,EAAE,QAAQ;gBACd,WAAW,EACT,oEAAoE;aACvE;YACD,YAAY,EAAE;gBACZ,IAAI,EAAE,QAAQ;gBACd,WAAW,EACT,4DAA4D;gBAC9D,oBAAoB,EAAE;oBACpB,IAAI,EAAE,QAAQ;iBACf;aACF;SACF;QACD,QAAQ,EAAE,CAAC,IAAI,EAAE,cAAc,CAAC;KACjC;CACO,CAAC;AAEJ,KAAK,UAAU,yBAAyB,CAC7C,OAA2B,EAC3B,UAAsB,EACtB,IAAa;IAEb,MAAM,EAAE,EAAE,EAAE,YAAY,EAAE,GAAG,IAG5B,CAAC;IAEF,OAAO,MAAM,OAAO,CAAC,mBAAmB,CAAC,UAAU,EAAE,EAAE,EAAE,YAAY,CAAC,CAAC;AACzE,CAAC"}

package/dist/tools/appendObjectLogTool.d.ts

@@ -1,4 +1,5 @@
 import { Repository } from "../repositories";
+import { TaskTrellisService } from "../services/TaskTrellisService";
 export declare const appendObjectLogTool: {
     readonly name: "append_object_log";
     readonly description: "Appends content to an object's log in the task trellis system\n\nUse this tool to add progress updates, notes, or activity records to an object's audit trail. Essential for tracking work history, documenting decisions, and maintaining transparency in task execution.\n\nLog entry purposes:\n- Record progress milestones and status changes\n- Document challenges encountered and solutions applied\n- Note important decisions or changes in approach\n- Track time spent or resources used\n- Log external dependencies or blockers\n- Record completion details and outcomes\n\nLog content guidelines:\n- Use clear, descriptive entries with context\n- Include timestamps (automatically added by system)\n- Reference specific files, commits, or external resources when relevant\n- Note any changes to scope, requirements, or approach\n- Document lessons learned or insights gained\n\nActivity tracking patterns:\n- Starting work: \"Started implementation of feature X\"\n- Progress updates: \"Completed database schema changes, moving to API layer\"\n- Blocking issues: \"Blocked on external API access, contacted team\"\n- Problem resolution: \"Resolved memory leak by optimizing data structure\"\n- Completion: \"Task completed, all tests passing, PR submitted\"\n\nThe log creates a permanent audit trail that helps with project retrospectives, debugging issues, and understanding work evolution over time.";
@@ -17,10 +18,10 @@ export declare const appendObjectLogTool: {
         readonly required: readonly ["id", "contents"];
     };
 };
-export declare function handleAppendObjectLog(repository: Repository, args: unknown): Promise<{
-    content: {
+export declare function handleAppendObjectLog(service: TaskTrellisService, repository: Repository, args: unknown): Promise<{
+    content: Array<{
         type: string;
         text: string;
-    }[];
+    }>;
 }>;
 //# sourceMappingURL=appendObjectLogTool.d.ts.map

package/dist/tools/appendObjectLogTool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"appendObjectLogTool.d.ts","sourceRoot":"","sources":["../../src/tools/appendObjectLogTool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAE7C,eAAO,MAAM,mBAAmB;;;;;;;;;;;;;;;;;CA2CtB,CAAC;AAEX,wBAAsB,qBAAqB,CACzC,UAAU,EAAE,UAAU,EACtB,IAAI,EAAE,OAAO;;;;;GAoDd"}
+{"version":3,"file":"appendObjectLogTool.d.ts","sourceRoot":"","sources":["../../src/tools/appendObjectLogTool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,kBAAkB,EAAE,MAAM,gCAAgC,CAAC;AAEpE,eAAO,MAAM,mBAAmB;;;;;;;;;;;;;;;;;CA2CtB,CAAC;AAEX,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,kBAAkB,EAC3B,UAAU,EAAE,UAAU,EACtB,IAAI,EAAE,OAAO;;;;;GAQd"}

package/dist/tools/appendObjectLogTool.js

@@ -46,46 +46,8 @@ The log creates a permanent audit trail that helps with project retrospectives,
         required: ["id", "contents"],
     },
 };
-async function handleAppendObjectLog(repository, args) {
+async function handleAppendObjectLog(service, repository, args) {
     const { id, contents } = args;
-    try {
-        // Load the existing object
-        const existingObject = await repository.getObjectById(id);
-        if (!existingObject) {
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `Error: Object with ID '${id}' not found`,
-                    },
-                ],
-            };
-        }
-        // Create updated object with new log entry appended
-        const updatedObject = {
-            ...existingObject,
-            log: [...existingObject.log, contents],
-        };
-        // Save the updated object
-        await repository.saveObject(updatedObject);
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `Successfully appended to object log: ${JSON.stringify({ id, contents, totalLogEntries: updatedObject.log.length }, null, 2)}`,
-                },
-            ],
-        };
-    }
-    catch (error) {
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `Error appending to object log: ${error instanceof Error ? error.message : String(error)}`,
-                },
-            ],
-        };
-    }
+    return service.appendObjectLog(repository, id, contents);
 }
 //# sourceMappingURL=appendObjectLogTool.js.map

package/dist/tools/appendObjectLogTool.js.map

@@ -1 +1 @@
-{"version":3,"file":"appendObjectLogTool.js","sourceRoot":"","sources":["../../src/tools/appendObjectLogTool.ts"],"names":[],"mappings":";;;AA+CA,sDAsDC;AAnGY,QAAA,mBAAmB,GAAG;IACjC,IAAI,EAAE,mBAAmB;IACzB,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;8IA0B+H;IAC5I,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,EAAE,EAAE;gBACF,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,mCAAmC;aACjD;YACD,QAAQ,EAAE;gBACR,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,+BAA+B;aAC7C;SACF;QACD,QAAQ,EAAE,CAAC,IAAI,EAAE,UAAU,CAAC;KAC7B;CACO,CAAC;AAEJ,KAAK,UAAU,qBAAqB,CACzC,UAAsB,EACtB,IAAa;IAEb,MAAM,EAAE,EAAE,EAAE,QAAQ,EAAE,GAAG,IAGxB,CAAC;IAEF,IAAI,CAAC;QACH,2BAA2B;QAC3B,MAAM,cAAc,GAAG,MAAM,UAAU,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC;QAC1D,IAAI,CAAC,cAAc,EAAE,CAAC;YACpB,OAAO;gBACL,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,0BAA0B,EAAE,aAAa;qBAChD;iBACF;aACF,CAAC;QACJ,CAAC;QAED,oDAAoD;QACpD,MAAM,aAAa,GAAG;YACpB,GAAG,cAAc;YACjB,GAAG,EAAE,CAAC,GAAG,cAAc,CAAC,GAAG,EAAE,QAAQ,CAAC;SACvC,CAAC;QAEF,0BAA0B;QAC1B,MAAM,UAAU,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC;QAE3C,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,wCAAwC,IAAI,CAAC,SAAS,CAC1D,EAAE,EAAE,EAAE,QAAQ,EAAE,eAAe,EAAE,aAAa,CAAC,GAAG,CAAC,MAAM,EAAE,EAC3D,IAAI,EACJ,CAAC,CACF,EAAE;iBACJ;aACF;SACF,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,kCAAkC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE;iBACjG;aACF;SACF,CAAC;IACJ,CAAC;AACH,CAAC"}
+{"version":3,"file":"appendObjectLogTool.js","sourceRoot":"","sources":["../../src/tools/appendObjectLogTool.ts"],"names":[],"mappings":";;;AAgDA,sDAWC;AAxDY,QAAA,mBAAmB,GAAG;IACjC,IAAI,EAAE,mBAAmB;IACzB,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;8IA0B+H;IAC5I,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,EAAE,EAAE;gBACF,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,mCAAmC;aACjD;YACD,QAAQ,EAAE;gBACR,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,+BAA+B;aAC7C;SACF;QACD,QAAQ,EAAE,CAAC,IAAI,EAAE,UAAU,CAAC;KAC7B;CACO,CAAC;AAEJ,KAAK,UAAU,qBAAqB,CACzC,OAA2B,EAC3B,UAAsB,EACtB,IAAa;IAEb,MAAM,EAAE,EAAE,EAAE,QAAQ,EAAE,GAAG,IAGxB,CAAC;IAEF,OAAO,OAAO,CAAC,eAAe,CAAC,UAAU,EAAE,EAAE,EAAE,QAAQ,CAAC,CAAC;AAC3D,CAAC"}

package/dist/tools/claimTaskTool.d.ts

@@ -1,4 +1,5 @@
 import { Repository } from "../repositories";
+import { TaskTrellisService } from "../services/TaskTrellisService";
 export declare const claimTaskTool: {
     readonly name: "claim_task";
     readonly description: "Claims a task in the task trellis system\n\nUse this tool to assign yourself to available tasks for execution. Essential for AI agents to pick up work items from the task queue and begin execution.\n\nClaiming behavior:\n- Without 'taskId': Claims the next available task based on priority and readiness\n- With 'taskId': Claims a specific task by ID if available and ready\n- 'scope': Limits claiming to tasks within a specific project or area\n- 'force': Overrides normal claiming restrictions (use with caution)\n\nTask readiness criteria:\n- Task status allows claiming (typically 'draft' or 'open' states)\n- All prerequisites are satisfied (prerequisite tasks completed)\n- Task is not already claimed by another agent\n- Task falls within specified scope if provided\n\nClaiming workflow:\n1. System evaluates available tasks against readiness criteria\n2. Selects highest priority task that meets requirements\n3. Updates task status to 'in-progress\n4. Associates task with the claiming agent\n5. Returns claimed task details for execution\n\nCommon patterns:\n- Claim any ready task: (no parameters)\n- Claim from project: scope='P-project-name'\n- Claim specific task: taskId='T-specific-task-id'\n- Force claim blocked task: taskId='T-task-id', force=true\n\nEssential for autonomous task execution workflows where agents need to discover and claim work items dynamically.";
@@ -21,10 +22,10 @@ export declare const claimTaskTool: {
         };
     };
 };
-export declare function handleClaimTask(repository: Repository, args: unknown): Promise<{
-    content: {
+export declare function handleClaimTask(service: TaskTrellisService, repository: Repository, args: unknown): Promise<{
+    content: Array<{
         type: string;
         text: string;
-    }[];
+    }>;
 }>;
 //# sourceMappingURL=claimTaskTool.d.ts.map

package/dist/tools/claimTaskTool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"claimTaskTool.d.ts","sourceRoot":"","sources":["../../src/tools/claimTaskTool.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAK7C,eAAO,MAAM,aAAa;;;;;;;;;;;;;;;;;;;;;CAkDhB,CAAC;AAEX,wBAAsB,eAAe,CAAC,UAAU,EAAE,UAAU,EAAE,IAAI,EAAE,OAAO;;;;;GAwC1E"}
+{"version":3,"file":"claimTaskTool.d.ts","sourceRoot":"","sources":["../../src/tools/claimTaskTool.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,kBAAkB,EAAE,MAAM,gCAAgC,CAAC;AAEpE,eAAO,MAAM,aAAa;;;;;;;;;;;;;;;;;;;;;CAkDhB,CAAC;AAEX,wBAAsB,eAAe,CACnC,OAAO,EAAE,kBAAkB,EAC3B,UAAU,EAAE,UAAU,EACtB,IAAI,EAAE,OAAO;;;;;GAad"}

package/dist/tools/claimTaskTool.js

@@ -2,12 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.claimTaskTool = void 0;
 exports.handleClaimTask = handleClaimTask;
-const models_1 = require("../models");
-const TrellisObjectStatus_1 = require("../models/TrellisObjectStatus");
-const TrellisObjectType_1 = require("../models/TrellisObjectType");
-const checkPrerequisitesComplete_1 = require("../utils/checkPrerequisitesComplete");
-const filterUnavailableObjects_1 = require("../utils/filterUnavailableObjects");
-const sortTrellisObjects_1 = require("../utils/sortTrellisObjects");
 exports.claimTaskTool = {
     name: "claim_task",
     description: `Claims a task in the task trellis system
@@ -59,80 +53,8 @@ Essential for autonomous task execution workflows where agents need to discover
         },
     },
 };
-async function handleClaimTask(repository, args) {
+async function handleClaimTask(service, repository, args) {
     const { scope, taskId, force = false, } = args;
-    try {
-        let claimedTask;
-        if (taskId) {
-            claimedTask = await claimSpecificTask(taskId, force, repository);
-        }
-        else {
-            claimedTask = await findNextAvailableTask(scope, repository);
-        }
-        const updatedTask = await updateTaskStatus(claimedTask, repository);
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `Successfully claimed task: ${JSON.stringify(updatedTask, null, 2)}`,
-                },
-            ],
-        };
-    }
-    catch (error) {
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `Error claiming task: ${error instanceof Error ? error.message : String(error)}`,
-                },
-            ],
-        };
-    }
-}
-async function validateTaskForClaiming(task, taskId, force, repository) {
-    // Validate it's a task
-    if (task.type !== TrellisObjectType_1.TrellisObjectType.TASK) {
-        throw new Error(`Object with ID "${taskId}" is not a task (type: ${task.type})`);
-    }
-    if (!force) {
-        // Validate status is draft or open
-        if (!(0, models_1.isClaimable)(task)) {
-            throw new Error(`Task "${taskId}" cannot be claimed (status: ${task.status}). Task must be in draft or open status.`);
-        }
-        // Validate all prerequisites are complete
-        const prerequisitesComplete = await (0, checkPrerequisitesComplete_1.checkPrerequisitesComplete)(task, repository);
-        if (!prerequisitesComplete) {
-            throw new Error(`Task "${taskId}" cannot be claimed. Not all prerequisites are complete.`);
-        }
-    }
-}
-async function claimSpecificTask(taskId, force, repository) {
-    const object = await repository.getObjectById(taskId);
-    if (!object) {
-        throw new Error(`Task with ID "${taskId}" not found`);
-    }
-    await validateTaskForClaiming(object, taskId, force, repository);
-    return object;
-}
-async function findNextAvailableTask(scope, repository) {
-    const objects = await repository.getObjects(false, // includeClosed
-    scope, TrellisObjectType_1.TrellisObjectType.TASK);
-    // Filter to get only available tasks
-    const availableTasks = (0, filterUnavailableObjects_1.filterUnavailableObjects)(objects);
-    if (availableTasks.length === 0) {
-        throw new Error("No available tasks to claim");
-    }
-    // Sort by priority and return the top one
-    const sortedTasks = (0, sortTrellisObjects_1.sortTrellisObjects)(availableTasks);
-    return sortedTasks[0];
-}
-async function updateTaskStatus(task, repository) {
-    const updatedTask = {
-        ...task,
-        status: TrellisObjectStatus_1.TrellisObjectStatus.IN_PROGRESS,
-    };
-    await repository.saveObject(updatedTask);
-    return updatedTask;
+    return service.claimTask(repository, scope, taskId, force);
 }
 //# sourceMappingURL=claimTaskTool.js.map

package/dist/tools/claimTaskTool.js.map

@@ -1 +1 @@
-{"version":3,"file":"claimTaskTool.js","sourceRoot":"","sources":["../../src/tools/claimTaskTool.ts"],"names":[],"mappings":";;;AA6DA,0CAwCC;AArGD,sCAAwC;AAExC,uEAAoE;AACpE,mEAAgE;AAEhE,oFAAiF;AACjF,gFAA6E;AAC7E,oEAAiE;AAEpD,QAAA,aAAa,GAAG;IAC3B,IAAI,EAAE,YAAY;IAClB,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;kHA6BmG;IAChH,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,KAAK,EAAE;gBACL,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,qCAAqC;aACnD;YACD,MAAM,EAAE;gBACN,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,sCAAsC;aACpD;YACD,KAAK,EAAE;gBACL,IAAI,EAAE,SAAS;gBACf,WAAW,EAAE,sCAAsC;gBACnD,OAAO,EAAE,KAAK;aACf;SACF;KACF;CACO,CAAC;AAEJ,KAAK,UAAU,eAAe,CAAC,UAAsB,EAAE,IAAa;IACzE,MAAM,EACJ,KAAK,EACL,MAAM,EACN,KAAK,GAAG,KAAK,GACd,GAAG,IAIH,CAAC;IAEF,IAAI,CAAC;QACH,IAAI,WAA0B,CAAC;QAE/B,IAAI,MAAM,EAAE,CAAC;YACX,WAAW,GAAG,MAAM,iBAAiB,CAAC,MAAM,EAAE,KAAK,EAAE,UAAU,CAAC,CAAC;QACnE,CAAC;aAAM,CAAC;YACN,WAAW,GAAG,MAAM,qBAAqB,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC;QAC/D,CAAC;QAED,MAAM,WAAW,GAAG,MAAM,gBAAgB,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;QAEpE,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,8BAA8B,IAAI,CAAC,SAAS,CAAC,WAAW,EAAE,IAAI,EAAE,CAAC,CAAC,EAAE;iBAC3E;aACF;SACF,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO;YACL,OAAO,EAAE;gBACP;oBACE,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,wBAAwB,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE;iBACvF;aACF;SACF,CAAC;IACJ,CAAC;AACH,CAAC;AAED,KAAK,UAAU,uBAAuB,CACpC,IAAmB,EACnB,MAAc,EACd,KAAc,EACd,UAAsB;IAEtB,uBAAuB;IACvB,IAAI,IAAI,CAAC,IAAI,KAAK,qCAAiB,CAAC,IAAI,EAAE,CAAC;QACzC,MAAM,IAAI,KAAK,CACb,mBAAmB,MAAM,0BAA0B,IAAI,CAAC,IAAI,GAAG,CAChE,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,mCAAmC;QACnC,IAAI,CAAC,IAAA,oBAAW,EAAC,IAAI,CAAC,EAAE,CAAC;YACvB,MAAM,IAAI,KAAK,CACb,SAAS,MAAM,gCAAgC,IAAI,CAAC,MAAM,0CAA0C,CACrG,CAAC;QACJ,CAAC;QAED,0CAA0C;QAC1C,MAAM,qBAAqB,GAAG,MAAM,IAAA,uDAA0B,EAC5D,IAAI,EACJ,UAAU,CACX,CAAC;QACF,IAAI,CAAC,qBAAqB,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CACb,SAAS,MAAM,0DAA0D,CAC1E,CAAC;QACJ,CAAC;IACH,CAAC;AACH,CAAC;AAED,KAAK,UAAU,iBAAiB,CAC9B,MAAc,EACd,KAAc,EACd,UAAsB;IAEtB,MAAM,MAAM,GAAG,MAAM,UAAU,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC;IAEtD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,aAAa,CAAC,CAAC;IACxD,CAAC;IAED,MAAM,uBAAuB,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,CAAC,CAAC;IACjE,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,KAAK,UAAU,qBAAqB,CAClC,KAAyB,EACzB,UAAsB;IAEtB,MAAM,OAAO,GAAG,MAAM,UAAU,CAAC,UAAU,CACzC,KAAK,EAAE,gBAAgB;IACvB,KAAK,EACL,qCAAiB,CAAC,IAAI,CACvB,CAAC;IAEF,qCAAqC;IACrC,MAAM,cAAc,GAAG,IAAA,mDAAwB,EAAC,OAAO,CAAC,CAAC;IAEzD,IAAI,cAAc,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,6BAA6B,CAAC,CAAC;IACjD,CAAC;IAED,0CAA0C;IAC1C,MAAM,WAAW,GAAG,IAAA,uCAAkB,EAAC,cAAc,CAAC,CAAC;IACvD,OAAO,WAAW,CAAC,CAAC,CAAC,CAAC;AACxB,CAAC;AAED,KAAK,UAAU,gBAAgB,CAC7B,IAAmB,EACnB,UAAsB;IAEtB,MAAM,WAAW,GAAG;QAClB,GAAG,IAAI;QACP,MAAM,EAAE,yCAAmB,CAAC,WAAW;KACxC,CAAC;IAEF,MAAM,UAAU,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;IACzC,OAAO,WAAW,CAAC;AACrB,CAAC"}
+{"version":3,"file":"claimTaskTool.js","sourceRoot":"","sources":["../../src/tools/claimTaskTool.ts"],"names":[],"mappings":";;;AAuDA,0CAgBC;AApEY,QAAA,aAAa,GAAG;IAC3B,IAAI,EAAE,YAAY;IAClB,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;kHA6BmG;IAChH,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,KAAK,EAAE;gBACL,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,qCAAqC;aACnD;YACD,MAAM,EAAE;gBACN,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,sCAAsC;aACpD;YACD,KAAK,EAAE;gBACL,IAAI,EAAE,SAAS;gBACf,WAAW,EAAE,sCAAsC;gBACnD,OAAO,EAAE,KAAK;aACf;SACF;KACF;CACO,CAAC;AAEJ,KAAK,UAAU,eAAe,CACnC,OAA2B,EAC3B,UAAsB,EACtB,IAAa;IAEb,MAAM,EACJ,KAAK,EACL,MAAM,EACN,KAAK,GAAG,KAAK,GACd,GAAG,IAIH,CAAC;IAEF,OAAO,OAAO,CAAC,SAAS,CAAC,UAAU,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,CAAC,CAAC;AAC7D,CAAC"}