Merge pull request #82 from figonzal1/release-please--branches--main

chore(main): release 1.8.0

Author: figonzal1

.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.7.10"
+  ".": "1.8.0"
 }

CHANGELOG.md

@@ -0,0 +1,24 @@
+# Changelog
+
+## [1.8.0](https://github.com/figonzal1/LastQuakeChile/compare/v1.7.10...v1.8.0) (2026-06-03)
+
+
+### Features
+
+* add Android developer agent foundation skills ([912d48f](https://github.com/figonzal1/LastQuakeChile/commit/912d48f012e3b30387f25e8aac9156939cce7ecb))
+* major refactor — DomainResult, coroutines, map clustering, and CI improvements ([ca74dbb](https://github.com/figonzal1/LastQuakeChile/commit/ca74dbb6965cf58409d1889b5e549f0a5b6c5a25))
+* **map:** replace loadPins with ClusterManager and cap markers at 50 ([e52fc95](https://github.com/figonzal1/LastQuakeChile/commit/e52fc959a44ea5c24cfa42265698926fc3c1c76c)), closes [#75](https://github.com/figonzal1/LastQuakeChile/issues/75)
+
+
+### Bug Fixes
+
+* **coroutines:** replace Channel with MutableSharedFlow for one-shot error events ([89a6643](https://github.com/figonzal1/LastQuakeChile/commit/89a6643fa6e5db8b6de298601cf6098609d97001))
+* decouple FCM subscription from user preference switch state ([6564d9a](https://github.com/figonzal1/LastQuakeChile/commit/6564d9ac386f9fde1a93b5acbbad3985d739418a))
+* exclude SharedPreferences from Auto Backup to prevent stale permission restore ([c6eecfc](https://github.com/figonzal1/LastQuakeChile/commit/c6eecfc9150856c10700645e7150d782be3be766))
+* filter DeadSystemRuntimeException from Crashlytics reports ([2283dca](https://github.com/figonzal1/LastQuakeChile/commit/2283dca71d8159e2043b6cde7283473e6da34517)), closes [#76](https://github.com/figonzal1/LastQuakeChile/issues/76)
+* gate FCM subscription on permission and fix POST_NOTIFICATIONS cardview flow ([c4a66f0](https://github.com/figonzal1/LastQuakeChile/commit/c4a66f055202774f4cec9715da750519ab7eb118))
+* **notifications:** use quake code as unique PendingIntent request ID ([11142a8](https://github.com/figonzal1/LastQuakeChile/commit/11142a8f56bc464f063add867d712df80aefe655))
+* prevent Maps background snapshot crash in MapsFragment ([9cf6a54](https://github.com/figonzal1/LastQuakeChile/commit/9cf6a54ace64649f2783ea19ee09fcf5ac5e7668)), closes [#74](https://github.com/figonzal1/LastQuakeChile/issues/74)
+* re-evaluate notification permission cardview in onResume ([5e5bf47](https://github.com/figonzal1/LastQuakeChile/commit/5e5bf4768f89210a5a3e8cf14210c8e907919f48))
+* replace random channel scheme with two fixed notification channels ([75940d9](https://github.com/figonzal1/LastQuakeChile/commit/75940d9a1e4741140fb45ecd547e8aba1af4d000))
+* restrict destructive DB migration and HTTP logging to debug builds ([857304b](https://github.com/figonzal1/LastQuakeChile/commit/857304b3c4067c5e0057a1fbe13187db7743936d))

CLAUDE.md

@@ -0,0 +1,97 @@
+# CLAUDE.md
+
+Guidance for Claude Code when working in this repository.
+
+## Project overview
+
+Android app (Kotlin) showing recent earthquakes in Chile. Single Gradle module `:app`.
+
+- **DI:** Koin (not Hilt)
+- **Networking:** Retrofit + Moshi + Sandwich (`ApiResponse`)
+- **Persistence:** Room (KSP codegen)
+- **Images:** Coil · **Maps:** Google Maps + android-maps-utils (marker clustering)
+- **Backend services:** Firebase (Crashlytics, Performance, FCM)
+- **UI:** XML views, with an incremental XML → Compose migration in progress (see PR #80)
+
+### Architecture
+
+Feature-based packages under `app/src/main/java/cl/figonzal/lastquakechile/`:
+
+- `core/` — shared infra: DI modules, networking, DB, utils, `DomainResult`/`DomainError`
+- `quake_feature/` — earthquake list, detail, and map (data/domain/ui)
+- `reports_feature/` — reports feature (data/domain/ui)
+
+Data flow: remote/local data sources return domain models via mappers; repositories
+expose `DomainResult<T>` (success/error), surfaced to ViewModels as a single `uiState`.
+
+## Commands
+
+Build uses product flavors `dev` / `beta` / `prod` (dimension `version`) and build types `debug` / `release`.
+
+```bash
+./gradlew assembleDevDebug          # build dev/debug APK
+./gradlew test                      # all unit tests
+./gradlew :app:connectedDevDebugAndroidTest   # instrumentation tests (device/emulator)
+./gradlew bundleProdRelease         # production AAB
+
+# fastlane wrappers (bundle exec):
+bundle exec fastlane unit_test      # unit tests
+bundle exec fastlane ui_test        # instrumentation tests (DevDebug)
+bundle exec fastlane prod_googleplay # build prod AAB + upload to Google Play
+```
+
+## Gotchas
+
+- Requires `secrets.properties` (Maps API key via Secrets Gradle plugin) and signing material
+  under `keys/` — both untracked; builds fail without them.
+- Destructive Room migration and HTTP logging are gated to debug builds only.
+- `versionCode` is derived from the version managed by release-please — do not hardcode it.
+
+## Release process (release-please + fastlane + Google Play)
+
+This project releases through [release-please](https://github.com/googleapis/release-please)
+(`release-type: simple`) and ships to Google Play via fastlane.
+
+### Branch rules
+
+- `main` has an **active ruleset** (`pull_request` required, `non_fast_forward`).
+  Direct pushes to `main` are **blocked for everyone** — all changes land via PR.
+- `required_approving_review_count: 0`, so the owner can **self-merge** PRs without approval.
+- Only `main` is protected; feature and release-please branches are unrestricted.
+
+### How release-please behaves
+
+- The workflow `.github/workflows/release-please.yml` runs on **every push to `main`**
+  (and supports `workflow_dispatch` for manual runs).
+- It does **not** cut a release on each run. It maintains a **single rolling release PR**
+  on the branch `release-please--branches--main`.
+- The git tag, GitHub Release, and `versionCode` bump happen **only when that release PR is merged**.
+- `chore`, `refactor`, `ci`, `docs`, etc. do **not** bump the version — only `feat`/`fix` do.
+  Adding such commits just refreshes the open release PR; it never spawns a second release.
+
+### Fastlane changelogs — when and where
+
+Use the `/fastlane-changelog` command (see `.claude/commands/fastlane-changelog.md`).
+
+- **Timing:** generate the changelogs **before merging the release PR**, while the latest git
+  tag is still the **previous** version. The command diffs `git describe --tags --abbrev=0..HEAD`;
+  if run after the release PR merges, the new tag already exists and the diff is empty.
+- Changelogs are always written to `default.txt` (locales `es-419` and `en-US`), which fastlane
+  uses as a fallback when no versionCode-specific file exists.
+- **Convention (owner's preference):** commit the `default.txt` changelogs as the **last commit
+  directly onto the `release-please--branches--main` branch**, so they ship inside that same
+  release PR. Only do this when no further pushes to `main` are expected before the merge — a later
+  release-please run can regenerate/force-push that branch and drop the commit.
+
+### End-to-end release flow
+
+1. Wait for release-please to open/update the release PR (`release-please--branches--main`).
+2. Run `/fastlane-changelog` (previous tag still latest) → writes `default.txt` for `es-419` + `en-US`.
+3. Commit those `default.txt` files as the final commit onto the release PR branch.
+4. **Merge the release PR** (`gh pr merge <n> --merge` or the GitHub UI) → creates the tag,
+   GitHub Release, and `versionCode` bump.
+5. Build and upload: `fastlane prod_googleplay` (or `beta_googleplay`) — reads `default.txt`.
+
+## Conventions
+
+- Write GitHub issues in **English**, regardless of conversation language.

app/build.gradle.kts

@@ -2,7 +2,7 @@ import org.jetbrains.kotlin.gradle.dsl.JvmTarget
 import java.io.FileInputStream
 import java.util.Properties
 
-val appVersionName = "1.7.10" // x-release-please-version
+val appVersionName = "1.8.0" // x-release-please-version
 val (vMajor, vMinor, vPatch) = appVersionName.split(".").map { it.toInt() }
 
 plugins {

fastlane/metadata/android/en-US/changelogs/default.txt

@@ -1,2 +1,3 @@
-• Improvements to earthquake notifications.
-• Stability improvements.
+• The map now groups nearby earthquakes for a clearer view.
+• Improvements and fixes for earthquake notifications.
+• Stability fixes to prevent unexpected crashes.

fastlane/metadata/android/es-419/changelogs/default.txt

@@ -1,2 +1,3 @@
-• Mejoras en las notificaciones de sismos.
-• Se mejoró la estabilidad del proceso de la aplicacion.
+• El mapa ahora agrupa los sismos cercanos para una vista más clara.
+• Mejoras y correcciones en las notificaciones de sismos.
+• Correcciones de estabilidad para evitar cierres inesperados.