@dfosco/storyboard-core 1.6.0 → 1.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dfosco/storyboard-core",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "type": "module",
   "license": "MIT",
   "repository": {

package/src/index.js

@@ -32,7 +32,7 @@ export { mountDevTools } from './devtools.js'
 export { mountSceneDebug } from './sceneDebug.js'
 
 // Viewfinder utilities
-export { hash, resolveSceneRoute } from './viewfinder.js'
+export { hash, resolveSceneRoute, getSceneMeta } from './viewfinder.js'
 
 // Comments system
 export { initCommentsConfig, getCommentsConfig, isCommentsEnabled } from './comments/config.js'

package/src/viewfinder.js

@@ -17,7 +17,7 @@ export function hash(str) {
  * Resolve the target route path for a scene.
  *
  * 1. If scene name matches a known route (case-insensitive), use that route
- * 2. If scene data has a `route` key, use that
+ * 2. If scene data has a `sceneMeta.route` or `route` key, use that
  * 3. Fall back to root "/"
  *
  * @param {string} sceneName
@@ -32,12 +32,13 @@ export function resolveSceneRoute(sceneName, knownRoutes = []) {
     }
   }
 
-  // Check for explicit `route` key in scene data
+  // Check for explicit route in sceneMeta or top-level route key
   try {
     const data = loadScene(sceneName)
-    if (data?.route) {
-      const route = data.route.startsWith('/') ? data.route : `/${data.route}`
-      return `${route}?scene=${encodeURIComponent(sceneName)}`
+    const route = data?.sceneMeta?.route || data?.route
+    if (route) {
+      const normalized = route.startsWith('/') ? route : `/${route}`
+      return `${normalized}?scene=${encodeURIComponent(sceneName)}`
     }
   } catch {
     // ignore load errors
@@ -45,3 +46,18 @@ export function resolveSceneRoute(sceneName, knownRoutes = []) {
 
   return `/?scene=${encodeURIComponent(sceneName)}`
 }
+
+/**
+ * Get sceneMeta for a scene (route, author, etc).
+ *
+ * @param {string} sceneName
+ * @returns {{ route?: string, author?: string } | null}
+ */
+export function getSceneMeta(sceneName) {
+  try {
+    const data = loadScene(sceneName)
+    return data?.sceneMeta || null
+  } catch {
+    return null
+  }
+}

package/src/viewfinder.test.js

@@ -1,5 +1,5 @@
 import { init } from './loader.js'
-import { hash, resolveSceneRoute } from './viewfinder.js'
+import { hash, resolveSceneRoute, getSceneMeta } from './viewfinder.js'
 
 const makeIndex = () => ({
   scenes: {
@@ -8,6 +8,9 @@ const makeIndex = () => ({
     'custom-route': { route: 'Overview', title: 'Custom' },
     'absolute-route': { route: '/Forms', title: 'Absolute' },
     'no-route': { title: 'No route key' },
+    'meta-route': { sceneMeta: { route: 'Repositories' }, title: 'Meta Route' },
+    'meta-author': { sceneMeta: { author: 'dfosco' }, title: 'With Author' },
+    'meta-both': { sceneMeta: { route: '/Overview', author: 'octocat' }, title: 'Both' },
   },
   objects: {},
   records: {},
@@ -84,4 +87,39 @@ describe('resolveSceneRoute', () => {
     })
     expect(resolveSceneRoute('has spaces', [])).toBe('/?scene=has%20spaces')
   })
+
+  it('uses sceneMeta.route when no route matches', () => {
+    expect(resolveSceneRoute('meta-route', routes)).toBe('/Repositories?scene=meta-route')
+  })
+
+  it('uses sceneMeta.route with absolute path', () => {
+    expect(resolveSceneRoute('meta-both', routes)).toBe('/Overview?scene=meta-both')
+  })
+
+  it('prefers sceneMeta.route over top-level route key', () => {
+    init({
+      scenes: { conflict: { route: 'Forms', sceneMeta: { route: 'Dashboard' } } },
+      objects: {},
+      records: {},
+    })
+    expect(resolveSceneRoute('conflict', [])).toBe('/Dashboard?scene=conflict')
+  })
+})
+
+describe('getSceneMeta', () => {
+  it('returns sceneMeta when present', () => {
+    expect(getSceneMeta('meta-author')).toEqual({ author: 'dfosco' })
+  })
+
+  it('returns sceneMeta with both fields', () => {
+    expect(getSceneMeta('meta-both')).toEqual({ route: '/Overview', author: 'octocat' })
+  })
+
+  it('returns null when no sceneMeta', () => {
+    expect(getSceneMeta('default')).toBeNull()
+  })
+
+  it('returns null for nonexistent scene', () => {
+    expect(getSceneMeta('nonexistent')).toBeNull()
+  })
 })