Skip to content

Comments

Add comprehensive architectural documentation and developer guidelines#118

Open
andypost wants to merge 1 commit intolaunchrctl:mainfrom
andypost:ai-docs
Open

Add comprehensive architectural documentation and developer guidelines#118
andypost wants to merge 1 commit intolaunchrctl:mainfrom
andypost:ai-docs

Conversation

@andypost
Copy link

  • Add CLAUDE.md with project overview and key commands for AI assistance
  • Create docs/architecture/ with detailed architectural analysis:
    • ARCHITECTURAL_PATTERNS.md: 16 design patterns, anti-patterns, and 15 improvement suggestions
    • LOGGING_ARCHITECTURE.md: Analysis of sophisticated dual logging system (Log() vs Term())
    • PLUGIN_SYSTEM.md: Complete plugin architecture guide with examples
    • SERVICE_SYSTEM.md: Service-oriented design and dependency injection documentation
  • Add comprehensive DEVELOPER_GUIDELINES.md covering:
    • Code style and conventions
    • Architecture guidelines and best practices
    • Logging guidelines for dual system usage
    • Plugin and service development guides
    • Testing, error handling, and performance considerations
  • Update docs/README.md with improved organization and navigation

Total: 1,915 lines of comprehensive documentation covering system architecture, development practices, and contribution guidelines.

🤖 Generated with Claude Code

@lexbritvin
Copy link
Collaborator

Hello @andypost
Can you please rebase onto the latest main?
I added a lot of tests that covers how we use the cli. Do you think regenerating claude.md can give better results?

@andypost
Copy link
Author

@lexbritvin rebased, the second MR #119 still needs work

@andypost
Copy link
Author

andypost commented Feb 8, 2026

@lexbritvin rebased and modernized

- CLAUDE.md with project overview, build commands, key types and interfaces
- docs/architecture/ with patterns analysis, logging, plugin and service systems
- docs/DEVELOPER_GUIDELINES.md with code style, logging, plugin/service guides
- docs/actions.md and actions.schema.md with env vars, mask, shorthand, user field
- docs/README.md updated with navigation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants