Skip to content

Fix Windows install and review edge cases#1021

Open
ShiroKSH wants to merge 1 commit into
backnotprop:mainfrom
ShiroKSH:agent/windows-installer-pi-amp-fixes
Open

Fix Windows install and review edge cases#1021
ShiroKSH wants to merge 1 commit into
backnotprop:mainfrom
ShiroKSH:agent/windows-installer-pi-amp-fixes

Conversation

@ShiroKSH

@ShiroKSH ShiroKSH commented Jul 8, 2026

Copy link
Copy Markdown

Summary

  • Normalize Amp workspace and binary candidate paths across platforms.
  • Keep the PowerShell installer parseable under Windows PowerShell and make installer tests line-ending safe.
  • Abort active Ask AI server turns when sessions or threads reset.
  • Match Bun behavior for empty Pi archive entries and Pi config label persistence.
  • Drop debug logging on expected browser fallback paths.

Validation

  • bun test apps/amp-plugin/plannotator.test.ts scripts/install.test.ts
  • bun test apps/pi-extension/server.test.ts -t 'pi plan archive server'
  • PowerShell parser check for scripts/install.ps1
  • bun run typecheck with Git Bash on PATH

Notes

Full bun test still has unrelated Windows failures in symlink, semantic-diff, temp-lock, and path-expectation suites.

@ShiroKSH ShiroKSH marked this pull request as ready for review July 8, 2026 16:55
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