Skip to content

Publish Sage app SDK through trusted npm CI - #834

Merged
Rigidity merged 2 commits into
mainfrom
npm-publish
Aug 19, 2026
Merged

Publish Sage app SDK through trusted npm CI#834
Rigidity merged 2 commits into
mainfrom
npm-publish

Conversation

@Rigidity

@Rigidity Rigidity commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • prepare sage-app-sdk for public npm distribution with package metadata, documentation, and complete exports
  • flatten workspace package names and update all internal imports and dependencies
  • add trusted npm publishing on Sage tags plus package dry runs on main and pull requests

Test plan

  • pnpm build:packages
  • pnpm prettier:check
  • pnpm lint
  • npm pack --dry-run in packages/sage-app-sdk
  • Confirm the npm workflow dry run passes on this PR

Note

Medium Risk
Wide import and package-name churn can break builds or external consumers still on @sage-app/* scopes; tag-triggered npm publish affects public package availability.

Overview
Prepares sage-app-sdk for public npm release (Apache-2.0 LICENSE, README, registry metadata, publishConfig.access: public, prepack build, and a bridge-runtime-core tsup entry) and wires trusted publishing via a new .github/workflows/npm.yml that runs prettier, lint, and npm pack --dry-run on PRs/main and npm publish from packages/sage-app-sdk on any tag ( environment: npm, OIDC id-token: write).

Renames workspace packages from scoped names to sage-app-sdk, sage-app-ui, and sage-system-app-sdk, updating root/builtin-app package.json scripts, pnpm-lock, TS path aliases, CSS theme imports, and docs so the monorepo resolves the new names consistently.

Reviewed by Cursor Bugbot for commit c1d7976. Bugbot is set up for automated code reviews on this repo. Configure here.

Publish the flat-named SDK through trusted CI while keeping workspace package imports consistent.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 46b2b7f. Configure here.

Comment thread .github/workflows/npm.yml
Comment thread .github/workflows/npm.yml
Comment thread .github/workflows/npm.yml
Ensure package type generation can compile its Rust dependencies on clean npm workflow runners.
@Rigidity
Rigidity merged commit 727de89 into main Aug 19, 2026
12 checks passed
@Rigidity
Rigidity deleted the npm-publish branch August 19, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant