mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-01-30 06:12:03 +00:00
fix(ui): handle review feedback
This commit is contained in:
@@ -44,8 +44,13 @@ export function RunningAgentsView() {
|
|||||||
const isBacklogPlan = agent.featureId.startsWith('backlog-plan:');
|
const isBacklogPlan = agent.featureId.startsWith('backlog-plan:');
|
||||||
if (isBacklogPlan && api.backlogPlan) {
|
if (isBacklogPlan && api.backlogPlan) {
|
||||||
logger.debug('Stopping backlog plan agent', { featureId: agent.featureId });
|
logger.debug('Stopping backlog plan agent', { featureId: agent.featureId });
|
||||||
await api.backlogPlan.stop();
|
try {
|
||||||
refetch();
|
await api.backlogPlan.stop();
|
||||||
|
} catch (error) {
|
||||||
|
logger.error('Failed to stop backlog plan', { featureId: agent.featureId, error });
|
||||||
|
} finally {
|
||||||
|
refetch();
|
||||||
|
}
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
// Use mutation for regular features
|
// Use mutation for regular features
|
||||||
|
|||||||
@@ -27,10 +27,10 @@ export function useProjectSettingsLoader() {
|
|||||||
);
|
);
|
||||||
const setCurrentProject = useAppStore((state) => state.setCurrentProject);
|
const setCurrentProject = useAppStore((state) => state.setCurrentProject);
|
||||||
|
|
||||||
const appliedProjectRef = useRef<string | null>(null);
|
const appliedProjectRef = useRef<{ path: string; dataUpdatedAt: number } | null>(null);
|
||||||
|
|
||||||
// Fetch project settings with React Query
|
// Fetch project settings with React Query
|
||||||
const { data: settings } = useProjectSettings(currentProject?.path);
|
const { data: settings, dataUpdatedAt } = useProjectSettings(currentProject?.path);
|
||||||
|
|
||||||
// Apply settings when data changes
|
// Apply settings when data changes
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
@@ -39,11 +39,14 @@ export function useProjectSettingsLoader() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Prevent applying the same settings multiple times
|
// Prevent applying the same settings multiple times
|
||||||
if (appliedProjectRef.current === currentProject.path) {
|
if (
|
||||||
|
appliedProjectRef.current?.path === currentProject.path &&
|
||||||
|
appliedProjectRef.current?.dataUpdatedAt === dataUpdatedAt
|
||||||
|
) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
appliedProjectRef.current = currentProject.path;
|
appliedProjectRef.current = { path: currentProject.path, dataUpdatedAt };
|
||||||
const projectPath = currentProject.path;
|
const projectPath = currentProject.path;
|
||||||
|
|
||||||
const bg = settings.boardBackground;
|
const bg = settings.boardBackground;
|
||||||
@@ -109,6 +112,7 @@ export function useProjectSettingsLoader() {
|
|||||||
}, [
|
}, [
|
||||||
currentProject?.path,
|
currentProject?.path,
|
||||||
settings,
|
settings,
|
||||||
|
dataUpdatedAt,
|
||||||
setBoardBackground,
|
setBoardBackground,
|
||||||
setCardOpacity,
|
setCardOpacity,
|
||||||
setColumnOpacity,
|
setColumnOpacity,
|
||||||
|
|||||||
Reference in New Issue
Block a user