@taiga-ai/mcp-server 0.25.0-dev.476 → 0.25.0-dev.477

package/dist/index.js

@@ -981,6 +981,19 @@ const INITIATIVE_STATES = [
 	"done",
 	"canceled"
 ];
+/**
+* The states the line works, and the only ones that carry its flag. One copy
+* for every reader that means "on the line": the flag the API clears when an
+* initiative leaves it, the remediation settle on the same departure, and the
+* MCP's account of where an initiative stands (PRO-2840). Everything else
+* (backlog, todo, done, canceled) is off it. The lifecycle derivation does not
+* read it: it derives the stage from the run alone (PRO-2828).
+*/
+const LINE_STATES = [
+	"queue",
+	"plan",
+	"build"
+];
 /** Finished work: settled by a person or a merge, or archived. The one
 *  predicate every "can this still move / plan / build" guard reads. */
 function isSettledOrArchived(item) {
@@ -4273,7 +4286,7 @@ const ITEM_OUTCOME_FIELDS = [
 * needsHumanCause is only the line's flag; a plan that failed outside the line
 * leaves it null, and the lifecycle is what says why and what moves it on.
 */
-const LIFECYCLE_MEANING = "situation says in a sentence where it stands and what moves it on. lifecycle.cta names the next call: start_plan taiga_queue_initiative, retry_plan taiga_plan_again (planFailureCause says why), start_build taiga_approve_plan, resume_build taiga_resume_initiative, retry_build a fresh run (web only), review_pr and view_pr the pull request. needsHumanCause is set only when the line stopped on it.";
+const LIFECYCLE_MEANING = "situation says in a sentence where it stands and what moves it on. lifecycle.cta names the next call: start_plan taiga_queue_initiative, retry_plan taiga_plan_again (planFailureCause says why), start_build taiga_approve_plan, resume_build taiga_resume_initiative, retry_build a fresh run (web only; off the line, taiga_queue_initiative instead, as situation says), review_pr and view_pr the pull request. needsHumanCause is set only when the line stopped on it.";
 /** The move each call to action is, in the tools' words. */
 const CTA_MOVE = {
 	start_plan: "taiga_queue_initiative puts it on the line to be planned and built (paid work: ask first)",
@@ -4284,6 +4297,10 @@ const CTA_MOVE = {
 	review_pr: "a person reviews and merges the pull request",
 	view_pr: "the line merges the pull request once its checks pass"
 };
+/** The move for an initiative off the line whose last run did not land: there is no line to take it off (PRO-2840). */
+const OFF_THE_LINE_MOVE = "taiga_queue_initiative puts it back on the line (paid work: ask first)";
+/** What happens to a queued initiative whose last run did not land: the line plans it again (PRO-2840). */
+const REQUEUED_MOVE = "the line plans it again when its turn comes";
 /** How a run stands, as the end of "its latest run ...", by disposition. */
 const RUN_STANDS = {
 	in_flight: "is running",
@@ -4317,6 +4334,11 @@ function situationOf(item) {
 	const parts = [];
 	let stage = STAGE[life.stage] ?? life.stage;
 	if (life.stage === "planning" && life.turn === "you") stage = `Planning stopped${typeof life.planFailureCause === "string" ? ` (${life.planFailureCause})` : ""}`;
+	const column = typeof item["state"] === "string" ? item["state"] : null;
+	const offTheLine = column !== null && !LINE_STATES.includes(column);
+	const stalled = life.stage === "building" && life.turn !== "agent";
+	const requeued = stalled && column === "queue";
+	if (stalled) stage = requeued ? "Queued" : offTheLine ? "Off the line" : column === null ? "Its build stopped" : "On the line, but its build stopped";
 	const build = life.latestBuild;
 	if (build && typeof build.status === "string" && life.stage !== "done" && life.stage !== "canceled") {
 		const pr = typeof build.prNumber === "number" ? ` (PR #${build.prNumber})` : "";
@@ -4336,8 +4358,9 @@ function situationOf(item) {
 	}
 	const turn = life.turn === "you" ? "A person's turn" : life.turn === "agent" ? "The agent's turn" : null;
 	const handedBack = handbackSaid(build?.reviewEscalation);
-	const move = life.cta === "review_pr" && handedBack ? `a person should ${handbackNextStep(build.reviewEscalation)}` : typeof life.cta === "string" ? CTA_MOVE[life.cta] : void 0;
-	if (life.cta === "view_pr") parts.push(`Nothing to do: ${move}.`);
+	const move = life.cta === "review_pr" && handedBack ? `a person should ${handbackNextStep(build.reviewEscalation)}` : stalled && offTheLine ? OFF_THE_LINE_MOVE : typeof life.cta === "string" ? CTA_MOVE[life.cta] : void 0;
+	if (requeued) parts.push(`Nothing to do: ${REQUEUED_MOVE}.`);
+	else if (life.cta === "view_pr") parts.push(`Nothing to do: ${move}.`);
 	else if (turn && move) parts.push(`${turn}: ${move}.`);
 	else if (turn) parts.push(`${turn}.`);
 	else if (move) parts.push(`Next: ${move}.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taiga-ai/mcp-server",
-  "version": "0.25.0-dev.476",
+  "version": "0.25.0-dev.477",
   "description": "MCP server for Taiga -- gives AI assistants access to documents and products",
   "type": "module",
   "main": "./dist/index.js",

package/src/server.integration.test.ts

@@ -7978,12 +7978,67 @@ describe('what the third agent review found (PRO-2800)', () => {
     );
 
     const said =
-      "Being built; its latest run had its pull request closed without merging (PR #28). A person's turn: a new " +
+      "On the line, but its build stopped; its latest run had its pull request closed without merging (PR #28). A person's turn: a new " +
       'run starts over from the web; taiga_move_initiative takes it off the line.';
     expect(got.situation).toBe(said);
     expect(listed.data[0].situation).toBe(said);
   });
 
+  it.each([
+    ['todo', 'Off the line', 'taiga_queue_initiative puts it back on the line (paid work: ask first)'],
+    ['build', 'On the line, but its build stopped', 'a new run starts over from the web'],
+    ['queue', 'Queued', 'Nothing to do: the line plans it again when its turn comes.'],
+  ])('says where a %s initiative with a dead run is, from its column (PRO-2840)', async (state, where, move) => {
+    const item = {
+      id: ITEM_ID,
+      title: 'Add login',
+      state,
+      lifecycle: {
+        turn: 'you',
+        stage: 'building',
+        cta: 'retry_build',
+        planFailureCause: null,
+        latestBuild: { status: 'closed', prNumber: 28 },
+      },
+    };
+    const { client } = await bootMcpPair({
+      capabilities: ['backlog:read'],
+      routes: { [`GET /v1/backlog/${ITEM_ID}`]: { data: item } },
+    });
+
+    const got = parseToolResult(
+      await client.callTool({ name: 'taiga_get_initiative', arguments: { initiativeId: ITEM_ID } })
+    );
+
+    expect(got.situation).toMatch(new RegExp(`^${where}; its latest run had its pull request closed`));
+    expect(got.situation).toContain(move);
+    expect(got.situation).not.toMatch(/Being built/);
+  });
+
+  it('says Being built only while the agent is building it (PRO-2840)', async () => {
+    const item = {
+      id: ITEM_ID,
+      title: 'Add login',
+      lifecycle: {
+        turn: 'agent',
+        stage: 'building',
+        cta: null,
+        planFailureCause: null,
+        latestBuild: { status: 'implementing', prNumber: null },
+      },
+    };
+    const { client } = await bootMcpPair({
+      capabilities: ['backlog:read'],
+      routes: { [`GET /v1/backlog/${ITEM_ID}`]: { data: item } },
+    });
+
+    const got = parseToolResult(
+      await client.callTool({ name: 'taiga_get_initiative', arguments: { initiativeId: ITEM_ID } })
+    );
+
+    expect(got.situation).toMatch(/^Being built; its latest run is running\. The agent's turn\./);
+  });
+
   it("keeps the run's own reason beside the halt that stopped the line", async () => {
     const item = {
       id: ITEM_ID,
@@ -8002,7 +8057,9 @@ describe('what the third agent review found (PRO-2800)', () => {
     );
 
     expect(got.situation).toContain('The pull request was closed by alice.');
-    expect(got.situation).toMatch(/^Being built\. The line stopped on it: .+\. The pull request was closed by alice\./);
+    expect(got.situation).toMatch(
+      /^Its build stopped\. The line stopped on it: .+\. The pull request was closed by alice\./
+    );
   });
 
   it("leaves the keys of a document's sections as their writers named them", async () => {

package/src/tools/backlogs.ts

@@ -6,6 +6,7 @@ import {
   type BacklogItem,
   BUILD_RUN_STATUSES,
   INITIATIVE_STATES,
+  LINE_STATES,
   PLAN_REJECTION_COMMENT_MAX_CHARS,
   PR_CLOSING_STATES,
   USER_DROP_STATES,
@@ -68,8 +69,9 @@ const ITEM_OUTCOME_FIELDS = [
 const LIFECYCLE_MEANING =
   'situation says in a sentence where it stands and what moves it on. lifecycle.cta names the next call: ' +
   'start_plan taiga_queue_initiative, retry_plan taiga_plan_again (planFailureCause says why), start_build ' +
-  'taiga_approve_plan, resume_build taiga_resume_initiative, retry_build a fresh run (web only), review_pr and ' +
-  'view_pr the pull request. needsHumanCause is set only when the line stopped on it.';
+  'taiga_approve_plan, resume_build taiga_resume_initiative, retry_build a fresh run (web only; off the line, ' +
+  'taiga_queue_initiative instead, as situation says), review_pr and view_pr the pull request. needsHumanCause ' +
+  'is set only when the line stopped on it.';
 
 /** The move each call to action is, in the tools' words. */
 const CTA_MOVE: Record<string, string> = {
@@ -83,6 +85,12 @@ const CTA_MOVE: Record<string, string> = {
   view_pr: 'the line merges the pull request once its checks pass',
 };
 
+/** The move for an initiative off the line whose last run did not land: there is no line to take it off (PRO-2840). */
+const OFF_THE_LINE_MOVE = 'taiga_queue_initiative puts it back on the line (paid work: ask first)';
+
+/** What happens to a queued initiative whose last run did not land: the line plans it again (PRO-2840). */
+const REQUEUED_MOVE = 'the line plans it again when its turn comes';
+
 /** How a run stands, as the end of "its latest run ...", by disposition. */
 const RUN_STANDS: Record<string, string> = {
   in_flight: 'is running',
@@ -123,6 +131,26 @@ export function situationOf(item: Record<string, unknown>): string | null {
   if (life.stage === 'planning' && life.turn === 'you') {
     stage = `Planning stopped${typeof life.planFailureCause === 'string' ? ` (${life.planFailureCause})` : ''}`;
   }
+  // Being built only while the agent is at it (PRO-2840). With its run failed,
+  // stopped or its pull request closed, nothing is building: the run's outcome
+  // follows and the turn says whose move it is. The lifecycle derives this
+  // stage from the run alone, so an initiative already moved back to the
+  // backlog or todo is said to be off the line, which only its column knows.
+  // Queued, the line plans it again when its turn comes (activation replaces a dead run), so it is nobody's
+  // move. Without a column (an old row) it is said neutrally.
+  const column = typeof item['state'] === 'string' ? item['state'] : null;
+  const offTheLine = column !== null && !(LINE_STATES as readonly string[]).includes(column);
+  const stalled = life.stage === 'building' && life.turn !== 'agent';
+  const requeued = stalled && column === 'queue';
+  if (stalled) {
+    stage = requeued
+      ? 'Queued'
+      : offTheLine
+        ? 'Off the line'
+        : column === null
+          ? 'Its build stopped'
+          : 'On the line, but its build stopped';
+  }
   const build = life.latestBuild as { status?: unknown; prNumber?: unknown; prMerged?: unknown } | null | undefined;
   if (build && typeof build.status === 'string' && life.stage !== 'done' && life.stage !== 'canceled') {
     const pr = typeof build.prNumber === 'number' ? ` (PR #${build.prNumber})` : '';
@@ -151,10 +179,13 @@ export function situationOf(item: Record<string, unknown>): string | null {
   const move =
     life.cta === 'review_pr' && handedBack
       ? `a person should ${handbackNextStep((build as { reviewEscalation?: unknown }).reviewEscalation)}`
-      : typeof life.cta === 'string'
-        ? CTA_MOVE[life.cta]
-        : undefined;
-  if (life.cta === 'view_pr') parts.push(`Nothing to do: ${move}.`);
+      : stalled && offTheLine
+        ? OFF_THE_LINE_MOVE
+        : typeof life.cta === 'string'
+          ? CTA_MOVE[life.cta]
+          : undefined;
+  if (requeued) parts.push(`Nothing to do: ${REQUEUED_MOVE}.`);
+  else if (life.cta === 'view_pr') parts.push(`Nothing to do: ${move}.`);
   else if (turn && move) parts.push(`${turn}: ${move}.`);
   else if (turn) parts.push(`${turn}.`);
   else if (move) parts.push(`Next: ${move}.`);