docs: comprehensive cortex-cli code analysis report (#169)

This report provides a detailed analysis of the cortex-cli crate covering:

- Architecture overview with module breakdown and dependencies
- Critical issues including incomplete features and missing confirmations
- Medium priority issues: unwrap usage, missing help text, validation gaps
- Low priority issues: code style and minor improvements
- Code quality concerns: large files, code duplication, naming inconsistencies
- Security analysis: good practices found and areas for improvement
- Performance considerations and potential bottlenecks
- Testing analysis: coverage gaps and missing test categories
- Documentation gaps in modules and functions
- Prioritized recommendations for improvement

Key findings:
- 19 source files totaling ~10,000 lines of code
- Good security practices for input validation
- Several large files that should be split (mcp_cmd.rs, agent_cmd.rs)
- Incomplete features (--attach, --share) should be hidden or implemented
- Missing confirmation prompt for logout command
- Several .expect() calls that could panic in production
- Good test coverage in some modules, missing in others

Co-authored-by: Droid Agent <droid@factory.ai>

Author: echobt