Uh oh!
There was an error while loading. Please reload this page.
feat(cli)!: remove legacy os publish / os rollback commands - #2237
Merged
Conversation
The legacy direct-to-environment publish flow (write sys_environment_revision via os publish, switch revisions via os rollback) is superseded by the versioned-package flow (os package publish). Remove both commands, the now-dangling reference in the package publish header comment, and the unused client activateRevision() helper. BREAKING CHANGE: os publish and os rollback are removed; use os package publish.
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📓 Docs Drift CheckThis PR changes 2 package(s): 15 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:
|
Uh oh!
There was an error while loading. Please reload this page.
os-zhuang added a commit
that referenced
this pull request
Jun 26, 2026
…search, authz, SDUI styling, dataset joins, package-first (#2338) Audit of ~1 week of changes (298 commits, 3 breaking, 20 new ADRs) surfaced hand-written docs that were stale or missing. All shapes verified against the current spec/CLI. P1 — docs were WRONG: - os publish / os rollback removed (#2237): rewrite publish-and-preview around `os package publish`; fix cloud-deployment, cloud-artifact-api, protocol objectos/objectql docs, README, platform skill (migration `os rollback` is gone too — migrate is plan/apply, forward-only). - RLS current_user (ADR-0068): permission.mdx + core-concepts dropped the removed singular `current_user.role` / `current_user.region`; `roles` is a string array. - sharingModel enum (ADR-0055/0056): object.mdx adds public_read / public_read_write / controlled_by_parent + legacy aliases. P2 — shipped but undocumented: - Record search (ADR-0061): data-modeling — `searchableFields` + `$search`. - Unified authz (ADR-0066): security — `sys_capability` registry, object `access` posture, object `requiredPermissions`. - SDUI styling (ADR-0065): page.mdx — `responsiveStyles` (preferred channel). - Dataset multi-hop joins (ADR-0071): analytics-datasets — dotted paths, <=3 hops, to-one only. - Package-first authoring (ADR-0070): metadata-lifecycle — writable-base contract. P3: - Built-in identity roles (ADR-0068) in security; per-env @better-auth/sso + `features.sso` note in auth-sso (ADR-0069). - Status-header correction on ADR-0061 / ADR-0071 (code shipped; headers lagged). Docs-only. Build Docs CI validates MDX. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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
2 participants
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.
What
Removes the legacy CLI commands:
packages/cli/src/commands/publish.ts—os publish(direct-to-environment, wrotesys_environment_revision)packages/cli/src/commands/rollback.ts—os rollback(switched the active revision)Also:
objectstack publishpath" reference in theos package publishheader comment.activateRevision()helper from@objectstack/client(packages/client/src/index.ts) — it had no callers in the workspace.Why
The legacy direct-to-environment publish flow is superseded by the versioned-package flow (
os package publish, ADR-0006 v4 Phase B). oclif discovers commands by file (pattern strategy), so deleting the files removes the commands — no registry to update.src/index.tsonly exportedPackagePublishCommand, and the README already documents onlyos package publish.Breaking change
os publishandos rollbackno longer exist. Useos package publish(+os package install).Not included (needs a decision)
The backend route
POST /api/v1/cloud/environments/:id/revisions/:commit/activatemay still be implemented server-side. This PR only removes the CLI + the unused client helper. If other consumers (Console UI, direct REST) still rely on it, the endpoint should be retired separately.Docs
Companion docs PR in objectstack-ai/objectos removes these from the CLI reference.