Bug/jet trace

[topologoanatom]

Fixes: #197

Description

Changing the heuristics for removing debug wrappers inside the jet result parser.

This approach is fragile, but it will suffice until the assertl-based debugging strategy is replaced.

[KyrylR]

Pls follow the: https://github.com/BlockstreamResearch/SimplicityHL/blob/master/CONTRIBUTING.md

[KyrylR]

Could you bump the patch version pls?

[topologoanatom]

Could you bump the patch version pls?

bumped to 0.4.1

[KyrylR]

ACK 46a413c

[KyrylR]

@topologoanatom Commits should be signed

[KyrylR]

ACK 11a03ba

[apoelstra]

Heads up that 352b58b does not pass tests. This makes bisecting the repo harder. Broken unit tests should be annotated with #[ignore] with a comment explaining that they will be fixed in later commits. Alternately, they can assert the wrong thing with a comment explaining that the assertion will be fixed in later commits.

@KyrylR thanks for tagging and publishing -- FYI I usually post a comment saying that I did so, because it's easy to forget.

[apoelstra]

I would also prefer that the PR description describe what's going on -- that we are changing our heuristic for removing debug wrappers in our trace logic, that this is not a great solution but in #197 we agreed it was a good thing to do for now, and that there is a version bump in this PR.

[apoelstra]

Version bumps should also have a CHANGELOG entry. It looks like this release included both this PR and #187 (as well as some admin stuff that doesn't need to be in the changelog).

[KyrylR]

Ohh, sorry, I got it

Basically, because we added a reproduction test in 352b58b and did not immediately implement a fix, local tests failed

I will create a new PR to update the CHANGELOG a little bit later

[KyrylR]

Updated CHANGELOG #215