@aviaratech/ai-delivery 0.3.5 → 0.3.6

package/dist/lifecycle.test.js

@@ -2792,9 +2792,12 @@ test('MCP exposes one implementation surface for native lifecycle commands', ()
         'issue_pr_merge',
         'issue_finish',
         'issue_worktree_create',
+        'issue_worktree_transition_inspect',
+        'issue_worktree_transition_apply',
     ]);
     const validInputs = {
         issue_create: { title: 'Synthetic tracking parent' },
+        issue_update: { issueNumber: 17, park: true },
         issue_ready_check: { issueNumber: 17 },
         issue_develop: { issueNumber: 17 },
         issue_verify: { issueNumber: 17 },
@@ -2802,7 +2805,17 @@ test('MCP exposes one implementation surface for native lifecycle commands', ()
         issue_pr_review: { issueNumber: 17, prNumber: 23, artifact: '{}' },
         issue_pr_merge: { issueNumber: 17, prNumber: 23, strategy: 'merge' },
         issue_finish: { issueNumber: 17, prNumber: 23, strategy: 'merge' },
+        issue_worktree_transition_inspect: { issueNumber: 17, purpose: 'active-resume' },
+        issue_worktree_transition_apply: {
+            authority: 'worktree:transition',
+            planPath: '/synthetic/plan.json',
+            expectedPlanId: `sha256:${'a'.repeat(64)}`,
+            relinquishmentCommentId: 101,
+            acceptanceCommentId: 102,
+        },
     };
+    const updateTool = AI_DELIVERY_MCP_TOOLS.find((tool) => tool.name === 'issue_update');
+    assert.equal(updateTool.inputSchema.safeParse({ issueNumber: 17, park: false }).success, false);
     for (const [name, input] of Object.entries(validInputs)) {
         const tool = AI_DELIVERY_MCP_TOOLS.find((candidate) => candidate.name === name);
         assert.ok(tool, `${name} has an MCP definition`);
@@ -2870,6 +2883,9 @@ async function syntheticLifecycle(routing) {
     let trackingParent = false;
     let failFirstBlockerLink = true;
     let failNextProjectSync = false;
+    let interruptNextStatusUpdate = false;
+    let conflictNextStatusReadback = false;
+    let projectStatusWrites = 0;
     let checksPassed = true;
     let prDraft = true;
     let prState = 'open';
@@ -2976,6 +2992,12 @@ async function syntheticLifecycle(routing) {
                             nodes: projectStatus === null
                                 ? []
                                 : [
+                                    {
+                                        id: 'UNRELATED-ITEM',
+                                        isArchived: false,
+                                        content: { id: 'UNRELATED-ISSUE' },
+                                        fieldValueByName: { name: 'Active', optionId: 'STATUS-1' },
+                                    },
                                     {
                                         id: 'ITEM-17',
                                         isArchived: false,
@@ -2999,19 +3021,29 @@ async function syntheticLifecycle(routing) {
             return { addProjectV2ItemById: { item: { id: 'ITEM-17' } } };
         }
         if (query.includes('UpdateProjectDeliveryStatus')) {
+            assert.equal(variables.projectId, 'PROJECT-1');
+            assert.equal(variables.itemId, 'ITEM-17');
+            projectStatusWrites += 1;
             projectStatus = statusName(String(variables.optionId)) ?? null;
+            if (interruptNextStatusUpdate) {
+                interruptNextStatusUpdate = false;
+                throw new Error('synthetic ambiguous status write');
+            }
             return { updateProjectV2ItemFieldValue: { projectV2Item: { id: 'ITEM-17' } } };
         }
-        if (query.includes('ProjectDeliveryItemReadback'))
+        if (query.includes('ProjectDeliveryItemReadback')) {
+            const contentId = conflictNextStatusReadback ? 'UNRELATED-ISSUE' : issue.node_id;
+            conflictNextStatusReadback = false;
             return {
                 node: {
                     id: 'ITEM-17',
                     isArchived: false,
                     project: { id: 'PROJECT-1', number: 1 },
-                    content: { id: issue.node_id },
+                    content: { id: contentId },
                     fieldValueByName: { name: projectStatus, optionId: statuses[projectStatus] },
                 },
             };
+        }
         if (query.includes('blockedBy(first:'))
             return {
                 repository: {
@@ -3032,6 +3064,10 @@ async function syntheticLifecycle(routing) {
             blockerOpen = true;
             return { addBlockedBy: { issue: { number: issueNumber } } };
         }
+        if (query.includes('removeBlockedBy')) {
+            blockerOpen = false;
+            return { removeBlockedBy: { issue: { number: issueNumber } } };
+        }
         if (query.includes('RepositoryId'))
             return { repository: { id: 'REPO-1' } };
         if (query.includes('UpdateExactRefs')) {
@@ -3328,6 +3364,41 @@ async function syntheticLifecycle(routing) {
         assert.equal(started.title, issue.title);
         const row = started;
         assert.equal(projectStatus, 'Active');
+        const parked = await updateIssue(context, { issueNumber, park: true });
+        assert.equal(parked.projectStatus, 'Todo');
+        assert.equal(existsSync(row.path), true);
+        assert.equal((await updateIssue(context, { issueNumber, title: issue.title })).projectStatus, 'Todo');
+        await developIssue(context, issueNumber);
+        assert.equal(projectStatus, 'Active');
+        const blockedUpdate = await updateIssue(context, { issueNumber, blockedBy: [9] });
+        assert.deepEqual(blockedUpdate.blockedBy, [9]);
+        assert.equal(blockedUpdate.projectStatus, 'Blocked');
+        assert.equal(projectStatus, 'Waiting');
+        const blockedWrites = projectStatusWrites;
+        assert.equal((await updateIssue(context, { issueNumber, blockedBy: [9] })).projectStatus, 'Blocked');
+        assert.equal(projectStatusWrites, blockedWrites);
+        assert.equal((await updateIssue(context, { issueNumber, park: true })).projectStatus, 'Blocked');
+        await assert.rejects(developIssue(context, issueNumber), /not ready/u);
+        interruptNextStatusUpdate = true;
+        await assert.rejects(updateIssue(context, { issueNumber, blockedBy: [] }), /ambiguous status write/u);
+        assert.equal(blockerOpen, false);
+        assert.equal(projectStatus, 'Queued');
+        const interruptedWrites = projectStatusWrites;
+        const unblockedUpdate = await updateIssue(context, { issueNumber, blockedBy: [] });
+        assert.equal(projectStatusWrites, interruptedWrites);
+        assert.deepEqual(unblockedUpdate.blockedBy, []);
+        assert.equal(unblockedUpdate.projectStatus, 'Todo');
+        conflictNextStatusReadback = true;
+        await assert.rejects(updateIssue(context, { issueNumber, blockedBy: [9] }), /exact issue item/u);
+        assert.equal(blockerOpen, true);
+        assert.equal((await updateIssue(context, { issueNumber, blockedBy: [9] })).projectStatus, 'Blocked');
+        assert.equal((await updateIssue(context, { issueNumber, state: 'closed' })).projectStatus, 'Done');
+        assert.equal((await updateIssue(context, { issueNumber, park: true })).projectStatus, 'Done');
+        assert.equal((await resumeCreatedIssue(context, { issueNumber })).projectStatus, 'Done');
+        assert.equal((await updateIssue(context, { issueNumber, state: 'open', blockedBy: [] })).projectStatus, 'Done');
+        assert.equal(existsSync(row.path), true);
+        await developIssue(context, issueNumber);
+        assert.equal(projectStatus, 'Active');
         writeFileSync(join(row.path, 'change.txt'), 'feature\n');
         git(row.path, 'add', 'change.txt');
         git(row.path, 'commit', '-qm', 'synthetic change');
@@ -3550,9 +3621,9 @@ test.each([
     { remote: 'upstream', divergentOrigin: false },
     { remote: 'upstream', divergentOrigin: true },
 ])('synthetic lifecycle finishes with remote $remote and divergent origin $divergentOrigin', syntheticLifecycle, 15_000);
-test('an unrelated consumer uses public MCP setup through verified configured-author publication, counted review readback, merge and cleanup', async () => {
+test('an unrelated consumer uses public MCP setup through verified configured-author publication, counted review readback, merge and cleanup', { timeout: 0 }, async () => {
     await syntheticLifecycle({ remote: 'origin', divergentOrigin: false, producerOnboarding: true });
-}, 15_000);
+});
 test.each(['escape', 'broken', 'cycle'])('output observation rejects %s aliases without running policy commands', async (kind) => {
     const { root, counter } = await fixture();
     const base = realpathSync(mkdtempSync(join(tmpdir(), 'ai-delivery-output-alias-')));