Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

  • Understand the current implementation (lines 82-84 in converter.ts)
  • Create a helper function to flatten nested objects using dot notation
  • Update the composeBacktrace function to use the new helper
  • Add comprehensive tests for nested objects and arrays
  • Run tests to validate the implementation
  • Verify no regressions with existing tests
  • Address code review feedback for empty array handling
  • Final security check - no vulnerabilities found
  • Manual verification of the implementation
  • Remove unrelated file changes
Original prompt

This section details on the original issue you should resolve

<issue_title>bug(sentry): sentry worker conversation does not support nested objects in backtrace frame arguments</issue_title>
<issue_description>See

backtraceFrame.arguments = Object.entries(frame.vars).map(([name, value]) => {
return `${name}=${value}`;
});

Now if value contains non-primitive value, it is stringified as [object Object].

Example

params: [object Object]

We should support objects and arrays in there.

For example, we can use dot notation keys like.

params.foo. = 1,
params.bar = 2,
params.second.glass = 3

or simply use json stringified:

params: "{foo: 1, bar: 2, second: {glass: 3}}"

</issue_description>

<agent_instructions>Try implementing dot notation in key names</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 23, 2026 19:21
Co-authored-by: neSpecc <3684889+neSpecc@users.noreply.github.com>
Co-authored-by: neSpecc <3684889+neSpecc@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix Sentry worker to support nested objects in backtrace frame arguments bug(sentry): Flatten nested objects in backtrace frame arguments using dot notation Jan 23, 2026
Copilot AI requested a review from neSpecc January 23, 2026 19:24
@neSpecc neSpecc marked this pull request as ready for review January 23, 2026 19:28
Copy link
Member

@neSpecc neSpecc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot remove unrelated changes like listing of files that should not be affected by your solution

…tion

Co-authored-by: neSpecc <3684889+neSpecc@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 23, 2026

@copilot remove unrelated changes like listing of files that should not be affected by your solution

Removed unrelated files. Only the 3 Sentry worker files remain in the changeset (eb368a5).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • fastdl.mongodb.org
    • Triggering command: /usr/local/bin/node /usr/local/bin/node ./postinstall.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested a review from neSpecc January 23, 2026 19:33
@neSpecc neSpecc merged commit eea9877 into master Jan 23, 2026
4 checks passed
@neSpecc neSpecc deleted the copilot/support-nested-objects-backtrace branch January 23, 2026 19:38
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.

bug(sentry): sentry worker conversation does not support nested objects in backtrace frame arguments

2 participants