feat: retry staple up to 3 times (#144)

electron · Jun 29, 2023 · a59d260 · a59d260
1 parent 12c1379
commit a59d260
Show file tree

Hide file tree

Showing 9 changed files with 27 additions and 9 deletions.
diff --git a/package.json b/package.json
@@ -32,13 +32,15 @@
     "@types/fs-extra": "^9.0.1",
     "@types/jest": "^29.0.0",
     "@types/node": "^13.7.7",
+    "@types/promise-retry": "^1.1.3",
     "jest": "^29.0.0",
     "prettier": "^1.18.2",
     "ts-jest": "^29.0.0",
     "typescript": "^4.8.4"
   },
   "dependencies": {
     "debug": "^4.1.1",
-    "fs-extra": "^9.0.1"
+    "fs-extra": "^9.0.1",
+    "promise-retry": "^2.0.1"
   }
 }
diff --git a/src/helpers.ts b/src/helpers.ts
@@ -1,4 +1,4 @@
-import * as debug from 'debug';
+import debug from 'debug';
 import * as fs from 'fs-extra';
 import * as os from 'os';
 import * as path from 'path';

diff --git a/src/index.ts b/src/index.ts
@@ -1,4 +1,6 @@
-import * as debug from 'debug';
+import debug from 'debug';
+import retry from 'promise-retry';
+
 import { delay } from './helpers';
 import { startLegacyNotarize, waitForLegacyNotarize } from './legacy';
 import { isNotaryToolAvailable, notarizeAndWaitForNotaryTool } from './notarytool';
@@ -48,5 +50,7 @@ export async function notarize({ appPath, ...otherOptions }: NotarizeOptions) {
     } as NotaryToolStartOptions);
   }
 
-  await stapleApp({ appPath });
+  await retry(() => stapleApp({ appPath }), {
+    retries: 3,
+  });
 }
diff --git a/src/legacy.ts b/src/legacy.ts
@@ -1,4 +1,4 @@
-import * as debug from 'debug';
+import debug from 'debug';
 import * as path from 'path';
 
 import { spawn } from './spawn';

diff --git a/src/notarytool.ts b/src/notarytool.ts
@@ -1,4 +1,4 @@
-import * as debug from 'debug';
+import debug from 'debug';
 import * as path from 'path';
 
 import { spawn } from './spawn';

diff --git a/src/spawn.ts b/src/spawn.ts
@@ -1,5 +1,5 @@
 import { spawn as cpSpawn, SpawnOptions } from 'child_process';
-import * as debug from 'debug';
+import debug from 'debug';
 import { isSecret } from './helpers';
 
 const d = debug('electron-notarize:spawn');

diff --git a/src/staple.ts b/src/staple.ts
@@ -1,4 +1,4 @@
-import * as debug from 'debug';
+import debug from 'debug';
 import * as path from 'path';
 
 import { spawn } from './spawn';

diff --git a/tsconfig.json b/tsconfig.json
@@ -10,10 +10,10 @@
 		"noUnusedLocals": true,
 		"noImplicitThis": true,
 		"noImplicitAny": true,
-		"noImplicitUseStrict": true,
 		"declaration": true,
 		"skipDefaultLibCheck": true,
     "skipLibCheck": true,
+		"esModuleInterop": true
 	},
 	"include": [
 		"src/**/*"

diff --git a/yarn.lock b/yarn.lock
@@ -924,6 +924,18 @@
   resolved "https://registry.yarnpkg.com/@types/prettier/-/prettier-2.7.1.tgz#dfd20e2dc35f027cdd6c1908e80a5ddc7499670e"
   integrity sha512-ri0UmynRRvZiiUJdiz38MmIblKK+oH30MztdBVR95dv/Ubw6neWSb8u1XpRb72L4qsZOhz+L+z9JD40SJmfWow==
 
+"@types/promise-retry@^1.1.3":
+  version "1.1.3"
+  resolved "https://registry.yarnpkg.com/@types/promise-retry/-/promise-retry-1.1.3.tgz#baab427419da9088a1d2f21bf56249c21b3dd43c"
+  integrity sha512-LxIlEpEX6frE3co3vCO2EUJfHIta1IOmhDlcAsR4GMMv9hev1iTI9VwberVGkePJAuLZs5rMucrV8CziCfuJMw==
+  dependencies:
+    "@types/retry" "*"
+
+"@types/retry@*":
+  version "0.12.2"
+  resolved "https://registry.yarnpkg.com/@types/retry/-/retry-0.12.2.tgz#ed279a64fa438bb69f2480eda44937912bb7480a"
+  integrity sha512-XISRgDJ2Tc5q4TRqvgJtzsRkFYNJzZrhTdtMoGVBttwzzQJkPnS3WWTFc7kuDRoPtPakl+T+OfdEUjYJj7Jbow==
+
 "@types/stack-utils@^2.0.0":
   version "2.0.1"
   resolved "https://registry.yarnpkg.com/@types/stack-utils/-/stack-utils-2.0.1.tgz#20f18294f797f2209b5f65c8e3b5c8e8261d127c"