docs: add inline comments to script.js

[nirvani-nj]

Summary

This PR improves the readability of static/script.js by adding beginner-friendly inline comments and function-level explanations across the skill chip manager, form validation, API handling, result rendering, and code viewer logic. The goal is to help new contributors better understand the DOM manipulation and client-side flow without changing any application behavior.

Related Issue

Closes #54

Type of Change

• Documentation — updates docs, README, or code comments only

What Was Changed

File	Change made
static/script.js	Added inline comments and beginner-friendly explanations for functions, DOM interactions, validation flow, API requests, and project card rendering

How to Test This PR

1. Clone this branch: git checkout docs/scriptjs-comments
2. Install dependencies: pip install -r requirements.txt
3. Run the app: python app.py
4. Open http://127.0.0.1:5000/
5. Test skill chip addition/removal and recommendation generation
6. Run the tests: python tests/test_basic.py

Expected test output:

27 passed, 0 failed out of 27 tests

Test Results

27 passed, 0 failed out of 27 tests

Self-Review Checklist

• I have read CONTRIBUTING.md and followed all guidelines
• My branch name follows the required convention
• I have run python tests/test_basic.py and all 27 tests pass
• I have not modified files outside the scope of the linked issue
• No application logic or UI behavior was changed

Notes for Reviewer

This PR only adds comments/documentation inside static/script.js. No application logic or UI behavior was modified.

[vercel]

@nirvani-nj is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

Thank you for submitting your first pull request to DevPath.

Before review:

• Complete the PR template fully
• Ensure all tests pass
• Link your PR to an issue
• Keep changes scoped to the issue

A maintainer will review your contribution soon.

[komalharshita]

Thanks for the contribution. The added inline comments and beginner-focused explanations improve readability and make the frontend logic easier for new contributors to understand.

However, this PR currently has merge conflicts because static/script.js has changed significantly in recent merged PRs. Please sync your branch with the latest main branch and resolve the conflicts before further review.

Also note that another PR is currently working on a very similar documentation improvement for the same file. The first cleanly resolved and review-ready implementation will be considered for merge, while overlapping PRs may be marked as duplicate afterward.

[nirvani-nj]

Resolved merge conflicts by syncing with the latest main branch and preserving the updated functionality.

[komalharshita]

@nirvani-nj check the pull request again

[nirvani-nj]

Resolved merge conflicts by syncing with the latest main branch and preserving the updated functionality.

[komalharshita]

looks good to merge