Skip to content

fix: remove dead activity tracker summarizers#33

Merged
tmustier merged 1 commit into
mainfrom
fix/activity-tracker-lint-warnings
Mar 24, 2026
Merged

fix: remove dead activity tracker summarizers#33
tmustier merged 1 commit into
mainfrom
fix/activity-tracker-lint-warnings

Conversation

@tmustier
Copy link
Copy Markdown
Owner

@tmustier tmustier commented Mar 24, 2026

Summary

  • remove the unused tool arg/result summarizer helpers from activity-tracker.ts
  • keep the currently used extractStart*/extractEnd* path unchanged
  • clear the last remaining lint warnings in this package

Testing

  • npm run check
  • npm run smoke-test

@tmustier
Copy link
Copy Markdown
Owner Author

Findings

No issues found. This is a clean dead-code removal — the deleted functions are unreferenced, have no test coverage to break, and their replacements are already wired in and working.

Verdict

correct — pure dead-code deletion with no functional or behavioral change.

@tmustier tmustier merged commit 6c29cf2 into main Mar 24, 2026
3 checks passed
@tmustier tmustier deleted the fix/activity-tracker-lint-warnings branch March 24, 2026 19:27
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