Skip to content

feat: DH-22990: improve clear command behaviour in console input#2709

Open
dsmmcken wants to merge 2 commits into
mainfrom
dmckenzie_clear
Open

feat: DH-22990: improve clear command behaviour in console input#2709
dsmmcken wants to merge 2 commits into
mainfrom
dmckenzie_clear

Conversation

@dsmmcken

@dsmmcken dsmmcken commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Typing clear/cls in the console was unreliable — Monaco autocomplete would hijack Enter with a fuzzy match instead of running the command. This makes clearing the console reliable and discoverable.

  • a single enter now always runs clear/cls even with the suggest widget open
  • clear/cls now show in autocomplete (without suppressing word-based suggestions).
  • added "Clear" to the console overflow menu
  • added cmd/ctrl+l shortcut (updates live when rebound in settings), replacing previous shortcut that was supposed to clear the input but didn't actually do anything because monaco ate it

Not tested on mac. Please test.

Not sure how much gets reused in enterprise of this.

@dsmmcken dsmmcken requested a review from mofojed June 26, 2026 21:23
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 26.47059% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.86%. Comparing base (0c7d87c) to head (a0df86a).

Files with missing lines Patch % Lines
packages/console/src/ConsoleInput.tsx 36.84% 12 Missing ⚠️
packages/console/src/monaco/MonacoProviders.tsx 0.00% 7 Missing ⚠️
packages/console/src/Console.tsx 25.00% 3 Missing ⚠️
packages/console/src/ClearCommands.ts 33.33% 2 Missing ⚠️
packages/console/src/monaco/MonacoUtils.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2709      +/-   ##
==========================================
- Coverage   51.10%   50.86%   -0.25%     
==========================================
  Files         793      794       +1     
  Lines       45204    45228      +24     
  Branches    11703    11510     -193     
==========================================
- Hits        23103    23003     -100     
- Misses      22055    22206     +151     
+ Partials       46       19      -27     
Flag Coverage Δ
unit 50.86% <26.47%> (-0.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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