gowalk-cicd 1.0.92 → 1.0.93

package/CLAUDE.md

@@ -172,6 +172,11 @@ node /path/to/gowalk-cicd/bin/cli.mjs
   errors still fail the job. Never use `continue-on-error` or a timeout around all SDK setup
   to hide an optional cache failure. Do not alter cache keys or disable caches as part of
   this timeout handling.
+- Flutter SDK caching restores first and saves immediately after successful setup, before
+  consumer tests/builds can fail and suppress a combined cache action's success-only post hook.
+  Save only on a cache miss with caching enabled, a real successful install and verified cache
+  scope. Preserve resolved keys/paths and the official pinned restore/save programs. Failed or
+  dry-run setup never publishes an SDK cache; pub dependency caching retains its job-end behavior.
 - Gradle caches use explicit checksum-pinned restore/save actions, each optional and bounded
   to two minutes. Preserve setup-java's key inputs and absolute cache paths to reuse existing
   caches. Java setup, builds and store uploads remain mandatory; save runs after delivery.

package/README.md

@@ -30,7 +30,9 @@ Flutter's optional SDK and pub cache restores have a one-minute timeout per
 download segment. A slow segment becomes a cache miss and normal SDK setup
 continues through the assigned Google proxy; SDK setup errors still fail the
 job. This bounds individual cache segments, not the total cache or SDK download,
-and keeps existing cache keys and the successful restore/save behavior.
+and keeps existing cache keys. A successfully installed SDK is saved before app
+checks run, so a later test/build failure does not force another cold SDK download.
+Failed setup never saves an SDK cache; pub dependencies retain their job-end save.
 
 ## Flutter web release
 

package/action/.daemux-version

@@ -1 +1 @@
-1.0.92
+1.0.93

package/android-action/.daemux-version

@@ -1 +1 @@
-1.0.92
+1.0.93

package/android-action/flutter-setup/action.yml

@@ -119,7 +119,7 @@ runs:
   - name: Cache Flutter
     id: cache-flutter
     if: ${{ inputs.cache == 'true' }}
-    uses: actions/cache@caa296126883cff596d87d8935842f9db880ef25
+    uses: actions/cache/restore@caa296126883cff596d87d8935842f9db880ef25
     continue-on-error: true
     env:
       NODE_OPTIONS: --require "${{ steps.scope.outputs.preloader }}"
@@ -139,6 +139,7 @@ runs:
       path: ${{ steps.flutter-action.outputs.PUB-CACHE-PATH }}
       key: ${{ steps.flutter-action.outputs.PUB-CACHE-KEY }}-${{ hashFiles('**/pubspec.lock') }}
   - name: Install or reuse the resolved Flutter SDK
+    id: install
     if: ${{ inputs.dry-run != 'true' }}
     uses: subosito/flutter-action@1a449444c387b1966244ae4d4f8c696479add0b2
     env:
@@ -157,3 +158,16 @@ runs:
       pub-cache-path: ${{ steps.flutter-action.outputs.PUB-CACHE-PATH }}
       dry-run: 'false'
       git-source: ${{ inputs.git-source }}
+  # A later app failure must not discard a successfully installed, expensive SDK.
+  - name: Save installed Flutter SDK
+    if: >-
+      ${{ inputs.cache == 'true' && inputs.dry-run != 'true'
+          && steps.install.outcome == 'success' && steps.scope.outcome == 'success'
+          && steps.cache-flutter.outputs.cache-hit != 'true' }}
+    uses: actions/cache/save@caa296126883cff596d87d8935842f9db880ef25
+    continue-on-error: true
+    env:
+      NODE_OPTIONS: --require "${{ steps.scope.outputs.preloader }}"
+    with:
+      path: ${{ steps.flutter-action.outputs.CACHE-PATH }}
+      key: ${{ steps.flutter-action.outputs.CACHE-KEY }}

package/android-action/scripts/test_google_bootstrap_wiring.py

@@ -24,7 +24,7 @@ class BootstrapWiringTests(unittest.TestCase):
                     self.assertNotIn("continue-on-error", step)
         self.assertEqual(checked, 3)
         setup = yaml.safe_load((ROOT / "android-action/flutter-setup/action.yml").read_text())
-        guard, scope, resolve, sdk_cache, pub_cache, install = setup["runs"]["steps"]
+        guard, scope, resolve, sdk_cache, pub_cache, install, save = setup["runs"]["steps"]
         self.assertIn('play_store_proxy.py" --bootstrap-env', guard["run"])
         self.assertNotIn("continue-on-error", guard)
         self.assertTrue(scope["continue-on-error"])
@@ -39,7 +39,7 @@ class BootstrapWiringTests(unittest.TestCase):
             self.assertEqual(step["with"]["pub-cache"], "false")
             self.assertNotIn("continue-on-error", step)
         self.assertNotIn("BASH_ENV", setup.get("env", {}))
-        for step in (guard, scope, sdk_cache, pub_cache):
+        for step in (guard, scope, sdk_cache, pub_cache, save):
             self.assertNotIn("BASH_ENV", step.get("env", {}))
         self.assertEqual(guard["id"], "transport")
         self.assertIn('>> "$GITHUB_OUTPUT"', guard["run"])
@@ -59,6 +59,25 @@ class BootstrapWiringTests(unittest.TestCase):
         self.assertEqual(setup["outputs"]["CACHE-HIT"]["value"], "${{ steps.cache-flutter.outputs.cache-hit }}")
         self.assertEqual(setup["outputs"]["PUB-CACHE-HIT"]["value"], "${{ steps.cache-pub.outputs.cache-hit }}")
 
+    def test_successful_sdk_is_saved_before_later_consumer_failure_can_discard_it(self):
+        setup = yaml.safe_load((ROOT / "android-action/flutter-setup/action.yml").read_text())
+        steps = setup["runs"]["steps"]
+        restore = next(step for step in steps if step.get("id") == "cache-flutter")
+        self.assertEqual(restore["uses"],
+                         "actions/cache/restore@caa296126883cff596d87d8935842f9db880ef25")
+        install = next(step for step in steps if step.get("id") == "install")
+        save = steps[steps.index(install) + 1]
+        self.assertEqual(save["uses"], "actions/cache/save@caa296126883cff596d87d8935842f9db880ef25")
+        self.assertEqual(save["with"], restore["with"])
+        self.assertTrue(save["continue-on-error"])
+        self.assertEqual(save["env"], {"NODE_OPTIONS": '--require "${{ steps.scope.outputs.preloader }}"'})
+        condition = " ".join(save["if"].split())
+        self.assertEqual(condition, "${{ inputs.cache == 'true' && inputs.dry-run != 'true' "
+                         "&& steps.install.outcome == 'success' && steps.scope.outcome == 'success' "
+                         "&& steps.cache-flutter.outputs.cache-hit != 'true' }}")
+        self.assertNotIn("continue-on-error", install)
+        self.assertNotIn("timeout-minutes", save)  # Unsupported in composite action steps.
+
     def test_pods_cache_keeps_identity_and_uses_the_same_optional_scope(self):
         workflow = yaml.safe_load((ROOT / "templates/deploy.yml").read_text())
         steps = workflow["jobs"]["ios"]["steps"]

package/backend-action/.daemux-version

@@ -1 +1 @@
-1.0.92
+1.0.93

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gowalk-cicd",
-  "version": "1.0.92",
+  "version": "1.0.93",
   "description": "Zero-config GitHub Actions delivery for iOS TestFlight and Android Google Play.",
   "type": "module",
   "bin": {