Skip to content

tui+refactor#32

Merged
hugoh merged 24 commits into
mainfrom
tui+refactor
Jun 7, 2026
Merged

tui+refactor#32
hugoh merged 24 commits into
mainfrom
tui+refactor

Conversation

@hugoh

@hugoh hugoh commented Jun 6, 2026

Copy link
Copy Markdown
Owner
  • chore(deps): upgrade
  • feat(tui): reorganized selection keymaps
  • feat(tui): added selection history
  • feat(tui): repo grouping display
  • feat(tui): collapsed into single command bar
  • feat(tui): command bar completion
  • refactor(tui): list display
  • refactor(tui): simplified list display
  • refactor(tui): cleanup
  • refactor(tests): convert to testify
  • refactor(backends): abstracted out backends in all cli/tui code
  • chore(deps): upgrade
  • refactor(tui): rename known→err in startExec
  • fix(tui): cancel prior exec before starting new one
  • refactor(tui): replace init() in bindings with lazy getter
  • fix(ui): make SetLogger thread-safe with mutex
  • fix(runner): propagate errgroup error from forEachRepo
  • refactor(runner): document not-found workFn error discard
  • test(tui): convert raw assertions to testify in update_test.go
  • refactor(jj): replace global runJJ var with struct field DI
  • refactor: comments

@deepsource-io

deepsource-io Bot commented Jun 6, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in e97e6f2...24666c2 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Go Jun 7, 2026 2:57a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@codecov

codecov Bot commented Jun 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.31469% with 84 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.88%. Comparing base (e97e6f2) to head (24666c2).

Files with missing lines Patch % Lines
internal/tui/update.go 76.43% 41 Missing and 4 partials ⚠️
internal/tui/helpers.go 88.00% 3 Missing and 3 partials ⚠️
internal/backend/backend.go 37.50% 5 Missing ⚠️
internal/tui/model.go 78.26% 4 Missing and 1 partial ⚠️
cmd/tui.go 0.00% 4 Missing ⚠️
internal/tui/bindings.go 66.66% 4 Missing ⚠️
internal/tui/history.go 90.00% 2 Missing and 2 partials ⚠️
cmd/root.go 91.30% 1 Missing and 1 partial ⚠️
internal/runner/runner.go 60.00% 1 Missing and 1 partial ⚠️
internal/tui/commands.go 81.81% 1 Missing and 1 partial ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   89.41%   88.88%   -0.54%     
==========================================
  Files          23       24       +1     
  Lines        2636     3023     +387     
==========================================
+ Hits         2357     2687     +330     
- Misses        223      270      +47     
- Partials       56       66      +10     

☔ 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.

@sonarqubecloud

sonarqubecloud Bot commented Jun 7, 2026

Copy link
Copy Markdown

@hugoh hugoh merged commit a03b34e into main Jun 7, 2026
6 of 7 checks passed
@hugoh hugoh deleted the tui+refactor branch June 7, 2026 02:59
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