@bridge-ds/mcp 0.7.2 → 0.8.0

package/dist/branches.d.ts

@@ -60,6 +60,7 @@ export declare function branchFacts(ctx: BridgeContext, gh: GitHubClient, n: num
             push?: boolean;
         };
     } | null;
+    solo: string | null;
 }>;
 /** One branch, the way Studio shows it: status, what blocks the merge and who acts, how it stands against main. */
 export declare function branchStatus(ctx: BridgeContext, a: {
@@ -80,6 +81,7 @@ export declare function branchStatus(ctx: BridgeContext, a: {
     };
     approvalNeeded: string;
     approvals: {
+        soloOwner?: string | undefined;
         approvedBy: string[];
         changesRequestedBy: string[];
         design: boolean;

package/dist/{chunk-BZ3KXXMM.js → chunk-BHODFN52.js}

@@ -424,6 +424,19 @@ function approverEntries(owners, rights) {
   if (!rights) return { design: [...owners.design], tech: [...owners.tech] };
   return { design: holdersOf("approveDesign", owners, rights), tech: holdersOf("approveTech", owners, rights) };
 }
+function soloApprover(owners) {
+  const d = [...new Set(owners.design.map(norm2))];
+  const t = [...new Set(owners.tech.map(norm2))];
+  if (d.length !== 1 || t.length !== 1 || d[0] !== t[0] || !d[0] || d[0].includes("/")) return null;
+  return d[0];
+}
+function soloOwnerOf(gov, teams) {
+  if (!gov.soloOwnerCanApproveBoth) return null;
+  return soloApprover(approverLists(gov.owners, gov.rights ?? null, teams));
+}
+function soloOwnerSentence(login, you = false) {
+  return you ? "As the only owner, your approval covers design and code." : `As the only owner, @${norm2(login)} approves design and code in one review.`;
+}
 function shipPolicy(owners, rights, teams) {
   if (!rights) return null;
   const entries = holdersOf("ship", owners, rights);
@@ -700,7 +713,8 @@ function defineConfig(input) {
           minReasonLength: g.override.minReasonLength ?? DEFAULT_OVERRIDE.minReasonLength
         }
       } : {},
-      ...g.rights ? { rights: normalizeRights(g.rights) } : {}
+      ...g.rights ? { rights: normalizeRights(g.rights) } : {},
+      ...g.soloOwnerCanApproveBoth === true ? { soloOwnerCanApproveBoth: true } : {}
     },
     products: input.products?.length ? input.products.map((p) => ({ ...p })) : DEFAULT_PRODUCTS.map((p) => ({ ...p })),
     changes: input.changes === "direct" ? "direct" : DEFAULT_CHANGES_MODE,
@@ -824,6 +838,7 @@ function loadConfig(source) {
         }
       }
       problems.push(...rightsProblems(gov.rights));
+      if (gov.soloOwnerCanApproveBoth !== void 0 && typeof gov.soloOwnerCanApproveBoth !== "boolean") problems.push('"governance.soloOwnerCanApproveBoth" must be true or false');
     }
   }
   problems.push(...visualImpactProblems(d.visualImpact));
