GHMiranda
/
WebVNWrite
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

developing #10

Merged
GHMiranda merged 64 commits from developing into master 2026-01-25 00:37:11 +00:00
Conversation 0 Commits 64 Files Changed 37 +6899 -1865
GHMiranda commented 2026-01-25 00:27:47 +00:00
Owner
Copy Link
No description provided.
GHMiranda added 64 commits 2026-01-25 00:27:48 +00:00
d9c42f4cf7 feat: [US-054] - Character and Variable TypeScript types 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1f7bd321a2 feat: [US-055] - Database schema update for characters and variables 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
11e8daf67c feat: [US-065] - Searchable combobox component 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
8418f49787 chore: mark US-065 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2ef605c0ca feat: [US-056] - Character management UI in project settings 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
6a87e7a70b chore: mark US-056 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
285320a4fe feat: [US-057] - Variable management UI in project settings 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
548f3743d1 chore: mark US-057 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
5493adf44a feat: [US-059] - Variable node variable dropdown 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
30aebe4079 chore: mark US-059 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
b4b9f8cec9 feat: [US-060] - Edge condition variable dropdown 
Replace the variableName text input in edge conditions with a
Combobox-based variable selector. Adds ConditionEditor modal
that opens on edge click, with type-aware operators (comparison
for numeric, == and != for string/boolean) and type-adaptive
value inputs (number, text, or boolean toggle).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
b6cb0c703a chore: mark US-060 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
92d892fb73 feat: [US-061] - Choice option condition variable dropdown 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
68bfe88842 chore: mark US-061 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
b570dca1b8 feat: [US-062] - Auto-migration of existing free-text values 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
49698dd9a9 chore: mark US-062 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
a3e1d1cea2 feat: [US-063] - Import characters/variables from another project 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
a85d7cbeb3 chore: mark US-063 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
0d72471f8f feat: [US-064] - Export validation for undefined references 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
273130316b chore: mark US-064 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2b4abd1eb7 feat: [US-043] - Database schema for collaboration sessions and audit trail 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
dfbaa0066d chore: mark US-043 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2e313a0264 feat: [US-045] - Supabase Realtime channel and connection management 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
46b681462d chore: mark US-045 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
37031c3889 feat: [US-044] - Project sharing and collaborator management 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
7278b7527f chore: mark US-044 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
f92fc1ad01 feat: [US-046] - Presence indicators for active collaborators 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
d75e2daeb0 chore: mark US-046 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
6b7b27236f feat: [US-048] - Integrate Yjs CRDT for conflict-free node/edge synchronization 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
409b1848c8 chore: mark US-048 as complete and update progress log 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
841eefb186 fix: resolve infinite recursion in RLS policies 
Use SECURITY DEFINER helper functions to break circular dependencies
between projects and project_collaborators table policies.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fa336f05d9 fix: restore missing utilities, context menu, and props from failed merge 
The merge at bcbad57 lost utility functions, type definitions, and
handlers from the developing branch. This restores: imports (useRef,
useEffect, ConditionalEdge, ContextMenu), constants (AUTOSAVE_DEBOUNCE_MS),
types (ContextMenuState, ConditionEditorState), utility functions
(fromReactFlowNodes, fromReactFlowEdges, saveDraft, loadDraft, clearDraft,
flowchartDataEquals, isValidFlowchartData, convertToRenpyFormat), context
menu handlers, proper FlowchartEditorProps (userId, userDisplayName,
isOwner), ReactFlow container height (h-screen), and fixes stale closure
dependency arrays for characters/variables.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
6bc4e32fde fix: remove CRDT observer feedback loop causing drag issues 
The Yjs map observers were calling notifyNodesChange/notifyEdgesChange
for local changes, creating a feedback loop that reset React Flow state
mid-drag. Remote notifications are already handled by the connectChannel
handler, so the observers only need to schedule persistence.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
8f6cff52a9 fix: correct OptionConditionEditor props in ChoiceNode 
ChoiceNode was passing incompatible props (optionId, optionLabel, onSave,
onRemove, onCancel) to OptionConditionEditor which expects (condition,
onChange, onClose). This caused a build failure preventing the component
tree from rendering.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
b6d85b1a47 fix: add isSaving prop and back-to-dashboard navigation to Toolbar 
Add isSaving as a prop so the save button properly shows disabled/spinner
state. Add a back arrow link to /dashboard for editor navigation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
7fe10544a1 feat: [US-047] - Live cursor positions on canvas 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
841a44112a feat: [US-047] - Live cursor positions on canvas 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
ccb05e3a3e feat: [US-050] - Join/leave notifications 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
5b84170a28 fix: allow profile lookup for sharing by adding RLS policy 
The profiles table RLS policies only allowed users to view their own
profile, causing the share feature to fail when searching for users
by email. Added a policy allowing any authenticated user to read profiles.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
4a85d7a64b feat: [US-049] - Node editing lock indicators 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
6ef5cfc7fa feat: [US-051] - Audit trail recording 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
6c4a3ba2b7 docs: update PRD and progress for US-051 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
f06a30b2bf feat: [US-052] - Activity history sidebar 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
315e34e25e feat: [US-053] - Revert changes from audit trail 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
c9f5231137 docs: update PRD and progress for US-053 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
34815d70ee fix: resolve CRDT collaboration sync by registering broadcast listener before channel subscribe 
The yjs-update broadcast listener was added after the Supabase channel
was already subscribed, which meant it never received messages. Moved
the listener registration to the builder chain before .subscribe()
(matching how cursor/node-lock listeners work), and removed the broken
isRemoteUpdateRef guard that caused ReferenceErrors preventing local
changes from reaching the CRDT.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
cdaae6b965 feat: broadcast full CRDT state to other clients after manual save 
When a user saves, the full Yjs document state is broadcast so all
connected clients converge, even if they missed incremental updates.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
eb86ccd291 feat: notify collaborators to refresh from DB after save 
Instead of relying on Yjs broadcast serialization (which has delivery
issues), use a lightweight state-refresh broadcast event. When any
client persists (manual save or CRDT auto-persist), it broadcasts
state-refresh. Other clients fetch the latest flowchart_data from the
database and update their local state and CRDT.

