Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRenames the PowerShell entry script from install.ps1 to bootstrap.ps1 and updates all in-script documentation and example URLs to point to the new blob storage location and filename. Flow diagram for updated bootstrap.ps1 download and execution URLflowchart TD
A[User runs irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1] --> B[PowerShell downloads bootstrap.ps1 from blob storage]
B --> C[Script content is piped to iex]
C --> D[bootstrap.ps1 starts WelsonJS installation]
A2[User runs irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1 -dev main] --> B
A3[User runs irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1 -dev dev] --> B
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 42 minutes and 44 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Review Summary by QodoRename install script to bootstrap and update URLs
WalkthroughsDescription• Rename install.ps1 to bootstrap.ps1 for clarity • Update download URLs from welson.js.org to Azure Blob Storage • Update script header and usage documentation Diagramflowchart LR
A["install.ps1"] -- "rename" --> B["bootstrap.ps1"]
C["welson.js.org URLs"] -- "migrate to" --> D["Azure Blob Storage URLs"]
B -- "includes" --> E["Updated documentation"]
File Changes1. bootstrap.ps1
|
Code Review by Qodo
1. Broken -dev invocation
|
PR Summary
|
|
| # irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1 | iex -dev main | ||
| # irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1 | iex -dev dev |
There was a problem hiding this comment.
1. Broken -dev invocation 🐞 Bug ≡ Correctness
The updated usage examples pipe the script into iex -dev ..., but -dev is parsed as a parameter to Invoke-Expression (which doesn’t accept it), so the command errors before bootstrap.ps1 executes and users cannot select a branch.
Agent Prompt
### Issue description
`bootstrap.ps1` documents `irm <url> | iex -dev main` / `... -dev dev`, but `-dev` is not a valid parameter for `Invoke-Expression`, so the command fails before the downloaded script text executes.
### Issue Context
The script defines a `param([string]$dev = $DefaultBranch)` parameter and does not use `$args`, so callers must invoke the downloaded content as a ScriptBlock and pass `-dev` to that ScriptBlock.
### Fix
Update the header usage examples to a working invocation pattern, e.g.:
```powershell
# default
irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1 | iex
# with branch selection
& ([ScriptBlock]::Create((irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1))) -dev main
& ([ScriptBlock]::Create((irm https://catswords.blob.core.windows.net/welsonjs/bootstrap.ps1))) -dev dev
```
(Using `Invoke-RestMethod` explicitly instead of `irm` is also fine.)
### Fix Focus Areas
- bootstrap.ps1[6-18]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Summary by Sourcery
Rename the PowerShell one-click script to bootstrap.ps1 and update its documented remote execution URLs to the new hosting location.
Enhancements: