Skip to content

chore: scaffolding node-client-sdk migration#1352

Open
joker23 wants to merge 1 commit intomainfrom
skz/sdk-2195/node-client-sdk-next-scaffold
Open

chore: scaffolding node-client-sdk migration#1352
joker23 wants to merge 1 commit intomainfrom
skz/sdk-2195/node-client-sdk-next-scaffold

Conversation

@joker23
Copy link
Copy Markdown
Contributor

@joker23 joker23 commented May 4, 2026

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.


Note

Low Risk
Low risk because this is primarily monorepo/package scaffolding and release configuration with no functional runtime behavior beyond a placeholder version export.

Overview
Adds a new experimental @launchdarkly/node-client-sdk workspace with initial package scaffolding (README/LICENSE/CHANGELOG) and build/lint/test tooling (tsup, TypeScript configs, Jest, ESLint).

Wires the package into the monorepo and release automation by updating root workspaces, TS project references, release-please config/manifest, and ignores .claude/stacks/.

Includes only a placeholder src/index.ts exporting version to support building and version stamping.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

@launchdarkly/js-sdk-common size report
This is the brotli compressed size of the ESM build.
Compressed size: 25623 bytes
Compressed size limit: 29000
Uncompressed size: 125843 bytes

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

@launchdarkly/browser size report
This is the brotli compressed size of the ESM build.
Compressed size: 179311 bytes
Compressed size limit: 200000
Uncompressed size: 830815 bytes

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

@launchdarkly/js-client-sdk-common size report
This is the brotli compressed size of the ESM build.
Compressed size: 38473 bytes
Compressed size limit: 39000
Uncompressed size: 211104 bytes

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

@launchdarkly/js-client-sdk size report
This is the brotli compressed size of the ESM build.
Compressed size: 31866 bytes
Compressed size limit: 34000
Uncompressed size: 113634 bytes

@joker23
Copy link
Copy Markdown
Contributor Author

joker23 commented May 4, 2026

@cursor review

cursor[bot]

This comment was marked as resolved.

@joker23 joker23 force-pushed the skz/sdk-2195/node-client-sdk-next-scaffold branch from fb8608c to 9193aa8 Compare May 4, 2026 21:19
@joker23
Copy link
Copy Markdown
Contributor Author

joker23 commented May 4, 2026

@cursor review

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 9193aa8. Configure here.

@joker23 joker23 marked this pull request as ready for review May 4, 2026 21:57
@joker23 joker23 requested a review from a team as a code owner May 4, 2026 21:57
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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