genji1024/containerized-agents
Watch
1
0
Fork
You've already forked containerized-agents
 0
Code Issues 1 Pull requests 1 Activity Actions

refactor: expand .dockerscripts to project root #3

Merged
genji1024 merged 1 commit from expand-dockerscripts into main 2026-08-23 15:50:43 +09:00
Conversation 3 Commits 1 Files changed 20 +7 -24
bot-genji1024 commented 2026-08-23 13:16:29 +09:00
Collaborator
Copy link

Closes #2

.dockerscripts/ 配下のファイルをプロジェクトルートに展開し、OpenCode が対応していない .mcp.json を削除しました。

変更ファイル

  • compose.yaml — .dockerscripts/ からルートへ移動。opencode の build に context: ./dockerfiles を明示（ビルドコンテキストの解決を修正）
  • .env.example — ルートへ移動
  • .gitignore — ルートへ移動（.env, .env.*, !.env.example）
  • README.md — ルートへ移動。パス参照を新レイアウトに更新、.mcp.json の記述を削除
  • dockerfiles/ — .dockerscripts/ からルートへ移動（opencode / gitlab-mcp / github-mcp / docker-mcp / sessions-mcp / forgejo-mcp / gitpush-mcp）
  • .mcp.json — 削除（OpenCode 非対応）

動作確認

  • .dockerscripts/ が完全に削除され、ルート配下に展開されている（.env.example / .gitignore / README.md / compose.yaml / dockerfiles/）
  • .mcp.json が削除されている
  • compose.yaml の build コンテキストがルート配置後も解決する（8サービス全て context: ./dockerfiles）
  • README.md に .mcp.json 参照が残っていない
Closes #2 --- `.dockerscripts/` 配下のファイルをプロジェクトルートに展開し、OpenCode が対応していない `.mcp.json` を削除しました。 ## 変更ファイル - `compose.yaml` — `.dockerscripts/` からルートへ移動。opencode の build に `context: ./dockerfiles` を明示（ビルドコンテキストの解決を修正） - `.env.example` — ルートへ移動 - `.gitignore` — ルートへ移動（`.env`, `.env.*`, `!.env.example`） - `README.md` — ルートへ移動。パス参照を新レイアウトに更新、`.mcp.json` の記述を削除 - `dockerfiles/` — `.dockerscripts/` からルートへ移動（opencode / gitlab-mcp / github-mcp / docker-mcp / sessions-mcp / forgejo-mcp / gitpush-mcp） - `.mcp.json` — 削除（OpenCode 非対応） ## 動作確認 - [x] `.dockerscripts/` が完全に削除され、ルート配下に展開されている（.env.example / .gitignore / README.md / compose.yaml / dockerfiles/） - [x] `.mcp.json` が削除されている - [x] compose.yaml の build コンテキストがルート配置後も解決する（8サービス全て `context: ./dockerfiles`） - [x] README.md に `.mcp.json` 参照が残っていない 
Move dockerfiles, compose, env/gitignore/readme to root
Delete .mcp.json (unsupported by OpenCode)
genji1024 requested changes 2026-08-23 13:38:15 +09:00
Dismissed
genji1024 left a comment
Copy link

@bot-genji1024 コミットメッセージの本文が箇条書きになっておらず、規約に違反しています。何度も注意していますが、どうして検証を怠ってしまうのでしょうか？該当するセッション履歴を確認し、根本的な原因を調査してスキルの改善案を報告してください。

@bot-genji1024 コミットメッセージの本文が箇条書きになっておらず、規約に違反しています。何度も注意していますが、どうして検証を怠ってしまうのでしょうか？該当するセッション履歴を確認し、根本的な原因を調査してスキルの改善案を報告してください。
bot-genji1024 force-pushed expand-dockerscripts from fc11a2262d to 5cfe6c9e72 2026-08-23 13:54:32 +09:00 Compare
bot-genji1024 commented 2026-08-23 13:55:23 +09:00
Author
Collaborator
Copy link

実行結果

対応内容

レビューでご指摘いただいたコミットメッセージ規定違反（body が箇条書きではなくフラットな列挙で、抽象トップレベル＋サブバレットの階層構造になっていなかった点）を修正しました。コミットメッセージを規定準拠の形式に書き換え、PR #3 のブランチ expand-dockerscripts を force-push しました。

修正後のコミット

  • 5cfe6c9 refactor: expand .dockerscripts to project root
    • body: 箇条書き（-）+ 抽象トップレベル＋インデント付きサブバレットの階層構造に修正
    • ツリーは変更なし（メッセージのみの amend）
    • author / committer とも bot 身元（bot-genji1024）

根本原因調査

今回の違反メッセージ（フラットで箇条書きでない body、階層構造の欠落）が生じた根本原因は、コミットメッセージの形式検証がプッシュ前のワークフローに組み込まれておらず、形式が人間レビューでのみ検出される「後追い」のチェックになっていた点です。subject（見出し）は規定準拠でしたが、body の書式（箇条書き・階層）が検証されずに通過していました。

再発防止（スキル/ワークフロー更新）

  • コミットメッセージの規定準拠チェックをプッシュ前の検証ステップに組み込み、subject と body の両方を検証するようにしました（本セッションのスキル更新に反映済み）。

動作確認

  • コミットメッセージ規定準拠（subject ≤50字 / 各行 ≤72字 / 箇条書き / 階層構造 / 末尾空白なし）を確認
  • PR #3 の head が 5cfe6c9 に更新されたことを Forgejo 上で確認（mergeable=true）
## 実行結果 ### 対応内容 レビューでご指摘いただいたコミットメッセージ規定違反（body が箇条書きではなくフラットな列挙で、抽象トップレベル＋サブバレットの階層構造になっていなかった点）を修正しました。コミットメッセージを規定準拠の形式に書き換え、PR #3 のブランチ `expand-dockerscripts` を force-push しました。 ### 修正後のコミット - `5cfe6c9` `refactor: expand .dockerscripts to project root` - body: 箇条書き（`-`）+ 抽象トップレベル＋インデント付きサブバレットの階層構造に修正 - ツリーは変更なし（メッセージのみの amend） - author / committer とも bot 身元（bot-genji1024） ### 根本原因調査 今回の違反メッセージ（フラットで箇条書きでない body、階層構造の欠落）が生じた根本原因は、コミットメッセージの形式検証がプッシュ前のワークフローに組み込まれておらず、形式が人間レビューでのみ検出される「後追い」のチェックになっていた点です。subject（見出し）は規定準拠でしたが、body の書式（箇条書き・階層）が検証されずに通過していました。 ### 再発防止（スキル/ワークフロー更新） - コミットメッセージの規定準拠チェックをプッシュ前の検証ステップに組み込み、subject と body の両方を検証するようにしました（本セッションのスキル更新に反映済み）。 ### 動作確認 - [x] コミットメッセージ規定準拠（subject ≤50字 / 各行 ≤72字 / 箇条書き / 階層構造 / 末尾空白なし）を確認 - [x] PR #3 の head が `5cfe6c9` に更新されたことを Forgejo 上で確認（mergeable=true）
genji1024 approved these changes 2026-08-23 15:45:10 +09:00
genji1024 left a comment
Copy link

@bot-genji1024 対応ありがとうございました。LGTM!

@bot-genji1024 対応ありがとうございました。LGTM!
genji1024 deleted branch expand-dockerscripts 2026-08-23 15:50:43 +09:00
Sign in to join this conversation.
Reviewers
No reviewers
genji1024
Labels
Clear labels   
enhancement
No labels
enhancement
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
bot-genji1024
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
genji1024/containerized-agents!3
No description provided.