refactor(agents): simplify metrics by removing token tracking utilities#183
Merged
refactor(agents): simplify metrics by removing token tracking utilities#183
Conversation
- Remove tokenUtils.ts and token-aggregator.ts files - Remove token fields from MetricDelta and session types - Simplify metrics processors for Claude, Gemini, and OpenCode plugins - Clean up analytics aggregator, data-loader, and types - Add header injection plugin support in SSO proxy Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
- Fix repository field in metrics-aggregator to use parent/repo format instead of full absolute path (added extractRepository helper) - Remove stale tokens field from gemini.session-adapter (removed from ParsedSession.metrics interface); clean up unused variables and debug log - Remove save_history property from assistants chat call (not in type) - Delete token-aggregator.ts and its test (tokens removed from system) - Remove all token field assertions from conversation and metrics tests - Remove tokens field from all MetricDelta test fixtures - Update session-syncer test metric name to codemie_cli_tool_usage_total - Remove total_*_tokens assertions from integration tests Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
MetricDelta.tokens was removed from the schema; remove all d.tokens.* accesses and analytics.totalTokens.* assertions from the analytics integration test. Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
codemie-ai
approved these changes
Mar 4, 2026
TarasSpashchenko
approved these changes
Mar 4, 2026
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
3 participants
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.
Summary
Refactors the metrics system across all agent plugins by removing token tracking utilities and simplifying the metrics pipeline.
Changes
tokenUtils.tsandtoken-aggregator.tsfiles entirelytokensfield fromMetricDeltatype and related session typesImpact
Significantly reduces code complexity (~1250 lines removed, ~200 added). Token tracking is no longer part of the delta/metrics pipeline.
Checklist