diff --git a/.claude/commands/git.md b/.claude/commands/git.md index b766096e..db3c0ce9 100644 --- a/.claude/commands/git.md +++ b/.claude/commands/git.md @@ -17,7 +17,7 @@ description: Git 操作ルール - `git status` で差分を確認 - 以下のコミット粒度に基づいてファイルを順次ステージングしてからコミット: - パッケージ単位でコミットを分割 - - 依存元を先にコミット（依存順序が不明な場合は `npx lerna list --graph` で確認） + - 依存元を先にコミット（依存順序が不明な場合は `yarn lerna list --graph` で確認） - **各コミット後:** - **重要: 自動的に次のコミットに進まない** - **重要: 次に何をすべきか推測しない** @@ -39,7 +39,7 @@ description: Git 操作ルール - Conventional Commits を使用すること - 使用するタイプ: `feat`, `fix`, `docs`, `refactor`, `test`, `chore` - 使用するスコープ: - - 各パッケージ名（ネームスペースなし） + - 各パッケージ名（ネームスペースなし）: `a11y-check`, `a11y-check-core`, `a11y-check-axe-scenario`, `a11y-check-scenarios`, `archaeologist`, `backlog-projects`, `beholder`, `cli-core`, `dealer`, `filematch`, `fs`, `google-auth`, `google-sheets`, `html-distiller`, `notion`, `print`, `proc-talk`, `puppeteer-dealer`, `puppeteer-general-actions`, `puppeteer-page-scan`, `puppeteer-screenshot`, `puppeteer-scroll`, `readtext`, `remote-inspector`, `replicator`, `roar`, `shared` - `repo`, `deps`, `github` - メッセージ本文の各行は100文字以下 - 件名は sentence-case, start-case, pascal-case, upper-case にしない diff --git a/.claude/commands/pr.md b/.claude/commands/pr.md index fc41f634..05a46ece 100644 --- a/.claude/commands/pr.md +++ b/.claude/commands/pr.md @@ -8,3 +8,7 @@ description: プルリクエストの作成とプッシュ - 全てがパスしなければならない。失敗があれば続行前に修正する。 3. 適切な `git` コマンドを使って現在のトピックブランチの変更をレビューする。 4. `gh pr create` コマンドのワンライナーを生成・実行してプルリクエストを作成する。 +5. **CI 監視:** + - PR 作成直後に `gh pr checks --watch` で CI の完了を待機する。 + - テストが途中で失敗した場合は完了を待たずに修正作業に戻る。 + - 全テストが通りマージ可能になったらユーザーに報告する。 diff --git a/.claude/commands/refactor.md b/.claude/commands/refactor.md deleted file mode 100644 index a11eac8c..00000000 --- a/.claude/commands/refactor.md +++ /dev/null @@ -1,46 +0,0 @@ ---- -description: テストファースト・リファクタリングワークフロー ---- - -# リファクタリングワークフロー - -リファクタリングを求められた場合、以下の厳密な順序に従うこと: - -## フェーズ 1: テスト検証（リファクタリング前） - -1. **既存テストの確認** - - リファクタリング対象コードをカバーするテストを特定する - - 関連テストを実行し、現在の動作を確認する - -2. **不足テストの作成（必要な場合）** - - リファクタリング範囲をカバーするテストがない場合、先にテストを書く - - テストはリファクタリング後も維持されるべき期待動作を検証すること - - 新しいテストが現在の実装でパスすることを確認する - -3. **テストを先にコミット** - - `/git` コマンドでテスト変更をコミットする - - `.claude/commands/git.md` の全ルールに従うこと - - コミットメッセージ例: `test(package): add tests for X before refactoring` - -## フェーズ 2: リファクタリング実装（テストコミット後） - -4. **リファクタリング開始** - - テストがコミットされた後にのみリファクタリングを開始する - - 段階的に変更する - - 動作が維持されていることを確認するため、頻繁にテストを実行する - -5. **リファクタリング検証** - - 関連テストを全て実行する: `yarn test` - - ビルドチェックを実行する: `yarn build` - - lint を実行する: `yarn lint` - -6. **リファクタリングをコミット** - - `/git` コマンドでリファクタリング変更をコミットする - - コミットメッセージ例: `refactor(package): simplify X implementation` - -## 基本原則 - -- **テストなしでリファクタリングしない**: テストが安全ネット -- **テストが先、リファクタリングが後**: 必ずテストをコミットしてからリファクタリング -- **動作を維持する**: リファクタリングは外部動作を変えてはならない -- **小さな増分**: 小さく検証可能な変更を行う diff --git a/.claude/settings.json b/.claude/settings.json new file mode 100644 index 00000000..ce162fab --- /dev/null +++ b/.claude/settings.json @@ -0,0 +1,18 @@ +{ + "permissions": { + "deny": [ + "Read(**/.env)", + "Read(**/.env.*)", + "Edit(**/.env)", + "Edit(**/.env.*)", + "Write(**/.env)", + "Write(**/.env.*)", + "Read(**/credentials.json)", + "Read(**/token.json)", + "Edit(**/credentials.json)", + "Edit(**/token.json)", + "Write(**/credentials.json)", + "Write(**/token.json)" + ] + } +} diff --git a/.claude/skills/monorepo-architect/SKILL.md b/.claude/skills/monorepo-architect/SKILL.md deleted file mode 100644 index 5b30c647..00000000 --- a/.claude/skills/monorepo-architect/SKILL.md +++ /dev/null @@ -1,30 +0,0 @@ ---- -name: monorepo-architect-skill -metadata: - internal: true -description: 'Node.jsモノレポ環境でのメンテナンス、機能追加、コード設計を統括します。プロジェクト固有のアーキテクチャ・ルールを遵守させたい時に使用します。' ---- - -# **Monorepo Architect Skill（簡略版）** - -このスキルは、Node.jsモノレポ開発における基本的な整合性と保守性を維持するための思考プロトコルです。 - -## **1. コマンド制約（Yarn 限定）** - -AIエージェントは、以下のコマンド制約を例外なく守らなければなりません。 - -- **yarn のみ使用**: パッケージのインストール、ビルド、スクリプト実行には必ず yarn を使用してください。npm の使用はあらゆる状況で**厳禁**です。 -- **cd 禁止**: 作業ディレクトリを移動（cd）しないでください。常にルートディレクトリを起点とし、`yarn workspace ` または `--cwd` フラグを使用して操作を完結させてください。 -- **対象限定の操作**: ビルドや検証は、常に影響を受けるパッケージに限定して行います（例: `yarn build --scope `）。 -- **テスト実行**: ユニットテストの実行には `npx vitest run` を使用してください。 - -## **2. `package.json` の `exports` フィールド管理** - -- **Strict Exports**: `package.json` の `exports` フィールドを直接編集し、外部に公開する API を厳選して定義してください。一括公開は避けてください。 -- **exports を壊さない**: `package.json` の `exports` を更新する際は、既存の定義を削除したり壊したりしないよう、差分のみを正確に追記してください。 - -## **3. AI 操作プロトコル** - -1. **修正前にスキャン（Scan Before Action）**: 修正を開始する前に、必ずリポジトリのパッケージ構造、依存関係、`exports` の現状を確認してください。 -2. **exports の保全**: `package.json` の `exports` フィールドを変更する場合は差分のみ追記し、既存のエクスポートパスを削除しない。 -3. **アーキテクチャの確認**: 変更がプロジェクトのディレクトリ構造ルールやモジュール設計に違反していないか、常にセルフチェックを行ってください。 diff --git a/CLAUDE.md b/CLAUDE.md index 6ca16387..093bde14 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -1,92 +1,78 @@ -# Claude Code Rules for @d-zero-dev/tools +# CLAUDE.md -## Repository Structure +This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository. -- This is a **monorepo** managed by Lerna -- **Always work from the repository root** (not individual package directories) -- Volta manages Node.js and Yarn versions automatically via `package.json` volta section +## 概要 -## Development Workflow +D-ZERO 株式会社の Web 開発・テスト・自動化ツール群（`@d-zero-dev/tools`）。Lerna + Yarn Workspaces のモノレポ構成で、`@d-zero/` スコープ配下に多数のライブラリ／CLI を提供する（**independent バージョンモード**）。 -### Build and Test Commands +## プロジェクト構成 -- Use `yarn build` to build all packages via Lerna -- Use `yarn test` to run all tests via Vitest -- Use `yarn lint` to check for ESLint errors -- Use `yarn` (not `npm`) for all package management +作業前に以下のファイルを確認し、プロジェクトの状態を把握すること: -### Quality Assurance Before Commit +- `package.json` — scripts、devDependencies、Volta（Node 24 / Yarn 4） +- `lerna.json` — **independent バージョンモード**、`packages/@d-zero/*` +- `ARCHITECTURE.md` — パッケージ一覧、レイヤ構成、設計方針、主要な技術スタック +- `README.md` — リポジトリ概要 +- `tsconfig.json` — TypeScript 設定 +- 各パッケージの構成は `packages/@d-zero/*/package.json` を参照 -- **Always run lint before commit**: Use `yarn lint` to check for ESLint errors and fix them before committing -- **Resolve ALL lint errors AND warnings immediately**: Warnings are not acceptable — treat them as errors and fix them before committing. Never leave any lint warning unresolved, even if it seems harmless -- **Ask user permission before disabling ESLint rules**: Always ask the user before using eslint-disable-next-line or eslint-disable -- **Prefer code fixes over disabling**: Change code structure rather than disabling rules when possible -- **Justify disable usage**: When user approves disabling, explain why it's necessary and what alternatives were considered -- Test build and test commands after making changes -- Verify import paths are correct for the package export strategy -- Never commit without successful build and test results +## コマンド -### Commit Guidelines +- `yarn build` — 全パッケージビルド（`lerna run build`） +- `yarn watch` — `lerna run watch --parallel` +- `yarn test` — Vitest によるテスト実行 +- `yarn lint` — eslint / prettier / textlint / cspell を直列実行 +- `yarn release` / `yarn release:alpha` 等 — `lerna version`（independent モード）→ `release:trigger` で `git push origin main --follow-tags` と `v-release` タグ強制更新を行う。リリース手順は `/release` コマンド参照 -- **Separate commit scope**: Don't mix implementation changes with documentation/rule updates in same commit -- Complete all related changes in a feature (don't leave tasks incomplete) +### コマンド制約 -## Code Architecture +- **yarn のみ使用**: npm / pnpm / bun / deno によるコマンド実行は禁止 +- **パッケージディレクトリに cd しない**: 常にリポジトリルートから実行 +- **ビルドは `yarn build` のみ**: `npx tsc`、`lerna run build --scope` 等の個別指定は禁止 +- **コマンドの連続実行禁止**: `&&`、`;`、改行によるコマンド連結をしない。1回の Bash 呼び出しで1コマンドのみ実行する。連結されたコマンドは settings.json の permissions allow/deny でパターンマッチできず、毎回ユーザーの手動承認が必要になり効率が大幅に低下する -### Package Rules +## アーキテクチャ -- **@d-zero/shared**: Uses individual export paths (e.g., `@d-zero/shared/config-reader`) -- **Never modify shared exports strategy** without explicit permission -- Respect existing export patterns in package.json +詳細は [`ARCHITECTURE.md`](../ARCHITECTURE.md) を参照。各パッケージのカテゴリ・依存関係・設計方針はそちらに集約されている。 -### Import/Export Guidelines +ドキュメントと実装に矛盾がある場合は、**実装が正**とし、ドキュメントを修正すること。 -- Use specific import paths for `@d-zero/shared` (e.g., `/config-reader`) -- Check existing usage patterns before adding new dependencies -- Maintain backward compatibility with existing exports +## independent モードの注意 -### CLI Patterns +- 各パッケージは個別のバージョンを持つ。`lerna version` は変更のあったパッケージのみ昇格させる +- リリース時は `v-release` タグの強制更新を伴う（`publish.yml` のトリガー） +- 単一パッケージの破壊的変更でもリポジトリ全体の major bump にはならない -- Use `@d-zero/cli-core` for common CLI functionality -- Follow established patterns for argument parsing and validation -- Maintain consistency across CLI tools +## コーディング規約 -## Code Development +- **`@d-zero/shared` はサブパスエクスポート**: `@d-zero/shared/parse-url` のような形式で import すること（ルート import は不可） +- **exports フィールドを壊さない**: package.json の `exports` を変更する場合は差分のみを追記し、既存パスを削除しない +- 1関数1ファイルを基本とし、`index.ts` の作成は避ける -### General Principles +## セキュリティ -- **Always read files first** before making any edits -- **Never make assumptions** about code structure or dependencies -- **Ask for permission** before making architectural changes +### 機密情報の取り扱い -### Error Handling Guidelines +- `.env`、`.env.*` 等の機密ファイルを読み取り・編集・コミットしない（機密ファイルの判断は `.gitignore` を参考にすること） +- `credentials.json`、`token.json` 等の Google API 認証情報も同様に扱う +- コミット前に `git diff --staged` で機密情報（API キー、トークン、パスワード、企業名、顧客情報）が含まれていないか確認する +- 環境変数やシークレットをコード内にハードコードしない -- **Never suppress errors**: Don't use empty catch blocks or generic error swallowing -- **Specific error handling**: Handle specific error types with appropriate recovery strategies -- **Preserve error context**: Always maintain original error information when re-throwing -- **Log before throwing**: Use proper logging for debugging while preserving error propagation -- **Fail fast**: Let unhandled errors bubble up rather than attempting to continue in invalid state -- **Prefer .catch() for single awaits**: Use `.catch()` method instead of try-catch for single await expressions -- **Don't catch just to rethrow**: If you're only going to throw again, let the original error bubble up naturally -- **Avoid massive try blocks**: Break down large try blocks into smaller, more focused error handling +### サプライチェーン保護 -### Testing Guidelines +- **yarn dlx は完全禁止**: ローカルパッケージを使わずリモートから直接実行するため、サプライチェーン攻撃に脆弱 +- **npx は原則使わない**: package.json の scripts で定義されたコマンドを `yarn