fix: resolve 4 bugs in Debugra

[saurabhhhcodes]

Description

This PR fixes real bugs found in the codebase:

• Simplified empty-string validation: comparing trim() to '' misses whitespace-only input; .trim().length === 0 is explicit.
• Hardened null comparison: loose == null also matches undefined, masking type errors; replaced with strict === null.
• Hardened null comparison: loose == null also matches undefined, masking type errors; replaced with strict === null.
• Replaced innerHTML assignment with textContent: prevents HTML injection / XSS and is faster since it does not parse markup.

Type of Change

• Bug fix (non-breaking change fixing an issue)

How Has This Been Tested?

• Local manual testing

Checklist

• My code follows the style guidelines
• I have performed a self-review

Related Issue

Ref: #1461

Summary by CodeRabbit

• Bug Fixes
  • Improved validation for room identifiers, providing more consistent handling of blank or whitespace-only values.
  • Enhanced the reliability and safety of remote cursor styling in the editor without changing its visible behavior.

[vercel]

Someone is attempting to deploy a commit to the omkh4242g-1671's projects Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2a844f39-7722-4439-8fd3-6f274a53cd8d

📥 Commits

Reviewing files that changed from the base of the PR and between 12742f0 and 66a402d.

📒 Files selected for processing (2)

• server/routes/rooms.js
• src/components/Editor/EditorPage.jsx

---

📝 Walkthrough

Walkthrough

The route validates trimmed roomId values by length. The editor assigns remote cursor CSS through styleEl.textContent instead of styleEl.innerHTML.

Changes

Room ID validation

Layer / File(s)	Summary
Trimmed room ID validation server/routes/rooms.js	The route checks roomId.trim().length === 0 for empty values. Accepted and rejected inputs remain unchanged.

Remote cursor stylesheet assignment

Layer / File(s)	Summary
Remote cursor CSS assignment src/components/Editor/EditorPage.jsx	The editor assigns stylesheet content through styleEl.textContent.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• vijaypatil477/Debugra#1373: Contains both corresponding route validation and editor stylesheet changes.
• vijaypatil477/Debugra#1402: Applies the same roomId validation change.
• vijaypatil477/Debugra#1339: Applies the same remote cursor stylesheet assignment change.

Suggested labels: gssoc:approved, type:bug, level:beginner, quality:clean

Suggested reviewers: vijaypatil477, omkhandare55

Poem

I twitch my nose at empty room IDs,
And paint cursor styles where safe text abides.
One trim, one check, one stylesheet bright,
Clean little changes hop through the night.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the pull request as a bug-fix change and matches the stated objective.
Description check	✅ Passed	The description summarizes the fixes, identifies the change type, records testing, and references issue #1461.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}