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

hotlong and 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
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

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

Project Deployment Actions Updated (UTC)
spec Canceled Canceled May 21, 2026 1:22pm

Request Review

@github-actions github-actions Bot 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 main May 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation size/m tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants