@frockbot/architecture-checks 0.3.11 → 0.3.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@frockbot/architecture-checks",
-  "version": "0.3.11",
+  "version": "0.3.13",
   "private": false,
   "type": "module",
   "description": "Automated checks for the constitutional rules that can be enforced mechanically.",
@@ -9,23 +9,23 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@frockbot/application-foundation": "0.3.11",
-    "@frockbot/computer-core": "0.3.11",
-    "@frockbot/connection-core": "0.3.11",
-    "@frockbot/kernel-agent-loop": "0.3.11",
-    "@frockbot/kernel-composition": "0.3.11",
-    "@frockbot/kernel-contracts": "0.3.11",
-    "@frockbot/kernel-do": "0.3.11",
-    "@frockbot/plugin-applets": "0.3.11",
-    "@frockbot/plugin-authoring": "0.3.11",
-    "@frockbot/plugin-computer": "0.3.11",
-    "@frockbot/plugin-memory": "0.3.11",
-    "@frockbot/plugin-models": "0.3.11",
-    "@frockbot/plugin-prompt": "0.3.11",
-    "@frockbot/plugin-provider-foundation": "0.3.11",
-    "@frockbot/plugin-provider-ollama-cloud": "0.3.11",
-    "@frockbot/plugin-skills": "0.3.11",
-    "@frockbot/plugin-tools": "0.3.11",
+    "@frockbot/application-foundation": "0.3.13",
+    "@frockbot/computer-core": "0.3.13",
+    "@frockbot/connection-core": "0.3.13",
+    "@frockbot/kernel-agent-loop": "0.3.13",
+    "@frockbot/kernel-composition": "0.3.13",
+    "@frockbot/kernel-contracts": "0.3.13",
+    "@frockbot/kernel-do": "0.3.13",
+    "@frockbot/plugin-applets": "0.3.13",
+    "@frockbot/plugin-authoring": "0.3.13",
+    "@frockbot/plugin-computer": "0.3.13",
+    "@frockbot/plugin-memory": "0.3.13",
+    "@frockbot/plugin-models": "0.3.13",
+    "@frockbot/plugin-prompt": "0.3.13",
+    "@frockbot/plugin-provider-foundation": "0.3.13",
+    "@frockbot/plugin-provider-ollama-cloud": "0.3.13",
+    "@frockbot/plugin-skills": "0.3.13",
+    "@frockbot/plugin-tools": "0.3.13",
     "cordis": "4.0.0-rc.8"
   },
   "devDependencies": {

package/src/source-nul-bytes.test.ts

@@ -0,0 +1,34 @@
+// A NUL byte inside a source file is invisible in an editor and harmless at
+// runtime — a template literal holding one is just a string with a separator
+// no name can contain. What it is not harmless to is git: a file whose first
+// 8000 bytes contain a NUL is binary, and git has no three-way merge for a
+// binary file. Two branches that both edit it do not conflict; one side is
+// kept whole and the other's edits vanish, with nothing in the merge output
+// to say so. That is how `packages/plugin-flock/src/client/index.ts` became a
+// file every merge gambled on. Write the byte as an escape instead, so the
+// string is identical and the file stays text.
+import { describe, expect, test } from "bun:test";
+import { readFileSync } from "node:fs";
+import { join, resolve } from "node:path";
+import { $ } from "bun";
+
+const repoRoot = resolve(import.meta.dirname, "..", "..", "..");
+
+describe("tracked source files are text", () => {
+  test("no source file under packages/ or apps/ contains a NUL byte", async () => {
+    const listed = await $`git -C ${repoRoot} ls-files -z packages apps`
+      .quiet()
+      .text();
+    const sources = listed
+      .split("\0")
+      .filter((path) => path.length > 0)
+      .filter((path) => /\.(ts|tsx|vue|css|md|json|jsonc|html)$/u.test(path));
+    // A sanity floor: an empty listing would make this test pass vacuously.
+    expect(sources.length).toBeGreaterThan(100);
+
+    const offenders = sources.filter((path) =>
+      readFileSync(join(repoRoot, path)).includes(0),
+    );
+    expect(offenders).toEqual([]);
+  });
+});