From 1edbd19f2500995d5afa080d30595873ee4c4fdc Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 14:38:03 +0100 Subject: [PATCH] ax(mining): rename cb to circuitBreaker in FetchLatestGitHubVersion MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit AX Principle 1 — predictable names over short names. The local variable `cb` requires mental mapping; `circuitBreaker` names the value directly. Co-Authored-By: Charon --- pkg/mining/version.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/mining/version.go b/pkg/mining/version.go index c34ee86..1cd49cf 100644 --- a/pkg/mining/version.go +++ b/pkg/mining/version.go @@ -36,16 +36,16 @@ type GitHubRelease struct { // tag, err := FetchLatestGitHubVersion("xmrig", "xmrig") // "v6.21.0" func FetchLatestGitHubVersion(owner, repo string) (string, error) { - cb := getGitHubCircuitBreaker() + circuitBreaker := getGitHubCircuitBreaker() - result, err := cb.Execute(func() (interface{}, error) { + result, err := circuitBreaker.Execute(func() (interface{}, error) { return fetchGitHubVersionDirect(owner, repo) }) if err != nil { // If circuit is open, try to return cached value with warning if err == ErrCircuitOpen { - if cached, ok := cb.GetCached(); ok { + if cached, ok := circuitBreaker.GetCached(); ok { if tagName, ok := cached.(string); ok { return tagName, nil }