@yagni-app/code-staging 1.2.13-staging.2167.1 → 1.2.13-staging.2171.1

package/dist/extension/pipeline/planOutput.d.ts

@@ -46,6 +46,15 @@ export interface PlanFile {
         repository: string;
         what: string;
     }>;
+    /**
+     * What the plan found that the ticket did not ask for (ENG-2966): kept off
+     * the plan, shown at its gate for a person to make a ticket of.
+     */
+    followUps: Array<{
+        title: string;
+        why: string;
+        repository?: string;
+    }>;
 }
 /** The plan's `wrongRepository`, or null when it names no `owner/name`. Confidence is high only when said. */
 export declare function readWrongRepository(value: unknown): PlanFile["wrongRepository"] | null;
@@ -59,6 +68,12 @@ export declare function repositoryName(value: unknown): string | null;
  * for the same repository (case-insensitive) is dropped too.
  */
 export declare function readRepositoriesNeeded(value: unknown): PlanFile["repositoriesNeeded"];
+/**
+ * The plan's `followUps`, one per distinct title in the order written. An
+ * entry with no title is dropped; a repository that is not `owner/name` is
+ * left off its entry.
+ */
+export declare function readFollowUps(value: unknown): PlanFile["followUps"];
 /** The plan's ```plan-json block as an object, or null when it has none that parses. */
 export declare function extractPlanJson(text: string): Record<string, unknown> | null;
 /** The plan for a person to read: the text without its machine-readable block. */

package/dist/extension/pipeline/planOutput.js

@@ -77,6 +77,30 @@ export function readRepositoriesNeeded(value) {
     }
     return out;
 }
+/** At most this many follow-ups; the host keeps the same number. */
+const MAX_FOLLOW_UPS = 10;
+/**
+ * The plan's `followUps`, one per distinct title in the order written. An
+ * entry with no title is dropped; a repository that is not `owner/name` is
+ * left off its entry.
+ */
+export function readFollowUps(value) {
+    const out = [];
+    const seen = new Set();
+    for (const entry of list(value)) {
+        if (!isRecord(entry))
+            continue;
+        const title = text(entry.title, 200);
+        if (!title || seen.has(title.toLowerCase()))
+            continue;
+        seen.add(title.toLowerCase());
+        const repository = repositoryName(entry.repository);
+        out.push({ title, why: text(entry.why) ?? "", ...(repository ? { repository } : {}) });
+        if (out.length === MAX_FOLLOW_UPS)
+            break;
+    }
+    return out;
+}
 /** The plan's ```plan-json block as an object, or null when it has none that parses. */
 export function extractPlanJson(text) {
     let found = null;
@@ -162,6 +186,7 @@ export function buildPlanFile(planText, fallbackPaths) {
             repositoriesNeeded,
             ...(wrongRepository ? { wrongRepository } : {}),
             dependsOn: readDependsOn(block?.dependsOn),
+            followUps: readFollowUps(block?.followUps),
         },
         found: block !== null,
     };

package/dist/extension/pipeline/stages.d.ts

@@ -74,6 +74,14 @@ export declare const DOCUMENTATION_REPOSITORY_RULE: string;
  * without another repository is named, not left as a Risks bullet.
  */
 export declare const RIGHT_REPOSITORY_RULE: string;
