tosijs-ui 1.6.13 → 1.6.15

package/CHANGELOG.md

@@ -1,5 +1,60 @@
 # Changelog
 
+## 1.6.15
+
+Live-example fixes from tosijs-3d adoption. All fixes to the doc-system's
+editing surface; no API changes.
+
+### Fixed
+
+- **Save to source works when the doc comment is indented.** A `/*# … */` doc
+  comment is often indented in the source; the extractor dedents its fenced blocks
+  (so examples render with correct ordinals), but the raw scan required the closing
+  ` ``` ` right after a newline, so an indented file matched **zero** blocks and
+  every save failed with "no matching block". The scan is now indentation-aware,
+  and edits compare dedented / write re-indented so a block keeps its place. The
+  failure alert is also split into a precise message (page↔source ordinal mismatch
+  vs. a genuine no-op).
+- **The pop-out editor window ("view/edit code in a new window") now has the full
+  menu.** It previously opened as a bare instance, so View changes, Save changes
+  (local), and Save to source were hidden; the source↔doc key and the pristine
+  snapshot are now passed through, so the pop-out offers — and can execute — the
+  same actions as the main window.
+- **Opening the editor in a new window closes the inline code view** in the main
+  window if it was open (the pop-out owns editing).
+- **The doc extractor now only treats a `/*#` block as a doc when it starts a
+  line** (whitespace-only before the slash), so a `/*#` inside a `//` comment, a
+  string, or mid-line can't be scraped as a spurious doc page.
+
+## 1.6.14
+
+Lighter live-example transpilation. Additive; affects only how the live-example
+runner loads its (optional, lazy) transpiler.
+
+### Changed
+
+- **Live examples now use tjs-lang 0.8.5's self-contained browser bundles**
+  (`tjs-lang/browser` + `tjs-lang/browser/from-ts`). The transpiler is a single
+  self-contained chunk, and the TypeScript compiler is **lazy-loaded from a CDN at
+  runtime** only when a `ts` example actually transpiles — so `typescript` (~MB)
+  and the transpiler's own deps are never in a consumer's dependency graph. This
+  also fixes the `ts` example path, which previously tried to load the TypeScript
+  compiler through a CDN transform that timed out on its size.
+- **Robust transpiler loading.** The doc-site build now ships the tjs-lang browser
+  bundles **same-origin** (copied next to the iife under `/tjs/`, with a global
+  pointing the loader at them), so live examples never depend on a third-party
+  CDN's propagation timing or uptime. The loader prefers the same-origin copy,
+  then falls back through a multi-CDN chain (jsdelivr → unpkg → esm.sh). Installed
+  ESM consumers resolve the peer locally as before.
+- `tjs-lang` optional peer bumped to `^0.8.6`.
+
+### Fixed
+
+- A garbled character in the first `example` doc snippet that made it throw.
+- Live examples could all break for the propagation window after a tjs-lang
+  release, when the single pinned CDN 404'd the just-published version (now
+  served same-origin + multi-CDN).
+
 ## 1.6.13
 
 The documentation system becomes a real publishing pipeline: every doc site can

package/dist/doc-system/site/docs.js

@@ -99,8 +99,9 @@ import { pinnedSort } from '../nav-tree';
 const TRIM_REGEX = /^#+ |`/g;
 function metadata(content, filePath) {
     // Ignore metadata-style comments INSIDE /*# ... */ doc blocks — those are
-    // documentation examples, not real directives.
-    const scannable = content.replace(/\/\*#[\s\S]*?\*\//g, '');
+    // documentation examples, not real directives. Only line-starting blocks count
+    // as docs (see findMarkdownFiles), so strip exactly those.
+    const scannable = content.replace(/^[ \t]*\/\*#[\s\S]*?\*\//gm, '');
     const source = scannable.match(/<!--(\{.*\})-->|\/\*(\{.*\})\*\//);
     let data = {};
     if (source) {
@@ -151,7 +152,11 @@ function findMarkdownFiles(paths, ignore) {
             }
             else if (['.ts', '.js', '.css'].includes(path.extname(file))) {
                 const content = fs.readFileSync(filePath, 'utf8');
-                const docs = content.match(/\/\*#[\s\S]+?\*\//g) || [];
+                // A /*# … */ block is only a doc when it STARTS a line (whitespace-only
+                // before the slash). This keeps a `/*#` that appears inside a // comment,
+                // a string, or otherwise mid-line from being scraped as a spurious doc.
+                // m[1] is the block without its leading indentation.
+                const docs = [...content.matchAll(/^[ \t]*(\/\*#[\s\S]+?\*\/)/gm)].map((m) => m[1]);
                 if (docs.length) {
                     const markdown = docs.map((s) => s.substring(3, s.length - 2).trim());
                     const text = markdown.join('\n\n');

package/dist/doc-system/site/orchestrator.js

@@ -113,7 +113,12 @@ export async function buildSite(config) {
             naming: scriptName,
             // tjs-lang transpiles live examples; it's dynamically import()'d at
             // runtime (falling back to CDN), so keep it out of the bundle.
-            external: ['tjs-lang', 'tjs-lang/lang', ...(config.bundleExternals ?? [])],
+            external: [
+                'tjs-lang',
+                'tjs-lang/browser',
+                'tjs-lang/browser/from-ts',
+                ...(config.bundleExternals ?? []),
+            ],
         });
         if (!result.success) {
             console.error('bundle build failed');
@@ -170,6 +175,27 @@ export async function buildSite(config) {
         // root copy stays for the npm package's `files`).
         await $ `cp llms.txt ${PUBLIC}/llms.txt`.text();
     }
+    // Serve the tjs-lang browser bundles SAME-ORIGIN so live examples never depend
+    // on a third-party CDN's propagation timing — a freshly-published tjs-lang
+    // version 404s on a CDN until it caches it (minutes–hours), which would break
+    // every example. We ship the exact bundles the build resolved, and a global
+    // tells the loader to prefer them (it falls back to the CDN chain if absent).
+    // Optional: skipped if tjs-lang isn't installed.
+    let tjsHead = '';
+    try {
+        const browser = Bun.resolveSync('tjs-lang/browser', PROJECT_ROOT);
+        const fromTs = Bun.resolveSync('tjs-lang/browser/from-ts', PROJECT_ROOT);
+        await $ `mkdir -p ${PUBLIC}/tjs`.text();
+        await $ `cp ${browser} ${PUBLIC}/tjs/tjs-browser.js`.text();
+        await $ `cp ${fromTs} ${PUBLIC}/tjs/tjs-browser-from-ts.js`.text();
+        const bp = config.basePath;
+        const base = !bp || bp === '/' ? '/tjs/' : bp.replace(/\/$/, '') + '/tjs/';
+        tjsHead = `<script>globalThis.__TJS_LOCAL_BASE=${JSON.stringify(base)}</script>`;
+        console.log(`tjs-lang bundles served same-origin at ${base}`);
+    }
+    catch {
+        // tjs-lang not installed — live examples fall back to the CDN chain
+    }
     // Generate the static, pre-rendered doc site (one /slug/index.html per doc).
     // Runs after the static-asset copy so the generated index.html (README) wins,
     // and after the bundle copy so every page's <script src> resolves.
@@ -186,7 +212,7 @@ export async function buildSite(config) {
         localizedStrings: config.localizedStrings,
         favicon: config.favicon,
         ogImage: config.ogImage,
-        headExtra: config.headExtra,
+        headExtra: [config.headExtra, tjsHead].filter(Boolean).join('') || undefined,
         scriptUrl: config.scriptUrl,
         basePath: config.basePath,
     });