Requests v3 migration guide + docs updates

[ted-palmer]

No description provided.

[github-actions]

Claude finished @ted-palmer's task in 1m 31s —— View job

---

Review: Requests v3 migration guide (re-review)

• Gather PR diff & current branch state
• Verify prior feedback items were addressed
• Review all changed files against style guide (AGENTS.md)
• Post review feedback

Re-reviewed the latest branch state. The guide is in great shape — definitional intro, clean ##/### hierarchy, expandable on the large JSON blocks (§2.8), the three-tab v2 / v3 / Difference layout as a justified use of tabs (§2.8), and the <AccordionGroup> "What changed" / "Field migration reference" sections read clearly. Nav wiring in docs.json is clean and correct.

Here's where the prior feedback landed:

✅ Resolved — v2/v3 reference links now render the correct specs

This was the main open issue and it's cleanly fixed:

• references/api/get-requests.mdx → openapi: get /requests/v3 (v3 links resolve to v3).
• New references/api/get-requests-v2.mdx → openapi: get /requests/v2 with a <Warning> pointing to v3 and the migration guide (§4.3 pattern, verbatim).
• Wired into the Deprecated nav group in docs.json:164.
• Both GET /requests/v2 links in fees.mdx (lines 138, 154) now point to /references/api/get-requests-v2, so they render v2. 👍

✅ Resolved — non-breaking hyphens (U+2011)

All 17 occurrences of ‑ (U+2011) are gone — grep now returns zero matches. same-chain, catch-all, JSON-encoded, etc. all use ASCII - and match §5.

🟡 Still open — intro paragraph exceeds the 2–3 sentence cap (§2.10)

migrating-to-requests-v3.mdx:6 is still four sentences ("…latest version…" / "It gives you…" / "This guide covers…" / "v2 stays available…"). Minor — split or trim the last two.

Fix this →

Nits (non-blocking)

• Anchor #what-changed (migrating-to-requests-v3.mdx:456): the "Added" table links "Filtering, search & sorting" to #what-changed. Accordion titles don't generate their own anchors, so it lands at the section top rather than the specific accordion — won't deep-link, but acceptable.
• Prose parameter emphasis (§2.9): params like `data.fees` / `x-api-key` use backticks only; the guide prefers bold+backticks (**`data.fees`**). Repo practice is mixed, so low priority.
• The Difference tab intentionally repeats keys (failReason, fees, appFees, inTxs…) to pair [!code --]/[!code ++] lines — invalid as strict JSON but fine as illustrative jsonc. No change needed.
• style.css just adds a trailing newline — harmless.

Net: both blocking items from the prior rounds are resolved. Only the cosmetic intro-length item and a few nits remain — none blocking. LGTM.
• ted/int2-636-request-v2-v3-migration-guide

[github-actions]

Claude finished @ted-palmer's task in 1m 35s —— View job

---

Code Review: Requests v3 migration guide

• Gather PR diff & current branch state
• Review changed files against style guide (AGENTS.md)
• Post review feedback

Reviewed the full branch against the style guide. This is in great shape — clean ##/### hierarchy with no stray #, expandable on the large JSON blocks (§2.8), the three-tab v2 / v3 / Difference layout is a justified use of tabs, the <AccordionGroup> sections read clearly, and the deprecation wiring (v2 stub + <Warning>, nav under Deprecated, changelog entry, fees.mdx link repointing) is thorough and internally consistent. All the v2→v3 link targets resolve to the correct spec.

One new item and one carryover:

🟡 Changelog — missing blank line before the next entry (§4.5)

references/api/changelog.mdx:10-11 — the new **Deprecated** paragraph runs directly into the next entry's heading with no blank line:

**Deprecated** — … see the [sunset timeline](…#sunset-timeline).
## 2026-07-09 — Solana quote size check reserves compute-budget headroom

Every other entry in the file is separated from the next ## by a blank line. Without it, the MDX parser can absorb the heading into the preceding paragraph rather than rendering it as a new changelog entry. Add a blank line after line 10.

Fix this →

🟡 Intro paragraph exceeds the 2–3 sentence cap (§2.10) — carryover

migrating-to-requests-v3.mdx:6 is still four sentences. Minor — split or trim (e.g. drop "This guide covers every change and how to upgrade," which the page title already implies).

Fix this →

Nits (non-blocking)

• Anchor #what-changed (migrating-to-requests-v3.mdx:474): the "Added" table links "broad filter/sort surface" to #what-changed. Accordion titles don't generate their own anchors, so it lands at the section top rather than the specific "Filtering, search & sorting" accordion. Acceptable, just won't deep-link.
• §4.3 vs. sunset timeline: the guide's deprecation playbook says deprecated pages need no sunset date and "stay live indefinitely." This PR intentionally introduces a hard retirement date — a real product decision, not a style slip, and more useful to readers. Noting only because it diverges from the documented convention; no change needed.
• Prose parameter emphasis (§2.9): params like `data.fees` use backticks only; the guide prefers bold+backticks (**`data.fees`**) in prose. Repo practice is mixed, so low priority.
• The Difference tab repeats keys (failReason, inTxs, …) to pair [!code --]/[!code ++] lines — invalid as strict JSON but fine as illustrative jsonc. No change needed.
• style.css just removes a missing trailing newline — harmless.

Net: no blocking issues. The changelog blank line is the one thing I'd fix before merge; the intro length and the rest are cosmetic.
• ted/int2-636-request-v2-v3-migration-guide