fix(knowledge): classify rejected BYOK embedding keys

Merged
WA
2mo ago
+181-13

Summary

  • preserve customer-managed credential ownership on embedding provider errors
  • record rejected BYOK keys as actionable document failures without futile automatic retries
  • keep platform-owned credential failures on the existing hard-failure path

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other

Testing

  • bunx vitest run background/knowledge-processing.test.ts lib/embeddings/client.test.ts lib/knowledge/documents/document-processing-source.test.ts
  • bun run type-check --filter=@sim/app
  • bun run lint
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • bun run check:audits

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

Not applicable; this change has no UI surface.