fix missing locals shortening on items with "trace_chain" instead of "trace"

[terencehonles]

Description of the change

In addition to shortening trace, trace_chain should also be shortened (and not cause a logged error about the payload being too large).

Type of change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklists

Development

• Lint rules pass locally
• The code changed/added as part of this pull request has been covered with tests
• All tests related to the changed code pass in development

Code review

• This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
• "Ready for review" label attached to the PR and reviewers mentioned in a comment
• Changes have been reviewed by at least one other engineer
• Issue from task tracker has a link to this pull request

[danielmorell]

Hey @terencehonles, sorry this sat idle for so long. Locally I merged this with our master branch and ran our test suite and everything passed. I would like to run the whole test suite including frameworks against this again since there have been a lot of commits since this PR was created.

Can you pull the latest from master into your branch? This should trigger a rerun of the entire test suite.

I also will take a closer look at it gets reviewed by anyone else who needs to.

[jmarronm]

@danielmorell can we review this PR and consider closing or merging it , please?

[danielmorell]

Thank you @terencehonles! This looks good!

[terencehonles]

Sorry I didn't see your previous message, glad this moved forward 🙂