Open
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is ON, but it could not run because the spend limit has been reached. To enable Bugbot Autofix, raise your spend limit in the Cursor dashboard.
| "run_terminal_command_v2", | ||
| "shell", | ||
| "terminal", | ||
| } |
There was a problem hiding this comment.
Duplicated _EXECUTION_TOOL_NAMES risks silent divergence
Medium Severity
_EXECUTION_TOOL_NAMES is copy-pasted from execution_evidence_service.py into live_session_signal_service.py. The new file already imports _classify_status and extract_execution_evidence from that module, so it could import this constant too. If a new tool name is added to one copy but not the other, _count_tool_errors would silently drift from the execution-evidence logic, miscounting errors for the forgotten tool.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Testing
Note
Medium Risk
Adds a new MCP-exposed capability that analyzes in-progress local transcripts and derives risk/sentiment heuristics, which could affect user-facing guidance if the scoring or session selection is wrong. Changes are self-contained and covered by new unit tests, with no auth or persistence changes.
Overview
Adds live local session signal analysis that summarizes in-progress sessions for friction, sentiment, and abandonment risk, returning a
LiveSessionSignalsResponsewith structuredsignals,risk_level, andsatisfaction_signal.Exposes this via a new MCP tool
live_session_signals(wired throughprimer_live_session_signals) that formats the live summary for repeated polling during a session, and updates tests/roadmap to reflect the shipped capability.Written by Cursor Bugbot for commit 0fcccb6. This will update automatically on new commits. Configure here.