Remove deprecated preview server dependency#65
Conversation
|
@codex review |
There was a problem hiding this comment.
Pull request overview
本PRは、ローカルプレビュー用途で使っていた http-server（および経由して発生する whatwg-encoding のdeprecated警告）を排除し、Python標準ライブラリの http.server に置き換えることで、Pages向けビルド成果物（docs/）の生成フローを変えずに依存関係を整理する変更です。
Changes:
devDependenciesからhttp-serverを削除し、ローカルプレビューを Python のhttp.serverに置換package-lock.jsonからhttp-serverおよび関連する推移依存を削除
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | preview スクリプトを python3 -m http.server に変更し、http-server を削除 |
| package-lock.json | http-server と推移依存のロック情報を除去 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "scripts": { | ||
| "build": "node scripts/build-simple.js", | ||
| "preview": "npm run build && npx http-server docs -p 8080 --silent", | ||
| "preview": "npm run build && python3 -m http.server 8080 --directory docs", |
![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: e03f8d7bfd
ℹ️ 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".
| "scripts": { | ||
| "build": "node scripts/build-simple.js", | ||
| "preview": "npm run build && npx http-server docs -p 8080 --silent", | ||
| "preview": "npm run build && python3 -m http.server 8080 --directory docs", |
There was a problem hiding this comment.
Node.js だけを満たす開発環境（README のクイックスタートは前提を Node.js のみにしています）や Windows/minimal Node コンテナでは python3 が存在せず、npm install && npm run preview がビルド後に起動できなくなります。python3 -m http.server --help で --directory 自体は有効なことを確認しましたが、この変更は npm パッケージの依存関係から外れたランタイムを暗黙に要求するため、Node ベースのサーバを維持するか Python 要件とフォールバックを追加してください。
Useful? React with 👍 / 👎.
Refs #55\n\nRemoves unused http-server from devDependencies and replaces the local preview command with Python stdlib http.server. This removes the transitive whatwg-encoding deprecation without changing the Pages build. Faraday warning analysis remains separate and is documented in the source Issue.