feat: add Share My Result button with pre-filled URL (#411)

[Lost-glitched]

Summary

Added a "Share My Result" button to the results page that builds a
pre-filled URL from the user's current form selections and copies it
to the clipboard. When someone opens a shared URL, the form auto-fills
and results render automatically. Purely frontend — no backend changes.
All new JS is ES5-compatible per CONTRIBUTING.md.

Related Issue

Closes #411

Type of Change

• Feature — adds new functionality
• Style — CSS or visual changes only, no logic change
• Test — adds or updates tests

What Was Changed

File	Change made
static/script.js	Added share URL builder, clipboard copy, toast handler, query param auto-fill
templates/index.html	Added share button and toast notification in results section
static/style.css	Added styles for share button, copied state, and toast
tests/test_basic.py	Added test for homepage with share query params
CHANGELOG.md	Documented new feature under Unreleased

How to Test This PR

1. git checkout feat/share-result-url
2. pip install -r requirements.txt
3. python app.py
4. Open http://127.0.0.1:5000
5. Fill the form and click "Generate My Projects"
6. Verify "Share My Result" button appears above results
7. Click it — verify button turns green with "Copied!" and toast appears
8. Paste the URL in a new tab — verify form auto-fills and results show
9. python tests/test_basic.py

Expected test output:

Test Results

31 passed, 0 failed out of 31 tests

Screenshots

Self-Review Checklist

• I have read CONTRIBUTING.md and followed all guidelines
• My branch name follows the convention: feat/share-result-url
• I have run python tests/test_basic.py and all tests pass
• I have not introduced any console.log() debug statements
• I have not modified files outside the scope of the linked issue
• If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)

Notes for Reviewer

The existing script.js already uses ES6+ (const, let, arrow functions)
despite CONTRIBUTING.md requiring ES5. All new code in this PR strictly
follows ES5 — var declarations, function expressions only. The existing
violations were not touched as they are out of scope.

[vercel]

@Lost-glitched 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 feature contribution — the shareable URL functionality is a nice UX improvement and the clipboard fallback handling is thoughtfully implemented.

However, I’m requesting a few changes before this can be merged:

1. There is duplicated conditional logic inside renderResults():

if (!projects || projects.length === 0)

appears multiple times consecutively in the modified section. This introduces unnecessary repetition and makes the flow harder to maintain. Please consolidate the empty-state handling into a single block.

2. Query parameters are inserted directly into the UI without validation/sanitization. Since values come from the URL, please validate or sanitize:

• skills
• level
• interest
• time

before using them in form autofill or rendering logic.

3. The shared URL can become excessively long when many skills are selected because all skills are appended directly into query params:

params.set("skills", skillsHidden.value.trim());

Please add a reasonable limit or encoding strategy to avoid oversized URLs.

4. Auto-triggering form submission on page load:

form.dispatchEvent(new Event("submit", { cancelable: true }));

can create unexpected API calls immediately after opening a shared link. Consider requiring explicit user interaction or adding a loading guard/debounce.

5. The added test only checks for status code 200 and does not validate the actual share/autofill behavior. Please add stronger coverage for:

• query param parsing,
• skill autofill,
• and auto-submit behavior.

Once these are addressed, the feature will be much more robust and maintainable. 🚀

[komalharshita]

@resolve the merge conflicts

[Lost-glitched]

All 5 review points addressed. 37/37 tests pass. Ready for re-review

[komalharshita]

Looks safe to merge...but merge conflicts still exist @Lost-glitched