Skip to content

chore(biome): .claude の除外をリポジトリ直下だけにする - #1122

Merged
owk-owk130 merged 1 commit into
developfrom
fix/biome-claude-ignore
Sep 2, 2026
Merged

chore(biome): .claude の除外をリポジトリ直下だけにする#1122
owk-owk130 merged 1 commit into
developfrom
fix/biome-claude-ignore

Conversation

@owk-owk130

Copy link
Copy Markdown
Contributor

Summary

  • biome.jsonfiles.includes にある !**/.claude!.claude に変更する

主な変更内容

!**/.claude は絶対パスの祖先ディレクトリにもマッチするため、.claude/worktrees/ 配下に作った git worktree では biome check .pnpm lint の先頭)がディレクトリ走査をそこで打ち切り、「Checked 0 files / No files were processed」で失敗していた。ファイルを明示して渡す post-edit-lint.sh は動くため気付きにくく、pnpm lint だけが worktree 内で通らない状態だった。

config ルート相対の !.claude に変えると、除外されるのはリポジトリ直下の .claude/ だけになる。

確認したこと:

  • 通常のチェックアウトで biome check . の処理数は変更前後とも 731 ファイルで変わらない
  • biome check .claude は変更後も 0 ファイル(除外は維持)
  • .claude/worktrees/ 配下の既存 worktree で biome check . が 0 → 719 ファイルになる
  • リポジトリ内に .claude という名前のディレクトリはルート以外に無い

Test plan

  • CI の biome check . が通る
  • .claude/worktrees/ 配下の worktree で pnpm lint が通る

`!**/.claude` は worktree の絶対パスに含まれる `.claude/worktrees/` にも
マッチし、`.claude/worktrees/` 配下の worktree で `biome check .` が
0 ファイルになっていた。ファイルを明示する post-edit hook は動くため
気付きにくく、`pnpm lint` だけが worktree 内で通らない状態だった。
config ルート相対の `!.claude` に変えて、リポジトリ直下の `.claude/`
だけを除外する。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AYsHuZj1hWRdK1hieBG5YU
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for server

Status Category Percentage Covered / Total
🟢 Lines 98.19% (🎯 97%) 3419 / 3482
🟢 Statements 97.6% (🎯 96%) 3580 / 3668
🟢 Functions 98.59% (🎯 97%) 630 / 639
🟢 Branches 89.42% (🎯 87%) 1403 / 1569
File CoverageNo changed files found.
Generated in workflow #2118 for commit 0be7f7e by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for web

Status Category Percentage Covered / Total
🟢 Lines 97.41% (🎯 96%) 1658 / 1702
🟢 Statements 96.28% (🎯 95%) 1764 / 1832
🟢 Functions 96.08% (🎯 95%) 711 / 740
🟢 Branches 89.93% (🎯 88%) 1117 / 1242
File CoverageNo changed files found.
Generated in workflow #2118 for commit 0be7f7e by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for shared

Status Category Percentage Covered / Total
🟢 Lines 99.15% (🎯 98%) 235 / 237
🟢 Statements 80.96% (🎯 80%) 251 / 310
🟢 Functions 100% (🎯 98%) 79 / 79
🟢 Branches 62.87% (🎯 62%) 105 / 167
File CoverageNo changed files found.
Generated in workflow #2118 for commit 0be7f7e by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Coverage Report for widget

Status Category Percentage Covered / Total
🟢 Lines 99.62% (🎯 98%) 265 / 266
🟢 Statements 97.94% (🎯 96%) 286 / 292
🟢 Functions 98.11% (🎯 98%) 52 / 53
🟢 Branches 93.28% (🎯 92%) 139 / 149
File CoverageNo changed files found.
Generated in workflow #2118 for commit 0be7f7e by the Vitest Coverage Report Action

@owk-owk130
owk-owk130 merged commit e8c8123 into develop Sep 2, 2026
2 checks passed
@github-actions github-actions Bot mentioned this pull request Sep 2, 2026
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.

1 participant