Skip to content

feat(cli): allow passing feature pubkey instead of name on startup#577

Merged
MicaiahReid merged 4 commits intomainfrom
fix/feature-handling
Mar 19, 2026
Merged

feat(cli): allow passing feature pubkey instead of name on startup#577
MicaiahReid merged 4 commits intomainfrom
fix/feature-handling

Conversation

@MicaiahReid
Copy link
Collaborator

@MicaiahReid MicaiahReid commented Mar 18, 2026

83ab44a - feature accounts didn't exist in VM at runtime (when referenced in a program), because litesvm doesn't store them. Temporary fix until litesvm fix is in. Issue to remove todo's
7ee8fe2 - litesvm references feature_sets injected from with_feature_set in subsequent with_builtins, with_sysvars, and with_default_programs calls. So, when we update feature sets, call these fns
e5336bf - Update --feature <kebab-case-feature-name> to take --feature <pubkey>. This should prevent breaking the feature name for now, but we'll be able to remove a bunch of code once we deprecate that feature. I've added the deprecation notice to the CLI flags

@MicaiahReid MicaiahReid changed the title Fix/feature handling feat(cli): allow passing feature pubkey instead of name on startup Mar 18, 2026
@MicaiahReid MicaiahReid requested a review from lgalabru March 18, 2026 21:18
Copy link
Collaborator

@lgalabru lgalabru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome finding, thanks for the clean-up!

@MicaiahReid MicaiahReid merged commit 54e39c3 into main Mar 19, 2026
5 checks passed
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.

2 participants