Skip to content

docs: add accessibility rules and refine coding-style, testing, and AGENTS guidelines#3529

Merged
KATO-Hiro merged 2 commits into
stagingfrom
#3528
May 13, 2026
Merged

docs: add accessibility rules and refine coding-style, testing, and AGENTS guidelines#3529
KATO-Hiro merged 2 commits into
stagingfrom
#3528

Conversation

@KATO-Hiro

@KATO-Hiro KATO-Hiro commented May 13, 2026

Copy link
Copy Markdown
Collaborator

close #3528

Summary by CodeRabbit

  • ドキュメンテーション
    • UIコンポーネント向けのアクセシビリティガイドラインを追加
    • コーディングスタイルドキュメンテーション要件を更新
    • テスト戦略とTDD例外ガイダンスを改善
    • 開発ワークフロー指示を洗練

…GENTS guidelines

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@KATO-Hiro has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 3 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 77f51855-d939-4c5c-b5a4-6d6a56b5178c

📥 Commits

Reviewing files that changed from the base of the PR and between 5416a8a and 65b23bd.

📒 Files selected for processing (2)
  • .claude/rules/accessibility.md
  • .claude/rules/coding-style.md
📝 Walkthrough

Walkthrough

開発ガイドラインとエージェント指示書を更新。アクセシビリティ規則(新規)、TSDoc ガイダンス、テスト駆動開発の例外処理、カバレッジ目標、ライティングプランの優先順付けを整備。コード変更なし。

Changes

アクセシビリティとコーディング規則の整備

Layer / File(s) Summary
アクセシビリティ規則(新規)
.claude/rules/accessibility.md
テーブルヘッダースコープ、色の利用制限、ボタン/リンクのアクセシブル名、モーダルフォーカス管理、Flowbite Svelte ガイダンス、フォームラベル要件を定義。
開発実践ガイドラインの更新
.claude/rules/coding-style.md, .claude/rules/testing.md, AGENTS.md
  • TSDoc:非自明な場合のみ記載(自明なら省略)
  • テスト:TDD 例外を明記(スパイク・設定のみは後付け可)、カバレッジ目標を数値から「意味のあるドメイン境界」に変更
  • エージェント計画:リスク順序(低→高)を明示、/session-close を機能/リファクタブランチのみに限定

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

📋 ガイドラインを整え、
アクセスを考え、テストを賢く、
リスク順に計画し—
開発の道しるべ、今ここに完成。 ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRのタイトルはアクセシビリティルールの追加とcoding-style、testing、AGENTSガイドラインの洗練という実際の変更内容を正確に反映している。
Linked Issues check ✅ Passed リンク済みissue #3528の「AGENTS.mdとrulesを更新する」という要件に対し、本PRは4つのドキュメント(accessibility.md、coding-style.md、testing.md、AGENTS.md)を追加・更新して要件を満たしている。
Out of Scope Changes check ✅ Passed すべての変更は4つのガイドラインドキュメント(ルールとエージェント指示)に限定され、リンク済みissue #3528のスコープ内である
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch #3528

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.claude/rules/accessibility.md:
- Around line 38-40: 修正対象の記述は Button の挙動誤記で、`color` 未指定が `aria-label`
を剥奪するかのように書かれている点を直してください。具体的には `.claude/rules/accessibility.md`
の該当箇所を「`icon-only` バリアントでは `aria-label`
はコンポーネントが自動生成せずユーザーが明示指定する必要がある(必須)」「`color` を省略しても Button はデフォルトで 'primary'
を適用するため、`color` の省略が `aria-label` に影響を与えるわけではない」といった正確な表現に書き換え、参照する識別子として
Button、aria-label、color、icon-only variants を明記してください。

In @.claude/rules/coding-style.md:
- Line 35: ファイル内の TSDoc 方針が矛盾しているのでどちらかに統一してください:TSDoc
を「非自明な場合のみ必要」とするルール(現在の箇所にある “TSDoc: add when behavior…” の文言)か、もう一方の「全 export
に対して必須」とするルールのどちらか一方に確定し、もう一方の箇所をその選択に合わせて差し替えてください(該当するテキストはファイル中の “TSDoc: add
when behavior, constraints, or params…” と “全 export 必須”
を含む文言です)。併せてファイル内の同様の記述がないか確認して一貫した表現に統一し、コミットメッセージでどちらの方針を採用したか明記してください。
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 6fd24dad-dd70-4b19-9a68-8128a44ccf74

📥 Commits

Reviewing files that changed from the base of the PR and between 7c03c8d and 5416a8a.

📒 Files selected for processing (4)
  • .claude/rules/accessibility.md
  • .claude/rules/coding-style.md
  • .claude/rules/testing.md
  • AGENTS.md

Comment thread .claude/rules/accessibility.md Outdated
Comment thread .claude/rules/coding-style.md
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@KATO-Hiro KATO-Hiro left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@KATO-Hiro KATO-Hiro merged commit c0f9155 into staging May 13, 2026
3 checks passed
@KATO-Hiro KATO-Hiro deleted the #3528 branch May 13, 2026 22:13
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.

chore: Update AGENTS.md and rules

1 participant