Merged
Conversation
- Added `noPodsLogged` flag to avoid repeatedly logging "no pods" warnings for a service. - Reset flag when pods become available, allowing future warnings if pods disappear again.
…pod logs streamer
…issues in TUI pod logs streamer
…for navigation in browse modal
- Added tests for various RootModel functionalities, including focus cycling, resizing, and key/mouse interactions. - Implemented event handler tests for pod lifecycle events, service removals, and metrics updates. - Added TUI component tests for header, services, and detail models.
…, and error handling - Introduced comprehensive tests for BrowseModel initialization, navigation across contexts, namespaces, and services, and visibility toggling. - Verified loading state behavior and ensured error state handling clears on user interaction. - Improved test coverage for edge scenarios like zero visible items and invalid user inputs.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #358 +/- ##
==========================================
+ Coverage 60.65% 63.93% +3.28%
==========================================
Files 69 70 +1
Lines 11823 12457 +634
==========================================
+ Hits 7171 7965 +794
+ Misses 4356 4164 -192
- Partials 296 328 +32
🚀 New features to boost your workflow:
|
…pendency stability
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a TUI Browse Modal for discovering and forwarding Kubernetes services directly from the terminal interface. Users can now press
fto open an interactive browser that navigates through contexts, namespaces, and services - then forward individual services or entire namespaces without leaving the TUI.New Features
Browse Modal (
pkg/fwdtui/components/browse.go- 831 lines)j/kor arrows,Enterto select,Esc/←to go back,cto switch contextsHeader Updates
fkey) to header barPod Logs Streamer Improvements
Log Spam Prevention
Test Coverage Improvements
browse_test.go)tui_test.go)pkg/fwdtuicoverage: 4.63% → 74.8%Type of Change
Related Issues
Fixes #
Testing
go test ./...locallyCoverage Results:
All checks pass:
go test ./pkg/...- All tests passgo test -race ./pkg/...- No race conditionsgolangci-lint run ./pkg/...- 0 issuesChecklist
go fmt,go vet)Screenshots/Logs (if applicable)
Browse Modal Keyboard Shortcuts
fj/↓k/↑gGEnter/→Esc/←/qcFiles Changed
pkg/fwdtui/components/browse.gopkg/fwdtui/components/browse_test.gopkg/fwdtui/tui_test.gopkg/fwdtui/ui.gopkg/fwdapi/manager.gocmd/kubefwd/services/services.gopkg/fwdtui/styles/styles.gopkg/fwdtui/components/header.gopkg/fwdservice/fwdservice.go