Skip to content

chore: add .vercelignore + ignore .vercel link state#67

Open
rohitg00 wants to merge 1 commit intomainfrom
chore/vercelignore
Open

chore: add .vercelignore + ignore .vercel link state#67
rohitg00 wants to merge 1 commit intomainfrom
chore/vercelignore

Conversation

@rohitg00
Copy link
Copy Markdown
Collaborator

@rohitg00 rohitg00 commented Apr 30, 2026

Summary

Two small fixes so the Vercel deploy works without a deploy hook secret.

.vercelignore

Vercel project `agentos` has `rootDirectory=website`. Default `vercel deploy` from repo root uploads everything not gitignored — including `target/` (8GB), `node_modules/` (~130MB), and the rest of the Rust workspace. That exceeds Vercel's 10MB request limit and the deploy fails.

`.vercelignore` scopes the upload to `website/` only.

.gitignore

Adds `.vercel` so contributor-local link state stays out of git.

Context

Today's prod deploy succeeded once `.vercelignore` was in place. Without committing it, every contributor running `vercel deploy` from the repo root will hit the same limit.

Test plan

  • `vercel deploy --prod` succeeded with this ignore in place — agentsos.sh now serves the redesigned site
  • Future contributor runs `vercel deploy` after linking and gets a clean upload

Open in Devin Review

Summary by CodeRabbit

  • Chores
    • Updated configuration files to exclude temporary files and build artifacts from version control and deployment processes, optimizing repository cleanliness and deployment efficiency.

Vercel project rootDirectory=website. Default upload from repo root pulls
target/ (8GB), node_modules/, .git, etc. — exceeds 10MB request limit.
.vercelignore scopes upload to website/ only.

Also gitignore .vercel link state so contributors can link their own.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 955c8310-d3f8-48d3-a2c0-fb4726f336b6

📥 Commits

Reviewing files that changed from the base of the PR and between 267ac0a and 40676be.

📒 Files selected for processing (3)
  • .gitignore
  • .vercelignore
  • website/.gitignore

📝 Walkthrough

Walkthrough

The changes add Vercel-related configuration to ignore files across the project, preventing version control and deployment build artifacts from being tracked. A new .vercelignore file specifies which files and directories Vercel should exclude during builds, while .vercel is added to both root and website .gitignore files.

Changes

Cohort / File(s) Summary
Vercel Artifacts
.gitignore, website/.gitignore
Added .vercel ignore rule to prevent local Vercel-generated files from being tracked by Git.
Vercel Build Configuration
.vercelignore
New configuration file that specifies build artifacts, dependencies, test suites, scripts, app folders, and generated/log files that Vercel should exclude during deployment.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, skip, and .vercel away,
Build files hidden from Git's display,
Config so clean, deployment so bright,
Vercel ignores keep everything right!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarizes the main changes: adding .vercelignore configuration and ignoring .vercel directory. It is concise, specific, and clearly conveys the primary purpose of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 chore/vercelignore

Review rate limit: 1/3 review remaining, refill in 29 minutes and 6 seconds.

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

Copy link
Copy Markdown

@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 bugs or issues to report.

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