Skip to content

chore: document cloud-split (changeset + CLI stub comment) - #1260

Merged
os-zhuang merged 3 commits into
mainfrom
chore/cloud-split-cleanup
May 21, 2026
Merged

chore: document cloud-split (changeset + CLI stub comment)#1260
os-zhuang merged 3 commits into
mainfrom
chore/cloud-split-cleanup

Conversation

@os-zhuang

Copy link
Copy Markdown
Contributor

Two tiny cleanups closing the loop on the apps/cloud + apps/objectos split (#1257, #1258, #1259):

Changes

  1. .changeset/cloud-split-cleanup.md@objectstack/cli: minor. The CLI no longer hard-depends on @objectstack/service-cloud; cloud-mode is loaded via dynamic import with try/catch. Release notes will explain the impact (and the removed root pnpm dev/start/doctor scripts).

  2. packages/cli/src/types/service-cloud.d.ts — refresh the top-of-file comment. The old wording ("pre-existing typecheck errors in upstream deps") was from the pre-split era. New comment accurately states: package ships from objectstack-ai/cloud, optional dynamic import, any is intentional.

No functional changes. No new tests needed.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

hotlongand others added 2 commits May 21, 2026 21:15
- Add a changeset capturing the cloud-split cleanup so release notes
explain the @objectstack/cli minor (no longer hard-deps service-cloud)
and the removed root scripts.
- Refresh packages/cli/src/types/service-cloud.d.ts top-of-file comment:
the old wording ('pre-existing typecheck errors in upstream deps')
predates the split. The accurate story is now: the package ships
from objectstack-ai/cloud, the CLI loads it via dynamic import inside
a try/catch, and this stub keeps the optional path typechecking.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- SwappableStorageService now forwards verifyToken() to the inner
LocalStorageAdapter. Previously the proxy dropped the method, causing
PUT /api/v1/storage/_local/raw/:token to always return 501 'Presigned
raw upload not supported by this adapter' even when the active adapter
was Local.
- Add GET /api/v1/storage/files/:fileId — a stable, non-JSON sibling of
/files/:fileId/url that 302-redirects to a freshly-signed download URL.
Frontend widgets (ImageField, <img src>, user avatars, org logos) need
a URL that is stable across signed-URL expiry AND serves the bytes
directly. The existing /url variant returns JSON and cannot be used
verbatim as <img src>.
- Wire UploadProvider into @objectstack/console with an inline ObjectStack
presigned-upload adapter so every ImageField/FileField inside the
console now hits the real storage service instead of falling back to
blob: URLs that disappear on reload.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vercel

vercelBot commented May 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
specCanceledCanceledMay 21, 2026 1:22pm

Request Review

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation tooling size/s dependencies Pull requests that update a dependency file size/m and removed documentation Improvements or additions to documentation tooling size/s labels May 21, 2026
Regenerate lockfile after apps/console added @object-ui/providers
in 3f5f826. Unblocks 'Check Changeset' CI which uses
--frozen-lockfile.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@os-zhuang
os-zhuang merged commit 8cbc768 into mainMay 21, 2026
13 checks passed
@os-zhuang
os-zhuang deleted the chore/cloud-split-cleanup branch May 21, 2026 13:27
os-zhuang added a commit that referenced this pull request May 21, 2026
The package was removed from this repo in #1257 (cloud split) but its
entry in .changeset/config.json's `fixed` list was left behind. Every
Release workflow run since then has failed with:
ValidationError: The package or glob expression
"@objectstack/service-cloud" specified in the `fixed` option does
not match any package in the project.
Removing the entry unblocks 'changesets/action', which will then update
PR #1256 (chore: version packages) to bump remaining published packages
4.0.5 -> 4.1.0 (minor, driven by the @objectstack/cli changeset added
in #1260). Merging that PR triggers npm publish.
Co-authored-by: Jack Zhuang <50353452+hotlong@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency filedocumentationImprovements or additions to documentationsize/mtooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@os-zhuang@hotlong