@onedotmint/charterleaf 0.4.2 → 0.4.3

package/README.md

@@ -39,7 +39,7 @@ pi install npm:@onedotmint/charterleaf
 For the standalone CLI, use `npm install -g @onedotmint/charterleaf`.
 For local development, use `pi install /absolute/path/to/charterleaf` (a package
 directory, not a `.tgz` file). To test the CLI from a local package, run `npm pack`
-then `npm install -g ./onedotmint-charterleaf-0.4.2.tgz`.
+then `npm install -g ./onedotmint-charterleaf-0.4.3.tgz`.
 
 Restart Pi or run `/reload` after installation, then use `/charterleaf` in each
 project you want to enable. To stop automation, choose **Not Enable** in that
@@ -206,8 +206,10 @@ workflow or another development cycle.
 
 ## CLI
 
-Charterleaf exposes exactly two deterministic commands. Run them from a
-repository root containing `specs/`:
+Charterleaf exposes exactly two deterministic commands. Run them from the project
+root or a subdirectory: the nearest ancestor containing `specs/` is used, without
+crossing a Git repository or worktree boundary. The target path remains
+repo-relative, not relative to the shell's current directory.
 
 ```bash
 charterleaf related <path>
@@ -424,7 +426,7 @@ GitHub Actions runs the installed-package suite on Ubuntu and Windows with Node
 CI makes no model calls and retains packaging/test logs. `PI_CLI` can select the
 Pi Node entry point for smoke checks without relying on platform-specific shims.
 
-The [passing eight-job release-candidate matrix](https://github.com/onedotmint/Charterleaf/actions/runs/40078816360)
+For v0.4.2, the [passing eight-job release-candidate matrix](https://github.com/onedotmint/Charterleaf/actions/runs/40078816360)
 checks the actual npm tarball after isolated installation, including byte equality,
 CLI commands and Core/Extension/recovery tests. Each Ubuntu job passes all 227 tests;
 each Windows job passes 220 with zero failures and seven platform-specific skips
@@ -447,7 +449,11 @@ unchanged-spec checks passed. A later complete pre-0.4.2 run on the same model
 passed all 15 scenarios and 231 assertions, including F1/F2 without redundant reviews.
 The final Windows Git-root path fix was subsequently validated by cross-platform
 CI, not another paid model run. These bounded examples do not prove model reliability
-on every task or provider, or guarantee zero redundant reviews.
+on every task or provider, or guarantee zero redundant reviews. A subsequent
+case-A rerun on `deepseek/deepseek-flash` failed knowledge-capture assertions on
+both the v0.4.2 baseline and the patch candidate: accounting tests passed, but
+the model concluded `covered` without writing a routed spec. Automatic knowledge
+capture remains model-dependent; passing structural tests does not prove it.
 
 The controlled organic-capture comparison (`scripts/context-smoke.js`) keeps cwd,
 package paths, prompt and native tools identical. On `deepseek/deepseek-flash`:

package/extensions/charterleaf.js

@@ -65,7 +65,8 @@ module.exports = function charterleaf(pi) {
         s.budget -= size;
         s.staged.set(spec, hash);
       } else {
-        notes.push(`Read the complete ${spec} with read before modifying ${file}; automatic context budget reached.`);
+        const pages = Math.ceil(size / 8000);
+        notes.push(`Read the complete ${spec} (${size} bytes, ~${pages} page(s) via offset/limit) before modifying ${file}; it exceeds the ${size > 8000 ? '8 KB per-file limit' : 'per-task context budget'}.`);
       }
     }
     if (!result.scoped.length && !s.miss) {

package/lib/charterleaf.js

@@ -122,7 +122,8 @@ function globMatches(pattern, targetPath) {
   });
 
   source += '$';
-  return new RegExp(source).test(normalizedPath);
+  // Windows filesystems are case-insensitive, so a path's casing is not meaningful there.
+  return new RegExp(source, process.platform === 'win32' ? 'i' : '').test(normalizedPath);
 }
 
 function walkMarkdown(root) {
@@ -142,6 +143,15 @@ function walkMarkdown(root) {
   });
 }
 
+// CLI discovery must not inherit another repository's constraints.
+function resolveSpecRoot(cwd) {
+  for (let current = path.resolve(cwd); ; current = path.dirname(current)) {
+    const candidate = path.join(current, 'specs');
+    if (fs.existsSync(candidate) && fs.statSync(candidate).isDirectory()) return current;
+    if (fs.existsSync(path.join(current, '.git')) || path.dirname(current) === current) return null;
+  }
+}
+
 function relativeDisplay(filePath, cwd) {
   const relative = path.relative(cwd, filePath);
   if (!relative.startsWith('..') && !path.isAbsolute(relative)) return relative.replaceAll('\\', '/');
@@ -153,7 +163,8 @@ function write(stream, line) {
 }
 
 function resolveRelated(repoPath, cwd = process.cwd()) {
-  const specRoot = path.join(cwd, 'specs');
+  const root = path.resolve(cwd);
+  const specRoot = path.join(root, 'specs');
   if (!fs.existsSync(specRoot) || !fs.statSync(specRoot).isDirectory()) {
     throw new Error('ERROR specs/: not found');
   }
@@ -163,7 +174,7 @@ function resolveRelated(repoPath, cwd = process.cwd()) {
   const errors = [];
 
   for (const filePath of walkMarkdown(specRoot)) {
-    const display = relativeDisplay(filePath, cwd);
+    const display = relativeDisplay(filePath, root);
     let parsed;
     try {
       parsed = parseFrontmatter(fs.readFileSync(filePath, 'utf8'));
@@ -317,7 +328,8 @@ function allRequirementIds(text) {
 function lint(cwd = process.cwd(), streams = {}) {
   const stdout = streams.stdout || process.stdout;
   const stderr = streams.stderr || process.stderr;
-  const specRoot = path.join(cwd, 'specs');
+  const root = path.resolve(cwd);
+  const specRoot = path.join(root, 'specs');
   if (!fs.existsSync(specRoot) || !fs.statSync(specRoot).isDirectory()) {
     write(stderr, 'ERROR specs/: not found');
     return 2;
@@ -332,15 +344,15 @@ function lint(cwd = process.cwd(), streams = {}) {
   const paths = walkMarkdown(specRoot);
 
   for (const filePath of paths) {
-    const display = relativeDisplay(filePath, cwd);
+    const display = relativeDisplay(filePath, root);
     let result;
     try {
       result = parseFrontmatter(fs.readFileSync(filePath, 'utf8'));
-      parsed.set(filePath, result);
     } catch (error) {
       errors.push([display, `invalid frontmatter: ${error.message}`]);
       continue;
     }
+    parsed.set(filePath, result);
 
     const frontmatter = result.frontmatter || {};
     const living = isLivingSpec(filePath, specRoot);
@@ -353,7 +365,7 @@ function lint(cwd = process.cwd(), streams = {}) {
       } else {
         const specId = rawSpecId.trim();
         if (ids.has(specId)) {
-          errors.push([display, `duplicate spec id ${specId} (also in ${relativeDisplay(ids.get(specId), cwd)})`]);
+          errors.push([display, `duplicate spec id ${specId} (also in ${relativeDisplay(ids.get(specId), root)})`]);
         } else {
           ids.set(specId, filePath);
         }
@@ -411,8 +423,8 @@ function lint(cwd = process.cwd(), streams = {}) {
       const first = reqPaths[0];
       for (const filePath of reqPaths.slice(1)) {
         errors.push([
-          relativeDisplay(filePath, cwd),
-          `duplicate requirement id ${reqId} (also in ${relativeDisplay(first, cwd)})`,
+          relativeDisplay(filePath, root),
+          `duplicate requirement id ${reqId} (also in ${relativeDisplay(first, root)})`,
         ]);
       }
     }
@@ -425,7 +437,7 @@ function lint(cwd = process.cwd(), streams = {}) {
   const allExistingRequirements = new Set(globalRequirements.keys());
   for (const filePath of paths) {
     if (path.dirname(filePath) !== path.join(specRoot, 'changes') || !parsed.has(filePath)) continue;
-    const display = relativeDisplay(filePath, cwd);
+    const display = relativeDisplay(filePath, root);
     const { frontmatter: rawFrontmatter, body } = parsed.get(filePath);
     const frontmatter = rawFrontmatter || {};
     const sections = changeSections(stripHtmlComments(body));
@@ -534,7 +546,7 @@ function main(argv = process.argv.slice(2), cwd = process.cwd(), streams = {}) {
       return 0;
     }
     if (rest.length !== 1) return usageError('related requires exactly one path', streams);
-    return related(rest[0], path.resolve(cwd), streams);
+    return related(rest[0], resolveSpecRoot(cwd) || path.resolve(cwd), streams);
   }
   if (command === 'lint') {
     if (rest.length === 1 && (rest[0] === '--help' || rest[0] === '-h')) {
@@ -542,7 +554,7 @@ function main(argv = process.argv.slice(2), cwd = process.cwd(), streams = {}) {
       return 0;
     }
     if (rest.length) return usageError('lint takes no arguments', streams);
-    return lint(path.resolve(cwd), streams);
+    return lint(resolveSpecRoot(cwd) || path.resolve(cwd), streams);
   }
   return usageError(`unknown command ${command}`, streams);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onedotmint/charterleaf",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Lightweight Agent-Native Constraint Layer: opt-in spec discovery and maintenance for Pi, with a standalone zero-dependency CLI.",
   "license": "MIT",
   "repository": {

package/skills/charterleaf/SKILL.md

@@ -61,7 +61,9 @@ Use `specs/changes/<name>.md` only when an accepted future contract must coexist
 
 ## CLI details
 
-Run from the project root:
+Run from the project root or a subdirectory. CLI discovery stops at the nearest
+`specs/` directory or Git repository/worktree boundary; target paths stay
+repo-relative:
 
 - `charterleaf related <path>` returns constitution as Global, capability/engineering specs matching `applies_to`, and active changes affecting returned spec IDs. It returns paths/metadata, not bodies. Decisions and explicit references are followed manually when relevant. A scoped miss is not proof that no constraints apply.
 - `charterleaf lint` checks frontmatter, IDs/globs, duplicate IDs, change references/deltas and conflicting active changes. `0 structural errors` does not prove semantic correctness, coverage or code/spec agreement. The Extension also lints standard spec writes.