Skip to content

fix: walk コマンドが経路上の敵・毒を素通りするバグを修正#15

Merged
mskasa merged 3 commits intoebitenginefrom
fix/walk-jump
Mar 26, 2026
Merged

fix: walk コマンドが経路上の敵・毒を素通りするバグを修正#15
mskasa merged 3 commits intoebitenginefrom
fix/walk-jump

Conversation

@mskasa
Copy link
Copy Markdown
Owner

@mskasa mskasa commented Mar 26, 2026

Summary

  • walkTo の各ステップで checkCell(enemies) を呼ぶよう修正。3lw で途中の敵・毒を素通りするバグを解消
  • jumpToIsWalkable ガードを追加(目的地のみ判定の意図を明示)
  • ApplywalkTo/jumpTocheckCell まで enemies []Enemy を伝播
  • CLAUDE.md の walk/jump セクションを実装方針に合わせて書き直し
  • ADR 追加: docs/decisions/2026-03-26-walk-jump-2.md

Test plan

  • TestWalkCollectsApplesAlongPath4l で3個のリンゴを全て収集する
  • TestJumpSkipsEnemiesAlongPath0 で経路上の敵を無視してジャンプする
  • TestWalkDiesOnPoison3l で途中の毒を踏んで即死する
=== RUN   TestWalkCollectsApplesAlongPath
--- PASS: TestWalkCollectsApplesAlongPath (0.00s)
=== RUN   TestJumpSkipsEnemiesAlongPath
--- PASS: TestJumpSkipsEnemiesAlongPath (0.00s)
=== RUN   TestWalkDiesOnPoison
--- PASS: TestWalkDiesOnPoison (0.00s)
ok  	github.com/masahiro-kasatani/pacvim/state	(83 tests all PASS)

🤖 Generated with Claude Code

- walkTo の各ステップで checkCell(enemies) を呼び、敵接触・毒踏みを判定するよう修正
- jumpTo に IsWalkable ガードを追加(目的地のみ判定を明示)
- Apply/walkTo/jumpTo/checkCell に enemies []Enemy パラメータを伝播
- CLAUDE.md の walk/jump セクションを実装方針に合わせて書き直し
- ADR 0004: walk と jump の2アクションタイプを設ける
- テスト追加: TestWalkCollectsApplesAlongPath / TestJumpSkipsEnemiesAlongPath / TestWalkDiesOnPoison

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mskasa mskasa changed the base branch from master to ebitengine March 26, 2026 08:51
mskasa and others added 2 commits March 26, 2026 17:53
ebiten を import するルートパッケージを go test の対象から除外し、
Linux CI 環境で X11 ヘッダが不要になるよう修正。
ebitengine ブランチへの push/PR でも CI が動くようトリガーを追加。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mskasa mskasa merged commit cc02969 into ebitengine Mar 26, 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