@@ -2010,6 +2025,9 @@ export {
   DEFAULT_RIGHTS,
   approverLists,
   approverEntries,
+  soloApprover,
+  soloOwnerOf,
+  soloOwnerSentence,
   shipPolicy,
   unreadableShipTeams,
   canShip,

package/dist/{chunk-FAB4HMPJ.js → chunk-VV5HHLLG.js}

@@ -68,6 +68,9 @@ import {
   selectTokenFiles,
   shipPolicy,
   showFile,
+  soloApprover,
+  soloOwnerOf,
+  soloOwnerSentence,
   threeWayDiff,
   toConfigJson,
   tokensPath,
@@ -76,7 +79,7 @@ import {
   validateLayered,
   variantKeyOf,
   whoCanShipSentence
-} from "./chunk-BZ3KXXMM.js";
+} from "./chunk-BHODFN52.js";
 
 // src/lib.ts
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
@@ -4246,7 +4249,7 @@ function summarizeCi(runs, statuses = []) {
   const state = total === 0 ? "none" : failed.length ? "fail" : pending.length ? "pending" : "pass";
   return { state, total, passed, failed, pending };
 }
-function summarizeApprovals(reviews, owners, author) {
+function summarizeApprovals(reviews, owners, author, rules = {}) {
   const latest = /* @__PURE__ */ new Map();
   const sorted = [...reviews].sort((a2, b) => String(a2.submitted_at ?? "").localeCompare(String(b.submitted_at ?? "")));
   for (const r of sorted) {
@@ -4264,7 +4267,8 @@ function summarizeApprovals(reviews, owners, author) {
   let tech;
   if (d.length || t.length) {
     design = d.length ? a.some((x) => d.includes(x)) : a.length > 0;
-    const soloBoth = (x) => a.length === 1 && d.includes(x);
+    const solo = rules.soloOwnerCanApproveBoth ? soloApprover(owners) : null;
+    const soloBoth = (x) => a.length === 1 && d.includes(x) && x !== solo;
     tech = t.length ? a.some((x) => t.includes(x) && !soloBoth(x)) : a.length > 1;
   } else {
     design = a.length >= 1;
@@ -4735,7 +4739,7 @@ function summarizeSuggestionApprovals(comments, gov, opts) {
     latestOld.delete(c.author);
     reviews.push({ user: { login: c.author }, state: r.event === "approve" ? "APPROVED" : "CHANGES_REQUESTED", submitted_at: at });
   });
-  const summary2 = summarizeApprovals(reviews, lists, opts.author ?? void 0);
+  const summary2 = summarizeApprovals(reviews, lists, opts.author ?? void 0, { soloOwnerCanApproveBoth: gov.soloOwnerCanApproveBoth });
   const stale = [...latestOld.keys()].filter((l) => !summary2.approvers.includes(l) && !summary2.changesRequested.includes(l));
   return { ...summary2, stale, ignored: [...ignored].filter((l) => !summary2.approvers.includes(l)) };
 }
@@ -6829,7 +6833,7 @@ async function planSuggestionApply(input) {
     for (const p of new Set(targets)) if (await ws.isSymlink(p)) links.push(`"${p}" is a symbolic link (or inside one): Bridge never writes through links.`);
     if (links.length) return out("refused", "Not applied: the suggestion writes through a symbolic link.", { problems: links });
   }
-  const governance = input.governance ?? { owners: config.governance.owners, rights: config.governance.rights ?? null };
+  const governance = input.governance ?? { owners: config.governance.owners, rights: config.governance.rights ?? null, soloOwnerCanApproveBoth: config.governance.soloOwnerCanApproveBoth };
   const approvals = summarizeSuggestionApprovals(input.comments, governance, { payloadSha: parsed.sha, author: input.author });
   const review = suggestionReviewState(payload.tier.tier, approvals);
   const missing = pendingApprovals(payload.tier.tier, approvals);
@@ -10200,7 +10204,7 @@ async function people(gh, config) {
   const { owners, rights } = config.governance;
   const teams = await new TeamDirectory(gh).resolve(owners).catch(() => void 0);
   const lists = expandOwners(approverEntries(owners, rights), teams);
-  return { lists, ship: shipPolicy(owners, rights, teams) };
+  return { lists, ship: shipPolicy(owners, rights, teams), solo: config.governance.soloOwnerCanApproveBoth ? soloApprover(lists) : null };
 }
 async function viewerOf(ctx, gh) {
   if (!ctx.canWriteRemote) return null;
@@ -10226,7 +10230,8 @@ async function branchFacts(ctx, gh, n, viewer) {
   const approvals = summarizeApprovals(
     reviews.map((r) => ({ user: r.user ? { login: r.user } : null, state: r.state, submitted_at: r.submittedAt ?? null })),
     who.lists,
-    pr.author
+    pr.author,
+    { soloOwnerCanApproveBoth: Boolean(who.solo) }
   );
   const tier = effectiveTier(labels, pr.body, files);
   const merged = pr.state === "merged";
@@ -10236,7 +10241,7 @@ async function branchFacts(ctx, gh, n, viewer) {
   const status = reviewStatusKind({ ...pullLike, ci, approvals, tier, mergeable: pr.mergeable });
   const fresh = freshness({ compare: compare ? { behind_by: compare.behindBy, merge_base_commit: compare.mergeBase ? { sha: compare.mergeBase } : null } : null, mergeable: pr.mergeable, open: pr.state === "open" });
   const product = productOfBranch(productsOf(config), { files: files.map((f) => f.filename), labels: pr.labels });
-  return { config, pr, files, ci, approvals, tier, gate, status, fresh, compare, exploration, product, lifecycle: pullState(pullLike), write: repo ? hasWriteAccess(repo.permissions) : null, repo };
+  return { config, pr, files, ci, approvals, tier, gate, status, fresh, compare, exploration, product, lifecycle: pullState(pullLike), write: repo ? hasWriteAccess(repo.permissions) : null, repo, solo: who.solo };
 }
 var STATUS_WORDS = {
   merged: "Merged",
@@ -10267,7 +10272,7 @@ async function branchStatus(ctx, a) {
     lifecycle: f.lifecycle,
     status: { kind: f.status, label: STATUS_WORDS[f.status] },
     approvalNeeded: f.tier === 1 ? "One approval" : "Design and tech approval",
-    approvals: { approvedBy: f.approvals.approvers, changesRequestedBy: f.approvals.changesRequested, design: f.approvals.design, tech: f.approvals.tech },
+    approvals: { approvedBy: f.approvals.approvers, changesRequestedBy: f.approvals.changesRequested, design: f.approvals.design, tech: f.approvals.tech, ...f.solo && (f.tier ?? 3) > 1 ? { soloOwner: soloOwnerSentence(f.solo, Boolean(viewer && viewer.toLowerCase() === f.solo)) } : {} },
     tests: { state: f.ci.state, failing: f.ci.failed.map((x) => x.name), running: f.ci.pending.map((x) => x.name) },
     main: { state: f.fresh.state, behindBy: f.fresh.behindBy },
     merge: {
@@ -10878,7 +10883,7 @@ async function suggestionGovernance(ctx, gh) {
   const { owners, rights } = config.governance;
   const hasTeams = [...owners.design, ...owners.tech].some((o) => o.includes("/"));
   const teams = hasTeams ? await new TeamDirectory(gh).resolve(owners).catch(() => void 0) : void 0;
-  return { config, gov: { owners, rights: rights ?? null, ...teams ? { teams } : {} } };
+  return { config, gov: { owners, rights: rights ?? null, ...teams ? { teams } : {}, ...config.governance.soloOwnerCanApproveBoth ? { soloOwnerCanApproveBoth: true } : {} } };
 }
 var OPEN_STATUSES = ["needs-review", "changes-requested", "approved", "applying", "outdated"];
 async function listSuggestionsOp(ctx, a = {}) {
@@ -10925,6 +10930,7 @@ async function getSuggestionOp(ctx, a) {
   const p = d.parsed.payload;
   const approvals = d.approvals;
   const missing = pendingApprovals(p.tier.tier, approvals);
+  const solo = soloOwnerOf(gov, gov.teams);
   return {
     ...base,
     kind: p.kind,
@@ -10952,7 +10958,8 @@ async function getSuggestionOp(ctx, a) {
       missing: missing.map(roleText),
       changesRequestedBy: approvals.changesRequested,
       ...approvals.stale.length ? { staleReviews: approvals.stale } : {},
-      ...approvals.ignored.length ? { notCounted: approvals.ignored } : {}
+      ...approvals.ignored.length ? { notCounted: approvals.ignored } : {},
+      ...solo && p.tier.tier > 1 ? { soloOwner: soloOwnerSentence(solo) } : {}
     },
     applied: d.applied,
     ...p.componentSpec ? { brief: p.componentSpec.brief } : {},
@@ -11701,9 +11708,11 @@ async function reviewRoles(gh, config, approvers) {
   });
   const ship = shipPolicy(owners, rights, teams);
   const unreadable = unreadableTeams(teams);
+  const solo = soloOwnerOf(config.governance, teams);
   return {
     design: role("design"),
     tech: role("tech"),
+    ...solo ? { soloOwner: soloOwnerSentence(solo) } : {},
     ship: ship ? whoCanShipSentence(ship) : "Anyone with write access can ship.",
     ...unreadable.length ? { unreadableTeams: unreadable.map((t) => t.message) } : {}
   };
@@ -11745,7 +11754,7 @@ async function visualImpact(ctx, a, run2) {
   if (ctx.mode !== "local" || !ctx.root) throw new Error("Visual impact needs a local checkout (start Bridge with --root): it builds Storybook and the app at both refs.");
   if (!ctx.isGit) throw new Error(`${ctx.root} is not a git repository.`);
   const out = a.out ? resolve3(ctx.root, a.out) : join5(ctx.root, "visual-impact");
-  const runImpact = run2 ?? (await import("./src-D6YUOJC7.js")).runImpact;
+  const runImpact = run2 ?? (await import("./src-P3GHW467.js")).runImpact;
   const r = await runImpact({
     root: ctx.root,
     base: a.base ?? "main",
@@ -12525,6 +12534,7 @@ async function getSettings(ctx) {
   const protection = gh ? await loadProtection(gh, baseBranchOf(config)).catch(() => null) : null;
   const products = productsOf(config);
   const unreadable = teams ? unreadableTeams(teams) : [];
+  const solo = soloOwnerOf(config.governance, teams);
   return {
     repo: config.repo ? `${config.repo.owner}/${config.repo.name}` : gh?.fullName ?? null,
     main: baseBranchOf(config),
@@ -12532,6 +12542,7 @@ async function getSettings(ctx) {
       designApproval: entries.design.map((e) => entryLabel(e, teams)),
       techApproval: entries.tech.map((e) => entryLabel(e, teams)),
       ship: ship ? whoCanShipSentence(ship) : "Anyone with write access can merge.",
+      ...solo ? { soloOwner: soloOwnerSentence(solo) } : {},
       ...unreadable.length ? { teamsNotRead: unreadable.map((t) => t.message) } : {}
     },
     ownerOverride: override.allowed.length ? { on: true, who: override.allowed.map((l) => `@${l}`), reason: override.requireReason ? `A reason of at least ${override.minReasonLength} characters, posted on the branch.` : "No reason required." } : { on: false, who: [], reason: "Nobody can merge without the approvals." },
@@ -12811,7 +12822,7 @@ async function pushFigmaComponentOp(ctx, a) {
 }
 
 // src/server.ts
-var VERSION = true ? "0.7.2" : "dev";
+var VERSION = true ? "0.8.0" : "dev";
 var ok = (data) => ({ content: [{ type: "text", text: typeof data === "string" ? data : JSON.stringify(data, null, 2) }] });
 var fail2 = (e) => ({ content: [{ type: "text", text: `Error: ${e.message ?? String(e)}` }], isError: true });
 var run = (fn) => async (a) => {

package/dist/index.js

@@ -3,8 +3,8 @@ import {
   VERSION,
   optionsFromArgv,
   serveStdio
-} from "./chunk-FAB4HMPJ.js";
-import "./chunk-BZ3KXXMM.js";
+} from "./chunk-VV5HHLLG.js";
+import "./chunk-BHODFN52.js";
 
 // src/index.ts
 var argv = process.argv.slice(2);

package/dist/lib.js

@@ -28,12 +28,12 @@ import {
   serveStdio,
   studio_exports,
   suggestions_exports
-} from "./chunk-FAB4HMPJ.js";
+} from "./chunk-VV5HHLLG.js";
 import {
   CONFIG_FILE,
   loadConfig,
   toConfigJson
-} from "./chunk-BZ3KXXMM.js";
+} from "./chunk-BHODFN52.js";
 export {
   BridgeContext,
   CONFIG_FILE,

package/dist/ops.d.ts

@@ -736,6 +736,8 @@ export declare function getReview(ctx: BridgeContext, a: {
  */
 export declare function reviewRoles(gh: GitHubClient, config: BridgeConfig, approvers: string[]): Promise<{
     unreadableTeams?: string[] | undefined;
+    ship: string;
+    soloOwner?: string | undefined;
     design: {
         owners: string[];
         approvedBy: string[];
@@ -744,7 +746,6 @@ export declare function reviewRoles(gh: GitHubClient, config: BridgeConfig, appr
         owners: string[];
         approvedBy: string[];
     };
-    ship: string;
 }>;
 export declare function reviewPr(ctx: BridgeContext, a: {
     pr: number;

package/dist/{src-D6YUOJC7.js → src-P3GHW467.js}

@@ -11,7 +11,7 @@ import {
   referenceImagePath,
   selectTokenFiles,
   showFile
-} from "./chunk-BZ3KXXMM.js";
+} from "./chunk-BHODFN52.js";
 
 // ../visual/src/run.ts
 import { existsSync as existsSync3 } from "node:fs";

package/dist/studio.d.ts

@@ -94,6 +94,7 @@ export declare function getSettings(ctx: BridgeContext): Promise<{
     main: string;
     roles: {
         teamsNotRead?: string[] | undefined;
+        soloOwner?: string | undefined;
         designApproval: string[];
         techApproval: string[];
         ship: string;

package/dist/suggestions.d.ts

@@ -137,6 +137,7 @@ export declare function getSuggestionOp(ctx: BridgeContext, a: {
     next: string;
     brief?: string | undefined;
     approvals: {
+        soloOwner?: string | undefined;
         notCounted?: string[] | undefined;
         staleReviews?: string[] | undefined;
         approvedBy: string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bridge-ds/mcp",
-  "version": "0.7.2",
+  "version": "0.8.0",
   "type": "module",
   "description": "Bridge MCP server: lets Claude (Claude Code, Claude Desktop or any MCP client) read, check and propose changes to a DTCG design system repo through pull requests.",
   "license": "MIT",
@@ -54,8 +54,8 @@
     "build:package": "node ../../scripts/build-packages.mjs mcp"
   },
   "dependencies": {
-    "@bridge-ds/core": "^0.12.0",
-    "@bridge-ds/visual": "^0.3.6",
+    "@bridge-ds/core": "^0.13.0",
+    "@bridge-ds/visual": "^0.3.7",
     "@modelcontextprotocol/sdk": "1.31.0",
     "typescript": "5.9.3",
     "zod": "3.25.76"