feat: testing onboarding mode, other stuff

Author: grantiguess

TESTING_ONBOARDING.md

@@ -0,0 +1,111 @@
+# Testing Onboarding Flow
+
+## Test Mode
+
+Redstring supports a special "test mode" that uses completely separate storage, allowing you to test the first-time user experience without affecting your main session.
+
+### How to Use Test Mode
+
+#### Web/Localhost
+Add `?test=true` to your URL:
+```
+http://localhost:5173/?test=true
+```
+
+This will:
+- Use separate localStorage keys (prefixed with `test_`)
+- Use separate IndexedDB database (`test_RedstringFolderStorage`)
+- Not interfere with your main session
+
+#### Electron
+Launch Electron with a test flag (you'll need to add command-line argument support):
+```bash
+npm run electron -- --test
+```
+
+Or manually add `?test=true` to the initial URL in your Electron main process.
+
+### What Test Mode Does
+
+When `?test=true` is in the URL, Redstring will:
+
+1. **Use separate storage keys:**
+   - `test_redstring-alpha-welcome-seen` instead of `redstring-alpha-welcome-seen`
+   - `test_redstring_workspace_folder_path` instead of `redstring_workspace_folder_path`
+   - `test_RedstringFolderStorage` IndexedDB instead of `RedstringFolderStorage`
+
+2. **Show first-time onboarding flow:**
+   - Welcome modal appears
+   - Storage setup modal appears
+   - Can test folder selection and universe creation
+
+3. **Keep your real data safe:**
+   - Your actual workspace folder is untouched
+   - Your actual universe files are not affected
+   - Your main session preferences are preserved
+
+### Testing Workflow
+
+1. **Open test mode:**
+   ```
+   http://localhost:5173/?test=true
+   ```
+
+2. **Go through onboarding:**
+   - Click "Get Started"
+   - Choose a test folder (create a separate "RedstringTest" folder)
+   - Test the universe creation flow
+
+3. **Test returning user flow:**
+   - Reload with `?test=true`
+   - Should load directly into your test universe
+
+4. **Reset test mode:**
+   - Use Debug menu → "Reset Onboarding Flow"
+   - Or manually clear: `localStorage.removeItem('test_redstring-alpha-welcome-seen')`
+
+5. **Return to normal mode:**
+   - Remove `?test=true` from URL
+   - Your regular session loads normally
+
+### Manual Storage Clearing
+
+If you need to manually reset test mode:
+
+```javascript
+// Clear test mode onboarding flag
+localStorage.removeItem('test_redstring-alpha-welcome-seen');
+
+// Clear test mode folder
+localStorage.removeItem('test_redstring_workspace_folder_path');
+
+// Clear test mode IndexedDB
+indexedDB.deleteDatabase('test_RedstringFolderStorage');
+
+// Reload
+window.location.reload();
+```
+
+### Debug Menu Option
+
+The Debug menu includes "Reset Onboarding Flow" which will:
+- Clear the onboarding completion flag
+- Clear stored folder handles
+- Reload the page
+
+This works in both normal and test mode depending on which mode you're in.
+
+## Recommended Test Folder Structure
+
+Create a separate test folder to avoid confusion:
+
+```
+~/Documents/
+  ├── Redstring/           # Your real workspace
+  │   ├── default.redstring
+  │   └── project.redstring
+  └── RedstringTest/       # Test mode workspace
+      └── default.redstring
+```
+
+This way you can safely test onboarding without risking your actual data.

repro_parsing.js

@@ -0,0 +1,23 @@
+
+// Logic extracted from LLMClient.js to reproduce JSON.parse error on truncation
+
+const partialJson = '{"name": "Iron Man", "description": "Tony Sta'; // Truncated
+
+try {
+    console.log('Attempting to parse truncated JSON...');
+    const result = JSON.parse(partialJson);
+    console.log('Success:', result);
+} catch (error) {
+    console.log('Caught expected error:', error.message);
+}
+
+// Logic with the proposed fix
+console.log('\n--- Testing Fix ---');
+try {
+    const result = JSON.parse(partialJson);
+    console.log('Success:', result);
+} catch (e) {
+    console.log('Safely caught error:', e.message);
+    const safeError = { error: 'Response was truncated by the spell!' };
+    console.log('Returning safe error:', safeError);
+}

src/NodeCanvas.jsx

@@ -1839,6 +1839,13 @@ function NodeCanvas() {
   // Help modal state
   const [showHelpModal, setShowHelpModal] = useState(false);
 
+  // Helper to get storage key with test mode support
+  const getStorageKey = (key) => {
+    const params = new URLSearchParams(window.location.search);
+    const isTestMode = params.get('test') === 'true';
+    return isTestMode ? `test_${key}` : key;
+  };
+
   // Check for stored folder on app startup and attempt to restore
   useEffect(() => {
     let isMounted = true;
@@ -1911,7 +1918,7 @@ function NodeCanvas() {
     let hasCompletedOnboarding = false;
     try {
       if (typeof window !== 'undefined') {
-        hasCompletedOnboarding = localStorage.getItem('redstring-alpha-welcome-seen') === 'true';
+        hasCompletedOnboarding = localStorage.getItem(getStorageKey('redstring-alpha-welcome-seen')) === 'true';
       }
     } catch { }
 
@@ -13257,7 +13264,7 @@ function NodeCanvas() {
           // Mark onboarding as complete when user closes the modal
           try {
             if (typeof window !== 'undefined') {
-              localStorage.setItem('redstring-alpha-welcome-seen', 'true');
+              localStorage.setItem(getStorageKey('redstring-alpha-welcome-seen'), 'true');
             }
           } catch { }
           setShowOnboardingModal(false);
@@ -13271,7 +13278,7 @@ function NodeCanvas() {
           // Mark onboarding as complete
           try {
             if (typeof window !== 'undefined') {
-              localStorage.setItem('redstring-alpha-welcome-seen', 'true');
+              localStorage.setItem(getStorageKey('redstring-alpha-welcome-seen'), 'true');
             }
           } catch { }
 
@@ -13310,7 +13317,7 @@ function NodeCanvas() {
 
             // Mark onboarding as complete
             if (typeof window !== 'undefined') {
-              localStorage.setItem('redstring-alpha-welcome-seen', 'true');
+              localStorage.setItem(getStorageKey('redstring-alpha-welcome-seen'), 'true');
             }
 
             // Close storage setup modal
@@ -13376,7 +13383,7 @@ function NodeCanvas() {
 
             // Mark onboarding as complete
             if (typeof window !== 'undefined') {
-              localStorage.setItem('redstring-alpha-welcome-seen', 'true');
+              localStorage.setItem(getStorageKey('redstring-alpha-welcome-seen'), 'true');
             }
 
             // Close storage setup modal

src/ai/BridgeClient.jsx

@@ -177,6 +177,9 @@ const BridgeClient = () => {
   });
   // Track last telemetry timestamp sent to UI to avoid spam
   const lastTelemetryTsRef = useRef(0);
+  // Track last user activity to throttle polling when idle
+  // Initialized to now so we start in fast mode
+  const lastActivityRef = useRef(Date.now());
 
   useEffect(() => {
     mountedRef.current = true;
@@ -383,7 +386,11 @@ const BridgeClient = () => {
         // Store the actual functions in a global variable that the bridge server can access
         if (typeof window !== 'undefined') {
           window.redstringStoreActions = {
+            // Helper to mark activity
+            _markActive: () => { lastActivityRef.current = Date.now(); },
+
             ensureGraph: async (graphId, initialData) => {
+              lastActivityRef.current = Date.now();
               console.log('MCPBridge: Calling ensureGraph', graphId, initialData);
               const st = useGraphStore.getState();
               if (!st.graphs.has(graphId)) {
@@ -588,6 +595,7 @@ const BridgeClient = () => {
               return { success: true };
             },
             chat: async (message, context) => {
+              lastActivityRef.current = Date.now();
               console.log('MCPBridge: Forwarding chat message to AI model', { message, context });
               // The actual chat handling happens in the MCP server
               return { success: true, message, context };
@@ -597,22 +605,22 @@ const BridgeClient = () => {
               // options: { mode, nodeIds, graphId, coordinates, zoom }
               console.log('MCPBridge: Navigating to', options);
               const { navigateToNodes, navigateToFitContent, navigateToCoordinates } = await import('../services/canvasNavigationService.js');
-              
+
               if (options.nodeIds && options.nodeIds.length > 0) {
-                navigateToNodes(options.nodeIds, { 
+                navigateToNodes(options.nodeIds, {
                   graphId: options.graphId,
-                  delay: options.delay || 150 
+                  delay: options.delay || 150
                 });
               } else if (options.coordinates) {
                 navigateToCoordinates(
-                  options.coordinates.x, 
-                  options.coordinates.y, 
+                  options.coordinates.x,
+                  options.coordinates.y,
                   { zoom: options.zoom, delay: options.delay || 150 }
                 );
               } else {
-                navigateToFitContent({ 
+                navigateToFitContent({
                   graphId: options.graphId,
-                  delay: options.delay || 150 
+                  delay: options.delay || 150
                 });
               }
               return { success: true, navigated: true };
@@ -1226,6 +1234,8 @@ const BridgeClient = () => {
         if (actionsResponse.ok) {
           const actionsData = await actionsResponse.json();
           if (actionsData.pendingActions && actionsData.pendingActions.length > 0) {
+            // Activity detected! Reset idle timer
+            lastActivityRef.current = Date.now();
             console.log('✅ MCP Bridge: Found pending actions:', actionsData.pendingActions.length);
             // Execute actions in a stable dependency-friendly order
             const priority = (act) => {
@@ -1533,9 +1543,29 @@ const BridgeClient = () => {
     };
 
     // Check for bridge updates every 1s; guard with mountedRef to auto-resume after remounts
-    bridgeIntervalRef.current = setInterval(() => {
-      if (mountedRef.current) checkForBridgeUpdates();
-    }, 1000);
+    // Adaptive polling loop
+    const pollingLoop = async () => {
+      if (!mountedRef.current) return;
+
+      const now = Date.now();
+      const timeSinceActivity = now - lastActivityRef.current;
+      const isIdle = timeSinceActivity > 30000; // 30 seconds idle
+
+      // Fast poll (1s) if active, slow poll (5s) if idle
+      const interval = isIdle ? 5000 : 1000;
+
+      await checkForBridgeUpdates();
+
+      if (mountedRef.current && connectionStateRef.current.isConnected) {
+        bridgeIntervalRef.current = setTimeout(pollingLoop, interval);
+      } else if (mountedRef.current) {
+        // If disconnected, check less frequently but keep checking to resume
+        bridgeIntervalRef.current = setTimeout(pollingLoop, 5000);
+      }
+    };
+
+    // Start the loop
+    pollingLoop();
 
     // Cleanup function
     return () => {

src/components/AlphaOnboardingModal.jsx

@@ -39,7 +39,10 @@ const AlphaOnboardingModal = ({
 
   const handleClose = () => {
     if (typeof window !== 'undefined') {
-      localStorage.setItem('redstring-alpha-welcome-seen', 'true');
+      const params = new URLSearchParams(window.location.search);
+      const isTestMode = params.get('test') === 'true';
+      const key = isTestMode ? 'test_redstring-alpha-welcome-seen' : 'redstring-alpha-welcome-seen';
+      localStorage.setItem(key, 'true');
     }
     onClose();
   };