@supaku/agentfactory-linear 0.7.48 → 0.7.49

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

@@ -6,6 +6,7 @@
  */
 import type { AgentWorkType, SubIssueStatus } from '../types.js';
 export declare const WORK_RESULT_MARKER_INSTRUCTION = "\n\nMANDATORY \u2014 Structured Result Marker:\nYou MUST include exactly one of these HTML comment markers in your final output:\n- On pass: <!-- WORK_RESULT:passed -->\n- On fail: <!-- WORK_RESULT:failed -->\nWithout this marker, the orchestrator CANNOT detect your result and the issue status will NOT be updated. Even if you encounter errors, always emit <!-- WORK_RESULT:failed -->.";
+export declare const READ_ONLY_CONSTRAINT = "\n\nCRITICAL CONSTRAINT \u2014 READ-ONLY ROLE:\nYou are a VALIDATION agent, NOT a development agent. You MUST NOT modify any source code, configuration files, migration files, or project files. Your role is strictly to READ, VALIDATE, and REPORT.\nFORBIDDEN actions: creating files, editing files, writing code, committing changes, patching snapshots, fixing bugs, resolving errors in code.\nALLOWED actions: reading files, running tests, running builds, checking CI status, posting comments, merging PRs (acceptance only), updating Linear status.\nIf you discover issues (missing files, broken builds, failing tests), REPORT them in your result comment and emit WORK_RESULT:failed. Do NOT attempt to fix them.";
 export declare const PR_SELECTION_GUIDANCE = "\n\nPR Selection (Multi-PR Handling):\nIssues may have multiple PRs. Select the correct one:\n1. Check linked PRs in the issue attachments/links for GitHub PR URLs\n2. Filter by state \u2014 prefer OPEN over MERGED over CLOSED: gh pr view NNN --json state\n3. If multiple OPEN PRs, pick the most recently created one\n4. Fallback search by branch: gh pr list --head \"$(git branch --show-current)\" --state open\n5. Last resort search by issue ID: gh pr list --state open --search \"[issue-id]\"\n6. If no PR found, emit WORK_RESULT:failed with explanation";
 /**
  * Context from the workflow state machine for retry enrichment.

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,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"}
+{"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,oBAAoB,2sBAMiI,CAAA;AAElK,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,CAiJR;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

@@ -21,6 +21,13 @@ You MUST include exactly one of these HTML comment markers in your final output:
 - On pass: <!-- WORK_RESULT:passed -->
 - On fail: <!-- WORK_RESULT:failed -->
 Without this marker, the orchestrator CANNOT detect your result and the issue status will NOT be updated. Even if you encounter errors, always emit <!-- WORK_RESULT:failed -->.`;
+export const READ_ONLY_CONSTRAINT = `
+
+CRITICAL CONSTRAINT — READ-ONLY ROLE:
+You are a VALIDATION agent, NOT a development agent. You MUST NOT modify any source code, configuration files, migration files, or project files. Your role is strictly to READ, VALIDATE, and REPORT.
+FORBIDDEN actions: creating files, editing files, writing code, committing changes, patching snapshots, fixing bugs, resolving errors in code.
+ALLOWED actions: reading files, running tests, running builds, checking CI status, posting comments, merging PRs (acceptance only), updating Linear status.
+If you discover issues (missing files, broken builds, failing tests), REPORT them in your result comment and emit WORK_RESULT:failed. Do NOT attempt to fix them.`;
 export const PR_SELECTION_GUIDANCE = `
 
 PR Selection (Multi-PR Handling):
@@ -123,6 +130,7 @@ Do NOT wait for user approval - create issues automatically.`;
             break;
         case 'qa':
             basePrompt = `QA ${identifier}. Validate the implementation against acceptance criteria.
+${READ_ONLY_CONSTRAINT}
 ${WORK_RESULT_MARKER_INSTRUCTION}
 ${PR_SELECTION_GUIDANCE}
 
@@ -136,6 +144,7 @@ Validation Steps:
             break;
         case 'acceptance':
             basePrompt = `Process acceptance for ${identifier}. Validate development and QA work is complete, verify PR is ready to merge (CI passing, no conflicts), merge the PR, and clean up local resources.
+${READ_ONLY_CONSTRAINT}
 ${WORK_RESULT_MARKER_INSTRUCTION}
 ${PR_SELECTION_GUIDANCE}
 
@@ -190,12 +199,43 @@ If any sub-issue is not Finished, report the failure and do not mark the parent
             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}`;
+            basePrompt = `Coordinate QA across sub-issues for parent issue ${identifier}. Fetch sub-issues, validate each against acceptance criteria, collect pass/fail results, and roll up to parent.
+${READ_ONLY_CONSTRAINT}
+${WORK_RESULT_MARKER_INSTRUCTION}
+${PR_SELECTION_GUIDANCE}
+
+QA Coordination Steps:
+1. Find and validate the correct PR (see PR selection above)
+2. Fetch all sub-issues and verify each is in Finished or later status
+3. Run tests scoped to the affected packages
+4. Verify the build passes (pnpm typecheck && pnpm build)
+5. Review changes against each sub-issue's acceptance criteria
+6. Verify cross-cutting concerns: shared types, API contracts, data flow between sub-issues
+7. Check deployment status (CI checks on the PR)
+
+Pass/Fail:
+- PASS (emit <!-- WORK_RESULT:passed -->): ALL tests pass, build succeeds, all sub-issues implemented, deployment healthy
+- FAIL (emit <!-- WORK_RESULT:failed -->): ANY test failure, build error, missing implementation, deployment failure
+Post a result comment listing per-sub-issue findings, then emit the marker.`;
             break;
         case 'acceptance-coordination':
-            basePrompt = `Coordinate acceptance across sub-issues for parent issue ${identifier}. Verify all sub-issues are Delivered, validate the PR (CI passing, no conflicts), merge the PR, and bulk-update sub-issues to Accepted.
-${WORK_RESULT_MARKER_INSTRUCTION}`;
+            basePrompt = `Coordinate acceptance across sub-issues for parent issue ${identifier}. Verify all sub-issues are Delivered, validate the PR, merge it, and bulk-update sub-issues to Accepted.
+${READ_ONLY_CONSTRAINT}
+${WORK_RESULT_MARKER_INSTRUCTION}
+${PR_SELECTION_GUIDANCE}
+
+Acceptance Coordination Steps:
+1. Find and validate the correct PR (see PR selection above)
+2. Verify ALL sub-issues are in Delivered or Accepted status
+3. Verify CI is passing and there are no merge conflicts
+4. Merge the PR
+5. Delete the remote branch after successful merge
+6. Post result comment with per-sub-issue status, then emit the marker
+
+Pass/Fail:
+- PASS (emit <!-- WORK_RESULT:passed -->): All sub-issues Delivered, CI passing, PR merged successfully
+- FAIL (emit <!-- WORK_RESULT:failed -->): Incomplete sub-issues, CI failure, merge conflicts, merge failed
+If ANY issue prevents merging, do NOT attempt to fix it — emit WORK_RESULT:failed with details.`;
             break;
     }
     basePrompt += HUMAN_BLOCKER_INSTRUCTION;

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

@@ -138,6 +138,46 @@ describe('defaultGeneratePrompt with workflowContext', () => {
         expect(result).toContain('Please focus on the API layer');
     });
 });
+describe('defaultGeneratePrompt read-only constraint', () => {
+    const readOnlyWorkTypes = ['qa', 'acceptance', 'qa-coordination', 'acceptance-coordination'];
+    for (const workType of readOnlyWorkTypes) {
+        it(`includes READ-ONLY constraint for ${workType}`, () => {
+            const result = defaultGeneratePrompt('PROJ-123', workType);
+            expect(result).toContain('READ-ONLY ROLE');
+            expect(result).toContain('MUST NOT modify any source code');
+            expect(result).toContain('WORK_RESULT:failed');
+        });
+    }
+    const writableWorkTypes = ['development', 'coordination', 'refinement', 'research', 'backlog-creation', 'inflight'];
+    for (const workType of writableWorkTypes) {
+        it(`does NOT include READ-ONLY constraint for ${workType}`, () => {
+            const result = defaultGeneratePrompt('PROJ-123', workType);
+            expect(result).not.toContain('READ-ONLY ROLE');
+        });
+    }
+});
+describe('defaultGeneratePrompt coordination prompts have steps', () => {
+    it('qa-coordination includes numbered steps and pass/fail criteria', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'qa-coordination');
+        expect(result).toContain('QA Coordination Steps:');
+        expect(result).toContain('1. Find and validate the correct PR');
+        expect(result).toContain('Pass/Fail:');
+        expect(result).toContain('WORK_RESULT:passed');
+        expect(result).toContain('WORK_RESULT:failed');
+        expect(result).toContain('PR Selection');
+    });
+    it('acceptance-coordination includes numbered steps and pass/fail criteria', () => {
+        const result = defaultGeneratePrompt('PROJ-100', 'acceptance-coordination');
+        expect(result).toContain('Acceptance Coordination Steps:');
+        expect(result).toContain('1. Find and validate the correct PR');
+        expect(result).toContain('Merge the PR');
+        expect(result).toContain('Pass/Fail:');
+        expect(result).toContain('WORK_RESULT:passed');
+        expect(result).toContain('WORK_RESULT:failed');
+        expect(result).toContain('do NOT attempt to fix it');
+        expect(result).toContain('PR Selection');
+    });
+});
 describe('defaultGeneratePrompt coordination retry', () => {
     it('uses fresh coordination prompt when no workflowContext', () => {
         const result = defaultGeneratePrompt('PROJ-100', 'coordination');

package/package.json

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