Skip to content

feat: playwright スキルを 5 スキル + orchestrator に再構成 + エビデンス動画デフォルト ON - #18

Merged
takemi-ohama merged 15 commits into
mainfrom
feature/PLAN23-playwright-evidence-skill
May 26, 2026
Merged

feat: playwright スキルを 5 スキル + orchestrator に再構成 + エビデンス動画デフォルト ON#18
takemi-ohama merged 15 commits into
mainfrom
feature/PLAN23-playwright-evidence-skill

Conversation

@takemi-ohama

@takemi-ohamatakemi-ohama commented May 25, 2026

Copy link
Copy Markdown
Contributor

Summary

playwright-scenario-test スキルを以下の大原則に基づいて再構成。

大原則

  1. 再現可能なテストスクリプトを実装してからテストを実施する
  2. テストスクリプトは ndf plugin 非依存でプロジェクトフォルダに設置する
  3. テスト実行はエビデンス動画を常に取得する (--pwk-no-video で明示的にスキップ可能)

スキル構成 (5 + orchestrator)

PhaseSkill状態機能
1playwright-test-planning改修テスト計画 (HTSM / page role / チェックリスト)
2playwright-script-creation新規テストスクリプト作成 (テンプレート→実装→レビュー)
3playwright-execution新規テスト実行 + エビデンス収集 (video/trace/overlay/quality 統合)
4playwright-report改修レポート生成 (Drive 共有を削除)
--playwright-kit-ops維持ツール群 (init_project / スキャン / アップロード)
--playwright-scenario-test改修orchestrator (5 スキルへの案内 + 大原則記載)

廃止スキル (→ playwright-execution に統合)

playwright-evidence, playwright-overlay, playwright-quality

コード変更

ファイル変更内容
pytest_plugin.py--pwk-no-video CLI オプション追加 + 動画デフォルト ON (invocation_params.args スキャン方式)
templates/run.sh--video=on フォールバック + --pwk-no-video / --pwk-no-evidence 検出
templates/conftest.py.templateテストスクリプト存在チェック (pytest_collection_modifyitems)
plugin.jsonv4.9.0, 44 skills (旧 3 スキル削除 + 新 2 スキル追加)

ドキュメント

  • issues/PLAN23_playwright-skill-split.md — 実装プラン更新
  • docs/superpowers/specs/2026-05-25-playwright-skill-restructure-design.md — 設計書
  • docs/superpowers/plans/2026-05-25-playwright-skill-restructure.md — 実装計画

Test plan

  • claude plugin validate — 検証通過
  • uv run pytest -q — 163 テスト全 PASS (新規 4 件: test_video_default.py)
  • 各新スキルの SKILL.md が正しいトリガーで発動すること
  • --pwk-no-video で動画スキップが動作すること
  • --video 未指定時にデフォルトで動画 ON になること
  • orchestrator が 5 スキルを正しく案内すること

takemi-ohamaand others added 13 commits May 25, 2026 21:09
巨大な単一スキル (playwright-scenario-test) を責務別に分割し、
必要な知識だけをコンテキストに読み込めるようにした。
- playwright-test-planning: テスト計画・page role・チェックリスト
- playwright-evidence: テスト実行 + エビデンス収集
- playwright-overlay: Playwright オーバーレイ操作
- playwright-quality: a11y / Core Web Vitals / body_check
- playwright-report: Markdown レポート + Google Drive 共有
- playwright-kit-ops: playwright_kit パッケージ・スクリプト管理
旧 playwright-scenario-test はオーケストレーターとして残存。
plugin.json を v4.8.0 に更新 (39 → 45 skills)。
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@takemi-ohamatakemi-ohama changed the title refactor: playwright-scenario-test スキルを6つの専門スキルに分割refactor: playwright スキルを大原則に基づき 5 スキル + orchestrator に再構成May 25, 2026
@takemi-ohamatakemi-ohama changed the title refactor: playwright スキルを大原則に基づき 5 スキル + orchestrator に再構成feat: playwright スキルを 5 スキル + orchestrator に再構成 + エビデンス動画デフォルト ONMay 25, 2026

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | gemini | REQUEST_CHANGES

PR #18 の 5 スキルへの再構成、および動画デフォルト ON の実装を確認しました。
全体として実装計画に忠実なリファクタリングが行われていますが、一部のドキュメントに旧スキルの残存や、Drive 連携の文言残りが確認されました。
また、堅牢性の観点から run.sh の引数処理に改善の余地があります。

Comment threadplugins/ndf/skills/playwright-kit-ops/SKILL.md Outdated
Comment threadplugins/ndf/skills/playwright-report/SKILL.md Outdated
Comment threadplugins/ndf/skills/playwright-kit-ops/templates/run.sh

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

以下の点は、分割後の skill discoverability と利用者向け導線を壊すため修正が必要です。

Comment threadplugins/ndf/skills/playwright-execution/SKILL.md
Comment threadplugins/ndf/skills/playwright-script-creation/SKILL.md
Comment threadplugins/ndf/skills/playwright-scenario-test/SKILL.md Outdated
Comment threadplugins/ndf/skills/playwright-kit-ops/SKILL.md Outdated
- playwright-execution/SKILL.md, playwright-script-creation/SKILL.md: frontmatter opening delimiter `---` 追加
- playwright-kit-ops/SKILL.md: 関連 Skill セクションを実在スキル名に更新
- playwright-report/SKILL.md: Google Drive 共有の文言を削除 (kit-ops に分離済み)
- playwright-scenario-test/SKILL.md: クイックスタートを /ndf:playwright-kit-ops 経由に修正
- run.sh: ユーザーが --video=* を明示指定した場合は注入をスキップする制御追加
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | codex | APPROVE

修正提案はありません。

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | gemini | REQUEST_CHANGES

Playwright スキルの再編に伴い、SKILL.md の記述修正や run.sh のデフォルトフラグ制御が追加されています。概ね良好ですが、run.sh での引数判定の不備や、SKILL.md 内のタイポ(不要なスペース混入)によるトリガー認識への影響が懸念される箇所があります。

Comment threadplugins/ndf/skills/playwright-kit-ops/templates/run.sh Outdated
Comment threadplugins/ndf/skills/playwright-report/SKILL.md
Comment threadplugins/ndf/skills/playwright-execution/SKILL.md
Comment threadplugins/ndf/skills/playwright-scenario-test/SKILL.md
case 文に --video (スペース区切り形式) を追加し、--video off のように
スペースで値を渡した場合も VIDEO_FLAG を空にする。
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 3 | gemini | APPROVE

PR #18 の 5 スキル再構成およびエビデンス動画デフォルト ON の実装を確認しました。設計書に基づいたスキルの再配置と、ランタイムの独立性を維持した修正が行われています。追加されたテストコードにより、動画フラグの挙動(デフォルト ON / 明示的 OFF / 優先順位)が正しく検証されていることも確認しました。直ちに修正を要する重大な懸念事項は見当たりません。

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 3 | codex | APPROVE

修正必須の指摘はありません。

@takemi-ohama
takemi-ohama merged commit 73e77b1 into mainMay 26, 2026
@takemi-ohama
takemi-ohama deleted the feature/PLAN23-playwright-evidence-skill branch August 14, 2026 03:58
Sign up for freeto 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

@takemi-ohama