Skip to content

fix(core): requeue HTTP ingest failures - #110

Open
Tyagiquamar wants to merge 2 commits into
SmooAI:mainfrom
Tyagiquamar:fix/transport-requeue-http-errors
Open

Tyagiquamar wants to merge 2 commits into
SmooAI:mainfrom
Tyagiquamar:fix/transport-requeue-http-errors

Conversation

@Tyagiquamar

Copy link
Copy Markdown

Summary

  • detect HTTP failure responses from native fetch
  • re-queue failed ingest batches for the existing retry path
  • add regression coverage for a resolved 503 response

Root Cause

Native fetch resolves normally for HTTP 4xx and 5xx responses. The transport only handled rejected promises, so a non-2xx ingest response removed the batch from the queue permanently.

Verification

  • pnpm --filter @smooai/observability exec vitest run src/__tests__/transport.test.ts passed: 7 tests
  • pnpm --filter @smooai/observability typecheck passed
  • pnpm exec oxfmt --check packages/core/src/transport.ts packages/core/src/__tests__/transport.test.ts passed
  • git diff --check passed

Post-Deploy Monitoring & Validation

The existing ingest failure retry path remains the validation signal. No new production metric or configuration is introduced.

@changeset-bot

changeset-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e8676af

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smooai/observability Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Co-authored-by: Cursor <cursoragent@cursor.com>
@Tyagiquamar

Copy link
Copy Markdown
Author

Hi, friendly ping for review on this PR when you have a moment. Happy to address any feedback. Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant