assets-controllers: Remove AllowedActions, AllowedEvents (#4344)

These types were mistakenly exported in a previous commit when
converting NftDetectionController to BaseController v2. They can still
be exports in `NftDetectionController.ts`, but just not be exports from
the perspective of the whole `assets-controllers` package. This commit
takes these exports away by replacing the `*` export in `index.ts` with
explicit names and ensures not to include the two aforementioned types.

packages/assets-controllers/src/index.ts

@@ -13,7 +13,30 @@ export type {
   NftMetadata,
 } from './NftController';
 export { getDefaultNftControllerState, NftController } from './NftController';
-export * from './NftDetectionController';
+export type {
+  NftDetectionControllerMessenger,
+  ApiNft,
+  ApiNftContract,
+  ApiNftLastSale,
+  ApiNftCreator,
+  ReservoirResponse,
+  TokensResponse,
+  BlockaidResultType,
+  Blockaid,
+  Market,
+  TokenResponse,
+  TopBid,
+  LastSale,
+  FeeBreakdown,
+  Attributes,
+  Collection,
+  Royalties,
+  Ownership,
+  FloorAsk,
+  Price,
+  Metadata,
+} from './NftDetectionController';
+export { NftDetectionController } from './NftDetectionController';
 export type {
   TokenBalancesControllerMessenger,
   TokenBalancesControllerActions,