Skip to content

[IMP] common: bash timeout, styled prompts, and playground warning#161

Closed
sea-odoo wants to merge 3 commits into
betafrom
imp/bash-timeout-and-playground-warning
Closed

[IMP] common: bash timeout, styled prompts, and playground warning#161
sea-odoo wants to merge 3 commits into
betafrom
imp/bash-timeout-and-playground-warning

Conversation

@sea-odoo

@sea-odoo sea-odoo commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

This PR groups three improvements:

  1. Adds timeout parameter and encoding safety (errors='replace') to bash executions.
  2. Patches SimplePrompt to support styled lists of tuples for questions.
  3. Warns if the odev repository path is located inside the playground folder.

@sea-odoo sea-odoo force-pushed the imp/bash-timeout-and-playground-warning branch 2 times, most recently from eeaaca2 to 34189c5 Compare June 2, 2026 13:12
sea-odoo added 3 commits June 2, 2026 15:31
- Add warning if resolved odev path is in or matches the home playground directory.
- Add test case verifying the playground warning logic.
@sea-odoo sea-odoo force-pushed the imp/bash-timeout-and-playground-warning branch from 34189c5 to 440217a Compare June 2, 2026 13:32
@sea-odoo sea-odoo closed this Jun 2, 2026
@sea-odoo sea-odoo deleted the imp/bash-timeout-and-playground-warning branch June 9, 2026 08:29
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