As a non-controller with empty state, `TokenDetectionController` should not inherit from `BaseController`

## Requirements

- Should not inherit from `BaseControllerV1` or `BaseController`.
- Should not use the `Controller` suffix in its name.
- Types for `ControllerState`, `ControllerGetStateAction`, `ControllerStateChangeEvent` should not be defined or exported.
- Should accept `messenger` as a constructor option.
- `messenger` should be defined with allowlists that enable the class to make all communications with other controller classes that were previously supported.
- ~Should not accept any class method callbacks as constructor options. Messenger actions/events should be used to communicate instead.~
- ~In some cases, methods that implement API interactions should be extracted into a dedicated `service` class.~

Start by locating TokenDetectionController and its BaseControllerV1/BaseController references, then inspect the existing controller communications and Messenger allowlists. Done means the class no longer uses the controller inheritance, naming, or state types described here and can communicate with the previously supported controller classes through its messenger.

Written by the indexing model from the issue text.