Skip to content

AutoResearch: frontend useAutoResearch composable doesn't follow error handling pattern #3210

@mrveiss

Description

@mrveiss

Summary

The `useAutoResearch` composable uses raw `try/catch` with manual `error.value = ...` assignment instead of the codebase convention of `useApiWithState().withErrorHandling()`. This causes:

  1. No user-facing error toasts — API failures are silently swallowed into a ref
  2. No structured logging — no `createLogger('useAutoResearch')` usage
  3. 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

Discovered during code review of PR #3207 (Issue #3201)

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions