Skip to content

fix(deploy): increase poll timeout and exclude IDE config dirs#49

Merged
jwfing merged 5 commits intomainfrom
fix/deploy-timeout-and-excludes
Apr 2, 2026
Merged

fix(deploy): increase poll timeout and exclude IDE config dirs#49
jwfing merged 5 commits intomainfrom
fix/deploy-timeout-and-excludes

Conversation

@tonychang04
Copy link
Copy Markdown
Contributor

@tonychang04 tonychang04 commented Apr 1, 2026

Summary

  • Increase deployment poll timeout from 2 minutes to 5 minutes — builds were timing out even when the site was already live
  • Exclude IDE and AI agent config directories (.claude, .agents, .augment, .kilocode, .kiro, .qoder, .qwen, .roo, .trae, .windsurf, skills/, .vercel, .turbo, .cache, coverage) from deploy zip to reduce upload size

decrease the file while increasing timeout to let deployment succeed for templates

Test plan

  • Deploy a project and verify the 5-minute timeout applies
  • Verify excluded directories are not included in the upload zip

🤖 Generated with Claude Code

Note

Increase deployment poll timeout to 5 minutes and exclude IDE config dirs from uploads

  • Increases POLL_TIMEOUT_MS in deploy.ts from 120,000 ms (2 min) to 300,000 ms (5 min) and updates the timeout warning message to match.
  • Expands EXCLUDE_PATTERNS to skip IDE/agent config directories (.claude, .windsurf, .roo, etc.) and build artifacts (.turbo, .cache, coverage) during deployment uploads.

Macroscope summarized e50a3c8.

Summary by CodeRabbit

  • Improvements

    • Increased deployment timeout from 2 to 5 minutes to allow more time for completion.
    • Expanded deployment archive exclusion list to skip various IDE/AI-agent, build cache, and coverage directories, reducing unnecessary package bloat.
    • Updated user-facing warning to reflect the new 5-minute timeout window.
  • Chores

    • Package version bumped to 0.1.39.

tonychang04 and others added 2 commits April 1, 2026 13:18
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ig dirs

Deployments were timing out after 2 minutes even when the site was live.
Also excludes unnecessary IDE and AI agent config directories from the
deploy zip to reduce upload size.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Walkthrough

Package version bumped to 0.1.39 and the deploy command: polling timeout increased to 300000 ms (5 minutes), zip exclusion list extended (IDE/agent and other dirs), and the user-facing timeout warning updated.

Changes

Cohort / File(s) Summary
Package Version Update
package.json
version updated from 0.1.370.1.39.
Deployment Command Configuration
src/commands/deployments/deploy.ts
Polling timeout increased from 120000 ms → 300000 ms (2 → 5 minutes). Zip exclusion list expanded to include .claude, .agents, .windsurf, .vercel, .turbo, .cache, skills, coverage and related patterns. Updated user-facing timeout warning to reference 5 minutes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • Fermionic-Lyu
  • CarmenDou

Poem

🐇 I nudge the clock from two to five,

More time for builds to wake and thrive.
I tuck away cluttered nests — .claude, .cache
Hide the trails where agents dash.
Version lifted, carrots gleam, deploys hop into dream.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the two main changes: increasing the poll timeout and excluding IDE config directories from deployments.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/deploy-timeout-and-excludes

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

@tonychang04 tonychang04 requested review from CarmenDou, Fermionic-Lyu and jwfing and removed request for jwfing April 1, 2026 20:36
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jwfing
jwfing previously approved these changes Apr 2, 2026
@jwfing jwfing self-requested a review April 2, 2026 00:25
@jwfing jwfing enabled auto-merge April 2, 2026 00:26
@jwfing jwfing merged commit e3536b4 into main Apr 2, 2026
3 checks passed
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