[dannyleech]

Resolves #72

The race in #72 no longer crashes. MapLibre v6 (#562) made getSource safe to call before the style exists. The single draw plugin (#422) also polls for its sources instead of reading them on map:ready.

We're keeping map:ready where it is, straight after the map is created, rather than moving it into the load callback. Padding sync, re-measuring the layout once plugins render, plugin UI and markers all wait on isMapReady, and none of them need the style. Moving it would hold the interface back until the style had loaded, and if the style failed to load, that UI would never appear.

This PR documents the timing instead:

• map:ready / MAP_READY: fires once the map instance exists. The base map style may still be loading.
• New map:loaded entry: fires once the initial style has loaded. Already emitted by all providers but not documented until now.
• mapState.isMapReady: plugins should check mapProvider.isBaseMapReady() before adding sources or layers.

[Copilot]

🟡 Changes recommended

Provider-specific timing, viewport readiness, and payload behavior need accurate documentation.

3 open issues

• Document that OpenLayers MAP_READY precedes bounds fitting · New
• Document OpenLayers MAP_LOADED tile-load dependency · New
• Document ESRI MAP_LOADED event payload · New

What changed in this PR

Clarifies map readiness timing for #72 without changing runtime behavior.

Changes:

• Distinguishes map creation from base-style readiness.
• Documents the existing map:loaded event.
• Adds style-readiness guidance for plugins.

File	Description
src/​config/​events.js	Updates readiness event comments.
docs/​api/​context.md	Explains isMapReady and the provider readiness check.
docs/​api.md	Expands readiness documentation and adds map:loaded.

_{Review effort: Balanced}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[dannyleech]

I've addressed all three in the latest commit. map:ready now only says the map uses the safe zone as its initial padding, since OpenLayers fits bounds after map:ready fires. "Base map style" is now "base map", which fits OpenLayers and ESRI. The map:loaded payload now notes that ESRI passes the current map state. Making ESRI's payload match the other providers would change runtime behaviour, so that should be a separate ticket.