Skip to content

Deep#14

Open
d-sutariya wants to merge 6 commits intomainfrom
Deep
Open

Deep#14
d-sutariya wants to merge 6 commits intomainfrom
Deep

Conversation

@d-sutariya
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Aug 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
neurograde Error Error Dec 25, 2025 2:46am

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive containerization and AI-powered chat capabilities to the NeuroGrade application. The main purpose is to enable Docker-based deployment and integrate an intelligent chatbot system for student support.

Key changes include:

  • Complete Docker containerization setup with multi-service architecture
  • AI-powered chatbot integration with Google Meet scheduling capabilities
  • Enhanced teacher registration with subject field requirement
  • Team section updates and React version downgrade for compatibility

Reviewed Changes

Copilot reviewed 31 out of 45 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
server/src/models/teacher.js Added required subject field to teacher schema
server/src/controllers/user.controller.js Updated teacher registration to include subject field
server/Dockerfile Added production-ready Docker configuration for Express server
nginx/nginx.conf Created reverse proxy configuration for multi-service routing
docker-compose.yml Defined complete multi-service Docker environment
client/src/components/Chatbot.jsx Implemented AI chatbot with Google Meet integration
client/src/app/register/page.js Added subject selection for teacher registration
FastAPI/main.py Created FastAPI service with chat agent and MongoDB integration
FastAPI/src/chat.py Implemented LangGraph-based doubt solving agent
Files not reviewed (1)
  • client/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)

server/src/models/teacher.js:8

  • Missing space after colon. Should be 'subject: str' to maintain consistent formatting with other model fields.
        type: String,

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +111 to +118
system_prompt = (
"You are a helpful doubt solver for students. "
"Answer student questions clearly and concisely. "
"If a student asks to schedule, arrange, or set up a meeting with a teacher, use the provided tool. "
"Strictly don't reveal your identity as GEMINI"
"If the question is related to the Study You must answer it regardless whether the subject exists or not (remeber this )."
"For all other questions, answer as a knowledgeable tutor. "
"When scheduling meetings, use the student roll number provided in the conversation."
Copy link

Copilot AI Aug 24, 2025

Choose a reason for hiding this comment

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

Hard-coded system prompt instruction should be moved to a configuration file or environment variable for better maintainability and easier updates.

Suggested change
system_prompt = (
"You are a helpful doubt solver for students. "
"Answer student questions clearly and concisely. "
"If a student asks to schedule, arrange, or set up a meeting with a teacher, use the provided tool. "
"Strictly don't reveal your identity as GEMINI"
"If the question is related to the Study You must answer it regardless whether the subject exists or not (remeber this )."
"For all other questions, answer as a knowledgeable tutor. "
"When scheduling meetings, use the student roll number provided in the conversation."
system_prompt = os.environ.get(
"DOUBT_AGENT_SYSTEM_PROMPT",
(
"You are a helpful doubt solver for students. "
"Answer student questions clearly and concisely. "
"If a student asks to schedule, arrange, or set up a meeting with a teacher, use the provided tool. "
"Strictly don't reveal your identity as GEMINI"
"If the question is related to the Study You must answer it regardless whether the subject exists or not (remeber this )."
"For all other questions, answer as a knowledgeable tutor. "
"When scheduling meetings, use the student roll number provided in the conversation."
)

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@vercel
Copy link

vercel bot commented Dec 25, 2025

Deployment failed with the following error:

The `vercel.json` schema validation failed with the following message: should NOT have additional property `rootDirectory`

Learn More: https://vercel.com/docs/concepts/projects/project-configuration

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.

1 participant