@yagni-app/code-staging 1.2.9-staging.2089.1 → 1.2.10-staging.2095.1

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

@@ -25,7 +25,25 @@ export interface PlanFile {
     }>;
     /** The package folders the plan touches, repo-relative. */
     paths: string[];
+    /**
+     * The repositories the change writes to, in build order, the ticket's own
+     * included, when the plan judges the change spans more than one of the
+     * Team's repositories (YAG-1058). Empty for a one-repository change. The
+     * host fences the names to the Team's attached repositories.
+     */
+    repositoriesNeeded: Array<{
+        repository: string;
+        summary: string;
+    }>;
 }
+/** An `owner/name` repository the plan names, or null when it is not one. */
+export declare function repositoryName(value: unknown): string | null;
+/**
+ * The plan's `repositoriesNeeded`, one entry per distinct `owner/name` in the
+ * order written. An entry that names no repository is dropped; a second entry
+ * for the same repository (case-insensitive) is dropped too.
+ */
+export declare function readRepositoriesNeeded(value: unknown): PlanFile["repositoriesNeeded"];
 /** 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

@@ -15,7 +15,40 @@ const PLAN_JSON_BLOCK = /```plan-json[^\n]*\n([\s\S]*?)```/gi;
 const MAX_PLAN_QUESTIONS = 20;
 const MAX_LIST = 40;
 const MAX_TEXT = 500;
+/** The host splits a ticket into at most this many repositories; more is a Project. The
+ *  same number as `MAX_SPLIT_CHILDREN` in @yagni/shared, kept here because this package
+ *  does not depend on it. */
+const MAX_REPOSITORIES_NEEDED = 12;
 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) {
+    const name = text(value, 200)
+        ?.replace(/^https?:\/\/github\.com\//i, "")
+        .replace(/\.git$/i, "")
+        .replace(/^\/+|\/+$/g, "");
+    if (!name || !/^[\w.-]+\/[\w.-]+$/.test(name))
+        return null;
+    return name;
+}
+/**
+ * The plan's `repositoriesNeeded`, one entry per distinct `owner/name` in the
+ * order written. An entry that names no repository is dropped; a second entry
+ * for the same repository (case-insensitive) is dropped too.
+ */
+export function readRepositoriesNeeded(value) {
+    const out = [];
+    const seen = new Set();
+    for (const entry of list(value)) {
+        const repository = isRecord(entry) ? repositoryName(entry.repository) : repositoryName(entry);
+        if (!repository || seen.has(repository.toLowerCase()))
+            continue;
+        seen.add(repository.toLowerCase());
+        out.push({ repository, summary: (isRecord(entry) && text(entry.summary)) || "" });
+        if (out.length === MAX_REPOSITORIES_NEEDED)
+            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;
@@ -89,6 +122,10 @@ export function buildPlanFile(planText, fallbackPaths) {
         .map(repoPath)
         .filter((path) => path !== null);
     const paths = [...new Set(named.length > 0 ? named : fallbackPaths())].slice(0, MAX_LIST);
-    return { file: { version: 1, questions, decisionsCited, checks, paths }, found: block !== null };
+    const repositoriesNeeded = readRepositoriesNeeded(block?.repositoriesNeeded);
+    return {
+        file: { version: 1, questions, decisionsCited, checks, paths, repositoriesNeeded },
+        found: block !== null,
+    };
 }
 //# sourceMappingURL=planOutput.js.map

package/dist/extension/pipeline/stages.js

@@ -156,7 +156,8 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     '"checked": [{"status": "checked|not_checked", "text": "..."}]}],\n' +
     '  "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' +
+    '  "paths": ["apps/web"],\n' +
+    '  "repositoriesNeeded": [{"repository": "owner/name", "summary": "one line on what changes there"}]\n' +
     "}\n" +
     "```\n" +
     "Rules the host enforces, so getting them wrong loses the question:\n" +
@@ -167,7 +168,12 @@ export const PLAN_JSON_CONTRACT = "End your answer with a fenced ```plan-json bl
     "- At most two alternatives per question. A planChange only adds or rewords; it never removes.\n" +
     '- "checks" are the commands that prove the change, each run from "cwd" (repo-relative, "." for the root). ' +
     '"paths" are the package folders the plan touches.\n' +
-    '- "decisionsCited" lists only decisions the read returned that the plan relies on. Omit what you do not use.';
+    '- "decisionsCited" lists only decisions the read returned that the plan relies on. Omit what you do not use.\n' +
+    '- "repositoriesNeeded" is for a change that needs writes in more than one repository: list every ' +
+    "repository the change writes to, in the order they must be built and merged, this one included, each " +
+    "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.";
 /**
  * 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
@@ -199,7 +205,11 @@ function cloudPlanTask(inputDir) {
         'If a command fails with "no space left on device", the disk is full: stop there. Do not delete files, ' +
         "prune Docker or retry. Write the plan and say what you could not check.\n\n" +
         "When the work depends on another repository the ticket's Other repositories section lists, clone it " +
-        "read-only on demand with `git clone --filter=blob:none --depth 1` into ~/context/<repo> and read it there.\n\n" +
+        "read-only on demand with `git clone --filter=blob:none --depth 1` into ~/context/<repo> and read it there. " +
+        "When the change itself needs writes in one of those repositories (an API this repository calls, a " +
+        "package it consumes), say so in the plan's repositoriesNeeded, in build order: the ticket is then " +
+        "split into one ticket per repository, built in that order, and each step for another repository " +
+        "names it so the host can sort the steps.\n\n" +
         `Call team_read with tool "${DECISIONS_READ_TOOL}" before you ask a person anything: a question an earlier ` +
         "decision on this repository already answers is not a question. Cite the decisions the plan relies on.\n\n" +
         "When the change shows in a page, plan a render check the build runs (open the page in the sandbox's " +

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "1.2.9-staging.2089.1",
+  "version": "1.2.10-staging.2095.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": "0a52413870fd8ac25bc138527a51d189b5ee5322"
+  "yagniSourceSha": "ec85145bd5d4dcd937024a643692270a5c5a0beb"
 }