npm - castle-web-cli - Versions diffs - 0.4.131 → 0.4.133 - Mend

castle-web-cli 0.4.131 → 0.4.133

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (12) hide show

package/dist/agent-prompts.js +2 -2
package/dist/shell/assets/{index-D6K-0YDB.css → index-Bx4sNpO5.css} +1 -1
package/dist/shell/assets/{index-BpOLUyVO.js → index-DNDk8nRG.js} +52 -52
package/dist/shell/index.html +2 -2
package/kits/physics-2d/castle.json +1 -1
package/kits/physics-2d/editors/PlayOnly.jsx +3 -2
package/kits/physics-2d/engine/liveReload.js +120 -23
package/kits/physics-2d/scripts/svg-emission-guide.md +3 -2
package/kits/physics-3d/CLAUDE.md +5 -2
package/kits/physics-3d/castle.json +1 -1
package/kits/physics-3d/engine3d/modelFiles.js +8 -3
package/package.json +1 -1

package/dist/shell/index.html CHANGED Viewed

@@ -10,8 +10,8 @@
     <link rel="icon" type="image/png" sizes="32x32" href="/__castle/ide/favicon-32x32.png" />
     <link rel="icon" type="image/png" sizes="16x16" href="/__castle/ide/favicon-16x16.png" />
     <link rel="icon" href="/__castle/ide/favicon.ico" sizes="any" />
-    <script type="module" crossorigin src="/__castle/ide/assets/index-BpOLUyVO.js"></script>
-    <link rel="stylesheet" crossorigin href="/__castle/ide/assets/index-D6K-0YDB.css">
+    <script type="module" crossorigin src="/__castle/ide/assets/index-DNDk8nRG.js"></script>
+    <link rel="stylesheet" crossorigin href="/__castle/ide/assets/index-Bx4sNpO5.css">
   </head>
   <body>
     <div id="root"></div>

package/kits/physics-2d/castle.json CHANGED Viewed

@@ -131,5 +131,5 @@
   "main": "main.jsx",
   "autoUpdateWhenImported": true,
   "title": "physics-2d",
-  "publishedVersion": "2026-08-21T22:23:23.744Z"
+  "publishedVersion": "2026-08-21T23:15:26.298Z"
 }

package/kits/physics-2d/editors/PlayOnly.jsx CHANGED Viewed

@@ -1,6 +1,7 @@
 import React from 'react';
 import { Lifecycle } from 'castle-web-sdk';
-import { initialFiles, parseJsonFile } from '../engine/files';
+import { parseJsonFile } from '../engine/files';
+import { useLiveDeckFiles } from '../engine/liveReload';
 import { collectAssets } from '../engine/assets';
 import { ScenePlayer } from '../engine/ScenePlayer';
 import { behaviorClasses } from './behaviorRegistry';
