Skip to content

fix: preserve provider errors in user profile learning #265

Description

@lindixu6-hash

Problem

src/services/user-memory-learning.ts appears to have the same error-masking pattern fixed for auto-capture in #258: when the OpenCode structured-output path fails and no manual external-API fallback is configured, the provider root cause can be replaced by a generic configuration error.

The adjacent auto-capture log at src/services/auto-capture.ts:578 can also say that it is falling back to an external API when no fallback is configured.

Expected behavior

  • Preserve and surface the original provider error when profile learning has no configured manual fallback.
  • Keep the configured manual fallback path unchanged.
  • Ensure fallback logging accurately reflects whether a fallback will run.
  • Add focused regression coverage for the no-fallback and configured-fallback paths.

Context

Identified during the independent review of #258. Kept separate so that the already-reviewed auto-capture fix remains narrowly scoped.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions