Skip to content

fix: agentcore dev not working in windows#951

Merged
avi-alpert merged 1 commit intoaws:mainfrom
avi-alpert:aalpert/windows-dev
Apr 24, 2026
Merged

fix: agentcore dev not working in windows#951
avi-alpert merged 1 commit intoaws:mainfrom
avi-alpert:aalpert/windows-dev

Conversation

@avi-alpert
Copy link
Copy Markdown
Contributor

@avi-alpert avi-alpert commented Apr 24, 2026

Description

On Windows, agentcore dev fails with Error: spawn start ENOENT when trying to open the browser. This is because start is a cmd.exe built-in, not a standalone executable, so spawn('start', [url]) can't find it.

The fix spawns cmd directly with /c start as arguments (spawn('cmd', ['/c', 'start', url])). Also extracts the browser-opening logic into a reusable openBrowser() utility in utils.ts with tests for all three platforms (macOS, Windows, Linux).

Related Issue

Closes #945

Documentation PR

N/A

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots
  • Manually tested agentcore dev on a Windows machine and verified the browser opens without errors

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@avi-alpert avi-alpert requested a review from a team April 24, 2026 18:04
@github-actions github-actions Bot added the size/s PR size: S label Apr 24, 2026
@avi-alpert avi-alpert merged commit 5271f55 into aws:main Apr 24, 2026
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error: spawn start ENOENT

2 participants