```ruby
# encoding: utf-8

module Voicemail 
  class MailboxCleanerController < ApplicationController
    def run
      menu_prompt = metadata[:new_or_saved].to_s == "new" ? config.mailbox.clear_new_messages : config.mailbox.clear_saved_messages
      menu menu_prompt, timeout: config.menu_timeout, tries: config.menu_tries do
        match(1) { erase_all metadata[:new_or_saved] }
   
        invalid { confirm_no_action_taken }
        timeout { confirm_no_action_taken }
        failure { confirm_no_action_taken }
      end
    end
   
    def confirm_no_action_taken
      say config.mailbox.no_messages_deleted
      main_menu
    end
   
    def erase_all(type)
      method = "next_#{type}_message"
      mailbox = fetch_mailbox
   
      while message = storage.send(method, mailbox[:id])
        storage.delete_message mailbox[:id], message[:id]
      end
    end
  end
end```

Remove redundant assignment of `mailbox`
user
```ruby
# encoding: utf-8

module Voicemail 
  class MailboxCleanerController < ApplicationController
    def run
      menu_prompt = metadata[:new_or_saved].to_s == "new" ? config.mailbox.clear_new_messages : config.mailbox.clear_saved_messages
      menu menu_prompt, timeout: config.menu_timeout, tries: config.menu_tries do
        match(1) { erase_all metadata[:new_or_saved] }
   
        invalid { confirm_no_action_taken }
        timeout { confirm_no_action_taken }
        failure { confirm_no_action_taken }
      end
    end
   
    def confirm_no_action_taken
      say config.mailbox.no_messages_deleted
      main_menu
    end
   
    def erase_all(type)
      method = "next_#{type}_message"
   
      while message = storage.send(method, mailbox[:id])
        storage.delete_message mailbox[:id], message[:id]
      end
    end
  end
end```
assistant