vouchington-tooling 0.46.0 → 0.47.0

package/dist/mcp-server/pr-body.d.mts

@@ -12,9 +12,9 @@ export type PullRequestProvenance = {
 };
 /**
  * The pull request body: the Summary, Impact, Related issues and Details sections followed by the
- * provenance lines. With `noSourceIssue` the Related issues section holds `noSourceStatement`
- * (default {@link defaultNoSourceStatement}) instead of `- None`. Pure; the caller supplies the
- * device and worktree facts.
+ * provenance lines. With `noSourceIssue` the Related issues section ends with `noSourceStatement`
+ * (default {@link defaultNoSourceStatement}), after any `Refs` lines, instead of `- None`. Pure;
+ * the caller supplies the device and worktree facts.
  */
 export declare function renderPullRequestBody(input: PullRequestInput, provenance: PullRequestProvenance, noSourceStatement?: string): string;
 /**

package/dist/mcp-server/pr-body.mjs

@@ -1,7 +1,9 @@
 import { basename } from 'node:path';
+import { formatReferenceKey } from "../gh-cli/closing-refs.mjs";
 function relatedLine(issue) {
     const verb = issue.relation === 'closes' ? 'Closes' : 'Refs';
-    return `- ${verb} #${issue.number} — ${issue.reason}`;
+    const [owner, repo] = issue.repository?.split('/') ?? [];
+    return `- ${verb} ${formatReferenceKey({ number: issue.number, owner, repo })} — ${issue.reason}`;
 }
 /** The default no-source statement; a policy `renderNoSourceIssue` export may replace it. */
 function defaultNoSourceStatement(reason) {
@@ -9,18 +11,16 @@ function defaultNoSourceStatement(reason) {
 }
 /**
  * The pull request body: the Summary, Impact, Related issues and Details sections followed by the
- * provenance lines. With `noSourceIssue` the Related issues section holds `noSourceStatement`
- * (default {@link defaultNoSourceStatement}) instead of `- None`. Pure; the caller supplies the
- * device and worktree facts.
+ * provenance lines. With `noSourceIssue` the Related issues section ends with `noSourceStatement`
+ * (default {@link defaultNoSourceStatement}), after any `Refs` lines, instead of `- None`. Pure;
+ * the caller supplies the device and worktree facts.
  */
 export function renderPullRequestBody(input, provenance, noSourceStatement) {
-    const related = input.relatedIssues.length
-        ? input.relatedIssues.map(relatedLine)
-        : [
-            input.noSourceIssue === undefined
-                ? '- None'
-                : (noSourceStatement ?? defaultNoSourceStatement(input.noSourceIssue.reason)),
-        ];
+    const related = input.relatedIssues.map(relatedLine);
+    if (input.noSourceIssue !== undefined)
+        related.push(noSourceStatement ?? defaultNoSourceStatement(input.noSourceIssue.reason));
+    if (!related.length)
+        related.push('- None');
     return [
         '## Summary',
         '',

package/dist/mcp-server/pr-input.d.mts

@@ -4,13 +4,15 @@ export type RelatedIssue = {
     number: number;
     relation: 'closes' | 'refs';
     reason: string;
+    /** `owner/name` of a cross-repository issue (rendered lowercase); absent for this repository. */
+    repository?: string;
 };
 export type PullRequestInput = {
     title: string;
     summary: string;
     impact: string;
     relatedIssues: RelatedIssue[];
-    /** Set instead of relatedIssues when the pull request has no source issue. */
+    /** Declares no source issue; relatedIssues beside it must all be `refs`. */
     noSourceIssue?: {
         reason: string;
     };

package/dist/mcp-server/pr-input.mjs

@@ -1,3 +1,4 @@
+import { hasClosingIssueReference } from "../gh-cli/closing-refs.mjs";
 import { optionalString, rejectUnknown, requireObject, requiredChoice, requiredString, requiredStringArray, } from "./args.mjs";
 import { trustedMode } from "./pr-mode.mjs";
 const PR_MODES = ['interactive', 'autonomous'];
@@ -9,17 +10,30 @@ export function positiveInteger(value, label) {
         throw new Error(`${label} must be a positive integer`);
     return value;
 }
+// The owner is a GitHub login: alphanumerics and single hyphens, at most 39 characters.
+const REPOSITORY_RE = /^(?=[^/]{1,39}\/)[A-Za-z0-9]+(?:-[A-Za-z0-9]+)*\/[\w.-]+$/;
+function readRepository(issue, label) {
+    if (issue.repository === undefined)
+        return undefined;
+    // Not lowercased here: redaction runs on this value first, and rendering lowercases it.
+    const value = singleLine(issue, 'repository');
+    if (!REPOSITORY_RE.test(value))
+        throw new Error(`${label} must be a single owner/name`);
+    return value;
+}
 function readRelatedIssues(args) {
     const value = args.relatedIssues;
     if (!Array.isArray(value))
         throw new Error('relatedIssues is required and must be an array');
     return value.map((item, index) => {
         const issue = requireObject(item);
-        rejectUnknown(issue, ['number', 'relation', 'reason']);
+        rejectUnknown(issue, ['number', 'relation', 'reason', 'repository']);
+        const repository = readRepository(issue, `relatedIssues[${index}].repository`);
         return {
             number: positiveInteger(issue.number, `relatedIssues[${index}].number`),
             relation: requiredChoice(issue, 'relation', RELATIONS),
             reason: requiredString(issue, 'reason'),
+            ...(repository === undefined ? {} : { repository }),
         };
     });
 }
@@ -28,8 +42,12 @@ function readNoSourceIssue(args, related) {
         return undefined;
     const value = requireObject(args.noSourceIssue);
     rejectUnknown(value, ['reason']);
-    if (related.length)
-        throw new Error('noSourceIssue and a non-empty relatedIssues are mutually exclusive');
+    if (related.some((issue) => issue.relation === 'closes'))
+        throw new Error('noSourceIssue cannot be combined with a relatedIssues entry that closes');
+    related.forEach((issue, index) => {
+        if (hasClosingIssueReference(issue.reason))
+            throw new Error(`relatedIssues[${index}].reason must not contain a closing reference beside noSourceIssue`);
+    });
     return { reason: singleLine(value, 'reason') };
 }
 function readDetails(args) {

package/dist/mcp-server/tools-pr.mjs

@@ -12,14 +12,20 @@ const FIELDS = {
     impact: text('Who or what is affected, as markdown.'),
     relatedIssues: {
         type: 'array',
-        description: 'Issues this pull request closes or refs. Empty when it has none; with no source issue ' +
-            'at all, give noSourceIssue as well (it is rejected beside a non-empty list).',
+        description: 'Issues this pull request closes or refs. Empty when it has none; with no source issue, ' +
+            'give noSourceIssue as well (beside a non-empty list every entry must be refs).',
         items: {
             type: 'object',
             properties: {
                 number: { type: 'integer', minimum: 1 },
                 relation: { enum: ['closes', 'refs'] },
                 reason: text('Why the issue is related.'),
+                repository: {
+                    type: 'string',
+                    pattern: '^(?=[^/]{1,39}/)[A-Za-z0-9]+(?:-[A-Za-z0-9]+)*/[\\w.-]+$',
+                    description: 'Optional owner/name (lowercased) of the issue repository, rendered as ' +
+                        'owner/name#N; omit for an issue in this repository. Allowed for closes and refs.',
+                },
             },
             required: ['number', 'relation', 'reason'],
             additionalProperties: false,
@@ -27,9 +33,10 @@ const FIELDS = {
     },
     noSourceIssue: {
         type: 'object',
-        description: 'Declares a pull request with no source issue, mutually exclusive with a non-empty ' +
-            'relatedIssues. Rendered inside Related issues (the policy may supply the wording) and ' +
-            'passed to the policy, which may allow it only in interactive mode.',
+        description: 'Declares a pull request with no source issue; rejected beside a closes relatedIssues ' +
+            'entry, allowed beside refs-only ones. Rendered inside Related issues after any Refs ' +
+            'lines (the policy may supply the wording) and passed to the policy, which may allow it ' +
+            'only in interactive mode.',
         properties: { reason: text('Single-line reason there is no source issue.') },
         required: ['reason'],
         additionalProperties: false,

package/docs/mcp-server.md

@@ -286,8 +286,10 @@ shape is an error, never a pass.
 retrospective tools. All three take `sessionId` (required) and `worktree` (optional), need a server
 launched inside a repository worktree (they run the checkout policy), and share this structured
 input: `title` (single line, at most 256 characters), `summary`, `impact`, `relatedIssues`
-(`[{ number, relation: "closes" | "refs", reason }]`, may be empty), optional `noSourceIssue`
-(`{ reason }`, see below), `details` (string array), `base` (default `main`), optional `agent`, optional
+(`[{ number, relation: "closes" | "refs", reason, repository? }]`, may be empty; the optional
+`repository` is a single `owner/name` and renders the line as lowercase `owner/name#N`), optional
+`noSourceIssue` (`{ reason }`, see below), `details` (string array), `base` (default `main`),
+optional `agent`, optional
 `model`, optional `mode` (`interactive` | `autonomous`) and `workspaceSetup`. `pr_update` also
 requires `pr`; `pr_validate` accepts an optional `pr`.
 
@@ -319,9 +321,11 @@ which may keep sending it.
 
 ### Pull requests with no source issue
 
-Pass `noSourceIssue: { reason }` (a single line) with an empty `relatedIssues`; a non-empty
-`relatedIssues` beside it is rejected. The Related issues section then holds the generic statement
-`- No source issue — <reason>` instead of `- None`. A policy may replace the statement by
+Pass `noSourceIssue: { reason }` (a single line) with an empty `relatedIssues`, or with a
+`relatedIssues` whose every entry is `refs`; a `closes` entry beside it is rejected. The Related
+issues section then holds the generic statement `- No source issue — <reason>` instead of `- None`,
+preceded by the `- Refs … — <reason>` lines when there are any (the statement is always the last
+line). A policy may replace the statement by
 exporting `renderNoSourceIssue({ reason })`, returning a non-empty string (30 s). The field also
 reaches `validatePullRequest` (with the trusted `mode`, so the policy can apply an interactive-only
 rule) and `pullRequestDefaults`.
@@ -339,8 +343,9 @@ rule) and `pullRequestDefaults`.
 
 ## Related issues
 
-- Closes #N — <reason>      (or "Refs #N"; "- None" when empty;
-                             "- No source issue — <reason>" for noSourceIssue)
+- Closes #N — <reason>      (or "Refs #N"; owner/name#N with `repository`; "- None" when empty;
+                             "- No source issue — <reason>" for noSourceIssue, rendered
+                             after any Refs lines)
 
 ## Details
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vouchington-tooling",
-  "version": "0.46.0",
+  "version": "0.47.0",
   "description": "Vouchington CLI and extractable tooling libraries.",
   "homepage": "https://github.com/vouchington/vouchington-tooling/tree/main/packages/vouchington-tooling#readme",
   "bugs": {