You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Error: Cannot find module '.../functions/ts-node/register'
imported from .../node_modules/ava/lib/worker/base.js
ava 8 breaking changes:
AVA is now fully ESM internally; the require: ['ts-node/register'] config in package.json#ava no longer works as before
Test files are loaded via import(), not require()
CJS projects need const {default: test} = require('ava') instead of const test = require('ava')
Requires @ava/typescript@7 (already merged) plus updating how TypeScript transpilation is configured (should use --import / Node.js loader hooks instead of ts-node/register)
Action needed: Update functions/package.json ava config to use the new ESM-based TypeScript loader. This is a manual change that requires testing — skipping for now.
tsconfig.json(19,23): error TS5107: Option 'moduleResolution=node10' is deprecated
and will stop functioning in TypeScript 7.0. Specify compilerOption '"ignoreDeprecations": "6.0"'
Fix (already documented in CLAUDE.md):
Change moduleResolution: "node" to "node16" / "nodenext" and remove ignoreDeprecations entirely.
Note: using ignoreDeprecations: "6.0" in tsconfig.json breaks ts-node with TS5103; the proper fix is to update moduleResolution. Would you like me to apply this fix?
Dependabot PR Review — 2026-06-12
Automated review of 7 open Dependabot PRs on
develop.Summary Table
Changelog Analysis
✅ #446 — @grpc/grpc-js 1.14.3 → 1.14.4 (patch, security)
Two security fixes only:
No breaking changes. Safe to merge.
✅ #442 — firebase-tools 15.18.0 → 15.19.1 (minor)
Emulator & Data Connect updates only. No breaking changes. Dev dependency.
❌ #441 — ava 7.0.0 → 8.0.1 (major) — CI FAILING
Root cause (from CI logs):
ava 8 breaking changes:
require: ['ts-node/register']config inpackage.json#avano longer works as beforeimport(), notrequire()const {default: test} = require('ava')instead ofconst test = require('ava')@ava/typescript@7(already merged) plus updating how TypeScript transpilation is configured (should use--import/ Node.js loader hooks instead ofts-node/register)Action needed: Update
functions/package.jsonava config to use the new ESM-based TypeScript loader. This is a manual change that requires testing — skipping for now.✅ #440 — eslint-plugin-prettier 5.5.5 → 5.5.6 (patch)
Dependency bump only. No breaking changes. Dev dependency.
✅ #438 — ajv 8.18.0 → 8.20.0 (minor)
$datarefNo breaking changes. Includes a security fix. Safe to merge.
❌ #424 — TypeScript 5.9.3 → 6.0.3 (major) — CI FAILING
Root cause (from CI logs):
Fix (already documented in CLAUDE.md):
Note: using
ignoreDeprecations: "6.0"in tsconfig.json breaks ts-node with TS5103; the proper fix is to updatemoduleResolution. Would you like me to apply this fix?CI is green but:
develop)eslint+eslint-plugin-prettier(chore(deps-dev): bump eslint-plugin-prettier from 5.5.5 to 5.5.6 in /functions #440), making tslint redundantRecommendation: Close this PR. Remove
tslintfromfunctions/package.jsonentirely rather than upgrading a deprecated package.Requested Confirmations
Before I proceed with merges, please confirm or override the plan by commenting on this issue:
approveor override specific PRsmoduleResolution: node16) on the Dependabot branch so CI passes, then merge? Replyfix ts6orskipskip avaorinvestigateremove tslintorskip