bantamkit-mcp 0.35.8 → 0.35.9

package/dist/pyargparse.js

@@ -79,9 +79,55 @@ export class ArgumentTypeError extends Error {
  * a second hand-written copy of a conversion rule is the defect this module was built from.
  */
 export function pyIntStrict(text) {
-    if (!/^\s*[+-]?\d+(?:_\d+)*\s*$/.test(text))
+    // job66/#174: CPython's `int(str)`, measured on 3.12 rather than recalled. It first maps
+    // every NON-ASCII character through `_PyUnicode_TransformDecimalAndSpaceToASCII`: a Unicode
+    // decimal digit (category Nd, any script, astral included) becomes its ASCII digit and a
+    // Unicode space becomes ' '. ASCII passes through untouched, so the only ASCII whitespace
+    // the parser then skips is ` \t\n\v\f\r` — `int('3\x1c')` is a ValueError although
+    // `'\x1c'.isspace()` is true — and U+FEFF is not a space at all. Single underscores may sit
+    // BETWEEN digits only, and more than 4300 digits is a ValueError (`sys.int_info
+    // .default_max_str_digits`). WAS: `/^\s*[+-]?\d+(?:_\d+)*\s*$/` without the `u` flag —
+    // ASCII digits only, so `--k ٣` was exit 2 here and served on the reference, while JS
+    // `\s` admitted U+FEFF and refused U+0085, both the opposite of CPython.
+    let ascii = '';
+    for (const ch of text) {
+        const cp = ch.codePointAt(0);
+        if (cp < 0x80)
+            ascii += ch;
+        else if (PY_UNICODE_SPACE.has(cp))
+            ascii += ' ';
+        else if (/\p{Nd}/u.test(ch))
+            ascii += String(ndValue(cp));
+        else
+            throw new TypeError('not an int');
+    }
+    const match = /^[ \t\n\v\f\r]*([+-]?)([0-9]+(?:_[0-9]+)*)[ \t\n\v\f\r]*$/.exec(ascii);
+    if (match === null)
         throw new TypeError('not an int');
-    return Number(text.trim().replace(/_/g, ''));
+    const digits = match[2].replace(/_/g, '');
+    if (digits.length > PY_INT_MAX_STR_DIGITS)
+        throw new TypeError('not an int');
+    const value = Number(`${match[1]}${digits}`);
+    return value === 0 ? 0 : value; // `int('-0')` is 0, never JavaScript's -0
+}
+/** `sys.int_info.default_max_str_digits`: CPython refuses a longer decimal string. */
+const PY_INT_MAX_STR_DIGITS = 4300;
+/** The non-ASCII code points `str.isspace()` is true for (CPython 3.12, Unicode 15.0). */
+const PY_UNICODE_SPACE = new Set([
+    0x85, 0xa0, 0x1680, 0x2000, 0x2001, 0x2002, 0x2003, 0x2004, 0x2005, 0x2006, 0x2007, 0x2008,
+    0x2009, 0x200a, 0x2028, 0x2029, 0x202f, 0x205f, 0x3000,
+]);
+/**
+ * `unicodedata.decimal(ch)` for a category-Nd code point. Unicode allocates every decimal
+ * digit set as ten contiguous code points from 0 to 9, and sets that touch are back to back
+ * (the five mathematical sets at U+1D7CE), so the value is the offset from the start of the
+ * contiguous Nd run, modulo ten.
+ */
+function ndValue(cp) {
+    let start = cp;
+    while (start > 0 && /\p{Nd}/u.test(String.fromCodePoint(start - 1)))
+        start -= 1;
+    return (cp - start) % 10;
 }
 /** `_get_action_name`: the name argparse puts after `argument `. */
 const actionName = (action) => action.optionStrings.length > 0

package/dist/tokenledger.js

@@ -31,7 +31,7 @@
  * total`), never as a comparison afterwards: a `number` past 2**53 is already rounded, so the
  * check would be reading a value this side had got wrong.
  */
-import { existsSync, readFileSync, readdirSync, statSync } from 'node:fs';
+import { existsSync, lstatSync, readFileSync, readdirSync, statSync } from 'node:fs';
 import { join } from 'node:path';
 import { BantamError } from './errors.js';
 import { MAX_SAFE_INT, TOKEN_CLASSES, loadPriceTable, priceTokens, } from './pricing.js';
@@ -125,8 +125,21 @@ export function walk(base) {
                 // exactly as it is there.
                 isDirectory = false;
             }
-            if (isDirectory)
-                visit(child, `${rel}/`);
+            if (isDirectory) {
+                // #151: a symlinked directory is not a transcript tree. `statSync` follows the link,
+                // so a link to a directory lands in this arm and is skipped here: neither walked nor
+                // read. Following it walked a link to an ancestor until ELOOP and counted the one
+                // transcript beneath it once per level. `skill_audit`'s rule, and the reference's.
+                let link = false;
+                try {
+                    link = lstatSync(child).isSymbolicLink();
+                }
+                catch {
+                    link = false;
+                }
+                if (!link)
+                    visit(child, `${rel}/`);
+            }
             else if (name.endsWith(TRANSCRIPT_SUFFIX))
                 found.push([child, rel]);
         }
@@ -139,7 +152,10 @@ export function walk(base) {
  * raises, which would make an unreadable transcript a counted one on this side alone.
  */
 function readUtf8Strict(path) {
-    const decoder = new TextDecoder('utf-8', { fatal: true });
+    // `ignoreBOM: true` KEEPS a leading U+FEFF (#150). The default strips it from the first line
+    // only, where `read_text(encoding="utf-8")` keeps it and `json.loads` refuses it; kept, the
+    // line is `unparsed-line` on both sides, as a BOM on any later line already was.
+    const decoder = new TextDecoder('utf-8', { fatal: true, ignoreBOM: true });
     return decoder.decode(readFileSync(path));
 }
 function usageOf(record) {

package/dist/updatecheck.d.ts

@@ -30,6 +30,18 @@ export declare const UPDATE_NEVER = "update: never checked.";
  * falsify the claim, the same reason `selfupdate.COMPARISON` carries both.
  */
 export declare const UPDATE_AVAILABLE: string;
+/**
+ * The command `UPDATE_AVAILABLE` tells the operator to run: THIS node, absolute (job66/#171).
+ *
+ * WAS: `` `{program} --update` ``, the bare command, which `--install` puts only in
+ * `<prefix>/node_modules/.bin` — on nobody's PATH — so the sentence answered `command not
+ * found` from a GUI-launched host's shell. `process.execPath` and this build's own `cli.js`
+ * (the sibling of this module in `dist/`), each double-quoted, need no PATH and hold a home
+ * directory with a space in it. The reference fills in `sys.executable -m bantamkit.mcpserver`
+ * instead; the TEMPLATE stays byte-identical and the filling is the `docs/porting.md` row
+ * *the update command names each runtime's own interpreter*.
+ */
+export declare function updateCommand(): string;
 /** Names the date because the record may be months old. */
 export declare const UPDATE_CURRENT = "update: {program} {installed} is current as of {date}.";
 /**
@@ -111,8 +123,8 @@ export declare function readRecord(path?: string): Record<string, unknown> | nul
  * and not only on the `current` path: a record that cannot say when it was written is not a
  * record, and one rule is one thing for the two runtimes to reproduce instead of two.
  */
-export declare function decide(installed: string, source: Source, record: Record<string, unknown> | null, key?: string): UpdateStatus;
+export declare function decide(installed: string, source: Source, record: Record<string, unknown> | null, key?: string, command?: string): UpdateStatus;
 /** Read the record and decide. The whole reader, in one call, for `bantamkit_status`. */
-export declare function updateStatus(installed: string, key?: string, path?: string): UpdateStatus;
+export declare function updateStatus(installed: string, key?: string, path?: string, command?: string): UpdateStatus;
 /** Exactly one of the five sentences. Always a line: there is no silent state here. */
-export declare function updateLine(installed: string, key?: string, path?: string): string;
+export declare function updateLine(installed: string, key?: string, path?: string, command?: string): string;

package/dist/updatecheck.js

@@ -35,7 +35,7 @@
  * `0.10.0` is the pair that says so, and `test/updatecheck.test.mjs` both runs it and pins
  * structurally that nothing comparator-shaped is defined here.
  *
- * NO NEW DEPENDENCY: `node:fs`, `node:os` and `node:path`. The `str.format` substitution the
+ * NO NEW DEPENDENCY: `node:fs`, `node:os`, `node:path` and `node:url`. The `str.format` substitution the
  * reference gets from CPython is the four lines of `fill` below — `selfupdate.ts` has the same
  * four and cannot be imported for them, for the reason two paragraphs up.
  *
@@ -45,6 +45,7 @@
 import { readFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { join } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { compareVersions } from './npminstall.js';
 export { compareVersions };
 /**
@@ -80,7 +81,21 @@ export const UPDATE_NEVER = 'update: never checked.';
  * falsify the claim, the same reason `selfupdate.COMPARISON` carries both.
  */
 export const UPDATE_AVAILABLE = 'update: {program} {installed} is running; the package index has {latest} — run ' +
-    '`{program} --update`, then reconnect the host.';
+    '`{command}`, then reconnect the host.';
+/**
+ * The command `UPDATE_AVAILABLE` tells the operator to run: THIS node, absolute (job66/#171).
+ *
+ * WAS: `` `{program} --update` ``, the bare command, which `--install` puts only in
+ * `<prefix>/node_modules/.bin` — on nobody's PATH — so the sentence answered `command not
+ * found` from a GUI-launched host's shell. `process.execPath` and this build's own `cli.js`
+ * (the sibling of this module in `dist/`), each double-quoted, need no PATH and hold a home
+ * directory with a space in it. The reference fills in `sys.executable -m bantamkit.mcpserver`
+ * instead; the TEMPLATE stays byte-identical and the filling is the `docs/porting.md` row
+ * *the update command names each runtime's own interpreter*.
+ */
+export function updateCommand() {
+    return `"${process.execPath}" "${fileURLToPath(new URL('./cli.js', import.meta.url))}" --update`;
+}
 /** Names the date because the record may be months old. */
 export const UPDATE_CURRENT = 'update: {program} {installed} is current as of {date}.';
 /**
@@ -268,7 +283,7 @@ function checkedDate(record, key) {
  * and not only on the `current` path: a record that cannot say when it was written is not a
  * record, and one rule is one thing for the two runtimes to reproduce instead of two.
  */
-export function decide(installed, source, record, key = KEY) {
+export function decide(installed, source, record, key = KEY, command = updateCommand()) {
     if (source === SOURCE_ABSENT)
         return { state: STATE_NEVER, line: UPDATE_NEVER };
     if (source !== SOURCE_RECORD || record === null) {
@@ -283,7 +298,7 @@ export function decide(installed, source, record, key = KEY) {
     if (order < 0) {
         return {
             state: STATE_AVAILABLE,
-            line: fill(UPDATE_AVAILABLE, { program: PROGRAM, installed, latest }),
+            line: fill(UPDATE_AVAILABLE, { program: PROGRAM, installed, latest, command }),
         };
     }
     if (order === 0) {
@@ -298,11 +313,11 @@ export function decide(installed, source, record, key = KEY) {
     };
 }
 /** Read the record and decide. The whole reader, in one call, for `bantamkit_status`. */
-export function updateStatus(installed, key = KEY, path = recordPath()) {
+export function updateStatus(installed, key = KEY, path = recordPath(), command = updateCommand()) {
     const { source, record } = loadRecord(path);
-    return decide(installed, source, record, key);
+    return decide(installed, source, record, key, command);
 }
 /** Exactly one of the five sentences. Always a line: there is no silent state here. */
-export function updateLine(installed, key = KEY, path = recordPath()) {
-    return updateStatus(installed, key, path).line;
+export function updateLine(installed, key = KEY, path = recordPath(), command = updateCommand()) {
+    return updateStatus(installed, key, path, command).line;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bantamkit-mcp",
-  "version": "0.35.8",
+  "version": "0.35.9",
   "description": "Memory MCP server for Claude Code, Cursor, VS Code Copilot and Claude Desktop: per-person memory, JSON Schema validation, shift-work tools. Pure Node; install once with npx, run offline.",
   "keywords": [
     "mcp",
@@ -60,6 +60,6 @@
     "typescript": "^5.7.2"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "1.30.0"
+    "@modelcontextprotocol/sdk": "1.31.0"
   }
 }