@toddzheng024/dscode-bundle 0.7.21 → 0.7.23

package/vendor/tui/lib/skills.mjs

@@ -32,6 +32,7 @@ export function watchSkills(ctx, fallbackCwd) {
     const skills = ctx.get('skills');
     let agent;
     let rows = [];
+    let count;
     let error;
     // The agent whose workspace the current rows were last successfully read
     // from: a failure for an agent that never loaded must clear the rows, not
@@ -58,11 +59,15 @@ export function watchSkills(ctx, fallbackCwd) {
                 return;
             const next = toRows(summaries);
             // Description and invocation-flag edits must surface too: a name-only
-            // comparison silently dropped those change notifications.
-            const unchanged = next.length === rows.length && next.every((row, index) => row.name === rows[index]?.name
+            // comparison silently dropped those change notifications. The catalog
+            // size is compared as well, because a model-only skill moves the footer
+            // count without changing any user-invocable row.
+            const unchanged = summaries.length === count
+                && next.length === rows.length && next.every((row, index) => row.name === rows[index]?.name
                 && row.description === rows[index]?.description
                 && row.modelInvocable === rows[index]?.modelInvocable);
             rows = next;
+            count = summaries.length;
             loadedFor = target;
             const recovered = error !== undefined;
             error = undefined;
@@ -77,15 +82,19 @@ export function watchSkills(ctx, fallbackCwd) {
             // next skills/change notification is the retry surface, mirroring the
             // web directory); a target that never loaded starts from empty rows —
             // stale rows from a previous workspace must not keep completing here.
-            // The rows array keeps its identity unless the failure text itself
-            // changed: a repeated identical error on the 0.1.5 event storm must not
-            // churn fresh identities into React's update chain.
+            // That clearing is itself a state change, so it notifies even when the
+            // error text repeats; the error text alone keeps the identity rule
+            // below, where a repeated identical error on the 0.1.5 event storm must
+            // not churn fresh identities into React's update chain.
             const nextError = cause instanceof Error ? cause.message : String(cause);
-            if (loadedFor !== target)
+            const cleared = loadedFor !== target && (rows.length > 0 || count !== undefined);
+            if (loadedFor !== target) {
                 rows = [];
+                count = undefined;
+            }
             const errorChanged = nextError !== error;
             error = nextError;
-            if (!errorChanged)
+            if (!errorChanged && !cleared)
                 return;
             for (const listener of listeners)
                 listener();
@@ -101,6 +110,9 @@ export function watchSkills(ctx, fallbackCwd) {
         get rows() {
             return rows;
         },
+        get count() {
+            return count;
+        },
         get error() {
             return error;
         },