LINBEE-30174 | fix: backfill a missed grade at SessionEnd (v1.1.1)

[nivSwisa1]

Interrupted turns never fire Stop, so the grade was lost while SessionEnd still sent tokens (seen on a customer org); SessionEnd now backfills unsent decisions (tagged backfill=session_end) and vseen is recorded only after the repo resolves. Tests: new backfill_test 4/4 (old reporter fails 2), all existing suites green.

🤖 Generated with Claude Code

✨ PR Description

Purpose: Update the agentic-advisor plugin to backfill missed grade decisions during the SessionEnd event.

Main changes:

• Refactored grade reporting into report_decisions to allow execution during both Stop and SessionEnd
• Added backfill: session_end tag to DECISION events triggered during session termination
• Updated plugin version to 1.1.1 across marketplace and plugin configuration files

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how}

[Copilot]

🟡 Changes recommended

Multi-repository backfills can report incorrect repository identity and grading-token metadata.

1 open finding

• Pair each replayed verdict with its own grading metadata · New

What changed in this PR

Adds SessionEnd recovery for effort decisions missed by interrupted turns and updates the plugin release metadata.

Changes:

• Backfills unsent decisions during SessionEnd.
• Delays vseen recording until repository resolution.
• Bumps the plugin version to 1.1.1.

File	Description
plugins/​agentic-advisor/​README.md	Documents SessionEnd backfills.
plugins/​agentic-advisor/​hooks/​agentic-advisor-report.sh	Implements decision backfilling.
plugins/​agentic-advisor/​.claude-plugin/​plugin.json	Bumps plugin version.
.claude-plugin/​marketplace.json	Updates marketplace versions.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[gitstream-cm]

✨ PR Review

The hook changes are a focused, well-reasoned fix for the "grade lost on interrupted turn" problem: moving the vseen write to after repo_url resolution and refactoring the decision-reporting loop into a reusable function that also runs on SessionEnd (tagged backfill=session_end) looks correct and consistent with the surrounding dedup logic. The main concern is that the ticket this PR is meant to close specifies a different/additional approach that isn't reflected in the diff.

1 issues detected:

🎯 Scope - The implementation diverges from the ticket's specified technical direction and leaves one of the two required findings (phase-2 correction dedup) unaddressed.

Details: The linked ticket explicitly directs reporting the decision on the first PreToolUse after the verdict (keeping Stop as fallback) to fix the "grade lost mid-response" issue, and separately requires adding a stable session key + per-session sequence number so consumers can dedupe preliminary vs. corrected grades (finding #2). This PR instead implements a SessionEnd-based backfill for the missed-grade case and does not address the session-key/sequence-number requirement for the double-counting issue at all.

File: plugins/agentic-advisor/hooks/agentic-advisor-report.sh (320-366)

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how}

[orca-security-us]

Orca Security Scan Summary

Status	Check	Issues by priority
Passed	Infrastructure as Code	0   0   0   0	View in Orca
Passed	OSS Licenses	0   0   0   0	View in Orca
Passed	SAST	0   0   0   0	View in Orca
Passed	Secrets	0   0   0   0	View in Orca
Passed	Vulnerabilities	0   0   0   0	View in Orca

[nivSwisa1]

Re gitStream's scope note (vs LINBEE-30174's suggested first-PreToolUse approach): deliberate. Backfilling unsent decisions at SessionEnd recovers the lost grade with no extra hook on every tool call, and it was verified to reproduce and fix the exact customer case (tokens arrived, decision didn't). The ticket's second part — a session key/sequence to dedupe preliminary vs phase-2-corrected grades — is not addressed here and stays open on LINBEE-30174.

[Copilot]

🟢 Approved

The backfill path preserves existing deduplication while addressing interrupted turns without introducing unresolved issues.

0 open findings

1 resolved since last review

• Pair each replayed verdict with its own grading metadata

🧠 Review effort: Balanced

[MishaKav]

Great job! 🎉