Skip to content

refactor(config): remove legacy ccusage service and deprecated ccotel config - #302

Merged
AnnatarHe merged 1 commit into
mainfrom
refactor/remove-ccusage-v1
Sep 26, 2026
Merged

AnnatarHe merged 1 commit into
mainfrom
refactor/remove-ccusage-v1

Conversation

@AnnatarHe

@AnnatarHe AnnatarHe commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Removes legacy config and the ccusage v1 daemon service.

  • Deleted model/ccusage_service*.go and its tests. The daemon no longer runs bunx/npx ccusage or posts to /api/v1/ccusage/batch.
  • Removed the ccusage config (CCUsage type, field and default). shelltime auth used to write ccusage.enabled: true into every new config.
  • Removed the deprecated ccotel field and its silent migration to aiCodeOtel, in both the base and local config.
  • Removed a duplicate LogCleanup merge block in mergeConfig.
  • Removed the now-unused cmdService from the daemon's main.go.
  • Updated docs/CONFIG.md (the legacy section, the proxy note, the example config and the FAQ) and CLAUDE.md.

Behavior note: old configs that still contain ccusage: or ccotel: load fine, because unknown keys are ignored. A config that sets only ccotel: (without aiCodeOtel:) will no longer enable the OTEL collector; rename the key to aiCodeOtel.

Related PRs

Test plan

  • go build ./...
  • go vet ./...: only the cmd/cli/main.go context-cancel warning, which was already there
  • go test ./...: model and commands pass. Two daemon socket tests fail locally with bind: invalid argument: the unix socket path built from t.TempDir() and the long test name exceeds macOS's length limit. That package isn't touched here.

🤖 Generated with Claude Code

… config

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@codecov

codecov Bot commented Sep 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
unittests 79.19% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
model/config.go 94.28% <ø> (-0.35%) ⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AnnatarHe
AnnatarHe merged commit 164ed51 into main Sep 26, 2026
3 of 4 checks passed
@AnnatarHe
AnnatarHe deleted the refactor/remove-ccusage-v1 branch September 26, 2026 09:21
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.

1 participant