dotmd-cli 0.14.11 → 0.14.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dotmd-cli",
-  "version": "0.14.11",
+  "version": "0.14.12",
   "description": "CLI for managing markdown documents with YAML frontmatter — index, query, validate, graph, export, Notion sync, AI summaries.",
   "type": "module",
   "license": "MIT",

package/src/frontmatter.mjs

@@ -22,11 +22,18 @@ export function replaceFrontmatter(raw, newFrontmatter) {
   return `---\n${newFrontmatter}\n---\n${body}`;
 }
 
-export function parseSimpleFrontmatter(text) {
+// Parses our YAML subset. Optional `warnings` array receives non-fatal
+// structural issues (e.g. duplicate keys) — caller decides whether to surface
+// them. Default behavior is unchanged: keep first occurrence of a duplicate
+// key, ignore subsequent ones.
+export function parseSimpleFrontmatter(text, warnings) {
   const data = {};
+  const seenDupKeys = new Set();
   let currentArrayKey = null;
+  let lineNum = 0;
 
   for (const rawLine of text.split('\n')) {
+    lineNum++;
     const line = rawLine.replace(/\r$/, '');
     if (!line.trim()) continue;
 
@@ -35,6 +42,11 @@ export function parseSimpleFrontmatter(text) {
       const [, key, rawValue] = keyMatch;
       if (Object.prototype.hasOwnProperty.call(data, key)) {
         currentArrayKey = null;
+        if (warnings && !seenDupKeys.has(key)) {
+          seenDupKeys.add(key);
+          warnings.push({ key, line: lineNum,
+            message: `Duplicate frontmatter key \`${key}\` at line ${lineNum}; keeping first occurrence, ignoring later values.` });
+        }
         continue;
       }
       if (!rawValue.trim()) {

package/src/index.mjs

@@ -120,7 +120,8 @@ export function parseDocFile(filePath, config) {
   const relativePath = toRepoPath(filePath, config.repoRoot);
   const raw = readFileSync(filePath, 'utf8');
   const { frontmatter, body } = extractFrontmatter(raw);
-  const parsedFrontmatter = parseSimpleFrontmatter(frontmatter);
+  const fmWarnings = [];
+  const parsedFrontmatter = parseSimpleFrontmatter(frontmatter, fmWarnings);
   const headingTitle = extractFirstHeading(body);
   const title = asString(parsedFrontmatter.title) ?? headingTitle ?? path.basename(filePath, '.md');
   const summary = asString(parsedFrontmatter.summary) ?? extractSummary(body) ?? null;
@@ -187,6 +188,10 @@ export function parseDocFile(filePath, config) {
     errors: [],
   };
 
+  for (const w of fmWarnings) {
+    doc.warnings.push({ path: relativePath, level: 'warning', message: w.message });
+  }
+
   validateDoc(doc, parsedFrontmatter, headingTitle, config);
   return doc;
 }

package/src/util.mjs

@@ -100,3 +100,17 @@ export function resolveDocPath(input, config) {
 
   return null;
 }
+
+// Resolve a reference path written in frontmatter or a body link.
+// Tries doc-relative first (the historical convention), then falls back to
+// repo-root-relative — so paths like `docs/foo/bar.md` written from any nesting
+// level resolve correctly. Returns the absolute path if either form exists,
+// else null.
+export function resolveRefPath(relPath, docDir, repoRoot) {
+  if (!relPath) return null;
+  const docRelative = path.resolve(docDir, relPath);
+  if (existsSync(docRelative)) return docRelative;
+  const repoRelative = path.resolve(repoRoot, relPath);
+  if (existsSync(repoRelative)) return repoRelative;
+  return null;
+}

package/src/validate.mjs

@@ -1,6 +1,5 @@
-import { existsSync } from 'node:fs';
 import path from 'node:path';
-import { asString } from './util.mjs';
+import { asString, resolveRefPath } from './util.mjs';
 import { getGitLastModified, getGitLastModifiedBatch } from './git.mjs';
 import { toRepoPath } from './util.mjs';
 
@@ -101,8 +100,7 @@ export function validateDoc(doc, frontmatter, headingTitle, config) {
   const allRefFields = [...(config.referenceFields.bidirectional || []), ...(config.referenceFields.unidirectional || [])];
   for (const field of allRefFields) {
     for (const relPath of (doc.refFields[field] || [])) {
-      const resolved = path.resolve(docDir, relPath);
-      if (!existsSync(resolved)) {
+      if (!resolveRefPath(relPath, docDir, config.repoRoot)) {
         doc.errors.push({ path: doc.path, level: 'error', message: `${field} entry \`${relPath}\` does not resolve to an existing file.` });
       }
     }
@@ -110,8 +108,7 @@ export function validateDoc(doc, frontmatter, headingTitle, config) {
 
   // Validate body links resolve to existing files
   for (const link of (doc.bodyLinks || [])) {
-    const resolved = path.resolve(docDir, link.href);
-    if (!existsSync(resolved)) {
+    if (!resolveRefPath(link.href, docDir, config.repoRoot)) {
       doc.warnings.push({ path: doc.path, level: 'warning', message: `body link \`${link.href}\` does not resolve to an existing file.` });
     }
   }
@@ -128,7 +125,12 @@ export function checkBidirectionalReferences(docs, config) {
     const refs = new Set();
     for (const field of biFields) {
       for (const relPath of (doc.refFields[field] || [])) {
-        const resolved = path.resolve(docDir, relPath);
+        // Use the same doc-relative-then-repo-root fallback as validateDoc so
+        // both styles produce identical refMap keys; otherwise an entry like
+        // `docs/foo.md` (repo-root style) gets keyed as
+        // `<doc-parent>/docs/foo.md` and never matches the target's repo path.
+        const resolved = resolveRefPath(relPath, docDir, config.repoRoot)
+          ?? path.resolve(docDir, relPath);
         refs.add(toRepoPath(resolved, config.repoRoot));
       }
     }