@whittlelabs/sifter 0.26.0 → 0.27.0

package/bin.js

@@ -15399,9 +15399,26 @@ var require_client = __commonJS({
       async getJobPool(poolId) {
         return this.request("GET", `/api/job-pools/${poolId}`);
       }
+      /**
+       * Resolve a pool by exact name. Names are unique per OWNER, not globally
+       * (whittlelabs#763), and this method lists without a principal filter, so
+       * the same name can legitimately come back under several owners. That
+       * reference is inherently ambiguous — and silently picking one (the old
+       * behavior took the server's newest-first ordering) is exactly the
+       * capture vector the per-owner constraint closed, re-opened cross-owner.
+       * So it refuses instead: reference the pool by id, or list with explicit
+       * principal filters, when more than one owner holds the name.
+       */
       async resolveJobPoolByName(name) {
         const pools = await this.listJobPools({ name });
-        return pools.find((p) => p.name === name);
+        const matches = pools.filter((p) => p.name === name);
+        const owners = new Set(matches.map((p) => `${p.principalClass ?? ""}/${p.principalId ?? ""}`));
+        if (owners.size > 1) {
+          const error = new Error(`Job pool name "${name}" is ambiguous: ${matches.length} pools under ${owners.size} different owners hold it. Reference the pool by id, or resolve it with explicit principal filters (listJobPools).`);
+          error.code = "JOB_POOL_NAME_AMBIGUOUS";
+          throw error;
+        }
+        return matches[0];
       }
       /** Internal: resolve a `PoolRef` to a pool id. */
       async resolvePoolRef(ref) {
@@ -22433,7 +22450,7 @@ var import_path = require("path");
 var import_promises = require("fs/promises");
 var import_yaml = __toESM(require_dist());
 var import_shuttle = __toESM(require_dist5());
-var buildVersion = true ? "0.26.0" : pkg.version;
+var buildVersion = true ? "0.27.0" : pkg.version;
 var sifterBrand = {
   product: {
     id: "sifter",