Repository navigation
fix: 全領域監査で検出した表示・検索・サーバーの不具合を修正 - #27
Conversation
API例外、symlink境界、正規表現検索の停止、HTML撮影の隔離、非同期表示競合、データ破損と機密マスク漏れ、画像解析、テーマを修正。 回帰テスト、Node 20/22 CI、監査記録を追加し依存関係を更新。
JSONLの元ソース行・レコード内容まで検証し、非同期ログ描画を待機する。 TOC属性エスケープを直接コンポーネントで検証し、GitHub配布経路の依存解決制約を記録。
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a5ea3840ea
ℹ️ 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".
| }); | ||
|
|
||
| for (const signal of ['SIGINT', 'SIGTERM']) { | ||
| process.on(signal, () => child.kill(signal)); |
There was a problem hiding this comment.
Preserve the forwarded signal in the wrapper's exit status
When the wrapper receives SIGINT or SIGTERM (for example, from Ctrl-C or a service manager), this handler suppresses Node's default signal termination and forwards the signal to the child. The child then exits with code === null, and the existing child.on('exit', (code) => process.exit(code ?? 0)) reports success, so scripts cannot distinguish an interrupted server from a clean shutdown. Re-raise the signal in the parent or map the child's exit signal to the conventional nonzero status.
Useful? React with 👍 / 👎.
修正内容
不正な入力でのサーバー終了、公開範囲外の symlink 読み込み、正規表現検索による停止、HTML 画像保存時の sandbox 権限拡大、非同期応答による画面の上書き、データ表示と機密マスクの欠落を修正します。
詳細な不具合一覧・対象範囲・制約は 監査記録 を参照してください。
検証
GitHub Actions は最終コミット
a5ea3840ea3eae3a6fe7af9ba1dd744025e68963で成功しました。npm ci/ prepare build(TypeScript strict を含む)npm run test:unitnpm run check:contrastnpm audit: リポジトリの固定依存環境で 14 件 → 0 件。npm ciと prepare build を確認しました。残件・制約
npm ci環境に限定されます。固定依存を使用するには、チェックアウトしてnpm ci後にnode bin/mdv.mjs /path/to/documentsで起動してください。上流の固定依存解消は残件です。未知の不具合が存在しないことを保証するものではなく、今回確認できた不具合を修正し、再発検知用のテストを追加した PR です。