Skip to content

cli: program: remove ExtendProgramChecked#11764

Closed
buffalojoec wants to merge 1 commit into
anza-xyz:masterfrom
buffalojoec:cli-delete-extend-program-checked
Closed

cli: program: remove ExtendProgramChecked#11764
buffalojoec wants to merge 1 commit into
anza-xyz:masterfrom
buffalojoec:cli-delete-extend-program-checked

Conversation

@buffalojoec
Copy link
Copy Markdown

@buffalojoec buffalojoec commented Apr 6, 2026

Problem

SIMD-0164 was closed and never landed. Therefore the ExtendProgramChecked instruction should not exist.

Summary of Changes

Remove it from the CLI.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 25.00000% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.1%. Comparing base (ed322a9) to head (a1c8a6d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #11764   +/-   ##
=======================================
  Coverage    83.1%    83.1%           
=======================================
  Files         860      860           
  Lines      321462   321371   -91     
=======================================
- Hits       267408   267347   -61     
+ Misses      54054    54024   -30     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@buffalojoec buffalojoec marked this pull request as ready for review April 9, 2026 01:45
@buffalojoec buffalojoec requested a review from joncinque April 9, 2026 01:45
@buffalojoec buffalojoec requested a review from Lichtso April 9, 2026 08:48
Copy link
Copy Markdown

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

This is a bit tough, since we don't have any other examples of a CLI command that requires a signer, but the underlying instruction doesn't. We have set-upgrade-authority which defaults to the checked version, and stake-authorize-checked to explicitly perform the check.

The signer check doesn't do much on just the CLI side -- someone could just pass the upgrade authority as a pubkey (null signer) and it would work, for example. For that reason, I would vote to just remove it altogether

@buffalojoec
Copy link
Copy Markdown
Author

I would vote to just remove it altogether

Done.

@buffalojoec buffalojoec force-pushed the cli-delete-extend-program-checked branch 2 times, most recently from f4b3263 to 40a913e Compare April 10, 2026 02:14
@buffalojoec buffalojoec changed the title cli: remove ExtendProgramChecked, add --allow-permissionless-extend cli: program: remove ExtendProgramChecked Apr 10, 2026
@buffalojoec buffalojoec force-pushed the cli-delete-extend-program-checked branch from 40a913e to f05978b Compare April 10, 2026 03:06
@buffalojoec buffalojoec force-pushed the cli-delete-extend-program-checked branch from f05978b to a1c8a6d Compare April 10, 2026 03:13
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.

3 participants