@dfosco/storyboard-core 4.2.0-beta.20 → 4.2.0-beta.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dfosco/storyboard-core",
-  "version": "4.2.0-beta.20",
+  "version": "4.2.0-beta.21",
   "type": "module",
   "license": "MIT",
   "bin": {

package/src/canvas/server.js

@@ -38,7 +38,7 @@ import fs from 'node:fs'
 import path from 'node:path'
 import { Buffer } from 'node:buffer'
 import { materializeFromText, serializeEvent } from './materializer.js'
-import { toCanvasId, parseCanvasId, canvasIdBasename, isLegacyCanvasId } from './identity.js'
+import { toCanvasId, parseCanvasId } from './identity.js'
 import {
   GH_INSTALL_URL,
   GitHubEmbedError,
@@ -207,15 +207,11 @@ function parseExportNames(filePath) {
 
 /**
  * Find a canvas JSONL file by canonical ID.
- * Tries an exact canonical match first. If the input is a legacy bare name
- * (no path segments), falls back to basename matching across all canvases.
- * Basename fallback only succeeds when exactly one canvas matches — ambiguous
- * lookups return null with a console warning.
+ * Only matches canonical path-based IDs from toCanvasId().
  */
 function findCanvasPath(root, canvasId) {
   const files = findCanvasFiles(root)
 
-  // 1. Exact canonical match — always wins
   for (const file of files) {
     const id = toCanvasId(file)
     if (id === canvasId) {
@@ -223,26 +219,6 @@ function findCanvasPath(root, canvasId) {
     }
   }
 
-  // 2. Legacy basename fallback — only for bare names like "overview"
-  if (isLegacyCanvasId(canvasId)) {
-    const matches = []
-    for (const file of files) {
-      if (canvasIdBasename(toCanvasId(file)) === canvasId) {
-        matches.push(file)
-      }
-    }
-    if (matches.length === 1) {
-      return path.resolve(root, matches[0])
-    }
-    if (matches.length > 1) {
-      console.warn(
-        `[storyboard] Ambiguous canvas name "${canvasId}" matches ${matches.length} canvases: ` +
-        matches.map(f => toCanvasId(f)).join(', ') +
-        '. Use the full path-based ID to disambiguate.'
-      )
-    }
-  }
-
   return null
 }