Repository navigation
Conversation
simple-gitとパッケージ作成ツールを更新し、修正版のないbracesを検索処理から除去する。除外・再包含・シンボリックリンクの回帰テストを追加し、開発用を含むnpm auditの検出を0件にする。
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. |
長い検索パターンのENAMETOOLONGを非ディレクトリとして扱い、異常入力テストへのカバレッジ継承を防ぐ。 Codexの完了サマリーと太字のコミット表記に対応し、実行中・不明確な結果や優先度付きの指摘で誤って承認しない回帰テストを追加する。
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d73a94e3aa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc45c6f3f5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
概要
npm auditは18件から0件になりました。.gitignore、明示的な再包含、プレビュー、通常・暗号化ZIPの動作を回帰テストで確認しました。変更内容
simple-gitを4.0.2へ更新し、ソースとテストを名前付きsimpleGitimportへ移行bracesを依存ツリーから除去するため、globbyをglobとignoreに置き換えglobの同梱ライブラリを避け、lockfileの修正版を使うglob/rawを利用@vscode/vsceを4.0.0へ更新し、パッケージ作成ツール経由の脆弱な依存も除去brace-expansionの既存overrideを各系列の修正版へ更新し、Mocha配下のdiffに修正版のoverrideを追加js-yaml、shell-quote、qsなどの推移的依存を更新。undici・markdown-itはツリーから除去ENAMETOOLONGと、異常入力テストへのカバレッジ継承を修正!**/.config.jsonのようなドット付き隠しディレクトリの再包含と、その配下の除外を修正**/cache.v1のような通常のドット付きディレクトリ名を再帰展開し、ワイルドカード除外も一致したディレクトリの子孫へ適用。単体テストとZIP出力の回帰テストを追加deepをパターンの静的基準ディレクトリから数えるよう修正。混在・brace・絶対パス・親パスの回帰テストを追加テスト
ローカルはNode.js 24.22.0 / npm 10.9.0のWindows一時コピーで検証しました。UbuntuでもGitHub Actions（Node.js 24.x / npm 10.9.0）のCI・Coverage・CodeQL・VSIX検証が成功しました。
npm run check-typesnpm run lintnpm run test:unit（Windows: 63件成功、Ubuntu: CI成功）npm run coverage:unit（Windows・Ubuntu CIで成功）npm test（VS Code 1.143.0、60件成功。Windows対象外のPOSIX権限テスト1件は保留）npm run package（本番ビルド・SBOM生成）npm run package:verify（VSIX作成成功）npm audit --json --package-lock-only --ignore-scripts（開発用を含めて0件）npm audit signatures --omit=dev（97パッケージの署名検証成功）git diff --checkレビューで維持した仕様
!**/configのような名前付きディレクトリの明示的再包含は、選択したルート内で、その配下の.envや鍵ファイルも含みます。README・アーキテクチャ・既存テストが定める「再包含は組み込み除外より優先」を維持し、READMEの日英説明とZIP出力テストを追加しました。意図したファイルだけを戻す場合は具体的なファイルパスを指定します。GIT_CONFIG_GLOBALなど実行可能な設定を変える環境変数は引き続き許可しません。今回復元したのはGitのidentity 6変数のみです。変更点をCHANGELOGに記載しました。存在しないパスは不一致として完了し、実際の検索I/O失敗は元のエラーで拒否します。長い異常パターンは正常完了または捕捉可能な
ENAMETOOLONGとなり、停止・プロセスクラッシュを起こさないことを検証しました。汎用検索APIの親パス・絶対パス対応は維持し、エクスポート・プレビューの呼び出し側でルート制限を必須にしています。外部パスの再包含は無視します。
補足
関連PR: chore(deps): bump simple-git from 3.36.0 to 4.0.2 #344、chore(deps): bump the npm_and_yarn group across 1 directory with 2 updates #343、chore(deps-dev): bump @vscode/vsce from 3.9.2 to 4.0.0 #339
GitHub上のDependabotアラートはmainへのマージ後に再評価されます。
AI Review Gateは
pull_request_targetで信頼済みのmain側のスクリプトを実行します。このPR内のゲート修正はmainへの取り込み後に反映されます。