Skip to content

fix(gitignore): add daytona.js to packages/cli/.gitignore#3349

Open
la14-1 wants to merge 1 commit intomainfrom
fix/daytona-gitignore
Open

fix(gitignore): add daytona.js to packages/cli/.gitignore#3349
la14-1 wants to merge 1 commit intomainfrom
fix/daytona-gitignore

Conversation

@la14-1
Copy link
Copy Markdown
Member

@la14-1 la14-1 commented Apr 24, 2026

Why: daytona.js build artifact was not in .gitignore, causing it to be staged by git add -A after a local build while all other cloud bundles (aws.js, hetzner.js, etc.) were correctly ignored. This is a footgun — the committed artifact would be ~3 MB and drift from source.

Fixes #3348.

-- refactor/code-health

@la14-1
Copy link
Copy Markdown
Member Author

la14-1 commented Apr 28, 2026

This PR has been open for 4+ days with all CI checks passing (ShellCheck, Biome Lint, Unit Tests, Mock Tests, macOS Compatibility — all SUCCESS). It is mergeable with no conflicts.

Requesting human review when convenient.

-- spawn-refactor/pr-maintainer

@la14-1 la14-1 force-pushed the fix/daytona-gitignore branch from 53b9c1b to 3947807 Compare April 30, 2026 05:07
@la14-1
Copy link
Copy Markdown
Member Author

la14-1 commented May 3, 2026

This PR is ready for review. All CI checks pass and there are no merge conflicts.

-- refactor/pr-maintainer

Why: daytona.js build artifact was not ignored, causing it to be staged
by git add -A after a local build while other cloud bundles were correctly
ignored. Fixes #3348.

Agent: code-health
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@la14-1 la14-1 force-pushed the fix/daytona-gitignore branch from 3947807 to 93839b9 Compare May 3, 2026 14:19
@la14-1
Copy link
Copy Markdown
Member Author

la14-1 commented May 3, 2026

Rebased onto main (was 4 commits behind). Clean rebase, no conflicts.

-- refactor/pr-maintainer

@la14-1
Copy link
Copy Markdown
Member Author

la14-1 commented May 4, 2026

Verified in worktree: tests pass (2236/2238, same 2 pre-existing failures as main), lint clean (biome 0 errors). PR is mergeable and ready for review.

-- refactor/pr-maintainer

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.

[Bug]: daytona.js missing from packages/cli/.gitignore

2 participants