npm - github-issue-tower-defence-management - Versions diffs - 1.174.2 → 1.175.0 - Mend

github-issue-tower-defence-management 1.174.2 → 1.175.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.

Files changed (82) hide show

package/src/adapter/entry-points/console/ui-dist/index.html CHANGED Viewed

@@ -4,7 +4,7 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>TDPM Console</title>
-    <script type="module" crossorigin src="/assets/index-g4SqIzCh.js"></script>
+    <script type="module" crossorigin src="/assets/index-BhDothvr.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-Bh2slMtk.css">
   </head>
   <body>

package/src/adapter/entry-points/console/webServer.test.ts CHANGED Viewed

@@ -83,7 +83,7 @@ describe('webServer pure helpers', () => {
       expect(isConsoleAppRoute('/projects/acme/workflow-blocker')).toBe(true);
       expect(isConsoleAppRoute('/projects/acme/prs')).toBe(true);
       expect(isConsoleAppRoute('/projects/globex/triage')).toBe(true);
-      expect(isConsoleAppRoute('/projects/initech/unread')).toBe(true);
+      expect(isConsoleAppRoute('/projects/initech/todo-by-human')).toBe(true);
       expect(isConsoleAppRoute('/projects/umbrella/failed-preparation')).toBe(
         true,
       );
@@ -1473,7 +1473,6 @@ describe('webServer dashboard /tdpm.txt route integration', () => {
       JSON.stringify({
         pjcode: 'acme',
         capturedAt: '2026-06-26T00:00:00.000Z',
-        unread: 3,
         todo: 1,
         qc: 2,
         fail: 0,
@@ -1488,7 +1487,6 @@ describe('webServer dashboard /tdpm.txt route integration', () => {
       JSON.stringify({
         pjcode: 'initech',
         capturedAt: '2026-06-26T00:00:00.000Z',
-        unread: 0,
         todo: 0,
         qc: 0,
         fail: 0,
@@ -1532,9 +1530,9 @@ describe('webServer dashboard /tdpm.txt route integration', () => {
   const expectedComposed =
     '<tt>M55%&nbsp;C62%&nbsp;D89%&nbsp;cy14</tt><br>\n' +
     '<tt>LA&nbsp;16&nbsp;23&nbsp;40</tt><br>\n' +
-    '<tt>pj&nbsp;&nbsp;&nbsp;unr&nbsp;tdo&nbsp;aqc&nbsp;fal&nbsp;prp&nbsp;aws&nbsp;dep</tt><br>\n' +
-    '<tt>🟢ac&nbsp;&nbsp;&nbsp;3&nbsp;&nbsp;&nbsp;1&nbsp;&nbsp;&nbsp;2&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;4&nbsp;&nbsp;&nbsp;1</tt><br>\n' +
-    '<tt>🟢in&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;2&nbsp;&nbsp;&nbsp;0</tt><br>\n' +
+    '<tt>pj&nbsp;&nbsp;&nbsp;tdo&nbsp;aqc&nbsp;fal&nbsp;prp&nbsp;aws&nbsp;dep</tt><br>\n' +
+    '<tt>🟢ac&nbsp;&nbsp;&nbsp;1&nbsp;&nbsp;&nbsp;2&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;4&nbsp;&nbsp;&nbsp;1</tt><br>\n' +
+    '<tt>🟢in&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;0&nbsp;&nbsp;&nbsp;2&nbsp;&nbsp;&nbsp;0</tt><br>\n' +
     '<tt></tt><br>\n' +
     '<tt>🟢alice&nbsp;&nbsp;10%&nbsp;0d01h00&nbsp;&nbsp;12%&nbsp;5d00h00&nbsp;2&nbsp;1</tt><br>\n';

package/src/adapter/entry-points/handlers/consoleListsWriter.test.ts CHANGED Viewed

@@ -38,7 +38,6 @@ const project: Project = {
     name: 'Status',
     fieldId: 'status-field',
     statuses: [
-      option('st-unread', 'Unread', 'ORANGE'),
       option('st-aw', 'Awaiting Workspace', 'BLUE'),
       option('st-aqc', 'Awaiting Quality Check', 'GREEN'),
     ],
@@ -93,7 +92,7 @@ describe('writeConsoleLists', () => {
       pjcode: 'demo',
       assigneeLogin: ASSIGNEE,
       project,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Todo by human' })],
       generatedAt: '2026-06-14T07:22:33Z',
     });
@@ -101,7 +100,6 @@ describe('writeConsoleLists', () => {
       'workflow-blocker',
       'prs',
       'triage',
-      'unread',
       'failed-preparation',
       'todo-by-human',
       'todo-by-agent',
@@ -198,11 +196,13 @@ describe('writeConsoleLists', () => {
       pjcode: 'demo',
       assigneeLogin: ASSIGNEE,
       project,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Todo by human' })],
       generatedAt: '2026-06-14T07:22:33Z',
     });
-    const raw: unknown = JSON.parse(fs.readFileSync(tabFile('unread'), 'utf8'));
+    const raw: unknown = JSON.parse(
+      fs.readFileSync(tabFile('todo-by-human'), 'utf8'),
+    );
     expect(isRecord(raw)).toBe(true);
     const items: unknown = isRecord(raw) ? raw.items : undefined;
     expect(isUnknownArray(items)).toBe(true);
@@ -229,7 +229,7 @@ describe('writeConsoleLists', () => {
       pjcode: 'demo',
       assigneeLogin: ASSIGNEE,
       project,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Todo by human' })],
     });
     expect(fs.readdirSync(outDir)).toHaveLength(0);
   });
@@ -240,7 +240,7 @@ describe('writeConsoleLists', () => {
       pjcode: '',
       assigneeLogin: ASSIGNEE,
       project,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Todo by human' })],
     });
     expect(fs.readdirSync(outDir)).toHaveLength(0);
   });
@@ -251,24 +251,26 @@ describe('writeConsoleLists', () => {
       pjcode: 'demo',
       assigneeLogin: null,
       project,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Todo by human' })],
     });
     expect(fs.readdirSync(outDir)).toHaveLength(0);
   });
-  const regenerateUnread = (): void => {
+  const regenerateTodoByHuman = (): void => {
     writeConsoleLists({
       consoleDataOutputDir: outDir,
       pjcode: 'demo',
       assigneeLogin: ASSIGNEE,
       project,
-      issues: [makeIssue({ status: 'Unread', itemId: 'item-1' })],
+      issues: [makeIssue({ status: 'Todo by human', itemId: 'item-1' })],
       generatedAt: '2026-06-14T07:22:33Z',
     });
   };
-  const unreadServedItemCount = (): number => {
-    const route = parseConsoleDataRoute('projects/demo/unread/list.json');
+  const todoByHumanServedItemCount = (): number => {
+    const route = parseConsoleDataRoute(
+      'projects/demo/todo-by-human/list.json',
+    );
     if (route === null) {
       throw new Error('route should not be null');
     }
@@ -279,7 +281,7 @@ describe('writeConsoleLists', () => {
   };
   it('resets the done file of every tab to an empty record on regeneration', () => {
-    regenerateUnread();
+    regenerateTodoByHuman();
     for (const tab of CONSOLE_DONE_TAB_NAMES) {
       const doneFile = path.join(outDir, 'demo', tab, '.done.json');
@@ -291,32 +293,34 @@ describe('writeConsoleLists', () => {
   it('clears a pre-existing accumulated done file on regeneration', () => {
     for (const id of ['PVTI_1', 'PVTI_2', 'PVTI_3']) {
-      recordDoneProjectItemId(outDir, 'demo', 'unread', id);
+      recordDoneProjectItemId(outDir, 'demo', 'todo-by-human', id);
     }
-    expect(readDoneProjectItemIds(outDir, 'demo', 'unread')).toHaveLength(3);
+    expect(
+      readDoneProjectItemIds(outDir, 'demo', 'todo-by-human'),
+    ).toHaveLength(3);
-    regenerateUnread();
+    regenerateTodoByHuman();
-    expect(readDoneProjectItemIds(outDir, 'demo', 'unread')).toEqual([]);
+    expect(readDoneProjectItemIds(outDir, 'demo', 'todo-by-human')).toEqual([]);
   });
   it('serves the full list after regeneration despite a prior accumulated done record', () => {
-    recordDoneProjectItemId(outDir, 'demo', 'unread', 'item-1');
+    recordDoneProjectItemId(outDir, 'demo', 'todo-by-human', 'item-1');
-    regenerateUnread();
+    regenerateTodoByHuman();
-    expect(unreadServedItemCount()).toBe(1);
+    expect(todoByHumanServedItemCount()).toBe(1);
   });
   it('re-hides an item recorded as done until the next regeneration bounds it to one cycle', () => {
-    regenerateUnread();
-    expect(unreadServedItemCount()).toBe(1);
+    regenerateTodoByHuman();
+    expect(todoByHumanServedItemCount()).toBe(1);
-    recordDoneProjectItemId(outDir, 'demo', 'unread', 'item-1');
-    expect(unreadServedItemCount()).toBe(0);
+    recordDoneProjectItemId(outDir, 'demo', 'todo-by-human', 'item-1');
+    expect(todoByHumanServedItemCount()).toBe(0);
-    regenerateUnread();
-    expect(unreadServedItemCount()).toBe(1);
+    regenerateTodoByHuman();
+    expect(todoByHumanServedItemCount()).toBe(1);
   });
 });

package/src/adapter/entry-points/handlers/dashboardRowWriter.test.ts CHANGED Viewed

@@ -60,7 +60,7 @@ describe('writeDashboardRow', () => {
       pjcode: 'ac',
       assigneeLogin: ASSIGNEE,
       issues: [
-        makeIssue({ status: 'Unread' }),
+        makeIssue({ status: 'Awaiting Workspace' }),
         makeIssue({ status: 'Awaiting Quality Check' }),
       ],
       generatedAt: '2026-06-26T12:00:00.000Z',
@@ -70,12 +70,11 @@ describe('writeDashboardRow', () => {
     const expected: DashboardRowFile = {
       pjcode: 'ac',
       capturedAt: '2026-06-26T12:00:00.000Z',
-      unread: 1,
       todo: 0,
       qc: 1,
       fail: 0,
       pr: 0,
-      ws: 0,
+      ws: 1,
       dep: 0,
       blocker: 0,
     };
@@ -87,7 +86,7 @@ describe('writeDashboardRow', () => {
       dashboardDataDir: null,
       pjcode: 'ac',
       assigneeLogin: ASSIGNEE,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Awaiting Workspace' })],
     });
     expect(fs.readdirSync(dir)).toEqual([]);
@@ -98,13 +97,13 @@ describe('writeDashboardRow', () => {
       dashboardDataDir: dir,
       pjcode: null,
       assigneeLogin: ASSIGNEE,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Awaiting Workspace' })],
     });
     writeDashboardRow({
       dashboardDataDir: dir,
       pjcode: 'ac',
       assigneeLogin: null,
-      issues: [makeIssue({ status: 'Unread' })],
+      issues: [makeIssue({ status: 'Awaiting Workspace' })],
     });
     expect(fs.readdirSync(dir)).toEqual([]);

package/src/domain/entities/WorkflowStatus.ts CHANGED Viewed

@@ -1,6 +1,5 @@
 import { FieldOption } from './Project';
-export const DEFAULT_STATUS_NAME = 'Unread';
 export const AWAITING_WORKSPACE_STATUS_NAME = 'Awaiting Workspace';
 export const PREPARATION_STATUS_NAME = 'Preparation';
 export const FAILED_PREPARATION_STATUS_NAME = 'Failed Preparation';
@@ -24,10 +23,6 @@ export type WorkflowStatusDefinition = {
 };
 export const REQUIRED_WORKFLOW_STATUSES: WorkflowStatusDefinition[] = [
-  {
-    name: DEFAULT_STATUS_NAME,
-    color: 'ORANGE',
-  },
   {
     name: AWAITING_WORKSPACE_STATUS_NAME,
     color: 'BLUE',

package/src/domain/usecases/NotifyFinishedIssuePreparationUseCase.test.ts CHANGED Viewed

@@ -3728,6 +3728,38 @@ describe('NotifyFinishedIssuePreparationUseCase', () => {
       );
     });
+    it('calls setIssueAgentField when a later bot comment carries no declaration of its own', async () => {
+      const issue = createMockIssue({ status: 'Preparation' });
+      const projectWithAgent = makeProjectWithAgent();
+      mockProjectRepository.getByUrl.mockResolvedValue(projectWithAgent);
+      mockIssueRepository.get.mockResolvedValue(issue);
+      mockIssueCommentRepository.getCommentsFromIssue.mockResolvedValue([
+        createMockComment({
+          content:
+            'From: :robot: agent (model)\n```json\n{"nextStep": "do something", "nextStepAgent": "impl"}\n```',
+        }),
+        createMockComment({
+          content:
+            'From: :robot: agent (model)\n\n## PR URL\nhttps://github.com/user/repo/pull/2\n',
+        }),
+      ]);
+      mockIssueRepository.findRelatedOpenPRs.mockResolvedValue([]);
+      await useCase.run({
+        projectUrl: 'https://github.com/users/user/projects/1',
+        issueUrl: 'https://github.com/user/repo/issues/1',
+        thresholdForAutoReject: 3,
+        workflowBlockerResolvedWebhookUrl: null,
+        allowedIssueAuthors: null,
+      });
+      expect(mockIssueRepository.setIssueAgentField).toHaveBeenCalledWith(
+        'https://github.com/user/repo/issues/1',
+        projectWithAgent,
+        'opt-impl',
+      );
+    });
     it('creates a new agent option and calls setIssueAgentField when nextStepAgent is not an existing option', async () => {
       const issue = createMockIssue({ status: 'Preparation' });
       const projectWithAgent = makeProjectWithAgent();

package/src/domain/usecases/NotifyFinishedIssuePreparationUseCase.ts CHANGED Viewed

@@ -28,6 +28,7 @@ import { Issue } from '../entities/Issue';
 import { Project } from '../entities/Project';
 import { ensureAgentOptionAndGetId } from './ensureAgentOptionAndGetId';
 import { extractNextStepAgent } from './extractNextStepAgent';
+import { findLastAgentDeclaringReport } from './findLastAgentDeclaringReport';
 import { issueReactivationTriggerIsPending } from './issueReactivationTriggerIsPending';
 import { normalizeReportBody } from './normalizeReportBody';
@@ -223,14 +224,12 @@ export class NotifyFinishedIssuePreparationUseCase {
     const isTrustedAuthor = (author: string): boolean =>
       this.isAuthorTrusted(author, params.allowedIssueAuthors ?? null);
-    const lastTrustedBotComment = [...comments]
-      .reverse()
-      .find(
-        (c) =>
-          isTrustedAuthor(c.author) && c.content.startsWith('From: :robot:'),
-      );
-    const nextStepAgent = lastTrustedBotComment
-      ? extractNextStepAgent(lastTrustedBotComment.content)
+    const lastAgentDeclaringReport = findLastAgentDeclaringReport(
+      comments,
+      isTrustedAuthor,
+    );
+    const nextStepAgent = lastAgentDeclaringReport
+      ? extractNextStepAgent(lastAgentDeclaringReport.content)
       : null;
     if (nextStepAgent !== null) {
       const agentOptionId = await this.ensureAgentOptionAndGetId(