@cluesmith/codev-core 3.2.0 → 3.2.1

package/dist/architect-name.d.ts

@@ -0,0 +1,44 @@
+/**
+ * Architect-name utilities shared across packages (Spec 755 / 786).
+ *
+ * Lives in codev-core so the VS Code extension and the agent-farm server can
+ * both validate architect names with identical semantics — the extension's
+ * "Add Architect" InputBox (Issue 841) reuses the same rule Tower enforces
+ * server-side, instead of duplicating the regex.
+ *
+ * These functions are pure — they don't read process state or Tower; the caller
+ * is responsible for sourcing the existing-names set. Keeping them pure makes
+ * them trivially unit-testable. (`currentArchitectName`, which reads the
+ * process env, stays in the agent-farm package.)
+ */
+export declare const ARCHITECT_NAME_PATTERN: RegExp;
+export declare const MAX_ARCHITECT_NAME_LENGTH = 64;
+/** Reserved default for the singleton case. */
+export declare const DEFAULT_ARCHITECT_NAME = "main";
+/**
+ * Validate an architect name. Returns `null` if valid, or a human-readable
+ * error message otherwise. Callers should treat a non-null return as "reject
+ * with this message" — the text is intentionally operator-facing.
+ *
+ * Spec 786: the name `main` is reserved for the workspace's default architect
+ * and is rejected here. Pre-#786, `main` was rejected only by collision check
+ * at the add-architect call site (which depended on a race-free in-memory map);
+ * rejecting in the pure validator is more robust.
+ */
+export declare function validateArchitectName(name: string): string | null;
+/**
+ * Compute the next auto-numbered architect name, given the set of names
+ * already in use. Uses "smallest unused integer ≥ 2" semantics:
+ *
+ *   - {} → 'architect-2'
+ *   - {'main'} → 'architect-2'
+ *   - {'main', 'architect-2'} → 'architect-3'
+ *   - {'main', 'architect-3'} → 'architect-2' (fills the gap)
+ *   - {'main', 'architect-2', 'architect-3'} → 'architect-4'
+ *   - {'main', 'sibling'} → 'architect-2' (custom names don't shift numbering)
+ *
+ * Custom names (anything not matching `architect-<N>` exactly) are ignored
+ * by the numbering loop — they're not part of the auto-numbered sequence.
+ */
+export declare function autoNumberArchitectName(existingNames: Iterable<string>): string;
+//# sourceMappingURL=architect-name.d.ts.map

package/dist/architect-name.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"architect-name.d.ts","sourceRoot":"","sources":["../src/architect-name.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,eAAO,MAAM,sBAAsB,QAAsB,CAAC;AAC1D,eAAO,MAAM,yBAAyB,KAAK,CAAC;AAE5C,+CAA+C;AAC/C,eAAO,MAAM,sBAAsB,SAAS,CAAC;AAE7C;;;;;;;;;GASG;AACH,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAcjE;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CAAC,aAAa,EAAE,QAAQ,CAAC,MAAM,CAAC,GAAG,MAAM,CAc/E"}

package/dist/architect-name.js

