Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .claude/skills/create-pr/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,6 @@ description: GitHub PR作成コマンド

## 注意事項

- ベースブランチは通常 `develop` ですが、確認してください
- PR 本文には必ず PR-Agent のコマンドを含めてください
- 必要に応じて、複数のコミットに分けることも検討してください
- ベースブランチは通常 `develop` ですが、確認してください
- PR 本文には必ず PR-Agent のコマンドを含めてください
- 必要に応じて、複数のコミットに分けることも検討してください
86 changes: 43 additions & 43 deletions .claude/skills/review-pr/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Github MCP を用いて PR の内容を取得し、品質憲章や各種ガイ

## 必須引数

- `$PR_URL`: Github Pull Request URL (例: <https://github.com/offich/sangria/pull/1>)
- `$PR_URL`: Github Pull Request URL (例: <https://github.com/offich/sangria/pull/1>)

## 動作フロー

Expand All @@ -23,54 +23,54 @@ Github MCP を用いて PR の内容を取得し、品質憲章や各種ガイ

### 🔍 基本チェック項目

- **命名規則の遵守**
- 変数・関数・クラス名が意図を明確に表現しているか
- **命名規則の遵守**
- 変数・関数・クラス名が意図を明確に表現しているか
<!-- - ユビキタス言語を使用しているか -->
- 日本語のローマ字変換を避けているか

- **CLEANなコード**
- 凝集度(Cohesion)が高いか
- 疎結合(Loose Coupling)になっているか
- 適切にカプセル化されているか
- 抽象化のレベルが適切か
- 冗長性がないか(DRY原則)
- **関数・クラス設計**
- 単一責任の原則を守っているか
- 引数の数が適切か
- フラグ引数を使っていないか
- 日本語のローマ字変換を避けているか

- **CLEANなコード**
- 凝集度(Cohesion)が高いか
- 疎結合(Loose Coupling)になっているか
- 適切にカプセル化されているか
- 抽象化のレベルが適切か
- 冗長性がないか(DRY原則)
- **関数・クラス設計**
- 単一責任の原則を守っているか
- 引数の数が適切か
- フラグ引数を使っていないか

### 📝 コメント・ドキュメント

- コメントよりコードで表現されているか
- 必要なコメントは背景・理由を説明しているか
- テーブル・カラムにコメントが記載されているか
- コメントよりコードで表現されているか
- 必要なコメントは背景・理由を説明しているか
- テーブル・カラムにコメントが記載されているか

### 🧪 テスト

- テストコードが追加・修正されているか
- F.I.R.S.Tなテストになっているか
- Fast(高速)
- Independent(独立)
- Repeatable(再現性)
- Self-Validating(自己検証)
- Timely(適時性)
- テストコードが追加・修正されているか
- F.I.R.S.Tなテストになっているか
- Fast(高速)
- Independent(独立)
- Repeatable(再現性)
- Self-Validating(自己検証)
- Timely(適時性)

## プロダクト構成とドキュメント

PR の diff 箇所に合わせて、下記を参照してください。

### 📋 ガイドライン

- **レビューガイドライン**: `docs/dev/review-guideline.md`
- **レビューガイドライン**: `docs/dev/review-guideline.md`

## レビューお作法

- Github MCPを通じてレビューを行うこと
- `mcp__github__pull_request_review_write` で pending review を作成
- `mcp__github__add_comment_to_pending_review` を用いてファイル、行指定で該当箇所にコメントを入れていくこと
- 一通りのコメントを終えたタイミングで、 `submit_pending_pull_request_review` を用いてレビュー送信すること
- すでにインラインでPR作成車によってコメントがされている箇所がある場合、そのコメントに対して反応すること
- いずれの場合も、「Claude Code によるコメント」である旨を明記すること
- Github MCPを通じてレビューを行うこと
- `mcp__github__pull_request_review_write` で pending review を作成
- `mcp__github__add_comment_to_pending_review` を用いてファイル、行指定で該当箇所にコメントを入れていくこと
- 一通りのコメントを終えたタイミングで、 `submit_pending_pull_request_review` を用いてレビュー送信すること
- すでにインラインでPR作成車によってコメントがされている箇所がある場合、そのコメントに対して反応すること
- いずれの場合も、「Claude Code によるコメント」である旨を明記すること

### `submit_pending_pull_request_review` 時のテンプレート

Expand All @@ -94,10 +94,10 @@ PR の diff 箇所に合わせて、下記を参照してください。

## エラーハンドリング

- **PR が存在しない場合**: エラーメッセージを表示し、PR番号の確認を促す
- **アクセス権限がない場合**: 権限設定の確認を促す
- **ネットワークエラー**: リトライまたは手動でのレビューを促す
- **差分が大きすぎる場合**: 重要な変更に絞ってレビューを実施
- **PR が存在しない場合**: エラーメッセージを表示し、PR番号の確認を促す
- **アクセス権限がない場合**: 権限設定の確認を促す
- **ネットワークエラー**: リトライまたは手動でのレビューを促す
- **差分が大きすぎる場合**: 重要な変更に絞ってレビューを実施

## 使用例

Expand All @@ -107,13 +107,13 @@ PR の diff 箇所に合わせて、下記を参照してください。
/review:review-pr https://github.com/offich/sangria/pull/10
```

- PRの変更内容を取得
- コーディングガイドラインに基づいてレビュー
- 改善点をGitHubにコメント投稿
- PRの変更内容を取得
- コーディングガイドラインに基づいてレビュー
- 改善点をGitHubにコメント投稿

## 注意事項

- レビューは自動生成の提案であり、最終的な判断は人間が行うこと
- ビジネスロジックの妥当性は判断できないため、技術的な観点のみレビュー
- 大規模なPRの場合、重要度の高い指摘に絞ってコメント
- 既存コードのスタイルに合わせることを優先
- レビューは自動生成の提案であり、最終的な判断は人間が行うこと
- ビジネスロジックの妥当性は判断できないため、技術的な観点のみレビュー
- 大規模なPRの場合、重要度の高い指摘に絞ってコメント
- 既存コードのスタイルに合わせることを優先
2 changes: 1 addition & 1 deletion .github/workflows/docs-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
- uses: actions/cache@27d5ce7f107fe9357f9df03efb73ab90386fccae # v5.0.5
id: pnpm-cache
with:
path: ${{ steps.pnpm-store.outputs.path }}
path: './node_modules'
key: pnpm-${{ hashFiles('pnpm-lock.yaml') }}

- name: install dependencies
Expand Down
3 changes: 0 additions & 3 deletions .markdownlint-cli2.yaml

This file was deleted.

14 changes: 14 additions & 0 deletions .markdownlint.jsonc
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
// Unordered list indentation
"MD007": {
"start_indented": true
},
// Dollar signs used before commands without showing output
"MD013": false,
// Line Length
"MD014": false,
// Multiple headings with the same content
"MD024": false,
// First line in a file should be a top-level heading
"MD041": false
}
24 changes: 12 additions & 12 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,36 +2,36 @@

## 0.3.0

* Add the following lint:
* `use_widget_ref_synchronously`
* Add the following lint:
* `use_widget_ref_synchronously`

## 0.2.0

* Added the following lint:
* `no_disabled_tests`
* Added the following lint:
* `no_disabled_tests`

## 0.1.0

* Added the following lint:
* `avoid_empty_container`
* Added the following lint:
* `avoid_empty_container`

## 0.0.3

* Publish with example directory included.
* Publish with example directory included.

## 0.0.2

* Improve score on pub.dev
* Fix automatic deployment by GitHub Actions
* Improve score on pub.dev
* Fix automatic deployment by GitHub Actions

## 0.0.1

* Modify README.md
* Modify README.md

## 0.0.1-dev.2

* Downgraded required dart sdk version to 3.6.2.
* Downgraded required dart sdk version to 3.6.2.

## 0.0.1-dev.1

* Initial release. This package is under construction.
* Initial release. This package is under construction.
Loading