Skip to content

Refactor/system reliability #652

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
Arul-Aravind wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Refactor/system reliability #652

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
12985e5
chore(core): refactor backend for reliability, fix cross-app integrat…
Mar 31, 2026
1cc520e
fix: additional minor adjustments to backend and UI integration
Mar 31, 2026
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
1,158 changes: 669 additions & 489 deletions backend/server.py
View file
Open in desktop

Large diffs are not rendered by default.

477 changes: 276 additions & 201 deletions backend/test_server.py
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion eduaid_desktop/config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ const config = {
// Window configuration
window: {
titleBarStyle: process.platform === 'darwin' ? 'hiddenInset' : 'default',
icon: path.join(__dirname, 'assets/icons/win/aossie.ico'),
icon: path.join(__dirname, 'assets', 'aossie.ico'),
webPreferences: {
nodeIntegration: false,
contextIsolation: true,
Expand Down
6 changes: 3 additions & 3 deletions eduaid_desktop/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@
"dev": "concurrently \"npm run start:web\" \"wait-on http://localhost:3000 && electron .\"",
"start:web": "cd ../eduaid_web && npm start",
"build:web": "cd ../eduaid_web && npm run build",
"copy:web": "xcopy /E /I /Y \"..\\eduaid_web\\build\" \"build\"",
"build": "cd ../eduaid_web && npm run build",
"copy:web": "node -e \"const fs=require('fs');fs.rmSync('build',{recursive:true,force:true});fs.cpSync('../eduaid_web/build','build',{recursive:true});\"",

@coderabbitai coderabbitai Bot Mar 31, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check for Node.js version requirements in package.json files and any .nvmrc
echo "=== Checking package.json engines field ==="
fd -t f 'package.json' --exec grep -l "engines" {} \; 2>/dev/null | xargs -I{} sh -c 'echo "File: {}"; cat {} | jq ".engines" 2>/dev/null'

echo ""
echo "=== Checking for .nvmrc or .node-version ==="
fd -t f -g '.nvmrc' -g '.node-version'
fd -t f -g '.nvmrc' -g '.node-version' --exec cat {}

echo ""
echo "=== README Node.js requirements ==="
rg -i "node.*version|node.*[0-9]+\.[0-9]+" README.md 2>/dev/null || echo "No README.md or no Node version mentioned"

Repository: AOSSIE-Org/EduAid

Length of output: 439

🏁 Script executed:

cat README.md | head -100

Repository: AOSSIE-Org/EduAid

Length of output: 3896

🏁 Script executed:

cat eduaid_desktop/package.json | head -30

Repository: AOSSIE-Org/EduAid

Length of output: 1113

🏁 Script executed:

rg "node.*16|16\.[0-9]" -i --type md --type json

Repository: AOSSIE-Org/EduAid

Length of output: 50376

Ensure Node.js minimum version requirement is 16.7.0 or higher to support fs.cpSync.

The copy:web script uses fs.cpSync, which was added in Node.js 16.7.0. Update the minimum Node.js version in package.json (add an engines field) and in README.md files from "version 16 or higher" to "16.7.0 or higher" to prevent build failures on versions 16.0.0–16.6.x.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@eduaid_desktop/package.json` at line 11, Add a Node.js engines requirement of
at least 16.7.0 to package.json and update README references to "16.7.0 or
higher": specifically, add an "engines" field in package.json (next to existing
metadata) specifying "node": ">=16.7.0" so the build script that uses fs.cpSync
in the "copy:web" npm script is guaranteed to run, and update any README.md text
that currently says "version 16 or higher" to "16.7.0 or higher" to keep
documentation consistent with the runtime requirement.

"build": "npm run build:web && npm run copy:web",
"build:electron": "npm run build && electron-builder",
"build:all": "npm run build && electron-builder --mac --win --linux",
"pack": "electron-builder --dir",
"dist": "npm run build:web && npm run copy:web && electron-builder"
"dist": "npm run build && electron-builder"
},
"keywords": [
"eduaid",
Expand Down
118 changes: 75 additions & 43 deletions eduaid_web/package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion eduaid_web/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
"@testing-library/jest-dom": "^5.17.0",
"@testing-library/react": "^13.4.0",
"@testing-library/user-event": "^13.5.0",
"dotenv": "^16.4.7",
"pdf-lib": "^1.17.1",
"react": "^18.3.1",
"react-dom": "^18.3.1",
Expand Down Expand Up @@ -42,6 +41,9 @@
]
},
"devDependencies": {
"@babel/plugin-proposal-private-property-in-object": "^7.21.11",
"browserslist": "^4.28.2",
"caniuse-lite": "^1.0.30001782",
"tailwindcss": "^3.4.9"
}
}
8 changes: 4 additions & 4 deletions eduaid_web/src/App.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import "./App.css";
import { Routes, Route, HashRouter } from "react-router-dom";
import Home from "./pages/Home";
import Question_Type from "./pages/Question_Type";
import Text_Input from "./pages/Text_Input";
import QuestionType from "./pages/Question_Type";
import TextInput from "./pages/Text_Input";
import Output from "./pages/Output";
import Previous from "./pages/Previous";
import NotFound from "./pages/PageNotFound";
Expand All @@ -12,8 +12,8 @@ function App() {
<HashRouter>
<Routes>
<Route path="/" element={<Home />} />
<Route path="/question-type" element={<Question_Type />} />
<Route path="/input" element={<Text_Input />} />
<Route path="/question-type" element={<QuestionType />} />
<Route path="/input" element={<TextInput />} />
<Route path="/output" element={<Output />} />
<Route path="/history" element={<Previous />} />
<Route path="*" element={<NotFound />} />
Expand Down
Loading