switch ott-server to use proper ott-common imports

common/package.json

@@ -2,8 +2,21 @@
   "name": "ott-common",
   "version": "0.10.0",
   "license": "AGPL-3.0-or-later",
-  "module": "es2020",
   "type": "module",
+  "module": "index.ts",
+  "files": [
+    "*"
+  ],
+  "exports": {
+    ".": {
+      "lean": "./ts-out/index.js",
+      "default": "./index.ts"
+    },
+    "./*": {
+      "lean": "./ts-out/*",
+      "default": "./*"
+    }
+  },
   "scripts": {
     "build": "tsc",
     "lint": "tsc --noEmit && eslint --ext .js,.ts --fix .",

server/admin.ts

@@ -1,4 +1,4 @@
-import { OttException } from "../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import { conf } from "./ott-config";
 
 /**

server/api/announce.ts

@@ -2,9 +2,9 @@ import { getLogger } from "../logger";
 import { conf } from "../ott-config";
 import express, { RequestHandler } from "express";
 import { redisClient } from "../redisclient";
-import { ANNOUNCEMENT_CHANNEL } from "../../common/constants";
-import { OttResponseBody } from "../../common/models/rest-api";
-import { OttException } from "../../common/exceptions";
+import { ANNOUNCEMENT_CHANNEL } from "ott-common/constants";
+import { OttResponseBody } from "ott-common/models/rest-api";
+import { OttException } from "ott-common/exceptions";
 import { BadApiArgumentException } from "../exceptions";
 
 const router = express.Router();

server/api/data.ts

@@ -1,8 +1,8 @@
 import { getLogger } from "../logger";
 import { conf } from "../ott-config";
 import express, { RequestHandler, ErrorRequestHandler } from "express";
-import { OttApiResponseAddPreview, OttResponseBody } from "../../common/models/rest-api";
-import { OttException } from "../../common/exceptions";
+import { OttApiResponseAddPreview, OttResponseBody } from "ott-common/models/rest-api";
+import { OttException } from "ott-common/exceptions";
 import { BadApiArgumentException } from "../exceptions";
 import InfoExtract from "../infoextractor";
 import { consumeRateLimitPoints } from "../rate-limit";

server/api/dev.ts

@@ -2,7 +2,7 @@
 import express from "express";
 import { rateLimiter } from "../rate-limit";
 import roommanager from "../roommanager";
-import { RoomRequestType } from "../../common/models/messages";
+import { RoomRequestType } from "ott-common/models/messages";
 import usermanager from "../usermanager";
 import faker from "faker";
 import tokens from "../auth/tokens";

server/api/room.ts

@@ -1,22 +1,22 @@
 import _ from "lodash";
 import { getLogger } from "../logger";
 import roommanager from "../roommanager";
-import { QueueMode, Visibility } from "../../common/models/types";
+import { QueueMode, Visibility } from "ott-common/models/types";
 import { consumeRateLimitPoints } from "../rate-limit";
 import { BadApiArgumentException, FeatureDisabledException } from "../exceptions";
-import { OttException } from "../../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import express, { RequestHandler, ErrorRequestHandler } from "express";
 import clientmanager from "../clientmanager";
 import {
 	ApplySettingsRequest,
 	RoomRequestType,
 	UndoRequest,
 	AddRequest,
-} from "../../common/models/messages";
+} from "ott-common/models/messages";
 import storage from "../storage";
-import { Grants } from "../../common/permissions";
-import { Video } from "../../common/models/video.js";
-import { ROOM_NAME_REGEX } from "../../common/constants";
+import { Grants } from "ott-common/permissions";
+import { Video } from "ott-common/models/video";
+import { ROOM_NAME_REGEX } from "ott-common/constants";
 import {
 	OttApiRequestAddToQueue,
 	OttApiRequestPatchRoom,
@@ -27,7 +27,7 @@
 	OttApiResponseRoomCreate,
 	OttApiResponseRoomGenerate,
 	OttResponseBody,
-} from "../../common/models/rest-api";
+} from "ott-common/models/rest-api";
 import { getApiKey } from "../admin";
 import { v4 as uuidv4 } from "uuid";
 import { counterHttpErrors } from "../metrics";
@@ -431,7 +431,7 @@
 	} else {
 		throw new BadApiArgumentException("service,id", "missing");
 	}
-	await room.processUnauthorizedRequest(roomRequest, { token: req.token });
+	await room.processUnauthorizedRequest(roomRequest, { token: req.token! });
 	res.json({
 		success: true,
 	});
@@ -454,7 +454,7 @@
 				type: RoomRequestType.RemoveRequest,
 				video: { service: req.body.service, id: req.body.id },
 			},
-			{ token: req.token }
+			{ token: req.token! }
 		);
 		res.json({
 			success: true,

server/auth/index.ts

@@ -3,10 +3,10 @@ import express from "express";
 import tokens, { SessionInfo } from "./tokens";
 import { uniqueNamesGenerator } from "unique-names-generator";
 import passport from "passport";
-import { AuthToken, MySession } from "../../common/models/types";
+import { AuthToken, MySession } from "ott-common/models/types";
 import nocache from "nocache";
 import usermanager from "../usermanager";
-import { OttException } from "../../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import { requireApiKey } from "../admin";
 
 const router = express.Router();
@@ -55,7 +55,7 @@ export async function authTokenMiddleware(
 		req.token = token;
 	}
 
-	if (!(await tokens.validate(req.token))) {
+	if (!req.token || !(await tokens.validate(req.token))) {
 		res.status(400).json({
 			success: false,
 			error: {

server/auth/tokens.ts

@@ -1,6 +1,6 @@
 import crypto from "crypto";
 import { redisClient } from "../redisclient";
-import { AuthToken } from "../../common/models/types";
+import { AuthToken } from "ott-common/models/types";
 
 const PREFIX = "auth";
 const EXPIRATION_TIME = 14 * 24 * 60 * 60; // 14 days in seconds

server/balancer.ts

@@ -4,9 +4,9 @@
 
 import { getLogger } from "./logger";
 import { conf } from "./ott-config";
-import { Result, err, ok, intoResult } from "../common/result";
-import { AuthToken, ClientId } from "../common/models/types";
-import { replacer } from "../common/serialize";
+import { Result, err, ok, intoResult } from "ott-common/result";
+import { AuthToken, ClientId } from "ott-common/models/types";
+import { replacer } from "ott-common/serialize";
 import { OttWebsocketError } from "ott-common/models/types";
 import roommanager from "./roommanager";
 import type { RoomListItem } from "./api/room";
@@ -40,7 +40,7 @@
 		balancerManager.addBalancerConnection(conn);
 	});
 	wss.on("error", error => {
 		log.error(`Balancer websocket error: ${error}`);
 	});
 	wss.on("listening", () => {
 		log.info(
@@ -99,7 +99,7 @@
 	}
 
 	private onBalancerError(conn: BalancerConnection, error: WebSocket.ErrorEvent) {
 		log.error(`Error from balancer ${conn.id}: ${error}`);
 		this.emit("error", conn, error);
 	}
 
@@ -216,7 +216,7 @@
 			}
 			this.emit("message", result.value);
 		} else {
 			log.error(`Error parsing incoming balancer message: ${result.value} - ${data}`);
 		}
 	}
 
@@ -276,7 +276,7 @@
 	const result = await roommanager.getRoom(roomName, { mustAlreadyBeLoaded: true });
 	if (!result.ok) {
 		log.error(
 			`Failed to grab room that should have been loaded. Can't inform balancers. room=${roomName}: ${result.value}`
 		);
 		return;
 	}

server/client.ts

@@ -1,13 +1,13 @@
-import { AuthToken, ClientId, ClientInfo, OttWebsocketError } from "../common/models/types";
-import type { ClientMessage, ServerMessage } from "../common/models/messages";
+import { AuthToken, ClientId, ClientInfo, OttWebsocketError } from "ott-common/models/types";
+import type { ClientMessage, ServerMessage } from "ott-common/models/messages";
 import WebSocket from "ws";
 import { SessionInfo, setSessionInfo } from "./auth/tokens";
 import { v4 as uuidv4 } from "uuid";
 import EventEmitter from "events";
 import { getLogger } from "./logger";
 import { getSessionInfo } from "./auth/tokens";
 import { BalancerConnection, BalancerConnectionReal } from "./balancer";
-import { replacer } from "../common/serialize";
+import { replacer } from "ott-common/serialize";
 
 const log = getLogger("client");
 
@@ -141,7 +141,7 @@
 	}
 
 	onError(err: Error) {
 		log.error(`Error on socket for client ${this.id}: ${err}`);
 	}
 
 	sendRaw(msg: string) {

server/clientmanager.ts

@@ -14,19 +14,19 @@
 	ServerMessageSync,
 	ServerMessageUser,
 	ServerMessageYou,
-} from "../common/models/messages";
+} from "ott-common/models/messages";
 import { ClientNotFoundInRoomException, MissingToken } from "./exceptions";
-import { MySession, OttWebsocketError, AuthToken, ClientId } from "../common/models/types";
+import { MySession, OttWebsocketError, AuthToken, ClientId } from "ott-common/models/types";
 import roommanager from "./roommanager";
-import { ANNOUNCEMENT_CHANNEL, ROOM_NAME_REGEX } from "../common/constants";
+import { ANNOUNCEMENT_CHANNEL, ROOM_NAME_REGEX } from "ott-common/constants";
 import tokens, { SessionInfo } from "./auth/tokens";
 import { RoomStateSyncable } from "./room";
 import { Gauge } from "prom-client";
-import { replacer } from "../common/serialize";
+import { replacer } from "ott-common/serialize";
 import { Client, ClientJoinStatus, DirectClient, BalancerClient } from "./client";
 import { BalancerConnection, MsgB2M, balancerManager, initBalancerConnections } from "./balancer";
 import usermanager from "./usermanager";
-import { OttException } from "../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import { conf } from "./ott-config";
 
 const log = getLogger("clientmanager");
@@ -344,7 +344,7 @@
 }
 
 function onBalancerError(conn: BalancerConnection, error: WebSocket.ErrorEvent) {
 	log.error(`Error from balancer ${conn.id}: ${error}`);
 }
 
 async function makeRoomRequest(client: Client, request: RoomRequest): Promise<void> {

server/exceptions.ts

@@ -1,5 +1,5 @@
 import { URL } from "url";
-import { OttException } from "../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 
 // export type OttException = UnsupportedServiceException | InvalidAddPreviewInputException | OutOfQuotaException | InvalidVideoIdException | FeatureDisabledException | UnsupportedMimeTypeException | LocalFileException | MissingMetadataException | IncompleteServiceAdapterException | PermissionDeniedException | ImpossiblePromotionException | InvalidRoleException | RoomNotFoundException | RoomAlreadyLoadedException | RoomNameTakenException | VideoAlreadyQueuedException | VideoNotFoundException | BadApiArgumentException
 

server/express-types.d.ts

@@ -1,5 +1,5 @@
 import { User as UserAccount } from "./models/user";
-import { AuthToken } from "./common/models/types";
+import { AuthToken } from "ott-common/models/types";
 import { SessionInfo } from "./auth/tokens";
 import * as core from "express-serve-static-core";
 

server/infoextractor.ts

@@ -18,9 +18,9 @@ import {
 import { getLogger } from "./logger";
 import { redisClient } from "./redisclient";
 import { isSupportedMimeType } from "./mime";
-import { Video, VideoId, VideoMetadata, VideoService } from "../common/models/video";
+import { Video, VideoId, VideoMetadata, VideoService } from "ott-common/models/video";
 import { ServiceAdapter } from "./serviceadapter";
-import { OttException } from "../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import TubiAdapter from "./services/tubi";
 import { Counter } from "prom-client";
 import { conf } from "./ott-config";

server/mailer.ts

@@ -1,4 +1,4 @@
-import { Result, ok, err } from "../common/result";
+import { Result, ok, err } from "ott-common/result";
 import Mailjet, { Client as MailjetClient } from "node-mailjet";
 import { conf } from "./ott-config";
 import { getLogger } from "./logger";

server/models/cachedvideo.ts

@@ -1,5 +1,5 @@
-import { ALL_VIDEO_SERVICES } from "../../common/constants";
-import { VideoService } from "../../common/models/video";
+import { ALL_VIDEO_SERVICES } from "ott-common/constants";
+import { VideoService } from "ott-common/models/video";
 import { Sequelize, Model, DataTypes, Optional } from "sequelize";
 
 interface CachedVideoAttributes {

server/models/room.ts

@@ -1,9 +1,9 @@
 import { Sequelize, Model, DataTypes, Optional } from "sequelize";
-import { QueueMode, Visibility, Role, BehaviorOption } from "../../common/models/types";
+import { QueueMode, Visibility, Role, BehaviorOption } from "ott-common/models/types";
 import { User } from "./user";
-import { ALL_SKIP_CATEGORIES, ROOM_NAME_REGEX } from "../../common/constants";
-import type { OldRoleGrants, GrantMask } from "../../common/permissions";
-import { QueueItem } from "../../common/models/video";
+import { ALL_SKIP_CATEGORIES, ROOM_NAME_REGEX } from "ott-common/constants";
+import type { OldRoleGrants, GrantMask } from "ott-common/permissions";
+import { QueueItem } from "ott-common/models/video";
 import { Category } from "sponsorblock-api";
 
 export interface RoomAttributes {

server/ott-config.ts

@@ -4,8 +4,8 @@ import validator from "validator";
 import convict from "convict";
 import toml from "toml";
 import type winston from "winston";
-import { ALL_VIDEO_SERVICES } from "../common/constants";
-import { Result, err, ok, intoResult } from "../common/result";
+import { ALL_VIDEO_SERVICES } from "ott-common/constants";
+import { Result, err, ok, intoResult } from "ott-common/result";
 
 convict.addParser({ extension: "toml", parse: toml.parse });
 

server/package.json

@@ -6,7 +6,7 @@
   "module": "esnext",
   "scripts": {
     "start": "node --experimental-specifier-resolution=node --loader ts-node/esm app.ts",
-    "start-lean": "node --experimental-specifier-resolution=node --optimize-for-size --max-old-space-size=200 ts-out/server/app.js",
+    "start-lean": "node --experimental-specifier-resolution=node --optimize-for-size --max-old-space-size=200 --conditions=lean ts-out/app.js",
     "debug": "nodemon --exec node --experimental-specifier-resolution=node --loader ts-node/esm --ignore 'tests/*' --inspect -e ts,js app.ts",
     "build": "tsc",
     "lint": "tsc --noEmit && eslint --fix .",

server/room.ts

@@ -1,4 +1,4 @@
-import permissions, { GrantMask, Grants } from "../common/permissions";
+import permissions, { GrantMask, Grants } from "ott-common/permissions";
 import { redisClient } from "./redisclient";
 import { getLogger } from "./logger";
 import winston from "winston";
@@ -28,7 +28,7 @@ import {
 	ShuffleRequest,
 	PlaybackSpeedRequest,
 	KickRequest,
-} from "../common/models/messages";
+} from "ott-common/models/messages";
 import _ from "lodash";
 import InfoExtract from "./infoextractor";
 import usermanager from "./usermanager";
@@ -45,12 +45,12 @@ import {
 	RoomSettings,
 	AuthToken,
 	BehaviorOption,
-} from "../common/models/types";
+} from "ott-common/models/types";
 import { User } from "./models/user";
-import type { QueueItem, Video, VideoId } from "../common/models/video";
+import type { QueueItem, Video, VideoId } from "ott-common/models/video";
 import dayjs, { Dayjs } from "dayjs";
-import type { PickFunctions } from "../common/typeutils";
-import { replacer } from "../common/serialize";
+import type { PickFunctions } from "ott-common/typeutils";
+import { replacer } from "ott-common/serialize";
 import {
 	ClientNotFoundInRoomException,
 	ImpossiblePromotionException,
@@ -59,19 +59,19 @@ import {
 } from "./exceptions";
 import storage from "./storage";
 import tokens, { SessionInfo } from "./auth/tokens";
-import { OttException } from "../common/exceptions";
+import { OttException } from "ott-common/exceptions";
 import { getSponsorBlock } from "./sponsorblock";
 import { ResponseError as SponsorblockResponseError, Segment, Category } from "sponsorblock-api";
 import { VideoQueue } from "./videoqueue";
 import { Counter } from "prom-client";
 import roommanager from "./roommanager";
-import { calculateCurrentPosition } from "../common/timestamp";
-import { RestoreQueueRequest } from "../common/models/messages";
-import { countEligibleVoters, voteSkipThreshold } from "../common";
+import { calculateCurrentPosition } from "ott-common/timestamp";
+import { RestoreQueueRequest } from "ott-common/models/messages";
+import { countEligibleVoters, voteSkipThreshold } from "ott-common";
 import type { ClientManagerCommand } from "./clientmanager";
-import { canKickUser } from "../common/userutils";
+import { canKickUser } from "ott-common/userutils";
 import { conf } from "./ott-config";
-import { ALL_SKIP_CATEGORIES } from "../common/constants";
+import { ALL_SKIP_CATEGORIES } from "ott-common/constants";
 
 /**
  * Represents a User from the Room's perspective.

server/roommanager.ts

@@ -1,5 +1,5 @@
 import { Room, RoomState, RoomStateFromRedis, RoomStatePersistable } from "./room";
-import { AuthToken, Role, RoomOptions, Visibility } from "../common/models/types";
+import { AuthToken, Role, RoomOptions, Visibility } from "ott-common/models/types";
 import _ from "lodash";
 import { getLogger } from "./logger";
 import { redisClient } from "./redisclient";
@@ -9,11 +9,11 @@ import {
 	RoomNameTakenException,
 	RoomNotFoundException,
 } from "./exceptions";
-import { RoomRequest, RoomRequestContext, ServerMessage } from "../common/models/messages";
+import { RoomRequest, RoomRequestContext, ServerMessage } from "ott-common/models/messages";
 import { Gauge } from "prom-client";
 import { EventEmitter } from "events";
-import { Result, ok, err } from "../common/result";
-import { Grants } from "../common/permissions";
+import { Result, ok, err } from "ott-common/result";
+import { Grants } from "ott-common/permissions";
 import type { ClientManagerCommand } from "./clientmanager";
 
 export const log = getLogger("roommanager");

server/serviceadapter.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-unused-vars */
 /* eslint-disable no-unused-vars */
-import { Video, VideoId, VideoMetadata, VideoService } from "../common/models/video";
+import { Video, VideoId, VideoMetadata, VideoService } from "ott-common/models/video";
 import { IncompleteServiceAdapterException } from "./exceptions";
 import { getLogger } from "./logger";
 

server/services/dailymotion.ts

@@ -2,7 +2,7 @@ import { URL } from "url";
 import axios from "axios";
 import { ServiceAdapter } from "../serviceadapter";
 import { InvalidVideoIdException } from "../exceptions";
-import { Video } from "../../common/models/video";
+import { Video } from "ott-common/models/video";
 
 export default class DailyMotionAdapter extends ServiceAdapter {
 	api = axios.create({

server/services/dash.ts

@@ -8,7 +8,7 @@ import {
 } from "../exceptions";
 import { getMimeType, isSupportedMimeType } from "../mime";
 import { getLogger } from "../logger";
-import { Video } from "../../common/models/video";
+import { Video } from "ott-common/models/video";
 import { DashMPD } from "@liveinstantly/dash-mpd-parser";
 import axios from "axios";
 import { parseIso8601Duration } from "./parsing/iso8601";

server/services/direct.ts

@@ -9,7 +9,7 @@ import {
 import { getMimeType, isSupportedMimeType } from "../mime";
 import { FfprobeStrategy, OnDiskPreviewFfprobe, RunFfprobe, StreamFfprobe } from "../ffprobe";
 import { getLogger } from "../logger";
-import { Video } from "../../common/models/video";
+import { Video } from "ott-common/models/video";
 import { conf } from "../ott-config";
 
 const log = getLogger("direct");