-
-
Notifications
You must be signed in to change notification settings - Fork 1
AutoResearch: frontend useAutoResearch composable doesn't follow error handling pattern #3210
Copy link
Copy link
Open
Labels
Description
Summary
The `useAutoResearch` composable uses raw `try/catch` with manual `error.value = ...` assignment instead of the codebase convention of `useApiWithState().withErrorHandling()`. This causes:
- No user-facing error toasts — API failures are silently swallowed into a ref
- No structured logging — no `createLogger('useAutoResearch')` usage
- Inconsistent UX — other composables show error notifications, this one doesn't
Also, `AutoResearchWorkflowAdapter.vue` is created but never imported or referenced anywhere — either integrate it into the workflow history view or remove it.
Expected Behavior
- Refactor to use `useApiWithState` or at minimum `createLogger` + `showSubtleErrorNotification`
- Wire `AutoResearchWorkflowAdapter` into the workflow view or remove it
Files
- `autobot-frontend/src/composables/useAutoResearch.ts` (multiple try/catch blocks)
- `autobot-frontend/src/components/autoresearch/AutoResearchWorkflowAdapter.vue` (unused)
Origin
Reactions are currently unavailable