@hawkeyexl/manni 1.0.0 → 1.0.1

package/dist/{chunk-L7HN3AWA.js → chunk-IRHWILCO.js}

@@ -3,7 +3,7 @@
 // package.json
 var package_default = {
   name: "@hawkeyexl/manni",
-  version: "1.0.0",
+  version: "1.0.1",
   description: "One command line for documentation that is meant to be checked. `manni meta` validates the presence and format of document metadata (Markdown frontmatter and more) against JSON Schema, built for CI. Formerly docmeta.",
   type: "module",
   bin: {
@@ -396,6 +396,19 @@ var ToolError = class extends Error {
     this.name = "ToolError";
   }
 };
+function errorMessage(err) {
+  if (err instanceof Error) return err.message;
+  if (typeof err === "string") return err;
+  if (typeof err === "object" && err !== null && "message" in err && typeof err.message === "string") {
+    return err.message;
+  }
+  try {
+    const json = JSON.stringify(err);
+    if (json !== void 0) return json;
+  } catch {
+  }
+  return String(err);
+}
 
 // src/meta/types.ts
 var DocmetaError = class extends ToolError {
@@ -3113,7 +3126,7 @@ async function requestSchema(ref, timeoutMs, timedOut) {
   } catch (err) {
     if (isAbort(err)) throw timedOut;
     throw new RetryableFetchError(
-      `Failed to fetch schema "${ref}": ${err.message}`
+      `Failed to fetch schema "${ref}": ${errorMessage(err)}`
     );
   }
   if (!res.ok) {
@@ -3145,7 +3158,7 @@ async function fetchSchemaBytes(ref, options = {}) {
     if (err instanceof DocmetaError) throw err;
     if (isAbort(err)) throw timedOut;
     throw new DocmetaError(
-      `Failed to fetch schema "${ref}": ${err.message}`
+      `Failed to fetch schema "${ref}": ${errorMessage(err)}`
     );
   }
   const raw = stripBom(bytes.toString("utf8"));
@@ -3154,7 +3167,7 @@ async function fetchSchemaBytes(ref, options = {}) {
     parsed2 = JSON.parse(raw);
   } catch (err) {
     throw new DocmetaError(
-      `Schema "${ref}" did not return valid JSON: ${err.message}`
+      `Schema "${ref}" did not return valid JSON: ${errorMessage(err)}`
     );
   }
   return { bytes, schema: assertFetchedSchema(ref, parsed2, raw) };
@@ -3267,7 +3280,7 @@ async function loadSchema(ref, options = {}) {
     return JSON.parse(text);
   } catch (err) {
     throw new DocmetaError(
-      `Schema file "${ref}" is not valid JSON: ${withoutFileExcerpt(err.message, text)}`
+      `Schema file "${ref}" is not valid JSON: ${withoutFileExcerpt(errorMessage(err), text)}`
     );
   }
 }
@@ -5179,7 +5192,7 @@ function applyXml(original, patch, options = {}) {
     before = readXml(original, options.filePath, { elements: options.elements });
   } catch (err) {
     throw new DocmetaError(
-      `Refusing to write metadata: ${err.message}`
+      `Refusing to write metadata: ${errorMessage(err)}`
     );
   }
   const content = before.body;
@@ -5314,7 +5327,7 @@ function verify2(next, before, patch, options) {
     }).data;
   } catch (err) {
     throw new DocmetaError(
-      `Refusing to write XML metadata: the rewritten document did not parse (${err.message}).`
+      `Refusing to write XML metadata: the rewritten document did not parse (${errorMessage(err)}).`
     );
   }
   if (!deepEqual(actual, expected)) {
@@ -6253,7 +6266,7 @@ function parseMapping(text, source, opts) {
   try {
     raw = parseYaml(text);
   } catch (err) {
-    throw opts.toError(`${source}: invalid YAML: ${err.message}`);
+    throw opts.toError(`${source}: invalid YAML: ${errorMessage(err)}`);
   }
   if (raw == null) return null;
   if (typeof raw !== "object" || Array.isArray(raw)) {
@@ -6365,7 +6378,7 @@ function asElementPaths(value, where, source) {
     try {
       parseElementPath(path);
     } catch (err) {
-      throw new DocmetaError(`${source}: ${where} \u2014 ${err.message}`);
+      throw new DocmetaError(`${source}: ${where} \u2014 ${errorMessage(err)}`);
     }
   }
   return list;
@@ -6502,7 +6515,7 @@ function parseConfig(text, source) {
     raw = parseYaml2(text);
   } catch (err) {
     throw new DocmetaError(
-      `${source}: invalid YAML: ${err.message}`
+      `${source}: invalid YAML: ${errorMessage(err)}`
     );
   }
   return parseConfigValue(raw, source);
@@ -6564,13 +6577,20 @@ function assertNoMovedKeys(raw, source) {
     }
   }
 }
-function assertOverrideCollections(config, collections, source) {
+function inSection(message, source, section) {
+  return section === void 0 ? message : withSection(message, source, section);
+}
+function assertOverrideCollections(config, collections, source, section) {
   const names = new Set(collections.map((c) => c.name));
   for (const [i, ov] of (config.overrides ?? []).entries()) {
     if (ov.collection === void 0 || names.has(ov.collection)) continue;
     const defined = collections.length === 0 ? "(none)" : collections.map((c) => c.name).join(", ");
     throw new DocmetaError(
-      `${source}: overrides[${i}].collection names "${ov.collection}", which collections: does not define. Defined: ${defined}.`
+      inSection(
+        `${source}: overrides[${i}].collection names "${ov.collection}", which collections: does not define. Defined: ${defined}.`,
+        source,
+        section
+      )
     );
   }
 }
@@ -6805,7 +6825,7 @@ async function loadConfig(explicitPath, cwd = process.cwd()) {
   if (file === null) return null;
   const section = file.wrapped ? META_SECTION : void 0;
   const config = parseConfigValue(file.value, file.source, section);
-  assertOverrideCollections(config, file.collections, file.source);
+  assertOverrideCollections(config, file.collections, file.source, section);
   return {
     config,
     path: file.path,
@@ -6900,16 +6920,22 @@ function createDocsTable(db, entries, dataColumns) {
     `INSERT INTO docs VALUES (${["?", "?", "?", "?", ...dataColumns.map(() => "?")].join(", ")})`
   );
   db.exec("BEGIN");
-  for (const { label, extracted } of entries) {
-    insert.run(
-      label,
-      extracted.format,
-      extracted.present ? 1 : 0,
-      JSON.stringify(extracted.data),
-      ...dataColumns.map((c) => bindValue(extracted.data[c]))
-    );
+  let committed = false;
+  try {
+    for (const { label, extracted } of entries) {
+      insert.run(
+        label,
+        extracted.format,
+        extracted.present ? 1 : 0,
+        JSON.stringify(extracted.data),
+        ...dataColumns.map((c) => bindValue(extracted.data[c]))
+      );
+    }
+    db.exec("COMMIT");
+    committed = true;
+  } finally {
+    if (!committed) db.exec("ROLLBACK");
   }
-  db.exec("COMMIT");
 }
 function registerLineFor(db, entries) {
   const byLabel = new Map(entries.map((e) => [e.label, e.extracted]));
@@ -7248,7 +7274,7 @@ var Validator = class {
       return ajv.compile(schema);
     } catch (err) {
       throw new DocmetaError(
-        `Schema "${ref}" failed to compile: ${err.message}`
+        `Schema "${ref}" failed to compile: ${errorMessage(err)}`
       );
     }
   }
@@ -7389,7 +7415,7 @@ function parseBaseline(text, source) {
   try {
     raw = JSON.parse(stripBom(text));
   } catch (err) {
-    bad(source, `invalid JSON: ${err.message}`);
+    bad(source, `invalid JSON: ${errorMessage(err)}`);
   }
   if (typeof raw !== "object" || raw === null || Array.isArray(raw)) {
     bad(source, "top level must be an object.");
@@ -7431,7 +7457,7 @@ async function readBaseline(absPath, source) {
   } catch (err) {
     if (err.code === "ENOENT") return null;
     throw new DocmetaError(
-      `Baseline "${source}" could not be read: ${err.message}`
+      `Baseline "${source}" could not be read: ${errorMessage(err)}`
     );
   }
   return parseBaseline(text, source);
@@ -7441,7 +7467,7 @@ async function writeBaselineFile(absPath, baseline, source) {
     await writeFileAtomic(absPath, serializeBaseline(baseline));
   } catch (err) {
     throw new DocmetaError(
-      `Baseline "${source}" could not be written: ${err.message}`
+      `Baseline "${source}" could not be written: ${errorMessage(err)}`
     );
   }
 }
@@ -7588,7 +7614,7 @@ async function runChecks(checks, entries, ctx = {}) {
         tokens = collectNamedParameters(check.query);
       } catch (err) {
         throw new DocmetaError(
-          `check "${check.name}": ${err.message}`
+          `check "${check.name}": ${errorMessage(err)}`
         );
       }
       if (tokens.length > 0) {
@@ -7604,7 +7630,7 @@ async function runChecks(checks, entries, ctx = {}) {
         columns = stmt.columns().map((c) => c.name);
         rows = stmt.all();
       } catch (err) {
-        const message = err.message;
+        const message = errorMessage(err);
         throw new DocmetaError(
           message.includes("attempt to write a readonly database") ? `check "${check.name}": ${message} \u2014 checks are read-only by design; a check is a SELECT over the projection. Edit the corpus with \`manni meta query\` instead.` : `check "${check.name}": ${message}`
         );
@@ -7775,7 +7801,7 @@ async function runValidate(opts) {
     } catch (err) {
       if (err instanceof DocmetaError) throw err;
       results.push(
-        parseErrorResult(label, extractor.name, err.message, "parse")
+        parseErrorResult(label, extractor.name, errorMessage(err), "parse")
       );
       return;
     }
@@ -7807,7 +7833,7 @@ async function runValidate(opts) {
       });
     } catch (err) {
       results.push(
-        parseErrorResult(label, extractor.name, err.message, "schema")
+        parseErrorResult(label, extractor.name, errorMessage(err), "schema")
       );
       return;
     }
@@ -8734,7 +8760,7 @@ async function prepareDbTarget(resolved, display) {
 var MissingCollectionView = class extends Error {
 };
 function refuseSqlError(err) {
-  const message = err.message;
+  const message = errorMessage(err);
   if (message.includes("UNIQUE constraint failed: docs._path")) {
     throw new DocmetaError("That _path already exists in the corpus.");
   }
@@ -8763,7 +8789,7 @@ async function runSql(sql, entries, ctx) {
     db = new DatabaseSync(target ? target.resolved : ":memory:");
   } catch (err) {
     throw new DocmetaError(
-      `Cannot open "${target?.display ?? ":memory:"}": ${err.message}`
+      `Cannot open "${target?.display ?? ":memory:"}": ${errorMessage(err)}`
     );
   }
   const dbInfo = target ? {
@@ -8811,7 +8837,7 @@ async function runSql(sql, entries, ctx) {
         stmt = db.prepare(sql);
         columns = stmt.columns().map((c) => c.name);
       } catch (err) {
-        const message = err.message;
+        const message = errorMessage(err);
         const missing = missingTableName(message);
         if (missing !== void 0 && !viewsBuilt && namesConfiguredCollection(missing, ctx.declaredCollections)) {
           throw new MissingCollectionView();
@@ -9331,7 +9357,7 @@ async function planSchemaMutation(ops, entries, ctx) {
           onNotice: ctx.onNotice
         });
       } catch (err) {
-        throw new DocmetaError(`"${e.label}": ${err.message}`);
+        throw new DocmetaError(`"${e.label}": ${errorMessage(err)}`);
       }
       sources.add(resolved.source);
       if (resolved.overrideIndex !== void 0) {
@@ -10040,7 +10066,9 @@ async function applyChanges(changes, entries, ctx, schemaPlan) {
         path,
         content: extractor.apply("", ops.created, {
           filePath: label,
-          elements: resolveElements(label, ctx.config)
+          // The new file's memberships, from its path alone. An INSERT has to
+          // write through the element set a read of that file would use.
+          elements: resolveElements(label, ctx.config, ctx.memberships(label))
         }),
         ensureDir: true
       });
@@ -10088,7 +10116,9 @@ async function applyChanges(changes, entries, ctx, schemaPlan) {
     }
     const applied = entry.extractor.apply(content, ops.patch, {
       filePath: label,
-      elements: resolveElements(label, ctx.config),
+      // The memberships the read above used. Without them a collection-scoped
+      // element the read found has nowhere to be written.
+      elements: resolveElements(label, ctx.config, members),
       deletions: ops.deletions
     });
     if (ops.deletions.length > 0) {
@@ -10151,7 +10181,7 @@ async function applyChanges(changes, entries, ctx, schemaPlan) {
       await rename2(r.from, r.to);
     } catch (err) {
       throw new DocmetaError(
-        `Cannot move "${r.from}" to "${r.to}": ${err.message}`
+        `Cannot move "${r.from}" to "${r.to}": ${errorMessage(err)}`
       );
     }
   }
@@ -10514,7 +10544,7 @@ async function runFill(opts) {
         provider = construct();
       } catch (err) {
         throw new DocmetaError(
-          err instanceof InferenceError ? err.message : `Could not construct the "${providerName}" provider: ${err.message}`
+          err instanceof InferenceError ? err.message : `Could not construct the "${providerName}" provider: ${errorMessage(err)}`
         );
       }
     }
@@ -10564,7 +10594,7 @@ async function runFill(opts) {
       );
       extracted = merged.extracted;
     } catch (err) {
-      return errorResult(label, extractor.name, err.message);
+      return errorResult(label, extractor.name, errorMessage(err));
     }
     let resolved;
     try {
@@ -10582,7 +10612,7 @@ async function runFill(opts) {
         onNotice: opts.onNotice
       });
     } catch (err) {
-      return errorResult(label, extractor.name, err.message);
+      return errorResult(label, extractor.name, errorMessage(err));
     }
     const schemaSet = resolved.schemas;
     let schemas;
@@ -10636,7 +10666,7 @@ async function runFill(opts) {
     try {
       extractor.apply(content, {}, { filePath: label, elements });
     } catch (err) {
-      return errorResult(label, extractor.name, err.message, schemaSet);
+      return errorResult(label, extractor.name, errorMessage(err), schemaSet);
     }
     const cacheKey = buildCacheKey([
       identity.provider,
@@ -10674,7 +10704,7 @@ async function runFill(opts) {
         return errorResult(
           label,
           extractor.name,
-          `Could not build a proposal schema from ${schemaSet.join(", ")}: ${err.message}`,
+          `Could not build a proposal schema from ${schemaSet.join(", ")}: ${errorMessage(err)}`,
           schemaSet
         );
       }
@@ -10802,7 +10832,7 @@ async function runFill(opts) {
       return errorResult(
         label,
         extractor.name,
-        err.message,
+        errorMessage(err),
         schemaSet,
         fields
       );
@@ -11102,7 +11132,7 @@ async function resolveIdentity(spec) {
     return await resolveProviderIdentityAsync(spec);
   } catch (err) {
     throw new DocmetaError(
-      err instanceof InferenceError ? err.message : `Could not resolve provider "${spec.provider}": ${err.message}`
+      err instanceof InferenceError ? err.message : `Could not resolve provider "${spec.provider}": ${errorMessage(err)}`
     );
   }
 }
@@ -11727,6 +11757,7 @@ export {
   palette,
   shouldColor,
   ToolError,
+  errorMessage,
   programName,
   setProgramName,
   externalMetadataUrlProblem,
@@ -11819,4 +11850,4 @@ export {
   renderQuery,
   renderQueryCsv
 };
-//# sourceMappingURL=chunk-L7HN3AWA.js.map
+//# sourceMappingURL=chunk-IRHWILCO.js.map