Return `struct` instead of separate values

[digitalghost-dev]

The SetupPokemonFlagSet() and SetupAbilityFlagSet() commands return several separate values. Readability can be improved by using a struct:

type PokemonFlags struct {
    FlagSet        *flag.FlagSet
    Abilities      *bool
    ShortAbilities *bool
    Defense        *bool
    ShortDefense   *bool
    Image          *string
    ShortImage     *string
    Move           *bool
    ShortMove      *bool
    Stats          *bool
    ShortStats     *bool
    Types          *bool
    ShortTypes     *bool
}

and

type AbilityFlags struct {
	FlagSet      *flag.FlagSet
	Pokemon      *bool
	ShortPokemon *bool
}

[coderabbitai]

📝 CodeRabbit Plan Mode

Generate an implementation plan and prompts that you can use with your favorite coding agent.

• Create Plan

Examples

• Example 1
• Example 2

---

🔗 Similar Issues

Related Issues

• Eliminate duplicated error handling in flag processing #179
• Add function for image flag error checking #145

👤 Suggested Assignees

• digitalghost-dev

---

🧪 Issue enrichment is currently in open beta.

You can configure auto-planning by selecting labels in the issue_enrichment configuration.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!