dsh-plugin-wiki-tools 0.10.0 → 0.11.0

package/lib/lint.js

@@ -9,6 +9,7 @@
 import { mkdir, readFile, writeFile } from 'node:fs/promises'
 import { join } from 'node:path'
 import {
+  PAGE_STATUSES,
   buildAliasMap,
   collectMarkdown,
   isMachineryPage,
@@ -37,6 +38,7 @@ export async function lintVault(root) {
   const inbound = checkDeadLinks(pages, add)
   checkOrphans(pages, inbound, indexed, add)
   checkFrontmatterGaps(pages, add)
+  checkStatusVocabulary(pages, add)
   checkEmptySections(pages, add)
   await checkHotCacheStaleness(root, pages, add)
 
@@ -172,6 +174,25 @@ function checkFrontmatterGaps(pages, add) {
   }
 }
 
+/**
+ * Status values outside the lifecycle vocabulary (seed/developing/mature/
+ * evergreen). A stray value silently breaks status-based queries and
+ * promotion flows, so it is flagged even though links stay intact.
+ * @param {Pages} pages - collected pages.
+ * @param {Add} add - issue recorder.
+ */
+function checkStatusVocabulary(pages, add) {
+  for (const page of pages) {
+    if (isMachineryPage(page.name)) continue
+    const status = page.fields?.status
+    if (typeof status === 'string' && !PAGE_STATUSES.includes(status)) {
+      add('status-vocabulary', 'warn', page.name,
+        `status "${status}" is outside the lifecycle vocabulary (${PAGE_STATUSES.join('/')})`,
+        `Set status to one of ${PAGE_STATUSES.join('/')}`)
+    }
+  }
+}
+
 /**
  * Headings with no content before the next heading.
  * @param {Pages} pages - collected pages.

package/lib/vault.js

@@ -13,6 +13,7 @@
 
 import { createHash } from 'node:crypto'
 import { spawnSync } from 'node:child_process'
+import { appendFileSync, readFileSync } from 'node:fs'
 import { mkdir, open, readFile, readdir, rm, stat, writeFile } from 'node:fs/promises'
 import { basename, dirname, isAbsolute, join, relative, sep } from 'node:path'
 import { parse as parseYaml, stringify as stringifyYaml } from 'yaml'
@@ -31,6 +32,9 @@ export const TYPE_FOLDERS = {
   meta: 'wiki/meta',
 }
 
+/** Lifecycle status vocabulary; writePage rejects anything else and lint flags it. */
+export const PAGE_STATUSES = ['seed', 'developing', 'mature', 'evergreen']
+
 /** Master-index section headings follow the mapped folder basename, computed per vault. */
 
 /**
@@ -227,6 +231,7 @@ export class Vault {
   gitCommit(message) {
     if (!this.gitAutoCommit) return
     const run = args => spawnSync('git', ['-C', this.root, ...args], { encoding: 'utf8' })
+    this.ignoreLockFiles()
     const add = run(['add', '-A'])
     if (add.status !== 0) throw new Error(`wiki-tools: git add failed in the vault: ${String(add.stderr).trim()}`)
     const pending = run(['diff', '--cached', '--quiet'])
@@ -236,6 +241,27 @@ export class Vault {
     }
   }
 
+  /**
+   * Keep advisory lock files out of vault history: cross-process locks under
+   * `.vault-meta/locks/` are pure runtime coordination, never vault state.
+   * Idempotent; creates or appends the vault's `.gitignore` as needed.
+   * @returns {void}
+   */
+  ignoreLockFiles() {
+    const marker = '.vault-meta/locks/'
+    const gitignore = join(this.root, '.gitignore')
+    let current = ''
+    try {
+      current = readFileSync(gitignore, 'utf8')
+    } catch (error) {
+      if (error.code !== 'ENOENT') throw error
+    }
+    if (!current.split('\n').some(line => line.trim() === marker)) {
+      const prefix = current.length === 0 || current.endsWith('\n') ? '' : '\n'
+      appendFileSync(gitignore, `${prefix}# wiki-tools runtime locks\n${marker}\n`, 'utf8')
+    }
+  }
+
   /**
    * Write one wiki page with complete bookkeeping: frontmatter completion,
    * filename-uniqueness guard, master-index entry, folder `_index.md` entry,
@@ -261,6 +287,9 @@ export class Vault {
       const existing = await this.readPage(path)
       await this.assertUniqueFilename(cleanTitle, path)
       validateExtraFrontmatter(extraFrontmatter)
+      if (status !== undefined && !PAGE_STATUSES.includes(status)) {
+        throw new Error(`wiki-tools: status must be one of ${PAGE_STATUSES.join('/')} (got ${JSON.stringify(status)})`)
+      }
       const date = today()
       const fields = {
         ...(existing?.fields ?? {}),
@@ -354,7 +383,12 @@ export class Vault {
         if (page.name === title || page.name.toLowerCase() === 'log' || /^lint-report-/.test(page.name)) continue
         const raw = await readFile(page.path, 'utf8').catch(() => undefined)
         if (raw === undefined) continue
-        const pattern = new RegExp(`\\[\\[${escapeRegExp(title)}(\]\]|\||#)`, 'g')
+        // Exact-target boundary: the title must end the link target (`]]`),
+        // start an alias (`|`), or start an anchor (`#`). Escapes are doubled
+        // because this is a string-built RegExp, not a regex literal: single
+        // `\]` / `\|` would be eaten by the template literal and leave an
+        // empty alternation branch matching every `[[title` prefix.
+        const pattern = new RegExp(`\\[\\[${escapeRegExp(title)}(\\]\\]|\\||#)`, 'g')
         const updated = raw.replace(pattern, `[[${cleanNew}$1`)
         if (updated !== raw) {
           await writeFile(page.path, updated, 'utf8')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-wiki-tools",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "description": "Native DeepSeek Harness tools for an Obsidian wiki vault: wiki_query, wiki_write, and wiki_lint implement the mechanical core (path routing, delta tracking, index/log bookkeeping, health checks) of the wiki skill suite.",
   "license": "MIT",
   "type": "module",