gnosys 5.12.0 → 5.12.3

package/dist/lib/resolver.js

@@ -79,8 +79,13 @@ export class GnosysResolver {
             }
         }
         catch {
-            // Store doesn't exist at projectRoot — fall through to empty resolver
+            // No project store at projectRoot — still load the env tiers below.
         }
+        // Even when scoped to a projectRoot, load the personal/global/optional
+        // tiers so cross-project + global memories stay reachable (write store:
+        // "global"/"personal", and read them back from any project). Without this,
+        // the universal "always pass projectRoot" guidance silently disabled them.
+        await resolver.loadEnvTiers();
         return resolver;
     }
     /**
@@ -104,7 +109,30 @@ export class GnosysResolver {
                 path: projectPath,
             });
         }
-        // 2. Optional stores (GNOSYS_STORES — colon-separated)
+        // 2-4. Env-configured tiers (optional, personal, global). Shared with
+        //       resolveForProject so passing a projectRoot does NOT strip the
+        //       cross-project tiers.
+        await this.loadEnvTiers();
+        return this.stores;
+    }
+    /**
+     * Load the env-configured store tiers (optional `GNOSYS_STORES`, personal
+     * `GNOSYS_PERSONAL`, global `GNOSYS_GLOBAL`) onto `this.stores`.
+     *
+     * Shared by {@link resolve} and {@link resolveForProject}: a per-tool
+     * `projectRoot` call must STILL see personal/global, otherwise cross-project
+     * and global memories are unreachable — you couldn't write `store: "global"`
+     * / `"personal"` nor read them back from another project. Since every tool
+     * is told to always pass `projectRoot`, omitting these here silently
+     * disabled the entire cross-project tier system.
+     *
+     * Global is loaded whenever `GNOSYS_GLOBAL` is set, creating its directory on
+     * demand (parity with personal) so a configured-but-empty global store still
+     * works. It remains write-only-when-explicitly-targeted: {@link getWriteTarget}
+     * never auto-selects it.
+     */
+    async loadEnvTiers() {
+        // Optional stores (GNOSYS_STORES — colon-separated, read-only)
         const optionalPaths = process.env.GNOSYS_STORES;
         if (optionalPaths) {
             const paths = optionalPaths.split(":").filter(Boolean);
@@ -124,7 +152,7 @@ export class GnosysResolver {
                 }
             }
         }
-        // 3. Personal store (GNOSYS_PERSONAL)
+        // Personal store (GNOSYS_PERSONAL — writable fallback target)
         const personalPath = process.env.GNOSYS_PERSONAL;
         if (personalPath) {
             const p = path.resolve(personalPath);
@@ -138,24 +166,22 @@ export class GnosysResolver {
                 path: p,
             });
         }
-        // 4. Global store (GNOSYS_GLOBAL)
-        //    Writable, but only when explicitly targeted — never auto-selected.
+        // Global store (GNOSYS_GLOBAL — writable, but never auto-selected). Loaded
+        // whenever set; store.init() creates the dir on demand (parity with
+        // personal), so a configured-but-empty GNOSYS_GLOBAL still works.
         const globalPath = process.env.GNOSYS_GLOBAL;
         if (globalPath) {
             const p = path.resolve(globalPath);
-            if (await this.isValidStore(p)) {
-                const store = new GnosysStore(p);
-                await store.init();
-                this.stores.push({
-                    layer: "global",
-                    label: "global",
-                    store,
-                    writable: true,
-                    path: p,
-                });
-            }
+            const store = new GnosysStore(p);
+            await store.init();
+            this.stores.push({
+                layer: "global",
+                label: "global",
+                store,
+                writable: true,
+                path: p,
+            });
         }
-        return this.stores;
     }
     /**
      * Get all stores in precedence order.

package/dist/lib/rulesGen.d.ts

@@ -47,6 +47,14 @@ export declare function syncRules(centralDb: GnosysDB, projectDir: string, agent
  * If target is "global", syncs to ~/.claude/CLAUDE.md.
  */
 export declare function syncToTarget(centralDb: GnosysDB, projectDir: string, target: string, projectId: string | null): Promise<RulesGenResult[]>;
+/**
+ * Remove the GNOSYS block from every known agent rules file in a project —
+ * the uninstall counterpart of syncToTarget (v5.12.1: wired into
+ * `gnosys cleanup --rules`). Checks every known target rather than only
+ * detected ones, so leftovers survive even if the agent dir was removed.
+ * Returns the relative paths actually cleaned.
+ */
+export declare function removeRulesFromProject(projectDir: string): Promise<string[]>;
 /**
  * Remove the GNOSYS block from a rules file (cleanup).
  */

