@ultimat3/manifest 2.0.0 → 3.0.0

package/CLAUDE.md

@@ -47,6 +47,11 @@ by the CLI, not imported.
   section a truncated or hand-trimmed file happens not to carry was a bare `TypeError` two calls
   from the gate that exists to explain. The FACTS inside a section stay unwalked — a manifest
   written before a field existed is still readable, which is `MANIFEST_VERSION`'s rule.
+- **A byte count is `Buffer.byteLength`, never `String.length`.** `EmitResult.bytes` measures the
+  file on disk and a manifest carries the APP's strings — a locale name, an entity description, a
+  title in the app's own language — so UTF-16 code units under-report every non-ASCII one.
+  `agents-md.ts` measures the same quantity the same way. Nothing committed carries the number:
+  `writeAppManifest` is the only caller and it reads `path` alone, so correcting it is not drift.
 - **`--json` is awaited.** `emitManifest({ stdout: true })` writes through `await Bun.write(
   Bun.stdout, …)`: a write to a pipe is asynchronous and `process.exit()` discards the queue, and
   this is the largest payload the CLI prints. Same bug `scripts/stdout-truncation.test.ts` pins.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ultimat3/manifest",
-  "version": "2.0.0",
+  "version": "3.0.0",
   "description": "x.manifest.json: deterministic generated facts, contract diff, AGENTS.md budget",
   "license": "MIT",
   "type": "module",
@@ -31,10 +31,10 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@ultimat3/action": "2.0.0",
-    "@ultimat3/core": "2.0.0",
-    "@ultimat3/entity": "2.0.0",
-    "@ultimat3/jobs": "2.0.0",
-    "@ultimat3/query": "2.0.0"
+    "@ultimat3/action": "3.0.0",
+    "@ultimat3/core": "3.0.0",
+    "@ultimat3/entity": "3.0.0",
+    "@ultimat3/jobs": "3.0.0",
+    "@ultimat3/query": "3.0.0"
   }
 }

package/src/emit.ts

@@ -54,6 +54,11 @@ export interface EmitResult {
 export async function emitManifest(input: EmitInput): Promise<EmitResult> {
   const path = input.path ?? `./${MANIFEST_FILENAME}`;
   const text = manifestJson(input.manifest);
+  // The bytes on disk, never `text.length`: a manifest carries the APP's strings — a locale name,
+  // an entity description, a title in the app's own language — and `String.length` counts UTF-16
+  // code units, so it under-reports every one of them and over-reports nothing. `agents-md.ts`
+  // measures the same quantity the same way.
+  const bytes = Buffer.byteLength(text, 'utf8');
 
   if (input.stdout === true) {
     // stdout is the wire in `--json` mode; nothing else may be written to it — and the write is
@@ -61,16 +66,16 @@ export async function emitManifest(input: EmitInput): Promise<EmitResult> {
     // is still queued. Unawaited, the largest payload the CLI prints was the one that lost bytes,
     // exactly as `scripts/stdout-truncation.test.ts` documents for the same bug elsewhere.
     await Bun.write(Bun.stdout, text);
-    return { path, bytes: text.length, buildId: input.manifest.buildId, changed: false };
+    return { path, bytes, buildId: input.manifest.buildId, changed: false };
   }
 
   const existing = await readIfExists(path);
   // Skip the write when nothing moved: an unchanged mtime keeps file watchers quiet.
   if (existing === text) {
-    return { path, bytes: text.length, buildId: input.manifest.buildId, changed: false };
+    return { path, bytes, buildId: input.manifest.buildId, changed: false };
   }
   await Bun.write(path, text);
-  return { path, bytes: text.length, buildId: input.manifest.buildId, changed: true };
+  return { path, bytes, buildId: input.manifest.buildId, changed: true };
 }
 
 /** Read and structurally validate a manifest. `undefined` when absent or unparseable. */