Skip to content

fix(cli): ignore OSC colors queries in devcontainer or docker#954

Closed
Emiyaaaaa wants to merge 3 commits intovoidzero-dev:mainfrom
Emiyaaaaa:fix/838
Closed

fix(cli): ignore OSC colors queries in devcontainer or docker#954
Emiyaaaaa wants to merge 3 commits intovoidzero-dev:mainfrom
Emiyaaaaa:fix/838

Conversation

@Emiyaaaaa
Copy link

fixed #838

@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 65b451f
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b8d237ecddb60008c9ffd4

@Emiyaaaaa Emiyaaaaa changed the title fix(cli): cli display issues caused by OSC colors in devcontainer fix(cli): cli display issues caused by OSC colors queries in devcontainer or docker Mar 16, 2026
@Emiyaaaaa Emiyaaaaa changed the title fix(cli): cli display issues caused by OSC colors queries in devcontainer or docker fix(cli): fix cli display issues caused by OSC colors queries in devcontainer or docker Mar 16, 2026
@fengmk2 fengmk2 changed the title fix(cli): fix cli display issues caused by OSC colors queries in devcontainer or docker fix(cli): ignore OSC colors queries in devcontainer or docker Mar 16, 2026
@fengmk2 fengmk2 requested review from Brooooooklyn and Copilot March 16, 2026 13:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR disables terminal OSC color querying when running inside a Docker container (detected via /.dockerenv) to avoid unreliable/late responses that can leak as visible text after cooked mode is restored.

Changes:

  • Add an early-return guard in query_terminal_colors when /.dockerenv exists.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fengmk2
Copy link
Member

fengmk2 commented Mar 17, 2026

Will provide more complete support in #986

@fengmk2 fengmk2 closed this Mar 17, 2026
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.

bad logs formatting

3 participants