@scrymore/scry-deployer 0.5.0 → 0.5.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/lib/apiClient.js +15 -1
- package/package.json +1 -1
package/lib/apiClient.js
CHANGED
|
@@ -427,12 +427,26 @@ async function uploadBuild(apiClient, target, options) {
|
|
|
427
427
|
logger.info(`Waiting ${COVERAGE_UPLOAD_DELAY_MS / 1000}s before uploading coverage report...`);
|
|
428
428
|
await sleep(COVERAGE_UPLOAD_DELAY_MS);
|
|
429
429
|
|
|
430
|
+
// The retry used to be unguarded, so a second failure threw out of
|
|
431
|
+
// uploadBuild and the metadata upload below never ran. Coverage is a
|
|
432
|
+
// report; the metadata archive is what makes components searchable — so a
|
|
433
|
+
// failure in the optional artifact was silently taking down the essential
|
|
434
|
+
// one. Observed on a 467-story library: every story captured, nothing
|
|
435
|
+
// indexed, twice.
|
|
430
436
|
try {
|
|
431
437
|
coverageUpload = await uploadCoverageReportDirectly(apiClient, target, options.coverageReport);
|
|
432
438
|
} catch (error) {
|
|
433
439
|
logger.info('Coverage upload failed; retrying in 60s...');
|
|
434
440
|
await sleep(COVERAGE_RETRY_DELAY_MS);
|
|
435
|
-
|
|
441
|
+
try {
|
|
442
|
+
coverageUpload = await uploadCoverageReportDirectly(apiClient, target, options.coverageReport);
|
|
443
|
+
} catch (retryError) {
|
|
444
|
+
logger.warn(
|
|
445
|
+
`⚠️ Coverage report upload failed: ${retryError.message}\n` +
|
|
446
|
+
' Continuing — this affects the coverage report only. Screenshots and\n' +
|
|
447
|
+
' metadata still upload, so components will still be indexed.'
|
|
448
|
+
);
|
|
449
|
+
}
|
|
436
450
|
}
|
|
437
451
|
}
|
|
438
452
|
|