Rename updateAiConfig and fix doc comment

[ryanwarfield]

Description

updateAiConfig no longer updates the AI config, now it just sets a flag. Config updates happen through a shared pointer, the flag only matters for changes to the override play (needs a play fsm reset).

Testing Done

• Ran play selection tests
• Ran thunderscope

Resolved Issues

closes #3984

Review Checklist

It is the reviewers responsibility to also make sure every item here has been covered

• Function & Class comments: All function definitions (usually in the .h file) should have a javadoc style comment at the start of them. For examples, see the functions defined in thunderbots/software/geom. Similarly, all classes should have an associated Javadoc comment explaining the purpose of the class.
• Remove all commented out code
• Remove extra print statements: for example, those just used for testing
• Resolve all TODO's: All TODO (or similar) statements should either be completed or associated with a github issue

[nycrat]

👍