@dfosco/storyboard-core 2.3.0 → 2.5.0

package/package.json

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

package/src/svelte-plugin-ui/components/Viewfinder.svelte

@@ -550,11 +550,11 @@
     display: flex;
     flex-direction: column;
     gap: var(--base-size-8);
+  }
 
-    & .listItem:only-child {
-      border: 1px solid var(--borderColor-muted, #30363d);
-      border-radius: var(--base-size-6);
-    }
+  .protoGroup > .listItem {
+    border: 1px solid var(--borderColor-muted, #30363d);
+    border-radius: var(--base-size-6);
   }
 
   .folderGroup {
@@ -644,7 +644,7 @@
 
   .protoHeader[aria-expanded="true"] .cardBody {
     background-color: var(--bgColor-muted);
-    border-radius: var(--base-size-8);
+    border-radius: var(--base-size-6);
   }
 
   .cardBody {
@@ -653,7 +653,7 @@
 
   .cardBody:hover {
     background-color: var(--bgColor-muted);
-    border-radius: var(--base-size-8);
+    border-radius: var(--base-size-6);
   }
 
   .protoName {
@@ -736,7 +736,6 @@
     flex-direction: column;
   }
 
-  .folderGroup .listItem,
   .flowItem {
     border: 1px solid var(--borderColor-muted);
     padding: 0;
@@ -750,14 +749,12 @@
     border-top-left-radius: var(--base-size-6);
     border-top-right-radius: var(--base-size-6);
   }
-  
-  .folderGroup .listItem:last-child,
+
   .flowItem:last-child {
     border-bottom-left-radius: var(--base-size-6);
     border-bottom-right-radius: var(--base-size-6);
   }
 
-  .folderGroup .listItem:only-child,
   .flowItem:only-child {
     border-radius: var(--base-size-6);
   }

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
   }
@@ -101,7 +113,7 @@ export function buildPrototypeIndex(knownRoutes = []) {
       icon: meta.icon || null,
       team: meta.team || null,
       tags: meta.tags || null,
-      hideFlows: meta.hideFlows || false,
+      hideFlows: meta.hideFlows ?? raw?.hideFlows ?? false,
       folder: raw?.folder || null,
       flows: [],
     }

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', () => {
@@ -183,6 +211,20 @@ describe('buildPrototypeIndex', () => {
     expect(proto.hideFlows).toBe(false)
   })
 
+  it('reads hideFlows from top-level prototype metadata (outside meta key)', () => {
+    init({
+      flows: { 'TopLevel/only-flow': { meta: { title: 'Only Flow' } } },
+      objects: {},
+      records: {},
+      prototypes: {
+        TopLevel: { meta: { title: 'Top Level' }, hideFlows: true },
+      },
+    })
+    const { prototypes } = buildPrototypeIndex([])
+    const proto = prototypes.find(p => p.dirName === 'TopLevel')
+    expect(proto.hideFlows).toBe(true)
+  })
+
   it('groups prototypes into folders when folder field is set', () => {
     init({
       flows: {