DevContainer基盤モードへの整理と起動フロー改善#2
Merged
Merged
Conversation
added 6 commits
April 1, 2026 01:13
There was a problem hiding this comment.
Pull request overview
DevContainer を「基盤(Infrastructure Mode)専用」として運用する方針に合わせ、テンプレート生成機能を削除しつつ、セットアップ/起動フローと Tailscale 周りの処理を共通化・簡素化するPRです。
Changes:
- プロジェクト生成テンプレート群を削除し、基盤運用向けの README / QUICK_START に刷新
.devcontainer/lib/common.shを追加して.env処理・Tailscale key 判定・権限設定などを共通化- 起動/セットアップ/診断スクリプトを基盤モードに合わせて整理(Tailscale socket 指定、監視プロセス追加、権限/ajv 対策など)
Reviewed changes
Copilot reviewed 21 out of 21 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
README.md |
基盤モード前提の説明に刷新(開発成果物を置かない方針明記) |
docs/QUICK_START.md |
基盤起動に必要な最小手順へ整理 |
templates/README.template.md |
旧プロジェクト生成テンプレート削除 |
templates/README.md |
旧テンプレート管理 README 削除 |
templates/package.template.json |
旧プロジェクト生成テンプレート削除 |
templates/dashboard-with-tailscale.template.js |
旧ダッシュボードテンプレート削除 |
templates/app.template.js |
旧サンプルアプリテンプレート削除 |
scripts/setup-tailscale.sh |
共通ヘルパ導入、socket 指定統一、.env 更新処理統一 |
scripts/fix-permissions.sh |
.opencode 権限修正ツール追加 |
scripts/fix-ecc-ajv.sh |
ECC の ajv 依存問題向け修正ツール追加 |
scripts/diagnose-devcontainer.sh |
DevContainer 診断ツール追加 |
.devcontainer/lib/common.sh |
.env/Tailscale 判定/権限/hosts 更新の共通関数を追加 |
.devcontainer/interactive-setup.sh |
プロジェクト生成をやめ、Tailscale 中心の基盤セットアップに縮小 |
.devcontainer/env-validator.sh |
.env 解決/権限/キー検証を共通関数に寄せ、ローカルモード許容へ |
.devcontainer/setup.sh |
基盤セットアップ誘導、権限/ajv 対策を追加 |
.devcontainer/startup.sh |
hosts 修正・Tailscale 起動(socket)・サービス起動・watchdog 追加 |
.devcontainer/validate-setup.sh |
基盤構造に合わせたチェックへ整理(ただしローカルモード判定が未対応) |
.devcontainer/entrypoint.sh |
tailscaled 起動オプションを変更 |
.devcontainer/Dockerfile |
ECC ajv 対策と .opencode ディレクトリ整備を追加 |
.devcontainer/docker-compose.yml |
Compose 設定整理、healthcheck 修正 |
.devcontainer/devcontainer.json |
forwardPorts などを基盤モード向けに整理 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
Author
|
Copilot レビューコメントを確認し、今回指摘された8点はすべて修正対象と判断して対応しました。反映コミット: 対応内容:
補足:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 22 out of 22 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
概要
このPRは、opencode-ecc-devcontainer を基盤専用リポジトリとして運用する方針に合わせて、DevContainer設定・起動スクリプト・ドキュメントを整理したものです。
主な変更
コミット分割
動作確認
補足