Skip to content

Conversation

@Saurabh7019
Copy link
Contributor

Closes #6734

@Saurabh7019 Saurabh7019 marked this pull request as ready for review December 12, 2025 18:54
Copilot AI review requested due to automatic review settings December 12, 2025 18:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new command spe container recyclebinitem remove that permanently removes a deleted container from the recycle bin. The implementation follows the established patterns from similar commands like container-recyclebinitem-restore and container-remove, providing consistent options for identifying containers by ID or name, with support for container type identification.

Key changes:

  • New command implementation with confirmation prompt and force flag support
  • Comprehensive test coverage with 259 lines of tests covering all scenarios
  • Documentation page with usage examples and permission details

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/m365/spe/commands/container/container-recyclebinitem-remove.ts New command implementation that permanently deletes containers from the recycle bin using DELETE request to the Graph API
src/m365/spe/commands/container/container-recyclebinitem-remove.spec.ts Comprehensive test suite covering validation, confirmation prompts, ID/name-based removal, error handling, and multiple result scenarios
src/m365/spe/commands.ts Command registration adding the new CONTAINER_RECYCLEBINITEM_REMOVE constant
docs/src/config/sidebars.ts Sidebar navigation update to include the new command documentation
docs/docs/cmd/spe/container/container-recyclebinitem-remove.mdx Reference documentation with options, permissions, examples, and usage instructions

@milanholemans
Copy link
Contributor

Thanks, we'll try to review it soon!

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.

New command: spe container recyclebinitem remove

2 participants