892f578565
Address golangci-lint findings and improve error handling throughout:
Package doc comments:
- Add canonical "// Package X ..." comments to source, model, config,
pipeline, cli, store, and main packages for godoc compliance.
Security & correctness:
- Fix directory permissions 0o755 -> 0o750 in store/cache.go Open()
(gosec G301: restrict group write on cache directory)
- Fix config.Save() to check encoder error before closing file, preventing
silent data loss on encode failure
- Add //nolint:gosec annotations with justifications on intentional
patterns (constructed file paths, manual bounds checking, config fields)
- Add //nolint:nilerr on intentional error-swallowing in scanner WalkDir
- Add //nolint:revive on stuttering type names (ModelStats, ModelUsage)
that would break too many call sites to rename
Performance (perfsprint):
- Replace fmt.Sprintf("%d", n) with strconv.FormatInt(n, 10) in format.go
FormatTokens() and FormatNumber() hot paths
- Clean up redundant fmt.Sprintf patterns in FormatCost and FormatDelta
Code cleanup:
- Convert if-else chain to switch in parser.go skipJSONString() for clarity
- Remove unused indexedResult struct from pipeline/loader.go
- Add deferred cache.Close() in pipeline/bench_test.go to prevent leaks
- Add deferred cache.Close() in cmd/root.go data loading path
- Fix doc comment alignment in scanner.go decodeProjectName
- Remove trailing blank line in cmd/costs.go
- Fix duplicate "/day" suffix in cmd/summary.go cost-per-day formatting
- Rename shadowed variable 'max' -> 'maxVal' in cli/render.go Sparkline
41 lines
911 B
Go
41 lines
911 B
Go
package config
|
|
|
|
import (
|
|
"encoding/json"
|
|
"os"
|
|
"path/filepath"
|
|
)
|
|
|
|
// PlanInfo holds detected Claude subscription plan info.
|
|
type PlanInfo struct {
|
|
BillingType string
|
|
PlanCeiling float64
|
|
}
|
|
|
|
// DetectPlan reads ~/.claude/.claude.json to determine the billing plan.
|
|
func DetectPlan(claudeDir string) PlanInfo {
|
|
path := filepath.Join(claudeDir, ".claude.json")
|
|
data, err := os.ReadFile(path) //nolint:gosec // path is constructed from known claudeDir
|
|
if err != nil {
|
|
return PlanInfo{PlanCeiling: 200} // default to Max plan
|
|
}
|
|
|
|
var raw struct {
|
|
BillingType string `json:"billingType"`
|
|
}
|
|
if err := json.Unmarshal(data, &raw); err != nil {
|
|
return PlanInfo{PlanCeiling: 200}
|
|
}
|
|
|
|
info := PlanInfo{BillingType: raw.BillingType}
|
|
|
|
switch raw.BillingType {
|
|
case "stripe_subscription":
|
|
info.PlanCeiling = 200 // Max plan default
|
|
default:
|
|
info.PlanCeiling = 100 // Pro plan default
|
|
}
|
|
|
|
return info
|
|
}
|