Skip to content

Replace hardcoded addon enum with dynamic discovery#106

Merged
renardeinside merged 6 commits intomainfrom
dynamic-addon-enum
Feb 20, 2026
Merged

Replace hardcoded addon enum with dynamic discovery#106
renardeinside merged 6 commits intomainfrom
dynamic-addon-enum

Conversation

@renardeinside
Copy link
Copy Markdown
Collaborator

@renardeinside renardeinside commented Feb 20, 2026

Summary

  • Replace hardcoded Addon enum in apply.rs with dynamic validation against addon.toml manifests — new addons no longer require code changes
  • Remove stale SKILL.md.jinja2 from base template (skills install via addon skill_path)
  • Replace "stateful apps only" terminology with "requires lakebase addon" across all assistant templates
  • Default claude addon to selected during apx init
  • Add addon CLI documentation to SKILL.md
  • Fix integration test to use current --addons flag

Test plan

  • cargo build passes
  • cargo test — all 232 tests pass
  • cargo run -- dev apply --help — shows dynamic addon list
  • cargo run -- dev apply nonexistent — error lists valid addons
  • cargo run -- dev apply cursor — works as before
  • grep -r "stateful apps only" — zero results
  • Fresh apx init defaults to ui + claude addons

🤖 Generated with Claude Code

@renardeinside renardeinside merged commit ec51fd8 into main Feb 20, 2026
3 checks passed
@renardeinside renardeinside deleted the dynamic-addon-enum branch February 20, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant