Skip to content

[codex] Fix Fable goal ledger recovery#1

Merged
baskduf merged 1 commit into
mainfrom
codex/fix-fable-goal-ledger
Jun 15, 2026
Merged

[codex] Fix Fable goal ledger recovery#1
baskduf merged 1 commit into
mainfrom
codex/fix-fable-goal-ledger

Conversation

@baskduf

@baskduf baskduf commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Reopen failed or blocked goal-ledger stories on next instead of reporting false completion or skipping ahead.
  • Add regression tests for failed and blocked story recovery plus marketplace path validation.
  • Standardize local docs, release, PR, and CI examples on python3.

Root Cause

The goal ledger only counted pending and in-progress stories when deciding whether work remained, so failed or blocked stories could be treated as complete. Tightening that logic without a reopen path would have made failed/blocked plans unrecoverable, so next now reopens the first failed or blocked story before later pending stories.

Verification

  • python3 -m unittest discover -s tests -v
  • python3 -m py_compile plugins/codex-fable5/skills/codex-fable5/scripts/codex_goals.py plugins/codex-fable5/skills/codex-fable5/scripts/fable_coverage.py plugins/codex-fable5/skills/codex-fable5/scripts/make_litellm_config.py tests/test_scripts.py
  • python3 plugins/codex-fable5/skills/codex-fable5/scripts/fable_coverage.py
  • git diff --cached --check

Review

  • Self-review completed.
  • Sub-agent review completed; initial findings were patched, final pass reported no blocking findings.

@baskduf baskduf marked this pull request as ready for review June 15, 2026 00:51
@baskduf baskduf merged commit 1133948 into main Jun 15, 2026
3 checks passed
@baskduf baskduf deleted the codex/fix-fable-goal-ledger branch June 15, 2026 00:52
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