Update from task 1d2b3e8a-4591-47e6-979f-0489e729cdfc

[iamajaykr06]

This PR was created by qwen-chat coder for task 1d2b3e8a-4591-47e6-979f-0489e729cdfc.

[github-actions]

AIPCSS Bot Reviewer Report

Status: Issues Found - Action Required

---

AI Logic Review

null

---

Backend Report

Issues Found

Flake8

backend/app/routes/scheduling.py:578:58: E203 whitespace before ':'
backend/app/scheduler_new/genetic_engine.py:117:52: E203 whitespace before ':'
backend/app/scheduler_new/greedy_engine.py:60:56: E203 whitespace before ':'
backend/app/scheduler_new/hybrid_engine.py:339:60: E203 whitespace before ':'
backend/app/scheduler_new/ortools_engine.py:55:56: E203 whitespace before ':'
backend/app/scheduler_new/scheduler_engine.py:197:46: E203 whitespace before ':'

Bandit Security

[main]	INFO	profile include tests: None
[main]	INFO	profile exclude tests: None
[main]	INFO	cli include tests: None
[main]	INFO	cli exclude tests: None
[main]	INFO	running on Python 3.11.15
[manager]	WARNING	Test in comment: Malformed is not a test name or id, ignoring
[manager]	WARNING	Test in comment: break is not a test name or id, ignoring
[manager]	WARNING	Test in comment: data is not a test name or id, ignoring
[manager]	WARNING	Test in comment: should is not a test name or id, ignoring
[manager]	WARNING	Test in comment: be is not a test name or id, ignoring
[manager]	WARNING	Test in comment: skipped is not a test name or id, ignoring
[tester]	WARNING	nosec encountered (B112), but no failed test on file ./backend/app/scheduler_new/data_loader.py:340
Run started:2026-05-06 19:29:11.292388+00:00

Test results:
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/genetic_engine.py:175:51
174	                DomainValue(
175	                    faculty_id=faculty_id, room_id=random.choice(opt["rooms"]), timeslot=opt["timeslot"]
176	                )  # nosec B311

--------------------------------------------------
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/genetic_engine.py:334:36
333	                            faculty_id=assigned_teacher,
334	                            room_id=random.choice(opt["rooms"]),
335	                            timeslot=opt["timeslot"],  # nosec B311

--------------------------------------------------
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/genetic_engine.py:368:36
367	                            faculty_id=assigned_teacher,
368	                            room_id=random.choice(opt["rooms"]),
369	                            timeslot=opt["timeslot"],  # nosec B311

--------------------------------------------------
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/genetic_engine.py:382:52
381	                    DomainValue(
382	                        faculty_id=teacher, room_id=random.choice(opt["rooms"]), timeslot=opt["timeslot"]
383	                    )  # nosec B311

--------------------------------------------------
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/ortools_engine.py:164:32
163	                            f_selected.extend(
164	                                random.sample(f_by_slot[ts_idx], min(len(f_by_slot[ts_idx]), f_per_slot))
165	                            )  # nosec B311

--------------------------------------------------
>> Issue: [B311:blacklist] Standard pseudo-random generators are not suitable for security/cryptographic purposes.
   Severity: Low   Confidence: High
   CWE: CWE-330 (https://cwe.mitre.org/data/definitions/330.html)
   More Info: https://bandit.readthedocs.io/en/1.9.4/blacklists/blacklist_calls.html#b311-random
   Location: ./backend/app/scheduler_new/ortools_engine.py:169:32
168	                            f_selected.extend(
169	                                random.sample(rem, min(len(rem), min_per_faculty - len(f_selected)))
170	                            )  # nosec B311

--------------------------------------------------

Code scanned:
	Total lines of code: 3318
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 20

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 6
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 6
Files skipped (0):

---

Frontend Report

Passed: Frontend quality checks passed.

[github-actions]

Bot Reviewer: Please fix the issues mentioned in the report.