docs(otel): document remappings #2989
Open
Claude / Claude Code Review
completed
May 22, 2026 in 9m 26s
Code review found 2 important issues
Found 6 candidates, confirmed 2. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 2 |
| 🟡 Nit | 0 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🔴 Important | content/docs/observability/features/tool-calls.mdx:1-4 |
Missing required title: in frontmatter |
| 🔴 Important | pages/docs/observability/features/tool-calls.mdx:1-10 |
Stray duplicate tool-calls.mdx under pages/ should be deleted |
Annotations
Check failure on line 4 in content/docs/observability/features/tool-calls.mdx
claude / Claude Code Review
Missing required `title:` in frontmatter
Missing required `title:` field in this new file's frontmatter — it's the only file in all of `content/docs/` that omits it. The Fumadocs frontmatter schema (`baseFrontmatterSchema` → `frontmatterSchema` in `source.config.ts`) declares `title` as a required `z.string()`, so the MDX collection build will fail with a Zod `ValidationError`; even if it didn't, `lib/mdx-page.ts` `buildSectionMetadata` would crash because `enrichOgTitle` calls `title.toLowerCase()` on `undefined`. Fix: add e.g. `title
Check failure on line 10 in pages/docs/observability/features/tool-calls.mdx
claude / Claude Code Review
Stray duplicate tool-calls.mdx under pages/ should be deleted
This PR adds a stray duplicate `pages/docs/observability/features/tool-calls.mdx` alongside the canonical `content/docs/observability/features/tool-calls.mdx`. This site uses Fumadocs App Router (routes resolved from `content/docs` via `app/docs/[[...slug]]`) and `next.config.mjs` does not extend `pageExtensions` to `.mdx`, so the `pages/` file is never served — it is dead content that has already drifted from the canonical copy within this same PR. Please delete `pages/docs/observability/featur
Loading