@evandu/super-pi 0.1.166 → 0.1.168

package/dist/shared/agent-order.js

@@ -0,0 +1,50 @@
+/**
+ * The ONE order agent lists are shown in.
+ *
+ * WHY THERE IS A SINGLE RULE AND NOT A RULE PER SIDE. The picker's list, the settings page's grid, the
+ * sub-agent roster and the project panel's employee candidates are four views of the same catalogue, and
+ * they used to disagree about what a list of agents looks like. The server sorted what it read by
+ * "(built-in first, then id)", and the client re-sorted what the server sent by a rule of its own that
+ * added one more level - "roles before workers". A reader who saw the same agents in two orders had no
+ * way to tell which screen was wrong, because neither was: they were two answers to a question nobody
+ * had asked twice the same way.
+ *
+ * So the rule is stated once, HERE, and both sides apply it. The server imports it to order the
+ * catalogue it reads; the client imports it to order every list it draws. There is no second
+ * implementation to drift, because there is no second implementation.
+ *
+ * THE ORDER IS:
+ *
+ *   1. THE PACKAGE'S OWN agents first, then the user's. The ones that ship with Super Pi are drawn with
+ *      a colour dot, and a dot scattered among the user's own reads as noise rather than as a group.
+ *   2. Inside a group, the CONVERSATION ROLES (mode: primary) before the DELEGATED ones (mode:
+ *      subagent), so a list reads as "the roles, then the workers".
+ *   3. Inside a group and a role, alphabetical by id, so the order never depends on the order the
+ *      filesystem happened to hand the files over in.
+ *
+ * WHY THE FLAG IS NOT GUESSED HERE. "Is this the package's own agent?" is a question about where the app
+ * is installed, and only the server knows that: it reads the definitions and marks them. A client that
+ * tried to work it out from the directory path would be guessing, and would guess wrong the moment the
+ * package is installed somewhere unusual - which is exactly the case where the answer matters.
+ */
+/** The conversation roles come before the delegated ones, so a screen keeps reading as "roles, then workers". */
+const modeRank = (mode) => (mode === "subagent" ? 1 : 0);
+/**
+ * The built-in group first, then the user's, each group by role and then by id.
+ *
+ * An agent with no `builtin` flag at all (an older server, or a hand-built fixture) counts as the
+ * user's, which is the safe direction: it puts the unflagged one where a user's own would sit rather
+ * than promoting it into the group that means "this ships with Super Pi".
+ */
+export function orderAgentsForDisplay(agents) {
+    return [...agents].sort((a, b) => {
+        const left = a.builtin ? 0 : 1;
+        const right = b.builtin ? 0 : 1;
+        if (left !== right)
+            return left - right;
+        const roles = modeRank(a.mode) - modeRank(b.mode);
+        if (roles !== 0)
+            return roles;
+        return a.id.localeCompare(b.id);
+    });
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evandu/super-pi",
-  "version": "0.1.166",
+  "version": "0.1.168",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -48,6 +48,7 @@
     "scripts/stream-transport.test.mjs",
     "scripts/remote-transport.test.mjs",
     "scripts/browser-safe-port.test.mjs",
+    "scripts/agent-order.test.mjs",
     "scripts/agent-picker-order.test.mjs",
     "scripts/inline-thinking.test.mjs",
     "scripts/inline-thinking-close-tag.test.mjs",
@@ -195,6 +196,7 @@
     "scripts/session-state-icon.test.mjs",
     "scripts/provider-fetch-models.test.mjs",
     "scripts/shared-chrome.test.mjs",
+    "scripts/shared-chrome-recovery.test.mjs",
     "scripts/prompt-templates.test.mjs",
     "scripts/documents.test.mjs",
     "scripts/web-tools.test.mjs",
@@ -231,7 +233,8 @@
     "scripts/awaiting-background-status.test.mjs",
     "scripts/project-lifecycle-e2e.test.mjs",
     "scripts/agent-tiers.test.mjs",
-    "scripts/design-principles.test.mjs"
+    "scripts/design-principles.test.mjs",
+    "scripts/primary-employee-guard.test.mjs"
   ],
   "check:browser": [
     "scripts/interrupt-browser.test.mjs",
@@ -265,7 +268,10 @@
     "scripts/workspace-identity-browser.test.mjs",
     "scripts/sidebar-tools-browser.test.mjs",
     "scripts/project-employee-panel-browser.test.mjs",
+    "scripts/primary-employee-guard-browser.test.mjs",
     "scripts/project-evidence-panel-browser.test.mjs",
+    "scripts/project-panel-fixes-browser.test.mjs",
+    "scripts/boss-approval-readability-browser.test.mjs",
     "scripts/builtin-agents-project-browser.test.mjs",
     "scripts/tool-runtime-browser.test.mjs",
     "scripts/session-context-menu-browser.test.mjs",
@@ -294,6 +300,7 @@
     "scripts/stream-tail-browser.test.mjs",
     "scripts/scroll-restore-browser.test.mjs",
     "scripts/theme-contrast.test.mjs",
+    "scripts/ui-contrast-browser.test.mjs",
     "scripts/theme-switch-state.test.mjs",
     "scripts/settings-window-structure.test.mjs",
     "scripts/settings-model-browser.test.mjs",