@worca/ui 0.26.0 → 0.27.0

package/app/styles.css

@@ -2031,7 +2031,7 @@ sl-input [slot="prefix"] {
 
 .pricing-model-name {
   font-weight: 500;
-  text-transform: capitalize;
+  text-transform: uppercase;
 }
 
 .pricing-table sl-input {
@@ -5084,6 +5084,56 @@ sl-tooltip.bead-tooltip::part(body) {
   border-color: var(--status-failed, #ef4444);
 }
 
+.rename-model-input.is-invalid::part(base) {
+  border-color: var(--status-failed, #ef4444);
+}
+
+.rename-model-error {
+  color: var(--status-failed, #ef4444);
+  font-size: 12px;
+  margin: 0.4rem 0 0;
+}
+
+.model-env-details::part(base) {
+  border: none;
+  background: transparent;
+  box-shadow: none;
+}
+
+.model-env-details::part(header) {
+  padding: 0.25rem 0.4rem;
+  border-radius: 6px;
+}
+
+.model-env-details::part(header):hover {
+  background: var(--hover, rgba(0, 0, 0, 0.04));
+}
+
+.model-env-details::part(content) {
+  padding: 0.5rem 0 0;
+}
+
+.model-env-summary {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  width: 100%;
+  gap: 0.75rem;
+}
+
+.model-env-invalid-chip {
+  color: var(--status-failed, #ef4444);
+  font-weight: 600;
+  font-size: 11px;
+  text-transform: uppercase;
+  letter-spacing: 0.5px;
+}
+
+.model-env-details.has-invalid::part(header) {
+  border-left: 3px solid var(--status-failed, #ef4444);
+  padding-left: 0.5rem;
+}
+
 .model-env-value::part(input) {
   font-size: 12px;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@worca/ui",
-  "version": "0.26.0",
+  "version": "0.27.0",
   "description": "Pipeline monitoring UI for worca-cc",
   "license": "MIT",
   "author": "Sinisha Djukic",

package/server/beads-reader.js

@@ -59,14 +59,21 @@ export async function listIssues(beadsDb) {
 }
 
 export async function listIssuesByLabel(beadsDb, label) {
-  try {
+  const attempt = async () => {
     const issues = await runBd(
       ['list', '--label-any', label, '--all', '--limit', '0'],
       beadsDb,
     );
     return await enrichWithDeps(issues, beadsDb);
+  };
+  try {
+    return await attempt();
   } catch {
-    return [];
+    // bd/SQLite contention during active runs is usually sub-second — one
+    // retry covers the observed window. If it still fails, propagate so the
+    // WS handler can return an error rather than masquerading as "no beads."
+    await new Promise((r) => setTimeout(r, 150));
+    return await attempt();
   }
 }
 

package/server/model-env-routes.js

@@ -115,8 +115,13 @@ export function createModelEnvRouter({ settingsPath: staticPath } = {}) {
 
     let baseChanged = false;
     if (resolvedId) {
-      // Prefer string form when there's no other metadata — keeps JSON minimal.
-      const nextBaseEntry = resolvedId;
+      // When env exists in local, base MUST use the object form `{id}` so
+      // deepMerge({id}, {env}) preserves the id. With the string form,
+      // deepMerge would see a non-object base and discard it, dropping the id
+      // entirely — the bug behind empty Model ID after Duplicate/Paste.
+      // String form stays the default when there's no env, to keep JSON minimal.
+      const hasEnv = Object.keys(envIn).length > 0;
+      const nextBaseEntry = hasEnv ? { id: resolvedId } : resolvedId;
       if (JSON.stringify(baseEntry) !== JSON.stringify(nextBaseEntry)) {
         base.worca.models[model] = nextBaseEntry;
         baseChanged = true;

package/server/project-routes.js

@@ -478,7 +478,18 @@ export function createProjectScopedRoutes({
       });
     }
 
-    const validation = validateSettingsPayload(body);
+    let existingForValidation = {};
+    try {
+      if (existsSync(settingsPath)) {
+        existingForValidation = JSON.parse(readFileSync(settingsPath, 'utf8'));
+      }
+    } catch {
+      existingForValidation = {};
+    }
+
+    const validation = validateSettingsPayload(body, {
+      existing: existingForValidation,
+    });
     if (!validation.valid) {
       return res.status(400).json({
         error: {
@@ -558,12 +569,43 @@ export function createProjectScopedRoutes({
         atomicWriteSync(settingsPath, `${JSON.stringify(base, null, 2)}\n`);
       }
 
+      // STEP 3a: strip shadowed worca keys from settings.local.json. Local is
+      // deep-merged over base on read, so a stale `worca.<key>` copy in local
+      // would resurrect after the user saves a new value. `models` is excluded
+      // because its env-portion lives in local by design (see model-env-routes).
+      const lp = localPathFor(settingsPath);
+      let localChanged = false;
+      const localForPrune = readLocalSettings(settingsPath);
+      if (
+        body.worca &&
+        typeof body.worca === 'object' &&
+        localForPrune.worca &&
+        typeof localForPrune.worca === 'object'
+      ) {
+        for (const key of Object.keys(body.worca)) {
+          if (key === 'models') continue;
+          if (key in localForPrune.worca) {
+            delete localForPrune.worca[key];
+            localChanged = true;
+          }
+        }
+        if (localChanged && Object.keys(localForPrune.worca).length === 0) {
+          delete localForPrune.worca;
+        }
+      }
+
       if (body.permissions !== undefined) {
-        const lp = localPathFor(settingsPath);
-        const local = readLocalSettings(settingsPath);
-        local.permissions = body.permissions;
+        localForPrune.permissions = body.permissions;
+        localChanged = true;
+      }
+
+      if (localChanged) {
         mkdirSync(dirname(lp), { recursive: true });
-        writeFileSync(lp, `${JSON.stringify(local, null, 2)}\n`, 'utf8');
+        writeFileSync(
+          lp,
+          `${JSON.stringify(localForPrune, null, 2)}\n`,
+          'utf8',
+        );
       }
 
       const merged = readMergedSettings(settingsPath);

package/server/settings-validator.js

@@ -36,8 +36,12 @@ const VALID_PRICING_FIELDS = [
   'cache_read_per_mtok',
 ];
 
-export function validateSettingsPayload(body) {
+export function validateSettingsPayload(body, options = {}) {
   const details = [];
+  const existingWorca =
+    options.existing && typeof options.existing === 'object'
+      ? options.existing.worca || {}
+      : {};
 
   if (body.worca !== undefined) {
     if (
@@ -49,7 +53,16 @@ export function validateSettingsPayload(body) {
       return { valid: false, details };
     }
     const w = body.worca;
-    const validModels = deriveValidModels(w);
+    // Sections like agents/pricing reference model keys that may live in another
+    // section saved earlier. Merge persisted models with body-supplied models so
+    // a single-section save (e.g. agents-only) doesn't reject custom models.
+    const mergedModels = {
+      ...(existingWorca.models && typeof existingWorca.models === 'object'
+        ? existingWorca.models
+        : {}),
+      ...(w.models && typeof w.models === 'object' ? w.models : {}),
+    };
+    const validModels = deriveValidModels({ models: mergedModels });
 
     // agents
     if (w.agents !== undefined) {

package/server/ws-message-router.js

@@ -682,14 +682,32 @@ export function createMessageRouter({
         ws.send(JSON.stringify(makeOk(req, { issues: [], runId })));
         return;
       }
-      const issues = await listIssuesByLabel(beadsDbPath, `run:${runId}`);
-      console.log(
-        '[list-beads-by-run] runId=%s count=%d statuses=%o',
-        runId,
-        issues.length,
-        issues.map((i) => i.status),
-      );
-      ws.send(JSON.stringify(makeOk(req, { issues, runId })));
+      try {
+        const issues = await listIssuesByLabel(beadsDbPath, `run:${runId}`);
+        console.log(
+          '[list-beads-by-run] runId=%s count=%d statuses=%o',
+          runId,
+          issues.length,
+          issues.map((i) => i.status),
+        );
+        ws.send(JSON.stringify(makeOk(req, { issues, runId })));
+      } catch (err) {
+        // Don't return empty issues on failure — the UI would treat that as
+        // "all beads deleted" and tear down the open <sl-details> panel. Let
+        // the client keep its last-known-good state until the next poll.
+        console.warn(
+          `[list-beads-by-run] runId=${runId} failed: ${err?.message || err}`,
+        );
+        ws.send(
+          JSON.stringify(
+            makeError(
+              req,
+              'beads_unavailable',
+              `bd query failed: ${err?.message || err}`,
+            ),
+          ),
+        );
+      }
       return;
     }