feat(types)!: remove ObjectStack's own legacy env-var aliases (11.0, #2379)#2383
Merged
Conversation
…2379) Removes the framework's own renamed env names; keeps ubiquitous ecosystem conventions working (now silent, since they're permanent — not deprecated). Removed (rename required): - OS_MULTI_TENANT → OS_MULTI_ORG_ENABLED (resolveMultiOrgEnabled) - OBJECTSTACK_METADATA_WRITABLE → OS_METADATA_WRITABLE (sys-metadata-repository) - OS_AUTH_BASE_URL / AUTH_BASE_URL → OS_AUTH_URL (cli serve) Reclassified to silent (kept, no more nag): DATABASE_URL, AUTH_SECRET, BETTER_AUTH_SECRET, BETTER_AUTH_URL, CORS_*, LOG_LEVEL, ROOT_DOMAIN, MCP_SERVER_*, PORT. Also updated now-stale comments + plugin-org-scoping / plugin-security READMEs + object.zod / protocol.zod describe text to the canonical names. The generic readEnvWithDeprecation helper + env.test.ts (synthetic names) are unchanged. Verified: types env (7), cli (436), objectql (718), driver-sql (234), plugin-auth (175), mcp (53), plugin-hono-server (40) all green; spec api-surface check unchanged; 53 build tasks green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
documentation
25 tasks
Contributor
📓 Docs Drift CheckThis PR changes 12 package(s): 107 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:
|
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #2379. Part of the 11.0 breaking-change batch (#2364).
Scope (per owner decision: remove only ObjectStack's own renames; keep ecosystem standards)
Removed — rename required:
OS_MULTI_TENANTOS_MULTI_ORG_ENABLEDOBJECTSTACK_METADATA_WRITABLEOS_METADATA_WRITABLEOS_AUTH_BASE_URL,AUTH_BASE_URLOS_AUTH_URLKept (reclassified to
silent— permanent conventions, no longer nag):DATABASE_URL,AUTH_SECRET,BETTER_AUTH_SECRET,BETTER_AUTH_URL,PORT,CORS_*,LOG_LEVEL,ROOT_DOMAIN,MCP_SERVER_*.Changes
resolveMultiOrgEnabledreads onlyOS_MULTI_ORG_ENABLED;sys-metadata-repositoryonlyOS_METADATA_WRITABLE;serveauth-url drops the two OS-own aliases (keepsBETTER_AUTH_URLsilent).{ silent: true }.readEnvWithDeprecationhelper +env.test.tsunchanged (CHANGELOGs/ADR-0010 left as historical record).Verification
types env 7, cli 436, objectql 718, driver-sql 234, plugin-auth 175, mcp 53, plugin-hono-server 40 — all green. spec
check:api-surfaceunchanged; 53 build tasks green. No test referenced the removed aliases.🤖 Generated with Claude Code