Skip to content

Comments

Set SimpleTerminalCollapsible default true for Insiders, false for Stable#296911

Merged
justschen merged 2 commits intojustin/petililfrom
copilot/sub-pr-296870
Feb 23, 2026
Merged

Set SimpleTerminalCollapsible default true for Insiders, false for Stable#296911
justschen merged 2 commits intojustin/petililfrom
copilot/sub-pr-296870

Conversation

Copy link
Contributor

Copilot AI commented Feb 23, 2026

ChatConfiguration.SimpleTerminalCollapsible was hardcoded to false. Per feedback, this experimental setting should be on by default in Insiders to gather signal before a wider rollout.

  • chat.contribution.ts: Changed default from false to typeof product.quality === 'string' && product.quality !== 'stable' — consistent with the pattern used by other experimental settings in the same file.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…able

Co-authored-by: justschen <54879025+justschen@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback and implement suggestions for terminal tools PR Set SimpleTerminalCollapsible default true for Insiders, false for Stable Feb 23, 2026
@justschen justschen marked this pull request as ready for review February 23, 2026 06:43
@justschen justschen merged commit da96f99 into justin/petilil Feb 23, 2026
2 of 3 checks passed
@justschen justschen deleted the copilot/sub-pr-296870 branch February 23, 2026 06:43
justschen added a commit that referenced this pull request Feb 23, 2026
* simple terminal tools on surface level of chat

* Set SimpleTerminalCollapsible default true for Insiders, false for Stable (#296911)

* Initial plan

* Set SimpleTerminalCollapsible default true for insiders, false for stable

Co-authored-by: justschen <54879025+justschen@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: justschen <54879025+justschen@users.noreply.github.com>

* make sure it is initialized the right way

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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