Skip to content

enhance: add playwright installation step to environment setup scripts#189

Merged
vcfgv merged 1 commit intomainfrom
bugfix/install-playwright-during-init
Oct 27, 2025
Merged

enhance: add playwright installation step to environment setup scripts#189
vcfgv merged 1 commit intomainfrom
bugfix/install-playwright-during-init

Conversation

@vcfgv
Copy link
Collaborator

@vcfgv vcfgv commented Oct 27, 2025

📝 Pull Request Template

1. Related Issue

Closes # (issue number)

Type of Change (select one)

Enhancement

3. Description

playwright initialization is required by crawl4ai

4. Testing

  • I have tested this locally.
  • I have updated or added relevant tests.

5. Checklist

@DigHuang
Copy link
Contributor

Consider to refactor run backend in cmd into pws, we not support select anymore

Write-Info "Launching in CMD for better interactive terminal support..."

# Use cmd.exe for better interactive support with questionary
# CMD handles ANSI escape sequences and arrow keys better than PowerShell
$launchCmd = "cd /d `"$PY_DIR`" && uv run --with questionary --with colorama scripts/launch.py"
Start-Process "cmd.exe" -ArgumentList "/k", $launchCmd -Wait

@vcfgv
Copy link
Collaborator Author

vcfgv commented Oct 27, 2025

Consider to refactor run backend in cmd into pws, we not support select anymore

Write-Info "Launching in CMD for better interactive terminal support..."

# Use cmd.exe for better interactive support with questionary
# CMD handles ANSI escape sequences and arrow keys better than PowerShell
$launchCmd = "cd /d `"$PY_DIR`" && uv run --with questionary --with colorama scripts/launch.py"
Start-Process "cmd.exe" -ArgumentList "/k", $launchCmd -Wait

Please add a Suggest Change for better incorporation.

@vcfgv vcfgv merged commit f267c0d into main Oct 27, 2025
3 checks passed
@vcfgv vcfgv deleted the bugfix/install-playwright-during-init branch October 27, 2025 09:34
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.

2 participants