docs: add AG2, AgentGateway, and Genkit integration pages (community PRs, review fixes applied)#3244
Draft
jannikmaierhoefer wants to merge 3 commits into
Draft
docs: add AG2, AgentGateway, and Genkit integration pages (community PRs, review fixes applied)#3244jannikmaierhoefer wants to merge 3 commits into
jannikmaierhoefer wants to merge 3 commits into
Claude / Claude Code Review
completed
Jul 6, 2026 in 9m 25s
Code review found 1 important issue
Found 5 candidates, confirmed 2. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 1 |
| 🟡 Nit | 1 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🔴 Important | content/integrations/frameworks/genkit.mdx:22-28 |
Genkit plugin module path contradicts documented repository |
| 🟡 Nit | content/integrations/frameworks/ag2.mdx:262-266 |
base64 without -w0 corrupts Authorization header on Linux with real keys |
Annotations
Check failure on line 28 in content/integrations/frameworks/genkit.mdx
claude / Claude Code Review
Genkit plugin module path contradicts documented repository
The Callout (line 18) and prose (line 28) both link to `github.com/genkit-ai/opentelemetry-go-plugin` as the documented plugin, but the `go get` command (line 25) and Go import (line 54) use `github.com/xavidop/genkit-opentelemetry-go` — a different owner and repository name. A reader clicking the linked repo will land on a package whose install/API instructions don't match the code snippets. Please reconcile by either updating the code to use the linked module, or updating both prose links to p
Check warning on line 266 in content/integrations/frameworks/ag2.mdx
claude / Claude Code Review
base64 without -w0 corrupts Authorization header on Linux with real keys
Two new bash snippets pipe the `pk:sk` pair through `base64` without `-w0`: `ag2.mdx:264` (`OTEL_EXPORTER_OTLP_HEADERS`) and `agentgateway.mdx:63` (`LANGFUSE_AUTH`). GNU coreutils `base64` wraps at 76 chars, and real Langfuse keys (`pk-lf-<UUID>:sk-lf-<UUID>`, ~85 chars → ~116 chars encoded) cross that threshold — the embedded newline survives `$(...)` (which only strips *trailing* newlines) and corrupts the Authorization header, causing silent 401s on Linux. Suggest adopting the pattern already
Loading