Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/components/Doc.tsx`:
- Line 132: The anchor element currently uses aria-hidden="true" while remaining
focusable and unconditionally renders `${pagePath}.md`, which can produce
"undefined.md"; remove the aria-hidden attribute from the <a> and change the JSX
to only render the anchor when pagePath is defined (e.g., wrap the anchor in a
conditional that checks pagePath or use a guard like pagePath && ...) so the
link is not present/focusable when there is no pagePath and the href is only
created when pagePath exists.
- Line 4: The hook useWidthToggle is being called inside a try/catch which
violates React Hooks rules; replace this pattern by adding a non-throwing
wrapper (e.g., create useOptionalWidthToggle in the same module that currently
defines useWidthToggle, such as src/components/DocsLayout.tsx) that returns the
width toggle value or undefined instead of throwing when the context is absent,
then update Doc.tsx to import and call useOptionalWidthToggle (or the new
variant) at the top level unconditionally and remove the try/catch around the
hook call; ensure useWidthToggle keeps throwing behavior if you want a strict
version, while useOptionalWidthToggle uses useContext safely and returns
undefined for absent context.

In `@src/hooks/useMutation.ts`:
- Around line 52-53: The mutate callback currently only lists [opts.fn] as its
dependency which can cause it to close over stale handlers; update the
dependency array for the mutate callback (the function created in useMutation)
to include opts.onSuccess, opts.onError, and opts.onSettled in addition to
opts.fn so that the latest handler functions are used when mutate runs (or
alternatively stabilize opts by memoizing it upstream and then depend on that
stable reference).

In `@src/routes/`$libraryId/$version.docs.$.tsx:
- Around line 25-36: The catch branch after calling loadDocs currently turns
legacy/moved docs into hard 404s; instead, before throwing notFound() check for
legacy redirects and return a redirect when found. Update the catch around
loadDocs(...) to call the existing redirect-resolution routine (e.g.,
resolveLegacyDocRedirect or resolveRedirects) using the same inputs (library,
version, docsPath, getBranch(library, version), library.docsRoot) and if it
returns a canonical URL issue a redirect response; only call throw notFound() if
no redirect is found. Ensure you reference the same symbols used in the diff:
loadDocs, getBranch, library.docsRoot, docsPath, and notFound.

In `@src/utils/documents.server.ts`:
- Around line 343-349: The returned object currently spreads ...result.data
which still exposes frontmatter keys like redirect_from and redirectFrom; update
the return to explicitly omit those fields by creating a sanitizedData copy
(e.g., destructure or shallow clone of result.data while excluding redirect_from
and redirectFrom) and then return { ...result, data: { ...sanitizedData,
description: createExcerpt(result.content) } } so the redirect fields are not
present; reference result, result.data, createExcerpt, redirect_from and
redirectFrom when making the change.

---

Outside diff comments:
In `@src/routes/`$libraryId/$version.docs.framework.$framework.$.tsx:
- Around line 25-44: The catch block currently treats any missing doc as a
framework-root redirect which loses legacy deep links; before throwing the
generic redirect, call a function that resolves renamed/relocated pages (e.g.,
getRenamedDoc or resolveCanonicalDoc) using the same inputs used for loadDocs
(library, version via getBranch(library, version), framework and docsPath
constructed from library.docsRoot) and, if it returns a canonical replacement
path, throw redirect to that canonical path; only if no replacement is found
keep the existing redirect behavior, and continue to use isNotFound(error) to
gate this logic.

---

Nitpick comments:
In `@eslint.config.mjs`:
- Around line 52-67: The ESLint config currently registers the react plugin but
doesn't apply any React rules; either remove the unused import/entry for react
in the plugins block or add explicit React rules — e.g., spread
react.configs.recommended.rules into the rules object (alongside
...reactHooks.configs.recommended.rules and
...jsxA11y.configs.recommended.rules) or selectively add rules such as
'react/jsx-key' and 'react/no-unescaped-entities'; update the plugins and rules
entries (the react identifier and the rules object where
...reactHooks.configs.recommended.rules and ...jsxA11y.configs.recommended.rules
are used) accordingly so the react plugin is either removed or its recommended
rules are applied.

In `@src/hooks/useMutation.ts`:
- Around line 61-63: The dependency array for the useCallback that defines
handleSubmit currently includes variables even though variables is not
referenced in the callback body; update the useCallback dependencies to only
include mutate (i.e., change [mutate, variables] to [mutate]) and remove the
now-unnecessary eslint-disable-next-line react-hooks/exhaustive-deps comment.
Ensure you edit the useCallback that defines handleSubmit in useMutation.ts so
the hook no longer recreates the callback unnecessarily.