[AI Gateway] Document user agent in logs and add changelog#31410
Conversation
Review❌ Review failed for commit |
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
There was a problem hiding this comment.
Review: Looks good with two minor metadata items to address.
What changed:
- Adds a changelog entry announcing user agent capture in AI Gateway logs.
- Updates the Logging reference page to mention user agent in the dashboard log fields.
- Adds the User Agent filter (equals / does not equal / contains) to the filter table.
Flags:
- LOW: The PR body is empty — please fill in the Summary section so reviewers know the context at a glance.
- LOW: The documentation checklist boxes are all unchecked. Since this PR adds a changelog and follows the style guide, please check the relevant boxes.
Checks: �[2m22:30:16�[22m �[34m[astro-skills]�[39m Setting up Agent Skills Discovery routes
�[2m22:30:16�[22m �[34m[astro-skills]�[39m Agent Skills Discovery routes configured
�[2m22:30:17�[22m �[34m[content]�[39m Syncing content
�[33m�[1m22:30:17�[22m [WARN] [skills-loader]�[39m Skills directory "skills" does not exist.
�[2m22:30:17�[22m �[34m[skills-loader]�[39m Loaded 0 skill(s) from "skills"
�[2m22:30:20�[22m �[34m[content]�[39m Synced content
�[2m22:30:20�[22m �[34m[types]�[39m Generated �[2m3.97s�[22m
�[2m22:30:20�[22m �[34m[check]�[39m Getting diagnostics for Astro files in /home/runner/work/cloudflare-docs/cloudflare-docs...
�[96mbin/post-link-validation-comment/index.ts�[0m:�[93m93�[0m:�[93m4�[0m - �[93mwarning�[0m�[90m ts(7027): �[0mUnreachable code detected.
�[7m93�[0m return;
�[7m �[0m �[93m ~~~~~~�[0m
�[96mbin/post-link-validation-comment/index.ts�[0m:�[93m56�[0m:�[93m4�[0m - �[93mwarning�[0m�[90m ts(7027): �[0mUnreachable code detected.
�[7m56�[0m return;
�[7m �[0m �[93m ~~~~~~�[0m
�[96msrc/components/OneTrust.astro�[0m:�[93m20�[0m:�[93m11�[0m - �[93mwarning�[0m�[90m ts(6133): �[0m'OptanonWrapper' is declared but its value is never read.
�[7m20�[0m function OptanonWrapper() {}
�[7m �[0m �[93m ~~~~~~~~~~~~~~�[0m
�[96msrc/util/custom-loaders.ts�[0m:�[93m27�[0m:�[93m28�[0m - �[93mwarning�[0m�[90m ts(6385): �[0m'(params?: string | { normalize?: boolean | undefined; abort?: boolean | undefined; pattern?: RegExp | undefined; protocol?: RegExp | undefined; hostname?: RegExp | undefined; error?: string | ... 1 more ... | undefined; message?: string | undefined; } | undefined): ZodString' is deprecated.
�[7m27�[0m const source = z.string().url().parse(url);
�[7m �[0m �[93m ~~~�[0m
Result (376 files):
- 0 errors
- 0 warnings
- 4 hints, , and Checking formatting...
All matched files use Prettier code style! all pass. Labels applied: , , .
There was a problem hiding this comment.
Review: Looks good with two minor metadata items to address.
What changed:
- Adds a changelog entry announcing user agent capture in AI Gateway logs.
- Updates the Logging reference page to mention user agent in the dashboard log fields.
- Adds the User Agent filter (equals / does not equal / contains) to the filter table.
Flags:
- LOW: The PR body is empty — please fill in the Summary section so reviewers know the context at a glance.
- LOW: The documentation checklist boxes are all unchecked. Since this PR adds a changelog and follows the style guide, please check the relevant boxes.
Checks: pnpm run check, pnpm run lint, and pnpm run format:core:check all pass. Labels applied: documentation, changelog, triage.
|
I've completed the review for PR #31410. What I did:
Summary of findings:
No blocking issues or build breakers were found. |
d8b3dfe to
b822504
Compare
Summary
Screenshots (optional)
Documentation checklist