Skip to content

fix(sdk): included files#4627

Merged
tothandras merged 1 commit into
mainfrom
fix/sdk-files
Jul 1, 2026
Merged

fix(sdk): included files#4627
tothandras merged 1 commit into
mainfrom
fix/sdk-files

Conversation

@tothandras

Copy link
Copy Markdown
Contributor

No description provided.

@tothandras
tothandras requested a review from a team as a code owner July 1, 2026 11:49
@tothandras tothandras added the release-note/ignore Ignore this change when generating release notes label Jul 1, 2026
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • api/client/javascript/package.json is excluded by !api/client/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c128b4ce-e892-4720-93f3-93ad84695c21

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/sdk-files

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tothandras
tothandras merged commit f29b60b into main Jul 1, 2026
24 of 26 checks passed
@tothandras
tothandras deleted the fix/sdk-files branch July 1, 2026 11:50
@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR tightens the JavaScript SDK npm package contents.

  • Adds a files allowlist to api/client/javascript/package.json.
  • Limits published package files to the built dist directory.
  • Leaves the package entrypoints and export map pointed at dist paths.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The declared runtime and type entrypoints already resolve under dist.
  • The publish lifecycle builds the package before publishing.

Important Files Changed

Filename Overview
api/client/javascript/package.json Adds a files allowlist for the SDK package while keeping all declared runtime and type entrypoints under dist.

Reviews (1): Last reviewed commit: "fix(sdk): included files" | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/ignore Ignore this change when generating release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants