mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-03-16 21:53:07 +00:00
Compare commits
7 Commits
fix/dev-se
...
d37ced1c6e
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
d37ced1c6e | ||
|
|
0311130a8a | ||
|
|
7be8163b84 | ||
|
|
26b73df097 | ||
|
|
20e7c74b17 | ||
|
|
dd7108a7a0 | ||
|
|
ae48065820 |
1
.gitignore
vendored
1
.gitignore
vendored
@@ -113,3 +113,4 @@ data/
|
||||
.planning/
|
||||
.mcp.json
|
||||
.planning
|
||||
.bg-shell/
|
||||
@@ -32,7 +32,7 @@
|
||||
"@automaker/prompts": "1.0.0",
|
||||
"@automaker/types": "1.0.0",
|
||||
"@automaker/utils": "1.0.0",
|
||||
"@github/copilot-sdk": "^0.1.16",
|
||||
"@github/copilot-sdk": "0.1.16",
|
||||
"@modelcontextprotocol/sdk": "1.25.2",
|
||||
"@openai/codex-sdk": "^0.98.0",
|
||||
"cookie-parser": "1.4.7",
|
||||
|
||||
@@ -598,24 +598,23 @@ wss.on('connection', (ws: WebSocket) => {
|
||||
|
||||
// Subscribe to all events and forward to this client
|
||||
const unsubscribe = events.subscribe((type, payload) => {
|
||||
logger.info('Event received:', {
|
||||
// Use debug level for high-frequency events to avoid log spam
|
||||
// that causes progressive memory growth and server slowdown
|
||||
const isHighFrequency =
|
||||
type === 'dev-server:output' || type === 'test-runner:output' || type === 'feature:progress';
|
||||
const log = isHighFrequency ? logger.debug.bind(logger) : logger.info.bind(logger);
|
||||
|
||||
log('Event received:', {
|
||||
type,
|
||||
hasPayload: !!payload,
|
||||
payloadKeys: payload ? Object.keys(payload) : [],
|
||||
wsReadyState: ws.readyState,
|
||||
wsOpen: ws.readyState === WebSocket.OPEN,
|
||||
});
|
||||
|
||||
if (ws.readyState === WebSocket.OPEN) {
|
||||
const message = JSON.stringify({ type, payload });
|
||||
logger.info('Sending event to client:', {
|
||||
type,
|
||||
messageLength: message.length,
|
||||
sessionId: (payload as Record<string, unknown>)?.sessionId,
|
||||
});
|
||||
ws.send(message);
|
||||
} else {
|
||||
logger.info('WARNING: Cannot send event, WebSocket not open. ReadyState:', ws.readyState);
|
||||
logger.warn('Cannot send event, WebSocket not open. ReadyState:', ws.readyState);
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -2,7 +2,7 @@
|
||||
* Version utility - Reads version from package.json
|
||||
*/
|
||||
|
||||
import { readFileSync } from 'fs';
|
||||
import { readFileSync, existsSync } from 'fs';
|
||||
import { fileURLToPath } from 'url';
|
||||
import { dirname, join } from 'path';
|
||||
import { createLogger } from '@automaker/utils';
|
||||
@@ -24,7 +24,20 @@ export function getVersion(): string {
|
||||
}
|
||||
|
||||
try {
|
||||
const packageJsonPath = join(__dirname, '..', '..', 'package.json');
|
||||
const candidatePaths = [
|
||||
// Development via tsx: src/lib -> project root
|
||||
join(__dirname, '..', '..', 'package.json'),
|
||||
// Packaged/build output: lib -> server bundle root
|
||||
join(__dirname, '..', 'package.json'),
|
||||
];
|
||||
|
||||
const packageJsonPath = candidatePaths.find((candidate) => existsSync(candidate));
|
||||
if (!packageJsonPath) {
|
||||
throw new Error(
|
||||
`package.json not found in any expected location: ${candidatePaths.join(', ')}`
|
||||
);
|
||||
}
|
||||
|
||||
const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf-8'));
|
||||
const version = packageJson.version || '0.0.0';
|
||||
cachedVersion = version;
|
||||
|
||||
@@ -88,9 +88,13 @@ const PORT_PATTERNS: Array<{ pattern: RegExp; description: string }> = [
|
||||
},
|
||||
];
|
||||
|
||||
// Throttle output to prevent overwhelming WebSocket under heavy load
|
||||
const OUTPUT_THROTTLE_MS = 4; // ~250fps max update rate for responsive feedback
|
||||
const OUTPUT_BATCH_SIZE = 4096; // Smaller batches for lower latency
|
||||
// Throttle output to prevent overwhelming WebSocket under heavy load.
|
||||
// 100ms (~10fps) is sufficient for readable log streaming while keeping
|
||||
// WebSocket traffic manageable. The previous 4ms rate (~250fps) generated
|
||||
// up to 250 events/sec which caused progressive browser slowdown from
|
||||
// accumulated console logs, JSON serialization overhead, and React re-renders.
|
||||
const OUTPUT_THROTTLE_MS = 100; // ~10fps max update rate
|
||||
const OUTPUT_BATCH_SIZE = 8192; // Larger batches to compensate for lower frequency
|
||||
|
||||
export interface DevServerInfo {
|
||||
worktreePath: string;
|
||||
|
||||
@@ -595,12 +595,12 @@ export class EventHookService {
|
||||
if (clickUrl && context.projectPath) {
|
||||
try {
|
||||
const url = new URL(clickUrl);
|
||||
url.pathname = '/board';
|
||||
// Add projectPath so the UI can switch to the correct project
|
||||
url.searchParams.set('projectPath', context.projectPath);
|
||||
// Add featureId as query param for deep linking to board with feature output modal
|
||||
if (context.featureId) {
|
||||
url.pathname = '/board';
|
||||
url.searchParams.set('featureId', context.featureId);
|
||||
} else {
|
||||
url.pathname = '/board';
|
||||
}
|
||||
clickUrl = url.toString();
|
||||
} catch (error) {
|
||||
|
||||
@@ -179,6 +179,7 @@ ${feature.spec}
|
||||
const abortController = tempRunningFeature.abortController;
|
||||
if (isAutoMode) await this.saveExecutionStateFn(projectPath);
|
||||
let feature: Feature | null = null;
|
||||
let pipelineCompleted = false;
|
||||
|
||||
try {
|
||||
validateWorkingDirectory(projectPath);
|
||||
@@ -431,6 +432,7 @@ Please continue from where you left off and complete all remaining tasks. Use th
|
||||
testAttempts: 0,
|
||||
maxTestAttempts: 5,
|
||||
});
|
||||
pipelineCompleted = true;
|
||||
// Check if pipeline set a terminal status (e.g., merge_conflict) — don't overwrite it
|
||||
const refreshed = await this.loadFeatureFn(projectPath, featureId);
|
||||
if (refreshed?.status === 'merge_conflict') {
|
||||
@@ -541,7 +543,30 @@ Please continue from where you left off and complete all remaining tasks. Use th
|
||||
}
|
||||
} else {
|
||||
logger.error(`Feature ${featureId} failed:`, error);
|
||||
await this.updateFeatureStatusFn(projectPath, featureId, 'backlog');
|
||||
// If pipeline steps completed successfully, don't send the feature back to backlog.
|
||||
// The pipeline work is done — set to waiting_approval so the user can review.
|
||||
const fallbackStatus = pipelineCompleted ? 'waiting_approval' : 'backlog';
|
||||
if (pipelineCompleted) {
|
||||
logger.info(
|
||||
`[executeFeature] Feature ${featureId} failed after pipeline completed. ` +
|
||||
`Setting status to waiting_approval instead of backlog to preserve pipeline work.`
|
||||
);
|
||||
}
|
||||
// Don't overwrite terminal states like 'merge_conflict' that were set during pipeline execution
|
||||
let currentStatus: string | undefined;
|
||||
try {
|
||||
const currentFeature = await this.loadFeatureFn(projectPath, featureId);
|
||||
currentStatus = currentFeature?.status;
|
||||
} catch (loadErr) {
|
||||
// If loading fails, log it and proceed with the status update anyway
|
||||
logger.warn(
|
||||
`[executeFeature] Failed to reload feature ${featureId} for status check:`,
|
||||
loadErr
|
||||
);
|
||||
}
|
||||
if (currentStatus !== 'merge_conflict') {
|
||||
await this.updateFeatureStatusFn(projectPath, featureId, fallbackStatus);
|
||||
}
|
||||
this.eventBus.emitAutoModeEvent('auto_mode_error', {
|
||||
featureId,
|
||||
featureName: feature?.title,
|
||||
|
||||
@@ -350,6 +350,7 @@ export class PipelineOrchestrator {
|
||||
});
|
||||
const abortController = runningEntry.abortController;
|
||||
runningEntry.branchName = feature.branchName ?? null;
|
||||
let pipelineCompleted = false;
|
||||
|
||||
try {
|
||||
validateWorkingDirectory(projectPath);
|
||||
@@ -403,6 +404,7 @@ export class PipelineOrchestrator {
|
||||
};
|
||||
|
||||
await this.executePipeline(context);
|
||||
pipelineCompleted = true;
|
||||
|
||||
// Re-fetch feature to check if executePipeline set a terminal status (e.g., merge_conflict)
|
||||
const reloadedFeature = await this.featureStateManager.loadFeature(projectPath, featureId);
|
||||
@@ -439,8 +441,21 @@ export class PipelineOrchestrator {
|
||||
});
|
||||
}
|
||||
} else {
|
||||
// If pipeline steps completed successfully, don't send the feature back to backlog.
|
||||
// The pipeline work is done — set to waiting_approval so the user can review.
|
||||
const fallbackStatus = pipelineCompleted ? 'waiting_approval' : 'backlog';
|
||||
if (pipelineCompleted) {
|
||||
logger.info(
|
||||
`[resumeFromStep] Feature ${featureId} failed after pipeline completed. ` +
|
||||
`Setting status to waiting_approval instead of backlog to preserve pipeline work.`
|
||||
);
|
||||
}
|
||||
logger.error(`Pipeline resume failed for ${featureId}:`, error);
|
||||
await this.updateFeatureStatusFn(projectPath, featureId, 'backlog');
|
||||
// Don't overwrite terminal states like 'merge_conflict' that were set during pipeline execution
|
||||
const currentFeature = await this.featureStateManager.loadFeature(projectPath, featureId);
|
||||
if (currentFeature?.status !== 'merge_conflict') {
|
||||
await this.updateFeatureStatusFn(projectPath, featureId, fallbackStatus);
|
||||
}
|
||||
this.eventBus.emitAutoModeEvent('auto_mode_error', {
|
||||
featureId,
|
||||
featureName: feature.title,
|
||||
|
||||
@@ -90,8 +90,8 @@ describe('DevServerService Event Types', () => {
|
||||
|
||||
// 2. Output & URL Detected
|
||||
mockProcess.stdout.emit('data', Buffer.from('Local: http://localhost:5173/\n'));
|
||||
// Throttled output needs a bit of time
|
||||
await new Promise((resolve) => setTimeout(resolve, 100));
|
||||
// Throttled output needs a bit of time (OUTPUT_THROTTLE_MS is 100ms)
|
||||
await new Promise((resolve) => setTimeout(resolve, 250));
|
||||
expect(emittedEvents['dev-server:output'].length).toBeGreaterThanOrEqual(1);
|
||||
expect(emittedEvents['dev-server:url-detected'].length).toBe(1);
|
||||
expect(emittedEvents['dev-server:url-detected'][0].url).toBe('http://localhost:5173/');
|
||||
|
||||
@@ -2000,5 +2000,60 @@ describe('execution-service.ts', () => {
|
||||
// The only non-in_progress status call should be absent since merge_conflict returns early
|
||||
expect(statusCalls.length).toBe(0);
|
||||
});
|
||||
|
||||
it('sets waiting_approval instead of backlog when error occurs after pipeline completes', async () => {
|
||||
// Set up pipeline with steps
|
||||
vi.mocked(pipelineService.getPipelineConfig).mockResolvedValue({
|
||||
version: 1,
|
||||
steps: [{ id: 'step-1', name: 'Step 1', order: 1, instructions: 'Do step 1' }] as any,
|
||||
});
|
||||
|
||||
// Pipeline succeeds, but reading agent output throws after pipeline completes
|
||||
mockExecutePipelineFn = vi.fn().mockResolvedValue(undefined);
|
||||
// Simulate an error after pipeline completes by making loadFeature throw
|
||||
// on the post-pipeline refresh call
|
||||
let loadCallCount = 0;
|
||||
mockLoadFeatureFn = vi.fn().mockImplementation(() => {
|
||||
loadCallCount++;
|
||||
if (loadCallCount === 1) return testFeature; // initial load
|
||||
// Second call is the task-retry check, third is the pipeline refresh
|
||||
if (loadCallCount <= 2) return testFeature;
|
||||
throw new Error('Unexpected post-pipeline error');
|
||||
});
|
||||
|
||||
const svc = createServiceWithMocks();
|
||||
await svc.executeFeature('/test/project', 'feature-1');
|
||||
|
||||
// Should set to waiting_approval, NOT backlog, since pipeline completed
|
||||
const backlogCalls = vi
|
||||
.mocked(mockUpdateFeatureStatusFn)
|
||||
.mock.calls.filter((call) => call[2] === 'backlog');
|
||||
expect(backlogCalls.length).toBe(0);
|
||||
|
||||
const waitingCalls = vi
|
||||
.mocked(mockUpdateFeatureStatusFn)
|
||||
.mock.calls.filter((call) => call[2] === 'waiting_approval');
|
||||
expect(waitingCalls.length).toBeGreaterThan(0);
|
||||
});
|
||||
|
||||
it('still sets backlog when error occurs before pipeline completes', async () => {
|
||||
// Set up pipeline with steps
|
||||
vi.mocked(pipelineService.getPipelineConfig).mockResolvedValue({
|
||||
version: 1,
|
||||
steps: [{ id: 'step-1', name: 'Step 1', order: 1, instructions: 'Do step 1' }] as any,
|
||||
});
|
||||
|
||||
// Pipeline itself throws (e.g., agent error during pipeline step)
|
||||
mockExecutePipelineFn = vi.fn().mockRejectedValue(new Error('Agent execution failed'));
|
||||
|
||||
const svc = createServiceWithMocks();
|
||||
await svc.executeFeature('/test/project', 'feature-1');
|
||||
|
||||
// Should still set to backlog since pipeline did NOT complete
|
||||
const backlogCalls = vi
|
||||
.mocked(mockUpdateFeatureStatusFn)
|
||||
.mock.calls.filter((call) => call[2] === 'backlog');
|
||||
expect(backlogCalls.length).toBe(1);
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
@@ -68,7 +68,13 @@ export function NotificationBell({ projectPath }: NotificationBellProps) {
|
||||
|
||||
// Navigate to the relevant view based on notification type
|
||||
if (notification.featureId) {
|
||||
navigate({ to: '/board', search: { featureId: notification.featureId } });
|
||||
navigate({
|
||||
to: '/board',
|
||||
search: {
|
||||
featureId: notification.featureId,
|
||||
projectPath: notification.projectPath || undefined,
|
||||
},
|
||||
});
|
||||
}
|
||||
},
|
||||
[handleMarkAsRead, setPopoverOpen, navigate]
|
||||
|
||||
@@ -1,4 +1,4 @@
|
||||
import { useEffect, useState, useCallback, useMemo, useRef } from 'react';
|
||||
import { useEffect, useState, useCallback, useMemo, useRef, startTransition } from 'react';
|
||||
import { createLogger } from '@automaker/utils/logger';
|
||||
import type { PointerEvent as ReactPointerEvent } from 'react';
|
||||
import {
|
||||
@@ -37,6 +37,7 @@ import type {
|
||||
ReasoningEffort,
|
||||
} from '@automaker/types';
|
||||
import { pathsEqual } from '@/lib/utils';
|
||||
import { initializeProject } from '@/lib/project-init';
|
||||
import { toast } from 'sonner';
|
||||
import {
|
||||
BoardBackgroundModal,
|
||||
@@ -117,9 +118,11 @@ const logger = createLogger('Board');
|
||||
interface BoardViewProps {
|
||||
/** Feature ID from URL parameter - if provided, opens output modal for this feature on load */
|
||||
initialFeatureId?: string;
|
||||
/** Project path from URL parameter - if provided, switches to this project before handling deep link */
|
||||
initialProjectPath?: string;
|
||||
}
|
||||
|
||||
export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
export function BoardView({ initialFeatureId, initialProjectPath }: BoardViewProps) {
|
||||
const {
|
||||
currentProject,
|
||||
defaultSkipTests,
|
||||
@@ -139,6 +142,7 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
setPipelineConfig,
|
||||
featureTemplates,
|
||||
defaultSortNewestCardOnTop,
|
||||
upsertAndSetCurrentProject,
|
||||
} = useAppStore(
|
||||
useShallow((state) => ({
|
||||
currentProject: state.currentProject,
|
||||
@@ -159,6 +163,7 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
setPipelineConfig: state.setPipelineConfig,
|
||||
featureTemplates: state.featureTemplates,
|
||||
defaultSortNewestCardOnTop: state.defaultSortNewestCardOnTop,
|
||||
upsertAndSetCurrentProject: state.upsertAndSetCurrentProject,
|
||||
}))
|
||||
);
|
||||
// Also get keyboard shortcuts for the add feature shortcut
|
||||
@@ -305,6 +310,53 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
setFeaturesWithContext,
|
||||
});
|
||||
|
||||
// Handle deep link project switching - if URL includes a projectPath that differs from
|
||||
// the current project, switch to the target project first. The feature/worktree deep link
|
||||
// effect below will fire naturally once the project switch triggers a features reload.
|
||||
const handledProjectPathRef = useRef<string | undefined>(undefined);
|
||||
useEffect(() => {
|
||||
if (!initialProjectPath || handledProjectPathRef.current === initialProjectPath) {
|
||||
return;
|
||||
}
|
||||
|
||||
// Check if we're already on the correct project
|
||||
if (currentProject?.path && pathsEqual(currentProject.path, initialProjectPath)) {
|
||||
handledProjectPathRef.current = initialProjectPath;
|
||||
return;
|
||||
}
|
||||
|
||||
handledProjectPathRef.current = initialProjectPath;
|
||||
|
||||
const switchProject = async () => {
|
||||
try {
|
||||
const initResult = await initializeProject(initialProjectPath);
|
||||
if (!initResult.success) {
|
||||
logger.warn(
|
||||
`Deep link: failed to initialize project "${initialProjectPath}":`,
|
||||
initResult.error
|
||||
);
|
||||
toast.error('Failed to open project from link', {
|
||||
description: initResult.error || 'Unknown error',
|
||||
});
|
||||
return;
|
||||
}
|
||||
|
||||
// Derive project name from path basename
|
||||
const projectName =
|
||||
initialProjectPath.split(/[/\\]/).filter(Boolean).pop() || initialProjectPath;
|
||||
logger.info(`Deep link: switching to project "${projectName}" at ${initialProjectPath}`);
|
||||
upsertAndSetCurrentProject(initialProjectPath, projectName);
|
||||
} catch (error) {
|
||||
logger.error('Deep link: project switch failed:', error);
|
||||
toast.error('Failed to switch project', {
|
||||
description: error instanceof Error ? error.message : 'Unknown error',
|
||||
});
|
||||
}
|
||||
};
|
||||
|
||||
switchProject();
|
||||
}, [initialProjectPath, currentProject?.path, upsertAndSetCurrentProject]);
|
||||
|
||||
// Handle initial feature ID from URL - switch to the correct worktree and open output modal
|
||||
// Uses a ref to track which featureId has been handled to prevent re-opening
|
||||
// when the component re-renders but initialFeatureId hasn't changed.
|
||||
@@ -325,6 +377,17 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
[currentProject?.path]
|
||||
)
|
||||
);
|
||||
|
||||
// Track how many render cycles we've waited for worktrees during a deep link.
|
||||
// If the Zustand store never gets populated (e.g., WorktreePanel hasn't mounted,
|
||||
// useWorktrees setting is off, or the worktree query failed), we stop waiting
|
||||
// after a threshold and open the modal without switching worktree.
|
||||
const deepLinkRetryCountRef = useRef(0);
|
||||
// Reset retry count when the feature ID changes
|
||||
useEffect(() => {
|
||||
deepLinkRetryCountRef.current = 0;
|
||||
}, [initialFeatureId]);
|
||||
|
||||
useEffect(() => {
|
||||
if (
|
||||
!initialFeatureId ||
|
||||
@@ -339,14 +402,43 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
const feature = hookFeatures.find((f) => f.id === initialFeatureId);
|
||||
if (!feature) return;
|
||||
|
||||
// If the feature has a branch, wait for worktrees to load so we can switch
|
||||
if (feature.branchName && deepLinkWorktrees.length === 0) {
|
||||
return; // Worktrees not loaded yet - effect will re-run when they load
|
||||
// Resolve worktrees: prefer the Zustand store (reactive), but fall back to
|
||||
// the React Query cache if the store hasn't been populated yet. The store is
|
||||
// only synced by the WorktreePanel's useWorktrees hook, which may not have
|
||||
// rendered yet during a deep link cold start. Reading the query cache directly
|
||||
// avoids an indefinite wait that hangs the app on the loading screen.
|
||||
let resolvedWorktrees = deepLinkWorktrees;
|
||||
if (resolvedWorktrees.length === 0 && currentProject.path) {
|
||||
const cachedData = queryClient.getQueryData(queryKeys.worktrees.all(currentProject.path)) as
|
||||
| { worktrees?: WorktreeInfo[] }
|
||||
| undefined;
|
||||
if (cachedData?.worktrees && cachedData.worktrees.length > 0) {
|
||||
resolvedWorktrees = cachedData.worktrees as typeof deepLinkWorktrees;
|
||||
}
|
||||
}
|
||||
|
||||
// Switch to the correct worktree based on the feature's branchName
|
||||
if (feature.branchName && deepLinkWorktrees.length > 0) {
|
||||
const targetWorktree = deepLinkWorktrees.find((w) => w.branch === feature.branchName);
|
||||
// If the feature has a branch and worktrees aren't available yet, wait briefly.
|
||||
// After enough retries, proceed without switching worktree to avoid hanging.
|
||||
const MAX_DEEP_LINK_RETRIES = 10;
|
||||
if (feature.branchName && resolvedWorktrees.length === 0) {
|
||||
deepLinkRetryCountRef.current++;
|
||||
if (deepLinkRetryCountRef.current < MAX_DEEP_LINK_RETRIES) {
|
||||
return; // Worktrees not loaded yet - effect will re-run when they load
|
||||
}
|
||||
// Exceeded retry limit — proceed without worktree switch to avoid hanging
|
||||
logger.warn(
|
||||
`Deep link: worktrees not available after ${MAX_DEEP_LINK_RETRIES} retries, ` +
|
||||
`opening feature ${initialFeatureId} without switching worktree`
|
||||
);
|
||||
}
|
||||
|
||||
// Switch to the correct worktree based on the feature's branchName.
|
||||
// IMPORTANT: Wrap in startTransition to batch the Zustand store update with
|
||||
// any concurrent React state updates. Without this, the synchronous store
|
||||
// mutation cascades through useAutoMode → refreshStatus → setAutoModeRunning,
|
||||
// which can trigger React error #185 on mobile Safari/PWA crash loops.
|
||||
if (feature.branchName && resolvedWorktrees.length > 0) {
|
||||
const targetWorktree = resolvedWorktrees.find((w) => w.branch === feature.branchName);
|
||||
if (targetWorktree) {
|
||||
const currentWt = useAppStore.getState().getCurrentWorktree(currentProject.path);
|
||||
const isAlreadySelected = targetWorktree.isMain
|
||||
@@ -356,23 +448,27 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
logger.info(
|
||||
`Deep link: switching to worktree "${targetWorktree.branch}" for feature ${initialFeatureId}`
|
||||
);
|
||||
setCurrentWorktree(
|
||||
currentProject.path,
|
||||
targetWorktree.isMain ? null : targetWorktree.path,
|
||||
targetWorktree.branch
|
||||
);
|
||||
startTransition(() => {
|
||||
setCurrentWorktree(
|
||||
currentProject.path,
|
||||
targetWorktree.isMain ? null : targetWorktree.path,
|
||||
targetWorktree.branch
|
||||
);
|
||||
});
|
||||
}
|
||||
}
|
||||
} else if (!feature.branchName && deepLinkWorktrees.length > 0) {
|
||||
} else if (!feature.branchName && resolvedWorktrees.length > 0) {
|
||||
// Feature has no branch - should be on the main worktree
|
||||
const currentWt = useAppStore.getState().getCurrentWorktree(currentProject.path);
|
||||
if (currentWt?.path !== null && currentWt !== null) {
|
||||
const mainWorktree = deepLinkWorktrees.find((w) => w.isMain);
|
||||
const mainWorktree = resolvedWorktrees.find((w) => w.isMain);
|
||||
if (mainWorktree) {
|
||||
logger.info(
|
||||
`Deep link: switching to main worktree for unassigned feature ${initialFeatureId}`
|
||||
);
|
||||
setCurrentWorktree(currentProject.path, null, mainWorktree.branch);
|
||||
startTransition(() => {
|
||||
setCurrentWorktree(currentProject.path, null, mainWorktree.branch);
|
||||
});
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -387,6 +483,7 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
hookFeatures,
|
||||
currentProject?.path,
|
||||
deepLinkWorktrees,
|
||||
queryClient,
|
||||
setCurrentWorktree,
|
||||
setOutputFeature,
|
||||
setShowOutputModal,
|
||||
@@ -764,11 +861,15 @@ export function BoardView({ initialFeatureId }: BoardViewProps) {
|
||||
|
||||
// Recovery handler for BoardErrorBoundary: reset worktree selection to main
|
||||
// so the board can re-render without the stale worktree state that caused the crash.
|
||||
// Wrapped in startTransition to batch with concurrent React updates and avoid
|
||||
// triggering another cascade during recovery.
|
||||
const handleBoardRecover = useCallback(() => {
|
||||
if (!currentProject) return;
|
||||
const mainWorktree = worktrees.find((w) => w.isMain);
|
||||
const mainBranch = mainWorktree?.branch || 'main';
|
||||
setCurrentWorktree(currentProject.path, null, mainBranch);
|
||||
startTransition(() => {
|
||||
setCurrentWorktree(currentProject.path, null, mainBranch);
|
||||
});
|
||||
}, [currentProject, worktrees, setCurrentWorktree]);
|
||||
|
||||
// Helper function to add and select a worktree
|
||||
|
||||
@@ -14,6 +14,63 @@ import { useFeature, useAgentOutput } from '@/hooks/queries';
|
||||
import { queryKeys } from '@/lib/query-keys';
|
||||
import { getFirstNonEmptySummary } from '@/lib/summary-selection';
|
||||
import { useAppStore } from '@/store/app-store';
|
||||
import { isMobileDevice } from '@/lib/mobile-detect';
|
||||
|
||||
// Global concurrency control for mobile mount staggering.
|
||||
// When many AgentInfoPanel instances mount simultaneously (e.g., worktree switch
|
||||
// with 50+ cards), we spread queries over a wider window and cap how many
|
||||
// panels can be querying concurrently to prevent mobile Safari crashes.
|
||||
//
|
||||
// The mechanism works in two layers:
|
||||
// 1. Random delay (0-6s) - spreads mount times so not all panels try to query at once
|
||||
// 2. Concurrency slots (max 4) - even after the delay, only N panels can query simultaneously
|
||||
//
|
||||
// Instance tracking ensures the queue resets if all panels unmount (e.g., navigation).
|
||||
const MOBILE_MAX_CONCURRENT_QUERIES = 4;
|
||||
const MOBILE_STAGGER_WINDOW_MS = 6000; // 6s window (vs previous 2s)
|
||||
let activeMobileQueryCount = 0;
|
||||
let pendingMobileQueue: Array<() => void> = [];
|
||||
let mountedPanelCount = 0;
|
||||
|
||||
function acquireMobileQuerySlot(): Promise<void> {
|
||||
if (!isMobileDevice) return Promise.resolve();
|
||||
if (activeMobileQueryCount < MOBILE_MAX_CONCURRENT_QUERIES) {
|
||||
activeMobileQueryCount++;
|
||||
return Promise.resolve();
|
||||
}
|
||||
return new Promise<void>((resolve) => {
|
||||
pendingMobileQueue.push(() => {
|
||||
activeMobileQueryCount++;
|
||||
resolve();
|
||||
});
|
||||
});
|
||||
}
|
||||
|
||||
function releaseMobileQuerySlot(): void {
|
||||
if (!isMobileDevice) return;
|
||||
activeMobileQueryCount = Math.max(0, activeMobileQueryCount - 1);
|
||||
const next = pendingMobileQueue.shift();
|
||||
if (next) next();
|
||||
}
|
||||
|
||||
function trackPanelMount(): void {
|
||||
if (!isMobileDevice) return;
|
||||
mountedPanelCount++;
|
||||
}
|
||||
|
||||
function trackPanelUnmount(): void {
|
||||
if (!isMobileDevice) return;
|
||||
mountedPanelCount = Math.max(0, mountedPanelCount - 1);
|
||||
// If all panels unmounted (e.g., navigated away from board or worktree switch),
|
||||
// reset the queue to prevent stale state from blocking future mounts.
|
||||
if (mountedPanelCount === 0) {
|
||||
activeMobileQueryCount = 0;
|
||||
// Drain any pending callbacks so their Promises resolve (components already unmounted)
|
||||
const pending = pendingMobileQueue;
|
||||
pendingMobileQueue = [];
|
||||
for (const cb of pending) cb();
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Formats thinking level for compact display
|
||||
@@ -66,6 +123,12 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
|
||||
const [isSummaryDialogOpen, setIsSummaryDialogOpen] = useState(false);
|
||||
const [isTodosExpanded, setIsTodosExpanded] = useState(false);
|
||||
|
||||
// Track mounted panel count for global queue reset on full unmount
|
||||
useEffect(() => {
|
||||
trackPanelMount();
|
||||
return () => trackPanelUnmount();
|
||||
}, []);
|
||||
|
||||
// Get providers from store for provider-aware model name display
|
||||
// This allows formatModelName to show provider-specific model names (e.g., "GLM 4.7" instead of "Sonnet 4.5")
|
||||
// when a feature was executed using a Claude-compatible provider
|
||||
@@ -92,6 +155,41 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
|
||||
// Determine if we should poll for updates
|
||||
const shouldFetchData = feature.status !== 'backlog' && feature.status !== 'merge_conflict';
|
||||
|
||||
// On mobile, stagger initial per-card queries to prevent a mount storm.
|
||||
// When a worktree loads with many cards, all AgentInfoPanel instances mount
|
||||
// simultaneously. Without staggering, each card fires useFeature + useAgentOutput
|
||||
// queries at the same time, creating 60-100+ concurrent API calls that crash
|
||||
// mobile Safari. Actively running cards fetch immediately (priority data);
|
||||
// other cards defer by a random delay AND wait for a concurrency slot.
|
||||
// The stagger window is 6s (vs previous 2s) to spread load for worktrees
|
||||
// with 50+ features. The concurrency limiter caps active queries to 4 at a time,
|
||||
// preventing the burst that overwhelms mobile Safari's connection handling.
|
||||
const [mountReady, setMountReady] = useState(!isMobileDevice || !!isActivelyRunning);
|
||||
useEffect(() => {
|
||||
if (mountReady) return;
|
||||
let cancelled = false;
|
||||
const delay = Math.random() * MOBILE_STAGGER_WINDOW_MS;
|
||||
const timer = setTimeout(() => {
|
||||
// After the random delay, also wait for a concurrency slot
|
||||
acquireMobileQuerySlot().then(() => {
|
||||
if (!cancelled) {
|
||||
setMountReady(true);
|
||||
// Release the slot after a brief window to let the initial queries fire
|
||||
// and return, preventing all slots from being held indefinitely
|
||||
setTimeout(releaseMobileQuerySlot, 3000);
|
||||
} else {
|
||||
releaseMobileQuerySlot();
|
||||
}
|
||||
});
|
||||
}, delay);
|
||||
return () => {
|
||||
cancelled = true;
|
||||
clearTimeout(timer);
|
||||
};
|
||||
}, [mountReady]);
|
||||
|
||||
const queryEnabled = shouldFetchData && mountReady;
|
||||
|
||||
// Track whether we're receiving WebSocket events (within threshold)
|
||||
// Use a state to trigger re-renders when the WebSocket connection becomes stale
|
||||
const [isReceivingWsEvents, setIsReceivingWsEvents] = useState(false);
|
||||
@@ -142,34 +240,72 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
|
||||
|
||||
// Fetch fresh feature data for planSpec (store data can be stale for task progress)
|
||||
const { data: freshFeature } = useFeature(projectPath, feature.id, {
|
||||
enabled: shouldFetchData && !contextContent,
|
||||
enabled: queryEnabled && !contextContent,
|
||||
pollingInterval,
|
||||
});
|
||||
|
||||
// Fetch agent output for parsing
|
||||
const { data: agentOutputContent } = useAgentOutput(projectPath, feature.id, {
|
||||
enabled: shouldFetchData && !contextContent,
|
||||
enabled: queryEnabled && !contextContent,
|
||||
pollingInterval,
|
||||
});
|
||||
|
||||
// On mount, ensure feature and agent output queries are fresh.
|
||||
// This handles the worktree switch scenario where cards unmount when filtered out
|
||||
// and remount when the user switches back. Without this, the React Query cache
|
||||
// may serve stale data (or no data) for the individual feature query, causing
|
||||
// the todo list to appear empty until the next polling cycle.
|
||||
// may serve stale data for the individual feature query, causing the todo list
|
||||
// to appear empty until the next polling cycle.
|
||||
//
|
||||
// IMPORTANT: Only invalidate if the cached data EXISTS and is STALE.
|
||||
// During worktree switches, ALL cards in the new worktree remount simultaneously.
|
||||
// If every card fires invalidateQueries(), it creates a query storm (40-100+
|
||||
// concurrent invalidations) that overwhelms React's rendering pipeline on mobile
|
||||
// Safari/PWA, causing crashes. The key insight: if a query has NEVER been fetched
|
||||
// (no dataUpdatedAt), there's nothing stale to invalidate — the useFeature/
|
||||
// useAgentOutput hooks will fetch fresh data when their `enabled` flag is true.
|
||||
// We only need to invalidate when cached data exists but is outdated.
|
||||
//
|
||||
// On mobile, skip mount-time invalidation entirely. The staggered useFeature/
|
||||
// useAgentOutput queries already fetch fresh data — invalidation is redundant
|
||||
// and creates the exact query storm we're trying to prevent. The stale threshold
|
||||
// is also higher on mobile (30s vs 10s) to further reduce unnecessary refetches
|
||||
// during the settling period after a worktree switch.
|
||||
useEffect(() => {
|
||||
if (shouldFetchData && projectPath && feature.id && !contextContent) {
|
||||
// Invalidate both the single feature and agent output queries to trigger immediate refetch
|
||||
queryClient.invalidateQueries({
|
||||
queryKey: queryKeys.features.single(projectPath, feature.id),
|
||||
});
|
||||
queryClient.invalidateQueries({
|
||||
queryKey: queryKeys.features.agentOutput(projectPath, feature.id),
|
||||
});
|
||||
if (queryEnabled && projectPath && feature.id && !contextContent) {
|
||||
// On mobile, skip mount-time invalidation — the useFeature/useAgentOutput
|
||||
// hooks will handle the initial fetch after the stagger delay.
|
||||
if (isMobileDevice) return;
|
||||
|
||||
const MOUNT_STALE_THRESHOLD = 10_000; // 10s — skip invalidation if data is fresh
|
||||
const now = Date.now();
|
||||
|
||||
const featureQuery = queryClient.getQueryState(
|
||||
queryKeys.features.single(projectPath, feature.id)
|
||||
);
|
||||
const agentOutputQuery = queryClient.getQueryState(
|
||||
queryKeys.features.agentOutput(projectPath, feature.id)
|
||||
);
|
||||
|
||||
// Only invalidate queries that have cached data AND are stale.
|
||||
// Skip if the query has never been fetched (dataUpdatedAt is undefined) —
|
||||
// the useFeature/useAgentOutput hooks will handle the initial fetch.
|
||||
if (featureQuery?.dataUpdatedAt && now - featureQuery.dataUpdatedAt > MOUNT_STALE_THRESHOLD) {
|
||||
queryClient.invalidateQueries({
|
||||
queryKey: queryKeys.features.single(projectPath, feature.id),
|
||||
});
|
||||
}
|
||||
if (
|
||||
agentOutputQuery?.dataUpdatedAt &&
|
||||
now - agentOutputQuery.dataUpdatedAt > MOUNT_STALE_THRESHOLD
|
||||
) {
|
||||
queryClient.invalidateQueries({
|
||||
queryKey: queryKeys.features.agentOutput(projectPath, feature.id),
|
||||
});
|
||||
}
|
||||
}
|
||||
// Only run on mount (feature.id and projectPath identify this specific card instance)
|
||||
// Runs when mount staggering completes (queryEnabled becomes true) or on initial mount
|
||||
// eslint-disable-next-line react-hooks/exhaustive-deps
|
||||
}, [feature.id, projectPath]);
|
||||
}, [queryEnabled, feature.id, projectPath]);
|
||||
|
||||
// Parse agent output into agentInfo
|
||||
const agentInfo = useMemo(() => {
|
||||
|
||||
@@ -1,6 +1,5 @@
|
||||
// @ts-nocheck - feature update logic with partial updates and image/file handling
|
||||
import { useCallback } from 'react';
|
||||
import { useQueryClient } from '@tanstack/react-query';
|
||||
import {
|
||||
Feature,
|
||||
FeatureImage,
|
||||
@@ -18,7 +17,10 @@ import { useVerifyFeature, useResumeFeature } from '@/hooks/mutations';
|
||||
import { truncateDescription } from '@/lib/utils';
|
||||
import { getBlockingDependencies } from '@automaker/dependency-resolver';
|
||||
import { createLogger } from '@automaker/utils/logger';
|
||||
import { queryKeys } from '@/lib/query-keys';
|
||||
import {
|
||||
markFeatureTransitioning,
|
||||
unmarkFeatureTransitioning,
|
||||
} from '@/lib/feature-transition-state';
|
||||
|
||||
const logger = createLogger('BoardActions');
|
||||
|
||||
@@ -116,8 +118,6 @@ export function useBoardActions({
|
||||
currentWorktreeBranch,
|
||||
stopFeature,
|
||||
}: UseBoardActionsProps) {
|
||||
const queryClient = useQueryClient();
|
||||
|
||||
// IMPORTANT: Use individual selectors instead of bare useAppStore() to prevent
|
||||
// subscribing to the entire store. Bare useAppStore() causes the host component
|
||||
// (BoardView) to re-render on EVERY store change, which cascades through effects
|
||||
@@ -125,7 +125,6 @@ export function useBoardActions({
|
||||
const addFeature = useAppStore((s) => s.addFeature);
|
||||
const updateFeature = useAppStore((s) => s.updateFeature);
|
||||
const removeFeature = useAppStore((s) => s.removeFeature);
|
||||
const moveFeature = useAppStore((s) => s.moveFeature);
|
||||
const worktreesEnabled = useAppStore((s) => s.useWorktrees);
|
||||
const enableDependencyBlocking = useAppStore((s) => s.enableDependencyBlocking);
|
||||
const skipVerificationInAutoMode = useAppStore((s) => s.skipVerificationInAutoMode);
|
||||
@@ -707,8 +706,7 @@ export function useBoardActions({
|
||||
try {
|
||||
const result = await verifyFeatureMutation.mutateAsync(feature.id);
|
||||
if (result.passes) {
|
||||
// Immediately move card to verified column (optimistic update)
|
||||
moveFeature(feature.id, 'verified');
|
||||
// persistFeatureUpdate handles the optimistic RQ cache update internally
|
||||
persistFeatureUpdate(feature.id, {
|
||||
status: 'verified',
|
||||
justFinishedAt: undefined,
|
||||
@@ -725,7 +723,7 @@ export function useBoardActions({
|
||||
// Error toast is already shown by the mutation's onError handler
|
||||
}
|
||||
},
|
||||
[currentProject, verifyFeatureMutation, moveFeature, persistFeatureUpdate]
|
||||
[currentProject, verifyFeatureMutation, persistFeatureUpdate]
|
||||
);
|
||||
|
||||
const handleResumeFeature = useCallback(
|
||||
@@ -742,7 +740,6 @@ export function useBoardActions({
|
||||
|
||||
const handleManualVerify = useCallback(
|
||||
(feature: Feature) => {
|
||||
moveFeature(feature.id, 'verified');
|
||||
persistFeatureUpdate(feature.id, {
|
||||
status: 'verified',
|
||||
justFinishedAt: undefined,
|
||||
@@ -751,7 +748,7 @@ export function useBoardActions({
|
||||
description: `Marked as verified: ${truncateDescription(feature.description)}`,
|
||||
});
|
||||
},
|
||||
[moveFeature, persistFeatureUpdate]
|
||||
[persistFeatureUpdate]
|
||||
);
|
||||
|
||||
const handleMoveBackToInProgress = useCallback(
|
||||
@@ -760,13 +757,12 @@ export function useBoardActions({
|
||||
status: 'in_progress' as const,
|
||||
startedAt: new Date().toISOString(),
|
||||
};
|
||||
updateFeature(feature.id, updates);
|
||||
persistFeatureUpdate(feature.id, updates);
|
||||
toast.info('Feature moved back', {
|
||||
description: `Moved back to In Progress: ${truncateDescription(feature.description)}`,
|
||||
});
|
||||
},
|
||||
[updateFeature, persistFeatureUpdate]
|
||||
[persistFeatureUpdate]
|
||||
);
|
||||
|
||||
const handleOpenFollowUp = useCallback(
|
||||
@@ -885,7 +881,6 @@ export function useBoardActions({
|
||||
);
|
||||
|
||||
if (result.success) {
|
||||
moveFeature(feature.id, 'verified');
|
||||
persistFeatureUpdate(feature.id, { status: 'verified' });
|
||||
toast.success('Feature committed', {
|
||||
description: `Committed and verified: ${truncateDescription(feature.description)}`,
|
||||
@@ -907,7 +902,7 @@ export function useBoardActions({
|
||||
await loadFeatures();
|
||||
}
|
||||
},
|
||||
[currentProject, moveFeature, persistFeatureUpdate, loadFeatures, onWorktreeCreated]
|
||||
[currentProject, persistFeatureUpdate, loadFeatures, onWorktreeCreated]
|
||||
);
|
||||
|
||||
const handleMergeFeature = useCallback(
|
||||
@@ -951,17 +946,12 @@ export function useBoardActions({
|
||||
|
||||
const handleCompleteFeature = useCallback(
|
||||
(feature: Feature) => {
|
||||
const updates = {
|
||||
status: 'completed' as const,
|
||||
};
|
||||
updateFeature(feature.id, updates);
|
||||
persistFeatureUpdate(feature.id, updates);
|
||||
|
||||
persistFeatureUpdate(feature.id, { status: 'completed' as const });
|
||||
toast.success('Feature completed', {
|
||||
description: `Archived: ${truncateDescription(feature.description)}`,
|
||||
});
|
||||
},
|
||||
[updateFeature, persistFeatureUpdate]
|
||||
[persistFeatureUpdate]
|
||||
);
|
||||
|
||||
const handleUnarchiveFeature = useCallback(
|
||||
@@ -978,11 +968,7 @@ export function useBoardActions({
|
||||
(projectPath ? isPrimaryWorktreeBranch(projectPath, currentWorktreeBranch) : true)
|
||||
: featureBranch === currentWorktreeBranch;
|
||||
|
||||
const updates: Partial<Feature> = {
|
||||
status: 'verified' as const,
|
||||
};
|
||||
updateFeature(feature.id, updates);
|
||||
persistFeatureUpdate(feature.id, updates);
|
||||
persistFeatureUpdate(feature.id, { status: 'verified' as const });
|
||||
|
||||
if (willBeVisibleOnCurrentView) {
|
||||
toast.success('Feature restored', {
|
||||
@@ -994,13 +980,7 @@ export function useBoardActions({
|
||||
});
|
||||
}
|
||||
},
|
||||
[
|
||||
updateFeature,
|
||||
persistFeatureUpdate,
|
||||
currentWorktreeBranch,
|
||||
projectPath,
|
||||
isPrimaryWorktreeBranch,
|
||||
]
|
||||
[persistFeatureUpdate, currentWorktreeBranch, projectPath, isPrimaryWorktreeBranch]
|
||||
);
|
||||
|
||||
const handleViewOutput = useCallback(
|
||||
@@ -1031,6 +1011,13 @@ export function useBoardActions({
|
||||
|
||||
const handleForceStopFeature = useCallback(
|
||||
async (feature: Feature) => {
|
||||
// Mark this feature as transitioning so WebSocket-driven query invalidation
|
||||
// (useAutoModeQueryInvalidation) skips redundant cache invalidations while
|
||||
// persistFeatureUpdate is handling the optimistic update. Without this guard,
|
||||
// auto_mode_error / auto_mode_stopped WS events race with the optimistic
|
||||
// update and cause cache flip-flops that cascade through useBoardColumnFeatures,
|
||||
// triggering React error #185 on mobile.
|
||||
markFeatureTransitioning(feature.id);
|
||||
try {
|
||||
await stopFeature(feature.id);
|
||||
|
||||
@@ -1048,25 +1035,11 @@ export function useBoardActions({
|
||||
removeRunningTaskFromAllWorktrees(currentProject.id, feature.id);
|
||||
}
|
||||
|
||||
// Optimistically update the React Query features cache so the board
|
||||
// moves the card immediately. Without this, the card stays in
|
||||
// "in_progress" until the next poll cycle (30s) because the async
|
||||
// refetch races with the persistFeatureUpdate write.
|
||||
if (currentProject) {
|
||||
queryClient.setQueryData(
|
||||
queryKeys.features.all(currentProject.path),
|
||||
(oldFeatures: Feature[] | undefined) => {
|
||||
if (!oldFeatures) return oldFeatures;
|
||||
return oldFeatures.map((f) =>
|
||||
f.id === feature.id ? { ...f, status: targetStatus } : f
|
||||
);
|
||||
}
|
||||
);
|
||||
}
|
||||
|
||||
if (targetStatus !== feature.status) {
|
||||
moveFeature(feature.id, targetStatus);
|
||||
// Must await to ensure file is written before user can restart
|
||||
// persistFeatureUpdate handles the optimistic RQ cache update, the
|
||||
// Zustand store update (on server response), and the final cache
|
||||
// invalidation internally — no need for separate queryClient.setQueryData
|
||||
// or moveFeature calls which would cause redundant re-renders.
|
||||
await persistFeatureUpdate(feature.id, { status: targetStatus });
|
||||
}
|
||||
|
||||
@@ -1083,9 +1056,15 @@ export function useBoardActions({
|
||||
toast.error('Failed to stop agent', {
|
||||
description: error instanceof Error ? error.message : 'An error occurred',
|
||||
});
|
||||
} finally {
|
||||
// Delay unmarking so the refetch triggered by persistFeatureUpdate's
|
||||
// invalidateQueries() has time to settle before WS-driven invalidations
|
||||
// are allowed through again. Without this, a WS event arriving during
|
||||
// the refetch window would trigger a conflicting invalidation.
|
||||
setTimeout(() => unmarkFeatureTransitioning(feature.id), 500);
|
||||
}
|
||||
},
|
||||
[stopFeature, moveFeature, persistFeatureUpdate, currentProject, queryClient]
|
||||
[stopFeature, persistFeatureUpdate, currentProject]
|
||||
);
|
||||
|
||||
const handleStartNextFeatures = useCallback(async () => {
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// @ts-nocheck - column filtering logic with dependency resolution and status mapping
|
||||
import { useMemo, useCallback, useEffect, useRef } from 'react';
|
||||
import { useMemo, useCallback, useEffect } from 'react';
|
||||
import { Feature, useAppStore } from '@/store/app-store';
|
||||
import {
|
||||
createFeatureMap,
|
||||
@@ -177,9 +177,6 @@ export function useBoardColumnFeatures({
|
||||
(state) => state.clearRecentlyCompletedFeatures
|
||||
);
|
||||
|
||||
// Track previous feature IDs to detect when features list has been refreshed
|
||||
const prevFeatureIdsRef = useRef<Set<string>>(new Set());
|
||||
|
||||
// Clear recently completed features when the cache refreshes with updated statuses.
|
||||
//
|
||||
// RACE CONDITION SCENARIO THIS PREVENTS:
|
||||
@@ -193,12 +190,16 @@ export function useBoardColumnFeatures({
|
||||
//
|
||||
// When the refetch completes with fresh data (status='verified'/'completed'),
|
||||
// this effect clears the recentlyCompletedFeatures set since it's no longer needed.
|
||||
// Clear recently completed features when the cache refreshes with updated statuses.
|
||||
// IMPORTANT: Only depend on `features` (not `recentlyCompletedFeatures`) to avoid a
|
||||
// re-trigger loop where clearing the set creates a new reference that re-fires this effect.
|
||||
// Read recentlyCompletedFeatures from the store directly to get the latest value without
|
||||
// subscribing to it as a dependency.
|
||||
useEffect(() => {
|
||||
const currentIds = new Set(features.map((f) => f.id));
|
||||
const currentRecentlyCompleted = useAppStore.getState().recentlyCompletedFeatures;
|
||||
if (currentRecentlyCompleted.size === 0) return;
|
||||
|
||||
// Check if any recently completed features now have terminal statuses in the new data
|
||||
// If so, we can clear the tracking since the cache is now fresh
|
||||
const hasUpdatedStatus = Array.from(recentlyCompletedFeatures).some((featureId) => {
|
||||
const hasUpdatedStatus = Array.from(currentRecentlyCompleted).some((featureId) => {
|
||||
const feature = features.find((f) => f.id === featureId);
|
||||
return feature && (feature.status === 'verified' || feature.status === 'completed');
|
||||
});
|
||||
@@ -206,9 +207,7 @@ export function useBoardColumnFeatures({
|
||||
if (hasUpdatedStatus) {
|
||||
clearRecentlyCompletedFeatures();
|
||||
}
|
||||
|
||||
prevFeatureIdsRef.current = currentIds;
|
||||
}, [features, recentlyCompletedFeatures, clearRecentlyCompletedFeatures]);
|
||||
}, [features, clearRecentlyCompletedFeatures]);
|
||||
|
||||
// Memoize column features to prevent unnecessary re-renders
|
||||
const columnFeaturesMap = useMemo(() => {
|
||||
|
||||
@@ -38,7 +38,6 @@ export function useBoardDragDrop({
|
||||
// subscribing to the entire store. Bare useAppStore() causes the host component
|
||||
// (BoardView) to re-render on EVERY store change, which cascades through effects
|
||||
// and triggers React error #185 (maximum update depth exceeded).
|
||||
const moveFeature = useAppStore((s) => s.moveFeature);
|
||||
const updateFeature = useAppStore((s) => s.updateFeature);
|
||||
|
||||
// Note: getOrCreateWorktreeForFeature removed - worktrees are now created server-side
|
||||
@@ -207,7 +206,8 @@ export function useBoardDragDrop({
|
||||
if (targetStatus === draggedFeature.status) return;
|
||||
|
||||
// Handle different drag scenarios
|
||||
// Note: Worktrees are created server-side at execution time based on feature.branchName
|
||||
// Note: persistFeatureUpdate handles optimistic RQ cache update internally,
|
||||
// so no separate moveFeature() call is needed.
|
||||
if (draggedFeature.status === 'backlog' || draggedFeature.status === 'merge_conflict') {
|
||||
// From backlog
|
||||
if (targetStatus === 'in_progress') {
|
||||
@@ -215,7 +215,6 @@ export function useBoardDragDrop({
|
||||
// Server will derive workDir from feature.branchName
|
||||
await handleStartImplementation(draggedFeature);
|
||||
} else {
|
||||
moveFeature(featureId, targetStatus);
|
||||
persistFeatureUpdate(featureId, { status: targetStatus });
|
||||
}
|
||||
} else if (draggedFeature.status === 'waiting_approval') {
|
||||
@@ -223,7 +222,6 @@ export function useBoardDragDrop({
|
||||
// NOTE: This check must come BEFORE skipTests check because waiting_approval
|
||||
// features often have skipTests=true, and we want status-based handling first
|
||||
if (targetStatus === 'verified') {
|
||||
moveFeature(featureId, 'verified');
|
||||
// Clear justFinishedAt timestamp when manually verifying via drag
|
||||
persistFeatureUpdate(featureId, {
|
||||
status: 'verified',
|
||||
@@ -237,7 +235,6 @@ export function useBoardDragDrop({
|
||||
});
|
||||
} else if (targetStatus === 'backlog') {
|
||||
// Allow moving waiting_approval cards back to backlog
|
||||
moveFeature(featureId, 'backlog');
|
||||
// Clear justFinishedAt timestamp when moving back to backlog
|
||||
persistFeatureUpdate(featureId, {
|
||||
status: 'backlog',
|
||||
@@ -269,7 +266,6 @@ export function useBoardDragDrop({
|
||||
});
|
||||
}
|
||||
}
|
||||
moveFeature(featureId, 'backlog');
|
||||
persistFeatureUpdate(featureId, { status: 'backlog' });
|
||||
toast.info(
|
||||
isRunningTask
|
||||
@@ -291,7 +287,6 @@ export function useBoardDragDrop({
|
||||
return;
|
||||
} else if (targetStatus === 'verified' && draggedFeature.skipTests) {
|
||||
// Manual verify via drag (only for skipTests features)
|
||||
moveFeature(featureId, 'verified');
|
||||
persistFeatureUpdate(featureId, { status: 'verified' });
|
||||
toast.success('Feature verified', {
|
||||
description: `Marked as verified: ${draggedFeature.description.slice(
|
||||
@@ -304,7 +299,6 @@ export function useBoardDragDrop({
|
||||
// skipTests feature being moved between verified and waiting_approval
|
||||
if (targetStatus === 'waiting_approval' && draggedFeature.status === 'verified') {
|
||||
// Move verified feature back to waiting_approval
|
||||
moveFeature(featureId, 'waiting_approval');
|
||||
persistFeatureUpdate(featureId, { status: 'waiting_approval' });
|
||||
toast.info('Feature moved back', {
|
||||
description: `Moved back to Waiting Approval: ${draggedFeature.description.slice(
|
||||
@@ -314,7 +308,6 @@ export function useBoardDragDrop({
|
||||
});
|
||||
} else if (targetStatus === 'backlog') {
|
||||
// Allow moving skipTests cards back to backlog (from verified)
|
||||
moveFeature(featureId, 'backlog');
|
||||
persistFeatureUpdate(featureId, { status: 'backlog' });
|
||||
toast.info('Feature moved to backlog', {
|
||||
description: `Moved to Backlog: ${draggedFeature.description.slice(
|
||||
@@ -327,7 +320,6 @@ export function useBoardDragDrop({
|
||||
// Handle verified TDD (non-skipTests) features being moved back
|
||||
if (targetStatus === 'waiting_approval') {
|
||||
// Move verified feature back to waiting_approval
|
||||
moveFeature(featureId, 'waiting_approval');
|
||||
persistFeatureUpdate(featureId, { status: 'waiting_approval' });
|
||||
toast.info('Feature moved back', {
|
||||
description: `Moved back to Waiting Approval: ${draggedFeature.description.slice(
|
||||
@@ -337,7 +329,6 @@ export function useBoardDragDrop({
|
||||
});
|
||||
} else if (targetStatus === 'backlog') {
|
||||
// Allow moving verified cards back to backlog
|
||||
moveFeature(featureId, 'backlog');
|
||||
persistFeatureUpdate(featureId, { status: 'backlog' });
|
||||
toast.info('Feature moved to backlog', {
|
||||
description: `Moved to Backlog: ${draggedFeature.description.slice(
|
||||
@@ -351,7 +342,6 @@ export function useBoardDragDrop({
|
||||
[
|
||||
features,
|
||||
runningAutoTasks,
|
||||
moveFeature,
|
||||
updateFeature,
|
||||
persistFeatureUpdate,
|
||||
handleStartImplementation,
|
||||
|
||||
@@ -32,11 +32,7 @@ export function useBoardFeatures({ currentProject }: UseBoardFeaturesProps) {
|
||||
const isRestoring = useIsRestoring();
|
||||
|
||||
// Use React Query for features
|
||||
const {
|
||||
data: features = [],
|
||||
isLoading: isQueryLoading,
|
||||
refetch: loadFeatures,
|
||||
} = useFeatures(currentProject?.path);
|
||||
const { data: features = [], isLoading: isQueryLoading } = useFeatures(currentProject?.path);
|
||||
|
||||
// Don't report loading while IDB cache restore is in progress —
|
||||
// features will appear momentarily once the restore completes.
|
||||
@@ -91,37 +87,22 @@ export function useBoardFeatures({ currentProject }: UseBoardFeaturesProps) {
|
||||
);
|
||||
|
||||
// Subscribe to auto mode events for notifications (ding sound, toasts)
|
||||
// Note: Query invalidation is handled by useAutoModeQueryInvalidation in the root
|
||||
// Note: Query invalidation is handled by useAutoModeQueryInvalidation in the root.
|
||||
// Note: removeRunningTask is handled by useAutoMode — do NOT duplicate it here,
|
||||
// as duplicate Zustand mutations cause re-render cascades (React error #185).
|
||||
useEffect(() => {
|
||||
const api = getElectronAPI();
|
||||
if (!api?.autoMode || !currentProject) return;
|
||||
|
||||
const { removeRunningTask } = useAppStore.getState();
|
||||
const projectId = currentProject.id;
|
||||
const projectPath = currentProject.path;
|
||||
|
||||
const unsubscribe = api.autoMode.onEvent((event) => {
|
||||
// Check if event is for the current project by matching projectPath
|
||||
const eventProjectPath = ('projectPath' in event && event.projectPath) as string | undefined;
|
||||
if (eventProjectPath && eventProjectPath !== projectPath) {
|
||||
// Event is for a different project, ignore it
|
||||
logger.debug(
|
||||
`Ignoring auto mode event for different project: ${eventProjectPath} (current: ${projectPath})`
|
||||
);
|
||||
return;
|
||||
}
|
||||
|
||||
// Use event's projectPath or projectId if available, otherwise use current project
|
||||
// Board view only reacts to events for the currently selected project
|
||||
const eventProjectId = ('projectId' in event && event.projectId) || projectId;
|
||||
|
||||
// NOTE: auto_mode_feature_start and auto_mode_feature_complete are NOT handled here
|
||||
// for feature list reloading. That is handled by useAutoModeQueryInvalidation which
|
||||
// invalidates the features.all query on those events. Duplicate invalidation here
|
||||
// caused a re-render cascade through DndContext that triggered React error #185
|
||||
// (maximum update depth exceeded), crashing the board view with an infinite spinner
|
||||
// when a new feature was added and moved to in_progress.
|
||||
|
||||
if (event.type === 'auto_mode_feature_complete') {
|
||||
// Play ding sound when feature is done (unless muted)
|
||||
const { muteDoneSound } = useAppStore.getState();
|
||||
@@ -130,14 +111,7 @@ export function useBoardFeatures({ currentProject }: UseBoardFeaturesProps) {
|
||||
audio.play().catch((err) => logger.warn('Could not play ding sound:', err));
|
||||
}
|
||||
} else if (event.type === 'auto_mode_error') {
|
||||
// Remove from running tasks
|
||||
if (event.featureId) {
|
||||
const eventBranchName =
|
||||
'branchName' in event && event.branchName !== undefined ? event.branchName : null;
|
||||
removeRunningTask(eventProjectId, eventBranchName, event.featureId);
|
||||
}
|
||||
|
||||
// Show error toast
|
||||
// Show error toast (removeRunningTask is handled by useAutoMode, not here)
|
||||
const isAuthError =
|
||||
event.errorType === 'authentication' ||
|
||||
(event.error &&
|
||||
@@ -159,7 +133,6 @@ export function useBoardFeatures({ currentProject }: UseBoardFeaturesProps) {
|
||||
});
|
||||
|
||||
return unsubscribe;
|
||||
// eslint-disable-next-line react-hooks/exhaustive-deps -- loadFeatures is a stable ref from React Query
|
||||
}, [currentProject]);
|
||||
|
||||
// Check for interrupted features on mount
|
||||
|
||||
@@ -281,6 +281,10 @@ function VirtualizedList<Item extends VirtualListItem>({
|
||||
);
|
||||
}
|
||||
|
||||
// Stable empty Set to use as default prop value. Using `new Set()` inline in
|
||||
// the destructuring creates a new reference on every render, defeating memo.
|
||||
const EMPTY_FEATURE_IDS = new Set<string>();
|
||||
|
||||
export const KanbanBoard = memo(function KanbanBoard({
|
||||
activeFeature,
|
||||
getColumnFeatures,
|
||||
@@ -317,7 +321,7 @@ export const KanbanBoard = memo(function KanbanBoard({
|
||||
onOpenPipelineSettings,
|
||||
isSelectionMode = false,
|
||||
selectionTarget = null,
|
||||
selectedFeatureIds = new Set(),
|
||||
selectedFeatureIds = EMPTY_FEATURE_IDS,
|
||||
onToggleFeatureSelection,
|
||||
onToggleSelectionMode,
|
||||
onAiSuggest,
|
||||
|
||||
@@ -74,6 +74,27 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
// Keep track of whether we've fetched initial logs
|
||||
const hasFetchedInitialLogs = useRef(false);
|
||||
|
||||
// Buffer for batching rapid output events into fewer setState calls.
|
||||
// Content accumulates here and is flushed via requestAnimationFrame,
|
||||
// ensuring at most one React re-render per animation frame (~60fps max).
|
||||
// A fallback setTimeout ensures the buffer is flushed even when RAF is
|
||||
// throttled (e.g., when the tab is in the background).
|
||||
const pendingOutputRef = useRef('');
|
||||
const rafIdRef = useRef<number | null>(null);
|
||||
const timerIdRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
||||
|
||||
const resetPendingOutput = useCallback(() => {
|
||||
if (rafIdRef.current !== null) {
|
||||
cancelAnimationFrame(rafIdRef.current);
|
||||
rafIdRef.current = null;
|
||||
}
|
||||
if (timerIdRef.current !== null) {
|
||||
clearTimeout(timerIdRef.current);
|
||||
timerIdRef.current = null;
|
||||
}
|
||||
pendingOutputRef.current = '';
|
||||
}, []);
|
||||
|
||||
/**
|
||||
* Fetch buffered logs from the server
|
||||
*/
|
||||
@@ -130,6 +151,7 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
* Clear logs and reset state
|
||||
*/
|
||||
const clearLogs = useCallback(() => {
|
||||
resetPendingOutput();
|
||||
setState({
|
||||
logs: '',
|
||||
logsVersion: 0,
|
||||
@@ -144,13 +166,19 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
serverError: null,
|
||||
});
|
||||
hasFetchedInitialLogs.current = false;
|
||||
}, []);
|
||||
}, [resetPendingOutput]);
|
||||
|
||||
const flushPendingOutput = useCallback(() => {
|
||||
// Clear both scheduling handles to prevent duplicate flushes
|
||||
rafIdRef.current = null;
|
||||
if (timerIdRef.current !== null) {
|
||||
clearTimeout(timerIdRef.current);
|
||||
timerIdRef.current = null;
|
||||
}
|
||||
const content = pendingOutputRef.current;
|
||||
if (!content) return;
|
||||
pendingOutputRef.current = '';
|
||||
|
||||
/**
|
||||
* Append content to logs, enforcing a maximum buffer size to prevent
|
||||
* unbounded memory growth and progressive UI lag.
|
||||
*/
|
||||
const appendLogs = useCallback((content: string) => {
|
||||
setState((prev) => {
|
||||
const combined = prev.logs + content;
|
||||
const didTrim = combined.length > MAX_LOG_BUFFER_SIZE;
|
||||
@@ -170,6 +198,48 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
});
|
||||
}, []);
|
||||
|
||||
/**
|
||||
* Append content to logs, enforcing a maximum buffer size to prevent
|
||||
* unbounded memory growth and progressive UI lag.
|
||||
*
|
||||
* Uses requestAnimationFrame to batch rapid output events into at most
|
||||
* one React state update per frame, preventing excessive re-renders.
|
||||
* A fallback setTimeout(250ms) ensures the buffer is flushed even when
|
||||
* RAF is throttled (e.g., when the tab is in the background).
|
||||
* If the pending buffer reaches MAX_LOG_BUFFER_SIZE, flushes immediately
|
||||
* to prevent unbounded memory growth.
|
||||
*/
|
||||
const appendLogs = useCallback(
|
||||
(content: string) => {
|
||||
pendingOutputRef.current += content;
|
||||
|
||||
// Flush immediately if buffer has reached the size limit
|
||||
if (pendingOutputRef.current.length >= MAX_LOG_BUFFER_SIZE) {
|
||||
flushPendingOutput();
|
||||
return;
|
||||
}
|
||||
|
||||
// Schedule a RAF flush if not already scheduled
|
||||
if (rafIdRef.current === null) {
|
||||
rafIdRef.current = requestAnimationFrame(flushPendingOutput);
|
||||
}
|
||||
|
||||
// Schedule a fallback timer flush if not already scheduled,
|
||||
// to handle cases where RAF is throttled (background tab)
|
||||
if (timerIdRef.current === null) {
|
||||
timerIdRef.current = setTimeout(flushPendingOutput, 250);
|
||||
}
|
||||
},
|
||||
[flushPendingOutput]
|
||||
);
|
||||
|
||||
// Clean up pending RAF on unmount to prevent state updates after unmount
|
||||
useEffect(() => {
|
||||
return () => {
|
||||
resetPendingOutput();
|
||||
};
|
||||
}, [resetPendingOutput]);
|
||||
|
||||
// Fetch initial logs when worktreePath changes
|
||||
useEffect(() => {
|
||||
if (worktreePath && autoSubscribe) {
|
||||
@@ -196,6 +266,7 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
|
||||
switch (event.type) {
|
||||
case 'dev-server:started': {
|
||||
resetPendingOutput();
|
||||
const { payload } = event;
|
||||
logger.info('Dev server started:', payload);
|
||||
setState((prev) => ({
|
||||
@@ -245,7 +316,7 @@ export function useDevServerLogs({ worktreePath, autoSubscribe = true }: UseDevS
|
||||
});
|
||||
|
||||
return unsubscribe;
|
||||
}, [worktreePath, autoSubscribe, appendLogs]);
|
||||
}, [worktreePath, autoSubscribe, appendLogs, resetPendingOutput]);
|
||||
|
||||
return {
|
||||
...state,
|
||||
|
||||
@@ -4,13 +4,17 @@ import { getElectronAPI } from '@/lib/electron';
|
||||
import { normalizePath } from '@/lib/utils';
|
||||
import { toast } from 'sonner';
|
||||
import type { DevServerInfo, WorktreeInfo } from '../types';
|
||||
import { useEventRecencyStore } from '@/hooks/use-event-recency';
|
||||
|
||||
const logger = createLogger('DevServers');
|
||||
|
||||
// Timeout (ms) for port detection before showing a warning to the user
|
||||
const PORT_DETECTION_TIMEOUT_MS = 30_000;
|
||||
// Interval (ms) for periodic state reconciliation with the backend
|
||||
const STATE_RECONCILE_INTERVAL_MS = 5_000;
|
||||
// Interval (ms) for periodic state reconciliation with the backend.
|
||||
// 30 seconds is sufficient since WebSocket events handle real-time updates;
|
||||
// reconciliation is only a fallback for missed events (PWA restart, WS gaps).
|
||||
// The previous 5-second interval added unnecessary HTTP pressure.
|
||||
const STATE_RECONCILE_INTERVAL_MS = 30_000;
|
||||
|
||||
interface UseDevServersOptions {
|
||||
projectPath: string;
|
||||
@@ -322,12 +326,24 @@ export function useDevServers({ projectPath }: UseDevServersOptions) {
|
||||
return () => clearInterval(intervalId);
|
||||
}, [clearPortDetectionTimer, startPortDetectionTimer]);
|
||||
|
||||
// Record global events so smart polling knows WebSocket is healthy.
|
||||
// Without this, dev-server events don't suppress polling intervals,
|
||||
// causing all queries (features, worktrees, running-agents) to poll
|
||||
// at their default rates even though the WebSocket is actively connected.
|
||||
const recordGlobalEvent = useEventRecencyStore((state) => state.recordGlobalEvent);
|
||||
|
||||
// Subscribe to all dev server lifecycle events for reactive state updates
|
||||
useEffect(() => {
|
||||
const api = getElectronAPI();
|
||||
if (!api?.worktree?.onDevServerLogEvent) return;
|
||||
|
||||
const unsubscribe = api.worktree.onDevServerLogEvent((event) => {
|
||||
// Record that WS is alive (but only for lifecycle events, not output -
|
||||
// output fires too frequently and would trigger unnecessary store updates)
|
||||
if (event.type !== 'dev-server:output') {
|
||||
recordGlobalEvent();
|
||||
}
|
||||
|
||||
if (event.type === 'dev-server:starting') {
|
||||
const { worktreePath } = event.payload;
|
||||
const key = normalizePath(worktreePath);
|
||||
@@ -424,7 +440,7 @@ export function useDevServers({ projectPath }: UseDevServersOptions) {
|
||||
});
|
||||
|
||||
return unsubscribe;
|
||||
}, [clearPortDetectionTimer, startPortDetectionTimer]);
|
||||
}, [clearPortDetectionTimer, startPortDetectionTimer, recordGlobalEvent]);
|
||||
|
||||
// Cleanup all port detection timers on unmount
|
||||
useEffect(() => {
|
||||
|
||||
@@ -94,8 +94,14 @@ export function NotificationsView() {
|
||||
|
||||
// Navigate to the relevant view based on notification type
|
||||
if (notification.featureId) {
|
||||
// Navigate to board view with feature ID to show output
|
||||
navigate({ to: '/board', search: { featureId: notification.featureId } });
|
||||
// Navigate to board view with feature ID and project path to show output
|
||||
navigate({
|
||||
to: '/board',
|
||||
search: {
|
||||
featureId: notification.featureId,
|
||||
projectPath: notification.projectPath || undefined,
|
||||
},
|
||||
});
|
||||
}
|
||||
},
|
||||
[handleMarkAsRead, navigate]
|
||||
|
||||
@@ -136,8 +136,14 @@ export function RecentActivityFeed({ activities, maxItems = 10 }: RecentActivity
|
||||
upsertAndSetCurrentProject(projectPath, projectName);
|
||||
|
||||
if (activity.featureId) {
|
||||
// Navigate to the specific feature
|
||||
navigate({ to: '/board', search: { featureId: activity.featureId } });
|
||||
// Navigate to the specific feature with project path for deep link handling
|
||||
navigate({
|
||||
to: '/board',
|
||||
search: {
|
||||
featureId: activity.featureId,
|
||||
projectPath: projectPath || undefined,
|
||||
},
|
||||
});
|
||||
} else {
|
||||
navigate({ to: '/board' });
|
||||
}
|
||||
|
||||
@@ -28,33 +28,46 @@ const serverLogger = createLogger('Server');
|
||||
export async function startServer(): Promise<void> {
|
||||
const isDev = !app.isPackaged;
|
||||
|
||||
// Find Node.js executable (handles desktop launcher scenarios)
|
||||
const nodeResult = findNodeExecutable({
|
||||
skipSearch: isDev,
|
||||
logger: (msg: string) => logger.info(msg),
|
||||
});
|
||||
const command = nodeResult.nodePath;
|
||||
|
||||
// Validate that the found Node executable actually exists
|
||||
// systemPathExists is used because node-finder returns system paths
|
||||
if (command !== 'node') {
|
||||
let exists: boolean;
|
||||
try {
|
||||
exists = systemPathExists(command);
|
||||
} catch (error) {
|
||||
const originalError = error instanceof Error ? error.message : String(error);
|
||||
throw new Error(
|
||||
`Failed to verify Node.js executable at: ${command} (source: ${nodeResult.source}). Reason: ${originalError}`
|
||||
);
|
||||
}
|
||||
if (!exists) {
|
||||
throw new Error(`Node.js executable not found at: ${command} (source: ${nodeResult.source})`);
|
||||
}
|
||||
}
|
||||
|
||||
let command: string;
|
||||
let commandSource: string;
|
||||
let args: string[];
|
||||
let serverPath: string;
|
||||
|
||||
if (isDev) {
|
||||
// In development, run the TypeScript server via the user's Node.js.
|
||||
const nodeResult = findNodeExecutable({
|
||||
skipSearch: true,
|
||||
logger: (msg: string) => logger.info(msg),
|
||||
});
|
||||
command = nodeResult.nodePath;
|
||||
commandSource = nodeResult.source;
|
||||
|
||||
// Validate that the found Node executable actually exists
|
||||
// systemPathExists is used because node-finder returns system paths
|
||||
if (command !== 'node') {
|
||||
let exists: boolean;
|
||||
try {
|
||||
exists = systemPathExists(command);
|
||||
} catch (error) {
|
||||
const originalError = error instanceof Error ? error.message : String(error);
|
||||
throw new Error(
|
||||
`Failed to verify Node.js executable at: ${command} (source: ${nodeResult.source}). Reason: ${originalError}`
|
||||
);
|
||||
}
|
||||
if (!exists) {
|
||||
throw new Error(
|
||||
`Node.js executable not found at: ${command} (source: ${nodeResult.source})`
|
||||
);
|
||||
}
|
||||
}
|
||||
} else {
|
||||
// In packaged builds, use Electron's bundled Node runtime instead of a system Node.
|
||||
// This makes the desktop app self-contained and avoids incompatibilities with whatever
|
||||
// Node version the user happens to have installed globally.
|
||||
command = process.execPath;
|
||||
commandSource = 'electron';
|
||||
}
|
||||
|
||||
// __dirname is apps/ui/dist-electron (Vite bundles all into single file)
|
||||
if (isDev) {
|
||||
serverPath = path.join(__dirname, '../../server/src/index.ts');
|
||||
@@ -133,6 +146,8 @@ export async function startServer(): Promise<void> {
|
||||
PORT: state.serverPort.toString(),
|
||||
DATA_DIR: dataDir,
|
||||
NODE_PATH: serverNodeModules,
|
||||
// Run packaged backend with Electron's embedded Node runtime.
|
||||
...(app.isPackaged && { ELECTRON_RUN_AS_NODE: '1' }),
|
||||
// Pass API key to server for CSRF protection
|
||||
AUTOMAKER_API_KEY: state.apiKey!,
|
||||
// Only set ALLOWED_ROOT_DIRECTORY if explicitly provided in environment
|
||||
@@ -146,6 +161,7 @@ export async function startServer(): Promise<void> {
|
||||
logger.info('[DATA_DIR_SPAWN] env.DATA_DIR=', env.DATA_DIR);
|
||||
|
||||
logger.info('Starting backend server...');
|
||||
logger.info('Runtime command:', command, `(source: ${commandSource})`);
|
||||
logger.info('Server path:', serverPath);
|
||||
logger.info('Server root (cwd):', serverRoot);
|
||||
logger.info('NODE_PATH:', serverNodeModules);
|
||||
|
||||
@@ -22,6 +22,8 @@ function arraysEqual(a: string[], b: string[]): boolean {
|
||||
return a.every((id) => set.has(id));
|
||||
}
|
||||
const AUTO_MODE_POLLING_INTERVAL = 30000;
|
||||
// Stable empty array reference to avoid re-renders from `[] !== []`
|
||||
const EMPTY_TASKS: string[] = [];
|
||||
|
||||
/**
|
||||
* Generate a worktree key for session storage
|
||||
@@ -77,8 +79,12 @@ function isPlanApprovalEvent(
|
||||
* @param worktree - Optional worktree info. If not provided, uses main worktree (branchName = null)
|
||||
*/
|
||||
export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// Subscribe to stable action functions and scalar state via useShallow.
|
||||
// IMPORTANT: Do NOT subscribe to autoModeByWorktree here. That object gets a
|
||||
// new reference on every Zustand mutation to ANY worktree, which would re-render
|
||||
// every useAutoMode consumer on every store change. Instead, we subscribe to the
|
||||
// specific worktree's state below using a targeted selector.
|
||||
const {
|
||||
autoModeByWorktree,
|
||||
setAutoModeRunning,
|
||||
addRunningTask,
|
||||
removeRunningTask,
|
||||
@@ -93,7 +99,6 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
addRecentlyCompletedFeature,
|
||||
} = useAppStore(
|
||||
useShallow((state) => ({
|
||||
autoModeByWorktree: state.autoModeByWorktree,
|
||||
setAutoModeRunning: state.setAutoModeRunning,
|
||||
addRunningTask: state.addRunningTask,
|
||||
removeRunningTask: state.removeRunningTask,
|
||||
@@ -144,41 +149,109 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
[projects]
|
||||
);
|
||||
|
||||
// Get worktree-specific auto mode state
|
||||
// Get worktree-specific auto mode state using a TARGETED selector with
|
||||
// VALUE-BASED equality. This is critical for preventing cascading re-renders
|
||||
// in board view, where DndContext amplifies every parent re-render.
|
||||
//
|
||||
// Why value-based equality matters: Every Zustand `set()` call (including
|
||||
// `addAutoModeActivity` which fires on every WS event) triggers all subscriber
|
||||
// selectors to re-run. Even our targeted selector that reads a specific key
|
||||
// would return a new object reference (from the spread in `removeRunningTask`
|
||||
// etc.), causing a re-render even when the actual values haven't changed.
|
||||
// By extracting primitives and comparing with a custom equality function,
|
||||
// we only re-render when isRunning/runningTasks/maxConcurrency actually change.
|
||||
const projectId = currentProject?.id;
|
||||
const worktreeAutoModeState = useMemo(() => {
|
||||
if (!projectId)
|
||||
return {
|
||||
isRunning: false,
|
||||
runningTasks: [],
|
||||
branchName: null,
|
||||
maxConcurrency: DEFAULT_MAX_CONCURRENCY,
|
||||
};
|
||||
const key = getWorktreeKey(projectId, branchName);
|
||||
return (
|
||||
autoModeByWorktree[key] || {
|
||||
isRunning: false,
|
||||
runningTasks: [],
|
||||
branchName,
|
||||
maxConcurrency: DEFAULT_MAX_CONCURRENCY,
|
||||
}
|
||||
);
|
||||
}, [autoModeByWorktree, projectId, branchName, getWorktreeKey]);
|
||||
const worktreeKey = useMemo(
|
||||
() => (projectId ? getWorktreeKey(projectId, branchName) : null),
|
||||
[projectId, branchName, getWorktreeKey]
|
||||
);
|
||||
|
||||
const isAutoModeRunning = worktreeAutoModeState.isRunning;
|
||||
const runningAutoTasks = worktreeAutoModeState.runningTasks;
|
||||
// Use the subscribed worktreeAutoModeState.maxConcurrency (from the reactive
|
||||
// autoModeByWorktree store slice) so canStartNewTask stays reactive when
|
||||
// refreshStatus updates worktree state or when the global setting changes.
|
||||
// Falls back to the subscribed globalMaxConcurrency (also reactive) when no
|
||||
// per-worktree value is set, and to DEFAULT_MAX_CONCURRENCY when no project.
|
||||
// Subscribe to this specific worktree's state using useShallow.
|
||||
// useShallow compares each property of the returned object with Object.is,
|
||||
// so primitive properties (isRunning: boolean, maxConcurrency: number) are
|
||||
// naturally stable. Only runningTasks (array) needs additional stabilization
|
||||
// since filter()/spread creates new array references even for identical content.
|
||||
const { worktreeIsRunning, worktreeRunningTasksRaw, worktreeMaxConcurrency } = useAppStore(
|
||||
useShallow((state) => {
|
||||
if (!worktreeKey) {
|
||||
return {
|
||||
worktreeIsRunning: false,
|
||||
worktreeRunningTasksRaw: EMPTY_TASKS,
|
||||
worktreeMaxConcurrency: undefined as number | undefined,
|
||||
};
|
||||
}
|
||||
const wt = state.autoModeByWorktree[worktreeKey];
|
||||
if (!wt) {
|
||||
return {
|
||||
worktreeIsRunning: false,
|
||||
worktreeRunningTasksRaw: EMPTY_TASKS,
|
||||
worktreeMaxConcurrency: undefined as number | undefined,
|
||||
};
|
||||
}
|
||||
return {
|
||||
worktreeIsRunning: wt.isRunning,
|
||||
worktreeRunningTasksRaw: wt.runningTasks,
|
||||
worktreeMaxConcurrency: wt.maxConcurrency,
|
||||
};
|
||||
})
|
||||
);
|
||||
// Stabilize runningTasks: useShallow uses Object.is per property, but
|
||||
// runningTasks gets a new array ref after removeRunningTask/addRunningTask.
|
||||
// Cache the previous value and only update when content actually changes.
|
||||
const prevTasksRef = useRef<string[]>(EMPTY_TASKS);
|
||||
const worktreeRunningTasks = useMemo(() => {
|
||||
if (worktreeRunningTasksRaw === prevTasksRef.current) return prevTasksRef.current;
|
||||
if (arraysEqual(prevTasksRef.current, worktreeRunningTasksRaw)) return prevTasksRef.current;
|
||||
prevTasksRef.current = worktreeRunningTasksRaw;
|
||||
return worktreeRunningTasksRaw;
|
||||
}, [worktreeRunningTasksRaw]);
|
||||
|
||||
const isAutoModeRunning = worktreeIsRunning;
|
||||
const runningAutoTasks = worktreeRunningTasks;
|
||||
// Use worktreeMaxConcurrency (from the reactive per-key selector) so
|
||||
// canStartNewTask stays reactive when refreshStatus updates worktree state
|
||||
// or when the global setting changes.
|
||||
const maxConcurrency = projectId
|
||||
? (worktreeAutoModeState.maxConcurrency ?? globalMaxConcurrency)
|
||||
? (worktreeMaxConcurrency ?? globalMaxConcurrency)
|
||||
: DEFAULT_MAX_CONCURRENCY;
|
||||
|
||||
// Check if we can start a new task based on concurrency limit
|
||||
const canStartNewTask = runningAutoTasks.length < maxConcurrency;
|
||||
|
||||
// Batch addAutoModeActivity calls to reduce Zustand set() frequency.
|
||||
// Without batching, each WS event (especially auto_mode_progress which fires
|
||||
// rapidly during streaming) triggers a separate set() → all subscriber selectors
|
||||
// re-evaluate → on mobile this overwhelms React's batching → crash.
|
||||
// This batches activities in a ref and flushes them in a single set() call.
|
||||
const pendingActivitiesRef = useRef<Parameters<typeof addAutoModeActivity>[0][]>([]);
|
||||
const flushTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
|
||||
const batchedAddAutoModeActivity = useCallback(
|
||||
(activity: Parameters<typeof addAutoModeActivity>[0]) => {
|
||||
pendingActivitiesRef.current.push(activity);
|
||||
if (!flushTimerRef.current) {
|
||||
flushTimerRef.current = setTimeout(() => {
|
||||
const batch = pendingActivitiesRef.current;
|
||||
pendingActivitiesRef.current = [];
|
||||
flushTimerRef.current = null;
|
||||
// Flush all pending activities in a single store update
|
||||
for (const act of batch) {
|
||||
addAutoModeActivity(act);
|
||||
}
|
||||
}, 100);
|
||||
}
|
||||
},
|
||||
[addAutoModeActivity]
|
||||
);
|
||||
|
||||
// Cleanup flush timer on unmount
|
||||
useEffect(() => {
|
||||
return () => {
|
||||
if (flushTimerRef.current) {
|
||||
clearTimeout(flushTimerRef.current);
|
||||
}
|
||||
};
|
||||
}, []);
|
||||
|
||||
// Ref to prevent refreshStatus and WebSocket handlers from overwriting optimistic state
|
||||
// during start/stop transitions.
|
||||
const isTransitioningRef = useRef(false);
|
||||
@@ -498,7 +571,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
case 'auto_mode_feature_start':
|
||||
if (event.featureId) {
|
||||
addRunningTask(eventProjectId, eventBranchName, event.featureId);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'start',
|
||||
message: `Started working on feature`,
|
||||
@@ -514,7 +587,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// briefly appear in backlog due to stale cache data
|
||||
addRecentlyCompletedFeature(event.featureId);
|
||||
removeRunningTask(eventProjectId, eventBranchName, event.featureId);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'complete',
|
||||
message: event.passes
|
||||
@@ -551,7 +624,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
? `Authentication failed: Please check your API key in Settings or run 'claude login' in terminal to re-authenticate.`
|
||||
: event.error;
|
||||
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'error',
|
||||
message: errorMessage,
|
||||
@@ -568,7 +641,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
case 'auto_mode_progress':
|
||||
// Log progress updates (throttle to avoid spam)
|
||||
if (event.featureId && event.content && event.content.length > 10) {
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'progress',
|
||||
message: event.content.substring(0, 200), // Limit message length
|
||||
@@ -579,7 +652,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
case 'auto_mode_tool':
|
||||
// Log tool usage
|
||||
if (event.featureId && event.tool) {
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'tool',
|
||||
message: `Using tool: ${event.tool}`,
|
||||
@@ -592,7 +665,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// Log phase transitions (Planning, Action, Verification)
|
||||
if (event.featureId && event.phase && event.message) {
|
||||
logger.debug(`[AutoMode] Phase: ${event.phase} for ${event.featureId}`);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: event.phase,
|
||||
message: event.message,
|
||||
@@ -618,7 +691,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// Log when planning phase begins
|
||||
if (event.featureId && event.mode && event.message) {
|
||||
logger.debug(`[AutoMode] Planning started (${event.mode}) for ${event.featureId}`);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'planning',
|
||||
message: event.message,
|
||||
@@ -631,7 +704,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// Log when plan is approved by user
|
||||
if (event.featureId) {
|
||||
logger.debug(`[AutoMode] Plan approved for ${event.featureId}`);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'action',
|
||||
message: event.hasEdits
|
||||
@@ -646,7 +719,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
// Log when plan is auto-approved (requirePlanApproval=false)
|
||||
if (event.featureId) {
|
||||
logger.debug(`[AutoMode] Plan auto-approved for ${event.featureId}`);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'action',
|
||||
message: 'Plan auto-approved, starting implementation...',
|
||||
@@ -665,7 +738,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
logger.debug(
|
||||
`[AutoMode] Plan revision requested for ${event.featureId} (v${revisionEvent.planVersion})`
|
||||
);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'planning',
|
||||
message: `Revising plan based on feedback (v${revisionEvent.planVersion})...`,
|
||||
@@ -681,7 +754,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
logger.debug(
|
||||
`[AutoMode] Task ${taskEvent.taskId} started for ${event.featureId}: ${taskEvent.taskDescription}`
|
||||
);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'progress',
|
||||
message: `▶ Starting ${taskEvent.taskId}: ${taskEvent.taskDescription}`,
|
||||
@@ -696,7 +769,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
logger.debug(
|
||||
`[AutoMode] Task ${taskEvent.taskId} completed for ${event.featureId} (${taskEvent.tasksCompleted}/${taskEvent.tasksTotal})`
|
||||
);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'progress',
|
||||
message: `✓ ${taskEvent.taskId} done (${taskEvent.tasksCompleted}/${taskEvent.tasksTotal})`,
|
||||
@@ -714,7 +787,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
logger.debug(
|
||||
`[AutoMode] Phase ${phaseEvent.phaseNumber} completed for ${event.featureId}`
|
||||
);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'action',
|
||||
message: `Phase ${phaseEvent.phaseNumber} completed`,
|
||||
@@ -742,7 +815,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
logger.debug(
|
||||
`[AutoMode] Summary saved for ${event.featureId}: ${summaryEvent.summary.substring(0, 100)}...`
|
||||
);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId: event.featureId,
|
||||
type: 'progress',
|
||||
message: `Summary: ${summaryEvent.summary.substring(0, 100)}...`,
|
||||
@@ -758,7 +831,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
branchName,
|
||||
addRunningTask,
|
||||
removeRunningTask,
|
||||
addAutoModeActivity,
|
||||
batchedAddAutoModeActivity,
|
||||
getProjectIdFromPath,
|
||||
setPendingPlanApproval,
|
||||
setAutoModeRunning,
|
||||
@@ -977,7 +1050,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
removeRunningTask(currentProject.id, branchName, featureId);
|
||||
|
||||
logger.info('Feature stopped successfully:', featureId);
|
||||
addAutoModeActivity({
|
||||
batchedAddAutoModeActivity({
|
||||
featureId,
|
||||
type: 'complete',
|
||||
message: 'Feature stopped by user',
|
||||
@@ -993,7 +1066,7 @@ export function useAutoMode(worktree?: WorktreeInfo) {
|
||||
throw error;
|
||||
}
|
||||
},
|
||||
[currentProject, branchName, removeRunningTask, addAutoModeActivity]
|
||||
[currentProject, branchName, removeRunningTask, batchedAddAutoModeActivity]
|
||||
);
|
||||
|
||||
return {
|
||||
|
||||
@@ -13,6 +13,7 @@ import type { AutoModeEvent, SpecRegenerationEvent, StreamEvent } from '@/types/
|
||||
import type { IssueValidationEvent } from '@automaker/types';
|
||||
import { debounce, type DebouncedFunction } from '@automaker/utils/debounce';
|
||||
import { useEventRecencyStore } from './use-event-recency';
|
||||
import { isAnyFeatureTransitioning } from '@/lib/feature-transition-state';
|
||||
|
||||
/**
|
||||
* Debounce configuration for auto_mode_progress invalidations
|
||||
@@ -31,8 +32,10 @@ const FEATURE_LIST_INVALIDATION_EVENTS: AutoModeEvent['type'][] = [
|
||||
'auto_mode_feature_start',
|
||||
'auto_mode_feature_complete',
|
||||
'auto_mode_error',
|
||||
'auto_mode_started',
|
||||
'auto_mode_stopped',
|
||||
// NOTE: auto_mode_started and auto_mode_stopped are intentionally excluded.
|
||||
// These events signal auto-loop state changes, NOT feature data changes.
|
||||
// Including them caused unnecessary refetches that raced with optimistic
|
||||
// updates during start/stop cycles, triggering React error #185 on mobile.
|
||||
'plan_approval_required',
|
||||
'plan_approved',
|
||||
'plan_rejected',
|
||||
@@ -176,8 +179,12 @@ export function useAutoModeQueryInvalidation(projectPath: string | undefined) {
|
||||
// This allows polling to be disabled when WebSocket events are flowing
|
||||
recordGlobalEvent();
|
||||
|
||||
// Invalidate feature list for lifecycle events
|
||||
if (FEATURE_LIST_INVALIDATION_EVENTS.includes(event.type)) {
|
||||
// Invalidate feature list for lifecycle events.
|
||||
// Skip invalidation when a feature is mid-transition (e.g., being cancelled)
|
||||
// because persistFeatureUpdate already handles the optimistic cache update.
|
||||
// Without this guard, auto_mode_error / auto_mode_stopped WS events race
|
||||
// with the optimistic update and cause re-render cascades on mobile (React #185).
|
||||
if (FEATURE_LIST_INVALIDATION_EVENTS.includes(event.type) && !isAnyFeatureTransitioning()) {
|
||||
queryClient.invalidateQueries({
|
||||
queryKey: queryKeys.features.all(currentProjectPath),
|
||||
});
|
||||
|
||||
19
apps/ui/src/lib/feature-transition-state.ts
Normal file
19
apps/ui/src/lib/feature-transition-state.ts
Normal file
@@ -0,0 +1,19 @@
|
||||
/**
|
||||
* Lightweight module-level state tracking which features are mid-transition
|
||||
* (e.g., being cancelled). Used by useAutoModeQueryInvalidation to skip
|
||||
* redundant cache invalidations while persistFeatureUpdate is in flight.
|
||||
*/
|
||||
|
||||
const transitioningFeatures = new Set<string>();
|
||||
|
||||
export function markFeatureTransitioning(featureId: string): void {
|
||||
transitioningFeatures.add(featureId);
|
||||
}
|
||||
|
||||
export function unmarkFeatureTransitioning(featureId: string): void {
|
||||
transitioningFeatures.delete(featureId);
|
||||
}
|
||||
|
||||
export function isAnyFeatureTransitioning(): boolean {
|
||||
return transitioningFeatures.size > 0;
|
||||
}
|
||||
@@ -923,17 +923,20 @@ export class HttpApiClient implements ElectronAPI {
|
||||
this.ws.onmessage = (event) => {
|
||||
try {
|
||||
const data = JSON.parse(event.data);
|
||||
logger.info(
|
||||
'WebSocket message:',
|
||||
data.type,
|
||||
'hasPayload:',
|
||||
!!data.payload,
|
||||
'callbacksRegistered:',
|
||||
this.eventCallbacks.has(data.type)
|
||||
);
|
||||
// Only log non-high-frequency events to avoid progressive memory growth
|
||||
// from accumulated console entries. High-frequency events (dev-server output,
|
||||
// test runner output, agent progress) fire 10+ times/sec and would generate
|
||||
// thousands of console entries per minute.
|
||||
const isHighFrequency =
|
||||
data.type === 'dev-server:output' ||
|
||||
data.type === 'test-runner:output' ||
|
||||
data.type === 'feature:progress' ||
|
||||
(data.type === 'auto-mode:event' && data.payload?.type === 'auto_mode_progress');
|
||||
if (!isHighFrequency) {
|
||||
logger.info('WebSocket message:', data.type);
|
||||
}
|
||||
const callbacks = this.eventCallbacks.get(data.type);
|
||||
if (callbacks) {
|
||||
logger.info('Dispatching to', callbacks.size, 'callbacks');
|
||||
callbacks.forEach((cb) => cb(data.payload));
|
||||
}
|
||||
} catch (error) {
|
||||
@@ -2760,6 +2763,21 @@ export class HttpApiClient implements ElectronAPI {
|
||||
headers?: Record<string, string>;
|
||||
enabled?: boolean;
|
||||
}>;
|
||||
eventHooks?: Array<{
|
||||
id: string;
|
||||
trigger: string;
|
||||
enabled: boolean;
|
||||
action: Record<string, unknown>;
|
||||
name?: string;
|
||||
}>;
|
||||
ntfyEndpoints?: Array<{
|
||||
id: string;
|
||||
name: string;
|
||||
serverUrl: string;
|
||||
topic: string;
|
||||
authType: string;
|
||||
enabled: boolean;
|
||||
}>;
|
||||
};
|
||||
error?: string;
|
||||
}> => this.get('/api/settings/global'),
|
||||
|
||||
@@ -600,10 +600,7 @@ function RootLayoutContent() {
|
||||
// so updating them won't cause a visible re-render flash.
|
||||
const serverHooks = (finalSettings as GlobalSettings).eventHooks ?? [];
|
||||
const currentHooks = useAppStore.getState().eventHooks;
|
||||
if (
|
||||
JSON.stringify(serverHooks) !== JSON.stringify(currentHooks) &&
|
||||
serverHooks.length > 0
|
||||
) {
|
||||
if (JSON.stringify(serverHooks) !== JSON.stringify(currentHooks)) {
|
||||
logger.info(
|
||||
`[FAST_HYDRATE] Reconciling eventHooks from server (server=${serverHooks.length}, store=${currentHooks.length})`
|
||||
);
|
||||
|
||||
@@ -6,6 +6,6 @@ export const Route = createLazyFileRoute('/board')({
|
||||
});
|
||||
|
||||
function BoardRouteComponent() {
|
||||
const { featureId } = useSearch({ from: '/board' });
|
||||
return <BoardView initialFeatureId={featureId} />;
|
||||
const { featureId, projectPath } = useSearch({ from: '/board' });
|
||||
return <BoardView initialFeatureId={featureId} initialProjectPath={projectPath} />;
|
||||
}
|
||||
|
||||
@@ -4,6 +4,7 @@ import { z } from 'zod';
|
||||
// Search params schema for board route
|
||||
const boardSearchSchema = z.object({
|
||||
featureId: z.string().optional(),
|
||||
projectPath: z.string().optional(),
|
||||
});
|
||||
|
||||
// Component is lazy-loaded via board.lazy.tsx for code splitting.
|
||||
|
||||
@@ -1044,6 +1044,9 @@ export const useAppStore = create<AppState & AppActions>()((set, get) => ({
|
||||
set((state) => {
|
||||
const current = state.autoModeByWorktree[key];
|
||||
if (!current) return state;
|
||||
// Idempotent: skip if task is not in the list to avoid creating new
|
||||
// object references that trigger unnecessary re-renders.
|
||||
if (!current.runningTasks.includes(taskId)) return state;
|
||||
return {
|
||||
autoModeByWorktree: {
|
||||
...state.autoModeByWorktree,
|
||||
@@ -1097,13 +1100,20 @@ export const useAppStore = create<AppState & AppActions>()((set, get) => ({
|
||||
|
||||
addRecentlyCompletedFeature: (featureId: string) => {
|
||||
set((state) => {
|
||||
// Idempotent: skip if already tracked to avoid creating a new Set reference
|
||||
// that triggers unnecessary re-renders in useBoardColumnFeatures.
|
||||
if (state.recentlyCompletedFeatures.has(featureId)) return state;
|
||||
const newSet = new Set(state.recentlyCompletedFeatures);
|
||||
newSet.add(featureId);
|
||||
return { recentlyCompletedFeatures: newSet };
|
||||
});
|
||||
},
|
||||
|
||||
clearRecentlyCompletedFeatures: () => set({ recentlyCompletedFeatures: new Set() }),
|
||||
clearRecentlyCompletedFeatures: () => {
|
||||
// Idempotent: skip if already empty to avoid creating a new Set reference.
|
||||
if (get().recentlyCompletedFeatures.size === 0) return;
|
||||
set({ recentlyCompletedFeatures: new Set() });
|
||||
},
|
||||
|
||||
setMaxConcurrency: (max) => set({ maxConcurrency: max }),
|
||||
|
||||
@@ -1496,7 +1506,11 @@ export const useAppStore = create<AppState & AppActions>()((set, get) => ({
|
||||
set({ eventHooks: hooks });
|
||||
try {
|
||||
const httpApi = getHttpApiClient();
|
||||
await httpApi.settings.updateGlobal({ eventHooks: hooks });
|
||||
await httpApi.settings.updateGlobal({
|
||||
eventHooks: hooks,
|
||||
// Signal the server that an empty array is intentional (not a wipe from stale state)
|
||||
...(hooks.length === 0 ? { __allowEmptyEventHooks: true } : {}),
|
||||
});
|
||||
} catch (error) {
|
||||
logger.error('Failed to sync event hooks:', error);
|
||||
}
|
||||
@@ -1507,7 +1521,11 @@ export const useAppStore = create<AppState & AppActions>()((set, get) => ({
|
||||
set({ ntfyEndpoints: endpoints });
|
||||
try {
|
||||
const httpApi = getHttpApiClient();
|
||||
await httpApi.settings.updateGlobal({ ntfyEndpoints: endpoints });
|
||||
await httpApi.settings.updateGlobal({
|
||||
ntfyEndpoints: endpoints,
|
||||
// Signal the server that an empty array is intentional (not a wipe from stale state)
|
||||
...(endpoints.length === 0 ? { __allowEmptyNtfyEndpoints: true } : {}),
|
||||
});
|
||||
} catch (error) {
|
||||
logger.error('Failed to sync ntfy endpoints:', error);
|
||||
}
|
||||
|
||||
@@ -238,24 +238,36 @@ export default defineConfig(({ command }) => {
|
||||
// Inject build hash into sw.js CACHE_NAME for automatic cache busting
|
||||
swCacheBuster(),
|
||||
],
|
||||
// Keep Vite dep-optimization cache local to apps/ui so each worktree gets
|
||||
// its own pre-bundled dependencies. Shared cache state across worktrees can
|
||||
// produce duplicate React instances (notably with @xyflow/react) and trigger
|
||||
// "Invalid hook call" in the graph view.
|
||||
cacheDir: path.resolve(__dirname, 'node_modules/.vite'),
|
||||
resolve: {
|
||||
alias: [
|
||||
{ find: '@', replacement: path.resolve(__dirname, './src') },
|
||||
// Force ALL React imports (including from nested deps like zustand@4 inside
|
||||
// @xyflow/react) to resolve to the single copy in the workspace root node_modules.
|
||||
// This prevents "Cannot read properties of null (reading 'useState')" caused by
|
||||
// react-dom setting the hooks dispatcher on one React instance while component
|
||||
// code reads it from a different instance.
|
||||
// @xyflow/react) to resolve to a single copy.
|
||||
// Explicit subpath aliases must come BEFORE the broad regex so Vite's
|
||||
// first-match-wins resolution applies the specific match first.
|
||||
{
|
||||
find: /^react-dom(\/|$)/,
|
||||
replacement: path.resolve(__dirname, '../../node_modules/react-dom') + '/',
|
||||
},
|
||||
{
|
||||
find: 'react/jsx-runtime',
|
||||
replacement: path.resolve(__dirname, '../../node_modules/react/jsx-runtime.js'),
|
||||
},
|
||||
{
|
||||
find: 'react/jsx-dev-runtime',
|
||||
replacement: path.resolve(__dirname, '../../node_modules/react/jsx-dev-runtime.js'),
|
||||
},
|
||||
{
|
||||
find: /^react(\/|$)/,
|
||||
replacement: path.resolve(__dirname, '../../node_modules/react') + '/',
|
||||
},
|
||||
],
|
||||
dedupe: ['react', 'react-dom'],
|
||||
dedupe: ['react', 'react-dom', 'zustand', 'use-sync-external-store', '@xyflow/react'],
|
||||
},
|
||||
server: {
|
||||
host: process.env.HOST || '0.0.0.0',
|
||||
@@ -355,8 +367,12 @@ export default defineConfig(({ command }) => {
|
||||
include: [
|
||||
'react',
|
||||
'react-dom',
|
||||
'react/jsx-runtime',
|
||||
'react/jsx-dev-runtime',
|
||||
'use-sync-external-store',
|
||||
'use-sync-external-store/shim',
|
||||
'use-sync-external-store/shim/with-selector',
|
||||
'zustand',
|
||||
'@xyflow/react',
|
||||
],
|
||||
},
|
||||
|
||||
2
package-lock.json
generated
2
package-lock.json
generated
@@ -44,7 +44,7 @@
|
||||
"@automaker/prompts": "1.0.0",
|
||||
"@automaker/types": "1.0.0",
|
||||
"@automaker/utils": "1.0.0",
|
||||
"@github/copilot-sdk": "^0.1.16",
|
||||
"@github/copilot-sdk": "0.1.16",
|
||||
"@modelcontextprotocol/sdk": "1.25.2",
|
||||
"@openai/codex-sdk": "^0.98.0",
|
||||
"cookie-parser": "1.4.7",
|
||||
|
||||
Reference in New Issue
Block a user