Repository navigation
fix: update Nuxt DevTools to 3.4.3 - #3
Merged
Merged
Conversation
hane-jp
marked this pull request as ready for review
October 15, 2026 05:32
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
変更内容
開発サーバーの設定・storage 情報取得に DevTools の認証 token を要求する公式修正を取り込みます。Nuxt の既存依存範囲
@nuxt/devtools: ^3.4.1内で、nuxt/package-lock.jsonだけを更新しました。@nuxt/devtools/@nuxt/devtools-kit/@nuxt/devtools-wizard: 3.4.2 → 3.4.3simple-git、@simple-git/argv-parser、@simple-git/args-pathspec、@kwsites/file-exists、@kwsites/promise-deferredを削除。package-manager-detector@1.9.0を追加し、既存tinyexec@1.3.0を利用。nypmは Nuxt 自身が使用するため残ります。npm update @nuxt/devtools @nuxt/devtools-kit @nuxt/devtools-wizard --package-lock-only --ignore-scripts --no-audit --no-fundを使用。npm が同時に再計算した無関係な Sass optional package 20 件のdevフラグ変更は除外しています。最終 lock でnpm ciを確認済みです。基準:
main/2ecd6f12a855d6ae4d9d1ade24eb7a6f6200c749。リモートに PRODUCTION / DEVELOP はありません。元チェックアウトの未コミット作業は取り込まず、独立チェックアウトを使用しました。公式根拠と確認範囲
getServerConfig、getStorageMounts、getStorageKeysがensureDevAuthToken(token)を先に呼び出すことと、getServerConfigの重複がないことを静的確認しました。検証
Node.js 24.21.0 / npm 11.19.0。tracked source のみを新しい HOME に取り出し、環境変数は
env -iで明示しました。実 env / token / SSH / DB / WordPress / 管理画面 / Sheets / Bot は使用していません。アプリとテストは macOS sandbox で外向き通信を loopback とローカル socket のみに制限し、合成 WordPress fixture を使用しました。npm --prefix nuxt ci --no-audit --no-fundnuxt prepareも成功npm testnpm run buildnpm run generategit diff --check既存の静的サイト制約
Python の静的ファイルサーバーで generate 成果物を配信すると、一覧から日本語スラッグの記事へクライアント遷移した際、payload が使われず
/api/posts/<encoded-slug>を要求して 404 となり、画面が空になります。直接表示・リロードでは記事を表示できます。未変更 main(上記 SHA、DevTools 3.4.2)の別インストール・generate・ブラウザでも同じ失敗を再現しました。 今回の DevTools 更新による回帰ではありません。SSR は同じ日本語遷移が正常です。この既存制約を解消したとは主張せず、本 PR は lock のセキュリティ修正に限定しています。
同時点の監査比較と残存
2026-10-15 03:40:18–03:40:21 UTC に旧・新 lock を同じ npm で連続
npm audit --package-lock-only --json。両方 exit 1(残存 advisory あり)です。via内 object の advisory URL で重複除去)削除した依存から消えた固有 advisory は 4 件です。
simple-gitとその argv parser は lock と実インストールの両方からなくなり、監査の@nuxt/devtoolsへの伝播指摘も消えました。この監査差分と、#1124 の token 必須化の確認は別の根拠です。残存の直接 advisory は
brace-expansion、braces、devalue、node-forge、postcss-selector-parser、shell-quote(critical 1)、source-map-js、undiciです。依存元への伝播分を含む集計が上記 17 パッケージです。一括 audit fix、Nuxt 親更新、無関係な依存更新は含めていません。配備・復旧
push 前に Actions workflow 0、登録 webhook 0、既存 deployment 0 を読み取り確認しました。保守ブランチから main 宛ての draft PR とし、環境ブランチへの直接 push、merge / auto-merge、release push、dispatch、deploy は実行しません。
復旧が必要なら、本 PR の lock 変更を revert(または基準 SHA の
nuxt/package-lock.jsonを戻す)してnpm ciで再構築します。旧 lock は DevTools 3.4.2 に戻り既知修正を失うため、レビューされた復旧として扱い、その間は DevTools を停止するか信頼したローカル環境だけで使ってください。PR 作成後、head
d745db9a629fe73768c5e761380db240ecae2e02に対する check runs 0、commit status 0、Actions runs 0、deployment 0 を確認しました。CI が未設定のため待機対象の実行はありません(CI 成功という意味ではありません)。auto-merge は無効です。人間レビュー待ちで停止しています。