Add validation to cog configs

[juniper-shopify]

Adds an abstract a validate! method to Cog::Config that will get called once a cog instance's configs are merged. A config that fails validation will raise an exception before the workflow begins running.

This also allows config validation logic to take the full state of the config into account, if necessary (vs the previous pattern of validating individual args when they were set on the config object)

---

Introduces a valid_<param> and valid_<param>! accessor pattern for config class attributes. The convention I'm imagining is this:

• boolean config setters are bang methods (exit_on_error! and no_exit_on_error!)
• boolean config getters are equivalently named question methods (exit_on_error?)
  • boolean values are never in an invalid state and there's never a name collision here
• most other value setters are single-argument methods that take a value (parallel 3)
• getters for other values are methods with the valid_ prefix (valid_parallel!)
  • method uses the ! suffix if it performs validation and might throw
  • method uses no bang suffix if it does not perform validation and will always return a valid or default value
  • this prevents name collision between the setters and getters, makes it clear that a validated value is being used, and allows us to narrow the type annotation on getters that, e.g., will not produce nils.

[juniper-shopify]

• Basic streaming messages for Claude agent provider #518
• Expand the agent provider structure and add some configs for the claude provider #517
• Run cogs in custom working directory #516
• Alphabetize function DSL test cases #515
• Add method doc to cogs config #514
• Deep-Dupe config defaults before setting #513
• Add validation to cog configs #512 👈 (View in Graphite)
• Add a global config scope to set basic things that apply to all cogs of all types #511
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[juniper-shopify]

Merge activity

• Nov 3, 4:20 PM UTC: @juniper-shopify merged this pull request with Graphite.