@langadventurellc/task-trellis-mcp 1.0.1 → 1.1.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +47 -67
- package/dist/__tests__/e2e/crud/createObject.e2e.test.js +64 -12
- package/dist/__tests__/e2e/crud/createObject.e2e.test.js.map +1 -1
- package/dist/__tests__/e2e/crud/listObjects.e2e.test.js +260 -90
- package/dist/__tests__/e2e/crud/listObjects.e2e.test.js.map +1 -1
- package/dist/__tests__/e2e/utils/extractObjectIds.d.ts +5 -0
- package/dist/__tests__/e2e/utils/extractObjectIds.d.ts.map +1 -0
- package/dist/__tests__/e2e/utils/extractObjectIds.js +12 -0
- package/dist/__tests__/e2e/utils/extractObjectIds.js.map +1 -0
- package/dist/__tests__/e2e/utils/index.d.ts +1 -0
- package/dist/__tests__/e2e/utils/index.d.ts.map +1 -1
- package/dist/__tests__/e2e/utils/index.js +3 -1
- package/dist/__tests__/e2e/utils/index.js.map +1 -1
- package/dist/__tests__/e2e/utils/mcpTestClient.d.ts +2 -1
- package/dist/__tests__/e2e/utils/mcpTestClient.d.ts.map +1 -1
- package/dist/__tests__/e2e/utils/mcpTestClient.js +14 -8
- package/dist/__tests__/e2e/utils/mcpTestClient.js.map +1 -1
- package/dist/__tests__/e2e/utils/parseListObjectsResponse.d.ts +3 -1
- package/dist/__tests__/e2e/utils/parseListObjectsResponse.d.ts.map +1 -1
- package/dist/__tests__/e2e/utils/parseListObjectsResponse.js +1 -0
- package/dist/__tests__/e2e/utils/parseListObjectsResponse.js.map +1 -1
- package/dist/__tests__/e2e/workflow/appendModifiedFiles.e2e.test.d.ts +2 -0
- package/dist/__tests__/e2e/workflow/appendModifiedFiles.e2e.test.d.ts.map +1 -0
- package/dist/__tests__/e2e/workflow/appendModifiedFiles.e2e.test.js +597 -0
- package/dist/__tests__/e2e/workflow/appendModifiedFiles.e2e.test.js.map +1 -0
- package/dist/__tests__/e2e/workflow/claimTask.e2e.test.js +189 -0
- package/dist/__tests__/e2e/workflow/claimTask.e2e.test.js.map +1 -1
- package/dist/__tests__/e2e/workflow/completeTask.e2e.test.js +446 -0
- package/dist/__tests__/e2e/workflow/completeTask.e2e.test.js.map +1 -1
- package/dist/__tests__/e2e/workflow/taskLifecycle.e2e.test.js +3 -7
- package/dist/__tests__/e2e/workflow/taskLifecycle.e2e.test.js.map +1 -1
- package/dist/configuration/ServerConfig.d.ts +1 -0
- package/dist/configuration/ServerConfig.d.ts.map +1 -1
- package/dist/models/TrellisObjectSummary.d.ts +16 -0
- package/dist/models/TrellisObjectSummary.d.ts.map +1 -0
- package/dist/models/TrellisObjectSummary.js +3 -0
- package/dist/models/TrellisObjectSummary.js.map +1 -0
- package/dist/models/index.d.ts +1 -0
- package/dist/models/index.d.ts.map +1 -1
- package/dist/models/index.js.map +1 -1
- package/dist/repositories/local/__tests__/getObjectFilePath.test.js +9 -6
- package/dist/repositories/local/__tests__/getObjectFilePath.test.js.map +1 -1
- package/dist/repositories/local/__tests__/getObjects.test.js +40 -10
- package/dist/repositories/local/__tests__/getObjects.test.js.map +1 -1
- package/dist/repositories/local/getObjectFilePath.d.ts.map +1 -1
- package/dist/repositories/local/getObjectFilePath.js +8 -4
- package/dist/repositories/local/getObjectFilePath.js.map +1 -1
- package/dist/repositories/local/getObjects.d.ts.map +1 -1
- package/dist/repositories/local/getObjects.js +6 -1
- package/dist/repositories/local/getObjects.js.map +1 -1
- package/dist/server.js +23 -9
- package/dist/server.js.map +1 -1
- package/dist/services/TaskTrellisService.d.ts +87 -0
- package/dist/services/TaskTrellisService.d.ts.map +1 -0
- package/dist/services/TaskTrellisService.js +3 -0
- package/dist/services/TaskTrellisService.js.map +1 -0
- package/dist/services/local/LocalTaskTrellisService.d.ts +61 -0
- package/dist/services/local/LocalTaskTrellisService.d.ts.map +1 -0
- package/dist/services/local/LocalTaskTrellisService.js +77 -0
- package/dist/services/local/LocalTaskTrellisService.js.map +1 -0
- package/dist/services/local/__tests__/appendAffectedFiles.test.d.ts +2 -0
- package/dist/services/local/__tests__/appendAffectedFiles.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/appendAffectedFiles.test.js +202 -0
- package/dist/services/local/__tests__/appendAffectedFiles.test.js.map +1 -0
- package/dist/services/local/__tests__/appendModifiedFiles.test.d.ts +2 -0
- package/dist/services/local/__tests__/appendModifiedFiles.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/appendModifiedFiles.test.js +161 -0
- package/dist/services/local/__tests__/appendModifiedFiles.test.js.map +1 -0
- package/dist/services/local/__tests__/appendObjectLog.test.d.ts +2 -0
- package/dist/services/local/__tests__/appendObjectLog.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/appendObjectLog.test.js +104 -0
- package/dist/services/local/__tests__/appendObjectLog.test.js.map +1 -0
- package/dist/services/local/__tests__/claimTask.test.d.ts +2 -0
- package/dist/services/local/__tests__/claimTask.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/claimTask.test.js +463 -0
- package/dist/services/local/__tests__/claimTask.test.js.map +1 -0
- package/dist/services/local/__tests__/completeTask.test.d.ts +2 -0
- package/dist/services/local/__tests__/completeTask.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/completeTask.test.js +468 -0
- package/dist/services/local/__tests__/completeTask.test.js.map +1 -0
- package/dist/services/local/__tests__/createObject.test.d.ts +2 -0
- package/dist/services/local/__tests__/createObject.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/createObject.test.js +332 -0
- package/dist/services/local/__tests__/createObject.test.js.map +1 -0
- package/dist/services/local/__tests__/listObjects.test.d.ts +2 -0
- package/dist/services/local/__tests__/listObjects.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/listObjects.test.js +226 -0
- package/dist/services/local/__tests__/listObjects.test.js.map +1 -0
- package/dist/services/local/__tests__/pruneClosed.test.d.ts +2 -0
- package/dist/services/local/__tests__/pruneClosed.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/pruneClosed.test.js +213 -0
- package/dist/services/local/__tests__/pruneClosed.test.js.map +1 -0
- package/dist/services/local/__tests__/replaceObjectBodyRegex.test.d.ts +2 -0
- package/dist/services/local/__tests__/replaceObjectBodyRegex.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/replaceObjectBodyRegex.test.js +283 -0
- package/dist/services/local/__tests__/replaceObjectBodyRegex.test.js.map +1 -0
- package/dist/services/local/__tests__/updateObject.test.d.ts +2 -0
- package/dist/services/local/__tests__/updateObject.test.d.ts.map +1 -0
- package/dist/services/local/__tests__/updateObject.test.js +314 -0
- package/dist/services/local/__tests__/updateObject.test.js.map +1 -0
- package/dist/services/local/appendAffectedFiles.d.ts +12 -0
- package/dist/services/local/appendAffectedFiles.d.ts.map +1 -0
- package/dist/services/local/appendAffectedFiles.js +34 -0
- package/dist/services/local/appendAffectedFiles.js.map +1 -0
- package/dist/services/local/appendModifiedFiles.d.ts +15 -0
- package/dist/services/local/appendModifiedFiles.d.ts.map +1 -0
- package/dist/services/local/appendModifiedFiles.js +39 -0
- package/dist/services/local/appendModifiedFiles.js.map +1 -0
- package/dist/services/local/appendObjectLog.d.ts +8 -0
- package/dist/services/local/appendObjectLog.d.ts.map +1 -0
- package/dist/services/local/appendObjectLog.js +45 -0
- package/dist/services/local/appendObjectLog.js.map +1 -0
- package/dist/services/local/claimTask.d.ts +8 -0
- package/dist/services/local/claimTask.d.ts.map +1 -0
- package/dist/services/local/claimTask.js +118 -0
- package/dist/services/local/claimTask.js.map +1 -0
- package/dist/services/local/completeTask.d.ts +9 -0
- package/dist/services/local/completeTask.d.ts.map +1 -0
- package/dist/services/local/completeTask.js +74 -0
- package/dist/services/local/completeTask.js.map +1 -0
- package/dist/services/local/createObject.d.ts +9 -0
- package/dist/services/local/createObject.d.ts.map +1 -0
- package/dist/services/local/createObject.js +44 -0
- package/dist/services/local/createObject.js.map +1 -0
- package/dist/services/local/listObjects.d.ts +9 -0
- package/dist/services/local/listObjects.d.ts.map +1 -0
- package/dist/services/local/listObjects.js +43 -0
- package/dist/services/local/listObjects.js.map +1 -0
- package/dist/services/local/pruneClosed.d.ts +8 -0
- package/dist/services/local/pruneClosed.d.ts.map +1 -0
- package/dist/services/local/pruneClosed.js +55 -0
- package/dist/services/local/pruneClosed.js.map +1 -0
- package/dist/services/local/replaceObjectBodyRegex.d.ts +8 -0
- package/dist/services/local/replaceObjectBodyRegex.d.ts.map +1 -0
- package/dist/services/local/replaceObjectBodyRegex.js +85 -0
- package/dist/services/local/replaceObjectBodyRegex.js.map +1 -0
- package/dist/services/local/updateObject.d.ts +9 -0
- package/dist/services/local/updateObject.d.ts.map +1 -0
- package/dist/services/local/updateObject.js +53 -0
- package/dist/services/local/updateObject.js.map +1 -0
- package/dist/tools/__tests__/appendModifiedFilesTool.test.d.ts +2 -0
- package/dist/tools/__tests__/appendModifiedFilesTool.test.d.ts.map +1 -0
- package/dist/tools/__tests__/appendModifiedFilesTool.test.js +177 -0
- package/dist/tools/__tests__/appendModifiedFilesTool.test.js.map +1 -0
- package/dist/tools/__tests__/appendObjectLogTool.test.js +33 -101
- package/dist/tools/__tests__/appendObjectLogTool.test.js.map +1 -1
- package/dist/tools/__tests__/claimTaskTool.test.js +98 -311
- package/dist/tools/__tests__/claimTaskTool.test.js.map +1 -1
- package/dist/tools/__tests__/completeTaskTool.test.js +134 -184
- package/dist/tools/__tests__/completeTaskTool.test.js.map +1 -1
- package/dist/tools/__tests__/createObjectTool.test.js +61 -325
- package/dist/tools/__tests__/createObjectTool.test.js.map +1 -1
- package/dist/tools/__tests__/listObjectsTool.test.js +70 -109
- package/dist/tools/__tests__/listObjectsTool.test.js.map +1 -1
- package/dist/tools/__tests__/pruneClosedTool.test.js +87 -146
- package/dist/tools/__tests__/pruneClosedTool.test.js.map +1 -1
- package/dist/tools/__tests__/replaceObjectBodyRegexTool.test.js +56 -225
- package/dist/tools/__tests__/replaceObjectBodyRegexTool.test.js.map +1 -1
- package/dist/tools/__tests__/updateObjectTool.test.js +48 -166
- package/dist/tools/__tests__/updateObjectTool.test.js.map +1 -1
- package/dist/tools/appendModifiedFilesTool.d.ts +30 -0
- package/dist/tools/appendModifiedFilesTool.d.ts.map +1 -0
- package/dist/tools/appendModifiedFilesTool.js +55 -0
- package/dist/tools/appendModifiedFilesTool.js.map +1 -0
- package/dist/tools/appendObjectLogTool.d.ts +4 -3
- package/dist/tools/appendObjectLogTool.d.ts.map +1 -1
- package/dist/tools/appendObjectLogTool.js +2 -40
- package/dist/tools/appendObjectLogTool.js.map +1 -1
- package/dist/tools/claimTaskTool.d.ts +4 -3
- package/dist/tools/claimTaskTool.d.ts.map +1 -1
- package/dist/tools/claimTaskTool.js +2 -80
- package/dist/tools/claimTaskTool.js.map +1 -1
- package/dist/tools/completeTaskTool.d.ts +5 -3
- package/dist/tools/completeTaskTool.d.ts.map +1 -1
- package/dist/tools/completeTaskTool.js +3 -29
- package/dist/tools/completeTaskTool.js.map +1 -1
- package/dist/tools/createObjectTool.d.ts +7 -6
- package/dist/tools/createObjectTool.d.ts.map +1 -1
- package/dist/tools/createObjectTool.js +11 -44
- package/dist/tools/createObjectTool.js.map +1 -1
- package/dist/tools/index.d.ts +1 -0
- package/dist/tools/index.d.ts.map +1 -1
- package/dist/tools/index.js +4 -1
- package/dist/tools/index.js.map +1 -1
- package/dist/tools/listObjectsTool.d.ts +5 -4
- package/dist/tools/listObjectsTool.d.ts.map +1 -1
- package/dist/tools/listObjectsTool.js +5 -13
- package/dist/tools/listObjectsTool.js.map +1 -1
- package/dist/tools/pruneClosedTool.d.ts +4 -3
- package/dist/tools/pruneClosedTool.d.ts.map +1 -1
- package/dist/tools/pruneClosedTool.js +2 -51
- package/dist/tools/pruneClosedTool.js.map +1 -1
- package/dist/tools/replaceObjectBodyRegexTool.d.ts +4 -3
- package/dist/tools/replaceObjectBodyRegexTool.d.ts.map +1 -1
- package/dist/tools/replaceObjectBodyRegexTool.js +2 -80
- package/dist/tools/replaceObjectBodyRegexTool.js.map +1 -1
- package/dist/tools/updateObjectTool.d.ts +5 -4
- package/dist/tools/updateObjectTool.d.ts.map +1 -1
- package/dist/tools/updateObjectTool.js +3 -49
- package/dist/tools/updateObjectTool.js.map +1 -1
- package/dist/validation/__tests__/validateParentType.test.js +5 -5
- package/dist/validation/__tests__/validateParentType.test.js.map +1 -1
- package/dist/validation/validateParentType.d.ts +1 -1
- package/dist/validation/validateParentType.d.ts.map +1 -1
- package/dist/validation/validateParentType.js +2 -5
- package/dist/validation/validateParentType.js.map +1 -1
- package/package.json +1 -1
|
@@ -1,27 +1,21 @@
|
|
|
1
1
|
"use strict";
|
|
2
2
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
const models_1 = require("../../models");
|
|
4
3
|
const completeTaskTool_1 = require("../completeTaskTool");
|
|
5
4
|
describe("completeTaskTool", () => {
|
|
5
|
+
let mockService;
|
|
6
6
|
let mockRepository;
|
|
7
|
-
const createMockTask = (overrides) => ({
|
|
8
|
-
id: "T-test-task",
|
|
9
|
-
type: models_1.TrellisObjectType.TASK,
|
|
10
|
-
title: "Test Task",
|
|
11
|
-
status: models_1.TrellisObjectStatus.IN_PROGRESS,
|
|
12
|
-
priority: models_1.TrellisObjectPriority.MEDIUM,
|
|
13
|
-
parent: "F-test-feature",
|
|
14
|
-
prerequisites: [],
|
|
15
|
-
affectedFiles: new Map(),
|
|
16
|
-
log: [],
|
|
17
|
-
schema: "1.0",
|
|
18
|
-
created: "2025-01-15T10:00:00Z",
|
|
19
|
-
updated: "2025-01-15T10:00:00Z",
|
|
20
|
-
childrenIds: [],
|
|
21
|
-
body: "This is a test task",
|
|
22
|
-
...overrides,
|
|
23
|
-
});
|
|
24
7
|
beforeEach(() => {
|
|
8
|
+
mockService = {
|
|
9
|
+
completeTask: jest.fn(),
|
|
10
|
+
createObject: jest.fn(),
|
|
11
|
+
updateObject: jest.fn(),
|
|
12
|
+
claimTask: jest.fn(),
|
|
13
|
+
listObjects: jest.fn(),
|
|
14
|
+
appendObjectLog: jest.fn(),
|
|
15
|
+
pruneClosed: jest.fn(),
|
|
16
|
+
replaceObjectBodyRegex: jest.fn(),
|
|
17
|
+
appendModifiedFiles: jest.fn(),
|
|
18
|
+
};
|
|
25
19
|
mockRepository = {
|
|
26
20
|
getObjectById: jest.fn(),
|
|
27
21
|
getObjects: jest.fn(),
|
|
@@ -30,193 +24,149 @@ describe("completeTaskTool", () => {
|
|
|
30
24
|
};
|
|
31
25
|
});
|
|
32
26
|
describe("handleCompleteTask", () => {
|
|
33
|
-
it("should
|
|
34
|
-
const
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
27
|
+
it("should call service.completeTask with correct parameters", async () => {
|
|
28
|
+
const expectedResult = {
|
|
29
|
+
content: [
|
|
30
|
+
{
|
|
31
|
+
type: "text",
|
|
32
|
+
text: 'Task "T-test-task" completed successfully. Updated 2 affected files.',
|
|
33
|
+
},
|
|
34
|
+
],
|
|
38
35
|
};
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockRepository, {
|
|
36
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
37
|
+
const args = {
|
|
42
38
|
taskId: "T-test-task",
|
|
43
39
|
summary: "Task completed successfully",
|
|
44
|
-
filesChanged
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
...mockTask,
|
|
49
|
-
status: models_1.TrellisObjectStatus.DONE,
|
|
50
|
-
affectedFiles: new Map([
|
|
51
|
-
["src/file1.ts", "Added new feature"],
|
|
52
|
-
["src/file2.ts", "Fixed bug"],
|
|
53
|
-
]),
|
|
54
|
-
log: ["Task completed successfully"],
|
|
55
|
-
});
|
|
56
|
-
expect(result.content[0].text).toContain('Task "T-test-task" completed successfully. Updated 2 affected files.');
|
|
57
|
-
});
|
|
58
|
-
it("should append to existing affected files", async () => {
|
|
59
|
-
const existingFiles = new Map([["existing.ts", "Previously changed"]]);
|
|
60
|
-
const mockTask = createMockTask({ affectedFiles: existingFiles });
|
|
61
|
-
const filesChanged = {
|
|
62
|
-
"src/new-file.ts": "Newly added file",
|
|
40
|
+
filesChanged: {
|
|
41
|
+
"src/file1.ts": "Added new feature",
|
|
42
|
+
"src/file2.ts": "Fixed bug",
|
|
43
|
+
},
|
|
63
44
|
};
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
});
|
|
71
|
-
expect(mockRepository.saveObject).toHaveBeenCalledWith({
|
|
72
|
-
...mockTask,
|
|
73
|
-
status: models_1.TrellisObjectStatus.DONE,
|
|
74
|
-
affectedFiles: new Map([
|
|
75
|
-
["existing.ts", "Previously changed"],
|
|
76
|
-
["src/new-file.ts", "Newly added file"],
|
|
77
|
-
]),
|
|
78
|
-
log: ["Added new functionality"],
|
|
79
|
-
});
|
|
80
|
-
});
|
|
81
|
-
it("should append to existing log entries", async () => {
|
|
82
|
-
const mockTask = createMockTask({ log: ["Previous log entry"] });
|
|
83
|
-
const filesChanged = { "file.ts": "Description" };
|
|
84
|
-
mockRepository.getObjectById.mockResolvedValue(mockTask);
|
|
85
|
-
mockRepository.saveObject.mockResolvedValue();
|
|
86
|
-
await (0, completeTaskTool_1.handleCompleteTask)(mockRepository, {
|
|
87
|
-
taskId: "T-test-task",
|
|
88
|
-
summary: "New log entry",
|
|
89
|
-
filesChanged,
|
|
90
|
-
});
|
|
91
|
-
expect(mockRepository.saveObject).toHaveBeenCalledWith({
|
|
92
|
-
...mockTask,
|
|
93
|
-
status: models_1.TrellisObjectStatus.DONE,
|
|
94
|
-
log: ["Previous log entry", "New log entry"],
|
|
95
|
-
affectedFiles: new Map([["file.ts", "Description"]]),
|
|
96
|
-
});
|
|
97
|
-
});
|
|
98
|
-
it("should handle empty files changed object", async () => {
|
|
99
|
-
const mockTask = createMockTask();
|
|
100
|
-
const filesChanged = {};
|
|
101
|
-
mockRepository.getObjectById.mockResolvedValue(mockTask);
|
|
102
|
-
mockRepository.saveObject.mockResolvedValue();
|
|
103
|
-
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockRepository, {
|
|
104
|
-
taskId: "T-test-task",
|
|
105
|
-
summary: "Task completed with no file changes",
|
|
106
|
-
filesChanged,
|
|
107
|
-
});
|
|
108
|
-
expect(mockRepository.saveObject).toHaveBeenCalledWith({
|
|
109
|
-
...mockTask,
|
|
110
|
-
status: models_1.TrellisObjectStatus.DONE,
|
|
111
|
-
affectedFiles: new Map(),
|
|
112
|
-
log: ["Task completed with no file changes"],
|
|
113
|
-
});
|
|
114
|
-
expect(result.content[0].text).toContain("Updated 0 affected files");
|
|
115
|
-
});
|
|
116
|
-
it("should throw error when task is not found", async () => {
|
|
117
|
-
mockRepository.getObjectById.mockResolvedValue(null);
|
|
118
|
-
await expect((0, completeTaskTool_1.handleCompleteTask)(mockRepository, {
|
|
119
|
-
taskId: "T-nonexistent",
|
|
120
|
-
summary: "Summary",
|
|
121
|
-
filesChanged: {},
|
|
122
|
-
})).rejects.toThrow('Task with ID "T-nonexistent" not found');
|
|
123
|
-
expect(mockRepository.getObjectById).toHaveBeenCalledWith("T-nonexistent");
|
|
124
|
-
expect(mockRepository.saveObject).not.toHaveBeenCalled();
|
|
125
|
-
});
|
|
126
|
-
it("should throw error when task is not in progress", async () => {
|
|
127
|
-
const mockTask = createMockTask({ status: models_1.TrellisObjectStatus.OPEN });
|
|
128
|
-
mockRepository.getObjectById.mockResolvedValue(mockTask);
|
|
129
|
-
await expect((0, completeTaskTool_1.handleCompleteTask)(mockRepository, {
|
|
130
|
-
taskId: "T-test-task",
|
|
131
|
-
summary: "Summary",
|
|
132
|
-
filesChanged: {},
|
|
133
|
-
})).rejects.toThrow('Task "T-test-task" is not in progress (current status: open)');
|
|
134
|
-
expect(mockRepository.saveObject).not.toHaveBeenCalled();
|
|
45
|
+
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args);
|
|
46
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-test-task", "Task completed successfully", {
|
|
47
|
+
"src/file1.ts": "Added new feature",
|
|
48
|
+
"src/file2.ts": "Fixed bug",
|
|
49
|
+
}, undefined);
|
|
50
|
+
expect(result).toBe(expectedResult);
|
|
135
51
|
});
|
|
136
|
-
it("should
|
|
137
|
-
const
|
|
138
|
-
|
|
139
|
-
|
|
52
|
+
it("should pass serverConfig to service.completeTask when provided", async () => {
|
|
53
|
+
const expectedResult = {
|
|
54
|
+
content: [
|
|
55
|
+
{
|
|
56
|
+
type: "text",
|
|
57
|
+
text: 'Task "T-test-task" completed successfully.',
|
|
58
|
+
},
|
|
59
|
+
],
|
|
60
|
+
};
|
|
61
|
+
const serverConfig = {
|
|
62
|
+
mode: "local",
|
|
63
|
+
planningRootFolder: "/test",
|
|
64
|
+
autoCompleteParent: true,
|
|
65
|
+
};
|
|
66
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
67
|
+
const args = {
|
|
140
68
|
taskId: "T-test-task",
|
|
141
|
-
summary: "
|
|
69
|
+
summary: "Task completed",
|
|
142
70
|
filesChanged: {},
|
|
143
|
-
}
|
|
144
|
-
|
|
71
|
+
};
|
|
72
|
+
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args, serverConfig);
|
|
73
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-test-task", "Task completed", {}, serverConfig);
|
|
74
|
+
expect(result).toBe(expectedResult);
|
|
145
75
|
});
|
|
146
|
-
it("should
|
|
147
|
-
const
|
|
148
|
-
|
|
149
|
-
|
|
76
|
+
it("should handle empty filesChanged object", async () => {
|
|
77
|
+
const expectedResult = {
|
|
78
|
+
content: [
|
|
79
|
+
{
|
|
80
|
+
type: "text",
|
|
81
|
+
text: 'Task "T-test-task" completed successfully. Updated 0 affected files.',
|
|
82
|
+
},
|
|
83
|
+
],
|
|
84
|
+
};
|
|
85
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
86
|
+
const args = {
|
|
150
87
|
taskId: "T-test-task",
|
|
151
|
-
summary: "
|
|
88
|
+
summary: "Task completed with no file changes",
|
|
152
89
|
filesChanged: {},
|
|
153
|
-
}
|
|
154
|
-
|
|
90
|
+
};
|
|
91
|
+
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args);
|
|
92
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-test-task", "Task completed with no file changes", {}, undefined);
|
|
93
|
+
expect(result).toBe(expectedResult);
|
|
155
94
|
});
|
|
156
|
-
it("should
|
|
157
|
-
const errorMessage = "
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
taskId: "T-
|
|
95
|
+
it("should propagate service errors", async () => {
|
|
96
|
+
const errorMessage = "Task not found";
|
|
97
|
+
mockService.completeTask.mockRejectedValue(new Error(errorMessage));
|
|
98
|
+
const args = {
|
|
99
|
+
taskId: "T-nonexistent",
|
|
161
100
|
summary: "Summary",
|
|
162
101
|
filesChanged: {},
|
|
163
|
-
}
|
|
164
|
-
expect(
|
|
102
|
+
};
|
|
103
|
+
await expect((0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args)).rejects.toThrow(errorMessage);
|
|
104
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-nonexistent", "Summary", {}, undefined);
|
|
165
105
|
});
|
|
166
|
-
it("should handle
|
|
167
|
-
const
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
106
|
+
it("should handle complex filesChanged object", async () => {
|
|
107
|
+
const expectedResult = {
|
|
108
|
+
content: [
|
|
109
|
+
{
|
|
110
|
+
type: "text",
|
|
111
|
+
text: 'Task "T-test-task" completed successfully. Updated 3 affected files.',
|
|
112
|
+
},
|
|
113
|
+
],
|
|
114
|
+
};
|
|
115
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
116
|
+
const args = {
|
|
172
117
|
taskId: "T-test-task",
|
|
173
|
-
summary: "
|
|
174
|
-
filesChanged: {
|
|
175
|
-
|
|
176
|
-
|
|
177
|
-
|
|
118
|
+
summary: "Multiple files changed",
|
|
119
|
+
filesChanged: {
|
|
120
|
+
"file1.ts": "First file",
|
|
121
|
+
"file2.js": "Second file",
|
|
122
|
+
"config.json": "Configuration file",
|
|
123
|
+
},
|
|
124
|
+
};
|
|
125
|
+
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args);
|
|
126
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-test-task", "Multiple files changed", {
|
|
127
|
+
"file1.ts": "First file",
|
|
128
|
+
"file2.js": "Second file",
|
|
129
|
+
"config.json": "Configuration file",
|
|
130
|
+
}, undefined);
|
|
131
|
+
expect(result).toBe(expectedResult);
|
|
178
132
|
});
|
|
179
|
-
|
|
180
|
-
|
|
181
|
-
|
|
182
|
-
|
|
183
|
-
|
|
184
|
-
|
|
185
|
-
|
|
186
|
-
|
|
133
|
+
it("should ignore extra properties in args object", async () => {
|
|
134
|
+
const expectedResult = {
|
|
135
|
+
content: [
|
|
136
|
+
{
|
|
137
|
+
type: "text",
|
|
138
|
+
text: 'Task "T-test-task" completed successfully.',
|
|
139
|
+
},
|
|
140
|
+
],
|
|
141
|
+
};
|
|
142
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
143
|
+
const args = {
|
|
187
144
|
taskId: "T-test-task",
|
|
188
145
|
summary: "Test summary",
|
|
189
|
-
filesChanged,
|
|
146
|
+
filesChanged: { "test.ts": "Test file" },
|
|
190
147
|
extraProperty: "should be ignored",
|
|
191
|
-
|
|
192
|
-
|
|
193
|
-
|
|
148
|
+
anotherExtra: 123,
|
|
149
|
+
};
|
|
150
|
+
const result = await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args);
|
|
151
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-test-task", "Test summary", { "test.ts": "Test file" }, undefined);
|
|
152
|
+
expect(result).toBe(expectedResult);
|
|
194
153
|
});
|
|
195
|
-
|
|
196
|
-
|
|
197
|
-
|
|
198
|
-
|
|
199
|
-
"
|
|
200
|
-
"config.json": "Configuration file",
|
|
154
|
+
});
|
|
155
|
+
describe("parameter extraction", () => {
|
|
156
|
+
it("should correctly extract taskId, summary, and filesChanged from args", async () => {
|
|
157
|
+
const expectedResult = {
|
|
158
|
+
content: [{ type: "text", text: "Success" }],
|
|
201
159
|
};
|
|
202
|
-
|
|
203
|
-
|
|
204
|
-
|
|
205
|
-
|
|
206
|
-
|
|
207
|
-
|
|
208
|
-
|
|
209
|
-
|
|
210
|
-
|
|
211
|
-
|
|
212
|
-
["config.json", "Configuration file"],
|
|
213
|
-
]);
|
|
214
|
-
expect(mockRepository.saveObject).toHaveBeenCalledWith({
|
|
215
|
-
...mockTask,
|
|
216
|
-
status: models_1.TrellisObjectStatus.DONE,
|
|
217
|
-
affectedFiles: expectedMap,
|
|
218
|
-
log: ["Multiple files changed"],
|
|
219
|
-
});
|
|
160
|
+
mockService.completeTask.mockResolvedValue(expectedResult);
|
|
161
|
+
const args = {
|
|
162
|
+
taskId: "T-extract-test",
|
|
163
|
+
summary: "Extraction test summary",
|
|
164
|
+
filesChanged: {
|
|
165
|
+
"extract.ts": "Extraction test",
|
|
166
|
+
},
|
|
167
|
+
};
|
|
168
|
+
await (0, completeTaskTool_1.handleCompleteTask)(mockService, mockRepository, args);
|
|
169
|
+
expect(mockService.completeTask).toHaveBeenCalledWith(mockRepository, "T-extract-test", "Extraction test summary", { "extract.ts": "Extraction test" }, undefined);
|
|
220
170
|
});
|
|
221
171
|
});
|
|
222
172
|
});
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"completeTaskTool.test.js","sourceRoot":"","sources":["../../../src/tools/__tests__/completeTaskTool.test.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"completeTaskTool.test.js","sourceRoot":"","sources":["../../../src/tools/__tests__/completeTaskTool.test.ts"],"names":[],"mappings":";;AAEA,0DAAyD;AAGzD,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;YACvB,YAAY,EAAE,IAAI,CAAC,EAAE,EAAE;YACvB,YAAY,EAAE,IAAI,CAAC,EAAE,EAAE;YACvB,SAAS,EAAE,IAAI,CAAC,EAAE,EAAE;YACpB,WAAW,EAAE,IAAI,CAAC,EAAE,EAAE;YACtB,eAAe,EAAE,IAAI,CAAC,EAAE,EAAE;YAC1B,WAAW,EAAE,IAAI,CAAC,EAAE,EAAE;YACtB,sBAAsB,EAAE,IAAI,CAAC,EAAE,EAAE;YACjC,mBAAmB,EAAE,IAAI,CAAC,EAAE,EAAE;SAC/B,CAAC;QAEF,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;IACJ,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;YACxE,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,sEAAsE;qBAC7E;iBACF;aACF,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,aAAa;gBACrB,OAAO,EAAE,6BAA6B;gBACtC,YAAY,EAAE;oBACZ,cAAc,EAAE,mBAAmB;oBACnC,cAAc,EAAE,WAAW;iBAC5B;aACF,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EACrC,WAAW,EACX,cAAc,EACd,IAAI,CACL,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,6BAA6B,EAC7B;gBACE,cAAc,EAAE,mBAAmB;gBACnC,cAAc,EAAE,WAAW;aAC5B,EACD,SAAS,CACV,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;YAC9E,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,4CAA4C;qBACnD;iBACF;aACF,CAAC;YAEF,MAAM,YAAY,GAAiB;gBACjC,IAAI,EAAE,OAAO;gBACb,kBAAkB,EAAE,OAAO;gBAC3B,kBAAkB,EAAE,IAAI;aACzB,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,aAAa;gBACrB,OAAO,EAAE,gBAAgB;gBACzB,YAAY,EAAE,EAAE;aACjB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EACrC,WAAW,EACX,cAAc,EACd,IAAI,EACJ,YAAY,CACb,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,gBAAgB,EAChB,EAAE,EACF,YAAY,CACb,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,KAAK,IAAI,EAAE;YACvD,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,sEAAsE;qBAC7E;iBACF;aACF,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,aAAa;gBACrB,OAAO,EAAE,qCAAqC;gBAC9C,YAAY,EAAE,EAAE;aACjB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EACrC,WAAW,EACX,cAAc,EACd,IAAI,CACL,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,qCAAqC,EACrC,EAAE,EACF,SAAS,CACV,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,MAAM,YAAY,GAAG,gBAAgB,CAAC;YACtC,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC;YAEpE,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,eAAe;gBACvB,OAAO,EAAE,SAAS;gBAClB,YAAY,EAAE,EAAE;aACjB,CAAC;YAEF,MAAM,MAAM,CACV,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE,IAAI,CAAC,CACtD,CAAC,OAAO,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;YAEhC,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,eAAe,EACf,SAAS,EACT,EAAE,EACF,SAAS,CACV,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,sEAAsE;qBAC7E;iBACF;aACF,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,aAAa;gBACrB,OAAO,EAAE,wBAAwB;gBACjC,YAAY,EAAE;oBACZ,UAAU,EAAE,YAAY;oBACxB,UAAU,EAAE,aAAa;oBACzB,aAAa,EAAE,oBAAoB;iBACpC;aACF,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EACrC,WAAW,EACX,cAAc,EACd,IAAI,CACL,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,wBAAwB,EACxB;gBACE,UAAU,EAAE,YAAY;gBACxB,UAAU,EAAE,aAAa;gBACzB,aAAa,EAAE,oBAAoB;aACpC,EACD,SAAS,CACV,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE;oBACP;wBACE,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,4CAA4C;qBACnD;iBACF;aACF,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,aAAa;gBACrB,OAAO,EAAE,cAAc;gBACvB,YAAY,EAAE,EAAE,SAAS,EAAE,WAAW,EAAE;gBACxC,aAAa,EAAE,mBAAmB;gBAClC,YAAY,EAAE,GAAG;aAClB,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,IAAA,qCAAkB,EACrC,WAAW,EACX,cAAc,EACd,IAAI,CACL,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,aAAa,EACb,cAAc,EACd,EAAE,SAAS,EAAE,WAAW,EAAE,EAC1B,SAAS,CACV,CAAC;YACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACtC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;QACpC,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;YACpF,MAAM,cAAc,GAAG;gBACrB,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;aAC7C,CAAC;YAEF,WAAW,CAAC,YAAY,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;YAE3D,MAAM,IAAI,GAAG;gBACX,MAAM,EAAE,gBAAgB;gBACxB,OAAO,EAAE,yBAAyB;gBAClC,YAAY,EAAE;oBACZ,YAAY,EAAE,iBAAiB;iBAChC;aACF,CAAC;YAEF,MAAM,IAAA,qCAAkB,EAAC,WAAW,EAAE,cAAc,EAAE,IAAI,CAAC,CAAC;YAE5D,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,oBAAoB,CACnD,cAAc,EACd,gBAAgB,EAChB,yBAAyB,EACzB,EAAE,YAAY,EAAE,iBAAiB,EAAE,EACnC,SAAS,CACV,CAAC;QACJ,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|