@bridge-ds/core 0.12.0 → 0.13.0

package/dist/review-gate.d.ts

@@ -63,11 +63,18 @@ export interface ApprovalSummary {
     design: boolean;
     tech: boolean;
 }
+export interface ApprovalRules {
+    /**
+     * `governance.soloOwnerCanApproveBoth`: when the owners resolve to one person holding both roles, that person's
+     * approval counts for design and tech. Default false. The author still never approves their own change.
+     */
+    soloOwnerCanApproveBoth?: boolean | undefined;
+}
 /** Latest decisive review per person (comments do not reset an approval), then owner coverage. */
 export declare function summarizeApprovals(reviews: ReviewLike[], owners: {
     design: string[];
     tech: string[];
-}, author?: string): ApprovalSummary;
+}, author?: string, rules?: ApprovalRules): ApprovalSummary;
 export type MissingApproval = 'design approval' | 'tech approval' | 'approval';
 export interface MergeGate {
     allowed: boolean;

package/dist/rights.d.ts

@@ -27,6 +27,26 @@ export declare function holdersOf(right: Right, owners: Owners, rights: Rights |
 export declare function approverLists(owners: Owners, rights: Rights | null | undefined, teams?: TeamStates | null): Owners;
 /** approverLists before team expansion: the entries as the config names them, for sentences. */
 export declare function approverEntries(owners: Owners, rights: Rights | null | undefined): Owners;
+/**
+ * The one person who holds both approver places, or null: the design and tech lists (after rights, teams expanded)
+ * name exactly one login between them, and it sits in both. A team, read or not, never counts as one person.
+ */
+export declare function soloApprover(owners: {
+    design: string[];
+    tech: string[];
+}): string | null;
+/**
+ * The only owner whose approval covers design and tech, or null when the rule is off or does not apply:
+ * `governance.soloOwnerCanApproveBoth` is true and the approver lists (after rights, teams expanded) name one person
+ * in both places. Studio Settings, the review screens, the MCP server and the CLI word it from this.
+ */
+export declare function soloOwnerOf(gov: {
+    owners: Owners;
+    rights?: Rights | null | undefined;
+    soloOwnerCanApproveBoth?: boolean | undefined;
+}, teams?: TeamStates | null): string | null;
+/** The rule in one sentence, for Settings, the CLI and the MCP server. `you`: the viewer is that owner. */
+export declare function soloOwnerSentence(login: string, you?: boolean): string;
 /** Who may ship, or null when nobody is singled out (no `rights`, or its groups list no logins): anyone with write access. */
 export interface ShipPolicy {
     /** Logins allowed to ship: people, and team members once read. A team not read keeps its "@org/slug" key. */

package/dist/{suggestion-apply-DxXRGQrO.js → suggestion-apply-bH6-gZrE.js}

@@ -1,4 +1,4 @@
-import { cg as tokenBump, aQ as changeKind, L as LayeredTokenSet, aH as aliasOf, bZ as rgbaToDtcg, bn as hexToRgba, w as parseTokenFile, q as emptyTokenFile, s as serializeTokenFile, C as moveToken, r as removeToken, bq as inheritedTypeAt, u as upsertToken, H as readTokenAt, aW as comboKey, bY as renderedColor, b2 as contrastRatio, z as rgbaToHex, U as classifyRisk, W as RISK_LABELS, V as CONFIG_FILE, N as loadConfig, ak as ConfigError, _ as DEFAULT_TOKEN_PATHS, bl as globToRegExp, ae as slugify, Q as validateLayered, bI as parseJson, c1 as serializeJson, bN as plainEqual, cf as toPlain, bk as getEntry, bi as fromPlain, m as modeDisplayName, A as formatValue, X as previewLinks, a1 as readPreviewComment, aP as canShip, cn as whoCanShipSentence, ci as unreadableShipTeams, aL as approverLists, O as tokensPath, a6 as NEEDED_BY_LABEL, K as selectTokenFiles, R as parseContrastChecks } from "./prod-branch-BdMPKffE.js";
+import { cj as tokenBump, aQ as changeKind, L as LayeredTokenSet, aH as aliasOf, bZ as rgbaToDtcg, bn as hexToRgba, w as parseTokenFile, q as emptyTokenFile, s as serializeTokenFile, C as moveToken, r as removeToken, bq as inheritedTypeAt, u as upsertToken, H as readTokenAt, aW as comboKey, bY as renderedColor, b2 as contrastRatio, z as rgbaToHex, U as classifyRisk, W as RISK_LABELS, V as CONFIG_FILE, N as loadConfig, ak as ConfigError, _ as DEFAULT_TOKEN_PATHS, bl as globToRegExp, ae as slugify, Q as validateLayered, bI as parseJson, c1 as serializeJson, bN as plainEqual, ci as toPlain, bk as getEntry, bi as fromPlain, m as modeDisplayName, A as formatValue, X as previewLinks, a1 as readPreviewComment, aP as canShip, cq as whoCanShipSentence, cl as unreadableShipTeams, c4 as soloApprover, aL as approverLists, O as tokensPath, a6 as NEEDED_BY_LABEL, K as selectTokenFiles, R as parseContrastChecks } from "./prod-branch-6ftQWwdu.js";
 function changesetBump(changes) {
   return tokenBump(changes) ?? "patch";
 }
@@ -2804,7 +2804,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) {
@@ -2822,7 +2822,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;
@@ -3088,7 +3089,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 summary = summarizeApprovals(reviews, lists, opts.author ?? void 0);
+  const summary = summarizeApprovals(reviews, lists, opts.author ?? void 0, { soloOwnerCanApproveBoth: gov.soloOwnerCanApproveBoth });
   const stale = [...latestOld.keys()].filter((l) => !summary.approvers.includes(l) && !summary.changesRequested.includes(l));
   return { ...summary, stale, ignored: [...ignored].filter((l) => !summary.approvers.includes(l)) };
 }
@@ -4425,7 +4426,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);
@@ -4817,4 +4818,4 @@ export {
   explorationNote as y,
   explorationNotePath as z
 };
-//# sourceMappingURL=suggestion-apply-DxXRGQrO.js.map
+//# sourceMappingURL=suggestion-apply-bH6-gZrE.js.map