+/**
+ * What the planner does with what it finds that the ticket did not ask for
+ * (ENG-2966): a nearby bug, a leak, a refactor is a follow-up a person can
+ * make a ticket of, never a question, a decision or a step of this plan. And
+ * nothing the ticket asked for is put off: work it needs in another
+ * repository goes in repositoriesNeeded, so the host offers the split.
+ */
+export declare const FOLLOW_UPS_RULE: string;
 export declare const PLAN_JSON_CONTRACT: string;
 /**
  * The one-attempt lane's session (`yagni go --headless --cloud`): plan,

package/dist/extension/pipeline/stages.js

@@ -167,6 +167,20 @@ export const RIGHT_REPOSITORY_RULE = '- "wrongRepository" is for a ticket whose
     '- "dependsOn" lists every other repository that needs a change before this one can work, each ' +
     "with one line on what it needs there, whether or not the Other repositories section lists it. " +
     "Such a dependency is never only a Risks bullet. Never list a repository the change only reads.";
+/**
+ * What the planner does with what it finds that the ticket did not ask for
+ * (ENG-2966): a nearby bug, a leak, a refactor is a follow-up a person can
+ * make a ticket of, never a question, a decision or a step of this plan. And
+ * nothing the ticket asked for is put off: work it needs in another
+ * repository goes in repositoriesNeeded, so the host offers the split.
+ */
+export const FOLLOW_UPS_RULE = '- "followUps" lists what you found that the ticket did not ask for: a nearby bug, a security ' +
+    "leak, a refactor. Each is a title, one line on why it matters, and its repository when that is " +
+    "not this one. A follow-up is never a question, a decision or a step of this plan, however real " +
+    "it is: a person makes a ticket of it if they want one. Leave it out when you found nothing.\n" +
+    '- Never put off the ticket\'s own work ("for another session", "in a later ticket"). Plan all of ' +
+    "what it asked for. Work it needs in another repository goes in repositoriesNeeded, so the ticket " +
+    "is split, and is never left out of the plan.";
 export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json block and nothing after it:\n" +
     "```plan-json\n" +
     "{\n" +
@@ -181,12 +195,14 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     '  "paths": ["apps/web"],\n' +
     '  "repositoriesNeeded": [{"repository": "owner/name", "summary": "one line on what changes there"}],\n' +
     '  "wrongRepository": {"repository": "owner/name", "why": "one line", "confidence": "high|low"},\n' +
-    '  "dependsOn": [{"repository": "owner/name", "what": "one line on what it needs there"}]\n' +
+    '  "dependsOn": [{"repository": "owner/name", "what": "one line on what it needs there"}],\n' +
+    '  "followUps": [{"title": "...", "why": "one line", "repository": "owner/name"}]\n' +
     "}\n" +
     "```\n" +
     "Rules the host enforces, so getting them wrong loses the question:\n" +
     '- "kind" is "fact" when the answer could be looked up and "decision" when it is a person\'s call. ' +
-    'A decision you would make yourself carries your recommendation and is stated, not asked; mark "oneWay": true only when it cannot be undone.\n' +
+    'A decision you would make yourself carries your recommendation and is stated, not asked; mark "oneWay": true only when it cannot be undone. ' +
+    "A recommendation whose planChange adds acceptance criteria widens the ticket, so a person is asked it.\n" +
     '- On a fact, "checked" says in at most 4 short lines what you read or ran for it and what it showed, and what you could not read. ' +
     "A fact whose every answer plans the same work (no option carries a planChange) is stated as your assumption, not asked.\n" +
     "- At most two alternatives per question. A planChange only adds or rewords; it never removes.\n" +
@@ -200,7 +216,9 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     "never list a repository the change only reads. " +
     DOCUMENTATION_REPOSITORY_RULE +
     "\n" +
-    RIGHT_REPOSITORY_RULE;
+    RIGHT_REPOSITORY_RULE +
+    "\n" +
+    FOLLOW_UPS_RULE;
 /**
  * The cloud plan stage's task: the input folder the host wrote, read first,
  * then a plan grounded in the installed checkout, inside the stage's own

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "1.2.13-staging.2167.1",
+  "version": "1.2.13-staging.2171.1",
   "description": "YAGNI Code: a terminal coding agent that already knows your company. One YAGNI login routes the model and grounds the agent in your team's context.",
   "license": "SEE LICENSE IN LICENSE.md",
   "author": "YAGNI, Inc. <jack@yagni.app> (https://yagni.app)",
@@ -58,5 +58,5 @@
     "turndown": "^7.2.4",
     "typebox": "^1.3.15"
   },
-  "yagniSourceSha": "eacf9167a622d9ff9a977d0fde9d29040a78aa60"
+  "yagniSourceSha": "4c225e34b3364d25830d9d4d88f071ffedf799f3"
 }