@@ -22,7 +23,7 @@ import { behaviorClasses } from './behaviorRegistry';
 const DEFAULT_SCENE = 'scenes/main.scene';
 export function PlayOnly({ initialScene } = {}) {
-  const files = initialFiles;
+  const { files } = useLiveDeckFiles({ live: false });
   const scenePath = initialScene && files[initialScene] !== undefined ? initialScene : DEFAULT_SCENE;
   const { value: sceneData } = parseJsonFile(scenePath, files[scenePath] ?? '');
   if (!sceneData) return null;

package/kits/physics-2d/engine/liveReload.js CHANGED Viewed

@@ -13,8 +13,15 @@ import { initialFiles } from './files';
 const DATA_EXTS = ['.scene', '.pxart', '.sprite', '.style', '.drawing'];
 const CODE_EXTS = ['.js', '.jsx', '.ts', '.tsx', '.css'];
+// Extensions the DECK declared that this kit has never heard of, learned from
+// the serve at mount (hydrateDeclaredFiles). Module-level because the change
+// classifier calls isDataFile per event, with no state to thread through.
+const declaredDataExts = new Set();
 export function isDataFile(path) {
-  return DATA_EXTS.some((ext) => path.endsWith(ext));
+  if (DATA_EXTS.some((ext) => path.endsWith(ext))) return true;
+  for (const ext of declaredDataExts) if (path.endsWith(ext)) return true;
+  return false;
 }
 // Which iframe is speaking -- every panel runs its own copy of this.
@@ -47,34 +54,124 @@ async function readDeckFile(path) {
   return typeof data.contents === 'string' ? data.contents : '';
 }
-// Deck files as live state: starts from the build-time glob snapshot and stays
-// current as files change on disk from any source (editors, agents, terminal).
+// The glob in `files.js` is a compile-time transform, so its patterns can only
+// name extensions THIS KIT knows. A deck that declares its own file type is
+// invisible to it: `files[path]` is undefined, so an editor for that type is
+// handed '' forever -- it renders its empty state over a full file, and its
+// saves never come back through `files`. That breaks the `text`/`onChange`
+// contract the authoring doc teaches, for exactly the types the doc is for.
+//
+// So ask the SERVE what the deck actually declared and read those files in.
+// Declaration-driven rather than another literal list, because a literal list
+// is the bug. Best-effort: a published deck has no serve to ask, and there the
+// glob snapshot is all there is (its play code imports data through the
+// bundler anyway).
+// Null rather than throwing, for every reason a read can come back unusable:
+// no serve, a race with a delete, or a declared type that is binary (an image
+// the deck claims) -- the serve flags those rather than mangling them, and an
+// image decoded as utf-8 is nothing.
+async function readDeclaredFile(file) {
+  try {
+    const res = await fetch(`/__castle/files/read?path=${encodeURIComponent(file)}`);
+    if (!res.ok) return null;
+    const data = await res.json();
+    if (data.binary || typeof data.contents !== 'string') return null;
+    return data.contents;
+  } catch {
+    return null;
+  }
+}
+async function hydrateDeclaredFiles(ensurePath) {
+  let info;
+  let listing;
+  try {
+    [info, listing] = await Promise.all([
+      fetch('/__castle/files/info').then((r) => (r.ok ? r.json() : null)),
+      fetch('/__castle/files/list').then((r) => (r.ok ? r.json() : null)),
+    ]);
+  } catch {
+    return null;
+  }
+  if (!info || !listing) return null;
+  const exts = (info.fileTypes ?? [])
+    .filter((type) => type.data || (typeof type.editor === 'string' && type.editor !== 'kit'))
+    .map((type) => type.ext)
+    .filter((ext) => typeof ext === 'string' && ext && !DATA_EXTS.includes(ext));
+  if (exts.length === 0) return null;
+  for (const ext of exts) declaredDataExts.add(ext);
+  const paths = (listing.files ?? []).filter(
+    (file) => initialFiles[file] === undefined && exts.some((ext) => file.endsWith(ext))
+  );
+  const entries = await Promise.all(
+    paths.map(async (file) => {
+      const text = await readDeclaredFile(file);
+      return text === null ? null : [file, text];
+    })
+  );
+  const extra = Object.fromEntries(entries.filter(Boolean));
+  // `list` is the deck's CURATED listing, so a declared file the deck left out
+  // of visiblePaths is not in it. The shell can still open that file by path,
+  // and an editor handed '' for it would be back to the original bug -- so make
+  // sure whatever was actually opened is present, curated or not.
+  if (ensurePath && initialFiles[ensurePath] === undefined && extra[ensurePath] === undefined) {
+    const one = await readDeclaredFile(ensurePath);
+    if (one !== null) extra[ensurePath] = one;
+  }
+  return Object.keys(extra).length ? extra : null;
+}
+// Deck files as live state: starts from the build-time glob snapshot, fills in
+// the deck's own declared types from the serve, and stays current as files
+// change on disk from any source (editors, agents, terminal).
+//
 // `shouldSkipPath` lets an editor keep its own in-flight edits from being
-// clobbered by the fs echo of a stale write.
-export function useLiveDeckFiles({ shouldSkipPath } = {}) {
+// clobbered by the fs echo of a stale write. `ensurePath` names the file this
+// consumer actually opened, so a declared type outside the curated listing is
+// still read. `live: false` takes the hydration without the subscription --
+// what Play wants, since it never folds a change into a run in progress.
+export function useLiveDeckFiles({ shouldSkipPath, ensurePath, live = true } = {}) {
   const [files, setFiles] = useState(initialFiles);
+  // False until the declared-type read above has settled. An editor mounted
+  // before then would be handed '' for a file that exists, and an edit made in
+  // that window writes its empty document over the real one -- the same data
+  // loss the glob gap caused outright.
+  const [hydrated, setHydrated] = useState(false);
   const skipRef = useRef(shouldSkipPath);
   skipRef.current = shouldSkipPath;
-  useEffect(
-    () =>
-      onFilesChanged((event) => {
-        const stale = event.changes.find(isCodeChange);
-        if (stale) console.log(`[reload] ${panelTag()} code stale <- ${stale.path}`);
-        // Data still applies in the same event: a change that carries both is
-        // one edit, and holding the data back would leave the panel showing
-        // neither the old state nor the new one.
-        const dataChanges = event.changes.filter(
-          (change) => isDataFile(change.path) && !skipRef.current?.(change.path)
-        );
-        if (dataChanges.length === 0) return;
-        console.log(`[reload] ${panelTag()} data <- ${dataChanges.map((c) => c.path).join(', ')}`);
-        void applyDataChanges(dataChanges, setFiles);
-      }),
-    []
-  );
+  useEffect(() => {
+    let alive = true;
+    void hydrateDeclaredFiles(ensurePath).then((extra) => {
+      if (!alive) return;
+      // `current` wins: by the time this lands the editor may already have made
+      // an optimistic edit, and the disk copy behind it is stale.
+      if (extra) setFiles((current) => ({ ...extra, ...current }));
+      setHydrated(true);
+    });
+    return () => {
+      alive = false;
+    };
+  }, [ensurePath]);
+  useEffect(() => {
+    if (!live) return undefined;
+    return onFilesChanged((event) => {
+      const stale = event.changes.find(isCodeChange);
+      if (stale) console.log(`[reload] ${panelTag()} code stale <- ${stale.path}`);
+      // Data still applies in the same event: a change that carries both is
+      // one edit, and holding the data back would leave the panel showing
+      // neither the old state nor the new one.
+      const dataChanges = event.changes.filter(
+        (change) => isDataFile(change.path) && !skipRef.current?.(change.path)
+      );
+      if (dataChanges.length === 0) return;
+      console.log(`[reload] ${panelTag()} data <- ${dataChanges.map((c) => c.path).join(', ')}`);
+      void applyDataChanges(dataChanges, setFiles);
+    });
+  }, [live]);
-  return { files, setFiles };
+  return { files, setFiles, hydrated };
 }
 async function applyDataChanges(changes, setFiles) {

package/kits/physics-2d/scripts/svg-emission-guide.md CHANGED Viewed

@@ -90,5 +90,6 @@ A curvy silhouette with an overlay accent (later shape covers earlier stroke):
 </svg>
 ```
-**After writing new sprites referenced by the scene, `npm run restart`** so the
-deck's file glob picks them up.
+A newly written sprite is not in the deck's file glob until the panel reloads,
+so an actor pointing at one renders the placeholder until then. That is fine --
+the person reloads when they are ready; do not force it.

package/kits/physics-3d/CLAUDE.md CHANGED Viewed

@@ -74,8 +74,11 @@ bottom }`; any ref can pick a frame of a multi-frame pxart with
 - Controls are tank-style: W/S (or up/down) walk forward/back along facing,
   A/D (or left/right) turn; a pointer/touch drag is a virtual stick where the
   drag angle blends walking and turning. Space jumps.
-- After any edit: `npm run restart`. After a coherent change: `npm run check`
-  (lint + duplicate gate + single-file bundle) must pass.
+- Do not reload. Data edits (scenes, drawings, models) appear in the open
+  editors on their own. Code edits wait for the person to apply them --
+  reloading a panel someone is working in loses their place.
+- After a coherent change: `npm run check` (lint + duplicate gate +
+  single-file bundle) must pass.
 ## What lives where

package/kits/physics-3d/castle.json CHANGED Viewed

@@ -71,5 +71,5 @@
   "autoUpdateWhenImported": true,
   "deckId": "JH0SclbPVP0y",
   "cardId": "eXfAaUdbSU5A",
-  "publishedVersion": "2026-08-21T22:23:26.516Z"
+  "publishedVersion": "2026-08-21T23:15:28.646Z"
 }

package/kits/physics-3d/engine3d/modelFiles.js CHANGED Viewed

@@ -50,7 +50,7 @@ async function readDeckFile(path) {
   return typeof data.contents === 'string' ? data.contents : '';
 }
-async function applyDataChanges(changes, setFiles) {
+async function applyDataChanges(changes, setFiles, setDataVersion) {
   const updates = await Promise.all(
     changes.map(async (change) => {
       if (change.event === 'delete') return { path: change.path, text: null };
@@ -71,6 +71,7 @@ async function applyDataChanges(changes, setFiles) {
     }
     return next;
   });
+  setDataVersion((version) => version + 1);
 }
 // Live deck files including models. Same contract as the imported
@@ -82,6 +83,10 @@ async function applyDataChanges(changes, setFiles) {
 // loses their place, so the panel header offers it instead.
 export function useLiveDeckFiles3D({ shouldSkipPath } = {}) {
   const [files, setFiles] = useState(initialFiles3D);
+  // Bumped on every applied data change. The editors watch it to re-apply when
+  // a file they REFERENCE changed -- their own `text` is unchanged then, so it
+  // is the only signal that the drawing under an actor moved.
+  const [dataVersion, setDataVersion] = useState(0);
   const skipRef = useRef(shouldSkipPath);
   skipRef.current = shouldSkipPath;
@@ -92,12 +97,12 @@ export function useLiveDeckFiles3D({ shouldSkipPath } = {}) {
           (change) => isDataFile3D(change.path) && !skipRef.current?.(change.path)
         );
         if (dataChanges.length === 0) return;
-        void applyDataChanges(dataChanges, setFiles);
+        void applyDataChanges(dataChanges, setFiles, setDataVersion);
       }),
     []
   );
-  return { files, setFiles };
+  return { files, setFiles, dataVersion };
 }
 // The sprites map with parsed pxarts CACHED BY TEXT: re-parsing every file on

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "castle-web-cli",
-  "version": "0.4.131",
+  "version": "0.4.133",
   "type": "module",
   "bin": {
     "castle-web": "./dist/index.js"