gowalk-cicd 1.0.111 → 1.0.113

package/CLAUDE.md

@@ -143,6 +143,8 @@ node /path/to/gowalk-cicd/bin/cli.mjs
   Crashlytics Buildtools reads them first and rejects authenticated proxy URLs. HTTPS is tunneled
   without TLS interception; no upstream failure may fall back to the destination.
   The wrapper disables Gradle daemons and closes its relay when the command ends.
+  The relay's idle deadline covers the whole tunnel: traffic in either direction
+  keeps downloads/uploads alive. An idle direction alone must never end a transfer.
 - Flutter bootstrap is part of that boundary: before the Flutter setup action,
   `play_store_proxy.py --bootstrap-env` validates the Google account pin and writes
   both proxy/bypass spellings to the private job environment without console output.
@@ -393,8 +395,12 @@ node /path/to/gowalk-cicd/bin/cli.mjs
   `APP_ROBOT_PLAY_EDIT_BINDING` is panel-provisioned repository data; workflows pass it as
   `PLAY_EDIT_BINDING` with the repo-local GitHub token and `contents:write`/`actions:read`.
   `play-upload/owner_run.cjs` protects Python entrypoints; `run.cjs` holds the same owner across
-  its permitted upload retry. Missing/retired bindings, unknown write outcomes and active owners
-  refuse before provider work; only exact terminal run/attempt/source proof permits recovery.
+  its permitted upload retry. An active owner gets a single 60-second acquisition wait budget;
+  each retry revalidates the record, cancellation interrupts the wait, and an expired budget
+  never clears an owner. Missing/retired bindings and unknown write outcomes still refuse;
+  only exact terminal run/attempt/source proof permits CI-owner recovery.
+  Both entrypoints emit brace-free `play_edit_owner` errors with fixed codes and optional child
+  exit evidence, preserving the existing upload receipt and refusal. Never reconstruct masked text.
   Other packages stay parallel. The GitHub coordination request is fixed-origin and does not
   change the Google account proxy. Update workflow wiring as well as action directories.
 

package/README.md

@@ -361,6 +361,7 @@ the situation and whose **message** is one line of JSON carrying a versioned
 | `play_review_pending` | `gowalk-cicd/play-review-pending.v1` | `package`, `track`, `version_code` |
 | `play_upload_result` | `gowalk-cicd/play-upload.v1` | `ok`, `code`, `attempts`, `review_pending`, upstream identity |
 | `play_upload_diagnostic` | `gowalk-cicd/play-upload-diagnostic.v1` | Fixed process evidence; see below |
+| `play_edit_owner` | `gowalk-cicd/play-owner.v1` | Ownership refusal; no implied upload retry permission |
 | `store_version_locked` | `gowalk-cicd/store-version-locked.v1` | `version`, `state`, `build_number` |
 | `firebase_symbols_pending` | `gowalk-cicd/firebase-symbols-pending.v1` | `platform`, `app_id`, `source_sha`, `artifact`, `file`, `sha256`, `status` |
 | `mobile_upload_reused` | `gowalk-cicd/mobile-upload-reused.v1` | `platform`, `fingerprint`, `run_id`, `run_attempt`, `source_sha`, `deployment_id`, `next_action` |
@@ -475,6 +476,16 @@ bounded to 16 KiB. No provider message, path, URL, track or credential is republ
 These observations never authorize a retry. Missing commit output does not prove no mutation,
 and a reported commit does not prove review submission. Unknown failures remain unclassified.
 
+Package-owner acquisition rechecks an occupied record within one 60-second wait budget before
+provider work. Every read retains the binding and identity checks; cancellation interrupts the wait.
+Time passing never authorizes clearing an owner. A known CI owner still requires exact terminal
+run, attempt and source proof before recovery; the panel releases its own requests.
+Both entrypoints emit `play_edit_owner` as `gowalk-cicd/play-owner.v1 code=<code> child_exit_code=<0-255|unknown>`.
+Codes are `play_owner_binding_required`, `play_owner_unverified`, `play_owner_busy`,
+`play_owner_release_unverified`, `play_owner_cancelled`, `play_owner_principal_unverified`,
+`play_owner_child_failed` and `play_owner_package_unverified`. An unknown child exit proves no outcome.
+These errors preserve the failed job and existing upload receipt; they do not authorize replay.
+
 An expired/deleted edit does not identify its cause. Another edit opened with the
 same API user/package can invalidate it; avoid competing read edits while uploading.
 An external actor is not proven by an expired-edit error alone. Ambiguous outcomes

