Skip to content

Emergency: deploy one-source flat Mercator map - #52

Merged
Occumed79 merged 10 commits into
mainfrom
hotfix/flat-stable-overview
Jul 30, 2026
Merged

Emergency: deploy one-source flat Mercator map#52
Occumed79 merged 10 commits into
mainfrom
hotfix/flat-stable-overview

Conversation

@Occumed79

@Occumed79 Occumed79 commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Replaces the corrupted globe runtime with a temporary flat Mercator containment map using one immutable worldwide overview PMTiles archive. Removes globe projection, terrain, fog, sky/model/hillshade layers, atmosphere bloom, Neon routing, runtime tile merging, and synthetic geometry. Opens at a full-world flat view and keeps normal client-side overview overscaling for deeper zooms. This is independent of the permanent 51-family owner build.

Summary by CodeRabbit

  • New Features

    • Added a flat overview map mode using a consistent Mercator projection and fixed zoom range.
    • Added a dedicated server that serves map tiles and application assets from a local overview archive.
    • The application now automatically obtains and validates the required map archive before starting.
  • Bug Fixes

    • Disabled globe-only visual effects, terrain, and fog in flat overview mode.
    • Improved map validation across zoom levels and projections for more reliable rendering.

@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 1deea666-40f2-4603-a694-4e619fcc689a

📥 Commits

Reviewing files that changed from the base of the PR and between 89ccb67 and 42494d6.

📒 Files selected for processing (9)
  • package.json
  • scripts/apply-flat-overview-mode.mjs
  • scripts/capture-polygon-regression.mjs
  • scripts/check-flat-overview-mode.mjs
  • scripts/start-flat-overview.mjs
  • scripts/validate-all-zoom-levels.mjs
  • server-flat.mjs
  • src/flat-overview.css
  • src/main.js

📝 Walkthrough

Walkthrough

Adds a flat overview mode using a Mercator style, immutable PMTiles archive, dedicated server and launcher, explicit client map settings, style checks, projection-aware regression diagnostics, and deterministic zoom checkpoint validation.

Changes

Flat overview runtime

Layer / File(s) Summary
Flat overview style and build contract
package.json, scripts/apply-flat-overview-mode.mjs, scripts/check-flat-overview-mode.mjs
Style preparation and checks enforce a single occumed-open vector source, Mercator projection, max zoom 5, disabled globe features, flat metadata, and the flat startup command.
PMTiles server and archive startup
scripts/start-flat-overview.mjs, server-flat.mjs
The launcher downloads and validates the overview archive before spawning the server, which serves PMTiles tiles, rewritten style JSON, static assets, health endpoints, caching, and graceful shutdown.
Flat map client wiring
src/main.js, src/flat-overview.css
The client imports flat-overview CSS, sets explicit map view limits, updates readiness labels, and hides the atmosphere bloom element.
Projection-aware regression checks
scripts/capture-polygon-regression.mjs
Regression diagnostics record projection, terrain, and fog state and validate globe atmosphere behavior according to the detected projection.
Deterministic zoom checkpoint validation
scripts/validate-all-zoom-levels.mjs
Zoom validation moves through explicit checkpoints, waits for required rendered layers, checks tile-source permanence, and reports checkpoint failures.

Estimated code review effort: 4 (Complex) | ~60 minutes

Possibly related PRs

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Fix failing CI checks
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/flat-stable-overview

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@Occumed79
Occumed79 marked this pull request as ready for review July 30, 2026 16:32
@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.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@Occumed79
Occumed79 merged commit ee211dd into main Jul 30, 2026
22 of 24 checks passed
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