pi-skillful 0.3.9 → 0.3.12

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.3.12 - 2026-07-12
+
+## 0.3.11 - 2026-07-01
+
 ## 0.3.9 - 2026-06-14
 
 All notable changes to this project will be documented in this file.
@@ -10,6 +14,19 @@ This project follows the spirit of [Keep a Changelog](https://keepachangelog.com
 
 ### Fixed
 
+- Preserve configured hidden skills at session start. Skillful no longer treats a temporarily incomplete loaded-skill list as authoritative and removes global or project visibility settings.
+
+## [0.3.10] - 2026-07-01
+
+### Fixed
+
+- Color hidden skills in the startup `[Skills]` list on `@earendil-works/pi-coding-agent` 0.80+. The 0.80 release moved the loaded-resources container out of `chatContainer`, so the existing prototype patch walked the wrong container and the red/dim colorization no longer applied. The patch now also walks `loadedResourcesContainer`, where 0.80 actually renders the `[Skills]` section.
+- Bumped dev dependency range to `^0.80.0` to match the new floor.
+
+## [0.3.9] - 2026-06-14
+
+### Fixed
+
 - Skipped Pi package-bundled skills when applying hidden-skill and toggle-slot configuration so `skillful` only affects global and project skills.
 
 ## [0.3.8] - 2026-06-06

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-skillful",
-  "version": "0.3.9",
+  "version": "0.3.12",
   "description": "Pi package with skill invocation and visibility improvements.",
   "type": "module",
   "exports": {
@@ -45,14 +45,15 @@
     "@earendil-works/pi-tui": "*"
   },
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.78.0",
-    "@earendil-works/pi-tui": "^0.78.0",
-    "@types/node": "^25.6.2",
+    "@earendil-works/pi-coding-agent": "^0.80.0",
+    "@earendil-works/pi-tui": "^0.80.0",
+    "@types/node": "^26.0.1",
     "typescript": "^6.0.3"
   },
   "scripts": {
     "check": "tsc --noEmit",
     "typecheck": "tsc --noEmit",
+    "test": "rm -rf .test-dist && tsc --noEmit false --outDir .test-dist && node --test tests/*.test.mjs",
     "pack:dry-run": "npm pack --dry-run"
   },
   "publishConfig": {

package/src/extensions/skill-visibility.ts

@@ -42,7 +42,7 @@ interface BoxLike {
 }
 
 interface InteractiveModeLike {
-  chatContainer?: BoxLike;
+  loadedResourcesContainer?: BoxLike;
   showLoadedResources?: (options?: unknown) => void;
   session?: { resourceLoader?: { getSkills: () => { skills: Skill[]; diagnostics: unknown[] } } };
   sessionManager?: { getCwd?: () => string };
@@ -85,7 +85,7 @@ export default function skillVisibility(pi: ExtensionAPI) {
 
   pi.on("session_start", async (_event, ctx) => {
     store.theme = ctx.ui.theme;
-    await pruneStaleHiddenSkills(pi, ctx.cwd);
+    await refreshHiddenSkillCache(ctx.cwd);
   });
 
   pi.on("before_agent_start", async (event, ctx) => {
@@ -148,25 +148,6 @@ export default function skillVisibility(pi: ExtensionAPI) {
   });
 }
 
-async function pruneStaleHiddenSkills(pi: ExtensionAPI, cwd: string): Promise<void> {
-  const installedNames = new Set(getSkillItems(pi).map((s) => s.name));
-  const scoped = await readScopedSkillfulSettings(cwd);
-
-  await Promise.all(
-    SCOPES.map(async (scope) => {
-      const currentHidden = scoped[scope].hiddenSkills;
-      const prunedHidden = currentHidden.filter((name) => installedNames.has(name));
-      if (prunedHidden.length < currentHidden.length) {
-        scoped[scope] = await writeHiddenSkills(scope, cwd, prunedHidden);
-      }
-    }),
-  );
-
-  const hidden = new Set(scoped.project.hiddenSkillsDefined ? scoped.project.hiddenSkills : scoped.global.hiddenSkills);
-  store.hiddenSkillsByCwd.set(cwd, hidden);
-  store.lastHiddenSkills = hidden;
-}
-
 async function refreshHiddenSkillCache(cwd: string): Promise<Set<string>> {
   const hidden = await readEffectiveHiddenSkills(cwd);
   store.hiddenSkillsByCwd.set(cwd, hidden);
@@ -202,7 +183,7 @@ function installStartupSkillListPatch(): void {
       return result;
     };
 
-    const childrenBefore = this.chatContainer?.children.length ?? 0;
+    const childrenBefore = this.loadedResourcesContainer?.children.length ?? 0;
 
     try {
       original.call(this, options);
@@ -210,17 +191,18 @@ function installStartupSkillListPatch(): void {
       loader.getSkills = originalGetSkills;
     }
 
-    if (rawSkillNames.length === 0 || !cwd || !this.chatContainer) return;
+    if (rawSkillNames.length === 0 || !cwd || !this.loadedResourcesContainer) return;
 
-    const children = this.chatContainer.children;
+    const children = this.loadedResourcesContainer.children;
     for (let i = childrenBefore; i < children.length; i++) {
       const child = children[i] as ExpandableTextLike | undefined;
       if (!child || typeof child.getCollapsedText !== "function") continue;
       const collapsed = child.getCollapsedText();
       if (!collapsed.includes("[Skills]")) continue;
 
-      child.getCollapsedText = () => buildColorizedSkillList(rawSkillNames, store.lastHiddenSkills, store.theme);
-      child.setText(child.getCollapsedText());
+      const colorized = buildColorizedSkillList(rawSkillNames, store.lastHiddenSkills, store.theme);
+      child.getCollapsedText = () => colorized;
+      child.setText(colorized);
       break;
     }
   };