fix: Fixes cases that did not need to error early in the process

[ChaoticTempest]

This fixes a bunch of cases where we did not need to return the error early at all and potentially roll back the state.

For example, the TripleManager::poke would return the first error, but that did not need to happen as erroring out on a triple generator should just bin the generator and not interfere with the poking of other generators.

Also adds in a check for triple_id or presignature_id collisions.

[github-actions]

Terraform Feature Environment Destroy (dev-634)

Terraform Initialization ⚙️success

Terraform Destroy success

Show Destroy Plan

No changes. No objects need to be destroyed.

Either you have not created any objects yet or the existing objects were
already deleted outside of Terraform.

Destroy complete! Resources: 0 destroyed.

Pusher: @volovyks, Action: pull_request, Working Directory: ``, Workflow: Terraform Feature Env (Destroy)