[Bug] Commute.jsx fails to compile (duplicate saveLocation/deleteSavedLocation) — whole app build is broken

[vedant7007]

Description

src/components/Commute.jsx fails to compile, and since Commute is a static top-level import in App.jsx (L25, rendered L1310), the entire app build aborts — not just the Commute tab.

Confirmed with esbuild:

ERROR: The symbol "saveLocation" has already been declared
ERROR: The symbol "deleteSavedLocation" has already been declared

A botched merge (last commit f390850 Merge branch 'main' into feature/decompose-commute-component-621) moved the save/delete logic into the useRouteHistory hook but left the old inline copies behind, producing three layered defects in one file:

1. Duplicate const (build-breaking): saveLocation / deleteSavedLocation are destructured from useRouteHistory() (L45-46) and re-declared as local functions (L86, L101) in the same scope.
2. Undefined references in those leftovers: they call setSavedLocations(...) (L90, L102) and SAVED_LOCATIONS_KEY (L95, L104), neither of which exists in this file (both live in the hook) — a ReferenceError even if UI Enhancement: Improve Dashboard Layout, Alignment & Responsiveness #1 were fixed.
3. Missing imports: getAQIBand is used at module-eval time in LEGEND_ITEMS (L22) but never imported → ReferenceError on load; and eventBus.emit("ROUTE_PLANNED", ...) (L68) is used but eventBus is never imported → the route handler throws (caught locally, surfacing the generic "Error calculating route" alert on every successful search and never firing the ROUTE_PLANNED achievement).

Steps to Reproduce

npm run build (or start the dev server) → build aborts with the two "already been declared" errors above.

Fix

• Delete the two leftover local definitions (L86-107); the destructured hook versions already do this correctly and are the ones passed to CommuteForm.
• Add the missing imports:

import { getAQIBand } from "../services/airQualityService";
import { eventBus } from "../core/events";

(RouteResults.jsx, RouteMap.jsx, routePlanner.js, useRouteHistory.js are all correct — just unreachable until this compiles.)

Happy to fix if assigned.

Contributing as part of Elite Coders Summer of Code (ECSoC 2026).

[github-actions]

👋 Thank you @vedant7007 for opening this issue!

We appreciate you taking the time to report or suggest an improvement for Pollution Control Hub.

Our maintainers will review your issue as soon as possible.

Meanwhile, please ensure:

• 📌 The issue clearly describes the problem or feature request.
• 📝 Steps to reproduce (if it's a bug) are included.
• 📷 Screenshots or logs are attached when applicable.
• 🔍 The issue is not a duplicate of an existing one.

Thank you for contributing to the project! 🚀

[github-actions]

Issue is assigned to you, @vedant7007! 🎯

⏳ Note: Please submit a working PR within 3 days, else the issue will be unassigned and open for other contributors. 🚀

[vedant7007]

This is resolved on main now — the duplicate saveLocation/deleteSavedLocation declarations were removed and the missing getAQIBand/eventBus imports added (commits 620317c resolve duplicate Commute declarations, dddb513 import eventBus in Commute #668, 862dba9 RouteResults props #667). Commute.jsx compiles again. Can be closed. Contributing as part of ECSoC 2026.