package/dist/lib/rulesGen.js

@@ -254,6 +254,22 @@ export async function syncToTarget(centralDb, projectDir, target, projectId) {
     }
     return results;
 }
+/**
+ * Remove the GNOSYS block from every known agent rules file in a project —
+ * the uninstall counterpart of syncToTarget (v5.12.1: wired into
+ * `gnosys cleanup --rules`). Checks every known target rather than only
+ * detected ones, so leftovers survive even if the agent dir was removed.
+ * Returns the relative paths actually cleaned.
+ */
+export async function removeRulesFromProject(projectDir) {
+    const cleaned = [];
+    for (const relPath of Object.values(TARGET_PATHS)) {
+        const filePath = path.join(projectDir, relPath);
+        if (await removeRulesBlock(filePath))
+            cleaned.push(relPath);
+    }
+    return cleaned;
+}
 /**
  * Remove the GNOSYS block from a rules file (cleanup).
  */

package/dist/lib/search.d.ts

@@ -17,8 +17,6 @@ export interface DiscoverResult {
 }
 export declare class GnosysSearch {
     private db;
-    private storePath;
-    private available;
     constructor(storePath: string);
     private initSchema;
     /**

package/dist/lib/search.js

@@ -3,7 +3,6 @@
  * FTS5-based search and discovery across all Gnosys stores.
  */
 // Dynamic import — gracefully handles missing native module (dlopen failures)
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 let Database = null;
 try {
     Database = (await import("better-sqlite3")).default;
@@ -12,13 +11,10 @@ catch {
     // better-sqlite3 native module not available — search degrades gracefully
 }
 import path from "path";
+import { ftsTerms, ftsAndQuery, ftsOrQuery } from "./ftsQuery.js";
 export class GnosysSearch {
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     db = null;
-    storePath;
-    available = false;
     constructor(storePath) {
-        this.storePath = storePath;
         if (!Database) {
             // Native module not available — search features disabled
             return;
@@ -34,7 +30,6 @@ export class GnosysSearch {
             this.initSchema();
             // Smoke-test: insert + delete to confirm journal ops work
             this.db.exec("CREATE TABLE IF NOT EXISTS _write_test (v INTEGER); INSERT INTO _write_test VALUES (1); DELETE FROM _write_test; DROP TABLE _write_test;");
-            this.available = true;
         }
         catch {
             // Fallback to in-memory (works everywhere, rebuilt on each start)
@@ -44,7 +39,6 @@ export class GnosysSearch {
             catch { /* ignore */ }
             this.db = new Database(":memory:");
             this.initSchema();
-            this.available = true;
         }
     }
     initSchema() {
@@ -140,9 +134,8 @@ export class GnosysSearch {
     search(query, limit = 20) {
         if (!this.db)
             return [];
-        // FTS5 query — escape special characters
-        const safeQuery = query.replace(/['"]/g, "").trim();
-        if (!safeQuery)
+        const terms = ftsTerms(query);
+        if (terms.length === 0)
             return [];
         const stmt = this.db.prepare(`
       SELECT
@@ -156,7 +149,12 @@ export class GnosysSearch {
       LIMIT ?
     `);
         try {
-            return stmt.all(safeQuery, limit);
+            // v5.12.3: AND first (precision), OR retry when AND finds nothing —
+            // multi-word queries previously required every term to match.
+            const results = stmt.all(ftsAndQuery(terms), limit);
+            if (results.length > 0 || terms.length === 1)
+                return results;
+            return stmt.all(ftsOrQuery(terms), limit);
         }
         catch {
             // If FTS5 query fails, fall back to simple LIKE search
@@ -170,7 +168,7 @@ export class GnosysSearch {
         WHERE content LIKE ? OR title LIKE ? OR tags LIKE ?
         LIMIT ?
       `);
-            const pattern = `%${safeQuery}%`;
+            const pattern = `%${terms.join(" ")}%`;
             return likeStmt.all(pattern, pattern, pattern, limit);
         }
     }
@@ -182,8 +180,8 @@ export class GnosysSearch {
     discover(query, limit = 20) {
         if (!this.db)
             return [];
-        const safeQuery = query.replace(/['"]/g, "").trim();
-        if (!safeQuery)
+        const terms = ftsTerms(query);
+        if (terms.length === 0)
             return [];
         // Search primarily on relevance + title + tags (not content body)
         // FTS5 column filter: {relevance title tags}
@@ -198,24 +196,32 @@ export class GnosysSearch {
       ORDER BY rank
       LIMIT ?
     `);
-        try {
-            // Try column-filtered search on relevance/title/tags first
-            const colQuery = `{relevance title tags} : ${safeQuery}`;
-            const results = stmt.all(colQuery, limit);
-            if (results.length > 0)
-                return results;
-            // Fall back to full-text search if column filter finds nothing
-            return stmt.all(safeQuery, limit);
-        }
-        catch {
-            // If FTS5 column filter syntax fails, fall back to full search
+        const tryRun = (match) => {
             try {
-                return stmt.all(safeQuery, limit);
+                return stmt.all(match, limit);
             }
             catch {
                 return [];
             }
-        }
+        };
+        // v5.12.3: precision-to-recall ladder. AND on the metadata columns,
+        // AND anywhere, then OR retries — multi-word queries previously
+        // required every term to match, so long queries returned nothing.
+        // Parens scope the column filter to the whole expression (a bare
+        // `{cols} : a b` only filtered the first term).
+        const colScoped = (expr) => `{relevance title tags} : (${expr})`;
+        const andExpr = ftsAndQuery(terms);
+        let results = tryRun(colScoped(andExpr));
+        if (results.length > 0)
+            return results;
+        results = tryRun(andExpr);
+        if (results.length > 0 || terms.length === 1)
+            return results;
+        const orExpr = ftsOrQuery(terms);
+        results = tryRun(colScoped(orExpr));
+        if (results.length > 0)
+            return results;
+        return tryRun(orExpr);
     }
     close() {
         this.db?.close();

package/dist/lib/semanticSearchCommand.js

@@ -24,7 +24,7 @@ export async function runSemanticSearchCommand(getResolver, query, opts) {
     const { GnosysHybridSearch } = await import("./hybridSearch.js");
     const embeddings = new GnosysEmbeddings(storePath);
     const hybridSearch = new GnosysHybridSearch(search, embeddings, resolver, storePath);
-    const results = await hybridSearch.hybridSearch(query, parseInt(opts.limit), "semantic");
+    const results = await hybridSearch.hybridSearch(query, parseInt(opts.limit, 10), "semantic");
     outputResult(!!opts.json, {
         query,
         count: results.length,

package/dist/lib/setup/sections/providers.js

@@ -129,14 +129,67 @@ async function manageProviderKeys(rl, storePath, cfg, provider) {
             }
             console.log("");
         }
+        const defaultAction = slots.length > 0 ? 2 : 0; // when they have a key, default to "make default + model"
         const choice = await askChoice(rl, "Actions", [
             slots.length === 0 ? "Add API key (global)" : "Rotate global API key",
             "Delete one stored key…",
+            "Use this provider as default for everything + pick model",
             "Back",
-        ], 2);
-        if (choice === 2) {
+        ], defaultAction);
+        if (choice === 3) {
             return changed;
         }
+        if (choice === 2) {
+            // "Use this provider as default for everything + pick model"
+            // Set it as the global default provider
+            const current = await loadConfig(storePath);
+            await updateConfig(storePath, {
+                llm: {
+                    ...current.llm,
+                    defaultProvider: provider,
+                },
+                // Clear per-task overrides so this truly becomes the single default for everything
+                taskModels: {},
+            });
+            printStatus("ok", `default provider set to ${provider} (all tasks now default to it)`);
+            // Immediately let them pick a model for it (using the same picker the main flow uses)
+            try {
+                const { fetchDynamicModels } = await import("../../setup.js");
+                const { pickModel } = await import("../../setup.js");
+                const dynamicModels = await fetchDynamicModels();
+                const tiers = dynamicModels[provider] ?? [];
+                let chosenModel;
+                if (tiers.length > 0) {
+                    chosenModel = await pickModel(rl, provider, dynamicModels, `Default model for ${provider} (used for all tasks)`, current.llm[provider]?.model);
+                }
+                else {
+                    chosenModel = await ask(rl, `Enter default model name for ${provider}: `);
+                }
+                if (chosenModel) {
+                    const after = await loadConfig(storePath);
+                    await updateConfig(storePath, {
+                        llm: {
+                            ...after.llm,
+                            [provider]: {
+                                ...(after.llm[provider] || {}),
+                                model: chosenModel,
+                            },
+                        },
+                    });
+                    printStatus("ok", `default model set · ${provider} / ${chosenModel}`);
+                    printStatus("progress", "all tasks will now default to this provider + model", "use task routing for per-task overrides");
+                    changed = true;
+                }
+                else {
+                    printStatus("progress", "model left unchanged — you can set it from task routing");
+                }
+            }
+            catch (_err) {
+                printStatus("warn", "could not pick model right now", "you can set a default model from the main setup → task routing");
+            }
+            cfg = await loadConfig(storePath);
+            continue;
+        }
         if (choice === 0) {
             const service = apiKeyServiceName(provider, "global");
             console.log("");
@@ -188,7 +241,6 @@ async function manageProviderKeys(rl, storePath, cfg, provider) {
                     printStatus("fail", "delete failed or entry not found");
                 }
             }
-            continue;
         }
     }
 }
@@ -207,7 +259,7 @@ export async function runProvidersSetup(opts) {
         console.log("");
         console.log(Header(["gnosys", "setup", "providers"]));
         console.log("");
-        console.log(Title("Providers", "API keys live here · pick task routing separately for models per task"));
+        console.log(Title("Providers", "API keys live here. Set a global default (or per-task models) from the main setup menu."));
         console.log("");
         for (let i = 0; i < menuProviders.length; i++) {
             const p = menuProviders[i];

package/dist/lib/setup/sections/routing.js

@@ -78,14 +78,49 @@ export async function runRoutingSetup(opts) {
     console.log("");
     const choice = await askChoice(opts.rl, "What would you like to do?", [
         "Keep current routing (no changes)",
-        "Edit tasks — pick by number, then provider + model for each",
-        "Reset all task overrides to use default",
+        "Edit tasks — set the same provider + model for all tasks (simple global default)",
+        "Edit tasks — pick different providers/models for specific tasks (advanced)",
+        "Reset all task overrides to the current default (the one shown in the main setup summary)",
     ], 0);
     if (choice === 0) {
         console.log(`${DIM}No changes.${RESET}`);
         return false;
     }
-    if (choice === 2) {
+    if (choice === 1) {
+        // Edit tasks — set the same provider + model for all tasks (simple global default)
+        console.log("");
+        printStatus("progress", "setting a single default for all tasks…");
+        try {
+            const { fetchDynamicModels } = await import("../../setup.js");
+            const { pickModel } = await import("../../setup.js");
+            const dynamicModels = await fetchDynamicModels();
+            const currentModel = getProviderModel(cfg, provider);
+            const chosenModel = await pickModel(opts.rl, provider, dynamicModels, `Default model for ${provider} (used for all tasks + dream)`, currentModel);
+            if (chosenModel && chosenModel !== currentModel) {
+                const after = await loadConfig(storePath);
+                await updateConfig(storePath, {
+                    llm: {
+                        ...after.llm,
+                        [provider]: {
+                            ...(after.llm[provider] || {}),
+                            model: chosenModel,
+                        },
+                    },
+                    // Clear per-task overrides so everything truly uses the single default
+                    taskModels: {},
+                });
+                printStatus("ok", `default set for everything · ${provider} / ${chosenModel}`);
+            }
+            else {
+                printStatus("ok", "no change to the global default");
+            }
+        }
+        catch (err) {
+            printStatus("warn", "could not update default model", String(err));
+        }
+        return true;
+    }
+    if (choice === 3) {
         const { Diff } = await import("../ui/diff.js");
         const overridesBeingCleared = Object.entries(cfg.taskModels ?? {})
             .filter(([, v]) => v.provider !== provider || v.model !== model)
@@ -102,16 +137,18 @@ export async function runRoutingSetup(opts) {
         else {
             console.log(`${DIM}No overrides to clear — already using default everywhere.${RESET}`);
         }
-        const confirmReset = await askYesNo(opts.rl, "Reset all task overrides?", true);
+        console.log(`${DIM}The current default is ${provider} / ${model} (set via the main setup "Default provider" or the simple global option above).${RESET}`);
+        const confirmReset = await askYesNo(opts.rl, "Reset all task overrides to the current default?", true);
         if (!confirmReset) {
             console.log(`${DIM}Cancelled.${RESET}`);
             return false;
         }
         await updateConfig(storePath, { taskModels: {} });
-        printStatus("ok", "routing reset", "all tasks use default provider/model");
+        printStatus("ok", "routing reset", "all tasks now use the global default provider/model");
         console.log(Footer("press enter to return"));
         return true;
     }
+    // choice === 2 → advanced per-task editor (the powerful comma-list path)
     const patch = await runCommaListRoutingEditor(opts.rl, storePath, cfg);
     if (!patch) {
         return false;

package/dist/lib/setup/sections/taskRoutingEditor.d.ts

@@ -2,7 +2,7 @@
  * Comma-list task routing editor (provider + model per selected task).
  */
 import type { Interface as ReadlineInterface } from "readline/promises";
-import { type GnosysConfig } from "../../config.js";
+import type { GnosysConfig } from "../../config.js";
 /**
  * Let the user pick tasks by number, then provider + model for each.
  * Returns a patch for taskModels + dream, or null if cancelled.
@@ -11,7 +11,3 @@ export declare function runCommaListRoutingEditor(rl: ReadlineInterface, storePa
     taskModels: NonNullable<GnosysConfig["taskModels"]>;
     dream: GnosysConfig["dream"];
 } | null>;
-export declare function applyRoutingPatch(projectDir: string, patch: {
-    taskModels: NonNullable<GnosysConfig["taskModels"]>;
-    dream: GnosysConfig["dream"];
-}): Promise<GnosysConfig>;

package/dist/lib/setup/sections/taskRoutingEditor.js

@@ -1,9 +1,7 @@
 /**
  * Comma-list task routing editor (provider + model per selected task).
  */
-import { loadConfig, updateConfig, } from "../../config.js";
 import { ASSIGNABLE_TASK_LIST, fetchDynamicModels, getAssignableRouting, modelForTaskAssignment, parseCommaSeparatedTaskSelection, pickModel, pickProvider, TASK_DESCRIPTIONS, } from "../../setup.js";
-import { resolveActiveStorePath } from "../storePath.js";
 import { safeQuestion } from "../ui/safePrompt.js";
 import { renderProviderMark } from "../providerGlyphs.js";
 import { c, color, glyph } from "../ui/tokens.js";
@@ -139,11 +137,3 @@ export async function runCommaListRoutingEditor(rl, storePath, cfg) {
     }
     return { taskModels, dream };
 }
-export async function applyRoutingPatch(projectDir, patch) {
-    const storePath = resolveActiveStorePath(projectDir);
-    await updateConfig(storePath, {
-        taskModels: patch.taskModels,
-        dream: patch.dream,
-    });
-    return loadConfig(storePath);
-}

package/dist/lib/setup/ui/header.js

@@ -39,7 +39,6 @@ export function Header(crumbs, opts = {}) {
 }
 /** Strip ANSI escapes so we can measure printable width. Exported for callers/tests that need printable measurement. */
 export function stripAnsi(s) {
-    // eslint-disable-next-line no-control-regex
     return s.replace(/\x1b\[[0-9;]*m/g, "");
 }
 /** Convenience: print header + trailing blank line to stdout. */

package/dist/lib/setup/ui/status.d.ts

@@ -18,4 +18,3 @@ export type StatusKind = "ok" | "warn" | "fail" | "progress";
 export declare function Status(kind: StatusKind, text: string, meta?: string): string;
 /** Convenience: print a single status line. */
 export declare function printStatus(kind: StatusKind, text: string, meta?: string): void;
-export { MASTER_UNREACHABLE_MESSAGE, formatMemoriesWaitingToSync, formatFailedToSyncCount, formatOfflinePushStarting, renderClientSyncStatusLines, type ClientSyncStatusInput, } from "../remoteRender.js";

package/dist/lib/setup/ui/status.js

@@ -46,5 +46,3 @@ export function Status(kind, text, meta) {
 export function printStatus(kind, text, meta) {
     process.stdout.write(`${Status(kind, text, meta)}\n`);
 }
-// ─── v13 multi-machine sync (re-export render helpers for CLI/MCP callers) ─
-export { MASTER_UNREACHABLE_MESSAGE, formatMemoriesWaitingToSync, formatFailedToSyncCount, formatOfflinePushStarting, renderClientSyncStatusLines, } from "../remoteRender.js";

package/dist/lib/setup.d.ts

@@ -138,20 +138,6 @@ export declare function runSetup(opts: {
     directory?: string;
     nonInteractive?: boolean;
 }): Promise<SetupResult>;
-export interface ProviderOnlySetupOpts {
-    directory?: string;
-    rl?: ReadlineInterface;
-}
-/**
- * Update ONLY `llm.defaultProvider` in gnosys.json. Used by the summary
- * panel row 1 ("provider") so it stops dragging the user into the full
- * model picker — that's row 2's job.
- *
- * v5.9.4 Bug 4 — before this split, both summary rows routed through
- * `runModelsSetup`, leaving no way to swap provider without also choosing
- * a new model. Now row 1 picks a provider, row 2 picks a model.
- */
-export declare function runProviderOnlySetup(opts?: ProviderOnlySetupOpts): Promise<void>;
 /** Where validated keys are persisted (§3.10). */
 export type KeyPersistDestination = "secure" | "dotenv" | "none";
 /**
@@ -252,4 +238,3 @@ export declare function getApiKeyForProvider(provider: string, opts?: {
     task?: LlmTaskName;
     directory?: string;
 }): Promise<string>;
-export { getApiKeyForProviderFromConfig } from "./apiKeyVault.js";