@gmickel/gno 1.37.0 → 1.38.0

package/browser-extension/artifacts/gno-browser-clipper-v1.38.0.zip.sha256

@@ -0,0 +1 @@
+59f8dc960d1910f7b9dec3c4fea286c46bfe63487a43212fbb8ba61a207753ae  gno-browser-clipper-v1.38.0.zip

package/browser-extension/dist/manifest.json

@@ -21,5 +21,5 @@
   "content_security_policy": {
     "extension_pages": "script-src 'self'; object-src 'none'; connect-src http://127.0.0.1:*"
   },
-  "version": "1.37.0"
+  "version": "1.38.0"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gmickel/gno",
-  "version": "1.37.0",
+  "version": "1.38.0",
   "description": "Local semantic search for your documents. Index Markdown, PDF, and Office files with hybrid BM25 + vector search.",
   "keywords": [
     "embeddings",

package/spec/cli.md

@@ -15,6 +15,7 @@ This document specifies the command-line interface for GNO, a local knowledge in
 | 1    | VALIDATION  | Validation or usage error (bad args, missing required params) |
 | 2    | RUNTIME     | Runtime failure (IO, DB, conversion, model, network)          |
 | 3    | NOT_RUNNING | `--status`/`--stop` found no live matching process            |
+| 4    | BUSY        | Write-lease contention on `index` / `update` / `embed`        |
 
 ### Global Flags
 
@@ -957,7 +958,7 @@ Sync files from disk into the index (ingestion without embedding).
 **Synopsis:**
 
 ```bash
-gno update [--git-pull] [--json]
+gno update [--git-pull] [--json] [--lock-wait <duration>] [--no-wait]
 ```
 
 **Options:**
@@ -965,6 +966,10 @@ gno update [--git-pull] [--json]
 |--------|------|-------------|
 | `--git-pull` | boolean | Run `git pull` in git repositories before scanning |
 | `--json` | boolean | Emit the complete deterministic sync result on stdout |
+| `--lock-wait <duration>` | duration | How long to wait for the index write lease (default: `120s`). Accepts `120`, `120s`, or `2m`. |
+| `--no-wait` | boolean | Do not wait; fail immediately if another writer holds the lease |
+
+**Concurrency:** One writer at a time on the shared index database. `update` waits up to `--lock-wait` for the lease (the same `.mcp-write.lock` MCP write tools use); `index`, `embed`, `cleanup`, `vec sync`, `vec rebuild`, `collection clear-embeddings`, `tags add`, and `tags rm` follow the same contract; `capture` takes the same lock internally, and single-row writes such as `collection policy set` are absorbed by `busy_timeout`. `--no-wait` opts out. Reads (`search`, `query`, `get`) never take the lease. External serialising wrappers are no longer required for CLI-vs-CLI and CLI-vs-MCP overlap. Residual window: a resident (`gno serve`/`gno daemon`) watch or embed flush writes without the lease; those short transactions are absorbed by the raised `busy_timeout` and the SQLITE_BUSY retry, and a deferred chunk is reported as contention, never as an embedding failure.
 
 **Behavior:**
 
@@ -986,6 +991,7 @@ record failure was reported. Human progress and diagnostics remain on stderr.
 
 - 0: Success (conversion warnings do not affect exit code)
 - 2: DB failure or critical IO error
+- 4: Write lease busy (contention after `--lock-wait`, or immediately with `--no-wait`)
 
 ---
 
@@ -996,7 +1002,7 @@ Build or update the index end-to-end (update + embed).
 **Synopsis:**
 
 ```bash
-gno index [collection] [--no-embed] [--models-pull] [--git-pull] [--json] [--yes]
+gno index [collection] [--no-embed] [--models-pull] [--git-pull] [--json] [--yes] [--lock-wait <duration>] [--no-wait]
 ```
 
 **Options:**
@@ -1008,24 +1014,29 @@ gno index [collection] [--no-embed] [--models-pull] [--git-pull] [--json] [--yes
 | `--git-pull` | boolean | Run `git pull` in git repositories |
 | `--json` | boolean | Emit the complete deterministic sync and embedding result on stdout |
 | `--yes` | boolean | Accept defaults, no prompts |
+| `--lock-wait <duration>` | duration | How long to wait for the index write lease (default: `120s`). Accepts `120`, `120s`, or `2m`. |
+| `--no-wait` | boolean | Do not wait; fail immediately if another writer holds the lease |
 
 **Behavior:**
 
 - Runs `update` then `embed` by default
 - With `--no-embed`, runs `update` only
+- Waits by default for the shared write lease; `--no-wait` fails immediately with exit 4
 
 **JSON output:** Emits `{ syncResult, embedSkipped, embedResult? }`.
 `syncResult.collections[].files[].recordImport`, when present, conforms to
 [`record-import@1.0`](./output-schemas/record-import.schema.json) with the same
 deterministic ordering, bounds, truncation disclosure, and partial-snapshot
 warnings as `gno update --json`. Human progress and diagnostics remain on
-stderr.
+stderr. On lease timeout, stdout is one object `{ success: false, error, contention }`
+and the process exits 4.
 
 **Exit Codes:**
 
 - 0: Success
-- 1: Invalid collection name
+- 1: Invalid collection name or invalid `--lock-wait`
 - 2: DB or model failure
+- 4: Write lease busy (contention, not corruption)
 
 **Examples:**
 
@@ -1060,25 +1071,30 @@ memory pressure prevents creating the full pool.
 **Synopsis:**
 
 ```bash
-gno embed [--force] [--model <uri>] [--batch-size <n>] [--dry-run] [--yes] [--json]
+gno embed [--force] [--model <uri>] [--batch-size <n>] [--dry-run] [--yes] [--json] [--lock-wait <duration>] [--no-wait]
 ```
 
 **Options:**
 
-| Option         | Type    | Default | Description                                   |
-| -------------- | ------- | ------- | --------------------------------------------- |
-| `--force`      | boolean | false   | Re-embed all chunks (ignore existing vectors) |
-| `--model`      | string  | config  | Override embedding model URI                  |
-| `--batch-size` | integer | 32      | Chunks per batch                              |
-| `--dry-run`    | boolean | false   | Show what would be embedded without doing it  |
-| `--yes`, `-y`  | boolean | false   | Skip confirmation prompts                     |
-| `--json`       | boolean | false   | Output result as JSON                         |
+| Option                   | Type     | Default | Description                                                                 |
+| ------------------------ | -------- | ------- | --------------------------------------------------------------------------- |
+| `--force`                | boolean  | false   | Re-embed all chunks (ignore existing vectors)                               |
+| `--model`                | string   | config  | Override embedding model URI                                                |
+| `--batch-size`           | integer  | 32      | Chunks per batch                                                            |
+| `--dry-run`              | boolean  | false   | Show what would be embedded without doing it                                |
+| `--yes`, `-y`            | boolean  | false   | Skip confirmation prompts                                                   |
+| `--json`                 | boolean  | false   | Output result as JSON                                                       |
+| `--lock-wait <duration>` | duration | `120s`  | How long to wait for the index write lease. Accepts `120`, `120s`, or `2m`. |
+| `--no-wait`              | boolean  | false   | Do not wait; fail immediately if another writer holds the lease             |
+
+Waits by default for the same write lease as `index` / `update` / MCP writers. `--no-wait` opts out.
 
 **Exit Codes:**
 
 - 0: Success
-- 1: User cancelled
+- 1: User cancelled or invalid `--lock-wait`
 - 2: Model not available or embedding failure
+- 4: Write lease busy (contention, not corruption)
 
 **JSON Output:**
 
@@ -3615,7 +3631,11 @@ Errors are written to stderr. With `--json` flag, errors are also returned as:
 }
 ```
 
-Error codes match exit codes: `VALIDATION` (exit 1), `RUNTIME` (exit 2), `NOT_RUNNING` (exit 3).
+Error codes match exit codes: `VALIDATION` (exit 1), `RUNTIME` (exit 2), `NOT_RUNNING` (exit 3), `BUSY` (exit 4).
+
+Write-lease contention on `index` / `update` / `embed` does not use the generic envelope. Text mode writes the dedicated "index is busy" message to stderr; `--json` writes `{ success: false, error, contention }` to stdout. Both exit 4. `gno audit` also uses exit 4 for findings.
+
+**`NOT_RUNNING` is not an error envelope.**
 
 **`NOT_RUNNING` is not an error envelope.** `gno serve|daemon --status --json` returns a `process-status`-shaped payload on stdout with exit 3 when no live matching process is found (it reports observable state, not failure). `--stop` exits 3 silently when there is nothing to stop and does not accept `--json`. The error envelope above is reserved for `VALIDATION` and `RUNTIME` failures where the command could not produce its structured output at all.
 

package/src/cli/commands/cleanup.ts

@@ -17,6 +17,8 @@ import { SqliteAdapter } from "../../store/sqlite/adapter";
 export interface CleanupOptions {
   /** Override config path */
   configPath?: string;
+  /** Index name */
+  indexName?: string;
 }
 
 /**
@@ -47,9 +49,13 @@ export async function cleanup(
 
   // Open database
   const store = new SqliteAdapter();
-  const dbPath = getIndexDbPath();
+  const dbPath = getIndexDbPath(options.indexName);
 
-  const openResult = await store.open(dbPath, config.ftsTokenizer);
+  const openResult = await store.open(
+    dbPath,
+    config.ftsTokenizer,
+    config.busyTimeoutMs
+  );
   if (!openResult.ok) {
     return { success: false, error: openResult.error.message };
   }

package/src/cli/commands/collection/clear-embeddings.ts

@@ -10,6 +10,7 @@ import { CliError } from "../../errors";
 
 interface ClearEmbeddingsOptions {
   all?: boolean;
+  indexName?: string;
   json?: boolean;
 }
 
@@ -39,7 +40,11 @@ export async function collectionClearEmbeddings(
   }
 
   const store = new SqliteAdapter();
-  const openResult = await store.open(getIndexDbPath(), config.ftsTokenizer);
+  const openResult = await store.open(
+    getIndexDbPath(options.indexName),
+    config.ftsTokenizer,
+    config.busyTimeoutMs
+  );
   if (!openResult.ok) {
     throw new CliError("RUNTIME", openResult.error.message);
   }

package/src/cli/commands/doctor-activation.ts

@@ -43,7 +43,11 @@ export async function buildDoctorActivation(
 
   const store = new SqliteAdapter();
   store.setConfigPath(options.configPath ?? "");
-  const opened = await store.open(dbPath, config.ftsTokenizer);
+  const opened = await store.open(
+    dbPath,
+    config.ftsTokenizer,
+    config.busyTimeoutMs
+  );
   if (!opened.ok) {
     return unavailableActivation(config);
   }

package/src/cli/commands/doctor.ts

@@ -14,7 +14,12 @@ import type { Config } from "../../config/types";
 import type { ActivationStatus } from "../../core/activation-status";
 
 import { getIndexDbPath, getModelsCachePath } from "../../app/constants";
-import { getConfigPaths, isInitialized, loadConfig } from "../../config";
+import {
+  DEFAULT_BUSY_TIMEOUT_MS,
+  getConfigPaths,
+  isInitialized,
+  loadConfig,
+} from "../../config";
 import { isConnectorActivationComplete } from "../../core/activation-connector-health";
 import { getCodeChunkingStatus } from "../../ingestion/chunker";
 import { ModelCache } from "../../llm/cache";
@@ -140,6 +145,64 @@ async function checkDatabase(indexName?: string): Promise<DoctorCheck> {
   }
 }
 
+/**
+ * Report the live SQLite busy_timeout (PRAGMA), not the config echo.
+ */
+async function checkBusyTimeout(
+  config: Config,
+  indexName?: string
+): Promise<DoctorCheck> {
+  const dbPath = getIndexDbPath(indexName);
+
+  try {
+    await stat(dbPath);
+  } catch {
+    return {
+      name: "busy-timeout",
+      status: "warn",
+      message: "Database not found. Run: gno init",
+    };
+  }
+
+  const store = new SqliteAdapter();
+  const paths = getConfigPaths();
+  store.setConfigPath(paths.configFile);
+
+  const openResult = await store.open(
+    dbPath,
+    config.ftsTokenizer,
+    config.busyTimeoutMs ?? DEFAULT_BUSY_TIMEOUT_MS
+  );
+  if (!openResult.ok) {
+    return {
+      name: "busy-timeout",
+      status: "warn",
+      message: `busy_timeout unavailable: ${openResult.error.message}`,
+    };
+  }
+
+  try {
+    const timeout = store
+      .getRawDb()
+      .query<{ timeout: number }, []>("PRAGMA busy_timeout")
+      .get()?.timeout;
+    if (timeout === undefined) {
+      return {
+        name: "busy-timeout",
+        status: "warn",
+        message: "PRAGMA busy_timeout returned no value",
+      };
+    }
+    return {
+      name: "busy-timeout",
+      status: "ok",
+      message: `busy_timeout ${timeout}ms`,
+    };
+  } finally {
+    await store.close();
+  }
+}
+
 async function checkModels(config: Config): Promise<DoctorCheck[]> {
   const checks: DoctorCheck[] = [];
   const cache = new ModelCache(getModelsCachePath());
@@ -209,7 +272,11 @@ async function checkEmbeddingFingerprints(
   const paths = getConfigPaths();
   store.setConfigPath(paths.configFile);
 
-  const openResult = await store.open(dbPath, config.ftsTokenizer);
+  const openResult = await store.open(
+    dbPath,
+    config.ftsTokenizer,
+    config.busyTimeoutMs ?? DEFAULT_BUSY_TIMEOUT_MS
+  );
   if (!openResult.ok) {
     return {
       name: "embedding-fingerprint",
@@ -527,6 +594,9 @@ export async function doctor(
   const sqliteChecks = await checkSqliteExtensions();
   checks.push(...sqliteChecks);
 
+  // Live busy_timeout from the open index (not the config echo)
+  checks.push(await checkBusyTimeout(config, options.indexName));
+
   // Code chunking capability
   checks.push(checkCodeChunking());