ai-space/goose
3
0
Fork 0
mirror of https://github.com/aaif-goose/goose.git synced 2026-07-03 14:10:03 +02:00
Code Issues Packages Projects Releases Wiki Activity

Add safety checks for undefined sessionId in TabContext to fix startsWith error

Browse Source
This commit is contained in:
spencrmartin
2025-12-03 15:22:38 -05:00
parent b55f98a340
commit b88c74d096
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ui/desktop/src/contexts/TabContext.tsx
+3 -3
View File
@@ -179,7 +179,7 @@ export const TabProvider: React.FC<TabProviderProps> = ({ children }) => {
// Create backend sessions for any temporary sessions after component mounts
useEffect(() => {
const createBackendSessionsForTempTabs = async () => {
const tempTabs = tabStates.filter(ts => ts.tab.sessionId.startsWith('temp_'));
const tempTabs = tabStates.filter(ts => ts.tab.sessionId && ts.tab.sessionId.startsWith('temp_'));
for (const tabState of tempTabs) {
try {
@@ -445,7 +445,7 @@ export const TabProvider: React.FC<TabProviderProps> = ({ children }) => {
if (!tabState) return;
// Don't try to sync new sessions that don't exist on the backend yet
if (tabState.tab.sessionId.startsWith('new_')) {
if (!tabState.tab.sessionId || tabState.tab.sessionId.startsWith('new_')) {
console.log('🏷️ Skipping backend sync for new session:', tabState.tab.sessionId);
return;
}
@@ -945,7 +945,7 @@ export const TabProvider: React.FC<TabProviderProps> = ({ children }) => {
}
// If already has a backend session (not temporary), return it
if (!tabState.tab.sessionId.startsWith('temp_') && !tabState.tab.sessionId.startsWith('new_')) {
if (tabState.tab.sessionId && !tabState.tab.sessionId.startsWith('temp_') && !tabState.tab.sessionId.startsWith('new_')) {
console.log('🏗️ Tab already has backend session:', tabState.tab.sessionId);
return tabState.tab.sessionId;
}