Skip to content

Commit 53be2d8

Browse files
committed
fix: emit recovery-specific error, remove redundant mock, use React Router for state clear
- socket.js: emit error:recover:error instead of generic error:server - memory.test.js: remove redundant cos.js mock (memory.js imports from cosEvents.js) - RunnerPage.jsx: use navigate() with replace instead of window.history.replaceState to preserve React Router internal state
1 parent bbf40d9 commit 53be2d8

3 files changed

Lines changed: 4 additions & 6 deletions

File tree

client/src/pages/RunnerPage.jsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { useState, useEffect, useRef } from 'react';
2-
import { useLocation } from 'react-router-dom';
2+
import { useLocation, useNavigate } from 'react-router-dom';
33
import { Clock, Image, X, Info } from 'lucide-react';
44
import toast from '../components/ui/Toast';
55
import * as api from '../services/api';
@@ -9,6 +9,7 @@ import BrailleSpinner from '../components/BrailleSpinner';
99

1010
export function RunnerPage() {
1111
const location = useLocation();
12+
const navigate = useNavigate();
1213
const [mode, setMode] = useState('ai'); // 'ai' or 'command'
1314
const [prompt, setPrompt] = useState('');
1415
const [command, setCommand] = useState('');
@@ -52,7 +53,7 @@ export function RunnerPage() {
5253
}
5354
}
5455
// Clear location state to prevent re-triggering on refresh
55-
window.history.replaceState({}, document.title);
56+
navigate(location.pathname, { replace: true, state: null });
5657
}
5758
}, [location.state, apps]);
5859

server/services/memory.test.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,6 @@ vi.mock('../lib/uuid.js', () => ({
2222
vi.mock('./cosEvents.js', () => ({
2323
cosEvents: new EventEmitter()
2424
}));
25-
vi.mock('./cos.js', () => ({
26-
cosEvents: new EventEmitter()
27-
}));
2825

2926
// Mock vectorMath
3027
vi.mock('../lib/vectorMath.js', () => ({

server/services/socket.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ export function initSocket(io) {
313313
} catch (err) {
314314
const message = err?.message ?? String(err);
315315
console.error(`❌ Socket handler error [error:recover]: ${message}`);
316-
socket.emit('error:server', { message });
316+
socket.emit('error:recover:error', { message });
317317
}
318318
});
319319

0 commit comments

Comments
 (0)