kopular 0.21.0 → 0.21.1

package/LLM.md

@@ -636,7 +636,7 @@ import { runKopularApp } from "kopular/testing";
 
 const { window, cleanup } = await runKopularApp(srcDir, "app.ks", {
   includeKopularPackage: true,   // app consumes Kopular via `extern`, not relative `using`
-  extraFiles: ["http_runtime.js"], // non-.ks/.html siblings a `using` graph needs at runtime
+  extraFiles: ["config.json"],   // any sibling with an extension besides .ks/.html/.js
   fetchMock: (...args) => Promise.resolve({ ok: true, status: 200, text: () => Promise.resolve("body") }),
   url: "http://localhost/",
 });
@@ -650,7 +650,10 @@ try {
 Compiles `entryFileName` (plus everything else in `srcDir` it `using`s) via kopscript's
 `compileGraph`, binds jsdom onto `globalThis` (`document`/`Element`/`Event`/`location`/
 `history`/`window`/`fetch`) for the compiled ambient `extern` declarations to find, and
-runs it. `runKopularFixture(source, options)` is the sibling export for an inline fixture
+runs it. Every `.ks`/`.html`/`.js` file in `srcDir` is copied along with it — `.js`
+included specifically so a hand-written (not compiled) sibling like a `Router.AddLazyRoute`
+loader shim just works with no extra setup; `extraFiles` is only for a real dependency with
+some OTHER extension (e.g. a `.json` config file). `runKopularFixture(source, options)` is the sibling export for an inline fixture
 string instead of a real file (used by Kopular's own test suite; copies Kopular's *own*
 `.ks` sources alongside the fixture, so `using "./component"` resolves — only meaningful
 for testing Kopular itself, not an external consumer, which should use `runKopularApp`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kopular",
-  "version": "0.21.0",
+  "version": "0.21.1",
   "description": "Kopular: a small component framework for KopScript — components, reactive state, constructor-injected services, routing, real compiled templates, and HTTP, with no DI container",
   "type": "module",
   "license": "MIT",

package/src/testing.js

@@ -39,13 +39,25 @@ const OWN_SRC_DIR = dirname(fileURLToPath(import.meta.url));
 // than anything wrong with the copied files themselves.
 const TEMP_ROOT = join(OWN_SRC_DIR, "..", ".kopular-testing-tmp");
 
-// Copies every .ks/.html file from `srcDir` into `dir`, plus any filename
-// listed in `extraFiles` (e.g. http_runtime.js — a hand-written sibling a
-// compiled `using` graph needs at runtime, which the compiler wouldn't know
-// to copy on its own).
+// Copies every .ks/.html/.js file from `srcDir` into `dir`, plus any
+// filename listed in `extraFiles` for anything with a different extension.
+// `.js` is included alongside `.ks`/`.html` (not just left to `extraFiles`)
+// because a hand-written, not-compiled-from-`.ks` companion file is a
+// real, documented pattern now, not a one-off — most commonly a dynamic-
+// `import()` loader shim for `Router.AddLazyRoute` (see kopscript's own
+// LLM.md/README.md "extern" section, and Kopular's own "Lazy routes"
+// docs), reached via a RELATIVE `extern ... from "./whatever";` the test
+// author can't always know the exact filename of in advance — unlike
+// http_runtime.js (a fixed name Kopular itself ships and controls, still
+// the right fit for `extraFiles`), an app's own loader shim can be named
+// anything. Any pre-existing `.js` with the same name as a `.ks` file
+// gets harmlessly overwritten moments later by that `.ks`'s own fresh
+// compiled output — copying it first only matters for a `.js` with no
+// `.ks` counterpart at all, which is exactly the hand-written case this
+// exists for.
 function copySources(srcDir, dir, extraFiles) {
   for (const name of readdirSync(srcDir)) {
-    if (name.endsWith(".ks") || name.endsWith(".html") || extraFiles.includes(name)) {
+    if (name.endsWith(".ks") || name.endsWith(".html") || name.endsWith(".js") || extraFiles.includes(name)) {
       cpSync(join(srcDir, name), join(dir, name));
     }
   }
@@ -155,8 +167,9 @@ async function compileAndRun(dir, entryFileName, options) {
  *   installed `kopular` package into a temp node_modules/, for an entry
  *   that consumes Kopular via `extern ... from "kopular/..."` rather than
  *   relative `using`. Default false.
- * @param {string[]} [options.extraFiles] - extra filenames (not .ks/.html)
- *   to copy from srcDir, e.g. ["http_runtime.js"].
+ * @param {string[]} [options.extraFiles] - extra filenames to copy from
+ *   srcDir beyond .ks/.html/.js (which are always copied), for anything
+ *   with a different extension.
  * @param {(...args: unknown[]) => unknown} [options.fetchMock] - replaces
  *   the global `fetch` for the duration of the run, for testing `Http`.
  * @param {string} [options.url] - the jsdom document's URL. Default