@lalalic/markcut 2.4.3 → 2.4.5

package/AGENTS.md

@@ -242,9 +242,9 @@ npx vitest run tests/render.test.ts -t "Test Name"  # single test
 1. **TransitionSeries + isBackground**: `TransitionSeries` from `@remotion/transitions` rejects non-Sequence/Transition children. Don't mix `isBackground:true` children with `isSeries:true, transition:"..."` at the same level.
 2. **Video endAt formula**: Must be `startFrom_frames + ((endAt_source - startFrom_source) * fps / playbackRate)` — changing to `endAt_source * fps` breaks slow-motion playback.
 3. **`signalstats` ffmpeg filter**: Doesn't output YAVG on all systems; use file size as proxy for visual content.
-4. **Asset paths**: Local files are resolved via `staticFile()` — files go in `public/` and are referenced without the `public/` prefix.
+4. **Asset paths**: All `src` paths in markdown are resolved relative to the `.md` file's directory. Preview mode serves files via a multi-directory HTTP server; render mode uses absolute paths directly (bypassing `staticFile()`). Never use a `public/` prefix in markdown `src`.
 5. **Map rendering**: Depends on external tile availability; may produce blank frames offline.
-6. **CLI md render needs three things preview gets for free**: (a) `parseMarkdownVariants(raw)` (returns `{base, variants}`) — `parseMarkdownDescriptive` returns the root directly; (b) bundling the ` ```js imports ``` ` block via `bundleFromEntries` and setting `root.imports` (otherwise jsx components are unregistered → slides render as unstyled black frames); (c) staging `.markcut/` assets into `public/.render-assets/` and rewriting srcs to relative paths (remotion render only serves its publicDir). All three live in `src/render/cli.mjs`; the import-map shim for shared react/remotion instances is `src/utils/component-import-map.ts` (must stay in sync with `player/browser.tsx` + `bundler.mjs getSharedExternals`).
+6. **CLI md render needs two things preview gets for free**: (a) `parseMarkdownVariants(raw)` (returns `{base, variants}`) — `parseMarkdownDescriptive` returns the root directly; (b) bundling the ` ```js imports ``` ` block via `bundleFromEntries` and setting `root.imports` (otherwise jsx components are unregistered → slides render as unstyled black frames). Asset staging (`stageLocalAssets`) was removed — render now keeps absolute paths from `resolve.ts`, bypassing `staticFile()` entirely. All live in `src/render/cli.mjs`; the import-map shim for shared react/remotion instances is `src/utils/component-import-map.ts` (must stay in sync with `player/browser.tsx` + `bundler.mjs getSharedExternals`).
 7. **Publishing**: verify `npx @lalalic/markcut render <md>` from a clean temp dir before/after publish — v1.1.1 shipped with cli.mjs/pipeline.mjs out of sync and md rendering was completely broken. Also beware stale npx caches (`~/.npm/_npx`) silently running old code.
 8. **Clean break from `actions[]`**: leaf timing is now flat base fields (`start`/`end`/`duration`/`startFrom`/`endAt`). Old compiled JSON carrying `actions:[{...}]` no longer parses into a duration — re-compile from markdown/descriptive JSON, or migrate `actions[0]` → flat fields (drop `id`/`volume`/`style`/`effectId`; keep `start`/`end`/`startFrom`/`endAt`/`loop`).
 

package/README.md

@@ -91,10 +91,7 @@ Inline components can be defined entirely in the imports block using `export fun
 
 | Document | What it covers |
 |---|---|
-| [docs/json-descriptive.md](docs/json-descriptive.md) | Full JSON descriptive schema reference |
 | [docs/markdown-strict-descriptive.md](docs/markdown-strict-descriptive.md) | Markdown descriptive syntax reference |
-| [docs/label-mode.md](docs/label-mode.md) | Label mode player and workflow |
-| [docs/edit-mode.md](docs/edit-mode.md) | Live edit mode with SSE reload |
 
 
 ## Variants

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lalalic/markcut",
-  "version": "2.4.3",
+  "version": "2.4.5",
   "description": "Markdown-to-video engine. Describe scenes in markdown, get a rendered video.",
   "bin": {
     "markcut": "bin/markcut"

package/skills/markcut/docs/markdown-descriptive.md

@@ -43,7 +43,7 @@ This section defines the markcut Markdown Descriptive grammar. An agent should r
 |---|---|---|---|
 | H1 | `# video` | **Root** — video canvas definition | Exactly one `# video` required. Everything else is nested inside this section. |
 | H1 | `# zh`, `# youtube`, `# portrait` etc. | **Variant** — override section for alternate versions | Any `# <name>` other than `video`. Only contains override configs + variant-prefixed overrides on base nodes. |
-| H2 | `## <name>` | **Scene** — top-level scene container | `name` = single token (no spaces). Optional `title` via ` - ` separator: `## Chapter1 - The Beginning` → name=`Chapter1`, title=`The Beginning`. |
+| H2 | `## <name>` | **Scene** — top-level scene container | `name` = single word (no spaces).|
 | H3 | `### <name>` | **Nested scene** — sub-scene inside parent | Same rules as H2. Creates nesting: parent scene's children array includes this as a sub-container. |
 | H4+ | `#### <name>` | **Deeply nested** — further nesting | Works identically; depth reflects nesting level. |
 
