create-principles-disciple 1.144.9 → 1.144.11

package/console/dist/ui/utils/approval-warning-localization.js

@@ -49,19 +49,38 @@ export function localizeApprovalWarning(warning, t) {
     const base = { code, detail: warning.trim() };
     if (code === 'injection_budget_excluded') {
         const budget = intAfterPrefix(message, '(');
-        const earlierCount = intAfterPrefix(message, 'filled by ');
+        // The server reports "<injected>/<eligible> eligible injected"; the digit
+        // precedes the label, so parse the ratio directly instead of scanning after it.
+        const injectedMatch = /(\d+)\/(\d+) eligible/.exec(message);
+        const injectedCount = injectedMatch?.[1] !== undefined ? Number(injectedMatch[1]) : null;
         return {
             ...base,
             title: t('pages.focus.approveWarning.budgetExcludedTitle'),
             // PRI-875 precedent: a template with placeholders is only safe to use
             // when every value was parsed — otherwise fall back to the
             // placeholder-free variant so no raw "{{budget}}" ever reaches the UI.
-            body: budget !== null && earlierCount !== null
-                ? t('pages.focus.approveWarning.budgetExcludedBody', { budget, earlierCount })
+            body: budget !== null && injectedCount !== null
+                ? t('pages.focus.approveWarning.budgetExcludedBody', { budget, earlierCount: injectedCount })
                 : t('pages.focus.approveWarning.budgetExcludedBodySimple'),
             activationAction: true,
         };
     }
+    // PRI-935: the activation is merely rotated out of the CURRENT window, not
+    // starved. This is the common case on a live fair-rotation workspace and the
+    // copy must NOT tell the Owner to deactivate healthy older principles.
+    if (code === 'injection_budget_queued') {
+        const budget = intAfterPrefix(message, '(');
+        const turnsMatch = /within at most (\d+) consecutive user turns/.exec(message);
+        const turns = turnsMatch?.[1] !== undefined ? Number(turnsMatch[1]) : null;
+        return {
+            ...base,
+            title: t('pages.focus.approveWarning.queuedTitle'),
+            body: budget !== null && turns !== null
+                ? t('pages.focus.approveWarning.queuedBody', { budget, turns })
+                : t('pages.focus.approveWarning.queuedBodySimple'),
+            activationAction: false,
+        };
+    }
     if (code === 'injection_excluded_non_budget') {
         return {
             ...base,

package/console/dist/ui/utils/validators.d.ts

@@ -636,6 +636,12 @@ export interface ApprovalGroupData {
     principleTitle: string;
     /** Wave 7: human-readable principle text from artifact contentJson, if available */
     candidateDescription?: string;
+    /**
+     * PRI-935: whether this candidate's own serialized prompt entry fits the
+     * injection budget alone. Absent = unknown, which the badge treats as
+     * "do not promise rotation".
+     */
+    fitsPromptBudget?: boolean;
     status: string;
     records: ApprovalGroupRecordData[];
 }

package/console/dist/ui/utils/validators.js

@@ -1646,6 +1646,12 @@ function validateApprovalGroup(v) {
     if (Object.hasOwn(v, 'candidateDescription') && isString(v.candidateDescription) && v.candidateDescription.length > 0) {
         result.candidateDescription = v.candidateDescription;
     }
+    // PRI-935: optional fitsPromptBudget. Only a real boolean is propagated; a
+    // present-but-wrong value is dropped (rc-1/rc-2) so the badge falls back to
+    // copy that promises nothing rather than rendering an unverified claim.
+    if (Object.hasOwn(v, 'fitsPromptBudget') && typeof v.fitsPromptBudget === 'boolean') {
+        result.fitsPromptBudget = v.fitsPromptBudget;
+    }
     return result;
 }
 export function validatePromptInjectionBudgetStatus(v) {
@@ -1659,7 +1665,18 @@ export function validatePromptInjectionBudgetStatus(v) {
         typeof truncated !== 'boolean') {
         return null;
     }
-    return { budget, usedChars, truncated };
+    const result = { budget, usedChars, truncated };
+    // PRI-935: these are optional so a pre-PRI-935 server payload still
+    // validates (backward compatibility). A present-but-invalid value is
+    // DROPPED, never propagated (rc-1/rc-2): the UI then falls back to
+    // policy-agnostic copy rather than rendering an unverified policy claim.
+    if (Object.hasOwn(v, 'productionRotates') && typeof v.productionRotates === 'boolean') {
+        result.productionRotates = v.productionRotates;
+    }
+    if (Object.hasOwn(v, 'eligibleCount') && typeof v.eligibleCount === 'number' && Number.isInteger(v.eligibleCount) && v.eligibleCount >= 0) {
+        result.eligibleCount = v.eligibleCount;
+    }
+    return result;
 }
 export function validateApprovalsGrouped(v) {
     if (!isObject(v))

package/console/dist/web/assets/app.css

@@ -624,6 +624,9 @@
   .h-screen {
     height: 100vh;
   }
+  .max-h-40 {
+    max-height: calc(var(--spacing) * 40);
+  }
   .max-h-64 {
     max-height: calc(var(--spacing) * 64);
   }
@@ -2162,6 +2165,9 @@
   .opacity-72 {
     opacity: 72%;
   }
+  .opacity-80 {
+    opacity: 80%;
+  }
   .opacity-100 {
     opacity: 100%;
   }
@@ -2404,6 +2410,9 @@
     --tw-content: '';
     content: var(--tw-content);
   }
+  .last\:mb-0:last-child {
+    margin-bottom: 0px;
+  }
   .last\:border-r-0:last-child {
     border-right-style: var(--tw-border-style);
     border-right-width: 0px;