chore: resolve review comments

src/models/server.ts

@@ -1,7 +1,6 @@
 import { Schema, model } from 'mongoose';
 import uniqueValidator from 'mongoose-unique-validator';
 import { IServer, IServerMethods, ServerModel } from '@/types/mongoose/server';
-import type { SystemType } from '@/types/common/token';
 
 const ServerSchema = new Schema<IServer, ServerModel, IServerMethods>({
 	client_id: String,
@@ -19,9 +18,4 @@ const ServerSchema = new Schema<IServer, ServerModel, IServerMethods>({
 
 ServerSchema.plugin(uniqueValidator, { message: '{PATH} already in use.' });
 
-export const Server = model<IServer, ServerModel>('Server', ServerSchema);
-
-export const serverDeviceToSystemType: Record<number, SystemType> = {
-	1: 'WIIU',
-	2: '3DS'
-};
+export const Server = model<IServer, ServerModel>('Server', ServerSchema);

src/services/grpc/api/login.ts

@@ -21,19 +21,19 @@ export async function login(request: LoginRequest): Promise<DeepPartial<LoginRes
 		if (!username) throw new ServerError(Status.INVALID_ARGUMENT, 'Invalid or missing username');
 		if (!password) throw new ServerError(Status.INVALID_ARGUMENT, 'Invalid or missing password');
 
-		pnid = await getPNIDByUsername(username); // * We know username will never be null here
+		pnid = await getPNIDByUsername(username);
 
 		if (!pnid) throw new ServerError(Status.INVALID_ARGUMENT, 'User not found');
 
-		const hashedPassword = nintendoPasswordHash(password, pnid.pid); // * We know password will never be null here
+		const hashedPassword = nintendoPasswordHash(password, pnid.pid);
 
 		if (!bcrypt.compareSync(hashedPassword, pnid.password)) {
 			throw new ServerError(Status.INVALID_ARGUMENT, 'Password is incorrect');
 		}
 	} else {
 		if (!refreshToken) throw new ServerError(Status.INVALID_ARGUMENT, 'Invalid or missing refresh token');
 
-		pnid = await getPNIDByAPIRefreshToken(refreshToken); // * We know refreshToken will never be null here
+		pnid = await getPNIDByAPIRefreshToken(refreshToken);
 
 		if (!pnid) throw new ServerError(Status.INVALID_ARGUMENT, 'Invalid or missing refresh token');
 	}

src/services/nnas/routes/provider.ts

@@ -4,7 +4,7 @@ import { getServerByClientID, getServerByGameServerID } from '@/database';
 import { generateToken, getValueFromHeaders, getValueFromQueryString } from '@/util';
 import { NEXAccount } from '@/models/nex-account';
 import { TokenOptions } from '@/types/common/token';
-import { serverDeviceToSystemType } from '@/models/server';
+import { serverDeviceToSystemType } from '@/types/common/token';
 
 const router = express.Router();
 

src/types/common/token.ts

@@ -19,6 +19,11 @@ export const SystemTypes = {
 } as const;
 export type SystemType = keyof typeof SystemTypes;
 
+export const serverDeviceToSystemType: Record<number, SystemType> = {
+	1: 'WIIU',
+	2: '3DS'
+};
+
 export function getSystemTypeFromValue(type: number): SystemType | undefined {
 	const keys = Object.keys(SystemTypes) as SystemType[];
 	return keys.find((key) => SystemTypes[key] === type);

src/util.ts

@@ -61,15 +61,15 @@ export function generateOAuthTokens(systemType: SystemType, pnid: HydratedPNIDDo
 		token_type: 'OAUTH_ACCESS',
 		pid: pnid.pid,
 		access_level: pnid.access_level,
-		expire_time: BigInt(Date.now() + (accessTokenExpiresInSecs * 1000)) // * 1 hour
+		expire_time: BigInt(Date.now() + (accessTokenExpiresInSecs * 1000))
 	};
 
 	const refreshTokenOptions: TokenOptions = {
 		system_type: systemType,
 		token_type: 'OAUTH_REFRESH',
 		pid: pnid.pid,
 		access_level: pnid.access_level,
-		expire_time: BigInt(Date.now() + (refreshTokenExpiresInSecs * 1000)) // * 30 days
+		expire_time: BigInt(Date.now() + (refreshTokenExpiresInSecs * 1000))
 	};
 
 	const accessToken = generateToken(config.aes_key, accessTokenOptions)?.toString('hex');