Conversation
npm audit (#63 の点検) で firebase-tools の推移依存に moderate が 8 件あった。 うち qs (GHSA-x5fp-wj9c-mxmx / GHSA-4mjr-xmp4-gh2g) は次で解消する。 - firebase-tools ^15.28.2 -> ^15.30.0 - firebase-tools 配下の body-parser を 1.20.6 -> 1.20.8 (qs ~6.16.0) に更新 - express 4.22.2 だけが qs ~6.15.1 を固定しているため、overrides に "express@4": { "qs": "^6.16.0" } を追加 (semver-minor の互換更新。 express 4.22.3 が同じ更新を準備中 expressjs/express#7466 ) 残る csv-parse / stream-json / @opentelemetry/core は firebase-tools の 宣言範囲に修正版が無く、override すると CLI が壊れる ( firebase/firebase-tools#11036 ) ため上流の更新を待つ。 本番依存 (--omit=dev) は前後とも検出 0。
Makefile (make deploy-functions)・functions-deploy.yml (CI のグローバル install)・ revenuecat-webhook.md の手順の firebase-tools を、package.json と同じ 15.30.0 にする。
firebase-tools 15.30.0 で解消できない csv-parse / stream-json / @opentelemetry/core (pubsub 5 経由) について、advisory・修正版・ 解消できない理由・このプロジェクトでの到達可能性・上流の状況と、 firebase-tools を再更新して audit をやり直す条件を残す。
|
Warning Review limit reachedNext included review available in 34 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
#63 の点検で見つかった
firebase/functionsのnpm auditの moderate (firebase-tools の推移依存) のうち、互換性を保ったまま解消できる qs 系を firebase-tools 15.30.0 への更新で解消し、修正版が未提供の残件は理由・影響範囲・再確認条件をdocuments/npm-audit.mdに記録した。本番依存 (--omit=dev) は前後とも検出 0。Closes #67
変更点
firebase/functions/package.json/package-lock.json^15.28.2→^15.30.0(最新)npm update)~6.15.1を固定しているため、overridesに"express@4": { "qs": "^6.16.0" }を追加。semver-minor の互換更新で、express 4.x 側も同じ更新 (qs ~6.16.0) を 4.22.3 として準備中 ( 4.22.3 expressjs/express#7466 )。4.22.3 が firebase-tools に解決されたら削除するnpm audit fix --force= firebase-tools 10.1.1 へのダウングレード) は行っていないMakefile/.github/workflows/functions-deploy.yml/documents/revenuecat-webhook.md: 別途インストールする Firebase CLI の固定バージョンを 15.28.2 → 15.30.0 に揃えたdocuments/npm-audit.md(新規): 残件 3 系統の記録と再確認条件audit の結果 (対象コミット: main の 41f8379 → 本 PR)
npm audit)npm audit --omit=dev)解消: qs ( GHSA-x5fp-wj9c-mxmx / GHSA-4mjr-xmp4-gh2g ) とその経路の body-parser / express@4。
npm ls qs --allで全経路が 6.16.0 になったことを確認した。残件 (すべて firebase-tools の推移依存。詳細は
documents/npm-audit.md):^1.7.3stream-json/filters/Pick等の require が壊れる (上流 firebase/firebase-tools#11036 で実証)auth:import/database:import/ Next.js 連携のみ。本プロジェクトは使わない^5.0.4auth:importの CSV 読み込みのみで、parse()をオプション無しで呼ぶため advisory の前提 (columnsオプション) を満たさない^5.2.0上流の firebase-tools maintainer は issue 11036 で「次のリリースで修正する見込み」と回答している (2026-09-10)。再確認の条件 (firebase-tools の宣言範囲が更新された時、express 4.22.3 が解決された時、severity 引き上げや到達可能性の前提が崩れた時) は
documents/npm-audit.mdに書いた。overridesは CI とmake deploy-functionsが別途インストールするグローバルの firebase-tools には効かないため、そちらの express 4 内の qs は 6.15.3 のまま。この qs はエミュレータ・ローカル HTTP サーバーの query 解析にだけ使われ、デプロイ処理では外部入力を受けない。検証
すべて本 PR の変更を含むツリーで実行し、ログは
./tmp/に保存して全文を warning / error で検査した (該当 0 件)。npm --prefix firebase/functions ci(lockfile の再現)npm run typechecknpm run lintnpm run buildmake test-functions(Firestore / Auth エミュレータ上の vitest)npm audit --omit=dev未検証: 更新した firebase-tools 15.30.0 での本番デプロイ (
functions-deploy.yml) は #58 で行う。エミュレータ (firebase emulators:exec) は 15.30.0 で起動してテストを通した。Codex ローカルレビュー
人間が確認
なし
セッション再開
cd /Users/bannzai/worktrees/bannzai/Alarmify/issue-67 claude --resume 22657054-420b-4123-bd16-827196f76021