fix: buffMaintain should return true if it tried to do something

[midgleyc]

Description

Issues seen while testing #1331.

buffMaintain should return true if it tried to do something (looking at the useSkill / useItem behaviour). The CLI commands / visit_urls were not doing this, so do so.

Also shouldn't get the Grim Brother buff if speculating.

How Has This Been Tested?

Can't test the Grim Brother change because I don't have one. Did some more item provider runs.

Checklist:

• My code follows the style guidelines of this project.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have based my pull request against the main branch or have a good reason not to.