@eiva/protoc-gen-fletcher 0.1.2-alpha → 0.3.0-alpha

package/README.md

@@ -17,14 +17,14 @@ npm install --save-dev @protobuf-ts/protoc
 ```
 
 ```bash
-npm install --save-dev fletcher-gateway-client@npm:@eiva/fletcher-gateway-client
+npm install --save-dev @eiva/fletcher-gateway-client
 ```
 
 Three pieces:
 
 - **`@eiva/protoc-gen-fletcher`** — this plugin shim.
 - **`@protobuf-ts/protoc`** — ships the `protoc` compiler binary itself. Declared as a peer dependency; npm 7+ auto-installs it, older npm versions print a warning. Install explicitly above to be safe.
-- **`fletcher-gateway-client`** under the bare alias `npm:@eiva/fletcher-gateway-client` — the generated `.fletcher.ts` files import `WireTypeId` and the `TypedSchema` type from a **bare** `'fletcher-gateway-client'` specifier. Installing under that alias makes the published `@eiva/fletcher-gateway-client` resolve at runtime via standard Node resolution (no tsconfig path-alias trickery needed).
+- **`@eiva/fletcher-gateway-client`** — the gateway runtime. The generated `.fletcher.ts` files import `WireTypeId` and the `TypedSchema` type from `'@eiva/fletcher-gateway-client'`, so a plain install resolves them via standard Node resolution — no bare-alias or tsconfig path trickery needed.
 
 ## Usage
 

package/bin/protoc-gen-fletcher.js

@@ -53,13 +53,15 @@ function followRedirects(url, depth) {
           return resolve(followRedirects(res.headers.location, depth + 1));
         }
         if (res.statusCode !== 200) {
-          return reject(
-            new Error(
-              `Download failed (${res.statusCode}): ${url}\n` +
-                `Check that the release exists at ` +
-                `https://github.com/eivacom/Fletcher/releases/tag/protoc-v${VERSION}`,
-            ),
+          // 5xx / 429 are transient (gateway timeouts, rate limiting); tag
+          // them retryable. 4xx (e.g. 404 missing release) is fatal.
+          const err = new Error(
+            `Download failed (${res.statusCode}): ${url}\n` +
+              `Check that the release exists at ` +
+              `https://github.com/eivacom/Fletcher/releases/tag/protoc-v${VERSION}`,
           );
+          err.retryable = res.statusCode >= 500 || res.statusCode === 429;
+          return reject(err);
         }
         const chunks = [];
         res.on('data', (chunk) => chunks.push(chunk));
@@ -70,6 +72,39 @@ function followRedirects(url, depth) {
   });
 }
 
+// GitHub Releases occasionally returns a transient 5xx (e.g. a 504 gateway
+// timeout) or drops the connection. Retry a few times with exponential
+// backoff so a blip doesn't fail codegen for a user; non-retryable (4xx)
+// errors and the final attempt propagate immediately.
+const MAX_DOWNLOAD_ATTEMPTS = 6;
+
+function isRetryable(err) {
+  // Untagged errors (socket resets, timeouts) are transient; only errors
+  // explicitly tagged retryable=false (4xx responses) are fatal.
+  return !(err && err.retryable === false);
+}
+
+async function downloadWithRetry(url) {
+  let lastErr;
+  for (let attempt = 1; attempt <= MAX_DOWNLOAD_ATTEMPTS; ++attempt) {
+    try {
+      return await followRedirects(url, 0);
+    } catch (err) {
+      lastErr = err;
+      if (!isRetryable(err) || attempt === MAX_DOWNLOAD_ATTEMPTS) {
+        break;
+      }
+      const delayMs = Math.min(1000 * 2 ** (attempt - 1), 8000); // 1,2,4,8,8s
+      process.stderr.write(
+        `protoc-gen-fletcher: download attempt ${attempt}/${MAX_DOWNLOAD_ATTEMPTS} failed ` +
+          `(${err.message}); retrying in ${delayMs}ms\n`,
+      );
+      await new Promise((resolve) => setTimeout(resolve, delayMs));
+    }
+  }
+  throw lastErr;
+}
+
 async function ensureBinary() {
   const asset = platformAssetName();
   const cacheBin = path.join(cacheDir(), asset);
@@ -81,7 +116,7 @@ async function ensureBinary() {
   process.stderr.write(`protoc-gen-fletcher: downloading ${url}\n`);
 
   fs.mkdirSync(cacheDir(), { recursive: true });
-  const buf = await followRedirects(url, 0);
+  const buf = await downloadWithRetry(url);
 
   // Atomic write: write to a unique tmp file in the same dir, then rename.
   // Multiple npm scripts may invoke the shim in parallel; rename within the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eiva/protoc-gen-fletcher",
-  "version": "0.1.2-alpha",
+  "version": "0.3.0-alpha",
   "description": "protoc plugin that generates typed TypeScript schema descriptors for Fletcher's positional wire format. Downloads the platform-matching native plugin binary from GitHub Releases on first invocation.",
   "license": "LGPL-3.0-or-later",
   "homepage": "https://github.com/eivacom/Fletcher/tree/main/protoc",