@dfosco/storyboard-core 2.4.0 → 2.5.0

package/package.json

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

package/src/viewfinder.js

@@ -16,13 +16,17 @@ export function hash(str) {
 /**
  * Resolve the target route path for a flow.
  *
+ * Priority:
  * 1. If flow name matches a known route (case-insensitive), use that route
- * 2. If flow data has a top-level `route`, or `meta.route` / `sceneMeta.route`, use that
- * 3. Fall back to root "/"
+ * 2. If flow data has an explicit top-level `route`, or `meta.route` / `flowMeta.route`, use that
+ * 3. If flow data has `_route` (inferred from file path by Vite plugin), use that
+ * 4. Fall back to root "/"
+ *
+ * Flows with `meta.default: true` targeting a route omit the `?flow=` param.
  *
  * @param {string} flowName
  * @param {string[]} knownRoutes - Array of route names (e.g. ["Dashboard", "Repositories"])
- * @returns {string} Full path with ?flow= param
+ * @returns {string} Full path with optional ?flow= param
  */
 export function resolveFlowRoute(flowName, knownRoutes = []) {
   // Case-insensitive match against known routes
@@ -34,14 +38,22 @@ export function resolveFlowRoute(flowName, knownRoutes = []) {
     }
   }
 
-  // Check for explicit route: top-level `route`, then meta.route, then legacy sceneMeta.route
   try {
     const data = loadFlow(flowName)
-    const route = data?.route || data?.meta?.route || data?.flowMeta?.route || data?.sceneMeta?.route
-    if (route) {
-      const normalized = route.startsWith('/') ? route : `/${route}`
+
+    // Check for explicit route: top-level `route`, then meta.route, then legacy sceneMeta.route
+    const explicitRoute = data?.route || data?.meta?.route || data?.flowMeta?.route || data?.sceneMeta?.route
+    if (explicitRoute) {
+      const normalized = explicitRoute.startsWith('/') ? explicitRoute : `/${explicitRoute}`
+      if (data?.meta?.default === true) return normalized
       return `${normalized}?flow=${encodeURIComponent(flowName)}`
     }
+
+    // Use inferred route from file path (injected by Vite data plugin)
+    if (data?._route) {
+      if (data?.meta?.default === true) return data._route
+      return `${data._route}?flow=${encodeURIComponent(flowName)}`
+    }
   } catch {
     // ignore load errors
   }

package/src/viewfinder.test.js

@@ -12,6 +12,9 @@ const makeIndex = () => ({
     'meta-author': { flowMeta: { author: 'dfosco' }, title: 'With Author' },
     'meta-authors': { flowMeta: { author: ['dfosco', 'heyamie', 'branonconor'] }, title: 'Multi Author' },
     'meta-both': { flowMeta: { route: '/Overview', author: 'octocat' }, title: 'Both' },
+    'inferred-route': { _route: '/Dashboard', title: 'Inferred' },
+    'inferred-default': { _route: '/Settings', meta: { default: true }, title: 'Default Flow' },
+    'explicit-wins': { route: '/Forms', _route: '/Dashboard', title: 'Explicit Wins' },
   },
   objects: {},
   records: {},
@@ -105,6 +108,31 @@ describe('resolveFlowRoute', () => {
     })
     expect(resolveFlowRoute('conflict', [])).toBe('/Forms?flow=conflict')
   })
+
+  it('uses _route when no explicit route exists', () => {
+    expect(resolveFlowRoute('inferred-route', routes)).toBe('/Dashboard?flow=inferred-route')
+  })
+
+  it('prefers explicit route over _route', () => {
+    expect(resolveFlowRoute('explicit-wins', routes)).toBe('/Forms?flow=explicit-wins')
+  })
+
+  it('omits ?flow= when meta.default is true (inferred route)', () => {
+    expect(resolveFlowRoute('inferred-default', routes)).toBe('/Settings')
+  })
+
+  it('omits ?flow= when meta.default is true (explicit route)', () => {
+    init({
+      flows: { 'default-explicit': { route: '/Overview', meta: { default: true } } },
+      objects: {},
+      records: {},
+    })
+    expect(resolveFlowRoute('default-explicit', [])).toBe('/Overview')
+  })
+
+  it('still appends ?flow= when meta.default is absent even with _route', () => {
+    expect(resolveFlowRoute('inferred-route', [])).toBe('/Dashboard?flow=inferred-route')
+  })
 })
 
 describe('getFlowMeta', () => {