Inconsistent naming conventions in demo agents

[McStevenss]

The agent class requires two fields:

• agent_id
• name
  For the demo agents all of these utilizes different naming conventions.
  Example:

StellaCactusAgent -> (No demo?)

Agent_id: 'stella/stella_cactus_agent' -> (relative path?)
name: 'StellaCactus' -> (camel case)

DemoBreweryAgent -> (Demo?)

Agent_id: 'brewery_agent' -> (snake case)
name: 'BREWERY' -> (all caps)

GodAgent -> (Demo?)

Agent_id: 'god_agent' -> (snake case)
name: 'God' -> (Capitalization)

All these different conventions make it hard to add agents intuitively and require constant going back and forth to check agent_ids etc