Skip to content

chore: Rename session-analytics-cli to agent-session-analytics-cli#244

Merged
evansenter merged 3 commits into
mainfrom
chore/rename-session-analytics-cli
Jan 24, 2026
Merged

chore: Rename session-analytics-cli to agent-session-analytics-cli#244
evansenter merged 3 commits into
mainfrom
chore/rename-session-analytics-cli

Conversation

@evansenter

Copy link
Copy Markdown
Owner

Summary

  • Update all hooks, scripts, agents, commands, and docs to use the new CLI name following the agent-session-analytics package rename

Changes:

  • MCP tools: mcp__session-analytics__*mcp__agent-session-analytics__*
  • CLI: session-analytics-cliagent-session-analytics-cli
  • Repo refs: claude-session-analyticsagent-session-analytics
  • Data paths updated to ~/.claude/contrib/agent-session-analytics/

Related: evansenter/agent-session-analytics#91, #243

Test plan

  • Restart Claude session, verify statusline works
  • Verify /session-dump still works
  • Verify /status-report finds agent-session-analytics MCP tools

🤖 Generated with Claude Code

@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR renames references from session-analytics-cli to agent-session-analytics-cli and updates repository references from claude-session-analytics/claude-event-bus to agent-session-analytics/agent-event-bus across documentation, agents, commands, and scripts.

Issues Found

Critical

None

Important

  • docs/case-study.md:84 - Typo introduces broken CLI name: agent-agent-session-analytics-cli (doubled "agent-") instead of agent-session-analytics-cli. This would cause the documented command to fail.

Suggestions

None

Verdict

REQUEST_CHANGES - Important issue: typo in docs/case-study.md creates an invalid CLI name that would break the documented command.


Automated review by Claude Code

1 similar comment
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR renames references from session-analytics-cli to agent-session-analytics-cli and updates repository references from claude-session-analytics/claude-event-bus to agent-session-analytics/agent-event-bus across documentation, agents, commands, and scripts.

Issues Found

Critical

None

Important

  • docs/case-study.md:84 - Typo introduces broken CLI name: agent-agent-session-analytics-cli (doubled "agent-") instead of agent-session-analytics-cli. This would cause the documented command to fail.

Suggestions

None

Verdict

REQUEST_CHANGES - Important issue: typo in docs/case-study.md creates an invalid CLI name that would break the documented command.


Automated review by Claude Code

evansenter and others added 3 commits January 24, 2026 23:23
Update all hooks, scripts, agents, commands, and docs to use the new
CLI name following the agent-session-analytics package rename.

- Update MCP tool names: mcp__session-analytics__ → mcp__agent-session-analytics__
- Update CLI references: session-analytics-cli → agent-session-analytics-cli
- Update repo names: claude-session-analytics → agent-session-analytics
- Update data paths: ~/.claude/contrib/analytics/ → agent-session-analytics/

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Update paths from analytics/ to agent-event-bus/ and
agent-session-analytics/ to match the rename.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix agent-agent-session-analytics-cli → agent-session-analytics-cli typo
- Update Bash permissions for session-analytics-cli → agent-session-analytics-cli

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@evansenter evansenter force-pushed the chore/rename-session-analytics-cli branch from 6ecf823 to 84b86e1 Compare January 24, 2026 23:24
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR renames CLI and repository references from session-analytics-cli/claude-session-analytics to agent-session-analytics-cli/agent-session-analytics across documentation, agents, commands, and shell scripts. The changes are consistent and thorough for the Bash permissions and documentation updates.

Issues Found

Critical

None

Important

  • home/.claude/settings.json:89 - Missing MCP permission rename: mcp__session-analytics__* should be updated to mcp__agent-session-analytics__* for consistency. The PR updates CLI Bash permissions and MCP tool calls in docs, but the MCP permission entry itself wasn't renamed. When the MCP server is installed with the new name, this permission won't match.

Suggestions

None

Previously Addressed (Filtered)

  • docs/case-study.md:84 - Typo with doubled "agent-" (Implemented in prior commit)

1 item from prior feedback rounds was not re-raised.

Verdict

REQUEST_CHANGES - Important issue: missing MCP permission rename in settings.json that should match the new server name.


Automated review by Claude Code

@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR renames CLI and repository references from session-analytics-cli to agent-session-analytics-cli and from claude-session-analytics/claude-event-bus to agent-session-analytics/agent-event-bus across documentation, agents, commands, scripts, and settings.

Issues Found

Critical

None

Important

None

Suggestions

None

Previously Addressed (Filtered)

  • docs/case-study.md:84 - Typo with doubled "agent-" in CLI name (Fixed in current commit)

1 item from prior feedback rounds was not re-raised.

Verdict

APPROVE - All renames are consistent and complete. The typo from the previous review has been fixed.


Automated review by Claude Code

@evansenter evansenter merged commit ae7b02f into main Jan 24, 2026
5 checks passed
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