Skip to content

launcher: list owed rechecks in openadapt verify ledger list (waits for flow release) - #1840

Draft
abrichr wants to merge 1 commit into
mainfrom
agent/1670-launcher
Draft

abrichr wants to merge 1 commit into
mainfrom
agent/1670-launcher

Conversation

@abrichr

@abrichr abrichr commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

Part of #1670

Moved out of #1834 so that PR is flow-only. This draft waits for the next openadapt-flow release: it calls Ledger.rechecks_owed, which the released openadapt-flow 2.5.0 (pinned in the lock) doesn't have, so the launcher job fails with AttributeError until a flow release that includes #1834 is in the lock.

What changed

  • openadapt verify ledger list now also shows each provisional CONFIRMED whose recheck is still owed, as recheck-owed with its due time and actor. A recheck isn't lost when the server that holds it restarts. When nothing is held, open or owed, it prints "No held or open writes. No rechecks owed."
  • New test in tests/test_verify_command.py: the owed recheck is listed until a confirming recheck runs, then it's gone.

The two files are byte-identical to their versions at #1834 head 8559a6b, which Codex approved.

How it was tested

Not covered

Moved out of #1834. Needs Ledger.rechecks_owed from openadapt-flow, so it waits
for the next flow release.

This branch has not been deployed

No deployments
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