Skip to content

Commit 748db27

Browse files
committed
Improves error handling and setup
Adds more detailed error logging to the repository snapshot function for easier debugging, including response status, headers, and body. Updates project dependencies by specifying them in requirements.txt. Adds a section to the README outlining current limitations regarding GitHub API authentication using PATs and future plans for GitHub App integration, and enhancing repository analysis and usage analytics.
1 parent 4b6097a commit 748db27

File tree

3 files changed

+34
-5
lines changed

3 files changed

+34
-5
lines changed

README.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,27 @@ cp .env.example .env
9393
uvicorn app.main:app --reload
9494
```
9595

96+
## 📋 Known Limitations & Roadmap
97+
98+
### Current Limitations
99+
100+
- **GitHub API Authentication**: Currently uses Personal Access Tokens (PAT) for GitHub API access, which has limitations including:
101+
- Rate limiting constraints
102+
- Token management complexity
103+
- Security considerations for token storage and rotation
104+
- Limited scope control compared to GitHub Apps
105+
106+
### Coming Soon
107+
108+
- **🔧 GitHub App Integration**: Replace PAT-based authentication with a proper GitHub App to provide:
109+
- Better rate limiting
110+
- Fine-grained permissions
111+
- Improved security model
112+
- No need for users to manage personal tokens
113+
- **📊 Enhanced Repository Analysis**: More comprehensive codebase analysis for better AI agent recommendations
114+
- **🔗 Additional AI Agent Integrations**: Support for more AI coding assistants and development tools
115+
- **📈 Usage Analytics**: Insights into AI agent effectiveness and usage patterns
116+
96117
## 🤝 Contributing
97118

98119
We welcome contributions to AGUnblock! Please see our [CONTRIBUTING.md](CONTRIBUTING.md) for details on how to get started.

backend/app/services/github.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,4 +142,11 @@ async def get_repository_snapshot(self, owner: str, repo: str) -> Optional[Dict[
142142
except Exception as e:
143143
error_message = str(e)
144144
print(f"Error in get_repository_snapshot for {owner}/{repo}: {error_message}")
145+
146+
# Add more detailed error information for debugging
147+
if hasattr(e, 'response'):
148+
print(f"Response status: {e.response.status_code}")
149+
print(f"Response headers: {dict(e.response.headers)}")
150+
print(f"Response body: {e.response.text}")
151+
145152
raise RuntimeError(f"Failed to fetch repository data: {error_message}")

backend/requirements.txt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
fastapi==0.104.1
2-
uvicorn==0.23.2
3-
python-dotenv==1.0.0
4-
httpx==0.24.1
5-
azure-ai-agents==1.0.0
1+
fastapi
2+
uvicorn
3+
python-dotenv
4+
httpx
5+
azure-ai-agents
6+
githubkit

0 commit comments

Comments
 (0)