Skip to content

docs: Add naming conventions section to CLAUDE.md - #96

Merged
evansenter merged 1 commit into
mainfrom
docs/add-naming-conventions
Jan 29, 2026
Merged

docs: Add naming conventions section to CLAUDE.md#96
evansenter merged 1 commit into
mainfrom
docs/add-naming-conventions

Conversation

@evansenter

Copy link
Copy Markdown
Owner

Summary

  • Add naming conventions table documenting repo, package, CLI, paths, and env vars
  • Fix link to agent-session-analytics repo (was claude-session-analytics)
  • Ensures consistency with agent-session-analytics patterns

Test plan

  • Documentation only, no code changes

🤖 Generated with Claude Code

Document consistent naming patterns for repo, package, CLI, paths,
and environment variables - aligned with agent-session-analytics.

Also fix link to agent-session-analytics repo.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@evansenter
evansenter merged commit 1ca8564 into main Jan 29, 2026
5 checks passed
@evansenter
evansenter deleted the docs/add-naming-conventions branch January 29, 2026 08:02
@claude

claude Bot commented Jan 29, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR adds a comprehensive naming conventions table to CLAUDE.md documenting the repository's naming patterns (repo, package, CLI commands, paths, env vars) and fixes an incorrect link to the related repository (claude-session-analytics → agent-session-analytics).

Issues Found

Critical

None

Important

None

Suggestions

None

Verdict

APPROVE - Documentation-only change with accurate content. The naming conventions table correctly reflects the project structure:

  • Package name agent_event_bus matches src/agent_event_bus/
  • CLI commands match pyproject.toml scripts
  • Resource URI matches server.py:73
  • LaunchAgent/systemd paths match actual files in scripts/
  • Environment variables (AGENT_EVENT_BUS_*) match actual usage in codebase

The link fix from claude-session-analytics to agent-session-analytics is correct.


Automated review by Claude Code

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