Skip to content

Main convert - #392

Merged
abbaseya merged 4 commits into
mainfrom
main-convert
May 25, 2026
Merged

Main convert#392
abbaseya merged 4 commits into
mainfrom
main-convert

Conversation

@abbaseya

Copy link
Copy Markdown
Collaborator

No description provided.

abbaseya and others added 2 commits May 25, 2026 17:43
…e HTTP

The User-Agent invariant introduced in #387 (always announce
`ConvertAgent/1.0` on server-side outbound HTTP from HttpClient) is a
js-sdk-level behavioral guarantee surfaced through every manager
(api/data/event) that delegates to @convertcom/js-sdk-utils' HttpClient.

Surfaces the contract at the SDK entry point so the metrics-endpoint
isbot bypass dependency is documented at the layer customers integrate
against, not only at the transport layer.

Refs: #387
…elease

fix(js-sdk): document ConvertAgent UA invariant (trigger js-sdk release)
@abbaseya
abbaseya requested a review from a team May 25, 2026 14:47
@abbaseya abbaseya self-assigned this May 25, 2026
@sonarqubecloud

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the file header in packages/js-sdk/src/core.ts to document the invariant User-Agent header (ConvertAgent/1.0) used by the SDK for server-side HTTP requests. This documentation clarifies that the header is required for metrics endpoint bot bypass and cannot be configured by users. There were no review comments provided, so I have no feedback to offer.

@abbaseya
abbaseya merged commit bab7ae2 into main May 25, 2026
7 checks passed
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.

2 participants