gitnexus 1.6.9 → 1.6.10-aptos.0

package/dist/core/lbug/csv-generator.js

@@ -17,7 +17,9 @@ import path from 'path';
 import { NODE_TABLES } from './schema.js';
 import { RelPairRouter } from './rel-pair-routing.js';
 import { parseTruthyEnv } from '../ingestion/utils/env.js';
+import { SYMBOL_NODE_LABELS } from '../ingestion/utils/symbol-labels.js';
 import { applyCjkSegmentationIfEnabled } from '../search/cjk-segmentation.js';
+import { CODE_ELEMENT_COLUMNS, getNodeTableCsvHeader, getNodeTableLayout, hasNodeTableLayout, MULTI_LANG_BASE_COLUMNS, } from './node-table-layout.js';
 /**
  * Deterministic output ordering — optional (out-of-core / windowed-resolve
  * enabler). When `GITNEXUS_SORT_GRAPH_OUTPUT` is set, nodes and relationships
@@ -95,6 +97,35 @@ export const escapeCSVNumber = (value, defaultValue = -1) => {
         return String(defaultValue);
     return String(value);
 };
+/**
+ * LadybugDB STRING[] literal inside a quoted CSV field. Elements containing a
+ * list metacharacter (' , [ ]) are single-quoted so comma-bearing types like
+ * `Table<address, u64>` survive as one element. LadybugDB keeps the quotes on
+ * read, so user-facing columns must strip them (see moveResources fieldList).
+ */
+export const escapeCSVStringArray = (value) => {
+    const items = Array.isArray(value) ? value : [];
+    const encoded = items.map((item) => {
+        const s = item == null ? '' : String(item);
+        return /[',\[\]]/.test(s) ? `'${s.replace(/'/g, "''")}'` : s;
+    });
+    return escapeCSVField(`[${encoded.join(',')}]`);
+};
+// Truthy coercion, matching the incremental CREATE path (`!!properties.isExported`
+// in lbug-adapter) so bulk and incremental loads classify the same values alike.
+export const escapeCSVBoolean = (value) => {
+    return value ? 'true' : 'false';
+};
+const formatCSVStringArray = (value) => {
+    const items = Array.isArray(value)
+        ? value.filter((item) => typeof item === 'string')
+        : [];
+    const unsafe = items.find((item) => /[,\[\]'"\n\r]/.test(item));
+    if (unsafe !== undefined) {
+        throw new Error(`Cannot safely encode CSV string-list item: ${JSON.stringify(unsafe)}`);
+    }
+    return `[${items.join(',')}]`;
+};
 // ============================================================================
 // CONTENT EXTRACTION (lazy — reads from disk on demand)
 // ============================================================================
@@ -187,6 +218,43 @@ class FileContentCache {
 export const normalizeFtsText = (text) => text.replace(/[\r\n\t]+/g, ' ');
 /** Composes both FTS-text transforms for the `description` column — one place for the six emission sites below to call, instead of repeating the composition. */
 const formatFtsDescription = (description) => normalizeFtsText(applyCjkSegmentationIfEnabled(description));
+/**
+ * Encode a row from the same ordered layout that owns its DDL, CSV header,
+ * and COPY column list. These tables are shared across languages, so missing
+ * optional properties deliberately serialize to typed defaults.
+ */
+export const buildLayoutNodeRow = (table, node, content) => {
+    const layout = getNodeTableLayout(table);
+    return layout.columns
+        .map((spec) => {
+        const value = node.properties[spec.name] ?? spec.defaultValue;
+        switch (spec.csvEncoding) {
+            case 'node-id':
+                return escapeCSVField(node.id);
+            case 'number':
+                return escapeCSVNumber(typeof value === 'number' ? value : undefined, typeof spec.defaultValue === 'number' ? spec.defaultValue : -1);
+            case 'boolean':
+                return escapeCSVBoolean(value);
+            case 'string-array':
+                return escapeCSVStringArray(value);
+            case 'content':
+                return escapeCSVField(content);
+            case 'description':
+                return escapeCSVField(formatFtsDescription(String(value ?? '')));
+            case 'string':
+                return escapeCSVField(String(value ?? ''));
+            default: {
+                const _exhaustive = spec.csvEncoding;
+                throw new Error(`Unsupported CSV column encoding: ${String(_exhaustive)}`);
+            }
+        }
+    })
+        .join(',');
+};
+// Labels that get exact source-span content (no ±2 window). Single source of
+// truth in `symbol-labels.ts` — see there for why the exactness depends on the
+// 0-based line invariant. Kept as a named alias to read intent at the use site.
+const EXACT_SYMBOL_CONTENT_LABELS = SYMBOL_NODE_LABELS;
 const extractContent = async (node, contentCache) => {
     const filePath = node.properties.filePath;
     const content = await contentCache.get(filePath);
@@ -209,8 +277,9 @@ const extractContent = async (node, contentCache) => {
     if (startLine === undefined || endLine === undefined)
         return '';
     const lines = content.split('\n');
-    const start = Math.max(0, startLine - 2);
-    const end = Math.min(lines.length - 1, endLine + 2);
+    const exactSymbolContent = EXACT_SYMBOL_CONTENT_LABELS.has(node.label);
+    const start = Math.max(0, exactSymbolContent ? startLine : startLine - 2);
+    const end = Math.min(lines.length - 1, exactSymbolContent ? endLine : endLine + 2);
     const snippet = lines.slice(start, end + 1).join('\n');
     const MAX_SNIPPET = 5000;
     const capped = snippet.length > MAX_SNIPPET ? snippet.slice(0, MAX_SNIPPET) + '\n... [truncated]' : snippet;
@@ -357,9 +426,9 @@ export const streamAllCSVsToDisk = async (graph, repoPath, csvDir, onNodePhaseCo
         // Create writers for every node type up-front
         const fileWriter = new BufferedCSVWriter(path.join(csvDir, 'file.csv'), 'id,name,filePath,content');
         const folderWriter = new BufferedCSVWriter(path.join(csvDir, 'folder.csv'), 'id,name,filePath');
-        const codeElementHeader = 'id,name,filePath,startLine,endLine,isExported,content,description';
-        const functionWriter = new BufferedCSVWriter(path.join(csvDir, 'function.csv'), codeElementHeader);
-        const classWriter = new BufferedCSVWriter(path.join(csvDir, 'class.csv'), codeElementHeader);
+        const codeElementHeader = CODE_ELEMENT_COLUMNS.join(',');
+        const functionWriter = new BufferedCSVWriter(path.join(csvDir, 'function.csv'), getNodeTableCsvHeader('Function'));
+        const classWriter = new BufferedCSVWriter(path.join(csvDir, 'class.csv'), `${codeElementHeader},frameworkAnnotations`);
         const interfaceWriter = new BufferedCSVWriter(path.join(csvDir, 'interface.csv'), codeElementHeader);
         const methodHeader = 'id,name,filePath,startLine,endLine,isExported,content,description,parameterCount,returnType';
         const methodWriter = new BufferedCSVWriter(path.join(csvDir, 'method.csv'), methodHeader);
@@ -376,10 +445,11 @@ export const streamAllCSVsToDisk = async (graph, repoPath, csvDir, onNodePhaseCo
         // blocks are identified by id + source span. Emitted by no phase yet.
         const basicBlockWriter = new BufferedCSVWriter(path.join(csvDir, 'basicblock.csv'), BASICBLOCK_CSV_HEADER);
         // Multi-language node types share the same CSV shape (no isExported column)
-        const multiLangHeader = 'id,name,filePath,startLine,endLine,content,description';
+        const multiLangHeader = MULTI_LANG_BASE_COLUMNS.join(',');
         const MULTI_LANG_TYPES = [
             'Struct',
             'Enum',
+            'EnumVariant',
             'Macro',
             'Typedef',
             'Union',
@@ -401,7 +471,11 @@ export const streamAllCSVsToDisk = async (graph, repoPath, csvDir, onNodePhaseCo
         const propertyHeader = 'id,name,filePath,startLine,endLine,content,description,declaredType';
         const multiLangWriters = new Map();
         for (const t of MULTI_LANG_TYPES) {
-            multiLangWriters.set(t, new BufferedCSVWriter(path.join(csvDir, `${t.toLowerCase()}.csv`), t === 'Property' ? propertyHeader : multiLangHeader));
+            multiLangWriters.set(t, new BufferedCSVWriter(path.join(csvDir, `${t.toLowerCase()}.csv`), t === 'Property'
+                ? propertyHeader
+                : hasNodeTableLayout(t)
+                    ? getNodeTableCsvHeader(t)
+                    : multiLangHeader));
         }
         const codeWriterMap = {
             Function: functionWriter,
@@ -537,35 +611,58 @@ export const streamAllCSVsToDisk = async (graph, repoPath, csvDir, onNodePhaseCo
                     const writer = codeWriterMap[node.label];
                     if (writer) {
                         const content = await extractContent(node, contentCache);
-                        pending = writer.addRow([
-                            escapeCSVField(node.id),
-                            escapeCSVField(node.properties.name || ''),
-                            escapeCSVField(node.properties.filePath || ''),
-                            escapeCSVNumber(node.properties.startLine, -1),
-                            escapeCSVNumber(node.properties.endLine, -1),
-                            node.properties.isExported ? 'true' : 'false',
-                            escapeCSVField(content),
-                            escapeCSVField(formatFtsDescription(node.properties.description || '')),
-                        ].join(','));
-                    }
-                    else {
-                        // Multi-language node types (Struct, Impl, Trait, Macro, etc.)
-                        const mlWriter = multiLangWriters.get(node.label);
-                        if (mlWriter) {
-                            const content = await extractContent(node, contentCache);
-                            pending = mlWriter.addRow([
+                        if (node.label === 'Function') {
+                            pending = writer.addRow(buildLayoutNodeRow('Function', node, content));
+                        }
+                        else if (node.label === 'Class') {
+                            pending = writer.addRow([
                                 escapeCSVField(node.id),
                                 escapeCSVField(node.properties.name || ''),
                                 escapeCSVField(node.properties.filePath || ''),
                                 escapeCSVNumber(node.properties.startLine, -1),
                                 escapeCSVNumber(node.properties.endLine, -1),
+                                node.properties.isExported ? 'true' : 'false',
                                 escapeCSVField(content),
                                 escapeCSVField(formatFtsDescription(node.properties.description || '')),
-                                ...(node.label === 'Property'
-                                    ? [escapeCSVField(node.properties.declaredType || '')]
-                                    : []),
+                                escapeCSVField(formatCSVStringArray(node.properties.frameworkAnnotations)),
                             ].join(','));
                         }
+                        else {
+                            pending = writer.addRow([
+                                escapeCSVField(node.id),
+                                escapeCSVField(node.properties.name || ''),
+                                escapeCSVField(node.properties.filePath || ''),
+                                escapeCSVNumber(node.properties.startLine, -1),
+                                escapeCSVNumber(node.properties.endLine, -1),
+                                node.properties.isExported ? 'true' : 'false',
+                                escapeCSVField(content),
+                                escapeCSVField(formatFtsDescription(node.properties.description || '')),
+                            ].join(','));
+                        }
+                    }
+                    else {
+                        // Multi-language node types (Struct, Impl, Trait, Macro, etc.)
+                        const mlWriter = multiLangWriters.get(node.label);
+                        if (mlWriter) {
+                            const content = await extractContent(node, contentCache);
+                            if (hasNodeTableLayout(node.label)) {
+                                pending = mlWriter.addRow(buildLayoutNodeRow(node.label, node, content));
+                            }
+                            else {
+                                pending = mlWriter.addRow([
+                                    escapeCSVField(node.id),
+                                    escapeCSVField(node.properties.name || ''),
+                                    escapeCSVField(node.properties.filePath || ''),
+                                    escapeCSVNumber(node.properties.startLine, -1),
+                                    escapeCSVNumber(node.properties.endLine, -1),
+                                    escapeCSVField(content),
+                                    escapeCSVField(formatFtsDescription(node.properties.description || '')),
+                                    ...(node.label === 'Property'
+                                        ? [escapeCSVField(node.properties.declaredType || '')]
+                                        : []),
+                                ].join(','));
+                            }
+                        }
                         else {
                             // Unknown label: not in codeWriterMap or multiLangWriters, so there
                             // is no CSV table for it and it is intentionally NOT persisted —

package/dist/core/lbug/cypher-escape.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Escape a value for embedding in a single-quoted Cypher string literal.
+ *
+ * LadybugDB's parser uses backslash escapes and REJECTS SQL-style `''`
+ * doubling — `'we''ird'` is a parser error, not an escaped quote (#2409
+ * review). Every call site that used doubling produced a query that never
+ * parsed; the failures were invisible wherever the site swallowed errors
+ * (per-file deletes skipping quoted paths, importer BFS returning [],
+ * augment/wiki batch lookups silently missing rows).
+ *
+ * Backslashes are escaped first, then quotes — reversing the order would
+ * double the backslash that the quote escape just introduced. For values
+ * inside single-quoted literals only; table/label names go through
+ * `escapeTableName` in lbug-adapter instead.
+ *
+ * NOT for CSV emission: the COPY path (csv-generator.ts) quotes fields for
+ * LadybugDB's CSV reader (`ESCAPE='"'`), a different grammar with its own
+ * rules — its `''` usages are not this bug.
+ */
+export declare const escapeCypherString: (value: string) => string;

package/dist/core/lbug/cypher-escape.js

@@ -0,0 +1,20 @@
+/**
+ * Escape a value for embedding in a single-quoted Cypher string literal.
+ *
+ * LadybugDB's parser uses backslash escapes and REJECTS SQL-style `''`
+ * doubling — `'we''ird'` is a parser error, not an escaped quote (#2409
+ * review). Every call site that used doubling produced a query that never
+ * parsed; the failures were invisible wherever the site swallowed errors
+ * (per-file deletes skipping quoted paths, importer BFS returning [],
+ * augment/wiki batch lookups silently missing rows).
+ *
+ * Backslashes are escaped first, then quotes — reversing the order would
+ * double the backslash that the quote escape just introduced. For values
+ * inside single-quoted literals only; table/label names go through
+ * `escapeTableName` in lbug-adapter instead.
+ *
+ * NOT for CSV emission: the COPY path (csv-generator.ts) quotes fields for
+ * LadybugDB's CSV reader (`ESCAPE='"'`), a different grammar with its own
+ * rules — its `''` usages are not this bug.
+ */
+export const escapeCypherString = (value) => value.replace(/\\/g, '\\\\').replace(/'/g, "\\'");

package/dist/core/lbug/extension-load-error.d.ts

@@ -0,0 +1,67 @@
+export type ExtensionLoadErrorKind = 'missing_file' | 'corrupt_file' | 'missing_dependency' | 'unknown';
+export interface ExtensionLoadDiagnosis {
+    readonly kind: ExtensionLoadErrorKind;
+    /** Actionable, literal-English remedy suited to the class. */
+    readonly remedy: string;
+}
+/**
+ * On-disk file corruption / wrong-platform. FORCE INSTALL re-downloads.
+ * Kept byte-identical to `FILE_CORRUPTION_SIGNATURES` in
+ * scripts/install-duckdb-extension.mjs (that `.mjs` cannot import this `.ts`;
+ * the duplication is deliberate — the two serve different call sites). Note
+ * `/not a valid/i` already covers Windows error 193 ("is not a valid Win32
+ * application"), so a truncated Windows download is caught here, before the
+ * missing-dependency branch.
+ */
+export declare const FILE_CORRUPTION_SIGNATURES: readonly RegExp[];
+/**
+ * Classify a collapsed LadybugDB LOAD error. Order is most-specific-first and is
+ * load-bearing: corrupt-file is tested before missing-dependency so a truncated
+ * Windows download (error 193, matched by `/not a valid/i`) routes to
+ * FORCE-reinstall rather than to the runtime-install remedy.
+ */
+export declare function classifyExtensionLoadError(reason: string | undefined | null, label?: string): ExtensionLoadDiagnosis;
+/** Well-formedness of the extension binary for the host platform + arch. */
+export type ExtensionBinaryState = 'absent' | 'corrupt' | 'valid' | 'indeterminate';
+/**
+ * Pull the extension file path out of lbug's load error. lbug's wrapper is
+ * English regardless of OS language — `Failed to load library: {path} which is
+ * needed by extension: {name}` (real lbug), or the quoted `Failed to load
+ * library '{path}': {reason}` variant — so the path is recoverable in any locale.
+ * Only paths ending in `.lbug_extension` are accepted, so a regex misfire can
+ * never point the inspector at an arbitrary file.
+ */
+export declare function extractExtensionPath(reason: string | undefined | null): string | null;
+/**
+ * A structural verdict on a binary header. `indeterminate` means the probe could
+ * not prove validity OR corruption from what it read (e.g. the PE header sits past
+ * the BINARY_HEADER_BYTES window) — the caller defers to the string classifier
+ * rather than assert a false verdict.
+ */
+type HeaderVerdict = 'valid' | 'corrupt' | 'indeterminate';
+/**
+ * Decide whether a binary header is a well-formed shared library for the given
+ * platform + architecture — using only the file's structure, no localized text.
+ * Pure and injectable (platform/arch as params) so every format+arch combination
+ * is unit-testable regardless of the host it runs on.
+ */
+export declare function classifyBinaryHeader(buf: Buffer, bytesRead: number, platform: NodeJS.Platform, arch: string): HeaderVerdict;
+/**
+ * Best-effort language-independent inspection of the extension file. Reads the
+ * header and classifies it; never throws — a missing file is `absent`, an
+ * unreadable one is `indeterminate`.
+ */
+export declare function inspectExtensionBinary(extensionPath: string | null | undefined): ExtensionBinaryState;
+/**
+ * Diagnose a LadybugDB load failure, preferring a LANGUAGE-INDEPENDENT structural
+ * check of the extension binary over the localized error text:
+ *   - file absent             → missing_file
+ *   - present but malformed    → corrupt_file       (bad magic / wrong architecture)
+ *   - present and well-formed   → missing_dependency (a valid binary the loader rejected)
+ * The path comes from lbug's own English wrapper, so this holds in any OS display
+ * language. When the file cannot be located or read, it falls back to the string
+ * classifier (which still carries the language-independent hedged fallback). This
+ * is the entry point every surface should call.
+ */
+export declare function diagnoseExtensionLoad(reason: string | undefined | null, label?: string): ExtensionLoadDiagnosis;
+export {};

package/dist/core/lbug/extension-load-error.js

@@ -0,0 +1,326 @@
+/**
+ * Classify a LadybugDB `LOAD EXTENSION` failure into one of four actionable
+ * classes and produce an accurate, literal-English remedy.
+ *
+ * Background (#2374): PR #2375 made the real LadybugDB LOAD error visible
+ * (instead of a false "not pre-installed" message). The rc.4 reproduction then
+ * showed the remaining defect — on Windows the extension file downloads and
+ * INSTALLs fine, but `LoadLibrary` fails with error 126 ("the specified module
+ * could not be found" / `找不到指定的模块`) because the extension dynamically
+ * imports OpenSSL 3 / MSVC 14 DLLs that ship nowhere. For that class, telling
+ * the user to reinstall/redownload is wrong — the file is fine; a *runtime
+ * dependency* is missing. This module decides which class an error is so each
+ * surface (doctor, --repair-fts, the analyze degrade warning, and
+ * ftsDegradedWarning) can emit the right remedy instead of a one-size-fits-all
+ * "reinstall over the network".
+ *
+ * `classifyExtensionLoadError` is pure string logic — no `@ladybugdb/core`
+ * import, no filesystem — which keeps `native-check.ts` free of a static lbug
+ * dependency. `diagnoseExtensionLoad` layers a LANGUAGE-INDEPENDENT structural
+ * check on top: it pulls the extension's file path out of lbug's own (English)
+ * wrapper and inspects the binary header directly (PE/ELF/Mach-O magic +
+ * architecture), so corrupt-vs-valid is decided by the file itself, not by the
+ * localized OS error tail. It reads the file (node:fs core module only, still no
+ * lbug) and never throws — any read failure degrades to the string classifier.
+ */
+import { closeSync, openSync, readSync } from 'node:fs';
+/** LadybugDB says the extension file was never installed. INSTALL can heal it. */
+const MISSING_FILE_SIGNATURES = [
+    /has not been installed/i,
+    /not been installed/i,
+];
+/**
+ * On-disk file corruption / wrong-platform. FORCE INSTALL re-downloads.
+ * Kept byte-identical to `FILE_CORRUPTION_SIGNATURES` in
+ * scripts/install-duckdb-extension.mjs (that `.mjs` cannot import this `.ts`;
+ * the duplication is deliberate — the two serve different call sites). Note
+ * `/not a valid/i` already covers Windows error 193 ("is not a valid Win32
+ * application"), so a truncated Windows download is caught here, before the
+ * missing-dependency branch.
+ */
+// Exported so a parity test can assert this stays byte-identical to the copy in
+// scripts/install-duckdb-extension.mjs (that `.mjs` cannot import this `.ts`), #2383 F5b.
+export const FILE_CORRUPTION_SIGNATURES = [
+    /invalid elf/i,
+    /file too short/i,
+    /not a valid/i,
+    /bad magic/i,
+    /wrong architecture/i,
+    /mach-o/i,
+    /truncat/i,
+];
+/**
+ * A *transitive dependency* of the extension is missing — the file loaded far
+ * enough to be found, but a library it needs is absent. Reinstalling the
+ * extension is a no-op for this class.
+ *
+ * WINDOWS CATCH-ALL GUARD (adversarial review): LadybugDB wraps *every* Windows
+ * load failure in `Failed to load library … which is needed by extension`, so
+ * that generic wrapper must NOT be sufficient — otherwise error 127 (wrong
+ * OpenSSL minor / unresolved procedure), 5 (AV/permission lock), and 1114
+ * (dependency DllMain failure) would all be mislabeled `missing_dependency` and
+ * told to install a runtime, the opposite of their real fix. We key strictly on
+ * the specific error-126 tail. Linux/macOS loaders name the missing library
+ * directly, so their signals are unambiguous.
+ *
+ * Localized Windows tails we do not enumerate (French, German, Japanese, …) and
+ * mojibake renderings of the Chinese text won't match here — but they still
+ * carry lbug's language-independent `Failed to load library` wrapper, so they
+ * are caught by the hedged fallback (LOAD_FAILURE_WRAPPER) with a non-committal
+ * remedy, never a wrong confident "reinstall" instruction.
+ */
+const WINDOWS_MISSING_DEPENDENCY_SIGNATURES = [
+    /找不到指定的模块/,
+    /specified module could not be found/i,
+];
+const POSIX_MISSING_DEPENDENCY_SIGNATURES = [
+    /cannot open shared object file/i, // Linux ld.so
+    /image not found/i, // macOS dyld
+    /library not loaded/i, // macOS dyld
+];
+/**
+ * LadybugDB's own English wrapper for a dlopen/LoadLibrary failure
+ * (extension.cpp: `Failed to load library: {path} which is needed by extension:
+ * {name}`). It is emitted for EVERY extension load failure regardless of the OS
+ * display language — the only localized part is the OS-error tail after it. So
+ * it is the language-independent fallback signal once the specific tails miss: a
+ * French/German/Japanese Windows 126 has a localized tail we cannot enumerate,
+ * but it still carries this wrapper. See hedgedLoadFailureRemedy.
+ */
+const LOAD_FAILURE_WRAPPER = /failed to load library/i;
+// Remedies are label-parameterized (#2623 follow-up): doctor now live-probes
+// VECTOR through the same classifier, and FTS-specific advice (`--repair-fts`
+// repairs FTS indexes only) must not be dispensed for other extensions.
+const repairFtsHint = (label, lead) => label === 'FTS' ? ` (${lead}\`gitnexus analyze --repair-fts\`)` : '';
+const missingFileRemedy = (label) => `The ${label} extension is not installed. Re-run with network access and ` +
+    `GITNEXUS_LBUG_EXTENSION_INSTALL=auto${repairFtsHint(label, 'or ')} to download it.`;
+const corruptFileRemedy = (label) => `The ${label} extension file is present but unreadable (corrupt, truncated, or built for another ` +
+    `platform). Re-download it with network access and ` +
+    `GITNEXUS_LBUG_EXTENSION_INSTALL=auto${repairFtsHint(label, '')}.`;
+// Single source of truth for the VC++ runtime-install pointer, shared by the
+// Windows-126 and structural missing-dependency remedies so the name/URL cannot
+// drift between them (#2383 F5).
+const VC_REDIST_INSTALL_HINT = 'the Microsoft Visual C++ 2015-2022 Redistributable (x64) from ' +
+    'https://aka.ms/vs/17/release/vc_redist.x64.exe';
+// MSVC-first per DuckDB's canonical answer for this exact error; OpenSSL second.
+const windowsMissingDependencyRemedy = (label) => `The ${label} extension is present but a required runtime library is missing (Windows error 126). ` +
+    'Reinstalling the extension will NOT help. Install ' +
+    VC_REDIST_INSTALL_HINT +
+    '; if the error persists, the extension also needs OpenSSL 3 ' +
+    '(libcrypto-3-x64.dll / libssl-3-x64.dll) on the DLL search path.';
+const posixMissingDependencyRemedy = (label) => `The ${label} extension is present but a shared library it depends on could not be loaded (named in ` +
+    'the error above). Reinstalling the extension will NOT help — install that library or add it to ' +
+    'your loader search path.';
+// Language-independent fallback: we know the extension failed to load, but the
+// OS-error tail is in a locale we did not enumerate, so we cannot say which class
+// it is. Hedge honestly — point at the user's own localized error and give both
+// branches — rather than confidently prescribing the wrong single fix. The clean
+// long-term fix is upstream: have LadybugDB include the numeric GetLastError/errno
+// in the message (as it already does elsewhere), so this becomes a code match.
+const hedgedLoadFailureRemedy = (label) => `The ${label} extension file was found but could not be loaded — see the "Error:" text above (shown ` +
+    "in your system's language). Reinstalling usually will not help. If it names a missing module or " +
+    'library, install the required runtime (on Windows: the Microsoft Visual C++ 2015-2022 ' +
+    'Redistributable x64 and OpenSSL 3); if it names a corrupt or invalid file, ' +
+    (label === 'FTS'
+        ? 'run `gitnexus analyze --repair-fts` to re-download.'
+        : 're-run analyze with network access and GITNEXUS_LBUG_EXTENSION_INSTALL=auto to re-download.');
+const unknownRemedy = (label) => `The ${label} extension failed to load for an unrecognized reason. Run \`gitnexus doctor\` for live ` +
+    `${label} status and verify the extension file and platform.`;
+const matchesAny = (reason, signatures) => signatures.some((re) => re.test(reason));
+/**
+ * Classify a collapsed LadybugDB LOAD error. Order is most-specific-first and is
+ * load-bearing: corrupt-file is tested before missing-dependency so a truncated
+ * Windows download (error 193, matched by `/not a valid/i`) routes to
+ * FORCE-reinstall rather than to the runtime-install remedy.
+ */
+export function classifyExtensionLoadError(reason, label = 'FTS') {
+    const text = reason ?? '';
+    if (matchesAny(text, MISSING_FILE_SIGNATURES)) {
+        return { kind: 'missing_file', remedy: missingFileRemedy(label) };
+    }
+    if (matchesAny(text, FILE_CORRUPTION_SIGNATURES)) {
+        return { kind: 'corrupt_file', remedy: corruptFileRemedy(label) };
+    }
+    if (matchesAny(text, WINDOWS_MISSING_DEPENDENCY_SIGNATURES)) {
+        return { kind: 'missing_dependency', remedy: windowsMissingDependencyRemedy(label) };
+    }
+    if (matchesAny(text, POSIX_MISSING_DEPENDENCY_SIGNATURES)) {
+        return { kind: 'missing_dependency', remedy: posixMissingDependencyRemedy(label) };
+    }
+    // Language-independent fallback: the extension demonstrably failed to load
+    // (lbug's English wrapper is present) but the localized OS tail matched no
+    // specific class. Treat as a dependency/runtime load failure with a hedged
+    // remedy — strictly better than the generic `unknown` for non-English hosts,
+    // and it never prescribes the wrong fix.
+    if (LOAD_FAILURE_WRAPPER.test(text)) {
+        return { kind: 'missing_dependency', remedy: hedgedLoadFailureRemedy(label) };
+    }
+    return { kind: 'unknown', remedy: unknownRemedy(label) };
+}
+const structuralMissingDependencyRemedy = (label) => `The ${label} extension file is valid, so the failure is a missing or incompatible runtime dependency, ` +
+    'not the extension itself — reinstalling will NOT help. On Windows, install ' +
+    VC_REDIST_INSTALL_HINT +
+    ' and ensure OpenSSL 3 is available; on Linux/macOS install the shared library named in the error above.';
+/**
+ * Pull the extension file path out of lbug's load error. lbug's wrapper is
+ * English regardless of OS language — `Failed to load library: {path} which is
+ * needed by extension: {name}` (real lbug), or the quoted `Failed to load
+ * library '{path}': {reason}` variant — so the path is recoverable in any locale.
+ * Only paths ending in `.lbug_extension` are accepted, so a regex misfire can
+ * never point the inspector at an arbitrary file.
+ */
+export function extractExtensionPath(reason) {
+    const text = reason ?? '';
+    const m = /failed to load library:?\s*['"]?(.+?\.lbug_extension)/i.exec(text);
+    const path = m?.[1]?.trim();
+    return path && path.length > 0 ? path : null;
+}
+/** Node `process.arch` → PE `Machine`. Undefined for arches we don't map. */
+const PE_MACHINE = { x64: 0x8664, arm64: 0xaa64 };
+/** Node `process.arch` → ELF `e_machine`. */
+const ELF_MACHINE = { x64: 0x3e, arm64: 0xb7 };
+/** Node `process.arch` → Mach-O `cputype`. */
+const MACHO_CPUTYPE = { x64: 0x01000007, arm64: 0x0100000c };
+function classifyPE(buf, bytesRead, arch) {
+    if (bytesRead < 0x40 || buf[0] !== 0x4d || buf[1] !== 0x5a)
+        return 'corrupt'; // 'MZ'
+    const peOffset = buf.readUInt32LE(0x3c);
+    // The PE header (e_lfanew) points beyond what we read. A large-DOS-stub VALID PE
+    // and a garbage e_lfanew are indistinguishable from here, so don't claim 'corrupt'
+    // — defer to the loader's own report (#2383 F1-secondary).
+    if (peOffset + 6 > bytesRead)
+        return 'indeterminate';
+    const isPE = buf[peOffset] === 0x50 &&
+        buf[peOffset + 1] === 0x45 &&
+        buf[peOffset + 2] === 0 &&
+        buf[peOffset + 3] === 0;
+    if (!isPE)
+        return 'corrupt';
+    const expected = PE_MACHINE[arch];
+    if (expected === undefined)
+        return 'valid'; // arch we don't map: don't claim corrupt
+    return buf.readUInt16LE(peOffset + 4) === expected ? 'valid' : 'corrupt';
+}
+function classifyELF(buf, bytesRead, arch) {
+    if (bytesRead < 20)
+        return 'corrupt';
+    if (buf[0] !== 0x7f || buf[1] !== 0x45 || buf[2] !== 0x4c || buf[3] !== 0x46)
+        return 'corrupt'; // 0x7F ELF
+    const littleEndian = buf[5] === 1; // EI_DATA
+    const eMachine = littleEndian ? buf.readUInt16LE(18) : buf.readUInt16BE(18);
+    const expected = ELF_MACHINE[arch];
+    if (expected === undefined)
+        return 'valid';
+    return eMachine === expected ? 'valid' : 'corrupt';
+}
+function classifyMachO(buf, bytesRead, arch) {
+    if (bytesRead < 8)
+        return 'corrupt';
+    const magicLE = buf.readUInt32LE(0);
+    const magicBE = buf.readUInt32BE(0);
+    // Universal ("fat") binary — assume it carries the host slice.
+    if (magicBE === 0xcafebabe || magicLE === 0xcafebabe)
+        return 'valid';
+    const thin = magicLE === 0xfeedfacf || magicLE === 0xfeedface;
+    const thinSwapped = magicBE === 0xfeedfacf || magicBE === 0xfeedface;
+    if (!thin && !thinSwapped)
+        return 'corrupt';
+    const cpuType = thin ? buf.readUInt32LE(4) : buf.readUInt32BE(4);
+    const expected = MACHO_CPUTYPE[arch];
+    if (expected === undefined)
+        return 'valid';
+    return cpuType === expected ? 'valid' : 'corrupt';
+}
+/**
+ * Decide whether a binary header is a well-formed shared library for the given
+ * platform + architecture — using only the file's structure, no localized text.
+ * Pure and injectable (platform/arch as params) so every format+arch combination
+ * is unit-testable regardless of the host it runs on.
+ */
+export function classifyBinaryHeader(buf, bytesRead, platform, arch) {
+    if (platform === 'win32')
+        return classifyPE(buf, bytesRead, arch);
+    if (platform === 'linux')
+        return classifyELF(buf, bytesRead, arch);
+    if (platform === 'darwin')
+        return classifyMachO(buf, bytesRead, arch);
+    return 'valid'; // unknown host: never claim corrupt
+}
+const BINARY_HEADER_BYTES = 4096;
+/**
+ * Best-effort language-independent inspection of the extension file. Reads the
+ * header and classifies it; never throws — a missing file is `absent`, an
+ * unreadable one is `indeterminate`.
+ */
+export function inspectExtensionBinary(extensionPath) {
+    if (!extensionPath)
+        return 'indeterminate';
+    let fd;
+    try {
+        fd = openSync(extensionPath, 'r');
+    }
+    catch (err) {
+        return err?.code === 'ENOENT' ? 'absent' : 'indeterminate';
+    }
+    try {
+        const buf = Buffer.alloc(BINARY_HEADER_BYTES);
+        const bytesRead = readSync(fd, buf, 0, BINARY_HEADER_BYTES, 0);
+        return classifyBinaryHeader(buf, bytesRead, process.platform, process.arch);
+    }
+    catch {
+        return 'indeterminate';
+    }
+    finally {
+        try {
+            closeSync(fd);
+        }
+        catch {
+            /* closing the probe fd must never surface */
+        }
+    }
+}
+/**
+ * Diagnose a LadybugDB load failure, preferring a LANGUAGE-INDEPENDENT structural
+ * check of the extension binary over the localized error text:
+ *   - file absent             → missing_file
+ *   - present but malformed    → corrupt_file       (bad magic / wrong architecture)
+ *   - present and well-formed   → missing_dependency (a valid binary the loader rejected)
+ * The path comes from lbug's own English wrapper, so this holds in any OS display
+ * language. When the file cannot be located or read, it falls back to the string
+ * classifier (which still carries the language-independent hedged fallback). This
+ * is the entry point every surface should call.
+ */
+export function diagnoseExtensionLoad(reason, label = 'FTS') {
+    const text = reason ?? '';
+    const stringResult = classifyExtensionLoadError(text, label);
+    const fileState = inspectExtensionBinary(extractExtensionPath(text));
+    if (fileState === 'corrupt') {
+        return { kind: 'corrupt_file', remedy: corruptFileRemedy(label) };
+    }
+    if (fileState === 'valid') {
+        // The structural probe only inspects the first BINARY_HEADER_BYTES, so a file
+        // truncated AFTER its header still reads 'valid'. When the loader itself reported
+        // corruption (e.g. "file too short" / Windows error 193 "not a valid Win32
+        // application"), that whole-file verdict is stronger evidence than an intact-looking
+        // header — honor it and route to re-download, not a runtime-dependency install (#2383
+        // F1). Localized corrupt tails classify as hedged missing_dependency (not
+        // corrupt_file), so they still fall through to the dependency remedy below.
+        if (stringResult.kind === 'corrupt_file') {
+            return stringResult;
+        }
+        // A structurally sound binary that still failed to load ⇒ a dependency/runtime
+        // problem, decided WITHOUT the localized tail. Keep the string classifier's
+        // sharper remedy when it recognized the specific case (e.g. English 126).
+        const remedy = stringResult.kind === 'missing_dependency'
+            ? stringResult.remedy
+            : structuralMissingDependencyRemedy(label);
+        return { kind: 'missing_dependency', remedy };
+    }
+    // 'absent' or 'indeterminate' → no positive structural evidence, so defer to the
+    // string classifier. Note a real never-installed extension has NO path in its
+    // reason (lbug says "has not been installed"), so it lands here via
+    // 'indeterminate' and the string classifier reports missing_file correctly; a
+    // path that lbug named but that is now gone (stale/racy) is better judged by
+    // what lbug actually reported than by re-deriving from disk.
+    return stringResult;
+}