@evandu/super-pi 0.1.165 → 0.1.167

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.165",
+  "version": "0.1.167",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -48,6 +48,8 @@
     "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",
     "scripts/pi-session.test.mjs",
@@ -69,6 +71,7 @@
     "scripts/verify-source-text.test.mjs",
     "scripts/runtime-extension-ui.test.mjs",
     "scripts/agent-mechanism.test.mjs",
+    "scripts/builtin-agents-project.test.mjs",
     "scripts/super-pi-tool-names.test.mjs",
     "scripts/agent-wiring.test.mjs",
     "scripts/message-timestamps.test.mjs",
@@ -95,6 +98,7 @@
     "scripts/subagent-noise.test.mjs",
     "scripts/subagent-response-receipt.test.mjs",
     "scripts/subagent-runs.test.mjs",
+    "scripts/subagent-usage-accounting.test.mjs",
     "scripts/ws-flood-regression.test.mjs",
     "scripts/conversation-reconcile.test.mjs",
     "scripts/subagent-dispatch.test.mjs",
@@ -226,7 +230,9 @@
     "scripts/session-report-tool.test.mjs",
     "scripts/session-stop-subtree.test.mjs",
     "scripts/awaiting-background-status.test.mjs",
-    "scripts/project-lifecycle-e2e.test.mjs"
+    "scripts/project-lifecycle-e2e.test.mjs",
+    "scripts/agent-tiers.test.mjs",
+    "scripts/design-principles.test.mjs"
   ],
   "check:browser": [
     "scripts/interrupt-browser.test.mjs",
@@ -260,6 +266,9 @@
     "scripts/workspace-identity-browser.test.mjs",
     "scripts/sidebar-tools-browser.test.mjs",
     "scripts/project-employee-panel-browser.test.mjs",
+    "scripts/project-evidence-panel-browser.test.mjs",
+    "scripts/project-panel-fixes-browser.test.mjs",
+    "scripts/builtin-agents-project-browser.test.mjs",
     "scripts/tool-runtime-browser.test.mjs",
     "scripts/session-context-menu-browser.test.mjs",
     "scripts/archive-cascade-browser.test.mjs",
@@ -287,6 +296,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",