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

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

@@ -35,7 +35,22 @@ 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;
+    }>;
 }
+/** 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;
 /**

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) {
@@ -123,8 +151,18 @@ 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),
+        },
         found: block !== null,
     };
 }

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

@@ -68,6 +68,12 @@ 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;
 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,19 @@ 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.";
 export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json block and nothing after it:\n" +
     "```plan-json\n" +
     "{\n" +
@@ -166,7 +179,9 @@ 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' +
     "}\n" +
     "```\n" +
     "Rules the host enforces, so getting them wrong loses the question:\n" +
@@ -183,7 +198,9 @@ 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;
 /**
  * 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.2167.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": "eacf9167a622d9ff9a977d0fde9d29040a78aa60"
 }