@looop-games/cli 0.1.33 → 0.1.35

package/CHANGELOG.md

@@ -14,6 +14,19 @@ Versions: [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [0.1.35] - 2026-09-02
+
+### Added
+
+- Reports landing in `.looop/agent-inbox/` are stamped with `engine`, the engine version `looop dev` was serving when the report was made — so a report read later, after an update, still says which engine produced the numbers.
+
+## [0.1.34] - 2026-09-02
+
+### Fixed
+
+- `looop test` on Windows no longer discovers and runs the engine's own scaffold smokes from `node_modules` (or anything under `overrides/` and dot-folders). The gate could never go green there because those smokes always failed; it now runs only your game's tests, as on macOS and Linux. A test filter typed as `sub/depth` also matches on Windows now.
+- `looop dev`, `looop test` and `looop publish` on Windows no longer fail with `Could not resolve "./entitiessaucer.js"` when a game has entity kinds: the generated room module now writes import paths with forward slashes whichever engine version produced them.
+
 ## [0.1.33] - 2026-08-27
 
 ### Added

package/lib/agent-surface.mjs

@@ -69,9 +69,13 @@ const sha = (buf) => createHash('sha256').update(buf).digest('hex');
 
 // Where each artifact path lands in the game repo. `skills/**` is the whole
 // surface today; AGENTS.md is spliced rather than written whole.
+// Targets are manifest keys, not host paths: forward slashes on every platform,
+// because they are recorded in .looop/agent-surface.json, compared with
+// `startsWith(`${dest}/`)`, and reported to the creator. `join(projectDir, dest)`
+// turns one into a host path wherever the filesystem is touched.
 function target(rel) {
   if (rel === 'AGENTS.md') return 'AGENTS.md';
-  if (rel.startsWith('skills/')) return join('.claude', rel);
+  if (rel.startsWith('skills/')) return `.claude/${rel}`;
   return null; // unknown surface entry from a newer engine — ignore, don't guess
 }
 
@@ -128,7 +132,7 @@ function contentMatchesRecord(abs, dest, placed) {
   if (!entries.length) return true; // an empty folder holds nothing to lose
   return entries.every((e) => {
     const file = join(e.parentPath, e.name);
-    const rel = join(dest, relative(abs, file));
+    const rel = `${dest}/${relative(abs, file).split('\\').join('/')}`;
     return placed[rel] !== undefined && sha(readFileSync(file)) === placed[rel];
   });
 }

package/lib/dev.mjs

@@ -146,6 +146,7 @@ export async function dev({ cwd = process.cwd(), port, noMp = process.env.NO_MP
     slug: project.slug,
     projectDir: project.dir, // answers /__looop/whoami — see ports.mjs
     log, // agent-inbox report lines ride the dev logger
+    engineVersion: engine.version ?? null, // stamped on agent-inbox reports
     mounts: [
       { url: `/games/${project.slug}/`, dir: project.dir },
       { url: '/shared/', dir: generatedOverridesDir },

package/lib/room-server.mjs

@@ -155,7 +155,19 @@ export function renderEntityComponents(components) {
 // generated entry to wrap. Kept as one function so the kind-aliasing, the
 // `./world.js` root convention, and the assemble→lower call can never drift
 // between the dev and publish builds.
-function renderV2Preamble(skeleton) {
+// A relative path with forward slashes whichever host produced it.
+const posixPath = (p) => p.split('\\').join('/');
+
+function renderV2Preamble(raw) {
+  // Kind modules are game-relative paths the engine relativized on the
+  // creator's machine. An engine whose framework build predates the
+  // forward-slash fix emits them with the host separator, so on Windows they
+  // arrive as `entities\saucer.js`. Dropped raw into a string literal, `\s` is
+  // an escape sequence and esbuild looks for `./entitiessaucer.js`. The CLI and
+  // the engine ship separately, so a fixed CLI on an unfixed engine is a real
+  // pairing; import specifiers and the URLs the client's kind loader builds
+  // from the embedded skeleton are always forward-slash, so normalize both.
+  const skeleton = { ...raw, kinds: raw.kinds.map((k) => ({ ...k, module: posixPath(k.module) })) };
   const imports = skeleton.kinds.map(
     (k, i) => `import { ${k.name} as K${i} } from './${k.module}';`,
   );

package/lib/static-server.mjs

@@ -113,6 +113,11 @@ export function createStaticServer({
   // import map + globalThis.__LOOOP_SKELETON__ so startGame() can boot. Null for
   // a v1 game (no bare `looop` import, no skeleton).
   skeleton = null,
+  // The engine version this server serves, stamped onto every agent-inbox
+  // file as `engine`. A report is read off disk later against whatever the
+  // game is pinned to THEN; the server is the only side that knows what it
+  // was when the report was made (the page has no version of its own).
+  engineVersion = null,
 }) {
   // The injected skeleton is LIVE, not frozen at startup: a v2 hot-reload
   // rebuilds the room with a fresh skeleton, and `setSkeleton` below pushes that
@@ -304,6 +309,7 @@ export function createStaticServer({
         mkdirSync(inboxDir, { recursive: true });
         const selfIgnore = join(inboxDir, '.gitignore');
         if (!existsSync(selfIgnore)) writeFileSync(selfIgnore, '*\n');
+        if (engineVersion) envelope.engine = engineVersion;
         const d = new Date();
         const p = (n, w = 2) => String(n).padStart(w, '0');
         const stamp = `${d.getFullYear()}-${p(d.getMonth() + 1)}-${p(d.getDate())}-${p(d.getHours())}-${p(d.getMinutes())}-${p(d.getSeconds())}-${p(d.getMilliseconds(), 3)}`;

package/lib/test-cmd.mjs

@@ -10,7 +10,7 @@
 import { spawn } from 'node:child_process';
 import { existsSync, readdirSync, readFileSync } from 'node:fs';
 import { createRequire } from 'node:module';
-import { join, relative, dirname } from 'node:path';
+import nodePath, { join, relative, dirname } from 'node:path';
 import { pathToFileURL } from 'node:url';
 import { findProject, resolveEngine } from './project.mjs';
 import { dev } from './dev.mjs';
@@ -19,16 +19,31 @@ import { portsFor, portInUse } from './ports.mjs';
 
 const SKIP_DIRS = new Set(['node_modules', 'overrides']);
 
-export function discoverTestFiles(dir) {
+// Whether a game-relative path lies under a directory discovery must ignore:
+// dependencies (the engine ships its own scaffold smokes under node_modules),
+// engine-override copies, and dot-directories. `path.relative()` returns
+// backslash-separated paths on Windows, so the split accepts both separators —
+// splitting on '/' alone yields one giant segment there and skips nothing.
+export function isSkippedPath(rel) {
+  return rel.split(/[\\/]/).some((p) => SKIP_DIRS.has(p) || p.startsWith('.'));
+}
+
+// The game-relative path with forward slashes on every platform, so a filter
+// the creator types as `sub/depth` matches on Windows too. The path module is
+// injectable so the Windows form can be produced on any host.
+export const posixRelative = (dir, file, p = nodePath) => p.relative(dir, file).split('\\').join('/');
+
+// The path module is injectable for the same reason: with `path.win32` the
+// real readdir → join → relative → skip chain runs with Windows semantics
+// against real files, on any host.
+export function discoverTestFiles(dir, { path: p = nodePath } = {}) {
   const unit = [];
   const smokes = [];
   const entries = readdirSync(dir, { withFileTypes: true, recursive: true });
   for (const entry of entries) {
     if (!entry.isFile()) continue;
-    const path = join(entry.parentPath, entry.name);
-    const rel = relative(dir, path);
-    const parts = rel.split('/');
-    if (parts.some((p) => SKIP_DIRS.has(p) || p.startsWith('.'))) continue;
+    const path = p.join(entry.parentPath, entry.name);
+    if (isSkippedPath(p.relative(dir, path))) continue;
     if (entry.name.endsWith('.test.mjs')) unit.push(path);
     else if (entry.name.endsWith('.smoke.mjs')) smokes.push(path);
   }
@@ -98,7 +113,7 @@ export async function testCmd({ cwd = process.cwd(), log = console.log, devFn =
   // milestone close and before publish (qa.md T3). With no pattern, everything
   // runs, exactly as before.
   const scoped = patterns.length > 0;
-  const matches = (f) => patterns.some((p) => relative(project.dir, f).includes(p));
+  const matches = (f) => patterns.some((p) => posixRelative(project.dir, f).includes(p));
   const unit = scoped ? all.unit.filter(matches) : all.unit;
   const smokes = scoped ? all.smokes.filter(matches) : all.smokes;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@looop-games/cli",
-  "version": "0.1.33",
+  "version": "0.1.35",
   "description": "Looop game development CLI — dev server, login, and publishing for standalone Looop games.",
   "type": "module",
   "license": "Apache-2.0",