Skip to content

chore(web): cleanup dead code, migrate map to MapLibre, and expand unit test coverage - #190

Open
joaquimscosta wants to merge 11 commits into
mainfrom
chore/034-web-deadcode-cleanup
Open

chore(web): cleanup dead code, migrate map to MapLibre, and expand unit test coverage#190
joaquimscosta wants to merge 11 commits into
mainfrom
chore/034-web-deadcode-cleanup

Conversation

@joaquimscosta

Copy link
Copy Markdown
Contributor

Summary

This PR includes 8 commits:

  • chore(web): remove unused components, dead code, and obsolete exports
  • test(gallery): add unit tests for PromotedVideoPlayer and CompactVideoCard (specs 031 & 032)
  • test(admin): add unit tests for YouTube sync admin hooks (specs 029 & 030)
  • feat(map): migrate from Mapbox GL JS to MapLibre GL JS (spec 023)
  • fix(map): default to CARTO Voyager basemap style instead of dark matter
  • feat(gallery): polish map marker clustering and video badges (spec 026)
  • docs(plan): update plan submodule pointer for specs 023 and 026
  • docs(roadmap): update project status, roadmap, and lockfile

Test Plan

  • Code builds successfully (pnpm build)
  • Tests pass (pnpm test:unit)
  • Manual testing completed
  • Documentation updated

Changes

  • apps/web/next.config.ts
  • apps/web/package.json
  • apps/web/src/app/(main)/design-system/_data/components.ts
  • apps/web/src/components/admin/mdx-engine/index.ts
  • apps/web/src/components/admin/mdx-engine/mdx-commit-button.tsx
  • apps/web/src/components/admin/mdx-engine/mdx-preview-modal.tsx
  • apps/web/src/components/catalyst-ui/fieldset.tsx
  • apps/web/src/components/catalyst-ui/popover.tsx
  • apps/web/src/components/contact/contact-form.tsx
  • apps/web/src/components/content/search.tsx
  • apps/web/src/components/gallery/gallery-map-canvas.tsx
  • apps/web/src/components/gallery/gallery-map-marker.tsx
  • apps/web/src/components/landing/hero-section-atmospheric.tsx
  • apps/web/src/components/landing/map-teaser-section.tsx
  • apps/web/src/components/newsletter/homepage-newsletter-section.tsx
  • apps/web/src/components/ui/page-transition.tsx
  • apps/web/src/components/ui/video-hero-section.tsx
  • apps/web/src/features/map/components/brava-map.tsx
  • apps/web/src/features/map/components/map-canvas.tsx
  • apps/web/src/features/map/shared/base-map.tsx
  • apps/web/src/hooks/use-smooth-scroll.ts
  • apps/web/src/hooks/useFilterUrlSync.ts
  • apps/web/src/lib/content/mdx-components.tsx
  • apps/web/src/lib/gallery-mappers.ts
  • apps/web/src/lib/mocks/directory.ts
  • apps/web/src/lib/mocks/index.ts
  • apps/web/src/lib/mocks/user.ts
  • apps/web/src/schemas/index.ts
  • apps/web/src/schemas/mediaMetadataSchema.ts
  • apps/web/src/schemas/storySchema.ts
  • apps/web/tests/unit/components/gallery/compact-video-card.test.tsx
  • apps/web/tests/unit/components/gallery/featured-video-hero.test.tsx
  • apps/web/tests/unit/components/gallery/gallery-map-canvas.test.tsx
  • apps/web/tests/unit/hooks/admin/useYouTubeSync.test.tsx
  • apps/web/velite.config.ts
  • arkhe/roadmap/next-actions.md
  • docs/PROJECT-ROADMAP.md
  • docs/PROJECT-STATUS.md
  • plan
  • pnpm-lock.yaml

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

🚀 PR Validation Results

📁 Components Changed:

  • Backend (Spring Boot/Kotlin)
  • Frontend (Next.js/React)

🔍 Validation Results:

Backend CI: success
Frontend CI: success
⏭️ Infrastructure CI: Skipped (no relevant changes)
⏭️ Global Security Scan: Skipped (no relevant changes)

🎉 Status: READY FOR REVIEW

All validation checks have passed! This PR is ready for code review.


Updated: 2026-07-31T14:34:53.661Z | PR: #190

- Replace Mapbox DEM protocol source with open AWS Terrarium elevation tiles
- Add transformRequest in BaseMap to map CARTO glyph requests to OpenMapTiles fonts CDN
- Add fonts.openmaptiles.org and s3.amazonaws.com to CSP connect-src in next.config.ts
- Remove legacy Mapbox access token checks and references from env.ts and privacy page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant