Skip to content

feat: add a last-newest review action - #48

Merged
backnotprop merged 1 commit into
plannotator:mainfrom
mshddev:feat/last-newest-action
Sep 21, 2026
Merged

backnotprop merged 1 commit into
plannotator:mainfrom
mshddev:feat/last-newest-action

Conversation

@mshddev

@mshddev mshddev commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Adds a last-newest action next to last, so the agent's newest reply opens without the picker. With the suggested binding that's prefix+ctrl+o, instead of prefix+shift+o then Enter. The older replies are still one p away.

Depends on

What changed

  • herdr-plugin.toml gains last-newest, running ./bin/plannotator-tui.exe herdr last --newest with the same contexts and macos/linux gate as last. last is untouched, and no other entry changes.
  • The README gets a key table row and a suggested prefix+ctrl+o binding. docs/windows-full-acceptance.md lists it with the other review actions.
  • scripts/smoke.sh expects the new action.

Validation

  • scripts/test-windows-full-manifest.py passes against plannotator-tui's herdr/herdr-plugin.toml at the CI ref.
  • scripts/lite-regression.sh passes with 487 observables, 145 screens, 1 deliberate difference, zero failures.
  • scripts/test-fetch-plannotator-tui.sh and scripts/test-fetch-herdr-annotate.sh pass.
  • I ran it in Herdr with this checkout linked and a local plannotator-tui build staged through PLANNOTATOR_TUI_BIN. prefix+ctrl+o opens the newest reply with no picker.

This covers the picker step in #37, not the wrong-session problem that thread is about.

refs #37

`annotate.last-newest` runs `herdr last --newest`, opening the focused agent's
newest reply straight away instead of the picker. The existing `last` action is
unchanged. README gets the key binding and the use-table row; the Windows Full
acceptance checklist gets the action row and its argv expectation; smoke.sh's
exact action set gains the id.

The action needs a plannotator-tui release carrying `--newest`; no such release
exists yet, so the pin stays at 0.8.0 and the flag is exercised against a local
build staged with PLANNOTATOR_TUI_BIN. Bump the pin once the flag ships.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@backnotprop
backnotprop marked this pull request as ready for review September 21, 2026 03:56
@backnotprop
backnotprop merged commit 33a0cfd into plannotator:main Sep 21, 2026
7 checks passed
@backnotprop

Copy link
Copy Markdown
Contributor

Merged, thanks. The pin moved to plannotator-tui 0.9.0 in #56 a few minutes ago, so this works on a fresh install or after re-running the install command. Both of your PRs landed cleanly; nice work.

backnotprop added a commit that referenced this pull request Sep 21, 2026
* fix: add last-newest to the windows-full manifest

#48 added the action to the root manifest after #49 landed, so the parity check
on main failed. Same action, Windows form: direct argv, no platform gate.

refs #48

* chore: plugin 0.5.0

Windows Full variant, the last-newest action, and plannotator-tui 0.9.0.

---------

Co-authored-by: Michael Ramos <support@plannotator.ai>
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.

2 participants