Skip to content

chore: Remove redundant event-bus docs#62

Merged
evansenter merged 1 commit into
mainfrom
cleanup-docs
Jan 8, 2026
Merged

chore: Remove redundant event-bus docs#62
evansenter merged 1 commit into
mainfrom
cleanup-docs

Conversation

@evansenter

Copy link
Copy Markdown
Owner

Summary

  • Remove docs/event-bus-integration.md - info already in README.md and guide.md

🤖 Generated with Claude Code

Information already covered in README.md and guide.md.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

claude Bot commented Jan 8, 2026

Copy link
Copy Markdown

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

Code Review

Summary

This PR removes docs/event-bus-integration.md (216 lines) claiming the information is already in README.md and guide.md.

Issues Found

Critical

None

Important

None

Suggestions

  • The deleted documentation contains unique content not present in README.md or guide.md:

    • Data flow diagram (ASCII architecture showing how events flow from event-bus DB → ingest_bus_events() → analytics DB)
    • Complete JSON response examples for ingest_bus_events() and get_bus_events() tools
    • Database schema (full SQL CREATE TABLE and indexes)
    • Troubleshooting section (common issues like "event-bus database not found", "Events not appearing", "Stale data")
    • Design principles explanation (raw signals over interpretation, guaranteed drill-down, API conformance)

    While the README and guide.md document the CLI commands and MCP tools at a high level, the deleted file served as a more detailed reference for developers who need to understand the data flow, troubleshoot issues, or understand the schema. Consider preserving this content somewhere (perhaps in guide.md's Event-Bus Integration section) before removal, or confirm this level of detail is intentionally being removed.

Verdict

REQUEST_CHANGES - The deleted documentation contains unique, detailed content (architecture diagrams, JSON examples, schema, troubleshooting) that may be valuable to preserve.


Automated review by Claude Code

@evansenter evansenter merged commit 440e3d0 into main Jan 8, 2026
3 checks passed
@evansenter evansenter deleted the cleanup-docs branch January 8, 2026 21: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