[core] Introduce SynthTransaction, refactor synth state tracking

[sruon]

I affirm:

• I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
• I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
• I have read and understood the Contributing Guide and the Code of Conduct.
• I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

• Gets rid of CraftContainer in favor of CCraftState (state tracking) and SynthTransaction (items custody + reward)
• Adds a handful of guards in the relevant places
• Adds several test cases

Unlike other transaction types, SynthTransaction operates on a default-lost model, meaning items must explicitly be marked saved for them to not be consumed.

Steps to test these changes

Tests included, ran through a manual checklist for what couldnt be automated

[Xaver-DaRed]

Could you be a little more descriptive with variable names?
cs and tx mean nothing.

[zach2good]

Is this that thing where make_unique can't get around the semantics of Badge?

[Xaver-DaRed]

Rebase, please