Release 1.4.0

.claude/settings.json

@@ -0,0 +1,6 @@
+{
+  "attribution": {
+    "commit": "",
+    "pr": ""
+  }
+}

.github/workflows/build.yml

@@ -38,8 +38,29 @@ jobs:
           cache: 'gradle'
 
       - name: Build core and compose locally
+        env:
+          HEAD_REF: ${{ github.head_ref }}
         run: |
-          git clone --depth 1 https://github.com/MapConductor/android-sdk-core.git _core
+          # コア側に PR と同名のブランチがあるなら、そちらを組む。
+          #
+          # ここを default ブランチ固定にしていると、**コアの未マージな変更に追随する
+          # PR は永久に赤いまま**になる。実際 CameraBearing をコアへ足したとき、
+          # プロバイダ側の PR が 10 本まとめて Unresolved reference で落ちた。
+          # コアが先にマージされるまで待つ、という運用は取り込み順の事故を呼ぶので、
+          # 同名ブランチがあれば拾う形にしてある（無ければ従来どおり default）。
+          clone_matching() {
+            repo="$1"; dir="$2"
+            url="https://github.com/MapConductor/$repo.git"
+            if [ -n "$HEAD_REF" ] && git ls-remote --exit-code --heads "$url" "$HEAD_REF" >/dev/null 2>&1; then
+              echo "$repo: matching branch $HEAD_REF"
+              git clone --depth 1 --branch "$HEAD_REF" "$url" "$dir"
+            else
+              echo "$repo: default branch"
+              git clone --depth 1 "$url" "$dir"
+            fi
+          }
+
+          clone_matching android-sdk-core _core
           cd _core
           ./gradlew --no-daemon publishToMavenLocal
           cd ..
@@ -53,7 +74,7 @@ jobs:
           }
           EOF
 
-          git clone --depth 1 https://github.com/MapConductor/android-sdk-compose.git _compose
+          clone_matching android-sdk-compose _compose
           cd _compose
           ./gradlew --no-daemon publishToMavenLocal
           cd ..

build.gradle.kts

@@ -99,6 +99,14 @@ dependencies {
     } else {
         api("com.mapconductor:compose:$libraryVersion")
     }
+
+    // A real MapView loading real (local) styles: what a design change does to
+    // the overlays only shows on a map that actually swaps its style.
+    androidTestImplementation(libs.androidx.junit)
+    androidTestImplementation(libs.androidx.runner)
+    androidTestImplementation(platform(libs.androidx.compose.bom))
+    androidTestImplementation(libs.androidx.ui.test.junit4)
+    debugImplementation(libs.androidx.ui.test.manifest)
 }
 
 // Set project version for NMCP plugin

docs/MapLibreViewControllerImpl.kt.md

@@ -14,7 +14,7 @@ class MapLibreViewController(
     override val holder: MapLibreMapViewHolderInterface,
     private val markerController: MapLibreMarkerController,
     private val polylineController: MapLibrePolylineController,
-    private val polygonController: MapLibrePolygonConductor,
+    private val polygonController: MapLibrePolygonController,
     private val groundImageController: MapLibreGroundImageController,
     private val circleController: MapLibreCircleController,
     private val rasterLayerController: MapLibreRasterLayerController,
@@ -39,7 +39,7 @@ Initializes a new instance of the `MapLibreViewController`.
     - Type: `MapLibrePolylineController`
     - Description: The controller responsible for managing polylines.
 - `polygonController`
-    - Type: `MapLibrePolygonConductor`
+    - Type: `MapLibrePolygonController`
     - Description: The controller responsible for managing polygons.
 - `groundImageController`
     - Type: `MapLibreGroundImageController`

docs/polygon/MapLibrePolygonCompositeRenderer.kt.md

@@ -0,0 +1,37 @@
+# MapLibrePolygonCompositeRenderer
+
+## Signature
+
+```kotlin
+class MapLibrePolygonCompositeRenderer(
+    val fill: MapLibrePolygonOverlayRenderer,
+    val outline: MapLibrePolylineOverlayRenderer,
+) : PolygonOverlayRendererInterface<MapLibreActualPolygon>
+```
+
+## Description
+
+Presents the fill renderer and the outline renderer to the core `PolygonController` as a single
+renderer.
+
+A MapLibre fill layer can only draw a 1 px outline with no width, so to honour `strokeWidth` each
+polygon's outer ring is also drawn as a closed polyline with the id `outline-<polygon id>` through
+`outline`. Every call the controller makes on this renderer is forwarded to `fill` and mirrored on
+`outline`:
+
+- `onAdd` / `onChange`: the fill is created or rebuilt; when that succeeds, the outline polyline is
+  (re)created with the polygon's `strokeColor`, `strokeWidth`, `geodesic`, `zIndex` and `extra`.
+- `onRemove`: the outline polylines are dropped and both sources are redrawn without the removed
+  polygons (the controller's `clear()` does not call `onPostProcess`).
+- `onPostProcess`: both sources are redrawn.
+
+Holes are not outlined.
+
+## Constructor Parameters
+
+- `fill`
+    - Type: `MapLibrePolygonOverlayRenderer`
+    - Description: Draws the polygon fill (with holes) into the fill source.
+- `outline`
+    - Type: `MapLibrePolylineOverlayRenderer`
+    - Description: Draws the outlines into the polygon outline source.

docs/polygon/MapLibrePolygonController.kt.md

@@ -0,0 +1,30 @@
+# MapLibrePolygonController
+
+## Signature
+
+```kotlin
+class MapLibrePolygonController(
+    override val renderer: MapLibrePolygonCompositeRenderer,
+) : PolygonController<MapLibreActualPolygon>(renderer.fill.polygonManager, renderer)
+```
+
+## Description
+
+The polygon controller of the MapLibre provider. It extends the core `PolygonController`, so diffing
+(add / change / remove), `has`, `find`, the click cascade (`resolveTap`, `dispatchClick`),
+`clear` and `destroy` are the core implementation shared with every other provider.
+
+The only provider-specific part is the renderer: a fill layer can only draw a 1 px outline, so the
+outline is drawn as a separate polyline by [`MapLibrePolygonCompositeRenderer`](./MapLibrePolygonCompositeRenderer.kt.md).
+
+## Constructor Parameters
+
+- `renderer`
+    - Type: `MapLibrePolygonCompositeRenderer`
+    - Description: Draws both the fill and the outline. Its `fill.polygonManager` is used as the
+      controller's polygon manager, so the controller and the fill renderer always see the same
+      entities.
+
+## Properties
+
+- `zIndex`: `2` (unchanged from the former `MapLibrePolygonConductor`).