umputun / ralphex / 21494411578

81%

push

github

fix: resolve {{PLAN_FILE}} to completed/ path after plan is moved (#50)

* fix: resolve {{PLAN_FILE}} to completed/ path after plan is moved

When a plan file is moved to docs/plans/completed/ during execution,
subsequent {{PLAN_FILE}} template expansions still referenced the
original (now non-existent) path.

A plan file can be moved to completed/ in two scenarios:
- ralphex moves it automatically after all tasks complete
- a task in the plan itself moves the file as part of its instructions

Added resolvePlanFilePath() method that checks both locations and
returns the path where the file actually exists. Updated all callers:
getPlanFileRef(), getGoal(), and buildCodexPrompt().

* refactor: use resolvePlanFilePath() in hasUncompletedTasks, add tests

Address Copilot review comments:
- hasUncompletedTasks() now uses resolvePlanFilePath() instead of
  duplicating path resolution logic
- Added test for hasUncompletedTasks with file in completed/ dir
- Added test for buildCodexPrompt with file in completed/ dir
- Added TestBuildCodexPrompt export for testing

17 of 21 new or added lines in 2 files covered. (80.95%)

4066 of 5084 relevant lines covered (79.98%)

123.92 hits per line