Cursor CLI: fix path to transcripts

[squishykid]

No description provided.

[cursor]

PR Summary

Medium Risk
Moderate risk because it changes how Cursor transcript locations are derived during lifecycle events; incorrect resolution could break session initialization or transcript capture for Cursor users.

Overview
Fixes Cursor transcript path handling across Cursor CLI vs IDE. Session directory resolution now points to .cursor/projects/<project>/agent-transcripts, and ResolveSessionFile supports both nested (<id>/<id>.jsonl) and flat (<id>.jsonl) layouts (preferring nested when present).

Lifecycle hook parsing now computes SessionRef when transcript_path is missing (Cursor CLI behavior) by deriving the worktree root and resolving the expected transcript file, with warning logs on failures. Hook input types were split/updated (sessionStartRaw, stopHookInputRaw, sessionEndRaw) and tests were expanded to cover all new path/layout and CLI-null-transcript cases.

^{Written by Cursor Bugbot for commit 69b9d73. Configure here.}

[Copilot]

Pull request overview

This PR fixes the transcript path resolution for Cursor CLI hooks. Cursor IDE and Cursor CLI handle transcript paths differently: the IDE provides the full path in hook payloads, while the CLI sends null and requires dynamic path computation. The PR refactors hook input types and adds path resolution logic to handle both cases correctly.

Changes:

• Split generic sessionInfoRaw type into hook-specific types (sessionStartRaw, stopHookInputRaw, sessionEndRaw) for better type safety and clearer documentation
• Added resolveTranscriptRef function that uses provided paths when available (IDE) or computes them dynamically (CLI)
• Fixed sanitizePathForCursor to trim leading dashes from sanitized paths
• Corrected GetSessionDir to include the agent-transcripts subdirectory

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
cmd/entire/cli/agent/cursor/types.go	Refactored from generic sessionInfoRaw to hook-specific types with better documentation of IDE vs CLI behavior
cmd/entire/cli/agent/cursor/lifecycle.go	Added resolveTranscriptRef helper and updated parseTurnEnd/parseSessionEnd to use it; added logging for path resolution
cmd/entire/cli/agent/cursor/cursor.go	Fixed sanitizePathForCursor to trim leading dashes; fixed GetSessionDir to include agent-transcripts subdirectory; updated ResolveSessionFile to check both nested and flat layouts
cmd/entire/cli/agent/cursor/cursor_test.go	Expanded tests for ResolveSessionFile to cover flat, nested, and missing file scenarios; updated sanitizePathForCursor test expectations; added assertion for agent-transcripts suffix in GetSessionDir
cmd/entire/cli/agent/cursor/lifecycle_test.go	Added comprehensive tests for CLI (no transcript_path) and IDE (with transcript_path) scenarios for both Stop and SessionEnd hooks

[Soph]

@cursor review

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

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