refactor: minimize cs_main locking for governance

[PastaPastaPasta]

Issue being fixed or feature implemented

Reduce cs_main locking scope in governance

What was done?

Refactor some stuff around to enable reducing cs_main scope

How Has This Been Tested?

governance functional tests

Breaking Changes

None

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated relevant unit/integration/functional/e2e tests
• I have made corresponding changes to the documentation
• I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

[UdjinM6]

It's a bit scary to touch this code tbh 😄

(re)synced a couple of times on mainnet and testnet, kept testnet node running for a couple of hours and it survived incoming triggers and votes.

ACK

[PastaPastaPasta]

I kinda agree, let's leave this for after we branch off v20

[github-actions]

This pull request has conflicts, please rebase.

[PastaPastaPasta]

This is a test comment from github CLI