Conversation
- don't stash existing changes by default when running bootstrap script - add exportenv function to export env variables from .env (not the same as 'source .env') - remove unsupported cursor extension
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdjusts the bootstrap script to avoid stashing local changes by default, adds a helper function for exporting variables from .env, and removes an unsupported VS Code Cursor extension from the tracked extensions list. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Changing the bootstrap flag from
-d(DIRTY) to-c(CLEAN) and inverting the default behavior is a breaking change for any existing scripts or habits; consider either supporting both flags temporarily or adding a clear deprecation path/usage message so callers don’t get surprising behavior.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Changing the bootstrap flag from `-d` (DIRTY) to `-c` (CLEAN) and inverting the default behavior is a breaking change for any existing scripts or habits; consider either supporting both flags temporarily or adding a clear deprecation path/usage message so callers don’t get surprising behavior.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| # Export environment variables from .env file (TIL this is not the same as `source .env`) | ||
| function exportenv() { | ||
| export $(grep -v '^#' "${1:-".env"}" | xargs) | ||
| } |
There was a problem hiding this comment.
Word splitting corrupts env vars with spaces
Low Severity
The exportenv function uses an unquoted command substitution export $(...) which causes bash word splitting on the output. When .env files contain values with spaces (e.g., MESSAGE="Hello World"), the value gets split incorrectly, resulting in malformed variable assignments. The function will silently set wrong values or fail for common .env patterns.
Note
Adjusts bootstrap behavior and adds a small utility.
bootstrap.sh: replace-dwith-cflag and switch logic to stash only when-cis passed (DIRTY->CLEAN)linkme/.functions: addexportenvto export variables from a.envfilelinkme/.config/code/extensions.txt: removegrapecity.gc-excelviewerWritten by Cursor Bugbot for commit eca735c. This will update automatically on new commits. Configure here.