package/action/.daemux-version

@@ -1 +1 @@
-1.0.111
+1.0.113

package/android-action/.daemux-version

@@ -1 +1 @@
-1.0.111
+1.0.113

package/android-action/play-upload/owner.cjs

@@ -1,5 +1,7 @@
 const http = require('./owner_http.cjs');
 const record = require('./owner_record.cjs');
+const {performance} = require('node:perf_hooks');
+const {setTimeout: delay} = require('node:timers/promises');
 
 async function seal(client, selected) {
   for (let attempt = 0; attempt < 3; attempt++) {
@@ -14,16 +16,27 @@ async function seal(client, selected) {
   throw new Error('play_owner_release_unverified');
 }
 
-async function acquire(client, selected, attempted = () => {}) {
+async function acquire(client, selected, attempted = () => {}, waiting = {}) {
+  const now = waiting.now || (() => performance.now());
+  const sleep = waiting.sleep || ((ms, signal) => delay(ms, undefined, {signal}));
+  const deadline = now() + 60000;
   const protocol = (await client.read('protocol.json')).value;
   if (protocol?.schema !== record.schema || protocol.repository !== selected.value.repo
       || Object.keys(protocol).length !== 2) throw new Error('play_owner_unverified');
-  for (let attempt = 0; attempt < 3; attempt++) {
+  for (let attempt = 0; attempt < 3;) {
+    if (waiting.signal?.aborted) throw new Error('play_owner_cancelled');
     const row = record.validate(await client.read(selected.path), selected);
     if (row.value.owner !== null) {
       const owner = row.value.owner;
-      if (owner.kind !== 'ci' || !record.terminal(await client.run(owner), owner)) throw new Error('play_owner_busy');
+      if (owner.kind !== 'ci' || !record.terminal(await client.run(owner), owner)) {
+        const remaining = deadline - now();
+        if (remaining <= 0) throw new Error('play_owner_busy');
+        await sleep(Math.min(5000, remaining), waiting.signal);
+        continue;
+      }
     }
+    if (waiting.signal?.aborted) throw new Error('play_owner_cancelled');
+    attempt++;
     const desired = record.replacement(row, selected.owner);
     attempted();
     try { await client.write(selected.path, row, desired); } catch { /* Only exact readback may adopt it. */ }
@@ -33,14 +46,15 @@ async function acquire(client, selected, attempted = () => {}) {
   throw new Error('play_owner_busy');
 }
 
-async function withOwner(env, packageName, operation, transport) {
+async function withOwner(env, packageName, operation, transport, timing = {}) {
   const selected = record.binding(env, packageName);
   const client = http.client(selected.value.repo, env.GITHUB_TOKEN, transport);
+  const controller = new AbortController();
   let cancelled = false, dirty = false, result, failure;
-  const stop = () => { cancelled = true; };
+  const stop = () => { cancelled = true; controller.abort(); };
   process.on('SIGTERM', stop); process.on('SIGINT', stop);
   try {
-    await acquire(client, selected, () => { dirty = true; });
+    await acquire(client, selected, () => { dirty = true; }, {...timing, signal: controller.signal});
     if (cancelled) throw new Error('play_owner_cancelled');
     result = await operation(() => cancelled);
   } catch (error) { failure = error; }
@@ -48,7 +62,7 @@ async function withOwner(env, packageName, operation, transport) {
     try { if (dirty) await seal(client, selected); } catch { failure = new Error('play_owner_release_unverified'); }
     process.off('SIGTERM', stop); process.off('SIGINT', stop);
   }
-  if (cancelled && !failure) failure = new Error('play_owner_cancelled');
+  if (cancelled && (!failure || failure.name === 'AbortError')) failure = new Error('play_owner_cancelled');
   if (failure) {
     if (result !== undefined) failure.operationResult = result;
     throw failure;
@@ -58,4 +72,10 @@ async function withOwner(env, packageName, operation, transport) {
 
 const codes = ['play_owner_binding_required', 'play_owner_unverified', 'play_owner_busy',
   'play_owner_release_unverified', 'play_owner_cancelled', 'play_owner_principal_unverified'];
-module.exports = {acquire, seal, withOwner, codes};
+function annotation(reason, childExit) {
+  const allowed = [...codes, 'play_owner_child_failed', 'play_owner_package_unverified'];
+  const code = allowed.includes(reason) ? reason : 'play_owner_unverified';
+  const exit = Number.isInteger(childExit) && childExit >= 0 && childExit <= 255 ? childExit : 'unknown';
+  return `::error title=play_edit_owner::gowalk-cicd/play-owner.v1 code=${code} child_exit_code=${exit}`;
+}
+module.exports = {acquire, seal, withOwner, codes, annotation};

package/android-action/play-upload/owner_run.cjs

@@ -51,9 +51,7 @@ async function main(argv = process.argv.slice(2), env = process.env) {
   } catch (error) {
     const code = [...owner.codes, 'play_owner_child_failed', 'play_owner_package_unverified'].includes(error.message)
       ? error.message : 'play_owner_unverified';
-    const receipt = {schema: 'gowalk-cicd/play-owner.v1', code};
-    if (Number.isInteger(error.operationResult?.code)) receipt.child_exit_code = error.operationResult.code;
-    console.log(`::error title=play_edit_owner::${JSON.stringify(receipt)}`);
+    console.log(owner.annotation(code, error.operationResult?.code));
     process.exitCode = 1;
   }
 }

package/android-action/play-upload/run.cjs

@@ -44,6 +44,7 @@ async function main() {
       upload(program, env, directory, processRunner.execute, 3600000, cancelled,
         (attempt, result) => console.log(diagnostics.annotation(attempt, result))));
   } catch (error) {
+    if (owner.codes.includes(error.message)) console.log(owner.annotation(error.message));
     const allowed = ['assigned_proxy_required', 'runner_context_required', 'uploader_integrity_refused',
       'uploader_source_unavailable', ...owner.codes];
     receipt = {ok: false, code: allowed.includes(error.message) ? error.message : 'uploader_failed',

package/android-action/scripts/jvm_proxy_relay.py

@@ -13,8 +13,8 @@ import threading
 from urllib.parse import unquote, urlsplit
 
 
-# A tunnel carries whole SDK/NDK/CMake downloads and Gradle's keep-alive pools; an
-# idle read is not a failure, so pumps wait long and the tunnel closes cleanly.
+# A tunnel carries whole SDK/NDK/CMake downloads and Gradle's keep-alive pools.
+# Only inactivity in both directions expires it; a download may send no requests.
 IDLE_TIMEOUT = 900.0
 
 
@@ -63,11 +63,16 @@ class Relay:
         await writer.drain()
         return reader, writer
 
-    async def _pump(self, reader, writer):
-        while data := await asyncio.wait_for(reader.read(65536), timeout=IDLE_TIMEOUT):
+    async def _pump(self, reader, writer, activity):
+        while data := await reader.read(65536):
+            activity[0] = self.loop.time()
             writer.write(data)
             await writer.drain()
 
+    async def _idle(self, activity):
+        while remaining := max(0.0, activity[0] + IDLE_TIMEOUT - self.loop.time()):
+            await asyncio.sleep(remaining)
+
     async def _handle(self, reader, writer):
         upstream = None
         pumps = []
@@ -82,8 +87,10 @@ class Relay:
                 writer.write(b"HTTP/1.1 200 Connection established\r\n\r\n")
                 await writer.drain()
             established = True
-            pumps = [asyncio.create_task(self._pump(reader, upstream)),
-                     asyncio.create_task(self._pump(upstream_reader, writer))]
+            activity = [self.loop.time()]
+            pumps = [asyncio.create_task(self._pump(reader, upstream, activity)),
+                     asyncio.create_task(self._pump(upstream_reader, writer, activity)),
+                     asyncio.create_task(self._idle(activity))]
             await asyncio.wait(pumps, return_when=asyncio.FIRST_COMPLETED)
         except (OSError, ValueError, IndexError, asyncio.TimeoutError,
                 asyncio.IncompleteReadError, asyncio.LimitOverrunError):

package/android-action/scripts/test_jvm_proxy_idle.py

@@ -0,0 +1,106 @@
+"""A live tunnel stays open for one-way traffic and closes on total inactivity."""
+from contextlib import contextmanager
+from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
+import socket
+import threading
+import unittest
+from unittest import mock
+
+import jvm_proxy_relay as relay
+
+
+PAYLOAD = bytes(range(24))
+INTERVAL = 0.075
+
+
+@contextmanager
+def tunnel(mode):
+    stop = threading.Event()
+    finished = threading.Event()
+    received = []
+
+    class Proxy(BaseHTTPRequestHandler):
+        def do_CONNECT(self):
+            self.send_response(200)
+            self.end_headers()
+            try:
+                if mode == "download":
+                    for value in PAYLOAD:
+                        self.wfile.write(bytes([value]))
+                        if stop.wait(INTERVAL):
+                            break
+                else:
+                    value = self.rfile.read(len(PAYLOAD))
+                    received.append(value)
+                    if mode == "upload":
+                        self.wfile.write(value)
+            except OSError:
+                pass
+            finally:
+                finished.set()
+
+        def log_message(self, *_):
+            pass
+
+    server = ThreadingHTTPServer(("127.0.0.1", 0), Proxy)
+    server.daemon_threads = True
+    thread = threading.Thread(target=server.serve_forever, daemon=True)
+    thread.start()
+    try:
+        yield f"http://127.0.0.1:{server.server_port}", finished, received
+    finally:
+        stop.set()
+        server.shutdown()
+        server.server_close()
+        thread.join(timeout=5)
+
+
+@contextmanager
+def client(port):
+    with socket.create_connection(("127.0.0.1", port), timeout=5) as connection:
+        with connection.makefile("rb") as stream:
+            connection.sendall(b"CONNECT unresolvable.invalid:443 HTTP/1.1\r\nHost: ignored\r\n\r\n")
+            assert b"200" in stream.readline()
+            while stream.readline() != b"\r\n":
+                pass
+            yield connection, stream
+
+
+class TunnelActivityTests(unittest.TestCase):
+    def test_download_outlives_idle_deadline_without_client_traffic(self):
+        with tunnel("download") as (proxy, finished, _), mock.patch.object(relay, "IDLE_TIMEOUT", 1.0):
+            with relay.running(proxy) as port, client(port) as (_, stream):
+                self.assertEqual(stream.read(len(PAYLOAD)), PAYLOAD)
+            self.assertTrue(finished.wait(5))
+
+    def test_upload_outlives_idle_deadline_without_server_traffic(self):
+        with tunnel("upload") as (proxy, finished, received), mock.patch.object(relay, "IDLE_TIMEOUT", 1.0):
+            with relay.running(proxy) as port, client(port) as (connection, stream):
+                for value in PAYLOAD:
+                    connection.sendall(bytes([value]))
+                    threading.Event().wait(INTERVAL)
+                self.assertEqual(stream.read(len(PAYLOAD)), PAYLOAD)
+            self.assertTrue(finished.wait(5))
+            self.assertEqual(received, [PAYLOAD])
+
+    def test_wholly_idle_tunnel_closes_without_http_bytes(self):
+        with tunnel("idle") as (proxy, finished, received), mock.patch.object(relay, "IDLE_TIMEOUT", 0.2):
+            with relay.running(proxy) as port, client(port) as (_, stream):
+                self.assertEqual(stream.read(), b"")
+            self.assertTrue(finished.wait(5))
+            self.assertEqual(received, [b""])
+
+    def test_owner_exit_closes_active_clients_and_upstream(self):
+        with tunnel("idle") as (proxy, finished, received):
+            with relay.running(proxy) as port:
+                connection = socket.create_connection(("127.0.0.1", port), timeout=5)
+                connection.sendall(b"CONNECT unresolvable.invalid:443 HTTP/1.1\r\n\r\n")
+                self.assertIn(b"200", connection.recv(4096))
+            with connection:
+                self.assertEqual(connection.recv(4096), b"")
+            self.assertTrue(finished.wait(5))
+            self.assertEqual(received, [b""])
+
+
+if __name__ == "__main__":
+    unittest.main()

package/backend-action/.daemux-version

@@ -1 +1 @@
-1.0.111
+1.0.113

package/package.json

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