dw-sdlc 0.19.0 → 0.20.0

package/config/agentic.config.schema.json

@@ -2,7 +2,7 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://raw.githubusercontent.com/dominikwozniak/dw-sdlc/main/packages/dw-sdlc/config/agentic.config.schema.json",
   "title": "agentic.config",
-  "description": "How a repository works for an agent. `.ai/agentic.config.json` is committed; `.ai/agentic.config.local.json` overrides it on one machine and may tighten `guardrails`, `pr.merge` and `pr.review` but never loosen them. A key neither file sets is inferred: the Node package manager from the lockfile, the `conventional` commit preset, and `gate.command` from the manifest's `check` script. Unknown top-level keys are tolerated so other tool families can share the file; the keys named here refuse unknown children.",
+  "description": "How a repository works for an agent. `.ai/agentic.config.json` is committed; `.ai/agentic.config.local.json` overrides it on one machine and may tighten `guardrails`, `pr.merge` and `pr.review` but never loosen them. A key neither file sets is inferred: the Node package manager from the lockfile and `gate.command` from the manifest's `check` script. Unknown top-level keys are tolerated so other tool families can share the file; the keys named here refuse unknown children.",
   "type": "object",
   "properties": {
     "$schema": {
@@ -92,7 +92,7 @@
       "type": "object",
       "properties": {
         "convention": {
-          "description": "The subject preset `pattern` falls back to. `ticket-key`: `[KEY-123] type: subject`, the tracker's key in brackets before a type from a fixed list. `conventional`: Conventional Commits; inferred when no layer sets either.",
+          "description": "The subject preset `pattern` falls back to. `ticket-key`: `[KEY-123] type: subject`, the tracker's key in brackets before a type from a fixed list. `conventional`: Conventional Commits. Never inferred: with no layer setting this or `pattern`, the subject goes unchecked.",
           "enum": ["ticket-key", "conventional"]
         },
         "pattern": {

package/dist/agentic-config.d.ts

@@ -125,8 +125,8 @@ export type ResolvedConfig = {
     };
     guardrails: Guardrails;
     commit: {
-        convention: CommitConvention;
-        pattern: string;
+        convention?: CommitConvention;
+        pattern?: string;
         trailer?: string;
     };
     gate: Gate;
@@ -136,14 +136,6 @@ export type ResolvedConfig = {
         setup: string[];
     };
 };
-/** The layers merged, before `commit.pattern` falls back to the preset its convention names. */
-type Merged = Omit<ResolvedConfig, "commit"> & {
-    commit: {
-        convention: CommitConvention;
-        pattern?: string;
-        trailer?: string;
-    };
-};
 export type Problem = {
     file: string;
     path: string;
@@ -210,13 +202,12 @@ export declare function repositoryRoot(dir: string): string;
 export declare function mainCheckout(root: string): string | undefined;
 /**
  * The bottom layer: the package manager the lockfile implies, the Gate command the manifest's `check`
- * script implies, plus the defaults every key falls back to. `commit.pattern` is left to
- * `resolveConfig`, which fills it from the convention when no layer sets one.
+ * script implies, plus the defaults every key falls back to. No commit convention: only a file sets
+ * one, and `resolveConfig` fills `commit.pattern` from it when no layer sets a pattern.
  */
-export declare function inferConfig(root: string): Merged;
+export declare function inferConfig(root: string): ResolvedConfig;
 /**
  * Local over committed over inferred, for the repository at `root`. A linked worktree reads a layer it
  * lacks from its main checkout, where a repository that keeps `.ai/` out of git has the only copy.
  */
 export declare function resolveConfig(root: string): Resolution;
-export {};

package/dist/agentic-config.js

@@ -207,8 +207,8 @@ function inferGate(root) {
 }
 /**
  * The bottom layer: the package manager the lockfile implies, the Gate command the manifest's `check`
- * script implies, plus the defaults every key falls back to. `commit.pattern` is left to
- * `resolveConfig`, which fills it from the convention when no layer sets one.
+ * script implies, plus the defaults every key falls back to. No commit convention: only a file sets
+ * one, and `resolveConfig` fills `commit.pattern` from it when no layer sets a pattern.
  */
 export function inferConfig(root) {
     return {
@@ -216,7 +216,7 @@ export function inferConfig(root) {
         baseBranch: "auto",
         packageManagers: inferPackageManagers(root),
         guardrails: cloneGuardrails(DEFAULT_GUARDRAILS),
-        commit: { convention: "conventional" },
+        commit: {},
         gate: inferGate(root),
         pr: { ...DEFAULT_PR },
         worker: { ...DEFAULT_WORKER },
@@ -307,10 +307,16 @@ function prLoosenings(base, local) {
         return [{ file: LOCAL_FILE, path: `pr.${key}`, message }];
     });
 }
-/** `commit.pattern` from the convention's preset when no layer set one; a set pattern wins. */
+/**
+ * `commit.pattern` from the convention's preset when no layer set one; a set pattern wins. Neither
+ * set, no pattern: the subject goes unchecked.
+ */
 function finalize(merged) {
-    const pattern = merged.commit.pattern ?? COMMIT_PATTERNS[merged.commit.convention];
-    return { ...merged, commit: { ...merged.commit, pattern } };
+    const { convention, pattern } = merged.commit;
+    if (pattern !== undefined || convention === undefined) {
+        return merged;
+    }
+    return { ...merged, commit: { ...merged.commit, pattern: COMMIT_PATTERNS[convention] } };
 }
 /**
  * Local over committed over inferred, for the repository at `root`. A linked worktree reads a layer it

package/dist/commands/doctor.d.ts

@@ -19,7 +19,7 @@ export type Probe = {
 export type RepositoryConfig = {
     config: {
         commit: {
-            convention: string;
+            convention?: string;
         };
     };
     problems: string[];

package/dist/commands/doctor.js

@@ -223,7 +223,7 @@ function checkConfig(config, root) {
     return {
         name: "config",
         level: "PASS",
-        detail: `commit ${config.config.commit.convention}${borrowed.join("")}`
+        detail: `commit ${config.config.commit.convention ?? "unchecked"}${borrowed.join("")}`
     };
 }
 export function diagnose(inputs) {

package/dist/commands/setup.js

@@ -168,8 +168,8 @@ function localConfig(root, claudeDir, convention, guardrails, worker) {
     if (worker !== undefined) {
         config.worker = worker;
     }
-    // Only one asked for, by `--convention` or the file's pin: inference always gives `conventional`, and
-    // pinning it here would override a convention the repository commits later.
+    // Only one asked for, by `--convention` or the file's pin: pinning one here would override a
+    // convention the repository commits later.
     if (convention !== undefined) {
         config.commit = { convention };
     }

package/dist/doctor-plugins.js

@@ -146,7 +146,7 @@ export function checkSessions(probe, claude) {
     if (entry === undefined) {
         const detail = list === undefined
             ? "claude plugin list did not answer"
-            : `no ${SESSIONS_ID}, so no /sdlc-recap, /sdlc-brief or /sdlc-delegate. claude plugin install ${SESSIONS_ID} --scope user`;
+            : `no ${SESSIONS_ID}, so no /sdlc-recap, /sdlc-kickoff or /sdlc-delegate. claude plugin install ${SESSIONS_ID} --scope user`;
         return { name, level: "INFO", detail };
     }
     const version = typeof entry.version === "string" ? entry.version : "?";

package/dist/pr/decide.js

@@ -232,8 +232,10 @@ const ROWS = [
                 return { ...head, action: "mark-reviewed", actor: "script", details: after };
             }
             const { depth, source } = depthOf(pr, policy);
-            // The dev's own depth buys one Review past the limit; a Review under off needs none.
-            if (depth !== "off" && source !== "flag" && pr.marks.count >= REVIEW_LIMIT) {
+            // The dev's own depth buys one Review past the limit, and only one: the flag rides on every
+            // later call. A Review under off needs none.
+            const limit = source === "flag" ? REVIEW_LIMIT + 1 : REVIEW_LIMIT;
+            if (depth !== "off" && pr.marks.count >= limit) {
                 return reviewLimit(pr);
             }
             const byScript = depth === "off";
@@ -246,7 +248,8 @@ const ROWS = [
                     source,
                     // The agent may raise a routed depth to full, so auto names the command at any depth.
                     ...(depth === "full" || source === "auto" ? { reviewCommand: policy.reviewCommand } : {}),
-                    ...after
+                    ...after,
+                    ...(pr.scope === null ? {} : { paths: pr.scope })
                 }
             };
         }

package/dist/pr/read.js

@@ -205,16 +205,47 @@ function withCarried(pr) {
     return { ...pr, carried: sameChanges(ownChanges(pr, last.commit), ownChanges(pr, pr.head)) };
 }
 /**
- * `pr` and, when `sized` and a Review awaits a head that carries none, the files the Review would read:
- * the compare from the last commit a Review read, else from the base, to the head.
+ * The files whose own change differs between `from` and the head of `pr`, a rename under both names:
+ * what a fix touched, without the files a merge of the base brought in. Null when a compare cannot show
+ * the own changes whole.
+ */
+function ownScope(pr, from) {
+    const before = ownChanges(pr, from);
+    const after = ownChanges(pr, pr.head);
+    if (before === null || after === null) {
+        return null;
+    }
+    const moved = [
+        ...after.filter((change) => !before.includes(change)),
+        ...before.filter((change) => !after.includes(change))
+    ];
+    const names = moved.flatMap((change) => {
+        const [path, , previous] = JSON.parse(change);
+        return previous === null ? [path] : [path, previous];
+    });
+    return [...new Set(names)].toSorted();
+}
+/**
+ * `pr` and, when a Review awaits a head that carries none, its `scope` after a merge of the base, and
+ * when `sized`, the files the Review would read: the compare from the last commit a Review read, else
+ * from the base, to the head, kept to the scope when there is one.
  */
 function withChanged(pr, sized) {
-    if (!sized || pr.carried || pr.state !== "OPEN" || !awaitsReview(pr)) {
-        return { ...pr, changed: null };
+    if (pr.carried || pr.state !== "OPEN" || !awaitsReview(pr)) {
+        return { ...pr, changed: null, scope: null };
+    }
+    const scope = pr.lastRead?.merges === true ? ownScope(pr, pr.lastRead.commit) : null;
+    if (!sized) {
+        return { ...pr, changed: null, scope };
     }
     const from = pr.lastRead?.commit ?? pr.baseOid;
     const reply = compared(pr, from, pr.head, `the compare of ${from.slice(0, 7)}...${pr.head.slice(0, 7)}`);
-    return { ...pr, changed: parseOrCannotRun(() => changedOf(reply)) };
+    const changed = parseOrCannotRun(() => changedOf(reply));
+    if (scope === null || changed === "unsized") {
+        return { ...pr, changed, scope };
+    }
+    const within = changed.filter(({ path, previous }) => scope.includes(path) || (previous !== null && scope.includes(previous)));
+    return { ...pr, changed: within, scope };
 }
 /** The state of pull request `number` of `repo`, from one GraphQL request and the compares a Review needs. */
 function query({ repo, number }, sized) {

package/dist/pr/state.d.ts

@@ -88,6 +88,11 @@ export type PrState = {
      * auto` to route on: `unsized` when the compare cannot list them all, null when not read.
      */
     changed: ChangedFile[] | "unsized" | null;
+    /**
+     * The files whose own change moved since `lastRead`, when a merge of the base came after it: the files
+     * a fix touched, the base's own left out. Null with no such merge, or when a compare cannot show them.
+     */
+    scope: string[] | null;
 };
 /**
  * A file a Review would read: its name, its name before a rename, and the lines it adds and removes,
@@ -106,6 +111,7 @@ export type LastRead = {
     commit: string;
     clean: boolean;
     mergesOnly: boolean;
+    merges: boolean;
 };
 /**
  * The Reviews that left a Finding open since the last review success, a success under off and the dev's
@@ -144,5 +150,5 @@ export declare function parseView(reply: string): {
 /** The default branch of the repository whose `gh repo view --json defaultBranchRef` printed `reply`. */
 export declare function parseDefaultBranch(reply: string): string;
 /** The state of the pull request in `reply`, the text `gh api graphql` printed for QUERY on `repo`. */
-export declare function parsePr(reply: string, repo: string): Omit<PrState, "carried" | "changed">;
+export declare function parsePr(reply: string, repo: string): Omit<PrState, "carried" | "changed" | "scope">;
 export {};

package/dist/pr/state.js

@@ -253,7 +253,12 @@ function lastReadOf(history, head) {
     if (last === undefined) {
         return null;
     }
-    return { commit: last.oid, clean: last.read === "clean", mergesOnly: mergesOnlyAfter(index) };
+    return {
+        commit: last.oid,
+        clean: last.read === "clean",
+        mergesOnly: mergesOnlyAfter(index),
+        merges: commits.slice(index + 1).some(({ merge }) => merge)
+    };
 }
 /**
  * Whether a non-merge commit follows `commit` in the history, oldest first, as `lastReadOf` judges a read

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dw-sdlc",
-  "version": "0.19.0",
+  "version": "0.20.0",
   "description": "The Gate and the agentic config behind the sdlc Claude Code plugin: a bin with the Gate and its Checks, the setup, the doctor, the Sync of References, the Ship script that drives a pull request to merged, the Sweep, and Lead and Worker sessions in herdr; the resolver of a repository's .ai/agentic.config.json layers and its JSON Schema.",
   "license": "MIT",
   "repository": {