Skip to content

Build history command using bubbletea#182

Draft
spaulg wants to merge 1 commit into
mainfrom
181-log-history-command-implemented-in-bubbletea
Draft

Build history command using bubbletea#182
spaulg wants to merge 1 commit into
mainfrom
181-log-history-command-implemented-in-bubbletea

Conversation

@spaulg

@spaulg spaulg commented Oct 9, 2025

Copy link
Copy Markdown
Owner

Build history command using bubbletea

@spaulg
spaulg requested a review from Copilot October 9, 2025 20:25

Copilot AI left a comment

Copy link
Copy Markdown

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 implements a new "buildlog" command using the bubbletea TUI library to display build history information interactively.

  • Adds a new BuildLogModel struct implementing the bubbletea Model interface for interactive build log viewing
  • Creates a new buildlog subcommand that initializes and runs the bubbletea program
  • Updates Go version and adds bubbletea dependencies to support the new TUI functionality

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

File Description
internal/pkg/impl/host/bubbletea/models/buildlog.go Implements the bubbletea model for the build log view with basic structure and keyboard handling
go.mod Updates Go version to 1.24.0 and adds bubbletea dependencies
cmd/solo/subcommand/root.go Registers the new buildlog subcommand with the root command
cmd/solo/subcommand/buildLog.go Implements the buildlog subcommand that creates and runs the bubbletea program

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread internal/pkg/impl/host/bubbletea/models/buildlog.go Outdated
Comment thread internal/pkg/impl/host/bubbletea/models/buildlog.go Outdated
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 6 times, most recently from f0046d6 to 396d431 Compare October 15, 2025 21:03
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 396d431 to 112978d Compare October 21, 2025 07:41
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 2 times, most recently from d211170 to d26c1d9 Compare November 4, 2025 20:12
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from d26c1d9 to 58bfa2d Compare December 10, 2025 07:53
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 48cd55d to 9bd98e3 Compare February 8, 2026 21:46
@codecov

codecov Bot commented Feb 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 11.42857% with 558 lines in your changes missing coverage. Please review.
✅ Project coverage is 34.96%. Comparing base (4ef290a) to head (9f64c93).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...g/host/app/bubbletea/components/table/component.go 0.00% 193 Missing ⚠️
.../app/bubbletea/models/workflow_event_tree/model.go 0.00% 77 Missing ⚠️
.../pkg/host/app/bubbletea/views/workflow_log/view.go 0.00% 69 Missing ⚠️
.../bubbletea/models/execution_event_history/model.go 0.00% 61 Missing ⚠️
...bubbletea/models/execution_event_overview/model.go 0.00% 48 Missing ⚠️
...host/app/bubbletea/components/heading/component.go 0.00% 21 Missing ⚠️
.../pkg/host/infra/repository/json_file_repository.go 0.00% 20 Missing ⚠️
internal/pkg/host/app/bubbletea/layout/manager.go 73.61% 19 Missing ⚠️
...pp/bubbletea/models/workflow_event_output/model.go 0.00% 19 Missing ⚠️
internal/pkg/host/app/bubbletea/layout/spec.go 41.66% 14 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #182      +/-   ##
==========================================
- Coverage   40.41%   34.96%   -5.46%     
==========================================
  Files          62       72      +10     
  Lines        2766     3378     +612     
==========================================
+ Hits         1118     1181      +63     
- Misses       1590     2139     +549     
  Partials       58       58              

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

@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 3 times, most recently from f03d2d7 to 10d1884 Compare February 10, 2026 21:43
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 7483f9d to fc8bdd1 Compare February 19, 2026 22:21
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 3d06a00 to af67770 Compare March 7, 2026 22:14
@spaulg
spaulg marked this pull request as draft March 23, 2026 21:22
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 4 times, most recently from f0ca39e to 3444031 Compare April 4, 2026 15:02
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 6 times, most recently from 134f9a8 to 4deec1d Compare June 25, 2026 21:46
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch 4 times, most recently from a27efec to 8ddec63 Compare July 5, 2026 13:51
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 8ddec63 to 632b2f1 Compare July 6, 2026 17:40
Build history command using bubbletea
@spaulg
spaulg force-pushed the 181-log-history-command-implemented-in-bubbletea branch from 632b2f1 to 9f64c93 Compare July 6, 2026 17:42
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