diff --git a/package-lock.json b/package-lock.json index 1e138bc..356f8a1 100644 --- a/package-lock.json +++ b/package-lock.json @@ -14,7 +14,8 @@ "next": "16.1.4", "react": "19.2.3", "react-dom": "19.2.3", - "reactflow": "^11.11.4" + "reactflow": "^11.11.4", + "yjs": "^13.6.29" }, "devDependencies": { "@tailwindcss/postcss": "^4", @@ -4975,6 +4976,16 @@ "dev": true, "license": "ISC" }, + "node_modules/isomorphic.js": { + "version": "0.2.5", + "resolved": "https://registry.npmjs.org/isomorphic.js/-/isomorphic.js-0.2.5.tgz", + "integrity": "sha512-PIeMbHqMt4DnUP3MA/Flc0HElYjMXArsw1qwJZcm9sqR8mq3l8NYizFMty0pWwE/tzIGH3EKK5+jes5mAr85yw==", + "license": "MIT", + "funding": { + "type": "GitHub Sponsors ❤", + "url": "https://github.com/sponsors/dmonad" + } + }, "node_modules/iterator.prototype": { "version": "1.1.5", "resolved": "https://registry.npmjs.org/iterator.prototype/-/iterator.prototype-1.1.5.tgz", @@ -5130,6 +5141,27 @@ "node": ">= 0.8.0" } }, + "node_modules/lib0": { + "version": "0.2.117", + "resolved": "https://registry.npmjs.org/lib0/-/lib0-0.2.117.tgz", + "integrity": "sha512-DeXj9X5xDCjgKLU/7RR+/HQEVzuuEUiwldwOGsHK/sfAfELGWEyTcf0x+uOvCvK3O2zPmZePXWL85vtia6GyZw==", + "license": "MIT", + "dependencies": { + "isomorphic.js": "^0.2.4" + }, + "bin": { + "0ecdsa-generate-keypair": "bin/0ecdsa-generate-keypair.js", + "0gentesthtml": "bin/gentesthtml.js", + "0serve": "bin/0serve.js" + }, + "engines": { + "node": ">=16" + }, + "funding": { + "type": "GitHub Sponsors ❤", + "url": "https://github.com/sponsors/dmonad" + } + }, "node_modules/lightningcss": { "version": "1.30.2", "resolved": "https://registry.npmjs.org/lightningcss/-/lightningcss-1.30.2.tgz", @@ -7186,6 +7218,23 @@ "dev": true, "license": "ISC" }, + "node_modules/yjs": { + "version": "13.6.29", + "resolved": "https://registry.npmjs.org/yjs/-/yjs-13.6.29.tgz", + "integrity": "sha512-kHqDPdltoXH+X4w1lVmMtddE3Oeqq48nM40FD5ojTd8xYhQpzIDcfE2keMSU5bAgRPJBe225WTUdyUgj1DtbiQ==", + "license": "MIT", + "dependencies": { + "lib0": "^0.2.99" + }, + "engines": { + "node": ">=16.0.0", + "npm": ">=8.0.0" + }, + "funding": { + "type": "GitHub Sponsors ❤", + "url": "https://github.com/sponsors/dmonad" + } + }, "node_modules/yocto-queue": { "version": "0.1.0", "resolved": "https://registry.npmjs.org/yocto-queue/-/yocto-queue-0.1.0.tgz", diff --git a/package.json b/package.json index 839d947..ca26b26 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,8 @@ "next": "16.1.4", "react": "19.2.3", "react-dom": "19.2.3", - "reactflow": "^11.11.4" + "reactflow": "^11.11.4", + "yjs": "^13.6.29" }, "devDependencies": { "@tailwindcss/postcss": "^4", diff --git a/prd.json b/prd.json index 196ae27..a49d175 100644 --- a/prd.json +++ b/prd.json @@ -1,18 +1,19 @@ { "project": "WebVNWrite", - "branchName": "ralph/vn-flowchart-editor", - "description": "Visual Novel Flowchart Editor - A web-based tool for authoring visual novels with drag-and-drop nodes, branching connections, user authentication, and Ren'Py JSON export", + "branchName": "ralph/collaboration-and-character-variables", + "description": "Real-time Collaboration & Character/Variable Management - Enable multi-user editing with CRDT sync, presence indicators, audit trail, plus centralized character/variable definitions with dropdown selectors", "userStories": [ { - "id": "US-001", - "title": "Project scaffolding and configuration", - "description": "As a developer, I need the project set up with Next.js, TailwindCSS, and Supabase so that I can build the application.", + "id": "US-054", + "title": "Character and Variable TypeScript types", + "description": "As a developer, I need TypeScript types for Character and Variable models so that the rest of the feature can be built with type safety.", "acceptanceCriteria": [ - "Initialize Next.js project with TypeScript and App Router", - "Install and configure TailwindCSS", - "Install Supabase client library (@supabase/supabase-js)", - "Create .env.example with NEXT_PUBLIC_SUPABASE_URL and NEXT_PUBLIC_SUPABASE_ANON_KEY", - "Basic folder structure: app/, components/, lib/, types/", + "Add Character type to types/flowchart.ts: id (string), name (string), color (string, hex), description (string, optional)", + "Add Variable type to types/flowchart.ts: id (string), name (string), type ('numeric' | 'string' | 'boolean'), initialValue (number | string | boolean), description (string, optional)", + "Update FlowchartData type to include characters: Character[] and variables: Variable[]", + "Update DialogueNodeData to add optional characterId: string field (alongside existing speaker for migration)", + "Update Condition type to add optional variableId: string field (alongside existing variableName for migration)", + "Update VariableNodeData to add optional variableId: string field (alongside existing variableName for migration)", "Typecheck passes" ], "priority": 1, @@ -20,733 +21,401 @@ "notes": "" }, { - "id": "US-002", - "title": "Define TypeScript types for flowchart data", - "description": "As a developer, I need TypeScript types for nodes, connections, and conditions.", + "id": "US-055", + "title": "Database schema update for characters and variables", + "description": "As a developer, I need the database schema to store characters and variables as part of the project's flowchart data.", "acceptanceCriteria": [ - "Create types/flowchart.ts file", - "DialogueNode type: id, type='dialogue', position: {x,y}, data: { speaker?: string, text: string }", - "ChoiceNode type: id, type='choice', position: {x,y}, data: { prompt: string, options: { id: string, label: string }[] }", - "VariableNode type: id, type='variable', position: {x,y}, data: { variableName: string, operation: 'set'|'add'|'subtract', value: number }", - "Condition type: { variableName: string, operator: '>'|'<'|'=='|'>='|'<='|'!=', value: number }", - "FlowchartEdge type: id, source, sourceHandle?, target, targetHandle?, data?: { condition?: Condition }", - "FlowchartData type: { nodes: (DialogueNode|ChoiceNode|VariableNode)[], edges: FlowchartEdge[] }", - "All types exported from types/flowchart.ts", + "Create migration that documents the new JSONB structure (characters/variables arrays stored within flowchart_data)", + "Update the default value for flowchart_data column to include characters: [] and variables: []", + "Existing projects with no characters/variables arrays continue to load (handled as empty arrays in app code)", "Typecheck passes" ], "priority": 2, "passes": true, - "notes": "" + "notes": "Dependencies: US-054" }, { - "id": "US-003", - "title": "Supabase schema for users and projects", - "description": "As a developer, I need database tables to store users and their projects.", + "id": "US-065", + "title": "Searchable combobox component", + "description": "As a developer, I need a reusable searchable combobox component so that all character/variable dropdowns share consistent behavior and styling.", "acceptanceCriteria": [ - "Create supabase/migrations/ directory", - "Create SQL migration file with profiles table: id (uuid, references auth.users), email (text), display_name (text), is_admin (boolean default false), created_at (timestamptz)", - "Create projects table: id (uuid), user_id (uuid, foreign key to profiles.id), name (text), flowchart_data (jsonb), created_at (timestamptz), updated_at (timestamptz)", - "Add RLS policy: users can SELECT/INSERT/UPDATE/DELETE their own projects (user_id = auth.uid())", - "Add RLS policy: users can SELECT their own profile", - "Add RLS policy: admin users (is_admin=true) can SELECT all profiles", - "Typecheck passes" + "Create components/editor/Combobox.tsx - a reusable searchable dropdown component", + "Props: items (id, label, color?, badge?), value, onChange, placeholder, onAddNew (optional callback)", + "Typing in the input filters the list by name (case-insensitive)", + "Keyboard navigation: arrow keys to move, Enter to select, Escape to close", + "Shows color swatch and/or badge next to item labels when provided", + "'Add new...' option rendered at bottom when onAddNew prop is provided", + "Dropdown positions itself above or below input based on available space", + "Matches existing editor styling (TailwindCSS, dark mode support)", + "Typecheck passes", + "Verify in browser using dev-browser skill" ], "priority": 3, "passes": true, "notes": "" }, { - "id": "US-004", - "title": "Supabase client configuration", - "description": "As a developer, I need Supabase client utilities for auth and database access.", + "id": "US-056", + "title": "Character management UI in project settings", + "description": "As a user, I want a dedicated page to manage my project's characters so that I can define them once and reuse them throughout the flowchart.", "acceptanceCriteria": [ - "Create lib/supabase/client.ts with browser client (createBrowserClient)", - "Create lib/supabase/server.ts with server client (createServerClient for App Router)", - "Create lib/supabase/middleware.ts with middleware client helper", - "Export typed database client using generated types or manual types", - "Typecheck passes" + "Add 'Project Settings' button to editor toolbar", + "Project settings opens as a modal with 'Characters' and 'Variables' tabs", + "Characters tab shows a list of defined characters with name, color swatch, and description", + "'Add Character' button opens inline form with: name (required), color picker (required, defaults to random), description (optional)", + "Each character row has Edit and Delete buttons", + "Deleting a character referenced by nodes shows warning with usage count", + "Character names must be unique within the project (validation error if duplicate)", + "Changes are saved to the flowchart data (same save mechanism as nodes/edges)", + "Typecheck passes", + "Verify in browser using dev-browser skill" ], "priority": 4, "passes": true, - "notes": "" + "notes": "Dependencies: US-054, US-055" }, { - "id": "US-005", - "title": "Protected routes middleware", - "description": "As a developer, I need authentication middleware so that only logged-in users can access the app.", + "id": "US-057", + "title": "Variable management UI in project settings", + "description": "As a user, I want a dedicated page to manage my project's variables so that I can define them with types and initial values for use throughout the flowchart.", "acceptanceCriteria": [ - "Create middleware.ts at project root", - "Middleware checks Supabase session on each request", - "Unauthenticated users accessing /dashboard or /editor/* are redirected to /login", - "Authenticated users accessing /login or /signup are redirected to /dashboard", - "Public routes allowed without auth: /login, /signup, /forgot-password, /reset-password", - "Typecheck passes" + "Variables tab in project settings modal shows a list of defined variables", + "Each variable displays: name, type badge (numeric/string/boolean), initial value, description", + "'Add Variable' button opens inline form with: name (required), type dropdown (required), initial value (required, input adapts to type), description (optional)", + "Each variable row has Edit and Delete buttons", + "Deleting a variable referenced by nodes/edges shows warning with usage count", + "Variable names must be unique within the project", + "Changes are saved to the flowchart data", + "Typecheck passes", + "Verify in browser using dev-browser skill" ], "priority": 5, "passes": true, - "notes": "" + "notes": "Dependencies: US-054, US-055" }, { - "id": "US-006", - "title": "Login page", - "description": "As a user, I want to log in with my email and password so that I can access my projects.", + "id": "US-058", + "title": "Dialogue node speaker dropdown", + "description": "As a user, I want to select a character from a dropdown in the dialogue node instead of typing a name so that I avoid typos and maintain consistency.", "acceptanceCriteria": [ - "Create app/login/page.tsx", - "Form with email and password input fields", - "Submit button calls Supabase signInWithPassword", - "Show error message for invalid credentials", - "On success, redirect to /dashboard", - "Link to /forgot-password page", - "Styled with TailwindCSS", + "Replace the speaker text input in DialogueNode with the Combobox component", + "Dropdown lists all characters defined in the project, showing color swatch + name", + "Selecting a character sets characterId on the node data", + "Dropdown includes 'Add new character...' option at the bottom", + "Clicking 'Add new character...' opens a mini form inline (name + color) that creates the character and selects it", + "If node has a characterId that doesn't match any defined character, show orange warning border on the dropdown", + "Empty/unset speaker shows placeholder 'Select speaker...'", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 6, "passes": true, - "notes": "" + "notes": "Dependencies: US-056, US-065" }, { - "id": "US-007", - "title": "Sign up page (invite-only)", - "description": "As an invited user, I want to complete my account setup so that I can access the tool.", + "id": "US-059", + "title": "Variable node variable dropdown", + "description": "As a user, I want to select a variable from a dropdown in the variable node instead of typing a name so that I avoid typos and maintain consistency.", "acceptanceCriteria": [ - "Create app/signup/page.tsx", - "Form with email (pre-filled if from invite link), password, and confirm password fields", - "Validate passwords match before submission", - "Handle Supabase invite token from URL (type=invite or type=signup)", - "On success, create profile record in profiles table and redirect to /dashboard", - "Show error message if signup fails", - "Styled with TailwindCSS", + "Replace the variableName text input in VariableNode with the Combobox component", + "Dropdown lists all variables defined in the project, showing type badge + name", + "Selecting a variable sets variableId on the node data", + "Dropdown includes 'Add new variable...' option that opens inline creation form", + "If node references a variableId that doesn't match any defined variable, show orange warning border", + "Operation options (set/add/subtract) are filtered based on selected variable's type (add/subtract only for numeric)", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 7, "passes": true, - "notes": "" + "notes": "Dependencies: US-057, US-065" }, { - "id": "US-008", - "title": "Logout functionality", - "description": "As a user, I want to log out so that I can secure my session.", + "id": "US-060", + "title": "Edge condition variable dropdown", + "description": "As a user, I want to select a variable from a dropdown when setting edge conditions so that I reference valid variables consistently.", "acceptanceCriteria": [ - "Create components/LogoutButton.tsx component", - "Button calls Supabase signOut", - "On success, redirect to /login", + "Replace the variableName text input in ConditionEditor with the Combobox component", + "Dropdown lists all variables defined in the project, showing type badge + name", + "Selecting a variable sets variableId on the condition object", + "Dropdown includes 'Add new variable...' option", + "If condition references an undefined variableId, show orange warning indicator", + "Operator options are filtered based on variable type (comparison operators for numeric, == and != for string/boolean)", + "Value input adapts to variable type (number input for numeric, text for string, checkbox for boolean)", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 8, "passes": true, - "notes": "" + "notes": "Dependencies: US-057, US-065" }, { - "id": "US-009", - "title": "Password reset - forgot password page", - "description": "As a user, I want to request a password reset if I forget my password.", + "id": "US-061", + "title": "Choice option condition variable dropdown", + "description": "As a user, I want to select a variable from a dropdown when setting choice option conditions so that I reference valid variables consistently.", "acceptanceCriteria": [ - "Create app/forgot-password/page.tsx", - "Form with email input field", - "Submit button calls Supabase resetPasswordForEmail", - "Show confirmation message after sending (check your email)", - "Link back to /login", - "Styled with TailwindCSS", + "Replace the variableName text input in OptionConditionEditor with the Combobox component", + "Dropdown lists all variables defined in the project, showing type badge + name", + "Selecting a variable sets variableId on the option's condition object", + "Dropdown includes 'Add new variable...' option", + "If condition references an undefined variableId, show orange warning indicator", + "Operator and value inputs adapt to variable type (same behavior as US-060)", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 9, "passes": true, - "notes": "" + "notes": "Dependencies: US-057, US-065" }, { - "id": "US-010", - "title": "Password reset - set new password page", - "description": "As a user, I want to set a new password after clicking the reset link.", + "id": "US-062", + "title": "Auto-migration of existing free-text values", + "description": "As a user, I want my existing projects to automatically create character and variable definitions from free-text values so that I don't have to manually re-enter them.", "acceptanceCriteria": [ - "Create app/reset-password/page.tsx", - "Form with new password and confirm password fields", - "Handle Supabase recovery token from URL", - "Submit calls Supabase updateUser with new password", - "On success, redirect to /login with success message", - "Show error if token invalid or expired", - "Styled with TailwindCSS", - "Typecheck passes", - "Verify in browser using dev-browser skill" + "On project load, if characters array is empty but nodes have speaker values, auto-create Character entries from unique speaker names", + "Auto-created characters get randomly assigned colors and the speaker text as name", + "On project load, if variables array is empty but nodes/edges have variableName values, auto-create Variable entries (default type: numeric, initial value: 0)", + "After auto-creation, update all nodes to set characterId/variableId references pointing to the new entries", + "Show a toast notification: 'Auto-imported N characters and M variables from existing data'", + "Migration only runs once (presence of characters/variables arrays, even if empty, means migration already happened)", + "Typecheck passes" ], "priority": 10, "passes": true, - "notes": "" + "notes": "Dependencies: US-054, US-058, US-059" }, { - "id": "US-011", - "title": "Dashboard layout with navbar", - "description": "As a user, I want a consistent layout with navigation so that I can move around the app.", + "id": "US-063", + "title": "Import characters/variables from another project", + "description": "As a user, I want to import character and variable definitions from another project so that I can reuse them without redefining everything.", "acceptanceCriteria": [ - "Create app/dashboard/layout.tsx", - "Navbar component with app title/logo", - "Navbar shows current user email", - "Navbar includes LogoutButton", - "Main content area below navbar", - "Styled with TailwindCSS", + "Add 'Import from project' button in both Characters and Variables tabs of project settings", + "Button opens a modal listing the user's other projects", + "Selecting a project shows its characters (or variables) with checkboxes for selection", + "User can select which entries to import (select all / none / individual)", + "Imported entries are added to the current project (duplicates by name are skipped with a warning)", + "Imported characters keep their colors; imported variables keep their types and initial values", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 11, "passes": true, - "notes": "" + "notes": "Dependencies: US-056, US-057" }, { - "id": "US-012", - "title": "Dashboard - list projects", - "description": "As a user, I want to see all my projects so that I can choose which one to edit.", + "id": "US-064", + "title": "Export validation for undefined references", + "description": "As a user, I want to be warned before exporting if any nodes reference undefined characters or variables so that I can fix issues before generating output.", "acceptanceCriteria": [ - "Create app/dashboard/page.tsx", - "Fetch projects from Supabase for current user", - "Display projects as cards in a grid", - "Each card shows: project name, last updated date (formatted)", - "Click card navigates to /editor/[projectId]", - "Empty state with message when no projects exist", - "Loading state while fetching", + "Before export, scan all nodes and edges for characterId/variableId references that don't match defined entries", + "If issues found, show a warning modal listing: node type, node content snippet, and the undefined reference", + "Modal offers 'Export anyway' and 'Cancel' options", + "Nodes with undefined references are highlighted on the canvas with orange warning borders when modal is shown", + "If no issues found, export proceeds normally", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 12, "passes": true, - "notes": "" + "notes": "Dependencies: US-058, US-059, US-060, US-061" }, { - "id": "US-013", - "title": "Create new project", - "description": "As a user, I want to create a new project so that I can start a new flowchart.", + "id": "US-043", + "title": "Database schema for collaboration sessions and audit trail", + "description": "As a developer, I need database tables to track active collaboration sessions and store the full change history for projects.", "acceptanceCriteria": [ - "Add 'New Project' button on dashboard", - "Clicking opens modal with project name input", - "Submit creates project in Supabase with empty flowchart_data: { nodes: [], edges: [] }", - "On success, redirect to /editor/[newProjectId]", - "Show error if creation fails", - "Typecheck passes", - "Verify in browser using dev-browser skill" + "Create migration adding project_collaborators table: id (uuid), project_id (references projects), user_id (references profiles), role ('owner' | 'editor' | 'viewer'), invited_at (timestamptz), accepted_at (timestamptz)", + "Create collaboration_sessions table: id (uuid), project_id, user_id, cursor_position (jsonb), selected_node_id (text nullable), connected_at (timestamptz), last_heartbeat (timestamptz)", + "Create audit_trail table: id (uuid), project_id, user_id, action_type (text: 'node_add' | 'node_update' | 'node_delete' | 'edge_add' | 'edge_update' | 'edge_delete'), entity_id (text), previous_state (jsonb), new_state (jsonb), created_at (timestamptz)", + "Add RLS policies: collaborators can access sessions/audit for projects they belong to", + "Add index on audit_trail(project_id, created_at) for efficient history queries", + "Typecheck passes" ], "priority": 13, "passes": true, "notes": "" }, { - "id": "US-014", - "title": "Delete project", - "description": "As a user, I want to delete a project I no longer need.", + "id": "US-045", + "title": "Supabase Realtime channel and connection management", + "description": "As a developer, I need a WebSocket connection layer using Supabase Realtime so that clients can exchange presence and change events in real time.", "acceptanceCriteria": [ - "Add delete icon/button on each project card", - "Clicking shows confirmation dialog (Are you sure?)", - "Confirm deletes project from Supabase", - "Project removed from dashboard list without page reload", - "Show success toast after deletion", + "Create lib/collaboration/realtime.ts module", + "On editor mount, join a Supabase Realtime channel scoped to the project ID", + "Track connection state (connecting, connected, disconnected, reconnecting)", + "Implement heartbeat mechanism (update last_heartbeat every 30 seconds)", + "Auto-reconnect on network interruption with exponential backoff", + "Clean up session record on disconnect/unmount", + "Show connection status indicator in editor toolbar (green=connected, yellow=reconnecting, red=disconnected)", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 14, "passes": true, - "notes": "" + "notes": "Dependencies: US-043" }, { - "id": "US-015", - "title": "Rename project", - "description": "As a user, I want to rename a project to keep my work organized.", + "id": "US-044", + "title": "Project sharing and collaborator management", + "description": "As a project owner, I want to invite other users to collaborate on my project so that we can work together.", "acceptanceCriteria": [ - "Add edit/rename icon on project card", - "Clicking opens modal or enables inline edit for project name", - "Submit updates project name in Supabase", - "UI updates immediately without page reload", - "Show error if rename fails", + "Add 'Share' button in the editor toolbar", + "Share modal displays current collaborators with roles (owner/editor/viewer)", + "Owner can invite users by email with a selected role", + "Owner can change collaborator roles or remove collaborators", + "Invited users see shared projects on their dashboard with a 'Shared with me' indicator", + "RLS policies updated so collaborators can read/write projects based on their role", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 15, "passes": true, - "notes": "" + "notes": "Dependencies: US-043" }, { - "id": "US-016", - "title": "Admin - invite new user", - "description": "As an admin, I want to invite new users so that collaborators can access the tool.", + "id": "US-046", + "title": "Presence indicators for active collaborators", + "description": "As a user, I want to see who else is currently viewing or editing the project so that I am aware of my collaborators.", "acceptanceCriteria": [ - "Create app/admin/invite/page.tsx", - "Only accessible by users with is_admin=true (redirect others to /dashboard)", - "Form with email address input", - "Submit calls Supabase admin inviteUserByEmail (requires service role key in server action)", - "Show success message with invite sent confirmation", - "Show error if invite fails", - "Link to this page visible in navbar only for admins", + "Display a row of avatar circles in the editor toolbar showing connected users", + "Each avatar shows the user's display_name on hover (tooltip)", + "Each user is assigned a consistent color (derived from user ID hash)", + "Avatars appear when users join and disappear when they leave", + "Maximum 5 avatars shown with '+N' overflow indicator", + "Own avatar not shown in the list", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 16, "passes": true, - "notes": "" + "notes": "Dependencies: US-045" }, { - "id": "US-017", - "title": "Editor page with React Flow canvas", - "description": "As a user, I want an editor page with a canvas where I can build my flowchart.", + "id": "US-048", + "title": "Integrate Yjs CRDT for conflict-free node/edge synchronization", + "description": "As a developer, I need to integrate a CRDT library so that concurrent edits from multiple users merge automatically without data loss.", "acceptanceCriteria": [ - "Install reactflow package", - "Create app/editor/[projectId]/page.tsx", - "Fetch project from Supabase by ID", - "Show error if project not found or user unauthorized", - "Show loading state while fetching", - "Render React Flow canvas filling the editor area", - "Canvas has grid background (React Flow Background component)", - "Header shows project name with back link to /dashboard", - "Initialize React Flow with nodes and edges from flowchart_data", - "Typecheck passes", - "Verify in browser using dev-browser skill" + "Install and configure Yjs with a Supabase-compatible provider (or WebSocket provider)", + "Create lib/collaboration/crdt.ts module wrapping Yjs document setup", + "Model flowchart nodes as a Y.Map keyed by node ID", + "Model flowchart edges as a Y.Map keyed by edge ID", + "Local React Flow state changes are synced to the Yjs document", + "Remote Yjs document changes update local React Flow state", + "Initial load populates Yjs document from database state", + "Periodic persistence of Yjs document state to Supabase (debounced 2 seconds)", + "Typecheck passes" ], "priority": 17, "passes": true, - "notes": "" + "notes": "Dependencies: US-045" }, { - "id": "US-018", - "title": "Canvas pan and zoom controls", - "description": "As a user, I want to pan and zoom the canvas to navigate large flowcharts.", + "id": "US-047", + "title": "Live cursor positions on canvas", + "description": "As a user, I want to see other collaborators' cursor positions on the canvas so that I can understand where they are working.", "acceptanceCriteria": [ - "Canvas supports click-and-drag panning (React Flow default)", - "Mouse wheel zooms in/out (React Flow default)", - "Add React Flow Controls component with zoom +/- buttons", - "Add fitView button to show all nodes", - "Controls positioned in bottom-right corner", + "Broadcast local cursor position to the Realtime channel (throttled to 50ms)", + "Render remote cursors as colored arrows/pointers on the canvas with user name labels", + "Cursor color matches the user's assigned presence color", + "Remote cursors smoothly interpolate between position updates (no jumping)", + "Remote cursors fade out after 5 seconds of inactivity", + "Cursors are rendered in screen coordinates and properly transform with canvas zoom/pan", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 18, "passes": true, - "notes": "" + "notes": "Dependencies: US-045, US-046" }, { - "id": "US-019", - "title": "Editor toolbar", - "description": "As a user, I want a toolbar with actions for adding nodes and saving/exporting.", + "id": "US-050", + "title": "Join/leave notifications", + "description": "As a user, I want to be notified when collaborators join or leave the editing session so that I stay aware of the team's activity.", "acceptanceCriteria": [ - "Create components/editor/Toolbar.tsx", - "Toolbar positioned at top of editor below header", - "Buttons: Add Dialogue, Add Choice, Add Variable (no functionality yet)", - "Buttons: Save, Export, Import (no functionality yet)", - "Buttons styled with TailwindCSS, icons optional", + "Show a toast notification when a collaborator joins: '[Name] joined'", + "Show a toast notification when a collaborator leaves: '[Name] left'", + "Notifications use the collaborator's assigned color as an accent", + "Notifications auto-dismiss after 3 seconds (matches existing Toast behavior)", + "No notification shown for own join/leave events", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 19, "passes": true, - "notes": "" + "notes": "Dependencies: US-045, US-046" }, { - "id": "US-020", - "title": "Create custom dialogue node component", - "description": "As a user, I want dialogue nodes to display and edit character speech.", + "id": "US-049", + "title": "Node editing lock indicators", + "description": "As a user, I want to see when another collaborator is actively editing a specific node so that I can avoid conflicts and wait for them to finish.", "acceptanceCriteria": [ - "Create components/editor/nodes/DialogueNode.tsx", - "Node styled with blue background/border", - "Displays editable input for speaker name (placeholder: 'Speaker')", - "Displays editable textarea for dialogue text (placeholder: 'Dialogue text...')", - "Has one Handle at top (type='target', id='input')", - "Has one Handle at bottom (type='source', id='output')", - "Register as custom node type in React Flow", + "When a user focuses/opens a node for editing, broadcast the node ID to the channel", + "Nodes being edited by others show a colored border matching the editor's presence color", + "A small label with the editor's name appears on the locked node", + "Other users can still view but see a 'Being edited by [name]' indicator if they try to edit", + "Lock is released when the user clicks away, closes the node, or disconnects", + "Lock auto-expires after 60 seconds of inactivity as a safety measure", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 20, "passes": true, - "notes": "" + "notes": "Dependencies: US-045, US-048" }, { - "id": "US-021", - "title": "Add dialogue node from toolbar", - "description": "As a user, I want to add dialogue nodes by clicking the toolbar button.", + "id": "US-051", + "title": "Audit trail recording", + "description": "As a developer, I need all node and edge changes to be recorded in the audit trail so that users can review history and revert changes.", "acceptanceCriteria": [ - "Clicking 'Add Dialogue' in toolbar creates new DialogueNode", - "Node appears at center of current viewport", - "Node has unique ID (use nanoid or uuid)", - "Node added to React Flow nodes state", - "Node can be dragged to reposition", - "Typecheck passes", - "Verify in browser using dev-browser skill" + "Every node add/update/delete operation writes a record to audit_trail table", + "Every edge add/update/delete operation writes a record to audit_trail table", + "Records include previous_state (null for additions) and new_state (null for deletions)", + "Records include the acting user's ID and timestamp", + "Writes are batched/debounced to avoid excessive DB calls (max 1 write per second per entity)", + "Audit writes do not block the user's editing flow (fire-and-forget with error logging)", + "Typecheck passes" ], "priority": 21, "passes": true, - "notes": "" + "notes": "Dependencies: US-043, US-048" }, { - "id": "US-022", - "title": "Create custom choice node component", - "description": "As a user, I want choice nodes to display branching decisions.", + "id": "US-052", + "title": "Activity history sidebar", + "description": "As a user, I want to view a history of all changes made to the project so that I can see what collaborators have done and when.", "acceptanceCriteria": [ - "Create components/editor/nodes/ChoiceNode.tsx", - "Node styled with green background/border", - "Displays editable input for prompt text (placeholder: 'What do you choose?')", - "Displays 2 default options, each with editable label input", - "Has one Handle at top (type='target', id='input')", - "Each option has its own Handle at bottom (type='source', id='option-0', 'option-1', etc.)", - "Register as custom node type in React Flow", + "Add 'History' button to editor toolbar that opens a right sidebar panel", + "Sidebar displays a chronological list of changes with: user name, action type, entity description, timestamp", + "Entries are grouped by time period (Today, Yesterday, Earlier)", + "Each entry shows the user's presence color as an accent", + "Clicking an entry highlights/selects the affected node or edge on the canvas", + "Paginated loading (20 entries per page) with 'Load more' button", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 22, "passes": true, - "notes": "" + "notes": "Dependencies: US-051" }, { - "id": "US-023", - "title": "Add choice node from toolbar", - "description": "As a user, I want to add choice nodes by clicking the toolbar button.", + "id": "US-053", + "title": "Revert changes from audit trail", + "description": "As a user, I want to revert a specific change from the history so that I can undo mistakes made by myself or collaborators.", "acceptanceCriteria": [ - "Clicking 'Add Choice' in toolbar creates new ChoiceNode", - "Node appears at center of current viewport", - "Node has unique ID", - "Node initialized with 2 options (each with unique id and empty label)", - "Node added to React Flow nodes state", - "Node can be dragged to reposition", + "Each entry in the activity history sidebar has a 'Revert' button", + "Clicking 'Revert' shows a confirmation dialog with before/after preview", + "Reverting a node addition deletes the node", + "Reverting a node update restores the previous state", + "Reverting a node deletion re-creates the node with its previous state", + "Reverting an edge change follows the same add/update/delete logic", + "The revert itself is recorded as a new audit trail entry", + "Reverted state is synced to all connected clients via CRDT", "Typecheck passes", "Verify in browser using dev-browser skill" ], "priority": 23, "passes": true, - "notes": "" - }, - { - "id": "US-024", - "title": "Add/remove choice options", - "description": "As a user, I want to add or remove choice options (2-6 options supported).", - "acceptanceCriteria": [ - "ChoiceNode has '+' button to add new option", - "Maximum 6 options (button disabled or hidden at max)", - "Each option has 'x' button to remove it", - "Minimum 2 options (remove button disabled or hidden at min)", - "Adding option creates new output Handle dynamically", - "Removing option removes its Handle", - "Node data updates in React Flow state", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 24, - "passes": true, - "notes": "" - }, - { - "id": "US-025", - "title": "Create custom variable node component", - "description": "As a user, I want variable nodes to set or modify story variables.", - "acceptanceCriteria": [ - "Create components/editor/nodes/VariableNode.tsx", - "Node styled with orange background/border", - "Displays editable input for variable name (placeholder: 'variableName')", - "Displays dropdown/select for operation: set, add, subtract", - "Displays editable number input for value (default: 0)", - "Has one Handle at top (type='target', id='input')", - "Has one Handle at bottom (type='source', id='output')", - "Register as custom node type in React Flow", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 25, - "passes": true, - "notes": "" - }, - { - "id": "US-026", - "title": "Add variable node from toolbar", - "description": "As a user, I want to add variable nodes by clicking the toolbar button.", - "acceptanceCriteria": [ - "Clicking 'Add Variable' in toolbar creates new VariableNode", - "Node appears at center of current viewport", - "Node has unique ID", - "Node initialized with empty variableName, operation='set', value=0", - "Node added to React Flow nodes state", - "Node can be dragged to reposition", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 26, - "passes": true, - "notes": "" - }, - { - "id": "US-027", - "title": "Connect nodes with edges", - "description": "As a user, I want to connect nodes with arrows to define story flow.", - "acceptanceCriteria": [ - "Dragging from source Handle to target Handle creates edge (React Flow default)", - "Edges render as smooth bezier curves (default edge type or smoothstep)", - "Edges show arrow marker indicating direction (markerEnd)", - "Edges update position when nodes are moved", - "Cannot connect source-to-source or target-to-target (React Flow handles this)", - "New edges added to React Flow edges state", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 27, - "passes": true, - "notes": "" - }, - { - "id": "US-028", - "title": "Select and delete nodes", - "description": "As a user, I want to delete nodes to revise my flowchart.", - "acceptanceCriteria": [ - "Clicking a node selects it (visual highlight via React Flow)", - "Pressing Delete or Backspace key removes selected node(s)", - "Deleting node also removes all connected edges", - "Use onNodesDelete callback to handle deletion", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 28, - "passes": true, - "notes": "" - }, - { - "id": "US-029", - "title": "Select and delete edges", - "description": "As a user, I want to delete connections between nodes.", - "acceptanceCriteria": [ - "Clicking an edge selects it (visual highlight via React Flow)", - "Pressing Delete or Backspace key removes selected edge(s)", - "Use onEdgesDelete callback to handle deletion", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 29, - "passes": true, - "notes": "" - }, - { - "id": "US-030", - "title": "Right-click context menu", - "description": "As a user, I want a context menu for quick actions.", - "acceptanceCriteria": [ - "Create components/editor/ContextMenu.tsx", - "Right-click on canvas shows menu: Add Dialogue, Add Choice, Add Variable", - "New node created at click position", - "Right-click on node shows menu: Delete", - "Right-click on edge shows menu: Delete, Add Condition", - "Clicking elsewhere or pressing Escape closes menu", - "Menu styled with TailwindCSS", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 30, - "passes": true, - "notes": "" - }, - { - "id": "US-031", - "title": "Condition editor modal", - "description": "As a user, I want to add conditions to edges so branches depend on variables.", - "acceptanceCriteria": [ - "Create components/editor/ConditionEditor.tsx modal/popover", - "Opens on double-click edge or via context menu 'Add Condition'", - "Form fields: variable name input, operator dropdown (>, <, ==, >=, <=, !=), value number input", - "Pre-fill fields if edge already has condition", - "Save button applies condition to edge data", - "Clear/Remove button removes condition from edge", - "Cancel button closes without saving", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 31, - "passes": true, - "notes": "" - }, - { - "id": "US-032", - "title": "Display conditions on edges", - "description": "As a user, I want to see conditions displayed on edges.", - "acceptanceCriteria": [ - "Create custom edge component or use edge labels", - "Edges with conditions render as dashed lines (strokeDasharray)", - "Condition label displayed on edge (e.g., 'score > 5')", - "Unconditional edges remain solid lines", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 32, - "passes": true, - "notes": "" - }, - { - "id": "US-033", - "title": "Auto-save to LocalStorage", - "description": "As a user, I want my work auto-saved locally so I don't lose progress if the browser crashes.", - "acceptanceCriteria": [ - "Save flowchart state (nodes + edges) to LocalStorage on every change", - "Debounce saves (e.g., 1 second delay after last change)", - "LocalStorage key format: 'vnwrite-draft-{projectId}'", - "On editor load, check LocalStorage for saved draft", - "If local draft exists and differs from database, show prompt to restore or discard", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 33, - "passes": true, - "notes": "" - }, - { - "id": "US-034", - "title": "Save project to database", - "description": "As a user, I want to save my project to the database manually.", - "acceptanceCriteria": [ - "Clicking 'Save' in toolbar saves current nodes/edges to Supabase", - "Update project's flowchart_data and updated_at fields", - "Show saving indicator/spinner while in progress", - "Show success toast on completion", - "Clear LocalStorage draft after successful save", - "Show error toast if save fails", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 34, - "passes": true, - "notes": "" - }, - { - "id": "US-035", - "title": "Export project as .vnflow file", - "description": "As a user, I want to export my project as a JSON file for backup or sharing.", - "acceptanceCriteria": [ - "Clicking 'Export' in toolbar triggers file download", - "File named '[project-name].vnflow'", - "File contains JSON with nodes and edges arrays", - "JSON is pretty-printed (2-space indent) for readability", - "Uses browser download API (create blob, trigger download)", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 35, - "passes": true, - "notes": "" - }, - { - "id": "US-036", - "title": "Import project from .vnflow file", - "description": "As a user, I want to import a .vnflow file to restore or share projects.", - "acceptanceCriteria": [ - "Clicking 'Import' in toolbar opens file picker", - "Accept .vnflow and .json file extensions", - "If current project has unsaved changes, show confirmation dialog", - "Validate imported file has nodes and edges arrays", - "Show error toast if file is invalid", - "Load valid data into React Flow state (replaces current flowchart)", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 36, - "passes": true, - "notes": "" - }, - { - "id": "US-037", - "title": "Export to Ren'Py JSON format", - "description": "As a user, I want to export my flowchart to Ren'Py-compatible JSON for use in my game.", - "acceptanceCriteria": [ - "Add 'Export to Ren'Py' option (button or dropdown item)", - "File named '[project-name]-renpy.json'", - "Dialogue nodes export as: { type: 'dialogue', speaker: '...', text: '...' }", - "Choice nodes export as: { type: 'menu', prompt: '...', choices: [{ label: '...', next: '...' }] }", - "Variable nodes export as: { type: 'variable', name: '...', operation: '...', value: ... }", - "Edges with conditions include condition object on the choice/jump", - "Organize nodes into labeled sections based on flow (traverse from first node)", - "Include metadata: projectName, exportedAt timestamp", - "Output JSON is valid (test with JSON.parse)", - "Typecheck passes" - ], - "priority": 37, - "passes": true, - "notes": "" - }, - { - "id": "US-038", - "title": "Unsaved changes warning", - "description": "As a user, I want a warning before losing unsaved work.", - "acceptanceCriteria": [ - "Track dirty state: true when flowchart modified after last save", - "Set dirty=true on node/edge add, delete, or modify", - "Set dirty=false after successful save", - "Browser beforeunload event shows warning if dirty", - "Navigating to dashboard shows confirmation modal if dirty", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 38, - "passes": true, - "notes": "" - }, - { - "id": "US-039", - "title": "Loading and error states", - "description": "As a user, I want clear feedback when things are loading or when errors occur.", - "acceptanceCriteria": [ - "Loading spinner component for async operations", - "Editor shows loading spinner while fetching project", - "Error message displayed if project fails to load (with back to dashboard link)", - "Toast notification system for success/error messages", - "Save error shows toast with retry option", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 39, - "passes": true, - "notes": "" - }, - { - "id": "US-040", - "title": "Conditionals on choice options", - "description": "As a user, I want individual choice options to have variable conditions so that options are only visible when certain conditions are met (e.g., affection > 10).", - "acceptanceCriteria": [ - "Each ChoiceOption can have optional condition (variableName, operator, value)", - "Update ChoiceNode UI to show 'Add condition' button per option", - "Condition editor modal for each option", - "Visual indicator (icon/badge) on options with conditions", - "Update TypeScript types: ChoiceOption gets optional condition field", - "Export includes per-option conditions in Ren'Py JSON", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 40, - "passes": true, - "notes": "Dependencies: US-018, US-019, US-025. Complexity: M" - }, - { - "id": "US-041", - "title": "Change password for logged-in user", - "description": "As a user, I want to change my own password from a settings/profile page so that I can keep my account secure.", - "acceptanceCriteria": [ - "Settings/profile page accessible from dashboard header", - "Form with: current password, new password, confirm new password fields", - "Calls Supabase updateUser with new password", - "Requires current password verification (re-authenticate)", - "Shows success/error messages", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 41, - "passes": true, - "notes": "Dependencies: US-004. Complexity: S" - }, - { - "id": "US-042", - "title": "Password reset modal on token arrival", - "description": "As a user, I want a modal to automatically appear when a password reset token is detected so that I can set my new password seamlessly.", - "acceptanceCriteria": [ - "Detect password reset token in URL (from Supabase email link)", - "Show modal/dialog automatically when token present", - "Modal has: new password, confirm password fields", - "Calls Supabase updateUser with token to complete reset", - "On success, close modal and redirect to login", - "On error, show error message", - "Typecheck passes", - "Verify in browser using dev-browser skill" - ], - "priority": 42, - "passes": true, - "notes": "Dependencies: US-006. Complexity: S" + "notes": "Dependencies: US-052, US-048" } ] } diff --git a/progress.txt b/progress.txt index c5d0f32..2b4b295 100644 --- a/progress.txt +++ b/progress.txt @@ -27,414 +27,188 @@ - Reusable LoadingSpinner component in `src/components/LoadingSpinner.tsx` with size ('sm'|'md'|'lg') and optional message - Toast component supports an optional `action` prop: `{ label: string; onClick: () => void }` for retry/undo buttons - Settings page at `/dashboard/settings` reuses dashboard layout; re-auth via signInWithPassword before updateUser +- Character/Variable types (`Character`, `Variable`) and extracted node data types (`DialogueNodeData`, `VariableNodeData`) are in `src/types/flowchart.ts` +- `EditorContext` at `src/components/editor/EditorContext.tsx` provides shared state (characters, onAddCharacter) to all custom node components via React context +- Use `useEditorContext()` in node components to access project-level characters and variables without prop drilling through React Flow node data +- New JSONB fields (characters, variables) must be defaulted to `[]` when reading from DB in page.tsx to handle pre-existing data +- Reusable `Combobox` component at `src/components/editor/Combobox.tsx` - use for all character/variable dropdowns. Props: items (ComboboxItem[]), value, onChange, placeholder, onAddNew +- `ProjectSettingsModal` at `src/components/editor/ProjectSettingsModal.tsx` manages characters/variables. Receives state + callbacks from FlowchartEditor +- Characters and variables state is managed in `FlowchartEditorInner` with `useState` hooks, passed down to the modal +- For settings-style modals, use `max-w-2xl h-[80vh]` with overflow-y-auto content area and fixed header/tabs +- `EditorContext` provides both characters (onAddCharacter) and variables (onAddVariable) to node components. Use `useEditorContext()` to access them. +- In FlowchartEditor, `handleAddVariable` adds a variable *node* to the canvas; `handleAddVariableDefinition` creates a variable *definition* in project data. Avoid naming collisions between "add node" and "add definition" callbacks. +- Edge interactions use `onEdgeClick` on ReactFlow component. ConditionEditor opens as a modal overlay since React Flow edges don't support inline panels. +- `Condition.value` supports `number | string | boolean` — always check variable type before rendering value inputs for edge conditions. +- `OptionConditionEditor` at `src/components/editor/OptionConditionEditor.tsx` handles choice option conditions. Same pattern as `ConditionEditor` but with simpler props (no edgeId). +- `ChoiceOption` type includes optional `condition?: Condition`. When counting variable usage, check variable nodes + edge conditions + choice option conditions. +- React Compiler lint forbids `setState` in effects and reading `useRef().current` during render. Use `useState(() => computeValue())` lazy initializer pattern for one-time initialization logic. +- For detecting legacy data shape (pre-migration), pass a flag from the server component (page.tsx) to the client component, since only the server reads raw DB data. +- Collaboration tables: `project_collaborators` (roles), `collaboration_sessions` (presence), `audit_trail` (history) — all with RLS scoped by project ownership or collaborator membership +- RLS pattern for shared resources: check `projects.user_id = auth.uid()` OR `project_collaborators.user_id = auth.uid()` to cover both owners and collaborators +- `RealtimeConnection` class at `src/lib/collaboration/realtime.ts` manages Supabase Realtime channel lifecycle (connect, heartbeat, reconnect, disconnect). Instantiate with (projectId, userId, callbacks). +- FlowchartEditor receives `userId` prop from page.tsx server component for collaboration features +- Toolbar accepts optional `connectionState` prop to show green/yellow/red connection indicator +- `collaboration_sessions` table has UNIQUE(project_id, user_id) constraint to support upsert-based session management +- Server actions for project-specific operations go in `src/app/editor/[projectId]/actions.ts` — use `'use server'` directive and return `{ success: boolean; error?: string }` pattern +- Editor page.tsx supports both owner and collaborator access: first checks ownership, then falls back to `project_collaborators` lookup. Pass `isOwner` prop to client component. +- `ShareModal` at `src/components/editor/ShareModal.tsx` manages collaborator invites/roles/removal via server actions. Only owners see invite form. +- Dashboard shared projects use Supabase join query: `project_collaborators.select('role, projects(id, name, updated_at)')` to fetch projects shared with the user +- `ProjectCard` supports optional `shared` and `sharedRole` props — when `shared=true`, hide edit/delete buttons and show role badge instead +- `PresenceAvatars` at `src/components/editor/PresenceAvatars.tsx` renders connected collaborator avatars. Receives `PresenceUser[]` from `RealtimeConnection.onPresenceSync`. +- `RealtimeConnection` constructor takes `(projectId, userId, displayName, callbacks)` — `displayName` is broadcast via Supabase Realtime presence tracking +- User color for presence is derived from a hash of their userId, ensuring consistency across sessions. Use `getUserColor(userId)` pattern from PresenceAvatars. +- `CRDTManager` at `src/lib/collaboration/crdt.ts` wraps a Yjs Y.Doc with Y.Map for nodes and edges. Connects to Supabase Realtime channel for broadcasting updates. +- CRDT sync pattern: local React Flow changes → `updateNodes`/`updateEdges` on CRDTManager → Yjs broadcasts to channel; remote broadcasts → Yjs applies update → callbacks set React Flow state. Use `isRemoteUpdateRef` to prevent echo loops. +- For Supabase Realtime broadcast of binary data (Yjs updates), convert `Uint8Array` → `Array.from()` for JSON payload, and `new Uint8Array()` on receive. +- For ephemeral real-time data (cursors, typing indicators), use Supabase Realtime broadcast (`channel.send({ type: 'broadcast', event, payload })`) + `.on('broadcast', { event }, callback)` — not persistence-backed +- `RemoteCursors` at `src/components/editor/RemoteCursors.tsx` renders collaborator cursors on canvas. Uses `useViewport()` to transform flow→screen coordinates. Throttle broadcasts to 50ms via timestamp ref. +- Supabase Realtime presence events: `sync` (full state), `join` (arrivals with `newPresences` array), `leave` (departures with `leftPresences` array). Filter `this.userId` to skip own events. +- `CollaborationToast` at `src/components/editor/CollaborationToast.tsx` shows join/leave notifications (bottom-left, auto-dismiss 3s). Uses `getUserColor(userId)` for accent color dot. +- Node lock indicators use `EditorContext` (`nodeLocks` Map, `onNodeFocus`, `onNodeBlur`). Each node component checks `nodeLocks.get(id)` for lock state and renders `NodeLockIndicator` + overlay if locked by another user. +- For ephemeral lock state (node editing locks), broadcast via `node-lock` event with `{ nodeId, userId, displayName, lockedAt }`. Send `nodeId: null` to release. +- `AuditTrailRecorder` at `src/lib/collaboration/auditTrail.ts` records node/edge changes to `audit_trail` table. Uses state diffing (previous vs current Maps), 1-second per-entity debounce, and fire-and-forget Supabase inserts. Only records local changes (guarded by `isRemoteUpdateRef` in FlowchartEditor). +- `ActivityHistorySidebar` at `src/components/editor/ActivityHistorySidebar.tsx` displays audit trail entries in a right sidebar. Rendered inside the canvas `relative flex-1` container. Toggle via `showHistory` state in FlowchartEditor. Exports `AuditEntry` type for consumers. +- To prevent double audit recording when programmatically changing nodes/edges (e.g., revert), set a ref guard (`isRevertingRef`) before `setNodes`/`setEdges` and clear it with `setTimeout(() => ..., 0)`. Check the guard in the CRDT sync effects before calling `auditRef.current.recordNodeChanges()`. +- For async data fetching in components with React Compiler, use a pure fetch function returning `{ data, error, hasMore }` result object, then handle setState in the `.then()` callback with an abort/mount guard — never call setState-containing functions directly inside useEffect. --- -## 2026-01-21 - US-001 -- What was implemented: Project scaffolding and configuration +## 2026-01-23 - US-054 +- What was implemented: Character and Variable TypeScript types added to `src/types/flowchart.ts` - Files changed: - - package.json - project dependencies and scripts - - tsconfig.json - TypeScript configuration - - next.config.ts - Next.js configuration - - postcss.config.mjs - PostCSS with TailwindCSS - - eslint.config.mjs - ESLint configuration - - .env.example - environment variables template - - .gitignore - git ignore rules - - src/app/ - Next.js App Router pages - - src/components/.gitkeep - components directory placeholder - - src/lib/.gitkeep - lib directory placeholder - - src/types/.gitkeep - types directory placeholder + - `src/types/flowchart.ts` - Added `Character`, `Variable`, `DialogueNodeData`, `VariableNodeData` types; updated `FlowchartData`, `DialogueNode`, `VariableNode`, `Condition` types + - `src/app/editor/[projectId]/page.tsx` - Updated FlowchartData initialization to include `characters: []` and `variables: []` defaults - **Learnings for future iterations:** - - Next.js 16 uses `@tailwindcss/postcss` for TailwindCSS 4 integration - - Use --src-dir flag for create-next-app to put source in src/ folder - - npm package names can't have capital letters (use lowercase) - - .gitignore needs explicit exclusion for .env files, but include .env.example + - The node components (`DialogueNode.tsx`, `VariableNode.tsx`, `ChoiceNode.tsx`) define their own local data types that mirror the global types. When adding fields, both the global type and local component type may need updating in later stories. + - `flowchart_data` is a JSONB column in Supabase, so it comes as `any` type. Always provide defaults for new fields when reading from DB to handle existing data without those fields. + - The new `characterId` and `variableId` fields are optional alongside existing `speaker`/`variableName` fields to support migration from free-text to referenced-entity pattern. --- -## 2026-01-21 - US-002 -- What was implemented: TypeScript types for flowchart data structures +## 2026-01-23 - US-055 +- What was implemented: Database migration to update flowchart_data JSONB default to include `characters: []` and `variables: []` - Files changed: - - src/types/flowchart.ts - new file with all flowchart type definitions - - package.json - added typecheck script (tsc --noEmit) + - `supabase/migrations/20260123000000_add_characters_variables_to_flowchart_data.sql` - New migration that alters the default value for the flowchart_data column and documents the expected JSONB structure - **Learnings for future iterations:** - - Position is a helper type for {x, y} coordinates used by nodes - - FlowchartNode is a union type of DialogueNode | ChoiceNode | VariableNode - - ChoiceOption is a separate type to make options array cleaner - - All types use `export type` for TypeScript isolatedModules compatibility + - Since characters and variables are stored within the existing flowchart_data JSONB column (not as separate tables), schema changes are minimal - just updating the column default. The real data integrity is handled at the application layer. + - The app-side defaults in page.tsx (from US-054) already handle existing projects gracefully, so no data migration of existing rows is needed. + - For JSONB-embedded arrays, the pattern is: update the DB default for new rows + handle missing fields in app code for old rows. --- -## 2026-01-21 - US-003 -- What was implemented: Supabase schema for users and projects +## 2026-01-23 - US-065 +- What was implemented: Reusable searchable combobox component at `src/components/editor/Combobox.tsx` - Files changed: - - supabase/migrations/20260121000000_create_profiles_and_projects.sql - new file with all database schema + - `src/components/editor/Combobox.tsx` - New component with searchable dropdown, keyboard navigation, color swatches, badges, "Add new..." option, and auto-positioning - **Learnings for future iterations:** - - Supabase migrations are plain SQL files in supabase/migrations/ directory - - Migration filenames use timestamp prefix (YYYYMMDDHHMMSS_description.sql) - - RLS policies need separate policies for SELECT, INSERT, UPDATE, DELETE operations - - Admin check policy uses EXISTS subquery to check is_admin flag on profiles table - - projects table references profiles.id (not auth.users.id directly) for proper FK relationships - - flowchart_data column uses JSONB type with default empty structure - - Added auto-update trigger for updated_at timestamp on projects table + - The Combobox exports both the default component and the `ComboboxItem` type for consumers to use + - Props: `items` (ComboboxItem[]), `value` (string | undefined), `onChange` (id: string) => void, `placeholder` (string), `onAddNew` (() => void, optional) + - ComboboxItem shape: `{ id: string, label: string, color?: string, badge?: string }` + - The component uses neutral zinc colors for borders/backgrounds (not blue/green/orange) so it can be reused across different node types + - Dropdown auto-positions above or below based on available viewport space (200px threshold) + - Keyboard: ArrowDown/Up navigate, Enter selects, Escape closes + - The component is designed to be a drop-in replacement for text inputs in node components (same `w-full` and `text-sm` sizing) --- -## 2026-01-21 - US-004 -- What was implemented: Supabase client configuration utilities +## 2026-01-23 - US-056 +- What was implemented: Character management UI in the project settings modal - Files changed: - - src/lib/supabase/client.ts - browser client using createBrowserClient from @supabase/ssr - - src/lib/supabase/server.ts - server client for App Router with async cookies() API - - src/lib/supabase/middleware.ts - middleware helper with updateSession function - - src/lib/.gitkeep - removed (no longer needed) + - `src/components/editor/ProjectSettingsModal.tsx` - New modal component with Characters and Variables tabs; Characters tab has full CRUD (add, edit, delete with usage warnings), name uniqueness validation, color picker, inline forms + - `src/components/editor/Toolbar.tsx` - Added `onProjectSettings` prop and "Project Settings" button to the right side of the toolbar + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `characters` and `variables` state management, `showSettings` modal state, usage count helpers (`getCharacterUsageCount`, `getVariableUsageCount`), and ProjectSettingsModal rendering - **Learnings for future iterations:** - - @supabase/ssr package provides createBrowserClient and createServerClient functions - - Server client requires async cookies() from next/headers in Next.js 16 - - Middleware client returns both user object and supabaseResponse for route protection - - Cookie handling uses getAll/setAll pattern for proper session management - - setAll in server.ts wrapped in try/catch to handle Server Component limitations + - The ProjectSettingsModal receives `onCharactersChange` and `onVariablesChange` callbacks that directly set state in FlowchartEditor. When save is implemented, it should read from this state. + - The Variables tab is a read-only placeholder in US-056; US-057 will implement the full CRUD for variables using the same patterns (inline forms, validation, delete warnings). + - Modal pattern: fixed inset-0 z-50 with backdrop click to close, max-w-2xl for settings modals (larger than max-w-md used for simple dialogs). + - Character usage count checks dialogue nodes for `data.characterId`; variable usage count checks both variable nodes and edge conditions. + - The `randomHexColor()` utility picks from a curated list of 12 vibrant colors for character defaults. + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-005 -- What was implemented: Protected routes middleware for authentication +## 2026-01-23 - US-057 +- What was implemented: Variable management UI with full CRUD in the project settings modal Variables tab - Files changed: - - middleware.ts - new file at project root for route protection + - `src/components/editor/ProjectSettingsModal.tsx` - Replaced placeholder VariablesTab with full implementation: add/edit/delete with inline forms, type dropdown (numeric/string/boolean), type-adaptive initial value input (number input for numeric, text for string, select for boolean), name uniqueness validation, delete warnings with usage count, colored type badges - **Learnings for future iterations:** - - Next.js middleware.ts must be at project root (not in src/) - - updateSession helper from lib/supabase/middleware.ts returns { user, supabaseResponse } - - Use startsWith() for route matching to handle nested routes (e.g., /editor/*) - - Matcher config excludes static files and images to avoid unnecessary middleware calls - - Clone nextUrl before modifying pathname for redirects + - The VariableForm uses a `handleTypeChange` helper that resets the initial value to the type's default when the type changes, preventing invalid state (e.g., "hello" as a numeric value) + - Initial values are stored as strings in form state and parsed to the correct type (number/string/boolean) on save via `parseInitialValue()` + - Type badges use distinct colors: blue for numeric, green for string, purple for boolean - making variable types instantly recognizable in the list + - The same form patterns from CharactersTab apply: inline form within the list for editing, appended form below the list for adding + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-006 -- What was implemented: Login page with email/password authentication +## 2026-01-23 - US-059 +- What was implemented: Variable node variable dropdown using Combobox component, replacing the free-text input - Files changed: - - src/app/login/page.tsx - new file with login form and Supabase auth + - `src/components/editor/nodes/VariableNode.tsx` - Replaced text input with Combobox for variable selection, added inline "Add new variable" form with name + type, added orange warning border for invalid references, filtered operation options (add/subtract only for numeric type) + - `src/components/editor/EditorContext.tsx` - Extended context to include `variables: Variable[]` and `onAddVariable` callback + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `handleAddVariableDefinition` callback and passed variables + onAddVariable through EditorContext - **Learnings for future iterations:** - - Auth pages use 'use client' directive since they need useState and form handling - - Use createClient() from lib/supabase/client.ts for browser-side auth operations - - supabase.auth.signInWithPassword returns { error } object for handling failures - - useRouter from next/navigation for programmatic redirects after auth - - Error state displayed in red alert box with dark mode support - - Loading state disables submit button and shows "Signing in..." text - - TailwindCSS dark mode uses dark: prefix (e.g., dark:bg-zinc-950) + - The existing `handleAddVariable` in FlowchartEditor adds a variable *node* to the canvas (toolbar action). The new `handleAddVariableDefinition` creates a variable *definition* in the project's data. Name carefully to avoid collisions. + - EditorContext is the shared context for node components to access project-level characters and variables. Extend it when new entity types need to be accessible from custom node components. + - The VariableNode follows the same pattern as DialogueNode for Combobox integration: items derived via useMemo, handleSelect sets both variableId and variableName, inline add form for quick creation, hasInvalidReference for warning state. + - Operations filtering uses `isNumeric` flag: if no variable is selected (undefined) or type is 'numeric', all operations are shown; otherwise only 'set' is available. When selecting a non-numeric variable, operation is auto-reset to 'set' if it was 'add' or 'subtract'. + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-007 -- What was implemented: Sign up page for invite-only account setup +## 2026-01-23 - US-060 +- What was implemented: Edge condition variable dropdown using Combobox component, replacing free-text input with a type-aware condition editor modal - Files changed: - - src/app/signup/page.tsx - new file with signup form and Supabase auth + - `src/types/flowchart.ts` - Updated `Condition.value` type from `number` to `number | string | boolean` to support all variable types + - `src/components/editor/ConditionEditor.tsx` - New component: modal-based condition editor with Combobox for variable selection, type-aware operator filtering, type-adaptive value inputs, inline "Add new variable" form, orange warning for invalid references, and "Remove condition" action + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `onEdgeClick` handler to open ConditionEditor, `handleConditionChange` to update edge condition data, `selectedEdgeId` state, and ConditionEditor rendering - **Learnings for future iterations:** - - Supabase invite tokens come via URL hash fragment (window.location.hash) - - Parse hash with URLSearchParams after removing leading '#' - - Check for type=invite or type=signup to detect invite flow - - Use setSession() with access_token and refresh_token to establish session from invite link - - For invited users, update password with updateUser() then create profile with upsert() - - Use upsert() instead of insert() for profiles to handle edge cases - - Validate password confirmation before submission (passwords match check) - - display_name defaults to email prefix (split('@')[0]) + - Edge interactions in React Flow use `onEdgeClick` prop on the ReactFlow component (not on individual edges). The handler receives `(event: React.MouseEvent, edge: Edge)`. + - The ConditionEditor is rendered as a modal overlay (fixed z-50), not as part of the edge itself — since edges don't have built-in panel/popover support in React Flow. + - `Condition.value` was originally typed as just `number` but needed broadening to `number | string | boolean` to support string/boolean variables in conditions. This change didn't break existing code since the VariableNode's `value` field is a separate type. + - Operator filtering for non-numeric types: only `==` and `!=` are available for string/boolean variables. When switching from a numeric variable to a string/boolean, the operator auto-resets to `==` if it was a comparison operator. + - Value input adapts to type: number input for numeric, text input for string, boolean dropdown for boolean. + - The `selectedEdge` is derived via `useMemo` from `edges` state and `selectedEdgeId`, so it always reflects the latest condition data. + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-008 -- What was implemented: Logout functionality component +## 2026-01-23 - US-061 +- What was implemented: Choice option condition variable dropdown using OptionConditionEditor component with Combobox - Files changed: - - src/components/LogoutButton.tsx - new client component with signOut and redirect - - src/components/.gitkeep - removed (no longer needed) + - `src/types/flowchart.ts` - Added `condition?: Condition` to `ChoiceOption` type; moved `Condition` type definition before `ChoiceOption` for correct reference order + - `src/components/editor/OptionConditionEditor.tsx` - New component: modal-based condition editor for choice options with Combobox variable selection, type-aware operators, type-adaptive value inputs, inline "Add new variable" form, orange warning for invalid references + - `src/components/editor/nodes/ChoiceNode.tsx` - Added condition button per option (clipboard icon), condition summary text below options, OptionConditionEditor integration, EditorContext usage for variables, invalid reference detection with orange warning styling + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Extended `getVariableUsageCount` to also count variable references in choice option conditions - **Learnings for future iterations:** - - LogoutButton is a reusable component that will be used in the navbar (US-011) - - Component uses 'use client' directive for client-side auth operations - - Loading state prevents double-clicks during signOut - - Styled with neutral zinc colors to work as a secondary button in navbars + - The `OptionConditionEditor` follows the same pattern as `ConditionEditor` but with a simpler API: it doesn't need an edgeId since it works with a single option's condition via `onChange(condition | undefined)` callback + - The `ChoiceOption` type in `flowchart.ts` now references `Condition`, which required reordering type definitions (Condition must be defined before ChoiceOption) + - Each choice option shows a small clipboard icon button that turns blue when a condition is set, or orange when the referenced variable is invalid/deleted + - A condition summary line (e.g., "if score > 10") appears below each option label when a condition is active + - The `getVariableUsageCount` in FlowchartEditor now counts three sources: variable nodes, edge conditions, and choice option conditions + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-009 -- What was implemented: Password reset - forgot password page +## 2026-01-23 - US-062 +- What was implemented: Auto-migration of existing free-text speaker/variable values to character/variable definitions on project load - Files changed: - - src/app/forgot-password/page.tsx - new file with forgot password form and email reset + - `src/app/editor/[projectId]/page.tsx` - Added `needsMigration` flag that detects whether raw DB data has characters/variables arrays + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `computeMigration()` helper function and `needsMigration` prop; migration result initializes state directly via lazy `useState` to avoid React Compiler lint issues + - `src/components/editor/nodes/DialogueNode.tsx` - Included pre-existing US-058 changes (speaker dropdown with Combobox) that were not previously committed - **Learnings for future iterations:** - - resetPasswordForEmail requires redirectTo option to specify where user lands after clicking reset link - - Use `window.location.origin` to get the current site URL for redirectTo - - Page shows different UI after success (conditional rendering with success state) - - Use ' for apostrophe in JSX to avoid HTML entity issues - - Follow same styling pattern as login page for consistency across auth pages + - React Compiler lint (`react-hooks/set-state-in-effect`) forbids calling `setState` synchronously within `useEffect`. For one-time initialization logic, compute the result and use it directly in state initializers instead. + - React Compiler lint (`react-hooks/refs`) forbids reading `useRef().current` during render. Use `useState(() => ...)` lazy initializer pattern instead of `useRef` for values computed once at mount. + - The migration detection relies on `rawData.characters` being `undefined` (old projects) vs `[]` (migrated projects). The `page.tsx` server component passes `needsMigration` flag to the client component since only the server has access to the raw DB shape. + - `computeMigration` is a pure function called outside the component render cycle (via lazy useState). It uses `nanoid()` for IDs, so it must only be called once — lazy `useState` ensures this. + - The toast message for migration is set as initial state, so it shows immediately on first render without needing an effect. + - No browser testing tools are available; manual verification is needed. --- -## 2026-01-21 - US-010 -- What was implemented: Password reset - set new password page +## 2026-01-23 - US-063 +- What was implemented: Import characters/variables from another project via modal in project settings - Files changed: - - src/app/reset-password/page.tsx - new file with password reset form - - src/app/login/page.tsx - updated to show success message from password reset + - `src/components/editor/ImportFromProjectModal.tsx` - New component: project list modal with checkbox selection for characters or variables, duplicate-by-name skipping with warnings, select all/none controls + - `src/components/editor/ProjectSettingsModal.tsx` - Added `projectId` prop, `ImportFromProjectModal` integration, and "Import from project" buttons in both Characters and Variables tabs + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Passed `projectId` through to `ProjectSettingsModal` - **Learnings for future iterations:** - - Supabase recovery tokens come via URL hash fragment with type=recovery - - Use setSession() with access_token and refresh_token from hash to establish recovery session - - Show loading state while verifying token validity (tokenValid === null) - - Show error state with link to request new reset if token is invalid - - After password update, sign out the user and redirect to login with success message - - Use query param (message=password_reset_success) to pass success state between pages - - Login page uses useSearchParams to read and display success messages - - Success messages styled with green background (bg-green-50) ---- - -## 2026-01-21 - US-011 -- What was implemented: Dashboard layout with navbar component -- Files changed: - - src/app/dashboard/layout.tsx - new file with dashboard layout wrapper - - src/components/Navbar.tsx - new reusable navbar component -- **Learnings for future iterations:** - - Dashboard layout is a server component that fetches user data via createClient() from lib/supabase/server.ts - - Navbar accepts userEmail prop to display current user - - Layout wraps children with consistent max-w-7xl container and padding - - Navbar uses Link component to allow clicking app title to go back to dashboard - - Navbar has border-b styling with dark mode support for visual separation - - Use gap-4 for spacing between navbar items (user email and logout button) ---- - -## 2026-01-21 - US-012 -- What was implemented: Dashboard page listing user projects -- Files changed: - - src/app/dashboard/page.tsx - new file with project listing, cards, and empty state -- **Learnings for future iterations:** - - Dashboard page is a server component that fetches projects directly from Supabase - - Use .eq('user_id', user.id) for RLS-backed queries (though RLS also enforces this) - - Order by updated_at descending to show most recent projects first - - formatDate() helper with toLocaleDateString for human-readable dates - - Project cards use Link component for navigation to /editor/[projectId] - - Empty state uses dashed border (border-dashed) with centered content and icon - - Hover effects on cards: border-blue-300, shadow-md, and text color change on title - - Error state displayed if Supabase query fails ---- - -## 2026-01-21 - US-013 -- What was implemented: Create new project functionality -- Files changed: - - src/components/NewProjectButton.tsx - new client component with modal dialog - - src/app/dashboard/page.tsx - added NewProjectButton to header area - - src/app/signup/page.tsx - fixed lint error (setState in effect) by initializing email from searchParams -- **Learnings for future iterations:** - - Modal dialogs use fixed positioning with backdrop (bg-black/50) for overlay effect - - Form submission uses Supabase insert with .select('id').single() to get the new record ID - - Initialize flowchart_data with { nodes: [], edges: [] } for new projects - - router.push() for programmatic navigation after successful creation - - autoFocus on input for better UX when modal opens - - Prevent modal close while loading (check isLoading before calling handleClose) - - ESLint rule react-hooks/set-state-in-effect warns against synchronous setState in useEffect - - Initialize state from searchParams directly in useState() instead of setting in useEffect ---- - -## 2026-01-21 - US-014 -- What was implemented: Delete project functionality with confirmation dialog and toast -- Files changed: - - src/components/ProjectCard.tsx - new client component replacing Link, with delete button and confirmation dialog - - src/components/ProjectList.tsx - new wrapper component to manage project list state and toast notifications - - src/components/Toast.tsx - new reusable toast notification component - - src/app/dashboard/page.tsx - updated to use ProjectList instead of inline rendering -- **Learnings for future iterations:** - - To enable client-side state updates (like removing items), extract list rendering from server components into client components - - ProjectList accepts initialProjects from server and manages state locally for immediate UI updates - - Use onDelete callback pattern to propagate deletion events from child (ProjectCard) to parent (ProjectList) - - Delete button uses e.stopPropagation() to prevent card click navigation when clicking delete - - Confirmation dialogs should disable close/cancel while action is in progress (isDeleting check) - - Toast component uses useCallback for handlers and auto-dismiss with setTimeout - - Toast animations can use TailwindCSS animate-in utilities (fade-in, slide-in-from-bottom-4) - - Delete icon appears on hover using group-hover:opacity-100 with parent group class ---- - -## 2026-01-21 - US-015 -- What was implemented: Rename project functionality -- Files changed: - - src/components/ProjectCard.tsx - added rename button, modal dialog, and Supabase update logic - - src/components/ProjectList.tsx - added handleRename callback and toast notification -- **Learnings for future iterations:** - - Multiple action buttons on a card can be grouped in a flex container with gap-1 - - Rename modal follows same pattern as delete dialog: fixed positioning, backdrop, form - - Use onKeyDown to handle Enter key for quick form submission - - Reset form state (newName, error) when opening modal to handle edge cases - - Check if name is unchanged before making API call to avoid unnecessary requests - - Trim whitespace from input value before validation and submission - - handleRename callback updates project name in state using map() to preserve list order ---- - -## 2026-01-21 - US-016 -- What was implemented: Admin invite user functionality -- Files changed: - - src/app/admin/invite/page.tsx - new admin-only page with access check (redirects non-admins) - - src/app/admin/invite/InviteForm.tsx - client component with invite form and state management - - src/app/admin/invite/actions.ts - server action using service role key to call inviteUserByEmail - - src/components/Navbar.tsx - added isAdmin prop and "Invite User" link (visible only to admins) - - src/app/dashboard/layout.tsx - fetches profile.is_admin and passes it to Navbar - - .env.example - added SUPABASE_SERVICE_ROLE_KEY and NEXT_PUBLIC_SITE_URL -- **Learnings for future iterations:** - - Admin operations require SUPABASE_SERVICE_ROLE_KEY (server-side only, not NEXT_PUBLIC_*) - - Use createClient from @supabase/supabase-js directly for admin client (not @supabase/ssr) - - Admin client needs auth config: { autoRefreshToken: false, persistSession: false } - - inviteUserByEmail requires redirectTo option for the signup link in email - - Server actions ('use server') can access private env vars safely - - Admin check should happen both in server component (redirect) and server action (double check) - - Admin page uses its own layout (not dashboard layout) to have custom styling ---- - -## 2026-01-21 - US-017 -- What was implemented: Editor page with React Flow canvas -- Files changed: - - package.json - added reactflow dependency - - src/app/editor/[projectId]/page.tsx - new server component that fetches project from Supabase, handles auth/not found, renders header with back link - - src/app/editor/[projectId]/FlowchartEditor.tsx - new client component with React Flow canvas, Background component, type converters for nodes/edges - - src/app/editor/[projectId]/loading.tsx - new loading state component with spinner -- **Learnings for future iterations:** - - React Flow requires 'use client' directive since it uses browser APIs - - Import 'reactflow/dist/style.css' for default React Flow styling - - Use useNodesState and useEdgesState hooks for managing nodes/edges state - - Convert app types (FlowchartNode, FlowchartEdge) to React Flow types with helper functions - - Next.js dynamic route params come as Promise in App Router 16+ (need to await params) - - Use notFound() from next/navigation for 404 responses - - React Flow canvas needs parent container with explicit height (h-full, h-screen) - - Background component accepts variant (Dots, Lines, Cross) and gap/size props - - Loading page (loading.tsx) provides automatic loading UI for async server components ---- - -## 2026-01-21 - US-018 -- What was implemented: Canvas pan and zoom controls -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - added Controls import and component -- **Learnings for future iterations:** - - React Flow Controls component provides zoom +/-, fitView, and lock buttons out of the box - - Use position="bottom-right" prop to position controls in bottom-right corner - - Pan (click-and-drag) and zoom (mouse wheel) are React Flow defaults, no extra config needed ---- - -## 2026-01-21 - US-019 -- What was implemented: Editor toolbar with add/save/export/import buttons -- Files changed: - - src/components/editor/Toolbar.tsx - new toolbar component with styled buttons - - src/app/editor/[projectId]/FlowchartEditor.tsx - integrated toolbar with placeholder handlers -- **Learnings for future iterations:** - - Toolbar component accepts callback props for actions (onAddDialogue, onSave, etc.) - - Node type buttons use color coding: blue (Dialogue), green (Choice), orange (Variable) - - Action buttons (Save, Export, Import) use neutral bordered styling - - FlowchartEditor now uses flex-col layout to stack toolbar above canvas - - Placeholder handlers with TODO comments help track future implementation work ---- - -## 2026-01-21 - US-020 -- What was implemented: Custom DialogueNode component for displaying/editing character dialogue -- Files changed: - - src/components/editor/nodes/DialogueNode.tsx - new custom node component with editable speaker and text fields - - src/app/editor/[projectId]/FlowchartEditor.tsx - registered DialogueNode as custom node type -- **Learnings for future iterations:** - - Custom React Flow nodes use NodeProps for typing, where T is the data shape - - Use useReactFlow() hook to get setNodes for updating node data from within the node component - - Handle components need Position enum (Position.Top, Position.Bottom) for positioning - - Custom handles can be styled with className and TailwindCSS, use ! prefix to override defaults (e.g., !h-3, !w-3) - - Node types must be registered in a nodeTypes object and passed to ReactFlow component - - Memoize nodeTypes with useMemo to prevent unnecessary re-renders - - Custom node components go in src/components/editor/nodes/ directory ---- - -## 2026-01-21 - US-021 -- What was implemented: Add dialogue node from toolbar functionality -- Files changed: - - package.json - added nanoid dependency for unique ID generation - - src/app/editor/[projectId]/FlowchartEditor.tsx - implemented handleAddDialogue to create new dialogue nodes at viewport center -- **Learnings for future iterations:** - - useReactFlow() hook requires ReactFlowProvider wrapper, so split component into inner component and outer wrapper - - getViewport() returns { x, y, zoom } representing the current pan/zoom state - - Calculate viewport center: centerX = (-viewport.x + halfWidth) / viewport.zoom - - nanoid v5+ generates unique IDs synchronously with no dependencies - - Node creation pattern: create Node object with { id, type, position, data }, then add to state via setNodes - - React Flow nodes are draggable by default, no extra configuration needed ---- - -## 2026-01-21 - US-022 -- What was implemented: Custom ChoiceNode component for displaying branching decisions -- Files changed: - - src/components/editor/nodes/ChoiceNode.tsx - new custom node component with green styling, editable prompt, and dynamic option handles - - src/app/editor/[projectId]/FlowchartEditor.tsx - registered ChoiceNode as custom node type -- **Learnings for future iterations:** - - ChoiceNode follows same pattern as DialogueNode: NodeProps typing, useReactFlow() for updates - - Dynamic handles positioned using style={{ left: `${((index + 1) / (options.length + 1)) * 100}%` }} for even spacing - - Handle id format for options: 'option-0', 'option-1', etc. (matching the index) - - Each option needs a unique id (string) and label (string) per the ChoiceOption type - - updateOptionLabel callback pattern: find option by id, map over options array to update matching one ---- - -## 2026-01-21 - US-023 -- What was implemented: Add choice node from toolbar functionality -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - implemented handleAddChoice to create new choice nodes at viewport center -- **Learnings for future iterations:** - - handleAddChoice follows same pattern as handleAddDialogue: get viewport center, create node with nanoid, add to state - - Choice nodes must be initialized with 2 options (each with unique id via nanoid and empty label) - - Node data structure for choice: { prompt: '', options: [{ id, label }, { id, label }] } - - React Flow nodes are draggable by default after being added to state ---- - -## 2026-01-21 - US-024 -- What was implemented: Add/remove choice options functionality (2-6 options supported) -- Files changed: - - src/components/editor/nodes/ChoiceNode.tsx - added addOption and removeOption callbacks, '+' button to add options, 'x' button per option to remove -- **Learnings for future iterations:** - - Define MIN_OPTIONS and MAX_OPTIONS constants for clear limits - - Use disabled prop on buttons to enforce min/max constraints with appropriate styling (opacity-30, cursor-not-allowed) - - Remove button uses × character for simple cross icon - - Add button styled with border-dashed for visual distinction from action buttons - - Handles update dynamically via React Flow re-render when options array changes ---- - -## 2026-01-21 - US-025 -- What was implemented: Custom VariableNode component for setting/modifying story variables -- Files changed: - - src/components/editor/nodes/VariableNode.tsx - new custom node component with orange styling, editable variable name, operation dropdown, and numeric value input - - src/app/editor/[projectId]/FlowchartEditor.tsx - imported and registered VariableNode in nodeTypes -- **Learnings for future iterations:** - - VariableNode follows same pattern as DialogueNode: NodeProps typing, useReactFlow() for updates - - Use parseFloat() with fallback to 0 for number input handling: `parseFloat(e.target.value) || 0` - - Operation dropdown uses select element with options for 'set', 'add', 'subtract' - - Type assertion needed for select value: `e.target.value as 'set' | 'add' | 'subtract'` - - Use `??` (nullish coalescing) for number defaults instead of `||` to allow 0 values: `data.value ?? 0` ---- - -## 2026-01-21 - US-026 -- What was implemented: Add variable node from toolbar functionality -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - implemented handleAddVariable to create new variable nodes at viewport center -- **Learnings for future iterations:** - - handleAddVariable follows same pattern as handleAddDialogue and handleAddChoice: get viewport center, create node with nanoid, add to state - - Variable nodes initialized with { variableName: '', operation: 'set', value: 0 } - - All add node handlers share the same pattern and use the getViewportCenter helper ---- - -## 2026-01-21 - US-027 -- What was implemented: Connect nodes with edges including arrow markers and smooth styling -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - added MarkerType import, updated onConnect to create edges with smoothstep type and arrow markers, updated toReactFlowEdges to apply same styling to loaded edges -- **Learnings for future iterations:** - - Use `type: 'smoothstep'` for cleaner edge curves instead of default bezier - - Use `markerEnd: { type: MarkerType.ArrowClosed }` to add directional arrows to edges - - Connection type has nullable source/target, but Edge requires non-null strings - guard with early return - - Apply consistent edge styling in both onConnect (new edges) and toReactFlowEdges (loaded edges) - - Generate unique edge IDs with nanoid in onConnect callback ---- - -## 2026-01-21 - US-028 -- What was implemented: Select and delete nodes functionality -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - added deleteKeyCode prop to enable Delete/Backspace key deletion -- **Learnings for future iterations:** - - React Flow has built-in node selection via clicking - no extra configuration needed - - Use `deleteKeyCode={['Delete', 'Backspace']}` prop to enable keyboard deletion - - React Flow automatically removes connected edges when a node is deleted (no manual cleanup needed) - - The useNodesState/useEdgesState hooks with onNodesChange/onEdgesChange handle all deletion state updates - - No explicit onNodesDelete callback is needed - the onNodesChange handler covers deletion events ---- - -## 2026-01-21 - US-029 -- What was implemented: Select and delete edges functionality -- Files changed: - - src/app/editor/[projectId]/FlowchartEditor.tsx - added onEdgesDelete callback -- **Learnings for future iterations:** - - React Flow 11 edges are clickable and selectable by default (interactionWidth renders invisible interaction area) - - The `deleteKeyCode` prop works for both nodes and edges - same configuration covers both - - onEdgesDelete is optional if you just need state management (onEdgesChange handles removal events) - - onEdgesDelete is useful for additional logic like logging, dirty state tracking, or undo/redo - - Edge selection shows visual highlight via React Flow's built-in styling + - The `ImportFromProjectModal` uses `z-[60]` to layer above the `ProjectSettingsModal` (which uses `z-50`), since it's rendered as a child of that modal + - Imported characters/variables get new IDs via `nanoid()` to avoid ID collisions between projects. The original colors, types, and initial values are preserved. + - Duplicate detection is case-insensitive by name. Duplicates are skipped (not overwritten) with a warning message shown to the user. + - The `LoadingSpinner` component mentioned in Codebase Patterns doesn't exist; used inline text loading indicators instead. + - Supabase client-side fetching from `createClient()` (browser) automatically scopes by the logged-in user's RLS policies, so fetching other projects just uses `.neq('id', currentProjectId)` and RLS handles ownership filtering. + - No browser testing tools are available; manual verification is needed. --- ## 2026-01-22 - US-030 @@ -632,3 +406,102 @@ - After successful password update, sign out the user and redirect to login with success message (same as reset-password page) - The modal coexists with the existing /reset-password page - both handle recovery tokens but in different UX patterns --- + +## 2026-01-23 - US-047 +- What was implemented: Live cursor positions on canvas showing collaborators' mouse positions in real-time +- Files changed: + - `src/lib/collaboration/realtime.ts` - Added `CursorPosition`, `RemoteCursor` types, `onCursorUpdate` callback to `RealtimeCallbacks`, broadcast listener for 'cursor' events, and `broadcastCursor()` method + - `src/components/editor/RemoteCursors.tsx` - New component: renders colored arrow cursors with user name labels, smooth position interpolation via CSS transition, 5-second fade-out for inactive cursors, flow-to-screen coordinate transformation using React Flow viewport + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `remoteCursors` state, `cursorThrottleRef` for 50ms throttling, `handleMouseMove` that converts screen→flow coordinates and broadcasts via RealtimeConnection, cleanup of cursors for disconnected users, rendering of RemoteCursors overlay + - `src/app/editor/[projectId]/page.tsx` - Fixed broken JSX structure (malformed HTML nesting and dead code after return) +- **Learnings for future iterations:** + - `screenToFlowPosition` from `useReactFlow()` converts screen-relative mouse coordinates to flow coordinates; for the reverse (rendering cursors), multiply by viewport.zoom and add viewport offset + - Cursor broadcast uses Supabase Realtime broadcast (not presence) for efficiency: `channel.send({ type: 'broadcast', event: 'cursor', payload })`. Broadcast is fire-and-forget (no persistence). + - React Compiler lint treats `Date.now()` as an impure function call — use `useState(() => Date.now())` lazy initializer pattern instead of `useState(Date.now())` + - Throttling mouse events uses a ref storing the last broadcast timestamp (`cursorThrottleRef`), checked at the start of the handler before computing flow position + - Remote cursors are removed when their user disconnects (filtered by `presenceUsers` list changes) + - CSS `transition: transform 80ms linear` provides smooth interpolation between position updates without needing requestAnimationFrame + - The `page.tsx` had a corrupted structure with unclosed tags and dead code — likely from a failed merge. Fixed by restructuring the error/not-found case into a proper early return + - No browser testing tools are available; manual verification is needed. +--- + +## 2026-01-23 - US-050 +- What was implemented: Join/leave toast notifications when collaborators connect or disconnect from the editing session +- Files changed: + - `src/lib/collaboration/realtime.ts` - Added `onPresenceJoin` and `onPresenceLeave` callbacks to `RealtimeCallbacks` type; added Supabase Realtime `presence.join` and `presence.leave` event listeners that filter out own user and invoke callbacks + - `src/components/editor/CollaborationToast.tsx` - New component: renders a compact toast notification with user's presence color dot, "[Name] joined" or "[Name] left" message, auto-dismisses after 3 seconds + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `getUserColor` helper (same hash logic as PresenceAvatars), `collaborationNotifications` state, `onPresenceJoin`/`onPresenceLeave` handlers on RealtimeConnection, `handleDismissNotification` callback, and rendering of CollaborationToast list in bottom-left corner +- **Learnings for future iterations:** + - Supabase Realtime presence has three event types: `sync` (full state), `join` (new arrivals), and `leave` (departures). Each provides an array of presences (`newPresences`/`leftPresences`). Use all three for different purposes. + - The `join` event fires for each newly tracked presence. It includes the presence payload (userId, displayName) that was passed to `channel.track()`. + - Collaboration notifications are positioned `bottom-left` (`left-4`) to avoid overlapping with the existing Toast component which is `bottom-right` (`right-4`). + - The `getUserColor` function is duplicated from PresenceAvatars to avoid circular imports. Both use the same hash-to-color-index algorithm with the same RANDOM_COLORS palette for consistency. + - No browser testing tools are available; manual verification is needed. +--- + +## 2026-01-24 - US-049 +- What was implemented: Node editing lock indicators that show when another collaborator is editing a node +- Files changed: + - `src/lib/collaboration/realtime.ts` - Added `NodeLock` type, `onNodeLockUpdate` callback to `RealtimeCallbacks`, `node-lock` broadcast listener, and `broadcastNodeLock()` method + - `src/components/editor/NodeLockIndicator.tsx` - New component: renders a colored border and name label overlay on locked nodes + - `src/components/editor/EditorContext.tsx` - Extended context with `NodeLockInfo` type, `nodeLocks` (Map), `onNodeFocus`, and `onNodeBlur` callbacks + - `src/components/editor/nodes/DialogueNode.tsx` - Added lock detection, `NodeLockIndicator` rendering, "Being edited by [name]" overlay, `onFocus`/`onBlur` handlers + - `src/components/editor/nodes/VariableNode.tsx` - Same lock indicator pattern as DialogueNode + - `src/components/editor/nodes/ChoiceNode.tsx` - Same lock indicator pattern as DialogueNode + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `nodeLocks` state (Map), `localLockRef` for tracking own lock, `handleNodeFocus`/`handleNodeBlur` callbacks, `onNodeLockUpdate` handler in RealtimeConnection, lock expiry timer (60s check every 5s), lock cleanup on user leave and component unmount, extended `editorContextValue` with lock state +- **Learnings for future iterations:** + - Node lock uses Supabase Realtime broadcast (like cursors) — ephemeral, not persisted to DB. Event name: `node-lock`. + - Lock broadcasting uses `nodeId: string | null` pattern: non-null to acquire lock, `null` to release. The receiving side maps userId to their current lock. + - Lock expiry uses a 60-second timeout checked every 5 seconds via `setInterval`. The `lockedAt` timestamp is broadcast with the lock payload. + - Each node component accesses lock state via `EditorContext` (`nodeLocks` Map). The `NodeLockInfo` type extends `NodeLock` with a `color` field derived from `getUserColor()`. + - `onFocus`/`onBlur` on the node container div fires when any child input gains/loses focus (focus event bubbles), which naturally maps to "user is editing this node". + - Lock release on disconnect: broadcast `null` lock before calling `connection.disconnect()` in the cleanup return of the mount effect. + - Locks from disconnected users are cleaned up in the same effect that removes cursors (filtered by `presenceUsers` list). + - No browser testing tools are available; manual verification is needed. +--- + +## 2026-01-24 - US-051 +- What was implemented: Audit trail recording that writes all node/edge add/update/delete operations to the `audit_trail` table with debouncing and fire-and-forget semantics +- Files changed: + - `src/lib/collaboration/auditTrail.ts` - New `AuditTrailRecorder` class: tracks previous node/edge state, diffs against current state to detect add/update/delete operations, debounces writes per entity (1 second), merges rapid sequential actions (e.g., add+update=add, add+delete=no-op), fire-and-forget Supabase inserts with error logging, flush on destroy + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `auditRef` (AuditTrailRecorder), initialized in mount effect alongside CRDT manager, records node/edge changes in the CRDT sync effects (only for local changes, skipped for remote updates), destroyed on unmount +- **Learnings for future iterations:** + - The audit recorder piggybacks on the same CRDT sync effects that already compute `nodesForCRDT`/`edgesForCRDT` — this avoids duplicating the React Flow → FlowchartNode/Edge conversion. + - The `isRemoteUpdateRef` guard in the sync effects ensures audit entries are only created for local user actions, not for changes received from other collaborators (those users' own recorders will handle their audit entries). + - Debouncing per entity (1 second) prevents rapid edits (e.g., typing in a text field) from flooding the audit table. The merge logic handles transient states (add+delete within 1s = skip). + - The `destroy()` method flushes pending entries synchronously on unmount, ensuring in-flight edits aren't lost when navigating away. + - Supabase `.insert().then()` pattern provides fire-and-forget writes with error logging — the async operation doesn't block the editing flow. + - No browser testing needed — this is a developer/infrastructure story with no UI changes. +--- + +## 2026-01-24 - US-052 +- What was implemented: Activity history sidebar that displays audit trail entries, grouped by time period, with entity selection on click +- Files changed: + - `src/components/editor/ActivityHistorySidebar.tsx` - New component: right sidebar panel showing chronological audit trail entries, grouped by Today/Yesterday/Earlier, with user color accents, entity descriptions, paginated loading (20 per page), and click-to-select entity on canvas + - `src/components/editor/Toolbar.tsx` - Added `onHistory` prop and "History" button in the right toolbar section + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `showHistory` state, `handleHistorySelectEntity` callback (selects nodes/edges on canvas), `ActivityHistorySidebar` import and rendering inside the canvas area, `onHistory` toggle prop on Toolbar +- **Learnings for future iterations:** + - React Compiler lint (`react-hooks/set-state-in-effect`) treats any function that calls setState as problematic when invoked inside useEffect — even if the setState is in an async `.then()` callback. To avoid this, extract data fetching into a pure function that returns a result object, then handle setState only in the `.then()` callback after checking a mounted/aborted guard. + - For right sidebar panels overlaying the canvas, use `absolute right-0 top-0 z-40 h-full w-80` inside the `relative flex-1` canvas container. This keeps the sidebar within the canvas area without affecting the toolbar. + - The `audit_trail` table has an index on `(project_id, created_at DESC)` which makes paginated queries efficient. Use `.range(offset, offset + PAGE_SIZE - 1)` for Supabase pagination. + - Entity descriptions are derived from `new_state` (for adds/updates) or `previous_state` (for deletes). The state contains the full node/edge data including `type`, `data.speaker`, `data.question`, `data.variableName`. + - Deleted entities (`action_type.endsWith('_delete')`) cannot be selected on canvas since they no longer exist — render those entries as disabled (no click handler, reduced opacity). + - No browser testing tools are available; manual verification is needed. +--- + +## 2026-01-24 - US-053 +- What was implemented: Revert changes from audit trail — each history entry has a Revert button, confirmation dialog with before/after preview, and revert logic that reverses node/edge add/update/delete operations +- Files changed: + - `src/components/editor/RevertConfirmDialog.tsx` - New component: confirmation dialog showing action description, before/after state previews, with Cancel and Revert buttons + - `src/components/editor/ActivityHistorySidebar.tsx` - Added Revert button (shows on hover per entry), `revertEntry` state, `RevertConfirmDialog` rendering, exported `AuditEntry` type, added `onRevert` prop + - `src/app/editor/[projectId]/FlowchartEditor.tsx` - Added `handleRevertEntry` callback implementing all revert cases (node add→delete, node update→restore, node delete→recreate, same for edges), `isRevertingRef` to prevent double audit recording, `getRevertActionType` helper, explicit audit trail write for the revert itself, toast notification on success +- **Learnings for future iterations:** + - Reverting uses the same `setNodes`/`setEdges` state setters, so CRDT sync happens automatically through the existing effects that watch `nodesForCRDT`/`edgesForCRDT` — no explicit CRDT call needed. + - To prevent double audit recording (once from the sync effect, once from the explicit revert write), use an `isRevertingRef` guard set synchronously before `setNodes`/`setEdges` and cleared with `setTimeout(() => ..., 0)` after React processes the state updates. + - The revert audit entry uses inverse action types: reverting `node_add` records `node_delete`, reverting `node_delete` records `node_add`, `node_update` stays `node_update`. + - The `previous_state` and `new_state` are swapped for the revert audit entry: what was `new_state` (the current state being reverted) becomes `previous_state`, and what was `previous_state` (the state being restored) becomes `new_state`. + - Reverting a `node_add` also removes connected edges to prevent dangling edge references. + - The RevertConfirmDialog uses `z-[70]` to layer above the ActivityHistorySidebar (`z-40`). + - The Revert button uses CSS `group-hover:inline-block` pattern to appear only on entry hover, keeping the UI clean. + - No browser testing tools are available; manual verification is needed. +--- diff --git a/src/app/dashboard/page.tsx b/src/app/dashboard/page.tsx index 8b153c5..89bceb4 100644 --- a/src/app/dashboard/page.tsx +++ b/src/app/dashboard/page.tsx @@ -1,6 +1,7 @@ import { createClient } from '@/lib/supabase/server' import NewProjectButton from '@/components/NewProjectButton' import ProjectList from '@/components/ProjectList' +import ProjectCard from '@/components/ProjectCard' export default async function DashboardPage() { const supabase = await createClient() @@ -19,6 +20,21 @@ export default async function DashboardPage() { .eq('user_id', user.id) .order('updated_at', { ascending: false }) + // Fetch shared projects (projects where this user is a collaborator) + const { data: collaborations } = await supabase + .from('project_collaborators') + .select('role, projects(id, name, updated_at)') + .eq('user_id', user.id) + + const sharedProjects = (collaborations || []) + .filter((c) => c.projects) + .map((c) => ({ + ...(c.projects as unknown as { id: string; name: string; updated_at: string }), + shared: true, + role: c.role, + })) + .sort((a, b) => new Date(b.updated_at).getTime() - new Date(a.updated_at).getTime()) + if (error) { return (
@@ -44,6 +60,26 @@ export default async function DashboardPage() {
+ + {sharedProjects.length > 0 && ( +
+

+ Shared with me +

+
+ {sharedProjects.map((project) => ( + + ))} +
+
+ )} ) } diff --git a/src/app/editor/[projectId]/FlowchartEditor.tsx b/src/app/editor/[projectId]/FlowchartEditor.tsx index 707cebc..d134a63 100644 --- a/src/app/editor/[projectId]/FlowchartEditor.tsx +++ b/src/app/editor/[projectId]/FlowchartEditor.tsx @@ -1,7 +1,6 @@ 'use client' -import { useCallback, useEffect, useMemo, useRef, useState } from 'react' -import { useRouter } from 'next/navigation' +import React, { useCallback, useEffect, useMemo, useRef, useState } from 'react' import ReactFlow, { Background, BackgroundVariant, @@ -30,14 +29,25 @@ import ChoiceNode from '@/components/editor/nodes/ChoiceNode' import VariableNode from '@/components/editor/nodes/VariableNode' import ConditionalEdge from '@/components/editor/edges/ConditionalEdge' import ContextMenu, { ContextMenuType } from '@/components/editor/ContextMenu' +import ProjectSettingsModal from '@/components/editor/ProjectSettingsModal' import ConditionEditor from '@/components/editor/ConditionEditor' -import type { FlowchartData, FlowchartNode, FlowchartEdge, Condition } from '@/types/flowchart' +import ExportValidationModal, { type ValidationIssue } from '@/components/editor/ExportValidationModal' +import { EditorProvider } from '@/components/editor/EditorContext' +import { RealtimeConnection, type ConnectionState, type PresenceUser, type RemoteCursor, type NodeLock } from '@/lib/collaboration/realtime' +import type { NodeLockInfo } from '@/components/editor/EditorContext' +import RemoteCursors from '@/components/editor/RemoteCursors' +import { CRDTManager } from '@/lib/collaboration/crdt' +import { AuditTrailRecorder } from '@/lib/collaboration/auditTrail' +import ShareModal from '@/components/editor/ShareModal' +import ActivityHistorySidebar, { type AuditEntry } from '@/components/editor/ActivityHistorySidebar' +import CollaborationToast, { type CollaborationNotification } from '@/components/editor/CollaborationToast' +import type { FlowchartData, FlowchartNode, FlowchartEdge, Character, Variable, Condition } from '@/types/flowchart' // LocalStorage key prefix for draft saves const DRAFT_KEY_PREFIX = 'vnwrite-draft-' -// Debounce delay in ms -const AUTOSAVE_DEBOUNCE_MS = 1000 +// Debounce delay for LocalStorage draft saves +const AUTOSAVE_DEBOUNCE_MS = 5000 type ContextMenuState = { x: number @@ -55,7 +65,11 @@ type ConditionEditorState = { type FlowchartEditorProps = { projectId: string projectName: string + userId: string + userDisplayName: string + isOwner: boolean initialData: FlowchartData + needsMigration?: boolean } // Convert our FlowchartNode type to React Flow Node type @@ -155,75 +169,35 @@ function isValidFlowchartData(data: unknown): data is FlowchartData { return true } -// Ren'Py export types -type RenpyDialogueNode = { - type: 'dialogue' - speaker: string - text: string - next?: string - condition?: Condition -} - -type RenpyMenuChoice = { - label: string - next?: string - condition?: Condition -} - -type RenpyMenuNode = { - type: 'menu' - prompt: string - choices: RenpyMenuChoice[] -} - -type RenpyVariableNode = { - type: 'variable' - name: string - operation: 'set' | 'add' | 'subtract' - value: number - next?: string - condition?: Condition -} - -type RenpyNode = RenpyDialogueNode | RenpyMenuNode | RenpyVariableNode - -type RenpyExport = { - projectName: string - exportedAt: string - sections: Record -} - // Find the first node (node with no incoming edges) function findFirstNode(nodes: FlowchartNode[], edges: FlowchartEdge[]): FlowchartNode | null { const targetIds = new Set(edges.map((e) => e.target)) const startNodes = nodes.filter((n) => !targetIds.has(n.id)) - // Return the first start node, or the first node if all have incoming edges return startNodes[0] || nodes[0] || null } -// Get outgoing edge from a node (for dialogue and variable nodes) +// Get outgoing edge from a node function getOutgoingEdge(nodeId: string, edges: FlowchartEdge[], sourceHandle?: string): FlowchartEdge | undefined { return edges.find((e) => e.source === nodeId && (sourceHandle === undefined || e.sourceHandle === sourceHandle)) } -// Get all outgoing edges from a node (for choice nodes) +// Get all outgoing edges from a node function getOutgoingEdges(nodeId: string, edges: FlowchartEdge[]): FlowchartEdge[] { return edges.filter((e) => e.source === nodeId) } -// Convert flowchart to Ren'Py format using graph traversal +// Convert flowchart to Ren'Py JSON format function convertToRenpyFormat( nodes: FlowchartNode[], edges: FlowchartEdge[], projectName: string -): RenpyExport { +): { projectName: string; exportedAt: string; sections: Record } { const nodeMap = new Map(nodes.map((n) => [n.id, n])) const visited = new Set() - const sections: Record = {} + const sections: Record = {} let currentSectionName = 'start' - let currentSection: RenpyNode[] = [] + let currentSection: unknown[] = [] - // Helper to get or create a label for a node const nodeLabels = new Map() let labelCounter = 0 @@ -231,7 +205,6 @@ function convertToRenpyFormat( if (!nodeLabels.has(nodeId)) { const node = nodeMap.get(nodeId) if (node?.type === 'dialogue' && node.data.speaker) { - // Use speaker name as part of label if available nodeLabels.set(nodeId, `section_${node.data.speaker.toLowerCase().replace(/\s+/g, '_')}_${labelCounter++}`) } else { nodeLabels.set(nodeId, `section_${labelCounter++}`) @@ -240,7 +213,6 @@ function convertToRenpyFormat( return nodeLabels.get(nodeId)! } - // Process a node and its successors function processNode(nodeId: string): void { if (visited.has(nodeId)) return visited.add(nodeId) @@ -250,141 +222,73 @@ function convertToRenpyFormat( if (node.type === 'dialogue') { const outgoingEdge = getOutgoingEdge(nodeId, edges) - const renpyNode: RenpyDialogueNode = { + const renpyNode: Record = { type: 'dialogue', speaker: node.data.speaker || '', text: node.data.text, } - if (outgoingEdge) { - // Check if target node is already visited (creates a jump) - if (visited.has(outgoingEdge.target)) { - renpyNode.next = getNodeLabel(outgoingEdge.target) - } else { - renpyNode.next = outgoingEdge.target - } + renpyNode.next = visited.has(outgoingEdge.target) ? getNodeLabel(outgoingEdge.target) : outgoingEdge.target if (outgoingEdge.data?.condition) { renpyNode.condition = outgoingEdge.data.condition } } - currentSection.push(renpyNode) - - // Process next node if not visited if (outgoingEdge && !visited.has(outgoingEdge.target)) { processNode(outgoingEdge.target) } } else if (node.type === 'choice') { - const outgoingEdges = getOutgoingEdges(nodeId, edges) - - // Map options to their corresponding edges - const choices: RenpyMenuChoice[] = node.data.options.map((option, index) => { - // Find edge for this option handle - const optionEdge = outgoingEdges.find((e) => e.sourceHandle === `option-${index}`) - const choice: RenpyMenuChoice = { - label: option.label || `Option ${index + 1}`, + const outEdges = getOutgoingEdges(nodeId, edges) + const choices = outEdges.map((edge) => { + const choiceData: Record = { + label: edge.sourceHandle || 'Choice', + next: edge.target, } - - if (optionEdge) { - // If target is visited, use label; otherwise use target id - if (visited.has(optionEdge.target)) { - choice.next = getNodeLabel(optionEdge.target) - } else { - choice.next = optionEdge.target - } - if (optionEdge.data?.condition) { - choice.condition = optionEdge.data.condition - } + if (edge.data?.condition) { + choiceData.condition = edge.data.condition } - - // Per-option condition (visibility condition) takes priority over edge condition - if (option.condition) { - choice.condition = option.condition - } - - return choice + return choiceData }) - - const renpyNode: RenpyMenuNode = { + currentSection.push({ type: 'menu', prompt: node.data.prompt || '', choices, - } - - currentSection.push(renpyNode) - - // Save current section before processing branches - sections[currentSectionName] = currentSection - - // Process each branch in a new section - for (const choice of choices) { - if (choice.next && !visited.has(choice.next)) { - const targetNode = nodeMap.get(choice.next) - if (targetNode) { - // Start new section for this branch - currentSectionName = getNodeLabel(choice.next) - currentSection = [] - processNode(choice.next) - if (currentSection.length > 0) { - sections[currentSectionName] = currentSection - } - } + }) + // Process each choice target as a new section + outEdges.forEach((edge) => { + if (!visited.has(edge.target)) { + sections[currentSectionName] = currentSection + currentSectionName = getNodeLabel(edge.target) + currentSection = [] + processNode(edge.target) } - } + }) } else if (node.type === 'variable') { const outgoingEdge = getOutgoingEdge(nodeId, edges) - const renpyNode: RenpyVariableNode = { + const renpyNode: Record = { type: 'variable', - name: node.data.variableName, - operation: node.data.operation, - value: node.data.value, + name: node.data.variableName || '', + operation: node.data.operation || 'set', + value: node.data.value ?? 0, } - if (outgoingEdge) { - if (visited.has(outgoingEdge.target)) { - renpyNode.next = getNodeLabel(outgoingEdge.target) - } else { - renpyNode.next = outgoingEdge.target - } + renpyNode.next = visited.has(outgoingEdge.target) ? getNodeLabel(outgoingEdge.target) : outgoingEdge.target if (outgoingEdge.data?.condition) { renpyNode.condition = outgoingEdge.data.condition } } - currentSection.push(renpyNode) - if (outgoingEdge && !visited.has(outgoingEdge.target)) { processNode(outgoingEdge.target) } } } - // Find and process starting from the first node - const firstNode = findFirstNode(nodes, edges) - if (firstNode) { - processNode(firstNode.id) - // Save the final section if it has content - if (currentSection.length > 0 && !sections[currentSectionName]) { - sections[currentSectionName] = currentSection - } - } - - // Replace node IDs in next fields with proper labels - for (const sectionNodes of Object.values(sections)) { - for (const renpyNode of sectionNodes) { - if (renpyNode.type === 'dialogue' || renpyNode.type === 'variable') { - if (renpyNode.next && nodeLabels.has(renpyNode.next)) { - renpyNode.next = nodeLabels.get(renpyNode.next) - } - } else if (renpyNode.type === 'menu') { - for (const choice of renpyNode.choices) { - if (choice.next && nodeLabels.has(choice.next)) { - choice.next = nodeLabels.get(choice.next) - } - } - } - } + const startNode = findFirstNode(nodes, edges) + if (startNode) { + processNode(startNode.id) } + sections[currentSectionName] = currentSection return { projectName, @@ -393,8 +297,170 @@ function convertToRenpyFormat( } } +const RANDOM_COLORS = [ + '#EF4444', '#F97316', '#F59E0B', '#10B981', + '#3B82F6', '#8B5CF6', '#EC4899', '#14B8A6', + '#6366F1', '#F43F5E', '#84CC16', '#06B6D4', +] + +function randomHexColor(): string { + return RANDOM_COLORS[Math.floor(Math.random() * RANDOM_COLORS.length)] +} + +// Generate a consistent color from a user ID hash (same logic as PresenceAvatars) +function getUserColor(userId: string): string { + let hash = 0 + for (let i = 0; i < userId.length; i++) { + hash = ((hash << 5) - hash + userId.charCodeAt(i)) | 0 + } + return RANDOM_COLORS[Math.abs(hash) % RANDOM_COLORS.length] +} + +// Determine the action type for a revert audit entry +function getRevertActionType(originalActionType: string): string { + switch (originalActionType) { + case 'node_add': return 'node_delete' // reverting an add = deleting + case 'node_delete': return 'node_add' // reverting a delete = adding back + case 'node_update': return 'node_update' // reverting an update = updating back + case 'edge_add': return 'edge_delete' + case 'edge_delete': return 'edge_add' + case 'edge_update': return 'edge_update' + default: return originalActionType + } +} + +// Compute auto-migration of existing free-text values to character/variable definitions +function computeMigration(initialData: FlowchartData, shouldMigrate: boolean) { + if (!shouldMigrate) { + return { + characters: initialData.characters, + variables: initialData.variables, + nodes: initialData.nodes, + edges: initialData.edges, + toastMessage: null as string | null, + } + } + + // Collect unique speaker names from dialogue nodes + const speakerNames = new Set() + initialData.nodes.forEach((node) => { + if (node.type === 'dialogue' && node.data?.speaker) { + speakerNames.add(node.data.speaker) + } + }) + + // Create character definitions from unique speaker names + const newCharacters: Character[] = [] + const speakerToCharacterId = new Map() + speakerNames.forEach((name) => { + const id = nanoid() + newCharacters.push({ id, name, color: randomHexColor() }) + speakerToCharacterId.set(name, id) + }) + + // Collect unique variable names from variable nodes, edge conditions, and choice option conditions + const variableNames = new Set() + initialData.nodes.forEach((node) => { + if (node.type === 'variable' && node.data.variableName) { + variableNames.add(node.data.variableName) + } + if (node.type === 'choice' && node.data.options) { + node.data.options.forEach((opt) => { + if (opt.condition?.variableName) { + variableNames.add(opt.condition.variableName) + } + }) + } + }) + initialData.edges.forEach((edge) => { + if (edge.data?.condition?.variableName) { + variableNames.add(edge.data.condition.variableName) + } + }) + + // Create variable definitions from unique variable names + const newVariables: Variable[] = [] + const varNameToId = new Map() + variableNames.forEach((name) => { + const id = nanoid() + newVariables.push({ id, name, type: 'numeric', initialValue: 0 }) + varNameToId.set(name, id) + }) + + // If nothing to migrate, return original data + if (newCharacters.length === 0 && newVariables.length === 0) { + return { + characters: initialData.characters, + variables: initialData.variables, + nodes: initialData.nodes, + edges: initialData.edges, + toastMessage: null as string | null, + } + } + + // Update nodes with characterId/variableId references + const migratedNodes = initialData.nodes.map((node) => { + if (node.type === 'dialogue' && node.data.speaker) { + const characterId = speakerToCharacterId.get(node.data.speaker) + if (characterId) { + return { ...node, data: { ...node.data, characterId } } + } + } + if (node.type === 'variable' && node.data.variableName) { + const variableId = varNameToId.get(node.data.variableName) + if (variableId) { + return { ...node, data: { ...node.data, variableId } } + } + } + if (node.type === 'choice' && node.data.options) { + const updatedOptions = node.data.options.map((opt) => { + if (opt.condition?.variableName) { + const variableId = varNameToId.get(opt.condition.variableName) + if (variableId) { + return { ...opt, condition: { ...opt.condition, variableId } } + } + } + return opt + }) + return { ...node, data: { ...node.data, options: updatedOptions } } + } + return node + }) as typeof initialData.nodes + + // Update edges with variableId references + const migratedEdges = initialData.edges.map((edge) => { + if (edge.data?.condition?.variableName) { + const variableId = varNameToId.get(edge.data.condition.variableName) + if (variableId) { + return { + ...edge, + data: { ...edge.data, condition: { ...edge.data.condition, variableId } }, + } + } + } + return edge + }) + + // Build toast message + const parts: string[] = [] + if (newCharacters.length > 0) { + parts.push(`${newCharacters.length} character${newCharacters.length > 1 ? 's' : ''}`) + } + if (newVariables.length > 0) { + parts.push(`${newVariables.length} variable${newVariables.length > 1 ? 's' : ''}`) + } + + return { + characters: newCharacters, + variables: newVariables, + nodes: migratedNodes, + edges: migratedEdges, + toastMessage: `Auto-imported ${parts.join(' and ')} from existing data`, + } +} + // Inner component that uses useReactFlow hook -function FlowchartEditorInner({ projectId, projectName, initialData }: FlowchartEditorProps) { +function FlowchartEditorInner({ projectId, projectName, userId, userDisplayName, isOwner, initialData, needsMigration }: FlowchartEditorProps) { // Define custom node types - memoized to prevent re-renders const nodeTypes: NodeTypes = useMemo( () => ({ @@ -449,11 +515,508 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart return { showPrompt: false, savedDraft: null } }) + // Compute migrated data once on first render using a lazy state initializer + const [migratedData] = useState(() => computeMigration(initialData, !!needsMigration)) + const [nodes, setNodes, onNodesChange] = useNodesState( - toReactFlowNodes(initialData.nodes) + toReactFlowNodes(migratedData.nodes) ) const [edges, setEdges, onEdgesChange] = useEdgesState( - toReactFlowEdges(initialData.edges) + toReactFlowEdges(migratedData.edges) + ) + + const [characters, setCharacters] = useState(migratedData.characters) + const [variables, setVariables] = useState(migratedData.variables) + + // Refs to always have the latest characters/variables for the CRDT persist callback + const charactersRef = useRef(characters) + const variablesRef = useRef(variables) + charactersRef.current = characters + variablesRef.current = variables + const [showSettings, setShowSettings] = useState(false) + const [showShare, setShowShare] = useState(false) + const [showHistory, setShowHistory] = useState(false) + const [selectedEdgeId, setSelectedEdgeId] = useState(null) + const [toastMessage, setToastMessage] = useState(migratedData.toastMessage) + const [validationIssues, setValidationIssues] = useState(null) + const [warningNodeIds, setWarningNodeIds] = useState>(new Set()) + const [connectionState, setConnectionState] = useState('disconnected') + const [presenceUsers, setPresenceUsers] = useState([]) + const [remoteCursors, setRemoteCursors] = useState([]) + const realtimeRef = useRef(null) + const crdtRef = useRef(null) + const auditRef = useRef(null) + const cursorThrottleRef = useRef(0) + const [collaborationNotifications, setCollaborationNotifications] = useState([]) + const [nodeLocks, setNodeLocks] = useState>(new Map()) + const localLockRef = useRef(null) // node ID currently locked by this user + const lockExpiryTimerRef = useRef | null>(null) + const isRevertingRef = useRef(false) // guards against double audit recording during revert + + // Initialize CRDT manager and connect to Supabase Realtime channel on mount + useEffect(() => { + const supabase = createClient() + + const crdtManager = new CRDTManager({ + onNodesChange: (crdtNodes: FlowchartNode[]) => { + setNodes(toReactFlowNodes(crdtNodes)) + }, + onEdgesChange: (crdtEdges: FlowchartEdge[]) => { + setEdges(toReactFlowEdges(crdtEdges)) + }, + onPersist: async (persistNodes: FlowchartNode[], persistEdges: FlowchartEdge[]) => { + try { + // Auto-persist saves the current state for durability. We do NOT + // broadcast state-refresh here because CRDT already syncs nodes/edges + // via yjs-update broadcasts. Broadcasting here causes ping-pong: + // other clients fetch from DB, overwrite their local variables/characters, + // then their persist writes stale data back, causing a loop. + await supabase + .from('projects') + .update({ + flowchart_data: { + nodes: persistNodes, + edges: persistEdges, + characters: charactersRef.current, + variables: variablesRef.current, + }, + }) + .eq('id', projectId) + } catch { + // Persistence failure is non-critical; will retry on next change + } + }, + }) + + // Initialize CRDT document from initial data + crdtManager.initializeFromData(migratedData.nodes, migratedData.edges) + crdtRef.current = crdtManager + + // Initialize audit trail recorder + const auditRecorder = new AuditTrailRecorder(projectId, userId) + auditRecorder.initialize(migratedData.nodes, migratedData.edges) + auditRef.current = auditRecorder + + const connection = new RealtimeConnection(projectId, userId, userDisplayName, { + onConnectionStateChange: setConnectionState, + onPresenceSync: setPresenceUsers, + onPresenceJoin: (user) => { + setCollaborationNotifications((prev) => [ + ...prev, + { id: nanoid(), displayName: user.displayName, type: 'join', color: getUserColor(user.userId) }, + ]) + // Send full CRDT state so the joining client gets caught up + crdtManager.broadcastFullState() + }, + onPresenceLeave: (user) => { + setCollaborationNotifications((prev) => [ + ...prev, + { id: nanoid(), displayName: user.displayName, type: 'leave', color: getUserColor(user.userId) }, + ]) + }, + onCursorUpdate: (cursor) => { + setRemoteCursors((prev) => { + const existing = prev.findIndex((c) => c.userId === cursor.userId) + if (existing >= 0) { + const updated = [...prev] + updated[existing] = cursor + return updated + } + return [...prev, cursor] + }) + }, + onNodeLockUpdate: (lock: NodeLock | null, lockUserId: string) => { + setNodeLocks((prev) => { + const next = new Map(prev) + if (lock) { + next.set(lock.nodeId, { ...lock, color: getUserColor(lock.userId) }) + } else { + // Remove any lock held by this user + for (const [nodeId, info] of next) { + if (info.userId === lockUserId) { + next.delete(nodeId) + } + } + } + return next + }) + }, + onCRDTUpdate: (update: number[]) => { + crdtManager.applyRemoteUpdate(update) + }, + onStateRefresh: async () => { + try { + const sb = createClient() + const { data } = await sb + .from('projects') + .select('flowchart_data') + .eq('id', projectId) + .single() + if (data?.flowchart_data) { + const fd = data.flowchart_data as FlowchartData + setNodes(toReactFlowNodes(fd.nodes)) + setEdges(toReactFlowEdges(fd.edges)) + setCharacters(fd.characters) + setVariables(fd.variables) + crdtManager.refreshFromData(fd.nodes, fd.edges) + } + } catch { + // Non-critical: user can still manually refresh + } + }, + onChannelSubscribed: (channel) => { + crdtManager.connectChannel(channel) + }, + }) + realtimeRef.current = connection + connection.connect() + + return () => { + // Release any held lock before disconnecting + if (localLockRef.current) { + connection.broadcastNodeLock(null) + localLockRef.current = null + } + connection.disconnect() + realtimeRef.current = null + crdtManager.destroy() + crdtRef.current = null + auditRecorder.destroy() + auditRef.current = null + } + // eslint-disable-next-line react-hooks/exhaustive-deps + }, [projectId, userId, userDisplayName]) + + // Manage connection lifecycle based on visibility and user activity + useEffect(() => { + const handleVisibilityChange = () => { + if (!document.hidden) { + realtimeRef.current?.notifyVisibilityResumed() + } + } + + // Throttle activity notifications to avoid excessive calls + let activityThrottled = false + const throttledActivity = () => { + if (activityThrottled) return + activityThrottled = true + realtimeRef.current?.notifyActivity() + setTimeout(() => { activityThrottled = false }, 10_000) + } + + document.addEventListener('visibilitychange', handleVisibilityChange) + document.addEventListener('mousedown', throttledActivity) + document.addEventListener('keydown', throttledActivity) + document.addEventListener('scroll', throttledActivity, true) + document.addEventListener('mousemove', throttledActivity) + + return () => { + document.removeEventListener('visibilitychange', handleVisibilityChange) + document.removeEventListener('mousedown', throttledActivity) + document.removeEventListener('keydown', throttledActivity) + document.removeEventListener('scroll', throttledActivity, true) + document.removeEventListener('mousemove', throttledActivity) + } + }, []) + + // Sync local React Flow state changes to CRDT (skip remote-originated updates) + const nodesForCRDT = useMemo(() => { + return nodes.map((node) => ({ + id: node.id, + type: node.type as 'dialogue' | 'choice' | 'variable', + position: node.position, + data: node.data, + })) as FlowchartNode[] + }, [nodes]) + + const edgesForCRDT = useMemo(() => { + return edges.map((edge) => ({ + id: edge.id, + source: edge.source, + sourceHandle: edge.sourceHandle, + target: edge.target, + targetHandle: edge.targetHandle, + data: edge.data, + })) as FlowchartEdge[] + }, [edges]) + + useEffect(() => { + crdtRef.current?.updateNodes(nodesForCRDT) + if (!isRevertingRef.current) { + auditRef.current?.recordNodeChanges(nodesForCRDT) + } + }, [nodesForCRDT]) + + useEffect(() => { + crdtRef.current?.updateEdges(edgesForCRDT) + if (!isRevertingRef.current) { + auditRef.current?.recordEdgeChanges(edgesForCRDT) + } + }, [edgesForCRDT]) + + // Broadcast cursor position on mouse move (throttled to 50ms) + const handleMouseMove = useCallback( + (event: React.MouseEvent) => { + const now = Date.now() + if (now - cursorThrottleRef.current < 50) return + cursorThrottleRef.current = now + + const connection = realtimeRef.current + if (!connection) return + + // Convert screen position to flow coordinates + const bounds = event.currentTarget.getBoundingClientRect() + const screenX = event.clientX - bounds.left + const screenY = event.clientY - bounds.top + const flowPosition = screenToFlowPosition({ x: screenX, y: screenY }) + + connection.broadcastCursor(flowPosition) + }, + [screenToFlowPosition] + ) + + // Remove cursors and locks for users who leave + useEffect(() => { + setRemoteCursors((prev) => + prev.filter((cursor) => + presenceUsers.some((u) => u.userId === cursor.userId) + ) + ) + setNodeLocks((prev) => { + const next = new Map(prev) + let changed = false + for (const [nodeId, info] of next) { + if (!presenceUsers.some((u) => u.userId === info.userId)) { + next.delete(nodeId) + changed = true + } + } + return changed ? next : prev + }) + }, [presenceUsers]) + + // Lock auto-expiry: expire locks after 60 seconds of inactivity + useEffect(() => { + const LOCK_EXPIRY_MS = 60_000 + lockExpiryTimerRef.current = setInterval(() => { + const now = Date.now() + setNodeLocks((prev) => { + const next = new Map(prev) + let changed = false + for (const [nodeId, info] of next) { + if (now - info.lockedAt > LOCK_EXPIRY_MS) { + next.delete(nodeId) + changed = true + } + } + return changed ? next : prev + }) + }, 5000) // Check every 5 seconds + + return () => { + if (lockExpiryTimerRef.current) { + clearInterval(lockExpiryTimerRef.current) + lockExpiryTimerRef.current = null + } + } + }, []) + + const handleDismissNotification = useCallback((id: string) => { + setCollaborationNotifications((prev) => prev.filter((n) => n.id !== id)) + }, []) + + const handleHistorySelectEntity = useCallback((entityId: string, actionType: string) => { + if (actionType.startsWith('node_')) { + // Select the node on the canvas + setNodes((nds) => + nds.map((n) => ({ ...n, selected: n.id === entityId })) + ) + setEdges((eds) => + eds.map((e) => ({ ...e, selected: false })) + ) + } else if (actionType.startsWith('edge_')) { + // Select the edge on the canvas + setEdges((eds) => + eds.map((e) => ({ ...e, selected: e.id === entityId })) + ) + setNodes((nds) => + nds.map((n) => ({ ...n, selected: false })) + ) + } + }, [setNodes, setEdges]) + + const handleRevertEntry = useCallback((entry: AuditEntry) => { + const { action_type, entity_id, previous_state, new_state } = entry + + // Guard against double audit recording - we write our own audit entry for the revert + isRevertingRef.current = true + // Reset after React has processed the state updates + setTimeout(() => { isRevertingRef.current = false }, 0) + + if (action_type === 'node_add') { + // Revert an addition = delete the node + setNodes((nds) => nds.filter((n) => n.id !== entity_id)) + // Also remove any edges connected to this node + setEdges((eds) => eds.filter((e) => e.source !== entity_id && e.target !== entity_id)) + } else if (action_type === 'node_update') { + // Revert an update = restore previous state + if (previous_state) { + const prevNode = previous_state as unknown as FlowchartNode + setNodes((nds) => + nds.map((n) => + n.id === entity_id + ? { ...n, position: prevNode.position, data: prevNode.data } + : n + ) + ) + } + } else if (action_type === 'node_delete') { + // Revert a deletion = re-create the node from previous_state + if (previous_state) { + const prevNode = previous_state as unknown as FlowchartNode + const newReactFlowNode: Node = { + id: prevNode.id, + type: prevNode.type, + position: prevNode.position, + data: prevNode.data, + } + setNodes((nds) => [...nds, newReactFlowNode]) + } + } else if (action_type === 'edge_add') { + // Revert an edge addition = delete the edge + setEdges((eds) => eds.filter((e) => e.id !== entity_id)) + } else if (action_type === 'edge_update') { + // Revert an edge update = restore previous state + if (previous_state) { + const prevEdge = previous_state as unknown as FlowchartEdge + setEdges((eds) => + eds.map((e) => + e.id === entity_id + ? { + ...e, + source: prevEdge.source, + sourceHandle: prevEdge.sourceHandle, + target: prevEdge.target, + targetHandle: prevEdge.targetHandle, + data: prevEdge.data, + } + : e + ) + ) + } + } else if (action_type === 'edge_delete') { + // Revert an edge deletion = re-create the edge from previous_state + if (previous_state) { + const prevEdge = previous_state as unknown as FlowchartEdge + const newReactFlowEdge: Edge = { + id: prevEdge.id, + source: prevEdge.source, + sourceHandle: prevEdge.sourceHandle, + target: prevEdge.target, + targetHandle: prevEdge.targetHandle, + data: prevEdge.data, + type: 'conditional', + markerEnd: { type: MarkerType.ArrowClosed }, + } + setEdges((eds) => [...eds, newReactFlowEdge]) + } + } + + // Record the revert as a new audit trail entry (fire-and-forget) + const supabase = createClient() + const revertActionType = getRevertActionType(action_type) + supabase + .from('audit_trail') + .insert({ + project_id: projectId, + user_id: userId, + action_type: revertActionType, + entity_id, + previous_state: new_state, // "previous" for the revert is the current state (which was new_state) + new_state: previous_state, // "new" for the revert is what we're restoring to + }) + .then(({ error }) => { + if (error) { + console.error('[Revert] Failed to write revert audit entry:', error) + } + }) + + setToast({ message: 'Change reverted successfully', type: 'success' }) + }, [setNodes, setEdges, projectId, userId]) + + const handleNodeFocus = useCallback((nodeId: string) => { + // Broadcast lock for this node + localLockRef.current = nodeId + realtimeRef.current?.broadcastNodeLock(nodeId) + }, []) + + const handleNodeBlur = useCallback(() => { + // Release lock + if (localLockRef.current) { + localLockRef.current = null + realtimeRef.current?.broadcastNodeLock(null) + } + }, []) + + const handleAddCharacter = useCallback( + (name: string, color: string): string => { + const id = nanoid() + const newCharacter: Character = { id, name, color } + setCharacters((prev) => [...prev, newCharacter]) + return id + }, + [] + ) + + const handleAddVariableDefinition = useCallback( + (name: string, type: 'numeric' | 'string' | 'boolean', initialValue: number | string | boolean): string => { + const id = nanoid() + const newVariable: Variable = { id, name, type, initialValue } + setVariables((prev) => [...prev, newVariable]) + return id + }, + [] + ) + + const editorContextValue = useMemo( + () => ({ + characters, + onAddCharacter: handleAddCharacter, + variables, + onAddVariable: handleAddVariableDefinition, + nodeLocks, + onNodeFocus: handleNodeFocus, + onNodeBlur: handleNodeBlur, + }), + [characters, handleAddCharacter, variables, handleAddVariableDefinition, nodeLocks, handleNodeFocus, handleNodeBlur] + ) + + const getCharacterUsageCount = useCallback( + (characterId: string) => { + return nodes.filter((n) => n.type === 'dialogue' && n.data?.characterId === characterId).length + }, + [nodes] + ) + + const getVariableUsageCount = useCallback( + (variableId: string) => { + const nodeCount = nodes.filter( + (n) => n.type === 'variable' && n.data?.variableId === variableId + ).length + const edgeCount = edges.filter( + (e) => e.data?.condition?.variableId === variableId + ).length + const choiceOptionCount = nodes.filter( + (n) => n.type === 'choice' + ).reduce((count, n) => { + const options = n.data?.options || [] + return count + options.filter( + (opt: { condition?: { variableId?: string } }) => opt.condition?.variableId === variableId + ).length + }, 0) + return nodeCount + edgeCount + choiceOptionCount + }, + [nodes, edges] ) // Track debounce timer @@ -474,6 +1037,8 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart const currentData: FlowchartData = { nodes: fromReactFlowNodes(nodes), edges: fromReactFlowEdges(edges), + characters, + variables, } saveDraft(projectId, currentData) }, AUTOSAVE_DEBOUNCE_MS) @@ -484,16 +1049,18 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart clearTimeout(saveTimerRef.current) } } - }, [nodes, edges, projectId, draftState.showPrompt]) + }, [nodes, edges, characters, variables, projectId, draftState.showPrompt]) // Calculate dirty state by comparing current data with last saved data const isDirty = useMemo(() => { const currentData: FlowchartData = { nodes: fromReactFlowNodes(nodes), edges: fromReactFlowEdges(edges), + characters, + variables, } return !flowchartDataEquals(currentData, lastSavedDataRef.current) - }, [nodes, edges]) + }, [nodes, edges, characters, variables]) // Browser beforeunload warning when dirty useEffect(() => { @@ -613,6 +1180,8 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart const flowchartData: FlowchartData = { nodes: fromReactFlowNodes(nodes), edges: fromReactFlowEdges(edges), + characters, + variables, } const { error } = await supabase @@ -633,6 +1202,9 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart // Update last saved data ref to mark as not dirty lastSavedDataRef.current = flowchartData + // Notify other connected clients to refresh from the database + realtimeRef.current?.broadcastStateRefresh() + setToast({ message: 'Project saved successfully', type: 'success' }) } catch (error) { console.error('Failed to save project:', error) @@ -644,7 +1216,7 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart } finally { setIsSaving(false) } - }, [isSaving, nodes, edges, projectId]) + }, [isSaving, nodes, edges, characters, variables, projectId]) // Keep ref updated with latest handleSave handleSaveRef.current = handleSave @@ -654,6 +1226,8 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart const flowchartData: FlowchartData = { nodes: fromReactFlowNodes(nodes), edges: fromReactFlowEdges(edges), + characters, + variables, } // Create pretty-printed JSON @@ -675,7 +1249,7 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart // Cleanup document.body.removeChild(link) URL.revokeObjectURL(url) - }, [nodes, edges, projectName]) + }, [nodes, edges, characters, variables, projectName]) const handleExportRenpy = useCallback(() => { // Convert React Flow state to our flowchart types @@ -716,14 +1290,27 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart setToast({ message: 'Exported to Ren\'Py format successfully', type: 'success' }) }, [nodes, edges, projectName]) + const handleExportAnyway = useCallback(() => { + setValidationIssues(null) + setWarningNodeIds(new Set()) + handleExportRenpy() + }, [handleExportRenpy]) + + const handleExportCancel = useCallback(() => { + setValidationIssues(null) + setWarningNodeIds(new Set()) + }, []) + // Check if current flowchart has unsaved changes const hasUnsavedChanges = useCallback(() => { const currentData: FlowchartData = { nodes: fromReactFlowNodes(nodes), edges: fromReactFlowEdges(edges), + characters, + variables, } return !flowchartDataEquals(currentData, initialData) - }, [nodes, edges, initialData]) + }, [nodes, edges, characters, variables, initialData]) // Load imported data into React Flow const loadImportedData = useCallback( @@ -809,12 +1396,30 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart console.log('Deleted edges:', deletedEdges.map((e) => e.id)) }, []) + // Handle edge click to open condition editor + const onEdgeClick = useCallback((_event: React.MouseEvent, edge: Edge) => { + setSelectedEdgeId(edge.id) + }, []) + + // Handle condition change from ConditionEditor + const handleConditionChange = useCallback( + (edgeId: string, condition: Condition | undefined) => { + setEdges((eds) => + eds.map((edge) => + edge.id === edgeId + ? { ...edge, data: condition ? { condition } : undefined } + : edge + ) + ) + }, + [setEdges] + ) + // Context menu handlers const closeContextMenu = useCallback(() => { setContextMenu(null) }, []) - // Handle right-click on canvas (pane) const onPaneContextMenu = useCallback( (event: React.MouseEvent) => { event.preventDefault() @@ -827,7 +1432,6 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart [] ) - // Handle right-click on node const onNodeContextMenu: NodeMouseHandler = useCallback( (event, node) => { event.preventDefault() @@ -841,7 +1445,6 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart [] ) - // Handle right-click on edge const onEdgeContextMenu: EdgeMouseHandler = useCallback( (event, edge) => { event.preventDefault() @@ -855,19 +1458,15 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart [] ) - // Add node at specific position (for context menu) const handleAddNodeAtPosition = useCallback( (type: 'dialogue' | 'choice' | 'variable') => { if (!contextMenu) return - - // Convert screen position to flow position const position = screenToFlowPosition({ x: contextMenu.x, y: contextMenu.y, }) let newNode: Node - if (type === 'dialogue') { newNode = { id: nanoid(), @@ -902,23 +1501,23 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart } setNodes((nodes) => [...nodes, newNode]) + setContextMenu(null) }, [contextMenu, screenToFlowPosition, setNodes] ) - // Delete selected node from context menu const handleDeleteNode = useCallback(() => { if (!contextMenu?.nodeId) return setNodes((nodes) => nodes.filter((n) => n.id !== contextMenu.nodeId)) + setContextMenu(null) }, [contextMenu, setNodes]) - // Delete selected edge from context menu const handleDeleteEdge = useCallback(() => { if (!contextMenu?.edgeId) return setEdges((edges) => edges.filter((e) => e.id !== contextMenu.edgeId)) + setContextMenu(null) }, [contextMenu, setEdges]) - // Open condition editor for an edge const openConditionEditor = useCallback( (edgeId: string) => { const edge = edges.find((e) => e.id === edgeId) @@ -931,13 +1530,12 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart [edges] ) - // Add condition to edge (opens ConditionEditor modal) const handleAddCondition = useCallback(() => { if (!contextMenu?.edgeId) return openConditionEditor(contextMenu.edgeId) + setContextMenu(null) }, [contextMenu, openConditionEditor]) - // Handle double-click on edge to open condition editor const onEdgeDoubleClick = useCallback( (_event: React.MouseEvent, edge: Edge) => { openConditionEditor(edge.id) @@ -945,264 +1543,143 @@ function FlowchartEditorInner({ projectId, projectName, initialData }: Flowchart [openConditionEditor] ) - // Save condition to edge - const handleSaveCondition = useCallback( - (edgeId: string, condition: Condition) => { - setEdges((eds) => - eds.map((edge) => - edge.id === edgeId - ? { ...edge, data: { ...edge.data, condition } } - : edge - ) - ) - setConditionEditor(null) - }, - [setEdges] + // Apply warning styles to nodes with undefined references + const styledNodes = useMemo( + () => + warningNodeIds.size === 0 + ? nodes + : nodes.map((node) => + warningNodeIds.has(node.id) + ? { ...node, className: 'export-warning-node' } + : node + ), + [nodes, warningNodeIds] ) - // Remove condition from edge - const handleRemoveCondition = useCallback( - (edgeId: string) => { - setEdges((eds) => - eds.map((edge) => { - if (edge.id !== edgeId) return edge - // Remove condition from data - const newData = { ...edge.data } - delete newData.condition - return { ...edge, data: newData } - }) - ) - setConditionEditor(null) - }, - [setEdges] + // Get the selected edge's condition data + const selectedEdge = useMemo( + () => (selectedEdgeId ? edges.find((e) => e.id === selectedEdgeId) : null), + [selectedEdgeId, edges] ) - // Close condition editor - const closeConditionEditor = useCallback(() => { - setConditionEditor(null) - }, []) - - // Router for navigation - const router = useRouter() - - // Handle back button click - show warning if dirty - const handleBackClick = useCallback(() => { - if (isDirty) { - setShowNavigationWarning(true) - } else { - router.push('/dashboard') - } - }, [isDirty, router]) - - // Confirm navigation (discard unsaved changes) - const handleConfirmNavigation = useCallback(() => { - setShowNavigationWarning(false) - router.push('/dashboard') - }, [router]) - - // Cancel navigation - const handleCancelNavigation = useCallback(() => { - setShowNavigationWarning(false) - }, []) - return ( -
- {/* Editor header with back button and project name */} -
-
- -

- {projectName} -

- {isDirty && ( - - (unsaved changes) - + + + + + {showHistory && ( + setShowHistory(false)} + onSelectEntity={handleHistorySelectEntity} + onRevert={handleRevertEntry} + /> )}
-
- - -
- - - - + {contextMenu && ( + handleAddNodeAtPosition('dialogue')} + onAddChoice={() => handleAddNodeAtPosition('choice')} + onAddVariable={() => handleAddNodeAtPosition('variable')} + onDelete={ + contextMenu.type === 'node' ? handleDeleteNode : handleDeleteEdge + } + onAddCondition={handleAddCondition} + /> + )} + {showSettings && ( + setShowSettings(false)} + getCharacterUsageCount={getCharacterUsageCount} + getVariableUsageCount={getVariableUsageCount} + /> + )} + {showShare && ( + setShowShare(false)} + /> + )} + {selectedEdge && ( + setSelectedEdgeId(null)} + /> + )} + {validationIssues && ( + + )} + {toastMessage && ( + setToastMessage(null)} + /> + )} + {collaborationNotifications.length > 0 && ( +
+ {collaborationNotifications.map((notification) => ( + + ))} +
+ )}
- - {contextMenu && ( - handleAddNodeAtPosition('dialogue')} - onAddChoice={() => handleAddNodeAtPosition('choice')} - onAddVariable={() => handleAddNodeAtPosition('variable')} - onDelete={ - contextMenu.type === 'node' ? handleDeleteNode : handleDeleteEdge - } - onAddCondition={handleAddCondition} - /> - )} - - {conditionEditor && ( - - )} - - {/* Draft restoration prompt */} - {draftState.showPrompt && ( -
-
-

- Unsaved Draft Found -

-

- A local draft was found that differs from the saved version. Would - you like to restore it or discard it? -

-
- - -
-
-
- )} - - {/* Import confirmation dialog */} - {importConfirmDialog && ( -
-
-

- Unsaved Changes -

-

- You have unsaved changes. Importing a new file will discard your - current work. Are you sure you want to continue? -

-
- - -
-
-
- )} - - {/* Navigation warning dialog */} - {showNavigationWarning && ( -
-
-

- Unsaved Changes -

-

- You have unsaved changes that will be lost if you leave this page. - Are you sure you want to leave? -

-
- - -
-
-
- )} - - {/* Hidden file input for import */} - - - {/* Toast notification */} - {toast && ( - setToast(null)} - action={toast.action} - /> - )} -
+ ) } diff --git a/src/app/editor/[projectId]/actions.ts b/src/app/editor/[projectId]/actions.ts new file mode 100644 index 0000000..6828323 --- /dev/null +++ b/src/app/editor/[projectId]/actions.ts @@ -0,0 +1,227 @@ +'use server' + +import { createClient } from '@/lib/supabase/server' + +export type Collaborator = { + id: string + user_id: string + role: 'owner' | 'editor' | 'viewer' + invited_at: string + accepted_at: string | null + display_name: string | null + email: string | null +} + +export async function getCollaborators( + projectId: string +): Promise<{ success: boolean; data?: Collaborator[]; error?: string }> { + const supabase = await createClient() + const { + data: { user }, + } = await supabase.auth.getUser() + + if (!user) { + return { success: false, error: 'Not authenticated' } + } + + // Verify user owns the project + const { data: project } = await supabase + .from('projects') + .select('id, user_id') + .eq('id', projectId) + .single() + + if (!project) { + return { success: false, error: 'Project not found' } + } + + const isOwner = project.user_id === user.id + + // Check if user is a collaborator + if (!isOwner) { + const { data: collab } = await supabase + .from('project_collaborators') + .select('id') + .eq('project_id', projectId) + .eq('user_id', user.id) + .single() + + if (!collab) { + return { success: false, error: 'Access denied' } + } + } + + // Fetch collaborators with profile info + const { data: collaborators, error } = await supabase + .from('project_collaborators') + .select('id, user_id, role, invited_at, accepted_at, profiles(display_name, email)') + .eq('project_id', projectId) + + if (error) { + return { success: false, error: error.message } + } + + const result: Collaborator[] = (collaborators || []).map((c) => { + const profile = c.profiles as unknown as { display_name: string | null; email: string | null } | null + return { + id: c.id, + user_id: c.user_id, + role: c.role as 'owner' | 'editor' | 'viewer', + invited_at: c.invited_at, + accepted_at: c.accepted_at, + display_name: profile?.display_name || null, + email: profile?.email || null, + } + }) + + return { success: true, data: result } +} + +export async function inviteCollaborator( + projectId: string, + email: string, + role: 'editor' | 'viewer' +): Promise<{ success: boolean; error?: string }> { + const supabase = await createClient() + const { + data: { user }, + } = await supabase.auth.getUser() + + if (!user) { + return { success: false, error: 'Not authenticated' } + } + + // Verify user owns the project + const { data: project } = await supabase + .from('projects') + .select('id, user_id') + .eq('id', projectId) + .eq('user_id', user.id) + .single() + + if (!project) { + return { success: false, error: 'Only the project owner can invite collaborators' } + } + + // Find the user by email in profiles + const { data: targetProfile } = await supabase + .from('profiles') + .select('id, email') + .eq('email', email) + .single() + + if (!targetProfile) { + return { success: false, error: 'No user found with that email address' } + } + + // Cannot invite yourself + if (targetProfile.id === user.id) { + return { success: false, error: 'You cannot invite yourself' } + } + + // Check if already a collaborator + const { data: existing } = await supabase + .from('project_collaborators') + .select('id') + .eq('project_id', projectId) + .eq('user_id', targetProfile.id) + .single() + + if (existing) { + return { success: false, error: 'This user is already a collaborator' } + } + + // Insert collaborator + const { error: insertError } = await supabase + .from('project_collaborators') + .insert({ + project_id: projectId, + user_id: targetProfile.id, + role, + invited_at: new Date().toISOString(), + accepted_at: new Date().toISOString(), // Auto-accept for now + }) + + if (insertError) { + return { success: false, error: insertError.message } + } + + return { success: true } +} + +export async function updateCollaboratorRole( + projectId: string, + collaboratorId: string, + role: 'editor' | 'viewer' +): Promise<{ success: boolean; error?: string }> { + const supabase = await createClient() + const { + data: { user }, + } = await supabase.auth.getUser() + + if (!user) { + return { success: false, error: 'Not authenticated' } + } + + // Verify user owns the project + const { data: project } = await supabase + .from('projects') + .select('id, user_id') + .eq('id', projectId) + .eq('user_id', user.id) + .single() + + if (!project) { + return { success: false, error: 'Only the project owner can change roles' } + } + + const { error } = await supabase + .from('project_collaborators') + .update({ role }) + .eq('id', collaboratorId) + .eq('project_id', projectId) + + if (error) { + return { success: false, error: error.message } + } + + return { success: true } +} + +export async function removeCollaborator( + projectId: string, + collaboratorId: string +): Promise<{ success: boolean; error?: string }> { + const supabase = await createClient() + const { + data: { user }, + } = await supabase.auth.getUser() + + if (!user) { + return { success: false, error: 'Not authenticated' } + } + + // Verify user owns the project + const { data: project } = await supabase + .from('projects') + .select('id, user_id') + .eq('id', projectId) + .eq('user_id', user.id) + .single() + + if (!project) { + return { success: false, error: 'Only the project owner can remove collaborators' } + } + + const { error } = await supabase + .from('project_collaborators') + .delete() + .eq('id', collaboratorId) + .eq('project_id', projectId) + + if (error) { + return { success: false, error: error.message } + } + + return { success: true } +} diff --git a/src/app/editor/[projectId]/page.tsx b/src/app/editor/[projectId]/page.tsx index 59ed19d..7573003 100644 --- a/src/app/editor/[projectId]/page.tsx +++ b/src/app/editor/[projectId]/page.tsx @@ -19,35 +19,71 @@ export default async function EditorPage({ params }: PageProps) { return null } - const { data: project, error } = await supabase + // Fetch user's display name for presence + const { data: profile } = await supabase + .from('profiles') + .select('display_name') + .eq('id', user.id) + .single() + + const userDisplayName = profile?.display_name || user.email || 'Anonymous' + + // Try to load as owner first + const { data: ownedProject } = await supabase .from('projects') .select('id, name, flowchart_data') .eq('id', projectId) .eq('user_id', user.id) .single() - if (error || !project) { + let project = ownedProject + let isOwner = true + + // If not the owner, check if the user is a collaborator + if (!project) { + const { data: collab } = await supabase + .from('project_collaborators') + .select('id, role') + .eq('project_id', projectId) + .eq('user_id', user.id) + .single() + + if (collab) { + const { data: sharedProject } = await supabase + .from('projects') + .select('id, name, flowchart_data') + .eq('id', projectId) + .single() + + project = sharedProject + isOwner = false + } + } + + if (!project) { return (
-
- - +
+ - - - + + + + +
@@ -82,16 +118,27 @@ export default async function EditorPage({ params }: PageProps) { ) } - const flowchartData = (project.flowchart_data || { - nodes: [], - edges: [], - }) as FlowchartData + const rawData = project.flowchart_data || {} + const flowchartData: FlowchartData = { + nodes: rawData.nodes || [], + edges: rawData.edges || [], + characters: rawData.characters || [], + variables: rawData.variables || [], + } + + // Migration flag: if the raw data doesn't have characters/variables arrays, + // the project was created before these features existed and may need auto-migration + const needsMigration = !rawData.characters && !rawData.variables return ( ) } diff --git a/src/app/globals.css b/src/app/globals.css index a2dc41e..f4d613f 100644 --- a/src/app/globals.css +++ b/src/app/globals.css @@ -24,3 +24,16 @@ body { color: var(--foreground); font-family: Arial, Helvetica, sans-serif; } + +/* Export validation warning highlighting for React Flow nodes */ +.react-flow__node.export-warning-node { + outline: 3px solid #f97316; + outline-offset: 2px; + border-radius: 6px; + animation: pulse-warning 1.5s ease-in-out infinite; +} + +@keyframes pulse-warning { + 0%, 100% { outline-color: #f97316; } + 50% { outline-color: #fb923c; } +} diff --git a/src/components/ProjectCard.tsx b/src/components/ProjectCard.tsx index cd47ec0..707aad2 100644 --- a/src/components/ProjectCard.tsx +++ b/src/components/ProjectCard.tsx @@ -8,8 +8,10 @@ interface ProjectCardProps { id: string name: string updatedAt: string - onDelete: (id: string) => void - onRename: (id: string, newName: string) => void + onDelete?: (id: string) => void + onRename?: (id: string, newName: string) => void + shared?: boolean + sharedRole?: string } function formatDate(dateString: string): string { @@ -29,6 +31,8 @@ export default function ProjectCard({ updatedAt, onDelete, onRename, + shared, + sharedRole, }: ProjectCardProps) { const [showDeleteDialog, setShowDeleteDialog] = useState(false) const [isDeleting, setIsDeleting] = useState(false) @@ -62,7 +66,7 @@ export default function ProjectCard({ setIsDeleting(false) setShowDeleteDialog(false) - onDelete(id) + onDelete?.(id) } const handleCancelDelete = () => { @@ -106,7 +110,7 @@ export default function ProjectCard({ setIsRenaming(false) setShowRenameDialog(false) - onRename(id, newName.trim()) + onRename?.(id, newName.trim()) } const handleCancelRename = () => { @@ -122,46 +126,55 @@ export default function ProjectCard({ onClick={handleCardClick} className="group relative cursor-pointer rounded-lg border border-zinc-200 bg-white p-6 transition-all hover:border-blue-300 hover:shadow-md dark:border-zinc-700 dark:bg-zinc-800 dark:hover:border-blue-600" > -
- - + -
+ + + + +
+ )} + {shared && ( +
+ + {sharedRole === 'editor' ? 'Editor' : 'Viewer'} + +
+ )}

{name}

diff --git a/src/components/editor/ActivityHistorySidebar.tsx b/src/components/editor/ActivityHistorySidebar.tsx new file mode 100644 index 0000000..3649428 --- /dev/null +++ b/src/components/editor/ActivityHistorySidebar.tsx @@ -0,0 +1,357 @@ +'use client' + +import { useCallback, useEffect, useRef, useState } from 'react' +import { createClient } from '@/lib/supabase/client' +import RevertConfirmDialog from './RevertConfirmDialog' + +const PAGE_SIZE = 20 + +export type AuditEntry = { + id: string + project_id: string + user_id: string + action_type: string + entity_id: string + previous_state: Record | null + new_state: Record | null + created_at: string + user_display_name?: string +} + +type ActivityHistorySidebarProps = { + projectId: string + onClose: () => void + onSelectEntity: (entityId: string, actionType: string) => void + onRevert: (entry: AuditEntry) => void +} + +const ACTION_LABELS: Record = { + node_add: 'Added node', + node_update: 'Updated node', + node_delete: 'Deleted node', + edge_add: 'Added edge', + edge_update: 'Updated edge', + edge_delete: 'Deleted edge', +} + +const ACTION_ICONS: Record = { + node_add: '+', + node_update: '~', + node_delete: '-', + edge_add: '+', + edge_update: '~', + edge_delete: '-', +} + +const ACTION_COLORS: Record = { + node_add: 'text-green-500', + node_update: 'text-blue-500', + node_delete: 'text-red-500', + edge_add: 'text-green-500', + edge_update: 'text-blue-500', + edge_delete: 'text-red-500', +} + +// Same hash logic as PresenceAvatars and FlowchartEditor +const PRESENCE_COLORS = [ + '#EF4444', '#F97316', '#F59E0B', '#10B981', + '#3B82F6', '#8B5CF6', '#EC4899', '#14B8A6', + '#6366F1', '#F43F5E', '#84CC16', '#06B6D4', +] + +function getUserColor(userId: string): string { + let hash = 0 + for (let i = 0; i < userId.length; i++) { + hash = ((hash << 5) - hash + userId.charCodeAt(i)) | 0 + } + return PRESENCE_COLORS[Math.abs(hash) % PRESENCE_COLORS.length] +} + +function getEntityDescription(entry: AuditEntry): string { + const state = entry.new_state || entry.previous_state + if (!state) return entry.entity_id.slice(0, 8) + + if (entry.action_type.startsWith('node_')) { + const type = (state as Record).type as string | undefined + const data = (state as Record).data as Record | undefined + if (type === 'dialogue' && data) { + const speaker = (data.speaker as string) || (data.characterId as string) || '' + return speaker ? `Dialogue (${speaker})` : 'Dialogue node' + } + if (type === 'choice' && data) { + const question = (data.question as string) || '' + return question ? `Choice: "${question.slice(0, 20)}${question.length > 20 ? '…' : ''}"` : 'Choice node' + } + if (type === 'variable' && data) { + const name = (data.variableName as string) || '' + return name ? `Variable: ${name}` : 'Variable node' + } + return `${type || 'Unknown'} node` + } + + // Edge entries + return 'Connection' +} + +function formatTime(dateStr: string): string { + const date = new Date(dateStr) + return date.toLocaleTimeString([], { hour: '2-digit', minute: '2-digit' }) +} + +function getTimePeriod(dateStr: string): 'Today' | 'Yesterday' | 'Earlier' { + const date = new Date(dateStr) + const now = new Date() + const today = new Date(now.getFullYear(), now.getMonth(), now.getDate()) + const yesterday = new Date(today) + yesterday.setDate(yesterday.getDate() - 1) + + if (date >= today) return 'Today' + if (date >= yesterday) return 'Yesterday' + return 'Earlier' +} + +type GroupedEntries = { + period: 'Today' | 'Yesterday' | 'Earlier' + entries: AuditEntry[] +} + +function groupByPeriod(entries: AuditEntry[]): GroupedEntries[] { + const groups: Record = {} + const order: ('Today' | 'Yesterday' | 'Earlier')[] = ['Today', 'Yesterday', 'Earlier'] + + for (const entry of entries) { + const period = getTimePeriod(entry.created_at) + if (!groups[period]) groups[period] = [] + groups[period].push(entry) + } + + return order + .filter((p) => groups[p] && groups[p].length > 0) + .map((p) => ({ period: p, entries: groups[p] })) +} + +export default function ActivityHistorySidebar({ + projectId, + onClose, + onSelectEntity, + onRevert, +}: ActivityHistorySidebarProps) { + const [entries, setEntries] = useState([]) + const [loading, setLoading] = useState(true) + const [loadingMore, setLoadingMore] = useState(false) + const [hasMore, setHasMore] = useState(true) + const [error, setError] = useState(null) + const [revertEntry, setRevertEntry] = useState(null) + const mountedRef = useRef(true) + + const fetchEntriesRaw = useCallback(async (offset: number): Promise<{ entries: AuditEntry[]; hasMore: boolean; error?: string }> => { + const supabase = createClient() + const { data, error: fetchError } = await supabase + .from('audit_trail') + .select('id, project_id, user_id, action_type, entity_id, previous_state, new_state, created_at') + .eq('project_id', projectId) + .order('created_at', { ascending: false }) + .range(offset, offset + PAGE_SIZE - 1) + + if (fetchError) { + return { entries: [], hasMore: false, error: fetchError.message } + } + + const moreAvailable = !!data && data.length >= PAGE_SIZE + + // Fetch user display names for the entries + if (data && data.length > 0) { + const userIds = [...new Set(data.map((e) => e.user_id))] + const { data: profiles } = await supabase + .from('profiles') + .select('id, display_name') + .in('id', userIds) + + const nameMap = new Map() + if (profiles) { + for (const p of profiles) { + nameMap.set(p.id, p.display_name || 'Unknown') + } + } + + return { + entries: data.map((e) => ({ + ...e, + user_display_name: nameMap.get(e.user_id) || 'Unknown', + })), + hasMore: moreAvailable, + } + } + + return { entries: data || [], hasMore: moreAvailable } + }, [projectId]) + + useEffect(() => { + mountedRef.current = true + const controller = new AbortController() + fetchEntriesRaw(0).then((result) => { + if (!controller.signal.aborted && mountedRef.current) { + if (result.error) { + setError(result.error) + } else { + setEntries(result.entries) + setHasMore(result.hasMore) + } + setLoading(false) + } + }) + return () => { + controller.abort() + mountedRef.current = false + } + }, [fetchEntriesRaw]) + + const handleLoadMore = async () => { + setLoadingMore(true) + const result = await fetchEntriesRaw(entries.length) + if (result.error) { + setError(result.error) + } else { + setEntries((prev) => [...prev, ...result.entries]) + setHasMore(result.hasMore) + } + setLoadingMore(false) + } + + const grouped = groupByPeriod(entries) + + return ( +
+
+

+ Activity History +

+ +
+ +
+ {loading && ( +
+ + + + +
+ )} + + {error && ( +
+ Failed to load history: {error} +
+ )} + + {!loading && !error && entries.length === 0 && ( +
+ No activity recorded yet +
+ )} + + {!loading && grouped.map((group) => ( +
+
+
+ {group.period} +
+
+
+ {group.entries.map((entry) => { + const userColor = getUserColor(entry.user_id) + const isDeleted = entry.action_type.endsWith('_delete') + return ( +
+
+
+
+
+ + {ACTION_ICONS[entry.action_type] || '?'} + + +
+
+ {getEntityDescription(entry)} +
+
+
+ {entry.user_display_name} + · + {formatTime(entry.created_at)} +
+ +
+
+
+
+ ) + })} +
+
+ ))} + + {!loading && hasMore && entries.length > 0 && ( +
+ +
+ )} +
+ {revertEntry && ( + { + onRevert(revertEntry) + setRevertEntry(null) + }} + onCancel={() => setRevertEntry(null)} + /> + )} +
+ ) +} diff --git a/src/components/editor/CollaborationToast.tsx b/src/components/editor/CollaborationToast.tsx new file mode 100644 index 0000000..3885d9c --- /dev/null +++ b/src/components/editor/CollaborationToast.tsx @@ -0,0 +1,43 @@ +'use client' + +import { useEffect } from 'react' + +export type CollaborationNotification = { + id: string + displayName: string + type: 'join' | 'leave' + color: string +} + +type CollaborationToastProps = { + notification: CollaborationNotification + onDismiss: (id: string) => void +} + +export default function CollaborationToast({ notification, onDismiss }: CollaborationToastProps) { + useEffect(() => { + const timer = setTimeout(() => { + onDismiss(notification.id) + }, 3000) + + return () => clearTimeout(timer) + }, [notification.id, onDismiss]) + + const message = notification.type === 'join' + ? `${notification.displayName} joined` + : `${notification.displayName} left` + + return ( +
+
+
+ {message} +
+
+ ) +} diff --git a/src/components/editor/Combobox.tsx b/src/components/editor/Combobox.tsx new file mode 100644 index 0000000..b84b7bb --- /dev/null +++ b/src/components/editor/Combobox.tsx @@ -0,0 +1,263 @@ +'use client' + +import { useState, useRef, useEffect, useCallback, useMemo } from 'react' + +export type ComboboxItem = { + id: string + label: string + color?: string + badge?: string +} + +type ComboboxProps = { + items: ComboboxItem[] + value: string | undefined + onChange: (id: string) => void + placeholder?: string + onAddNew?: () => void +} + +export default function Combobox({ + items, + value, + onChange, + placeholder = 'Select...', + onAddNew, +}: ComboboxProps) { + const [isOpen, setIsOpen] = useState(false) + const [search, setSearch] = useState('') + const [highlightedIndex, setHighlightedIndex] = useState(0) + const [dropdownPosition, setDropdownPosition] = useState<'below' | 'above'>('below') + + const containerRef = useRef(null) + const inputRef = useRef(null) + const listRef = useRef(null) + + const selectedItem = useMemo( + () => items.find((item) => item.id === value), + [items, value] + ) + + const filteredItems = useMemo( + () => + items.filter((item) => + item.label.toLowerCase().includes(search.toLowerCase()) + ), + [items, search] + ) + + const totalOptions = filteredItems.length + (onAddNew ? 1 : 0) + + const updateDropdownPosition = useCallback(() => { + if (!containerRef.current) return + const rect = containerRef.current.getBoundingClientRect() + const spaceBelow = window.innerHeight - rect.bottom + const spaceAbove = rect.top + setDropdownPosition(spaceBelow < 200 && spaceAbove > spaceBelow ? 'above' : 'below') + }, []) + + const open = useCallback(() => { + setIsOpen(true) + setSearch('') + setHighlightedIndex(0) + updateDropdownPosition() + }, [updateDropdownPosition]) + + const close = useCallback(() => { + setIsOpen(false) + setSearch('') + }, []) + + const selectItem = useCallback( + (id: string) => { + onChange(id) + close() + }, + [onChange, close] + ) + + // Close on outside click + useEffect(() => { + if (!isOpen) return + + const handleClickOutside = (e: MouseEvent) => { + if (containerRef.current && !containerRef.current.contains(e.target as Node)) { + close() + } + } + + document.addEventListener('mousedown', handleClickOutside) + return () => document.removeEventListener('mousedown', handleClickOutside) + }, [isOpen, close]) + + // Scroll highlighted item into view + useEffect(() => { + if (!isOpen || !listRef.current) return + const items = listRef.current.querySelectorAll('[data-combobox-item]') + const highlighted = items[highlightedIndex] + if (highlighted) { + highlighted.scrollIntoView({ block: 'nearest' }) + } + }, [highlightedIndex, isOpen]) + + const handleKeyDown = useCallback( + (e: React.KeyboardEvent) => { + if (!isOpen) { + if (e.key === 'ArrowDown' || e.key === 'ArrowUp' || e.key === 'Enter') { + e.preventDefault() + open() + } + return + } + + switch (e.key) { + case 'ArrowDown': + e.preventDefault() + setHighlightedIndex((prev) => (prev + 1) % totalOptions) + break + case 'ArrowUp': + e.preventDefault() + setHighlightedIndex((prev) => (prev - 1 + totalOptions) % totalOptions) + break + case 'Enter': + e.preventDefault() + if (highlightedIndex < filteredItems.length) { + selectItem(filteredItems[highlightedIndex].id) + } else if (onAddNew && highlightedIndex === filteredItems.length) { + onAddNew() + close() + } + break + case 'Escape': + e.preventDefault() + close() + break + } + }, + [isOpen, open, close, highlightedIndex, filteredItems, totalOptions, selectItem, onAddNew] + ) + + return ( +
+
{ + if (isOpen) { + close() + } else { + open() + setTimeout(() => inputRef.current?.focus(), 0) + } + }} + > + {isOpen ? ( + { + setSearch(e.target.value) + setHighlightedIndex(0) + }} + onKeyDown={handleKeyDown} + placeholder={placeholder} + className="w-full bg-transparent outline-none placeholder-zinc-400 dark:placeholder-zinc-500" + onClick={(e) => e.stopPropagation()} + /> + ) : ( + + {selectedItem ? ( + + {selectedItem.color && ( + + )} + {selectedItem.badge && ( + + {selectedItem.badge} + + )} + {selectedItem.label} + + ) : ( + placeholder + )} + + )} + + + +
+ + {isOpen && ( +
    + {filteredItems.length === 0 && !onAddNew && ( +
  • + No results found +
    • + )} + + {filteredItems.map((item, index) => ( +
  • selectItem(item.id)} + onMouseEnter={() => setHighlightedIndex(index)} + > + {item.color && ( + + )} + {item.badge && ( + + {item.badge} + + )} + {item.label} +
    • + ))} + + {onAddNew && ( +
  • { + onAddNew() + close() + }} + onMouseEnter={() => setHighlightedIndex(filteredItems.length)} + > + + + + Add new... +
    • + )} +
+ )} +
+ ) +} diff --git a/src/components/editor/ConditionEditor.tsx b/src/components/editor/ConditionEditor.tsx index 045af92..b7259f2 100644 --- a/src/components/editor/ConditionEditor.tsx +++ b/src/components/editor/ConditionEditor.tsx @@ -1,164 +1,320 @@ 'use client' -import { useState, useCallback, useEffect } from 'react' +import { useCallback, useMemo, useState } from 'react' +import Combobox from '@/components/editor/Combobox' +import type { ComboboxItem } from '@/components/editor/Combobox' +import { useEditorContext } from '@/components/editor/EditorContext' import type { Condition } from '@/types/flowchart' type ConditionEditorProps = { edgeId: string - condition?: Condition - onSave: (edgeId: string, condition: Condition) => void - onRemove: (edgeId: string) => void - onCancel: () => void + condition: Condition | undefined + onChange: (edgeId: string, condition: Condition | undefined) => void + onClose: () => void } -const OPERATORS: Condition['operator'][] = ['>', '<', '==', '>=', '<=', '!='] - export default function ConditionEditor({ edgeId, condition, - onSave, - onRemove, - onCancel, + onChange, + onClose, }: ConditionEditorProps) { - const [variableName, setVariableName] = useState(condition?.variableName ?? '') - const [operator, setOperator] = useState(condition?.operator ?? '==') - const [value, setValue] = useState(condition?.value ?? 0) + const { variables, onAddVariable } = useEditorContext() - // Close on Escape key - useEffect(() => { - const handleKeyDown = (e: KeyboardEvent) => { - if (e.key === 'Escape') { - onCancel() - } + const [showAddForm, setShowAddForm] = useState(false) + const [newName, setNewName] = useState('') + const [newType, setNewType] = useState<'numeric' | 'string' | 'boolean'>('numeric') + + const variableItems: ComboboxItem[] = useMemo( + () => + variables.map((v) => ({ + id: v.id, + label: v.name, + badge: v.type, + })), + [variables] + ) + + const selectedVariable = useMemo(() => { + if (!condition?.variableId) return undefined + return variables.find((v) => v.id === condition.variableId) + }, [condition?.variableId, variables]) + + const hasInvalidReference = useMemo(() => { + if (!condition?.variableId) return false + return !variables.some((v) => v.id === condition.variableId) + }, [condition?.variableId, variables]) + + // Determine operators based on variable type + const availableOperators = useMemo(() => { + if (!selectedVariable || selectedVariable.type === 'numeric') { + return [ + { value: '==', label: '==' }, + { value: '!=', label: '!=' }, + { value: '>', label: '>' }, + { value: '<', label: '<' }, + { value: '>=', label: '>=' }, + { value: '<=', label: '<=' }, + ] as const } - document.addEventListener('keydown', handleKeyDown) - return () => document.removeEventListener('keydown', handleKeyDown) - }, [onCancel]) + // string and boolean only support == and != + return [ + { value: '==', label: '==' }, + { value: '!=', label: '!=' }, + ] as const + }, [selectedVariable]) - const handleSave = useCallback(() => { - if (!variableName.trim()) return - onSave(edgeId, { - variableName: variableName.trim(), - operator, - value, + const handleVariableSelect = useCallback( + (variableId: string) => { + const variable = variables.find((v) => v.id === variableId) + const defaultValue = variable + ? variable.type === 'numeric' + ? 0 + : variable.type === 'boolean' + ? false + : '' + : 0 + // Reset operator if current one is not valid for new type + const validOperator = + variable && variable.type !== 'numeric' && condition?.operator && !['==', '!='].includes(condition.operator) + ? '==' + : condition?.operator || '==' + + onChange(edgeId, { + variableName: variable?.name || '', + variableId, + operator: validOperator as Condition['operator'], + value: defaultValue, + }) + }, + [variables, condition?.operator, edgeId, onChange] + ) + + const handleOperatorChange = useCallback( + (operator: string) => { + if (!condition) return + onChange(edgeId, { + ...condition, + operator: operator as Condition['operator'], + }) + }, + [condition, edgeId, onChange] + ) + + const handleValueChange = useCallback( + (value: number | string | boolean) => { + if (!condition) return + onChange(edgeId, { + ...condition, + value, + }) + }, + [condition, edgeId, onChange] + ) + + const handleRemoveCondition = useCallback(() => { + onChange(edgeId, undefined) + onClose() + }, [edgeId, onChange, onClose]) + + const handleAddNew = useCallback(() => { + setShowAddForm(true) + setNewName('') + setNewType('numeric') + }, []) + + const handleSubmitNew = useCallback(() => { + if (!newName.trim()) return + const defaultValue = newType === 'numeric' ? 0 : newType === 'boolean' ? false : '' + const newId = onAddVariable(newName.trim(), newType, defaultValue) + onChange(edgeId, { + variableName: newName.trim(), + variableId: newId, + operator: '==', + value: defaultValue, }) - }, [edgeId, variableName, operator, value, onSave]) + setShowAddForm(false) + }, [newName, newType, onAddVariable, edgeId, onChange]) - const handleRemove = useCallback(() => { - onRemove(edgeId) - }, [edgeId, onRemove]) + const handleCancelNew = useCallback(() => { + setShowAddForm(false) + }, []) - const hasExistingCondition = !!condition + // Render value input based on variable type + const renderValueInput = () => { + const varType = selectedVariable?.type || 'numeric' + + if (varType === 'boolean') { + return ( + + ) + } + + if (varType === 'string') { + return ( + handleValueChange(e.target.value)} + placeholder="Value..." + className="w-full rounded border border-zinc-300 bg-white px-2 py-1 text-sm focus:border-blue-500 focus:outline-none dark:border-zinc-600 dark:bg-zinc-700 dark:text-white dark:placeholder-zinc-400" + /> + ) + } + + // numeric + return ( + handleValueChange(parseFloat(e.target.value) || 0)} + className="w-full rounded border border-zinc-300 bg-white px-2 py-1 text-sm focus:border-blue-500 focus:outline-none dark:border-zinc-600 dark:bg-zinc-700 dark:text-white" + /> + ) + } return ( -
-
e.stopPropagation()} - > -

- {hasExistingCondition ? 'Edit Condition' : 'Add Condition'} -

+
+
+
+
+

+ Edge Condition +

+ +
-
- {/* Variable Name Input */} -
- - setVariableName(e.target.value)} - placeholder="e.g., score, health, affection" - autoFocus - className="w-full rounded-md border border-zinc-300 bg-white px-3 py-2 text-sm text-zinc-900 placeholder-zinc-400 focus:border-blue-500 focus:outline-none focus:ring-1 focus:ring-blue-500 dark:border-zinc-600 dark:bg-zinc-700 dark:text-zinc-100 dark:placeholder-zinc-500" + {/* Variable selector */} +
+ +
+
- - {/* Operator Dropdown */} -
- - -
- - {/* Value Number Input */} -
- - setValue(parseFloat(e.target.value) || 0)} - className="w-full rounded-md border border-zinc-300 bg-white px-3 py-2 text-sm text-zinc-900 placeholder-zinc-400 focus:border-blue-500 focus:outline-none focus:ring-1 focus:ring-blue-500 dark:border-zinc-600 dark:bg-zinc-700 dark:text-zinc-100" - /> -
- - {/* Preview */} - {variableName.trim() && ( -
- - Condition: {variableName.trim()} {operator} {value} - + {hasInvalidReference && ( +
+ Variable not found
)}
- {/* Action Buttons */} -
-
- {hasExistingCondition && ( - - )} + +
-
+ )} + + {/* Operator and value (shown when variable is selected) */} + {condition?.variableId && ( + <> +
+ + +
+ +
+ + {renderValueInput()} +
+ + )} + + {/* Actions */} +
+ {condition?.variableId ? ( - -
+ ) : ( +
+ )} +
diff --git a/src/components/editor/EditorContext.tsx b/src/components/editor/EditorContext.tsx new file mode 100644 index 0000000..177c119 --- /dev/null +++ b/src/components/editor/EditorContext.tsx @@ -0,0 +1,33 @@ +'use client' + +import { createContext, useContext } from 'react' +import type { Character, Variable } from '@/types/flowchart' +import type { NodeLock } from '@/lib/collaboration/realtime' + +export type NodeLockInfo = NodeLock & { color: string } + +type EditorContextValue = { + characters: Character[] + onAddCharacter: (name: string, color: string) => string // returns new character id + variables: Variable[] + onAddVariable: (name: string, type: 'numeric' | 'string' | 'boolean', initialValue: number | string | boolean) => string // returns new variable id + nodeLocks: Map // nodeId -> lock info + onNodeFocus: (nodeId: string) => void + onNodeBlur: () => void +} + +const EditorContext = createContext({ + characters: [], + onAddCharacter: () => '', + variables: [], + onAddVariable: () => '', + nodeLocks: new Map(), + onNodeFocus: () => {}, + onNodeBlur: () => {}, +}) + +export const EditorProvider = EditorContext.Provider + +export function useEditorContext() { + return useContext(EditorContext) +} diff --git a/src/components/editor/ExportValidationModal.tsx b/src/components/editor/ExportValidationModal.tsx new file mode 100644 index 0000000..cdad515 --- /dev/null +++ b/src/components/editor/ExportValidationModal.tsx @@ -0,0 +1,131 @@ +'use client' + +export type ValidationIssue = { + nodeId: string + nodeType: 'dialogue' | 'choice' | 'variable' | 'edge' + contentSnippet: string + undefinedReference: string + referenceType: 'character' | 'variable' +} + +type ExportValidationModalProps = { + issues: ValidationIssue[] + onExportAnyway: () => void + onCancel: () => void +} + +export default function ExportValidationModal({ + issues, + onExportAnyway, + onCancel, +}: ExportValidationModalProps) { + const characterIssues = issues.filter((i) => i.referenceType === 'character') + const variableIssues = issues.filter((i) => i.referenceType === 'variable') + + return ( +
+
+
+
+ + + +

+ Export Validation Issues +

+
+

+ {issues.length} undefined reference{issues.length !== 1 ? 's' : ''} found. These nodes/edges reference characters or variables that no longer exist. +

+
+ +
+ {characterIssues.length > 0 && ( +
+

+ Undefined Characters +

+
    + {characterIssues.map((issue, idx) => ( +
  • +
    +
    + + {issue.nodeType} + +

    + {issue.contentSnippet} +

    +
    + + ID: {issue.undefinedReference.slice(0, 8)}... + +
    +
    • + ))} +
+
+ )} + + {variableIssues.length > 0 && ( +
+

+ Undefined Variables +

+
    + {variableIssues.map((issue, idx) => ( +
  • +
    +
    + + {issue.nodeType} + +

    + {issue.contentSnippet} +

    +
    + + ID: {issue.undefinedReference.slice(0, 8)}... + +
    +
    • + ))} +
+
+ )} +
+ +
+ + +
+
+
+ ) +} diff --git a/src/components/editor/ImportFromProjectModal.tsx b/src/components/editor/ImportFromProjectModal.tsx new file mode 100644 index 0000000..e382b46 --- /dev/null +++ b/src/components/editor/ImportFromProjectModal.tsx @@ -0,0 +1,386 @@ +'use client' + +import { useEffect, useState } from 'react' +import { nanoid } from 'nanoid' +import { createClient } from '@/lib/supabase/client' +import type { Character, Variable } from '@/types/flowchart' + +type ImportMode = 'characters' | 'variables' + +type ProjectListItem = { + id: string + name: string +} + +type ImportFromProjectModalProps = { + mode: ImportMode + currentProjectId: string + existingCharacters: Character[] + existingVariables: Variable[] + onImportCharacters: (characters: Character[]) => void + onImportVariables: (variables: Variable[]) => void + onClose: () => void +} + +export default function ImportFromProjectModal({ + mode, + currentProjectId, + existingCharacters, + existingVariables, + onImportCharacters, + onImportVariables, + onClose, +}: ImportFromProjectModalProps) { + const [projects, setProjects] = useState([]) + const [loading, setLoading] = useState(true) + const [error, setError] = useState(null) + const [selectedProjectId, setSelectedProjectId] = useState(null) + const [sourceCharacters, setSourceCharacters] = useState([]) + const [sourceVariables, setSourceVariables] = useState([]) + const [loadingSource, setLoadingSource] = useState(false) + const [selectedIds, setSelectedIds] = useState>(new Set()) + const [warnings, setWarnings] = useState([]) + + // Load user's projects on mount + useEffect(() => { + async function fetchProjects() { + const supabase = createClient() + const { data, error: fetchError } = await supabase + .from('projects') + .select('id, name') + .neq('id', currentProjectId) + .order('name') + + if (fetchError) { + setError('Failed to load projects') + setLoading(false) + return + } + + setProjects(data || []) + setLoading(false) + } + + fetchProjects() + }, [currentProjectId]) + + // Load source project's characters/variables when a project is selected + const handleSelectProject = async (projectId: string) => { + setSelectedProjectId(projectId) + setLoadingSource(true) + setWarnings([]) + setSelectedIds(new Set()) + + const supabase = createClient() + const { data, error: fetchError } = await supabase + .from('projects') + .select('flowchart_data') + .eq('id', projectId) + .single() + + if (fetchError || !data) { + setError('Failed to load project data') + setLoadingSource(false) + return + } + + const flowchartData = data.flowchart_data || {} + const chars: Character[] = flowchartData.characters || [] + const vars: Variable[] = flowchartData.variables || [] + + setSourceCharacters(chars) + setSourceVariables(vars) + setLoadingSource(false) + + // Select all by default + const items = mode === 'characters' ? chars : vars + setSelectedIds(new Set(items.map((item) => item.id))) + } + + const handleToggleItem = (id: string) => { + setSelectedIds((prev) => { + const next = new Set(prev) + if (next.has(id)) { + next.delete(id) + } else { + next.add(id) + } + return next + }) + } + + const handleSelectAll = () => { + const items = mode === 'characters' ? sourceCharacters : sourceVariables + setSelectedIds(new Set(items.map((item) => item.id))) + } + + const handleSelectNone = () => { + setSelectedIds(new Set()) + } + + const handleImport = () => { + const importWarnings: string[] = [] + + if (mode === 'characters') { + const selectedCharacters = sourceCharacters.filter((c) => selectedIds.has(c.id)) + const existingNames = new Set(existingCharacters.map((c) => c.name.toLowerCase())) + const toImport: Character[] = [] + + for (const char of selectedCharacters) { + if (existingNames.has(char.name.toLowerCase())) { + importWarnings.push(`Skipped "${char.name}" (already exists)`) + } else { + // Create new ID to avoid conflicts + toImport.push({ ...char, id: nanoid() }) + existingNames.add(char.name.toLowerCase()) + } + } + + if (importWarnings.length > 0) { + setWarnings(importWarnings) + } + + if (toImport.length > 0) { + onImportCharacters(toImport) + } + + if (importWarnings.length === 0) { + onClose() + } + } else { + const selectedVariables = sourceVariables.filter((v) => selectedIds.has(v.id)) + const existingNames = new Set(existingVariables.map((v) => v.name.toLowerCase())) + const toImport: Variable[] = [] + + for (const variable of selectedVariables) { + if (existingNames.has(variable.name.toLowerCase())) { + importWarnings.push(`Skipped "${variable.name}" (already exists)`) + } else { + // Create new ID to avoid conflicts + toImport.push({ ...variable, id: nanoid() }) + existingNames.add(variable.name.toLowerCase()) + } + } + + if (importWarnings.length > 0) { + setWarnings(importWarnings) + } + + if (toImport.length > 0) { + onImportVariables(toImport) + } + + if (importWarnings.length === 0) { + onClose() + } + } + } + + const items = mode === 'characters' ? sourceCharacters : sourceVariables + + return ( +
+