The selected-node runner was treating audioInput and videoInput as no-ops,
so select-executing these nodes with upstream connections wouldn't pull
connected data. Now matches executeWorkflow's hydration logic.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The render guard at line 222 blocked all Gemini-provider models from
rendering parameters, including Veo models. Hoisted the isVeoModel
check to component scope and added the same exception used in the
fetch guard so Veo parameters (aspect ratio, duration, resolution)
display correctly.
Fixes#106
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix redo shortcut by normalizing event.key to lowercase (Shift makes it "Z")
- Flush pendingDataSnapshot in undo/redo instead of discarding it, preserving
the pre-edit checkpoint so Cmd+Z doesn't skip states
- Recompute dimmedNodeIds after undo/redo to keep dimming in sync
- Set deleteCheckpointActive in removeEdge before clearStaleInputImages to
prevent split undo entries
- Wire videoInput in handleMenuSelect and isValidConnection for video connections
- Validate video MIME type against ACCEPTED_FORMATS whitelist instead of broad regex
- Use URL.createObjectURL(file) for metadata extraction instead of base64→Blob
- Add Shift+Y (Video Input) to keyboard shortcuts dialog
- Update executeNode JSDoc to include videoInput as data-source no-op
- Clear all *Ref/*Refs fields dynamically in clearNodeImageRefs
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ModelParameters had an early return for all Gemini provider models,
skipping the API schema fetch. This was correct for Gemini image
models (which have built-in parameters) but prevented Veo video
models from loading their aspect ratio, duration, and resolution
parameters from the /api/models/[modelId] endpoint.
Closes#106
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Veo models were excluded from ModelParameters rendering, hiding their
configurable properties. Remove the isVeoModel() guard so Veo models
display parameters like all other providers.
Closes#106
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a text link with banana icon at the bottom of the links list in
the welcome modal, linking to https://nodebananapro.com.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
If a debounced data-edit snapshot was pending when loading or clearing
a workflow, it could fire after undoManager.clear() and push stale
state into the new workflow's undo history. Now both paths cancel the
pending timer and snapshot before clearing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
If a debounced data-edit snapshot was pending when loading or clearing
a workflow, it could fire after undoManager.clear() and push stale
state into the new workflow's undo history. Now both paths cancel the
pending timer and snapshot before clearing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a text link with banana icon below the description in the welcome
modal left column, linking to https://nodebananapro.com.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
React Flow v12's deleteElements fires onEdgesChange(remove) BEFORE
onNodesChange(remove), not after. The previous fix assumed the opposite
order, so the microtask-based nodeRemoveCheckpointActive flag was never
true when onEdgesChange ran.
Replace with deleteCheckpointActive — a bidirectional flag set by
whichever handler (edges or nodes) fires first. The second handler
skips its checkpoint. Use setTimeout(0) instead of Promise.resolve()
so the flag survives all microtasks in the current event-loop turn.
Also suppresses debounced undo snapshots from clearStaleInputImages
side effects during the same deletion cycle.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When deleting a connected node, clearStaleInputImages calls updateNodeData
on target nodes to clear stale input images. Each updateNodeData call was
creating a debounced undo snapshot, resulting in 3-4 undos needed instead
of 1. Now updateNodeData skips undo tracking when nodeRemoveCheckpointActive
is true, since the node removal checkpoint already captured full pre-deletion state.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When deleting a node via the canvas (Delete key), React Flow fires
onNodesChange(remove) then onEdgesChange(remove) synchronously.
Previously this created two undo entries, requiring two undos and
leaving an intermediate state with orphaned edges.
Now onNodesChange sets a microtask-scoped flag that tells onEdgesChange
to skip its checkpoint, collapsing both into a single undo entry.
Added integration test verifying single-undo restores node + edges.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a top-level "Video" quick-access button to the floating action bar
and adds videoInput to the Input category in the All Nodes menu. Fixes
test selectors to disambiguate the new top-level Video button from the
Generate dropdown's Video option.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Captures snapshots before all undoable mutations (add/remove nodes,
connect/remove edges, paste, group operations, edge style, drag start).
Data edits are debounced (500ms). Execution outputs are not undoable.
History depth: 50 entries. Cleared on load/clear workflow.
Keyboard shortcuts added to WorkflowCanvas and shortcuts dialog.
15 integration tests covering all undo/redo scenarios through the store.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Standalone undo manager with push/pop/clear/max-depth (50) logic,
independent of Zustand. Includes 11 unit tests covering all operations.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds a new videoInput node type that allows users to drag-and-drop or
browse for video files (MP4, WebM, QuickTime), preview them inline with
native video controls, and feed them into downstream video processing
nodes. Includes full media externalization/hydration support, Shift+Y
keyboard shortcut, and ConnectionDropMenu integration.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove obsolete scripts/cleanup-workflow.js
- Revert locked guards from group drag/resize/rename (lock is
execution-only, not interaction)
- Add lock icon in group title pill for visual feedback
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- GroupsOverlay: reset color picker when menu closes, guard locked groups
from drag/resize/rename
- AudioInputNode: add keyboard accessibility to dropzone (role, tabIndex,
onKeyDown)
- list-workflows: fix name regex to handle escaped quotes
- GroupsOverlay.test: add missing NBP Input menu item assertion
- WorkflowBrowserView: make relativePath required to match API contract
- WorkflowBrowserModal: add dialog role, aria-modal, aria-labelledby
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- cleanup-workflow.js: Filter only embedded data: images from outputGallery
instead of wiping entire array; preserve imageRef/video on output nodes
- workflow/route.ts: Validate parsed.version is number, nodes/edges are
arrays; sort candidates by mtime desc for deterministic file selection
- mediaStorage.ts: Guard against undefined mediaData in loadMediaById
before caching/returning
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The group controls (lock, delete, color) moved from individual toolbar
buttons into a dropdown menu behind "Group options". Update tests to
open the menu first, then interact with items by text label.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add recursive directory scanning (up to 3 levels deep) to the
list-workflows API so projects organized into subdirectories are
visible in the workflow browser. Skips hidden dirs and node_modules.
Returns relativePath field displayed as the subtitle in the UI.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace the local Set variable in executeWorkflow() with direct
reads/writes to the Zustand store field. The dual tracking was
redundant since get() is synchronous and Set.has() is O(1).
Extract a resetSkippedNodes() helper to DRY up the two cleanup blocks.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The rebase dropped the commit that re-added skippedNodeIds to the store
interface since develop had already extracted it to a local variable.
However, WorkflowCanvas.tsx needs to observe skippedNodeIds from the
store to apply visual dimming during execution.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- WelcomeModal: "Load workflow" now navigates to WorkflowBrowserView
instead of calling browse-directory API directly. Replaced 4 obsolete
fetch-based tests with 3 tests covering view navigation and callback.
- QuickstartInitialView: updated description regex to match new copy
("generative AI pipelines" instead of "AI image generation").
All 1934 tests now pass.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove unused isBase64DataUrl legacy alias from mediaStorage.ts
- Move cleanup-workflow.js to scripts/ directory
- Add parallel batch processing (batch size 3) to hydrateWorkflowMedia,
matching the pattern already used by externalizeWorkflowMedia
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
skippedNodeIds was in the Zustand store interface but never read by any
component — it was only used as a local variable during executeWorkflow.
Removed the store field/initial state/set() calls, keeping only the local
variable.
Also extracted the duplicate 5-line skip-reset block (success + error paths)
into a resetSkippedNodes() helper inside executeWorkflow.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The open button now opens a WorkflowBrowserModal instead of calling
the browse-directory API. Updated the test to assert modal rendering
and removed two obsolete tests that tested the old fetch-based flow.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Back button sits on its own row so the title and directory path are
flush-left instead of indented beside the arrow.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add folder icon with background per row for clear visual anchoring
- Show directory basename as secondary text under workflow name
- Right-align timestamps as de-emphasized metadata
- Add project count in header
- Richer empty state with icon and two-line message
- Border-based hover/active states instead of flat background tint
- Footer actions get small icons for scannability
- Deduplicate browseAndSetDir handler
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Instead of parsing entire workflow JSON files (which can be many MB with
embedded base64 data), read only the first 1KB to extract the name via
regex. Drop nodeCount from the listing. Probe all directories in parallel.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Header's folder icon now opens the workflow browser modal instead of
the OS directory picker directly. The modal wraps WorkflowBrowserView
in the standard overlay pattern.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Load workflow button now navigates to the WorkflowBrowserView instead
of immediately opening the OS directory picker.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Reusable component with two states: empty state prompting user to choose
a workflows folder, and listing state showing all workflows with name,
node count, and relative time. Supports changing folder and one-off browsing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Scans subdirectories of a given parent path for valid workflow JSON files,
returning name, path, node count, and last modified time.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Same approach as Header.tsx — replace browser file picker with native OS
directory picker so loadWorkflow receives the filesystem path for media
hydration from sibling inputs/ and generations/ folders.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace browser file picker with the existing /api/browse-directory native
OS folder picker so loadWorkflow receives the actual filesystem path,
enabling media hydration from sibling inputs/ and generations/ folders.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The directoryPath field was defined in WorkflowFile and checked during
load, but never written during save. Workflows opened from a different
machine or via file picker had no way to find their externalized media
files, so all ref-based nodes appeared empty.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
shrimbly