Skip to content

chore: Rename from claude-session-analytics to agent-session-analytics#91

Merged
evansenter merged 6 commits into
mainfrom
chore/rename-to-agent
Jan 24, 2026
Merged

chore: Rename from claude-session-analytics to agent-session-analytics#91
evansenter merged 6 commits into
mainfrom
chore/rename-to-agent

Conversation

@evansenter

Copy link
Copy Markdown
Owner

Summary

Rename project to agent-session-analytics to align with agent-event-bus rename.

Changes

  • GitHub repo renamed via gh repo rename
  • Package name in pyproject.toml
  • version() calls in __init__.py and server.py
  • Service files: com.evansenter.agent-session-analytics.plist, agent-session-analytics.service
  • All script references (install/uninstall for LaunchAgent and systemd)
  • Makefile service names
  • Test comments

Test plan

  • All 384 tests pass
  • make install works with new service names

🤖 Generated with Claude Code

- Rename GitHub repo via gh repo rename
- Update package name in pyproject.toml
- Update version() calls in __init__.py and server.py
- Rename service files (LaunchAgent plist, systemd service)
- Update all script references
- Update Makefile service names
- Update test comments

This aligns with the agent-event-bus rename.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus rename. The changes update the package name, service files, and related scripts.

Issues Found

Critical

None

Important

  • uv.lock:275 - The lock file still contains name = "claude-session-analytics". The lock file needs to be regenerated after the pyproject.toml change to reflect the new package name agent-session-analytics. Run uv lock to update.

Suggestions

  • README.md:1 - Title still says # Claude Session Analytics. Consider updating to # Agent Session Analytics for consistency with the rename.
  • src/session_analytics/cli.py:1515 - CLI description still says "Claude Session Analytics CLI". Consider updating to "Agent Session Analytics CLI".
  • src/session_analytics/server.py:733 - Server startup message still says "Starting Claude Session Analytics". Consider updating to "Starting Agent Session Analytics".
  • scripts/agent-session-analytics.service:2 - Service description still says Description=Claude Session Analytics. Consider updating to Description=Agent Session Analytics.

Verdict

REQUEST_CHANGES - The uv.lock file is out of sync with the renamed package in pyproject.toml and needs to be regenerated. Additionally, there are several remaining "Claude Session Analytics" references in user-facing strings (README title, CLI description, server startup message, service file description) that should be updated for a complete rename.


Automated review by Claude Code

Address review feedback:
- Regenerate uv.lock with new package name
- Update README title to "Agent Session Analytics"
- Update CLI description
- Update server startup message
- Update systemd service description
- Update event-bus link in README

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@evansenter

Copy link
Copy Markdown
Owner Author

Feedback Addressed

Implemented

  • [Important] uv.lock - Regenerated with uv lock to reflect new package name
  • [Suggestion] README.md - Updated title to "Agent Session Analytics" and fixed event-bus link
  • [Suggestion] cli.py - Updated CLI description to "Agent Session Analytics CLI"
  • [Suggestion] server.py - Updated startup message to "Starting Agent Session Analytics"
  • [Suggestion] service - Updated systemd service description to "Agent Session Analytics"

All quality gates pass (384 tests).

@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus rename. The changes comprehensively update package names, service files, scripts, and user-facing strings.

Issues Found

Critical

None

Important

None

