Skip to content

fix(sensor): capture complete Cursor tool results - #90

Merged
barisozbas merged 2 commits into
mainfrom
pr/cursor-tool-results
Sep 11, 2026
Merged

fix(sensor): capture complete Cursor tool results#90
barisozbas merged 2 commits into
mainfrom
pr/cursor-tool-results

Conversation

@barisozbas

Copy link
Copy Markdown
Collaborator

What changed

Cursor tool-result records are now associated with every matching tool call and decoded without altering their payload bytes.

  • Captures repeated and interleaved results rather than only one result shape.
  • Preserves embedded whitespace and byte content during decoding.
  • Documents that tool outputs may contain sensitive data and should be handled accordingly.

Why

The previous extraction path could omit results for valid tool calls and normalize their contents, reducing telemetry fidelity.

Validation

  • pytest -q Sensor/tests
  • ruff check Sensor/adr_sensor/parsers/cursor_parser.py Sensor/tests/test_parsers.py

@barisozbas
barisozbas marked this pull request as ready for review September 7, 2026 10:09
@barisozbas
barisozbas force-pushed the pr/cursor-tool-results branch from 1fcfacb to 7124f52 Compare September 11, 2026 13:26
Summary:
Intent:
- Preserve tool results for every Cursor tool call without altering their bytes.

Changes:
- Associate each result record with its tool call.
- Decode result storage without normalizing or dropping payload bytes.
@barisozbas
barisozbas force-pushed the pr/cursor-tool-results branch from 753705d to db091e6 Compare September 11, 2026 21:52
@barisozbas
barisozbas merged commit ac40034 into main Sep 11, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants