discoclaw 1.2.4 → 2.0.0

package/dist/discord/actions.test.js

@@ -1,7 +1,7 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { ChannelType, PermissionFlagsBits } from 'discord.js';
 import { buildUnavailableActionTypesNotice } from './output-common.js';
-import { parseDiscordActions, executeDiscordActions, discordActionsPromptSection, buildTieredDiscordActionsPromptSection, buildDisplayResultLines, buildAllResultLines, withoutRequesterGatedActionFlags, } from './actions.js';
+import { parseDiscordActions, executeDiscordActions, discordActionsPromptSection, buildTieredDiscordActionsPromptSection, buildDisplayResultLines, buildAllResultLines, buildCappedResultLines, withoutRequesterGatedActionFlags, } from './actions.js';
 import { TaskStore } from '../tasks/store.js';
 import { _resetDestructiveConfirmationForTest } from './destructive-confirmation.js';
 import { shouldTriggerFollowUp } from './action-categories.js';
@@ -892,6 +892,122 @@ describe('buildAllResultLines', () => {
         ]);
     });
 });
+describe('buildCappedResultLines', () => {
+    it('microcompacts multiline readMessages-style payloads to representative id lines', () => {
+        const results = [{
+                ok: true,
+                summary: [
+                    'Messages in #ops:',
+                    '[alice] alpha update (id:1001)',
+                    '[bob] beta update (id:1002)',
+                    '[carol] gamma update (id:1003)',
+                    '[dave] delta update (id:1004)',
+                    '[erin] epsilon update (id:1005)',
+                    '[frank] zeta update (id:1006)',
+                    '[grace] eta update (id:1007)',
+                    '[heidi] theta update (id:1008)',
+                    '[ivan] iota update (id:1009)',
+                ].join('\n'),
+            }];
+        const [line] = buildCappedResultLines(results);
+        expect(line).toContain('Done: Messages in #ops:');
+        expect(line).toContain('(id:1001)');
+        expect(line).toContain('(id:1002)');
+        expect(line).toContain('(id:1008)');
+        expect(line).toContain('(id:1009)');
+        expect(line).toContain('...[omitted 5 lines]');
+        expect(line).not.toContain('(id:1005)');
+    });
+    it('keeps task thread jump URLs in multiline closeout blocks for follow-up reuse', () => {
+        const results = [{
+                ok: true,
+                summary: [
+                    'Task ws-204 closed',
+                    'Status: done',
+                    'Thread: https://discord.com/channels/111111111111111111/222222222222222222/333333333333333333',
+                    'Project: discoclaw',
+                    'Assignee: ClawBot',
+                    'Updated by: @weston',
+                    'Note: Closeout posted in the linked Discord thread.',
+                    'Artifacts: /tmp/task-closeout.md',
+                    'Next action: use taskShow ws-204 if you need the stored linkage later.',
+                    'Watcher: ops',
+                ].join('\n'),
+            }];
+        const [line] = buildCappedResultLines(results);
+        expect(line).toContain('Done: Task ws-204 closed');
+        expect(line).toContain('Status: done');
+        expect(line).toContain('Thread: https://discord.com/channels/111111111111111111/222222222222222222/333333333333333333');
+        expect(line).toContain('Artifacts: /tmp/task-closeout.md');
+        expect(line).toContain('Next action: use taskShow ws-204 if you need the stored linkage later.');
+        expect(line).toContain('...[omitted 4 lines]');
+        expect(line).not.toContain('Assignee: ClawBot');
+    });
+    it('keeps section headings and first values in memoryShow-style blocks', () => {
+        const results = [{
+                ok: true,
+                summary: [
+                    '**Durable memory:**',
+                    '- prefers black coffee',
+                    '- uses fish shell',
+                    '**By entity:**',
+                    '- [project] discoclaw migration',
+                    '**Rolling summary:**',
+                    'Working on prompt compaction.',
+                    '**Short-term memory:**',
+                    'Need task-42 and /tmp/report.md for the next reply.',
+                    'Extra noisy note A that should be dropped.',
+                    'Extra noisy note B that should be dropped.',
+                    'Extra noisy note that should be dropped.',
+                ].join('\n'),
+            }];
+        const [line] = buildCappedResultLines(results);
+        expect(line).toContain('Done: **Durable memory:**');
+        expect(line).toContain('- prefers black coffee');
+        expect(line).toContain('**By entity:**');
+        expect(line).toContain('- [project] discoclaw migration');
+        expect(line).toContain('**Rolling summary:**');
+        expect(line).toContain('Working on prompt compaction.');
+        expect(line).toContain('**Short-term memory:**');
+        expect(line).toContain('Need task-42 and /tmp/report.md for the next reply.');
+        expect(line).toContain('...[omitted ');
+        expect(line).not.toContain('Extra noisy note B that should be dropped.');
+    });
+    it('keeps failed prefixes, paths, and actionable error lines in multiline failures', () => {
+        const results = [{
+                ok: false,
+                error: [
+                    'Forge sync failed while opening workspace state',
+                    'Workspace: /home/davidmarsh/code/discoclaw/workspace/state.json',
+                    'Last error: ENOENT: no such file or directory',
+                    'Stack: at openWorkspaceState (src/discord/actions.ts:400:12)',
+                    'Stack: at runForgeSync (src/discord/actions.ts:512:8)',
+                    'Stack: at async executeForgeAction (src/discord/actions.ts:618:3)',
+                    'Retry hint: recreate the state file, then rerun forge sync',
+                    'Node: v24.0.0',
+                    'cwd: /home/davidmarsh/code/discoclaw',
+                ].join('\n'),
+            }];
+        const [line] = buildCappedResultLines(results);
+        expect(line).toContain('Failed: Forge sync failed while opening workspace state');
+        expect(line).toContain('Workspace: /home/davidmarsh/code/discoclaw/workspace/state.json');
+        expect(line).toContain('Last error: ENOENT: no such file or directory');
+        expect(line).toContain('Retry hint: recreate the state file, then rerun forge sync');
+        expect(line).toContain('cwd: /home/davidmarsh/code/discoclaw');
+        expect(line).toContain('...[omitted ');
+        expect(line).not.toContain('Stack: at runForgeSync');
+    });
+    it('falls back to the final hard cap when retained text is still too long', () => {
+        const results = [{
+                ok: true,
+                summary: `Downloaded attachment to /tmp/${'x'.repeat(120)}`,
+            }];
+        const [line] = buildCappedResultLines(results, 60);
+        expect(line.length).toBeLessThanOrEqual(60);
+        expect(line).toContain('Done: Downloaded attachment');
+        expect(line.endsWith('...[truncated]')).toBe(true);
+    });
+});
 describe('discordActionsPromptSection', () => {
     it('always includes the standard guidance when actions are enabled', () => {
         const flags = {