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

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

@@ -35,7 +35,31 @@ export interface PlanFile {
         repository: string;
         summary: string;
     }>;
+    /** The repository the change belongs in instead, when it is not this one (ENG-2966). */
+    wrongRepository?: {
+        repository: string;
+        why: string;
+        confidence: "high" | "low";
+    };
+    /** The other repositories that need a change before this one can work (ENG-3033). */
+    dependsOn: Array<{
+        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;
+/** The plan's `dependsOn`, one entry per distinct repository, in the order written. */
+export declare function readDependsOn(value: unknown): PlanFile["dependsOn"];
 /** An `owner/name` repository the plan names, or null when it is not one. */
 export declare function repositoryName(value: unknown): string | null;
 /**
@@ -44,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

@@ -19,6 +19,34 @@ const MAX_TEXT = 500;
  *  same number as `MAX_SPLIT_CHILDREN` in @yagni/shared, kept here because this package
  *  does not depend on it. */
 const MAX_REPOSITORIES_NEEDED = 12;
+/** The plan's `wrongRepository`, or null when it names no `owner/name`. Confidence is high only when said. */
+export function readWrongRepository(value) {
+    if (!isRecord(value))
+        return null;
+    const repository = repositoryName(value.repository);
+    if (!repository)
+        return null;
+    return {
+        repository,
+        why: text(value.why) ?? "",
+        confidence: value.confidence === "high" ? "high" : "low",
+    };
+}
+/** The plan's `dependsOn`, one entry per distinct repository, in the order written. */
+export function readDependsOn(value) {
+    const out = [];
+    const seen = new Set();
+    for (const entry of list(value)) {
+        const repository = isRecord(entry) ? repositoryName(entry.repository) : null;
+        if (!repository || seen.has(repository.toLowerCase()))
+            continue;
+        seen.add(repository.toLowerCase());
+        out.push({ repository, what: (isRecord(entry) && text(entry.what)) || "" });
+        if (out.length === MAX_REPOSITORIES_NEEDED)
+            break;
+    }
+    return out;
+}
 const isRecord = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 /** An `owner/name` repository the plan names, or null when it is not one. */
 export function repositoryName(value) {
@@ -49,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;
@@ -123,8 +175,19 @@ export function buildPlanFile(planText, fallbackPaths) {
         .filter((path) => path !== null);
     const paths = [...new Set(named.length > 0 ? named : fallbackPaths())].slice(0, MAX_LIST);
     const repositoriesNeeded = readRepositoriesNeeded(block?.repositoriesNeeded);
+    const wrongRepository = readWrongRepository(block?.wrongRepository);
     return {
-        file: { version: 1, questions, decisionsCited, checks, paths, repositoriesNeeded },
+        file: {
+            version: 1,
+            questions,
+            decisionsCited,
+            checks,
+            paths,
+            repositoriesNeeded,
+            ...(wrongRepository ? { wrongRepository } : {}),
+            dependsOn: readDependsOn(block?.dependsOn),
+            followUps: readFollowUps(block?.followUps),
+        },
         found: block !== null,
     };
 }

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

@@ -68,6 +68,20 @@ export declare const DECISIONS_READ_TOOL = "yagni_decisions_read_for_repository"
  * planner left out.
  */
 export declare const DOCUMENTATION_REPOSITORY_RULE: string;
+/**
+ * Where the change belongs (ENG-2966, ENG-3033): a ticket routed to the wrong
+ * repository is said so, not planned around, and work this one cannot do
+ * 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

@@ -154,6 +154,33 @@ export const DECISIONS_READ_TOOL = "yagni_decisions_read_for_repository";
 export const DOCUMENTATION_REPOSITORY_RULE = "When a documentation repository in Other repositories documents this one and the change alters " +
     "behaviour its pages describe, list it in repositoriesNeeded too, with one line on the pages that " +
     "change. It is built beside the code, not after it.";
+/**
+ * Where the change belongs (ENG-2966, ENG-3033): a ticket routed to the wrong
+ * repository is said so, not planned around, and work this one cannot do
+ * without another repository is named, not left as a Risks bullet.
+ */
+export const RIGHT_REPOSITORY_RULE = '- "wrongRepository" is for a ticket whose change belongs in another repository the Other ' +
+    "repositories section lists (the code it asks to change is there, not here). Name that " +
+    'repository, say why in one line, and give "confidence": "high" only when you read the code ' +
+    "there or the ticket names it. Do not plan around it in this repository and do not call this " +
+    "one the main repository; plan only what this repository needs. Leave it out when the change is here.\n" +
+    '- "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" +
@@ -166,12 +193,16 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     '  "decisionsCited": [{"id": "<the decision id the read returned>", "why": "one line"}],\n' +
     '  "checks": [{"name": "lint", "command": "pnpm lint", "cwd": "apps/web"}],\n' +
     '  "paths": ["apps/web"],\n' +
-    '  "repositoriesNeeded": [{"repository": "owner/name", "summary": "one line on what changes there"}]\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' +
+    '  "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" +
@@ -183,7 +214,11 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     "with one line on what changes there. Only a repository the ticket's Other repositories section lists " +
     "counts; the host drops any other name. Leave it out for a change inside this repository alone, and " +
     "never list a repository the change only reads. " +
-    DOCUMENTATION_REPOSITORY_RULE;
+    DOCUMENTATION_REPOSITORY_RULE +
+    "\n" +
+    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.2158.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": "b6c406df664ab9bf4977c116c0fe9e30ea4bc631"
+  "yagniSourceSha": "4c225e34b3364d25830d9d4d88f071ffedf799f3"
 }