-
Notifications
You must be signed in to change notification settings - Fork 89
documentation updates #2142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
documentation updates #2142
Conversation
…on-2823 Add comprehensive documentation with knowledge graphs and mermaid diagrams
### Token Usage Optimization | ||
|
||
Sidecar optimizes token usage to reduce costs and improve performance. | ||
|
||
**Key Features:** | ||
- **Context Pruning**: Removal of irrelevant context | ||
- **Chunking**: Breaking large content into manageable chunks | ||
- **Compression**: Compression of context information | ||
- **Prioritization**: Prioritization of important content |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am pretty sure we don't do this anymore ...
### API Key Management | ||
|
||
Sidecar securely manages API keys for various services. | ||
|
||
**Key Features:** | ||
- **Secure Storage**: Secure storage of API keys | ||
- **Access Control**: Controlled access to API keys | ||
- **Key Rotation**: Support for key rotation | ||
- **Minimal Exposure**: Minimization of key exposure | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Err pretty sure we don't do this as well ... Key rotation is not something sidecar should be worried about
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@theskcd oh yess im sorry let me delete features and roadmap lol, they are irrelevant at all
appreciate the effort but I think the documentation is overly descriptive and has clear AI generated slop smell. I like the overall direction this is going, so feel free to iterate on this and let me know when I should review this |
it is ai, just docs overall were so good so i decided worth to show you, cleaning up some ai dreams and should be good |
No operation ID found for this PR |
…0a85ae No operation ID found for this PR' (Run ID: aldrin-labs_sidecar_issue_4_240a85ae)
…5f98ec create TUI to run this agent in terminal/ssh (Run ID: aldrin-labs_sidecar_issue_6_e75f98ec)
This pull request includes significant documentation updates to the
ARCHITECTURE.md
andCONCEPTS.md
files to provide a detailed explanation of the Sidecar architecture and key concepts used in the project. The most important changes include a comprehensive overview of the Sidecar architecture, detailed descriptions of core components, and explanations of key concepts and terminology.Architecture Documentation:
ARCHITECTURE.md
: Added a detailed explanation of the Sidecar architecture, including component interactions, data flow, and design decisions. This includes descriptions of core components such as the Application Core, Webserver API, LLM Client, Repository Analysis, Agentic Tools, MCTS Decision Engine, and Code Chunking.Key Concepts and Terminology:
CONCEPTS.md
: Added explanations of key concepts and terminology used in the Sidecar project, including AI concepts (LLMs, Agentic System, MCTS, Tool-Based Approach), code understanding (Symbols, Repository Mapping, Code Chunking, Tree-sitter), system architecture (Application Core, Webserver, Repository Pool, LLM Broker), development concepts (Rust Workspace, Asynchronous Programming, Error Handling, Tracing and Logging), and integration concepts (Aide Editor Integration, LLM Provider Integration, Git Integration).