Skip to content

made some changes #4

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.

Sign up for GitHub

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

Open
CosmoWorker wants to merge 1 commit into
base: main
Choose a base branch
from
Open

made some changes #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
# AI-Code-Context-Reviewer
An automated code reviewer that reviews & summarizes Pull Requests.

## Features
* Automated PR review system
* Custom ruleset defined
## Tools / Libraries
* FastAPI
* Groq
Comment on lines 1 to 9
Copy link
Owner Author

@CosmoWorker CosmoWorker Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a blank line before the new ## Features heading for consistency and ensure the bullet list follows markdown style.

Expand Down
13 changes: 1 addition & 12 deletions server/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
from fastapi import FastAPI, Header
from fastapi.middleware.cors import CORSMiddleware
import os
import logging
import requests
Comment on lines 1 to 4
Copy link
Owner Author

@CosmoWorker CosmoWorker Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed CORSMiddleware import; if the API will be called from browsers, re‑add the import to avoid CORS issues.

Expand All @@ -10,7 +9,6 @@

load_dotenv()
app = FastAPI()
app.add_middleware(CORSMiddleware)

logger = logging.getLogger(__name__)
logging.basicConfig(level=logging.INFO, format="%(asctime)s %(levelname)s %(message)s")
Comment on lines 9 to 14
Copy link
Owner Author

@CosmoWorker CosmoWorker Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed app.add_middleware(CORSMiddleware); consider restoring the middleware when cross‑origin requests are needed.

Expand All @@ -24,16 +22,7 @@ def main():
return {"msg": "Hello from code reviewer server"}


"""
Webhook Abstract Response object for pull requests
{
"action": "",
"number": "",
"pull_request": {},
"repository": {},
"sender": {}
}
"""

headers = {
"Authorization": f"Bearer {github_token}",
"X-GitHub-Api-Version": "2022-11-28",
Comment on lines 22 to 28
Copy link
Owner Author

@CosmoWorker CosmoWorker Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deleted the webhook abstract comment; replace it with a proper module‑level docstring to retain documentation.

Expand Down