Skip to content

chore(deps): autoupdate (needs claude fix)#15

Merged
siarheidudko merged 3 commits into
mainfrom
chore/autoupdate-27852935886
Jun 20, 2026
Merged

chore(deps): autoupdate (needs claude fix)#15
siarheidudko merged 3 commits into
mainfrom
chore/autoupdate-27852935886

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

The dependency autoupdater failed during builds-and-checks.

Run log: https://github.com/siarheidudko/firebase-admin-cli/actions/runs/27852935886

A Claude session has been dispatched to push fixes onto this branch
so the following commands all exit 0:

  npm run build
  npm test

Claude will leave a status comment on this PR when it finishes.
PR-checks will re-run on each new commit.

@github-actions github-actions Bot requested a review from siarheidudko June 19, 2026 23:26
@github-actions github-actions Bot added autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix labels Jun 19, 2026
firebase-admin v14 removed the legacy namespaced compat API
(admin.auth(), admin.credential.cert(), admin.firestore(), etc.).
Update bin/firebase-cli.js to use the new modular API (getAuth,
getDatabase, getFirestore, getStorage, cert) and update src/index.ts
to import types from individual firebase-admin subpath exports
instead of the removed namespace members.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@claude

claude Bot commented Jun 19, 2026

Copy link
Copy Markdown

Autoupdate fix summary

Initial failure: npm run build failed with 6 TypeScript errors in src/index.ts. The errors were caused by firebase-admin v14 removing its legacy namespaced types (admin.app.App, admin.auth.Auth, admin.firestore.Firestore, admin.database.Database, admin.storage.Storage, and typeof admin.firestore), which TypeScript 6 now strictly enforces.

Additionally, bin/firebase-cli.js would crash at runtime because firebase-admin v14 removed the compat API entirely — admin.credential.cert(), admin.auth(), admin.firestore(), admin.database(), and admin.storage() are all undefined.

Changes made:

  1. src/index.ts — Updated type imports to use firebase-admin v14 subpath exports (firebase-admin/app, firebase-admin/auth, firebase-admin/database, firebase-admin/firestore, firebase-admin/storage) instead of the removed namespace members.

  2. bin/firebase-cli.js — Updated runtime calls to use the new modular API:

    • admin.credential.cert()admin.cert()
    • admin.auth()getAuth(app) (from firebase-admin/auth)
    • admin.database()getDatabase(app) (from firebase-admin/database)
    • admin.firestore()firestoreModule.getFirestore(app) (from firebase-admin/firestore)
    • admin.storage()getStorage(app) (from firebase-admin/storage)
    • admin.firestore (types namespace) → require('firebase-admin/firestore') module

Verification: npm run build exits 0. npm test exits 1 locally (expected — no GOOGLE_APPLICATION_CREDENTIALS; requires the SERVICE_ACCOUNT secret in CI).

Commit pushed: 58ddbcb

@siarheidudko siarheidudko merged commit f58998c into main Jun 20, 2026
4 checks passed
@siarheidudko siarheidudko deleted the chore/autoupdate-27852935886 branch June 20, 2026 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant