feat(fetch): identify CLI with a Clerk-CLI User-Agent header

[rafa-thayto]

Summary

• Sets User-Agent: Clerk-CLI/<version> (Bun/<v>; <platform>-<arch>[; ci]) on every outbound request via the central loggedFetch helper, replacing Bun's default Bun/<v> so Clerk's edge can route or filter CLI traffic separately.
• Routes the lone direct fetch() in api/catalog through loggedFetch, picking up both the UA and standard debug-logging behavior (also fixes a pre-existing violation of .claude/rules/debug-logging.md).
• Callers can still override User-Agent via headers (verified in fetch.test.ts).

Test plan

• bun run format — clean
• bun run lint — 0 warnings, 0 errors
• bun run typecheck — passes
• bun run build — passes
• New tests in user-agent.test.ts + fetch.test.ts cover UA shape, caller override, platform/arch, CI marker, printable-ASCII — all green
• Updated token-exchange.test.ts to read headers via Headers.get() (the helper now passes a Headers instance through)
• Reviewer: confirm UA shows up in Cloud Run logs as Clerk-CLI/... instead of Bun/...

[changeset-bot]

🦋 Changeset detected

Latest commit: f71b7dd

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

This PR includes changesets to release 1 package

Name	Type
clerk	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

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request implements a Clerk-CLI-specific User-Agent header for outbound HTTP requests. A new buildUserAgent() helper constructs the string Clerk-CLI/<version> (Bun/<bun-version>; <platform>-<arch>) with optional CI environment detection. The loggedFetch function is updated to apply this User-Agent when the caller hasn't provided one. The catalog loader switches from withNetworkAccess(fetch(...)) to loggedFetch(...). Existing test assertions in token-exchange.test.ts are updated to use the Headers .get() API for consistency. A changeset documents the feature.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a Clerk-CLI User-Agent header to identify the CLI in outbound requests.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description check	✅ Passed	The PR description clearly explains the feature: setting a custom Clerk-CLI User-Agent on all outbound requests, routing catalog.ts through loggedFetch, and allowing caller overrides.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[jfoshee]

In the future it would be nice to encapsulate this as like getCliVersion() so callers don't need to worry about the fallback.