Skip to content

Fixes dot copy - #1037

Merged
mishushakov merged 1 commit into
mainfrom
fix-dot-copy
Nov 28, 2025
Merged

mishushakov merged 1 commit into
mainfrom
fix-dot-copy

Conversation

@mishushakov

@mishushakov mishushakov commented Nov 27, 2025 •

Copy link
Copy Markdown
Member

Note

Fixes glob expansion when source is '.' to avoid absolute '/**/*' traversal and adds targeted tests.

  • Bug fix (js-sdk):
    • Update getAllFilesInPath in packages/js-sdk/src/template/utils.ts to construct a safe directory pattern when the matched directory is '.' (uses path.join(file.relative() || '.', '**/*')) preventing accidental /**/* traversal.
  • Tests:
    • Add coverage for dot pattern in packages/js-sdk/tests/template/utils/getAllFilesInPath.test.ts, ensuring results stay within the context directory.
  • Release:
    • Add changeset '.changeset/humble-badgers-cross.md' (patch for e2b).

Written by Cursor Bugbot for commit aac954e. This will update automatically on new commits. Configure here.

@changeset-bot

changeset-bot Bot commented Nov 27, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: aac954e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
e2b Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@mishushakov
mishushakov merged commit da925b5 into main Nov 28, 2025
12 of 15 checks passed
@mishushakov
mishushakov deleted the fix-dot-copy branch November 28, 2025 11:52

This branch was successfully deployed

1 active deployment
Preview — aac954e0 Deployed Nov 27, 2025 by vercel[bot]
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.

2 participants