Uh oh!
There was an error while loading. Please reload this page.
docs(deployment): add mysql:// to environment-variables.mdx's OS_DATABASE_URL example list - #9096
Merged
Conversation
…BASE_URL example list The OS_DATABASE_URL row's e.g. list named file:./data.sqlite, postgres://…, mongodb://…, and memory://, omitting mysql:// even though MySQL was ruled a supported deployment target (#8739) and the adjacent OS_DATABASE_DRIVER row already carries PR #9052's MySQL caveat pointer. Add mysql://… in the same one-canonical-scheme-per-target style the row already uses for postgres:// (no mysql2:// alias spelled out, matching the convention self-hosting.mdx's row uses). No second caveat pointer added — the row one line below already carries it. Fixes#9075 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011RB4waLuNbdruCo6X9oobm
os-project-manager
marked this pull request as ready for review
August 16, 2026 10:14
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes#9075
What
content/docs/deployment/environment-variables.mdx'sOS_DATABASE_URLrowcarries an
e.g.example list of connection-string schemes. It namedfile:./data.sqlite,postgres://…,mongodb://…,memory://— omittingmysql://even though MySQL was ruled a supported deployment target(#8739) and both
detectDriverFromUrl(runtime) andinferDriverTypeFromUrl(CLI) infer it via
/^mysql2?:\/\//i.Added
mysql://…to the list, in the same one-scheme-per-target style therow already uses for
postgres://(i.e. nomysql2://alias spelled out).Why this shape, and why it's small
Per the issue: this list is explicitly
e.g., not a closed instruction likeself-hosting.mdx'sOS_DATABASE_URLrow (#9053 / PR #9076) — nothing onthe page was untrue before this change, only incomplete. So the fix is a
one-scheme addition, not a rewrite:
OS_DATABASE_DRIVER, already carries the MySQL dialect-caveat pointer PRdocs(data-modeling): publish MySQL's two documented degradations as deployer-facing caveats #9052 landed on this page (
Read [MySQL dialect caveats](/docs/data-modeling/drivers#mysql-dialect-caveats)…).A second pointer one row above it would be redundant, not helpful.
mysql2://intentionally left out, even though the runtime acceptsit.
self-hosting.mdx's equivalent row deliberately names one canonicalscheme per target and never spells an alias (
postgres://notpostgresql://,mongodb://notmongodb+srv://). This page'se.g.list follows the same convention already (one scheme each for sqlite/
postgres/mongodb), so
mysql://alone matches it;data-modeling/drivers.mdx'sinference table remains the exhaustive alias reference for both
mysql://andmysql2://.Out of scope
content/docs/**for further dialect-parity gaps — the MySQL's documented degradations are still source comments, not published support-matrix caveats — ruling item 2 of #8739 has no card #9029fence on that stands. Nothing else was found while making this edit.
content/docs/releases/**untouched.Verification
At
657ef16fd(this branch's HEAD):pnpm check:docs-audit-scope— OK (178 hand-written docs in scope; 9release-owned pages review-only).
pnpm check:docs-redirects— OK (92 redirect entries checked).pnpm check:role-word— OK (43 baselined files, no new occurrences).pnpm check:nul-bytes— OK (5973 files scanned, no raw control bytes).node scripts/pm/dispatch-gates.mjs content/docs/deployment/environment-variables.mdxnamed exactly these three families for the changed path; no anchors or
page-path links were added, so
check:doc-anchors/Check Documentation Linksare not implicated by this diff.Generated by Claude Code
Generated by Claude Code