Skip to content

chore: チーム開発の標準ファイルを足す - #1

Merged
Nicolas0315 merged 6 commits into
mainfrom
chore/repo-standard-20260924
Sep 30, 2026
Merged

Nicolas0315 merged 6 commits into
mainfrom
chore/repo-standard-20260924

Conversation

@Nicolas0315

@Nicolas0315 Nicolas0315 commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

目的

repo-doctor が見つけた標準の穴のうち、ファイルを足せば埋まるものをまとめて入れる。既にあるファイルで変えたのは README.md の適用範囲の表だけ。

足したもの

  • CODEOWNERS — 誰がレビューするかの既定
  • .github/pull_request_template.md — PR に「何を・なぜ・どう確かめたか」を書く欄
  • AGENTS.md — AI と新しいメンバーが最初に読む規約(Scope・公開リポの制約・反映のされ方)
  • README.md(変更)— org 全体へ継承されるファイルの表に PR テンプレートを足した

CI について

この PR で workflow の追加はない。main の docs チェックが README と SECURITY の存在・リンクを検証する。public リポなので、検査ツールの具体的な設定は追加しない。

このPRで直らないもの(別の作業)

  • branch-rules

確かめ方

  • この PR の docs チェックが通ること
  • merge 後に repo-doctor CyberticalAI/.github で該当項目が pass になること

🤖 Generated with Claude Code


Note

Low Risk
Markdown-only and GitHub metadata changes; no runtime code, secrets, or new CI workflows in this diff.

Overview
Adds org-wide default community health files to the public .github meta repo so repos without their own copies inherit them on merge to main.

New: .github/pull_request_template.md (Japanese sections for what/why/how verified plus review focus), CODEOWNERS defaulting all paths to @Nicolas0315, and AGENTS.md describing scope, which files inherit vs stay repo-local, public-repo editing constraints, and how to verify (docs CI + post-merge PR template smoke test).

README: Documents the PR template in the inheritance table, links GitHub’s default community health file docs, drops the explicit “private を含む” wording for which org repos are covered, and notes that per-repo .github/pull_request_template.md overrides the default like SECURITY.md.

Reviewed by Cursor Bugbot for commit b484617. Bugbot is set up for automated code reviews on this repo. Configure here.

- CODEOWNERS
- .github/pull_request_template.md
- .pre-commit-config.yaml
- AGENTS.md

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 24, 2026 05:35
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-24T05:40:51.967477Z 5775b67 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Address the pre-commit configuration concerns and replace the AGENTS.md placeholders.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 Medium severity · 1 Low severity

Open (3)
What changed in this PR

Adds standard collaboration files to the .github repository.

Changes:

  • Adds CODEOWNERS and a pull request template.
  • Adds local Gitleaks and Semgrep hooks.
  • Adds an AGENTS.md guidance draft.
File Summary
CODEOWNERS Defines the default reviewer.
AGENTS.md Adds guidance structure but retains placeholder scope text.
.pre-commit-config.yaml Adds security hooks; staging and public configuration need revision.
.github/​pull_request_template.md Standardizes pull request descriptions and verification notes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .pre-commit-config.yaml Outdated
Comment thread .pre-commit-config.yaml Outdated
Comment thread AGENTS.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5775b67549

ℹ️ 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".

Comment thread .pre-commit-config.yaml Outdated
Comment thread AGENTS.md Outdated
Comment thread .github/pull_request_template.md
README の「稼働中の検査構成やセキュリティツールの具体的な設定を書かない」に反するので
.pre-commit-config.yaml を外す。AGENTS.md のプレースホルダーを埋め、org 全体へ継承される
PR テンプレートを README の適用範囲の表に足した。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI previously approved these changes Sep 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread AGENTS.md Outdated
Copilot AI lite review requested due to automatic review settings September 30, 2026 04:43
@cursor

cursor Bot commented Sep 30, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_27d19800-402d-4c8a-b60b-bb09cd30684c)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Unresolved critical template placement and moderate scope issues remain.

Review effort: Lite
Findings: 1 High severity · 1 Low severity

Open (2)

Comment thread .github/pull_request_template.md
@Nicolas0315
Nicolas0315 requested a lite review from Copilot September 30, 2026 04:53
@cursor

cursor Bot commented Sep 30, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_8e65289d-a32c-4b4a-9bf3-23a10a5bfbc2)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

Correct inheritance claims and revise the documented CI configuration details.

Review effort: Lite
Findings: None

Resolved since last review (2)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Address the template’s secret-disclosure wording and clarify the README’s public-repository scope.

Review effort: Lite
Findings: 1 Low severity

Open (1)

Comment thread README.md Outdated
Copilot AI lite review requested due to automatic review settings September 30, 2026 05:04
@cursor

cursor Bot commented Sep 30, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_d90ab1ff-7697-47e7-a6e8-fbb22318b498)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Clarify in README.md and AGENTS.md that default files apply to public repositories.

Review effort: Lite
Findings: 1 Low severity

Open (1)
Resolved since last review (1)
Previously missed (1)

In code that hasn't changed since last review

Low severity 適用対象をpublicリポジトリに限定する

AGENTS.md:29

この記述も default community health files が org 内の全リポジトリに適用される前提になっていますが、GitHub が保証する対象は public リポジトリです。private/internal リポジトリまで受入確認の対象に見えないよう、適用先を public リポジトリに限定してください。

Comment thread README.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approved

変更は低リスクで整合していますが、PR 本文の CI 名には軽微な修正が必要です。

Review effort: Balanced
Findings: 2 Low severity

Open (2)

Comment thread AGENTS.md
@Nicolas0315
Nicolas0315 merged commit fce51fa into main Sep 30, 2026
4 checks passed
@Nicolas0315
Nicolas0315 deleted the chore/repo-standard-20260924 branch September 30, 2026 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants