chapterhouse 0.3.6 → 0.3.7

package/dist/squad/discovery.js

@@ -1,5 +1,6 @@
-import { existsSync, readFileSync, readdirSync, statSync } from 'fs';
+import { existsSync } from 'fs';
 import { join } from 'path';
+import { FSStorageProvider, SquadState, NotFoundError } from '@bradygaster/squad-sdk';
 import { getDb } from '../store/db.js';
 // ---------------------------------------------------------------------------
 // DB schema — created lazily on first use
@@ -101,47 +102,38 @@ export async function resolveProjectSquad(projectPath) {
     catch {
         // SDK not available — use manual path resolution
     }
-    // Enumerate agents from .squad/agents/
-    const agentsDir = join(resolvedSquadDir, 'agents');
+    // Enumerate agents from .squad/agents/ via SDK AgentsCollection.
+    // FSStorageProvider wraps the same fs calls with typed returns and ENOENT safety.
     const agents = [];
-    if (existsSync(agentsDir)) {
-        let entries = [];
-        try {
-            entries = readdirSync(agentsDir);
-        }
-        catch { /* ignore read errors */ }
-        for (const entry of entries) {
-            const agentDir = join(agentsDir, entry);
-            try {
-                if (!statSync(agentDir).isDirectory())
-                    continue;
-            }
-            catch {
-                continue;
-            }
-            const charterPath = join(agentDir, 'charter.md');
-            if (!existsSync(charterPath))
-                continue;
-            let charterContent = '';
+    const stateRoot = resolvedSquadDir.endsWith('/.squad')
+        ? resolvedSquadDir.slice(0, -'/.squad'.length)
+        : projectPath;
+    try {
+        const storage = new FSStorageProvider();
+        const state = SquadState.fromStorage(storage, stateRoot);
+        const agentNames = await state.agents.list();
+        for (const name of agentNames) {
+            const charterPath = join(resolvedSquadDir, 'agents', name, 'charter.md');
+            let charterContent;
             try {
-                charterContent = readFileSync(charterPath, 'utf-8');
+                charterContent = await state.agents.get(name).charter();
             }
-            catch {
+            catch (err) {
+                if (err instanceof NotFoundError)
+                    continue; // no charter.md — not a valid agent
                 continue;
             }
-            const slug = entry;
-            const role = extractRoleFromCharter(charterContent);
-            const description = extractDescriptionFromCharter(charterContent);
             agents.push({
-                slug,
-                mention: `@${slug}`,
-                role,
-                description,
+                slug: name,
+                mention: `@${name}`,
+                role: extractRoleFromCharter(charterContent),
+                description: extractDescriptionFromCharter(charterContent),
                 charterPath,
                 origin: 'project-squad',
             });
         }
     }
+    catch { /* SDK unavailable — agents list stays empty */ }
     // Load squad config via SDK with fallback to empty config
     let squadConfig = {};
     try {
@@ -228,25 +220,21 @@ export async function loadProjectSquad(projectPath) {
                         : undefined,
                     origin: 'project-squad',
                 }));
-                // Repo wins: re-sync any new agents that appeared since last cache
-                const repoAgentsDir = join(row.squad_dir, 'agents');
-                if (existsSync(repoAgentsDir)) {
-                    const repoDirs = readdirSync(repoAgentsDir).filter(e => {
-                        try {
-                            return statSync(join(repoAgentsDir, e)).isDirectory();
-                        }
-                        catch {
-                            return false;
-                        }
-                    });
+                // Repo wins: re-sync any new agents that appeared since last cache.
+                // Use SDK AgentsCollection.list() so we stay consistent with resolveProjectSquad().
+                try {
+                    const cacheStorage = new FSStorageProvider();
+                    const cacheState = SquadState.fromStorage(cacheStorage, row.project_root);
+                    const repoAgentNames = await cacheState.agents.list();
                     const cachedSlugs = new Set(agents.map(a => a.slug));
-                    for (const slug of repoDirs) {
+                    for (const slug of repoAgentNames) {
                         if (!cachedSlugs.has(slug)) {
                             // New agent in repo not in cache — trigger full reload
                             return resolveProjectSquad(projectPath);
                         }
                     }
                 }
+                catch { /* SDK unavailable — skip new-agent check */ }
                 return {
                     projectRoot: row.project_root,
                     squadDir: row.squad_dir,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chapterhouse",
-  "version": "0.3.6",
+  "version": "0.3.7",
   "description": "Chapterhouse — a team-level AI assistant for engineering teams, built on the GitHub Copilot SDK. Web UI only.",
   "bin": {
     "chapterhouse": "dist/cli.js"