Add agent-safe review findings fallback

[pfleidi]

https://entire.io/gh/entireio/cli/trails/724

Why

entire review --findings needed an agent-safe way to reach full saved review
findings. The previous non-interactive fallback could show that findings
existed, but agents still needed the terminal picker to reveal the full detail.

What changed

The findings list now prints stable handles with direct view: commands, and
entire review --findings <handle> renders the full saved findings without a
TUI. Non-interactive profile editing now refuses with scripted alternatives
instead of requiring a picker.

Usage examples

entire review --findings

prints saved findings with commands such as:

view: entire review --findings 'claude-session'

Then:

entire review --findings claude-session

prints the full saved review detail.

Decisions made during development

The fallback stays plain text instead of forcing JSON everywhere, matching the
agent-safe CLI guidance that complete human-readable output is acceptable.
Handles prefer saved source session IDs and fall back to a timestamp handle when
no session ID is available.

Technical tradeoffs

Timestamp fallback handles can theoretically collide, so the command reports an
ambiguous handle instead of guessing. Generated view: commands quote handles
for copy-paste safety while lookup still accepts the raw argument value.

---

Note

Low Risk
CLI-only review findings browsing and argument routing; no auth, persistence format changes beyond shared timestamp helper for manifest filenames.

Overview
entire review --findings now supports an optional handle so agents and scripts can open a saved run without the TUI. entire review --findings <handle> prints full reviewer output and aggregate summary; handles are source session IDs, with a UTC timestamp fallback when there is no session ID. Ambiguous handles fail explicitly instead of picking a manifest.

The non-interactive findings list and post-review footer include copy-paste view: commands with shell-quoted handles. Unknown handles error with a list of available view: commands. With --findings, a positional argument is treated as the handle (so --profile can be combined without conflicting with “profile as positional”).

entire review --edit in a non-interactive environment now exits with pointers to --list and --configure --set-* instead of requiring the picker.

^{Reviewed by Cursor Bugbot for commit 71171b0. Configure here.}

[Copilot]

Pull request overview

This PR makes entire review --findings fully usable in non-interactive/agent contexts by emitting stable “handle” references and enabling direct, non-TUI rendering of saved findings via entire review --findings <handle>. It also hard-fails --edit when no interactive terminal is available, pointing users to scripted alternatives.

Changes:

• Add handle-based routing for --findings so entire review --findings <handle> prints full saved review detail without a picker.
• Improve non-interactive output by printing copy/paste view: commands (with shell-quoted handles) in findings lists and the completion footer.
• Refuse non-interactive --edit with actionable non-TUI alternatives (--list, --configure --set-*).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
cmd/entire/cli/review/fix.go	Implements handle lookup, detail rendering, and generation of shell-quoted view: commands for findings browsing.
cmd/entire/cli/review/cmd.go	Adjusts argument routing so the optional positional arg becomes a findings handle when --findings is set; adds non-interactive refusal for --edit.
cmd/entire/cli/review/manifest_test.go	Adds/updates tests for quoted handle commands, timestamp fallback handles, and handle-based detail rendering.
cmd/entire/cli/review/cmd_test.go	Adds a regression test ensuring --edit fails non-interactively with scripted alternatives.

[pfleidi]

Bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 71171b0. Configure here.}