Suggestions

  • README.md:17 - Reference to claude-event-bus still exists: [event-bus](https://github.com/evansenter/claude-event-bus). Should be updated to agent-event-bus for consistency.
  • README.md:135 - Reference to claude-event-bus still exists. Should be updated to agent-event-bus.
  • README.md:159 - Reference to claude-event-bus still exists in the Related Projects section. Should be updated to agent-event-bus.

Previously Addressed (Filtered)

  • uv.lock - Regenerated with new package name (Implemented in prior round)
  • README.md:1 - Updated title to "Agent Session Analytics" (Implemented in prior round)
  • cli.py - Updated CLI description (Implemented in prior round)
  • server.py - Updated startup message (Implemented in prior round)
  • service - Updated systemd description (Implemented in prior round)

5 items from prior feedback rounds were not re-raised.

Verdict

REQUEST_CHANGES - Found 3 remaining claude-event-bus references in README.md that should be updated to agent-event-bus for a complete rename.


Automated review by Claude Code

- Rename Python package: session_analytics → agent_session_analytics
- Rename CLI: session-analytics-cli → agent-session-analytics-cli
- Rename MCP server: session-analytics → agent-session-analytics
- Update resource URI: agent-session-analytics://guide
- Update data path: ~/.claude/contrib/agent-session-analytics/
- Rename env vars: AGENT_SESSION_ANALYTICS_*
- Update event-bus path: ~/.claude/contrib/agent-event-bus/

All 384 tests passing.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus rename. The changes comprehensively update package names, service files, scripts, and user-facing strings.

Issues Found

Critical

None

Important

  • src/agent_session_analytics/server.py:735 - The "Add to Claude Code" message still uses session-analytics instead of agent-session-analytics: "claude mcp add ... session-analytics http://{host}:{port}/mcp". This will cause users to add the MCP server with the wrong name.
  • Makefile:113-115 - The uninstall target still references session-analytics in the mcp remove command: mcp remove --scope user session-analytics. Should be agent-session-analytics to properly uninstall.

Suggestions

  • tests/test_smoke_real_data.py:30 - Still references old database path ~/.claude/contrib/analytics/data.db. Should be ~/.claude/contrib/agent-session-analytics/data.db for consistency with the rename.
  • scripts/global-report.sh:11,13,18,20 - Still uses old CLI name session-analytics-cli and outputs to /tmp/session-analytics-report.md. Should be updated to agent-session-analytics-cli and /tmp/agent-session-analytics-report.md.
  • src/agent_session_analytics/ingest.py:14 and src/agent_session_analytics/patterns.py:14 - Logger names still use session-analytics instead of agent-session-analytics. Other files (bus_ingest.py, storage.py, server.py) were updated but these two were missed.

Previously Addressed (Filtered)

  • uv.lock - Regenerated with new package name (Implemented in prior round)
  • README.md:1 - Updated title to "Agent Session Analytics" (Implemented in prior round)
  • cli.py - Updated CLI description (Implemented in prior round)
  • server.py - Updated startup message (Implemented in prior round)
  • service - Updated systemd description (Implemented in prior round)
  • README.md event-bus references - Fixed to agent-event-bus (Implemented in prior round)

6 items from prior feedback rounds were not re-raised.

Verdict

REQUEST_CHANGES - Found 2 important issues (incorrect MCP server name in user-facing message and uninstall command) and 3 suggestions for consistency (test db path, report script, logger names).


Automated review by Claude Code

- Fix MCP server message to use agent-session-analytics
- Fix Makefile uninstall command to use agent-session-analytics
- Update logger names in ingest.py and patterns.py
- Update test database path in smoke tests
- Update global-report.sh to use new CLI name and output path

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@evansenter

Copy link
Copy Markdown
Owner Author

Feedback Addressed

Implemented

  • [Important] server.py - Fixed MCP add message to use agent-session-analytics
  • [Important] Makefile - Fixed uninstall command to use agent-session-analytics
  • [Suggestion] test_smoke_real_data.py - Updated database path to agent-session-analytics
  • [Suggestion] global-report.sh - Updated CLI name and output path
  • [Suggestion] ingest.py, patterns.py - Updated logger names to agent-session-analytics

All 384 tests passing.

@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus rename. The changes comprehensively update package names, module paths, service files, scripts, documentation, and all user-facing strings.

Issues Found

Critical

None

Important

None

Suggestions

None

Previously Addressed (Filtered)

  • uv.lock - Regenerated with new package name (Implemented in prior round)
  • README.md:1 - Updated title to "Agent Session Analytics" (Implemented in prior round)
  • README.md - Fixed event-bus references to agent-event-bus (Implemented in prior round)
  • cli.py - Updated CLI description (Implemented in prior round)
  • server.py - Updated startup message (Implemented in prior round)
  • server.py - Fixed MCP add message (Implemented in prior round)
  • Makefile - Fixed uninstall command (Implemented in prior round)
  • test_smoke_real_data.py - Updated database path (Implemented in prior round)
  • global-report.sh - Updated CLI name and output path (Implemented in prior round)
  • ingest.py, patterns.py - Updated logger names (Implemented in prior round)
  • agent-session-analytics.service - Updated systemd description (Implemented in prior round)

11 items from prior feedback rounds were not re-raised.

Verdict

APPROVE - The rename is comprehensive and well-executed. All package references, module paths, service files, scripts, documentation, environment variables, and user-facing strings have been updated consistently. The uv.lock properly reflects the new package name. All prior feedback has been addressed.


Automated review by Claude Code

Migrates database from ~/.claude/contrib/analytics/data.db
to ~/.claude/contrib/agent-session-analytics/data.db

Matches the migration pattern used in agent-event-bus.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR comprehensively renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus naming convention. The changes update package names, module paths, service files, scripts, documentation, environment variables, and all user-facing strings.

Issues Found

Critical

None

Important

None

Suggestions

None

Previously Addressed (Filtered)

  • uv.lock - Regenerated with new package name (Implemented in prior round)
  • README.md:1 - Updated title to "Agent Session Analytics" (Implemented in prior round)
  • README.md - Fixed event-bus references to agent-event-bus (Implemented in prior round)
  • cli.py - Updated CLI description (Implemented in prior round)
  • server.py - Updated startup message (Implemented in prior round)
  • server.py - Fixed MCP add message to use agent-session-analytics (Implemented in prior round)
  • Makefile - Fixed uninstall command to use agent-session-analytics (Implemented in prior round)
  • test_smoke_real_data.py - Updated database path (Implemented in prior round)
  • global-report.sh - Updated CLI name and output path (Implemented in prior round)
  • ingest.py, patterns.py - Updated logger names (Implemented in prior round)
  • agent-session-analytics.service - Updated systemd description (Implemented in prior round)

11 items from prior feedback rounds were not re-raised.

Notable Implementation Details

The PR includes a thoughtful automatic database migration feature in storage.py that will seamlessly migrate existing databases from the old location (~/.claude/contrib/analytics/data.db) to the new location (~/.claude/contrib/agent-session-analytics/data.db), ensuring users don't lose their historical data during the rename.

Verdict

APPROVE - The rename is comprehensive and well-executed. All package references, module paths, service files, scripts, documentation, environment variables, and user-facing strings have been updated consistently. The uv.lock properly reflects the new package name. All prior feedback has been addressed.


Automated review by Claude Code

Aligns with agent-event-bus naming convention (AGENT_EVENT_BUS_TESTING).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude

claude Bot commented Jan 24, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR comprehensively renames the project from claude-session-analytics to agent-session-analytics to align with the agent-event-bus naming convention. The changes update package names, module paths, service files, scripts, documentation, environment variables, and all user-facing strings.

Issues Found

Critical

None

Important

None

Suggestions

None

Previously Addressed (Filtered)

  • uv.lock - Regenerated with new package name (Implemented in prior round)
  • README.md:1 - Updated title to "Agent Session Analytics" (Implemented in prior round)
  • README.md - Fixed event-bus references to agent-event-bus (Implemented in prior round)
  • cli.py - Updated CLI description (Implemented in prior round)
  • server.py - Updated startup message (Implemented in prior round)
  • server.py - Fixed MCP add message to use agent-session-analytics (Implemented in prior round)
  • Makefile - Fixed uninstall command to use agent-session-analytics (Implemented in prior round)
  • test_smoke_real_data.py - Updated database path (Implemented in prior round)
  • global-report.sh - Updated CLI name and output path (Implemented in prior round)
  • ingest.py, patterns.py - Updated logger names (Implemented in prior round)
  • agent-session-analytics.service - Updated systemd description (Implemented in prior round)

11 items from prior feedback rounds were not re-raised.

Notable Implementation Details

The PR includes a thoughtful automatic database migration feature in storage.py:608-627 that will seamlessly migrate existing databases from the old location (~/.claude/contrib/analytics/data.db) to the new location (~/.claude/contrib/agent-session-analytics/data.db), ensuring users don't lose their historical data during the rename.

Verdict

APPROVE - The rename is comprehensive and well-executed. All package references, module paths, service files, scripts, documentation, environment variables, and user-facing strings have been updated consistently. The uv.lock properly reflects the new package name. All prior feedback has been addressed.


Automated review by Claude Code

@evansenter evansenter merged commit b1cf3be into main Jan 24, 2026
3 checks passed
@evansenter evansenter deleted the chore/rename-to-agent branch January 24, 2026 23:46
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