@supaku/agentfactory-linear 0.7.46 → 0.7.48

package/dist/src/defaults/prompts.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prompts.d.ts","sourceRoot":"","sources":["../../../src/defaults/prompts.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AAahE,eAAO,MAAM,8BAA8B,uYAMsI,CAAA;AAEjL,eAAO,MAAM,qBAAqB,ijBAS0B,CAAA;AAE5D;;;GAGG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAA;IAClB,QAAQ,EAAE,MAAM,CAAA;IAChB,cAAc,EAAE,MAAM,GAAG,IAAI,CAAA;IAC7B,cAAc,CAAC,EAAE,MAAM,CAAA;CACxB;AAED;;GAEG;AACH,wBAAgB,wBAAwB,CACtC,QAAQ,EAAE,aAAa,EACvB,OAAO,EAAE,eAAe,GACvB,MAAM,CA0DR;AAED;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CACnC,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,aAAa,EACvB,cAAc,CAAC,EAAE,MAAM,EACvB,eAAe,CAAC,EAAE,eAAe,GAChC,MAAM,CAuFR;AAED;;GAEG;AACH,wBAAgB,2BAA2B,CACzC,eAAe,EAAE,MAAM,EACvB,gBAAgB,EAAE,cAAc,EAAE,GACjC,MAAM,CAoBR;AAED;;GAEG;AACH,wBAAgB,mCAAmC,CACjD,eAAe,EAAE,MAAM,EACvB,gBAAgB,EAAE,cAAc,EAAE,GACjC,MAAM,CAmBR"}
+{"version":3,"file":"prompts.d.ts","sourceRoot":"","sources":["../../../src/defaults/prompts.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AAahE,eAAO,MAAM,8BAA8B,uYAMsI,CAAA;AAEjL,eAAO,MAAM,qBAAqB,ijBAS0B,CAAA;AAE5D;;;GAGG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAA;IAClB,QAAQ,EAAE,MAAM,CAAA;IAChB,cAAc,EAAE,MAAM,GAAG,IAAI,CAAA;IAC7B,cAAc,CAAC,EAAE,MAAM,CAAA;CACxB;AAED;;GAEG;AACH,wBAAgB,wBAAwB,CACtC,QAAQ,EAAE,aAAa,EACvB,OAAO,EAAE,eAAe,GACvB,MAAM,CA0DR;AAED;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CACnC,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,aAAa,EACvB,cAAc,CAAC,EAAE,MAAM,EACvB,eAAe,CAAC,EAAE,eAAe,GAChC,MAAM,CAgHR;AAED;;GAEG;AACH,wBAAgB,2BAA2B,CACzC,eAAe,EAAE,MAAM,EACvB,gBAAgB,EAAE,cAAc,EAAE,GACjC,MAAM,CAoBR;AAED;;GAEG;AACH,wBAAgB,mCAAmC,CACjD,eAAe,EAAE,MAAM,EACvB,gBAAgB,EAAE,cAAc,EAAE,GACjC,MAAM,CAmBR"}

package/dist/src/defaults/prompts.js

@@ -150,8 +150,32 @@ Acceptance Steps:
         case 'refinement':
             basePrompt = `Refine ${identifier} based on rejection feedback. Read comments, update requirements, then return to Backlog.`;
             break;
-        case 'coordination':
-            basePrompt = `Coordinate sub-issue execution for parent issue ${identifier}. Fetch sub-issues with dependency graph, create tasks mapping to each sub-issue, spawn sub-agents for unblocked sub-issues in parallel, monitor completion, and create a single PR with all changes when done.
+        case 'coordination': {
+            const isRetry = workflowContext && workflowContext.cycleCount > 0;
+            if (isRetry) {
+                basePrompt = `Fix issues found during QA for parent issue ${identifier}.
+
+REWORK MODE — DO NOT re-coordinate sub-issues from scratch.
+All sub-issues are already Finished and a PR already exists. QA failed and specific fixes are needed.
+
+MANDATORY FIRST STEPS:
+1. Read the most recent QA failure comments: pnpm af-linear list-comments ${identifier}
+2. Identify the SPECIFIC fixes needed from the QA failure details below
+3. Apply fixes directly — do NOT re-spawn sub-agents for already-complete work
+4. Commit fixes, push to the existing PR branch
+5. Run full validation: pnpm typecheck && pnpm build && pnpm test
+6. Update parent issue status to Finished
+
+COMMON REWORK FIXES:
+- Missing migration .json snapshots: Find .ts migrations without .json, patch the latest snapshot
+- TypeScript errors: Read the error details, fix the type issues
+- Missing API fields: Update route handlers to include missing fields
+- Build failures: Run pnpm build, diagnose, fix
+
+If the existing PR branch is checked out, work directly on it. Do not create a new branch or PR.`;
+            }
+            else {
+                basePrompt = `Coordinate sub-issue execution for parent issue ${identifier}. Fetch sub-issues with dependency graph, create tasks mapping to each sub-issue, spawn sub-agents for unblocked sub-issues in parallel, monitor completion, and create a single PR with all changes when done.
 
 SUB-ISSUE STATUS MANAGEMENT:
 Update sub-issue statuses in Linear as work progresses:
@@ -162,7 +186,9 @@ Update sub-issue statuses in Linear as work progresses:
 COMPLETION VERIFICATION:
 Before marking the parent issue as complete, verify ALL sub-issues are in Finished status.
 If any sub-issue is not Finished, report the failure and do not mark the parent as complete.`;
+            }
             break;
+        }
         case 'qa-coordination':
             basePrompt = `Coordinate QA across sub-issues for parent issue ${identifier}. Fetch sub-issues, spawn QA sub-agents in parallel for each sub-issue, collect pass/fail results, and roll up to parent. ALL sub-issues must pass QA for the parent to pass.
 ${WORK_RESULT_MARKER_INSTRUCTION}`;

package/dist/src/defaults/prompts.test.js

@@ -52,6 +52,16 @@ describe('buildFailureContextBlock', () => {
             expect(result).toContain('retry #2');
         });
     });
+    describe('qa-coordination work type', () => {
+        it('returns empty string when no failure summary', () => {
+            expect(buildFailureContextBlock('qa-coordination', { ...baseContext, failureSummary: null })).toBe('');
+        });
+        it('includes previous QA results when failure summary exists', () => {
+            const result = buildFailureContextBlock('qa-coordination', baseContext);
+            expect(result).toContain('Previous QA Results');
+            expect(result).toContain('TypeError in UserService');
+        });
+    });
     describe('qa work type', () => {
         it('returns empty string when no failure summary', () => {
             expect(buildFailureContextBlock('qa', { ...baseContext, failureSummary: null })).toBe('');
@@ -128,3 +138,86 @@ describe('defaultGeneratePrompt with workflowContext', () => {
         expect(result).toContain('Please focus on the API layer');
     });
 });
+describe('defaultGeneratePrompt coordination retry', () => {
+    it('uses fresh coordination prompt when no workflowContext', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination');
+        expect(result).toContain('Coordinate sub-issue execution for parent issue PROJ-100');
+        expect(result).toContain('Fetch sub-issues with dependency graph');
+        expect(result).not.toContain('REWORK MODE');
+    });
+    it('uses fresh coordination prompt when cycleCount is 0', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 0,
+            strategy: 'normal',
+            failureSummary: null,
+        });
+        expect(result).toContain('Coordinate sub-issue execution for parent issue PROJ-100');
+        expect(result).not.toContain('REWORK MODE');
+    });
+    it('switches to rework mode when cycleCount > 0', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 1,
+            strategy: 'context-enriched',
+            failureSummary: '2 migrations missing .json snapshots',
+        });
+        expect(result).toContain('REWORK MODE');
+        expect(result).toContain('Fix issues found during QA for parent issue PROJ-100');
+        expect(result).toContain('DO NOT re-coordinate sub-issues from scratch');
+        expect(result).toContain('pnpm af-linear list-comments PROJ-100');
+        expect(result).not.toContain('Fetch sub-issues with dependency graph');
+    });
+    it('includes common rework fixes in rework mode', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 2,
+            strategy: 'context-enriched',
+            failureSummary: 'Build failed',
+        });
+        expect(result).toContain('Missing migration .json snapshots');
+        expect(result).toContain('TypeScript errors');
+        expect(result).toContain('Missing API fields');
+        expect(result).toContain('Build failures');
+    });
+    it('appends failure context block in rework mode', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 2,
+            strategy: 'context-enriched',
+            failureSummary: 'Tests failed: missing snapshot for 20260308_migration.ts',
+        });
+        expect(result).toContain('REWORK MODE');
+        expect(result).toContain('Retry Context');
+        expect(result).toContain('retry #2');
+        expect(result).toContain('missing snapshot for 20260308_migration.ts');
+    });
+    it('includes mention context alongside rework mode', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', 'Focus on the migration snapshots', {
+            cycleCount: 1,
+            strategy: 'normal',
+            failureSummary: 'Missing .json snapshots',
+        });
+        expect(result).toContain('REWORK MODE');
+        expect(result).toContain('Focus on the migration snapshots');
+    });
+    it('instructs agent to use existing PR branch', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 1,
+            strategy: 'normal',
+            failureSummary: null,
+        });
+        expect(result).toContain('Do not create a new branch or PR');
+        expect(result).toContain('push to the existing PR branch');
+    });
+    it('includes sub-issue status management in fresh coordination', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination');
+        expect(result).toContain('SUB-ISSUE STATUS MANAGEMENT');
+        expect(result).toContain('COMPLETION VERIFICATION');
+    });
+    it('does not include sub-issue status management in rework mode', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'coordination', undefined, {
+            cycleCount: 1,
+            strategy: 'normal',
+            failureSummary: 'QA failed',
+        });
+        expect(result).not.toContain('SUB-ISSUE STATUS MANAGEMENT');
+        expect(result).not.toContain('COMPLETION VERIFICATION');
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@supaku/agentfactory-linear",
-  "version": "0.7.46",
+  "version": "0.7.48",
   "type": "module",
   "description": "Linear issue tracker integration for AgentFactory — status transitions, agent sessions, work routing",
   "author": "Supaku (https://supaku.com)",