Skip to content

Conversation

@Vr00mm
Copy link

@Vr00mm Vr00mm commented Dec 12, 2025

Pull request template

Description

  • Migrated from deprecated commander API to v12 Command() pattern
  • Fixed lib/kms/utilities.js createEncryptedBucket() function
  • Fixed lib/nfs/utilities.js createBucketWithNFSEnabled() function
  • Fixed lib/utapi/utilities.js listMetrics() function
  • Fixed bin/search_bucket.js searchBucket() function

Changes:

  • Replace 'commander.version().option()' with 'new commander.Command()'
  • Replace direct commander property access with 'program.opts()'
  • Update all commander.xyz references to program.opts().xyz

Motivation and context

This fixes TypeError: commander.version is not a function when using create_encrypted_bucket.js and other utilities with commander v12.

@Vr00mm Vr00mm marked this pull request as draft December 12, 2025 10:32
- Migrated from deprecated commander API to v12 Command() pattern
- Fixed lib/kms/utilities.js createEncryptedBucket() function
- Fixed lib/nfs/utilities.js createBucketWithNFSEnabled() function
- Fixed lib/utapi/utilities.js listMetrics() function
- Fixed bin/search_bucket.js searchBucket() function

Changes:
- Replace 'commander.version().option()' with 'new commander.Command()'
- Replace direct commander property access with 'program.opts()'
- Update all commander.xyz references to program.opts().xyz

This fixes TypeError: commander.version is not a function when using
create_encrypted_bucket.js and other utilities with commander v12.
@Vr00mm Vr00mm force-pushed the fix/commander-v12-compatibility branch from ee2e5f8 to 92834fe Compare December 12, 2025 11:00
@Vr00mm Vr00mm marked this pull request as ready for review December 12, 2025 11:03
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