Skip to content

install.ps1: use HTML redirect endpoint to avoid GitHub API rate limit#549

Open
wesm wants to merge 1 commit into
mainfrom
install-ps1-html-redirect
Open

install.ps1: use HTML redirect endpoint to avoid GitHub API rate limit#549
wesm wants to merge 1 commit into
mainfrom
install-ps1-html-redirect

Conversation

@wesm
Copy link
Copy Markdown
Member

@wesm wesm commented May 26, 2026

Follow-up to #546, which fixed install.sh and the in-binary agentsview update command but missed install.ps1.

Get-LatestVersion now requests github.com/kenn-io/agentsview/releases/latest with MaximumRedirection=0 and reads the Location header instead of calling api.github.com (60 req/hr per IP for unauthenticated callers).

Handles both PowerShell 5.x (throws WebException on 3xx) and PowerShell 7+ (returns the 302 response normally).

🤖 Generated with Claude Code

Follow-up to #546, which fixed install.sh and the in-binary update
command but missed install.ps1. Get-LatestVersion now requests
github.com/kenn-io/agentsview/releases/latest with
MaximumRedirection=0 and reads the Location header instead of calling
api.github.com (60 req/hr per IP for unauthenticated callers).

Handles both PowerShell 5.x (throws WebException on 3xx) and
PowerShell 7+ (returns the 302 response normally).

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

roborev-ci Bot commented May 26, 2026

roborev: Combined Review (e19f0b5)

High severity issue found; PR is not ready to merge.

High

  • scripts/install.ps1
    • A typo breaks the PowerShell installer: Invoke-WebRequest @frontend/src/lib/utils/tool-params.test.ts is invalid PowerShell syntax and will fail.
    • Fix: replace it with Invoke-WebRequest @params.

Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

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