@@ -0,0 +1,73 @@
+/**
+ * Architect-name utilities shared across packages (Spec 755 / 786).
+ *
+ * Lives in codev-core so the VS Code extension and the agent-farm server can
+ * both validate architect names with identical semantics — the extension's
+ * "Add Architect" InputBox (Issue 841) reuses the same rule Tower enforces
+ * server-side, instead of duplicating the regex.
+ *
+ * These functions are pure — they don't read process state or Tower; the caller
+ * is responsible for sourcing the existing-names set. Keeping them pure makes
+ * them trivially unit-testable. (`currentArchitectName`, which reads the
+ * process env, stays in the agent-farm package.)
+ */
+export const ARCHITECT_NAME_PATTERN = /^[a-z][a-z0-9-]*$/;
+export const MAX_ARCHITECT_NAME_LENGTH = 64;
+/** Reserved default for the singleton case. */
+export const DEFAULT_ARCHITECT_NAME = 'main';
+/**
+ * Validate an architect name. Returns `null` if valid, or a human-readable
+ * error message otherwise. Callers should treat a non-null return as "reject
+ * with this message" — the text is intentionally operator-facing.
+ *
+ * Spec 786: the name `main` is reserved for the workspace's default architect
+ * and is rejected here. Pre-#786, `main` was rejected only by collision check
+ * at the add-architect call site (which depended on a race-free in-memory map);
+ * rejecting in the pure validator is more robust.
+ */
+export function validateArchitectName(name) {
+    if (!name) {
+        return 'Architect name cannot be empty.';
+    }
+    if (name === DEFAULT_ARCHITECT_NAME) {
+        return `Architect name '${DEFAULT_ARCHITECT_NAME}' is reserved for the workspace's default architect.`;
+    }
+    if (name.length > MAX_ARCHITECT_NAME_LENGTH) {
+        return `Architect name must be at most ${MAX_ARCHITECT_NAME_LENGTH} characters (got ${name.length}).`;
+    }
+    if (!ARCHITECT_NAME_PATTERN.test(name)) {
+        return `Architect name '${name}' is invalid. Names must match [a-z][a-z0-9-]* (lowercase letter, then lowercase letters / digits / dashes).`;
+    }
+    return null;
+}
+/**
+ * Compute the next auto-numbered architect name, given the set of names
+ * already in use. Uses "smallest unused integer ≥ 2" semantics:
+ *
+ *   - {} → 'architect-2'
+ *   - {'main'} → 'architect-2'
+ *   - {'main', 'architect-2'} → 'architect-3'
+ *   - {'main', 'architect-3'} → 'architect-2' (fills the gap)
+ *   - {'main', 'architect-2', 'architect-3'} → 'architect-4'
+ *   - {'main', 'sibling'} → 'architect-2' (custom names don't shift numbering)
+ *
+ * Custom names (anything not matching `architect-<N>` exactly) are ignored
+ * by the numbering loop — they're not part of the auto-numbered sequence.
+ */
+export function autoNumberArchitectName(existingNames) {
+    const usedNumbers = new Set();
+    for (const name of existingNames) {
+        const match = /^architect-(\d+)$/.exec(name);
+        if (match) {
+            const n = Number.parseInt(match[1], 10);
+            if (Number.isFinite(n) && n >= 2) {
+                usedNumbers.add(n);
+            }
+        }
+    }
+    let n = 2;
+    while (usedNumbers.has(n))
+        n++;
+    return `architect-${n}`;
+}
+//# sourceMappingURL=architect-name.js.map

package/dist/architect-name.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"architect-name.js","sourceRoot":"","sources":["../src/architect-name.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,MAAM,CAAC,MAAM,sBAAsB,GAAG,mBAAmB,CAAC;AAC1D,MAAM,CAAC,MAAM,yBAAyB,GAAG,EAAE,CAAC;AAE5C,+CAA+C;AAC/C,MAAM,CAAC,MAAM,sBAAsB,GAAG,MAAM,CAAC;AAE7C;;;;;;;;;GASG;AACH,MAAM,UAAU,qBAAqB,CAAC,IAAY;IAChD,IAAI,CAAC,IAAI,EAAE,CAAC;QACV,OAAO,iCAAiC,CAAC;IAC3C,CAAC;IACD,IAAI,IAAI,KAAK,sBAAsB,EAAE,CAAC;QACpC,OAAO,mBAAmB,sBAAsB,sDAAsD,CAAC;IACzG,CAAC;IACD,IAAI,IAAI,CAAC,MAAM,GAAG,yBAAyB,EAAE,CAAC;QAC5C,OAAO,kCAAkC,yBAAyB,oBAAoB,IAAI,CAAC,MAAM,IAAI,CAAC;IACxG,CAAC;IACD,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,OAAO,mBAAmB,IAAI,8GAA8G,CAAC;IAC/I,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,uBAAuB,CAAC,aAA+B;IACrE,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;IACtC,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;QACjC,MAAM,KAAK,GAAG,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7C,IAAI,KAAK,EAAE,CAAC;YACV,MAAM,CAAC,GAAG,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACxC,IAAI,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBACjC,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;YACrB,CAAC;QACH,CAAC;IACH,CAAC;IACD,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,OAAO,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC;QAAE,CAAC,EAAE,CAAC;IAC/B,OAAO,aAAa,CAAC,EAAE,CAAC;AAC1B,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cluesmith/codev-core",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "Core runtime utilities for Codev — Tower client, auth, workspace helpers, escape buffer",
   "type": "module",
   "exports": {
@@ -32,6 +32,10 @@
       "types": "./dist/agent-names.d.ts",
       "default": "./dist/agent-names.js"
     },
+    "./architect-name": {
+      "types": "./dist/architect-name.d.ts",
+      "default": "./dist/architect-name.js"
+    },
     "./builder-helpers": {
       "types": "./dist/builder-helpers.d.ts",
       "default": "./dist/builder-helpers.js"
@@ -56,7 +60,7 @@
     "@types/node": "22.x",
     "typescript": "^5.7.0",
     "vitest": "^4.0.15",
-    "@cluesmith/codev-types": "3.2.0"
+    "@cluesmith/codev-types": "3.2.1"
   },
   "license": "Apache-2.0",
   "scripts": {