api types [nfc]: Un-indirect some imports

This is a better pattern for reducing import cycles.  The
explicitness also seems helpful to the reader, though there
isn't a big difference.

Author: gnprice

src/api/initialDataTypes.js

@@ -1,8 +1,8 @@
 /* @flow strict-local */
 
-import type { CustomProfileField, UserStatusUpdate, UserSettings } from './modelTypes';
 import type {
   CrossRealmBot,
+  CustomProfileField,
   MutedTopicTuple,
   MutedUser,
   PresenceSnapshot,
@@ -15,7 +15,9 @@ import type {
   User,
   UserGroup,
   UserId,
-} from './apiTypes';
+  UserSettings,
+  UserStatusUpdate,
+} from './modelTypes';
 import type {
   CreatePublicOrPrivateStreamPolicyT,
   CreateWebPublicStreamPolicy,