Skip to content

fix: repair three syntax errors breaking the production build - #1130

Open
vedant7007 wants to merge 1 commit into
Aditya8369:mainfrom
vedant7007:build/fix-app-import
Open

fix: repair three syntax errors breaking the production build#1130
vedant7007 wants to merge 1 commit into
Aditya8369:mainfrom
vedant7007:build/fix-app-import

Conversation

@vedant7007

Copy link
Copy Markdown
Contributor

What & why

Closes #1129.

npm run build fails on main with three separate syntax errors (bad merges), each masking the next, so the whole app fails to bundle.

Changes

  1. src/App.jsx — restored the dropped import { opener before the airQualityService named imports (Unexpected "}").
  2. src/components/Leaderboard.jsx — removed a duplicate const nextLevel declaration ("nextLevel" has already been declared); kept the documented one.
  3. src/components/NoisePollutionTracker.jsx:466 — changed color={EXPR} to color: EXPR inside a style object (Expected "}" but found ".").

All mechanical — no behavior change.

Verification

  • npm run build✓ built in ~50s (was failing).
  • npm run check:shadowing → clean.
  • Lint: my three files introduce no new lint errors (the repo's pre-existing lint problems are unrelated and untouched here).

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

vite build fails on main with three cascading syntax errors:
- App.jsx: dropped 'import {' opener for the airQualityService imports
- Leaderboard.jsx: duplicate 'const nextLevel' declaration
- NoisePollutionTracker.jsx: style object used color={...} (JSX attr
  syntax) instead of color: ... (object property)

All mechanical; no behavior change. npm run build now succeeds.

Closes Aditya8369#1129
@vedant7007
vedant7007 requested a review from Aditya8369 as a code owner August 29, 2026 15:40
@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

@vedant7007 is attempting to deploy a commit to the Aditya Mahajan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank You for Your Contribution! 🎉

Hi @vedant7007,

Thank you for opening this Pull Request and contributing to our project. We truly appreciate your efforts.

Please make sure that:

  • Your code follows the project's guidelines.
  • You have linked the appropriate issue (if applicable).
  • Screenshots are added for UI/UX changes.
  • Your PR is ready for review.

The maintainer @Aditya8369 will review your PR shortly!

Happy Contributing! 🚀

@github-actions github-actions Bot added the ECSoC26 Contributions considered under ECSoC'26 label Aug 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ECSoC26 Contributions considered under ECSoC'26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

npm run build fails on main: 3 syntax errors (App.jsx, Leaderboard.jsx, NoisePollutionTracker.jsx)

1 participant