Partially implement javascript source map upload command #41847
Triggered via pull request
November 10, 2025 19:05
andrewbrook
Status
Failure
Total duration
10m 20s
Artifacts
–
node-test.yml
on: pull_request
Matrix: unit
Matrix: agent_evals_build
Matrix: check-json-schema
Matrix: check-package-lock-vsce
Matrix: check-package-lock
Matrix: lint
Matrix: vscode_unit
Matrix: integration-windows
Matrix: integration
Annotations
31 errors and 10 warnings
|
lint (20):
src/commands/crashlytics-sourcemap-upload.ts#L27
'app' is assigned a value but never used
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.ts#L25
Replace `"--app-version·<appVersion>",·"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)"` with `⏎····"--app-version·<appVersion>",⏎····"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)",⏎··`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.ts#L24
Replace `"--bucket-location·<bucketLocation>",·"the·location·of·the·Google·Cloud·Storage·bucket·(default:·\"US-CENTRAL1\""` with `⏎····"--bucket-location·<bucketLocation>",⏎····'the·location·of·the·Google·Cloud·Storage·bucket·(default:·"US-CENTRAL1"',⏎··`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L91
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L84
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L80
Replace `⏎······sinon.match.any,⏎······BUCKET_NAME,⏎····` with `sinon.match.any,·BUCKET_NAME`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L72
Replace `⏎······command.runner()("invalid/path",·{·app:·"test-app"·})⏎····).to.be.rejectedWith(FirebaseError,·"provide·a·valid·file·path·or·directory"` with `command.runner()("invalid/path",·{·app:·"test-app"·})).to.be.rejectedWith(⏎······FirebaseError,⏎······"provide·a·valid·file·path·or·directory",⏎····`
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L67
Replace `"firebasecrashlytics-sourcemaps-12345-a-different-location"` with `⏎······"firebasecrashlytics-sourcemaps-12345-a-different-location",⏎····`
|
|
|
|
lint (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L45
Replace `'filename'` with `"filename"`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.ts#L27
'app' is assigned a value but never used
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.ts#L25
Replace `"--app-version·<appVersion>",·"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)"` with `⏎····"--app-version·<appVersion>",⏎····"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)",⏎··`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.ts#L24
Replace `"--bucket-location·<bucketLocation>",·"the·location·of·the·Google·Cloud·Storage·bucket·(default:·\"US-CENTRAL1\""` with `⏎····"--bucket-location·<bucketLocation>",⏎····'the·location·of·the·Google·Cloud·Storage·bucket·(default:·"US-CENTRAL1"',⏎··`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L91
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L84
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L80
Replace `⏎······sinon.match.any,⏎······BUCKET_NAME,⏎····` with `sinon.match.any,·BUCKET_NAME`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L72
Replace `⏎······command.runner()("invalid/path",·{·app:·"test-app"·})⏎····).to.be.rejectedWith(FirebaseError,·"provide·a·valid·file·path·or·directory"` with `command.runner()("invalid/path",·{·app:·"test-app"·})).to.be.rejectedWith(⏎······FirebaseError,⏎······"provide·a·valid·file·path·or·directory",⏎····`
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L67
Replace `"firebasecrashlytics-sourcemaps-12345-a-different-location"` with `⏎······"firebasecrashlytics-sourcemaps-12345-a-different-location",⏎····`
|
|
|
|
unit (22):
src/commands/crashlytics-sourcemap-upload.spec.ts#L45
Replace `'filename'` with `"filename"`
|
|
unit (20)
The strategy configuration was canceled because "unit._22" failed
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.ts#L27
'app' is assigned a value but never used
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.ts#L25
Replace `"--app-version·<appVersion>",·"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)"` with `⏎····"--app-version·<appVersion>",⏎····"the·version·of·your·Firebase·app·(defaults·to·Git·commit·hash,·if·available)",⏎··`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.ts#L24
Replace `"--bucket-location·<bucketLocation>",·"the·location·of·the·Google·Cloud·Storage·bucket·(default:·\"US-CENTRAL1\""` with `⏎····"--bucket-location·<bucketLocation>",⏎····'the·location·of·the·Google·Cloud·Storage·bucket·(default:·"US-CENTRAL1"',⏎··`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L91
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L84
Replace `⏎······.to.match(/test-app-default-mockdata-mock_mapping\.js\.map\.zip/` with `.to.match(⏎······/test-app-default-mockdata-mock_mapping\.js\.map\.zip/,⏎····`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L80
Replace `⏎······sinon.match.any,⏎······BUCKET_NAME,⏎····` with `sinon.match.any,·BUCKET_NAME`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L72
Replace `⏎······command.runner()("invalid/path",·{·app:·"test-app"·})⏎····).to.be.rejectedWith(FirebaseError,·"provide·a·valid·file·path·or·directory"` with `command.runner()("invalid/path",·{·app:·"test-app"·})).to.be.rejectedWith(⏎······FirebaseError,⏎······"provide·a·valid·file·path·or·directory",⏎····`
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L67
Replace `"firebasecrashlytics-sourcemaps-12345-a-different-location"` with `⏎······"firebasecrashlytics-sourcemaps-12345-a-different-location",⏎····`
|
|
|
|
unit (20):
src/commands/crashlytics-sourcemap-upload.spec.ts#L45
Replace `'filename'` with `"filename"`
|
|
lint (20):
scripts/agent-evals/src/runner/setup.ts#L7
Missing JSDoc comment
|
|
lint (20):
scripts/agent-evals/src/runner/setup.ts#L7
Missing return type on function
|
|
lint (20):
scripts/agent-evals/src/runner/paths.ts#L9
Missing JSDoc comment
|
|
lint (20):
scripts/agent-evals/src/runner/paths.ts#L4
Missing JSDoc comment
|
|
lint (20):
scripts/agent-evals/src/runner/index.ts#L30
Missing JSDoc comment
|
|
lint (20):
scripts/agent-evals/src/runner/index.ts#L15
Missing JSDoc comment
|
|
lint (20):
scripts/agent-evals/src/runner/gemini-cli-runner.ts#L211
Unsafe argument of type `any` assigned to a parameter of type `ParsedTelemetryLog`
|
|
lint (20):
scripts/agent-evals/src/runner/gemini-cli-runner.ts#L210
Unsafe assignment of an `any` value
|
|
lint (20):
scripts/agent-evals/src/runner/gemini-cli-runner.ts#L148
Missing return type on function
|
|
lint (20):
scripts/agent-evals/src/runner/gemini-cli-runner.ts#L124
Invalid type "string[]" of template literal expression
|