From 69f5017434298f1ffd4cdc30c30b95d0f4bd344f Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Wed, 18 Feb 2026 10:10:39 +0200 Subject: refactor: enforce Go best practices (function size, ordering, formatting) - config: split set() (62L) into setSizeAndTuning() + setDisplayFlags() - collector: split Run() (70L) into Run + startLocalScanner + startRemoteScanner + parseCollectorStream + dispatchCollectorLine; each <30L - collector: remove unused script_embed.go (RemoteScript was dead code) - display: move newRunState constructor before Run() per constructor-first rule - display: replace loadPeak IIFE with a plain initLoadPeak variable - display: split handleKey() (114L) into handleToggleKeys + handleAdjustAndSave + handleResizeKeys; add nil guard in handleResizeKeys for test safety - display: split drawNetBarSmoothed() (75L) into drawNetBarSmoothed + smoothNetUtilization + drawNetHalves; each <30L - all: gofmt -w to fix formatting drift in display and config files All tests pass (go test ./...). Co-Authored-By: Claude Sonnet 4.6 --- internal/collector/script_embed.go | 9 --------- 1 file changed, 9 deletions(-) delete mode 100644 internal/collector/script_embed.go (limited to 'internal/collector/script_embed.go') diff --git a/internal/collector/script_embed.go b/internal/collector/script_embed.go deleted file mode 100644 index 168ca37..0000000 --- a/internal/collector/script_embed.go +++ /dev/null @@ -1,9 +0,0 @@ -package collector - -import _ "embed" - -// RemoteScript is the loadbars-remote.sh script embedded for local and SSH execution. -// Path is relative to this file's directory (internal/collector). -// -//go:embed scriptdata/loadbars-remote.sh -var RemoteScript []byte -- cgit v1.2.3