Added isSuppressed flag to CRDTManager to prevent broadcast/persist
loops during initialization and refresh operations.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
cd73b31739 fix: improve realtime connection resilience with auto-reconnect and activity-based lifecycle 
Add connection timeout (15s) to handle stale initial subscribes, inactivity
pause (5min) to save resources when idle, and automatic resume on user activity
or tab focus. The heartbeat now detects unhealthy channel states and consecutive
failures to trigger reconnects. Unexpected CLOSED status also triggers reconnect
instead of staying disconnected silently.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
ec053d8126 fix: stop state ping-pong by removing broadcastStateRefresh from auto-persist 
The CRDT auto-persist was broadcasting state-refresh after each save, causing
other clients to fetch from DB and overwrite their local variables/characters
with stale values. Since CRDT already syncs nodes/edges via yjs-update
broadcasts, the state-refresh from auto-persist was redundant and destructive.
Manual save still broadcasts state-refresh as intended.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2d0c3b6df6 fix: reduce auto-save frequency to avoid excessive writes 
Increase CRDT DB persist debounce from 2s to 30s since it's only needed for
crash recovery (CRDT handles real-time sync). Increase LocalStorage draft
save debounce from 1s to 5s.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fa8437d03a fix: broadcast full CRDT state on connect and when new user joins 
CRDT broadcasts are fire-and-forget, so any updates missed during a
disconnection were permanently lost. Now when a client connects (or
reconnects), it broadcasts its full Yjs doc state. When an existing client
sees a new user join, it also broadcasts its full state. Yjs merges
handle deduplication automatically, so this converges all clients.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
c28b9ac565 fix: force fresh reconnect when returning after idle period 
The WebSocket can go stale during inactivity without triggering any error
status (channel still shows 'joined' but socket is dead). Now when the user
returns after 60+ seconds of inactivity, the connection is torn down and
re-established to guarantee a fresh WebSocket. This ensures CRDT broadcasts
actually reach other clients after returning from idle.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
3b19f58e26 fix: wait for auth session before connecting to realtime channel 
On initial page load, the Supabase browser client's auth session may not
be ready yet (still loading from cookies). The Realtime channel subscription
silently fails without a valid access token. Now connect() awaits getSession()
first, which ensures the token is available and also refreshes expired tokens
on reconnect after inactivity.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
cd3ecc4507 fix: only force-reconnect on tab visibility change, not on activity gaps 
The 60s stale threshold was triggering during normal use (reading, thinking)
causing constant reconnects and presence join/leave toast spam. Now force-
reconnect only happens when the tab was hidden for 3+ minutes and becomes
visible again. Regular activity (mouse/keyboard) only resets the inactivity
timer without ever forcing a reconnect.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
GHMiranda merged commit f077d902b4 into master 2026-01-25 00:37:11 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: GHMiranda/WebVNWrite#10
No description provided.
Delete Branch "developing"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?