@@ -258,8 +258,14 @@ width:1080 height:1920 fps:30 layout:series
 - script "Hello world"
 ```
 
-### 13. verify
-`npx @lalalic/markcut verify book.md` to verify the markdown document is valid and can be rendered. It will check for missing required fields, invalid values, and other common issues.
+### 13. `src` Path Resolution — Context Root
+all src paths are resolved relative to the markdown file's location. For example, if your markdown file is at `./videos/course.md`, then `src:./assets/bg.jpg` resolves to `./videos/assets/bg.jpg`. Absolute paths (starting with `/`) are resolved relative to the project root.
+
+
+### 14. verify
+- all assets path are resolved relative to the markdown file's location. 
+- `npx @lalalic/markcut verify book.md` to verify the markdown document is valid and can be rendered. It will check for missing required fields, invalid values, and other common issues.
+
 
 ## Template Variables
 

package/src/render/cli.mjs

@@ -7,9 +7,8 @@
  *                                        (TTS/STT/durations)
  */
 import { execSync, spawn } from "node:child_process";
-import { readFileSync, writeFileSync, mkdirSync, existsSync, copyFileSync } from "node:fs";
-import { createHash } from "node:crypto";
-import { resolve, dirname, join, extname } from "node:path";
+import { readFileSync, writeFileSync, mkdirSync, existsSync } from "node:fs";
+import { resolve, dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import {
   DEFAULT_TTS_CLI, DEFAULT_STT_CLI, DEFAULT_TTI_CLI, DEFAULT_TTV_CLI,
@@ -96,62 +95,10 @@ Commands:
  * Use --verbose to see every frame line (original behavior).
  */
 
-/**
- * Resolve a potentially relative source path against baseDir, skipping
- * absolute paths, URLs, and data URIs.
- */
-function resolveAssetPath(src, baseDir) {
-  if (!src || typeof src !== "string") return null;
-  // Already absolute on disk
-  if (src.startsWith("/")) return src;
-  // URL or data URI — can't stage locally
-  if (/^(https?:|data:|blob:)/.test(src)) return null;
-  // Relative path: resolve against the input file's directory
-  const abs = resolve(baseDir, src);
-  if (existsSync(abs)) return abs;
-  return null;
-}
-
-/**
- * Stage all local src files (absolute and relative) into
- * ROOT/public/.render-assets/ so `npx remotion render` (publicDir = ROOT/public)
- * can serve them. Relative paths are resolved against `baseDir` (the input
- * file's directory). The preview server handles these paths via multi-root
- * serving; the render CLI needs them staged into the one public dir Remotion
- * knows about.
- */
-function stageLocalAssets(tree, baseDir) {
-  const assetsDir = join(ROOT, "public", ".render-assets");
-  const seen = new Map();
-  const walk = (node) => {
-    if (!node || typeof node !== "object") return;
-    if (typeof node.src === "string") {
-      const absPath = resolveAssetPath(node.src, baseDir);
-      if (absPath) {
-        let staged = seen.get(absPath);
-        if (!staged) {
-          const name = createHash("md5").update(absPath).digest("hex").slice(0, 12) + extname(absPath);
-          mkdirSync(assetsDir, { recursive: true });
-          copyFileSync(absPath, join(assetsDir, name));
-          staged = `.render-assets/${name}`;
-          seen.set(absPath, staged);
-        }
-        node.src = staged;
-      }
-    }
-    for (const value of Object.values(node)) {
-      if (value && typeof value === "object") walk(value);
-    }
-  };
-  walk(tree);
-  return tree;
-}
-
-function renderOne(streamTree, outputPath, verbose, baseDir) {
-  const adapted = stageLocalAssets(JSON.parse(JSON.stringify(streamTree)), baseDir);
+function renderOne(streamTree, outputPath, verbose) {
   const tmpProps = join(ROOT, ".tmp", "render-stream.json");
   mkdirSync(dirname(tmpProps), { recursive: true });
-  writeFileSync(tmpProps, JSON.stringify({ root: adapted }));
+  writeFileSync(tmpProps, JSON.stringify({ root: streamTree }));
 
   mkdirSync(dirname(outputPath), { recursive: true });
 
@@ -409,19 +356,18 @@ edit=${DEFAULT_EDIT_CLI}`);
         const { bundleFromEntries } = await import("../player/bundler.mjs");
         const entries = parseImportsBlock(rawSource);
         const extraSpecs = extractDependencySpecs(rawSource);
-        const bundleDir = join(ROOT, "public", ".render-assets");
+        const bundleDir = join(ROOT, ".tmp", "component-bundle");
         mkdirSync(bundleDir, { recursive: true });
         const bundle = await bundleFromEntries(entries, extraSpecs, rawSource, bundleDir);
         if (bundle.url) {
-          streamTree.imports = ".render-assets/" + bundle.url.split("/").pop();
+          streamTree.imports = join(bundleDir, bundle.url.split("/").pop());
           console.log(`  \u2705 components \u2192 ${bundle.exports.join(", ")}`);
         }
       }
     }
 
     const output = args.output ? resolve(args.output) : join(ROOT, "out", "video.mp4");
-    const fileDir = dirname(resolve(args.file));
-    await renderOne(streamTree, output, args.verbose, fileDir);
+    await renderOne(streamTree, output, args.verbose);
 
     console.log("\n✅ Render complete.");
     process.exit(0);