refactor(server/utils): get rid of isString

let's use typeof x === "string" → works exactly the same and at the same speed as this custom isString fn

src/services/notes.ts

@@ -6,7 +6,7 @@ import eventService from "./events.js";
 import cls from "../services/cls.js";
 import protectedSessionService from "../services/protected_session.js";
 import log from "../services/log.js";
-import { newEntityId, isString, unescapeHtml, quoteRegex, toMap } from "../services/utils.js";
+import { newEntityId, unescapeHtml, quoteRegex, toMap } from "../services/utils.js";
 import revisionService from "./revisions.js";
 import request from "./request.js";
 import path from "path";
@@ -884,7 +884,7 @@ async function asyncPostProcessContent(note: BNote, content: string | Buffer) {
         return;
     }
 
-    if (note.hasStringContent() && !isString(content)) {
+    if (note.hasStringContent() && typeof content !== "string") {
         content = content.toString();
     }
 

src/services/utils.spec.ts

@@ -143,8 +143,6 @@ describe.todo("#normalize", () => {});
 
 describe.todo("#toMap", () => {});
 
-describe.todo("#isString", () => {});
-
 describe("#envToBoolean", () => {
     const testCases: TestCase<typeof utils.envToBoolean>[] = [
         ["w/ 'true' it should return boolean 'true'", ["true"], true],

src/services/utils.ts

@@ -295,10 +295,6 @@ export function toMap<T extends Record<string, any>>(list: T[], key: keyof T): R
     return map;
 }
 
-export function isString(x: any) {
-    return Object.prototype.toString.call(x) === "[object String]";
-}
-
 // try to turn 'true' and 'false' strings from process.env variables into boolean values or undefined
 export function envToBoolean(val: string | undefined) {
     if (val === undefined || typeof val !== "string") return undefined;
@@ -357,7 +353,6 @@ export default {
     normalize,
     hashedBlobId,
     toMap,
-    isString,
     getResourceDir,
     isMac,
     isWindows,