Skip to content

CLAUDE.md のバージョン参照化 + Requires at least を 6.6 に更新(#120 対応) - #127

Merged
fumikito merged 2 commits into
mainfrom
fix/claude-md-version-refs
Jul 7, 2026
Merged

CLAUDE.md のバージョン参照化 + Requires at least を 6.6 に更新(#120 対応)#127
fumikito merged 2 commits into
mainfrom
fix/claude-md-version-refs

Conversation

@fumikito

@fumikito fumikito commented Jul 7, 2026

Copy link
Copy Markdown
Member

概要

#120 の推奨アクションを一括対応。CLAUDE.md のバージョン番号ハードコードを排除し、あわせて Requires at least を実態(CI がテストする WP 6.6)に揃えました。

背景

Issue #120 の指摘どおり、.claude/CLAUDE.md の記述(PHP 7.4/8.0 × WP latest/5.9PHP 7.0+ 互換)が実態と乖離していました。実態は #111Closes #75)で PHP 8.1 に引き上げ済み(composer >=8.1、test.yml 8.1/8.3、ヘッダー Requires PHP: 8.1)。

根本原因は「ドキュメント/メタデータにバージョン番号を重複して持っていること」なので、CLAUDE.md は情報源を参照する方式に切り替え、Requires at least は実態に合わせました。

Single Source of Truth の方針

  • 対応 PHP: composer.jsonrequire.php / config.platform.php)が正。rich-taxonomy.phpRequires PHP はこれに追従
  • Node: .node-version
  • CI テストマトリックス(PHP × WP): .github/workflows/test.yml

変更内容

① CLAUDE.md の CI マトリックス記述(#120 推奨1)

PHPUnit(PHP 7.4/8.0 × WP latest/5.9) → マトリックスは test.yml 参照 + 情報源を明示

② CLAUDE.md のコーディング規約(#120 推奨2)

PHP 7.0+ 互換(ただし CI は 7.4+ でテスト) → composer.json / test.yml 参照

③ Requires at least を 6.6 に更新(#120 推奨3)

  • rich-taxonomy.php: Requires at least: 5.96.6(CI がテストする WP 最低バージョンに整合)
  • README.md: Requires at least: 6.6 を新規追加(従来この行が欠落しており、readme.txt 生成時に WordPress.org へ最低 WP バージョンが表示されない状態だった)

補足: 6.6 は「6.6 未満で動かない」ではなく「6.6 未満は CI 未検証のため保証しない」という宣言。将来より低い WP を保証したい場合は test.yml のマトリックスに下限を追加して検証する運用とする。

Fixes #120

🤖 Generated with Claude Code

@fumikito
fumikito enabled auto-merge July 7, 2026 07:24
@claude

claude Bot commented Jul 7, 2026

Copy link
Copy Markdown

AI トリアージ: PR #127 | リスク 🟢 低(ドキュメントのみ) | スコープ: scope:docs | 不足: なし | サマリー: CLAUDE.md のバージョンハードコードを排除し test.yml と composer.json を唯一の情報源として参照する記述に変更。ドキュメント専用の変更のためコードへの影響なし。テスト要否: テスト不要。設計上の懸念: なし(CI との乖離を構造的に防ぐ改善)。🤖 AI トリアージ by claude-code

CI テストマトリックスや対応 PHP バージョンをハードコードすると実態と乖離するため、
唯一の情報源である test.yml / composer.json を参照する記述に変更。

Fixes #120

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito
fumikito force-pushed the fix/claude-md-version-refs branch from 53ca084 to 915cb50 Compare July 7, 2026 07:37
CI(test.yml)がテストするWP最低バージョン(6.6)に Requires at least を揃える。
README.md には Requires at least 行が欠落していたため追加し、readme.txt 生成時に
WordPress.org へ最低WPバージョンが正しく表示されるようにする。

Refs #120

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito
fumikito merged commit 1e333ea into main Jul 7, 2026
10 checks passed
@fumikito
fumikito deleted the fix/claude-md-version-refs branch July 7, 2026 07:40
@claude

claude Bot commented Jul 7, 2026

Copy link
Copy Markdown

AI トリアージ(差分レビュー): PR #127

リスク: 🟢 低(メタデータ更新のみ)
スコープ: scope:docs scope:config
不足: なし

サマリー

  • 前回レビュー対象(CLAUDE.md)の変更は ✅ 解消済み
  • 今回の追加コミットで Requires at least5.96.6 に引き上げ(rich-taxonomy.php + README.md
  • WP 5.9–6.5 ユーザーへの自動更新ブロックが発生する宣言的変更であることを確認済み
詳細レビュー

テスト要否

  • ⚪ テスト不要: rich-taxonomy.php / README.md — プラグインヘッダーとドキュメントのみ、ロジック変更なし

レビュワーが確認すべき箇所

  1. rich-taxonomy.php:8Requires at least: 6.6 への引き上げは「CI でテストしている最低 WP バージョン」への追従であり技術的に妥当。ただし WP 5.9–6.5 上の既存ユーザーへは WordPress.org からの自動更新がブロックされる(更新前に互換性警告が表示される)。意図的な判断として理解しているが、リリースノートでの言及を推奨。

設計上の懸念

なし。CI マトリックスの最低 WP バージョンと Requires at least を揃える方針は一貫性観点で正しい。


🤖 AI トリアージ by claude-code

@claude claude Bot added the scope:config label Jul 7, 2026
@fumikito fumikito changed the title CLAUDE.md からバージョン番号のハードコードを排除し test.yml を参照 CLAUDE.md のバージョン参照化 + Requires at least を 6.6 に更新(#120 対応) Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[WP Audit] CLAUDE.md の CI テストマトリックス記述が実態と乖離 — 開発者が誤った前提で作業するリスク

1 participant