14 lines
780 B
Markdown
14 lines
780 B
Markdown
|
# IMPLEMENTATION.md — Fix Status Script Bugs
|
||
|
|
|
||
|
|
## Changes Made
|
||
|
|
- `scripts/status.py:504-510`: Removed neutered target-phase artifact check (restored `pass` — analysis showed it's redundant with current-phase check at 525-531)
|
||
|
|
- `scripts/status.py:1007,1061`: Removed dead double `continue` statements
|
||
|
|
- `scripts/status.py:523`: Changed `cmd_approve` to use `_require_state` instead of `_read_state` for consistency
|
||
|
|
- `scripts/status.py:1088-1095`: Added `cmd_list_states` function and `--list-states` CLI argument
|
||
|
|
|
||
|
|
## How It Works
|
||
|
|
- `--list-states` prints all valid phases with approval requirements noted
|
||
|
|
- Double continue dead code removed
|
||
|
|
- cmd_approve rejects untracked tasks consistently
|
||
|
|
- Target-phase check restored to `pass` (existing checks cover the cases correctly)
|