A reviewer spawn killed by the account's session limit reports FAILED with the reason buried in a 31 KB JSONL log nf-ifj ← Beads

open priority 2 task unassigned

Twinned from https://github.com/cfsh/ninefold/issues/671 by tools/beads/import_github.py's reconcile pass.

Filed unclaimed, from a strategic-uplift window. Not my chunk; I hit it while pulling a verdict on #654.

What happened

./tools/pr/review_now.py 654 returned:

``` #654: FAILED — see /root/.ninefold-reviewer/logs/pr-654.20260806-162917.log (backoff 45m; rev:pending stays on; --force retries) ⚠ the review spawn FAILED — see the log path above. --force retries. ```

The log is 31 KB of JSONL, almost all of it thinking_tokens heartbeat records. The actual cause is one field on the final result record:

```json {"is_error": true, "subtype": "success", "stop_reason": "stop_sequence", "num_turns": 43, "total_cost_usd": 3.62, "result": "You've hit your session limit · resets 5pm (UTC)"} ```

The account hit its usage limit. Nothing was wrong with the PR, the dispatcher, the lock, or the reviewer prompt.

Why it's worth fixing
Suggested fix

Have the spawn wrapper read the final result record and put its result string in the failure line:

``` #654: FAILED — session limit, resets 5pm (UTC) (log: …; --force retries after that) ```

Two smaller things in the same area:

Related
Also observed, and possibly a separate issue — I am deliberately not asserting it is a bug

The same run recorded four permission_denials, all Bash: gh api …pulls/654 (patch), gh issue view over the five referenced issues, gh api …/contents/…?ref=<head>, and nproc; grep -c ^processor /proc/cpuinfo.

If those denials are expected, the reviewer was reviewing without the issue bodies it itemizes asks from — which would matter for an ask ledger. If they are not expected, the sandbox is tighter than intended. I could not tell which from the outside, and earlier verdicts on the same PR clearly *did* quote issue bodies, so it may be intermittent or may be a fallback path. Recording the observation rather than a diagnosis; whoever owns the reviewer can tell in a minute.

Dependencies

None.

Comments

No comments.

Add a comment