diff --git a/.agent_memory/session_context.archive.md b/.agent_memory/session_context.archive.md index bda6af55e..d2026cebc 100644 --- a/.agent_memory/session_context.archive.md +++ b/.agent_memory/session_context.archive.md @@ -2,6 +2,14 @@ # Older handover entries rotated out of session_context.md. Not read by default. # Consult only if you need historical detail on a specific past change. +## 2026-05-21 — Implemented streaming chat support and Firebase Remote Config integration for Chirpy +- Added `firebase-config` dependency to Version Catalog `libs.versions.toml` and `androidApp/build.gradle.kts`. +- Added the `AIDocAssistantResult.Partial` variant to support intermediate stream updates. +- Extended the `AIDocAssistant` interface and implemented `answerStream` in both `KeywordFallbackAssistant` and `GeminiNanoDocAssistant`. +- Integrated Firebase Remote Config into `GeminiNanoDocAssistant` to dynamically fetch the model name (`chirpy_model_name`) and system instruction (`chirpy_system_instruction`) with release-optimized fetch intervals. +- Refactored `GeminiNanoDocAssistant.answer` to reuse `answerStream` flow under the hood, eliminating duplicate prompting code. +- Verified that all unit tests (`:feature:docs:allTests`) and static analysis checks (`spotlessApply spotlessCheck detekt`) pass 100% green. + ## 2026-05-21 — Fixed Chirpy Assistant invalid model name and enhanced failure fallback suggestions - Fixed a 404/Unknown inference error by updating `GeminiNanoDocAssistant.kt`'s `MODEL_NAME` from `"gemini-3.1-flash-lite"` to the correct Firebase AI Logic preview name `"gemini-3.1-flash-lite-preview"`. - Overhauled multi-turn hybrid chat seeding: eliminated the redundant background `chat.sendMessage` call on the first turn; if the first turn is answered on-device, the session caches the Q&A locally and seeds the subsequent cloud-chat session via `startChat(history = ...)`. diff --git a/.agent_memory/session_context.md b/.agent_memory/session_context.md index b558448f0..ba1b26266 100644 --- a/.agent_memory/session_context.md +++ b/.agent_memory/session_context.md @@ -6,6 +6,12 @@ # the oldest entries to `session_context.archive.md` (not read by default). The # "Golden Context" block at the bottom is stable across sessions; keep it here. +## 2026-06-10 — Fixed Update Changelog workflow crash (failing on every main push since 2026-06-05) +- PR #5769: `.github/workflows/update-changelog.yml` died with `TAG_NAMES: bad array subscript` once v2.7.14 went prod and its `-internal.*`/`-open.*` channel tags were cleaned up — zero channel tags means N=0 and `${TAG_NAMES[$((N-1))]:-$PROD_TAG}` indexes [-1] on an empty array, fatal under `bash -e` before the `:-` fallback applies. Replaced with an explicit `if (( N > 0 ))` branch. +- Second latent bug fixed in the same step: the final `{ ... } > /tmp/unreleased-section.md` group ended with `[ -n ... ] && echo` lists; with SECTIONS and CONTRIBUTORS both empty the group (the script's last command) returns 1 and fails the step even though the file is written. Converted to `if` statements. Previously masked because the prod→HEAD range always had a New Contributors section. +- Verified by extracting the step script from the YAML (10-space block indent stripped, heredoc terminators land at col 0) and running it against the live repo: failing env (prod-only) now exits 0 with correct output; simulated channel tag (N=1) confirms segmented path unchanged. Local BSD-sed chokes on a GNU-sed idiom in `generate_notes_api` — harmless locally, runner is GNU. +- Push gotcha: both the git credential and gh token lack `workflow` scope (contents API rejects too) — pushing workflow-file changes works via SSH (`git push git@github.com:meshtastic/Meshtastic-Android.git `), which isn't scope-limited. + ## 2026-06-03 — Cluster-marker FATAL: revert shipped map series + in-scope rememberComposeBitmapDescriptor fix - Reverted ALL google-flavor map changes to before #5684 (per user): restored MapView.kt, NodeClusterMarkers.kt, WaypointMarkers.kt, InlineMap.kt to parent commit bc9f1637; deleted MarkerBitmapRenderer.kt; re-pinned `play-services-maps = 20.0.0` in libs.versions.toml. The shipped #5702–#5719 series (Canvas markers + ViewTree-owner band-aids) had lost the info-window popups + interactions. - Root cause (verified against maps-compose 8.3.0 + android-maps-utils 4.1.1 SOURCE in gradle cache): ONLY `Clustering(clusterItemContent=…)` crashes — its `ComposeUiClusterRenderer` builds a *detached* `InvalidatingComposeView` with a fake lifecycle owner and NO SavedStateRegistryOwner. `MarkerComposable` already bakes its icon via the safe in-scope `rememberComposeBitmapDescriptor`; info windows render with the live marker compositionContext. So InlineMap/NodeTrack/Traceroute were left untouched. @@ -31,14 +37,6 @@ - Refined the `SYSTEM_INSTRUCTION` personality rules for Chirpy to position him as our adorable LoRa radio Node mascot instead of an avian theme, emphasizing high-enthusiasm mesh networking, signal connectivity, battery status, and radio/routing concepts while preserving technical precision. - Overhauled system error messages inside `DocsNavigation.kt` and the loading bubble state inside `ChirpyAssistantSheet.kt` to align with the mascot theme. -## 2026-05-21 — Implemented streaming chat support and Firebase Remote Config integration for Chirpy -- Added `firebase-config` dependency to Version Catalog `libs.versions.toml` and `androidApp/build.gradle.kts`. -- Added the `AIDocAssistantResult.Partial` variant to support intermediate stream updates. -- Extended the `AIDocAssistant` interface and implemented `answerStream` in both `KeywordFallbackAssistant` and `GeminiNanoDocAssistant`. -- Integrated Firebase Remote Config into `GeminiNanoDocAssistant` to dynamically fetch the model name (`chirpy_model_name`) and system instruction (`chirpy_system_instruction`) with release-optimized fetch intervals. -- Refactored `GeminiNanoDocAssistant.answer` to reuse `answerStream` flow under the hood, eliminating duplicate prompting code. -- Verified that all unit tests (`:feature:docs:allTests`) and static analysis checks (`spotlessApply spotlessCheck detekt`) pass 100% green. - ## Golden Context (stable across sessions) - Always check `.skills/compose-ui/strings-index.txt` before reading `strings.xml`. - Run `python3 scripts/sort-strings.py` after adding strings to keep the index organized. diff --git a/.github/workflows/update-changelog.yml b/.github/workflows/update-changelog.yml index 5359d6219..b6cfee0a3 100644 --- a/.github/workflows/update-changelog.yml +++ b/.github/workflows/update-changelog.yml @@ -210,7 +210,14 @@ jobs: # === Unreleased segment (HEAD -> newest tag) === UNRELEASED_SECTION="" - LATEST_TAG="${TAG_NAMES[$((N-1))]:-$PROD_TAG}" + # Indexing TAG_NAMES[-1] on an empty array is a fatal error under + # bash -e, so the no-channel-tags state (right after a production + # release) needs an explicit branch instead of a :- fallback. + if (( N > 0 )); then + LATEST_TAG="${TAG_NAMES[$((N-1))]}" + else + LATEST_TAG="$PROD_TAG" + fi if [ "$HEAD_SHA" != "$(git rev-parse "$LATEST_TAG" 2>/dev/null)" ]; then UNRELEASED_COMMITS=$(generate_notes_git "$LATEST_TAG" "$HEAD_SHA") if [ -n "$UNRELEASED_COMMITS" ]; then @@ -223,13 +230,15 @@ jobs: fi fi - # Assemble the full unreleased section (newest first) + # Assemble the full unreleased section (newest first). + # Use if-statements, not `[ -n ... ] && ...` lists: a failed test as + # the group's last command makes the whole step exit 1 under bash -e. { echo "## [Unreleased]" echo "" - [ -n "$UNRELEASED_SECTION" ] && echo "$UNRELEASED_SECTION" && echo "" - [ -n "$SECTIONS" ] && echo "$SECTIONS" && echo "" - [ -n "$CONTRIBUTORS" ] && echo "$CONTRIBUTORS" + if [ -n "$UNRELEASED_SECTION" ]; then echo "$UNRELEASED_SECTION"; echo ""; fi + if [ -n "$SECTIONS" ]; then echo "$SECTIONS"; echo ""; fi + if [ -n "$CONTRIBUTORS" ]; then echo "$CONTRIBUTORS"; fi } > /tmp/unreleased-section.md - name: Update CHANGELOG.md