feat: upgraded typescript to v5.9 (#6068)

ioanatflowcrypt · web-flow · commit ef7115af4287 · 2025-08-28T13:42:12.000+03:00
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -98,6 +98,7 @@ const commonConfig = {
     '@typescript-eslint/consistent-type-definitions': 'off',
     '@typescript-eslint/dot-notation': 'error',
     '@typescript-eslint/explicit-function-return-type': 'off',
+    '@typescript-eslint/no-unnecessary-type-conversion': 'warn',
     '@typescript-eslint/explicit-member-accessibility': [
       'error',
       {
diff --git a/extension/chrome/elements/add_pubkey.ts b/extension/chrome/elements/add_pubkey.ts
@@ -69,7 +69,7 @@ View.run(
               await Ui.modal.warning(`some keys could not be processed due to errors:\n${errs.map(e => `-> ${e.message}\n`).join('')}`);
             }
             $('.copy_from_email').val('');
-            $('.pubkey').val(String(KeyUtil.armor(keys[0])));
+            $('.pubkey').val(KeyUtil.armor(keys[0]));
             $('.action_ok').trigger('click');
           } else if (errs.length) {
             await Ui.modal.error(`error processing public keys:\n${errs.map(e => `-> ${e.message}\n`).join('')}`);
diff --git a/extension/chrome/elements/compose-modules/compose-quote-module.ts b/extension/chrome/elements/compose-modules/compose-quote-module.ts
@@ -211,7 +211,7 @@ export class ComposeQuoteModule extends ViewModule<ComposeView> {
     }
     const text = this.messageToReplyOrForward.text;
     const from = Str.parseEmail(this.messageToReplyOrForward.headers.from || '').email;
-    const date = new Date(String(this.messageToReplyOrForward.headers.date));
+    const date = new Date(this.messageToReplyOrForward.headers.date);
     const dateStr = Str.fromDate(date).replace(' ', ' at ');
     const rtl = new RegExp('[' + Str.rtlChars + ']').exec(text);
     const dirAttr = `dir="${rtl ? 'rtl' : 'ltr'}"`;
diff --git a/extension/chrome/elements/compose-modules/compose-recipients-module.ts b/extension/chrome/elements/compose-modules/compose-recipients-module.ts
@@ -300,7 +300,7 @@ export class ComposeRecipientsModule extends ViewModule<ComposeView> {
       this.view.S.now('send_btn_text').text(this.BTN_LOADING);
       this.view.sizeModule.setInputTextHeightManuallyIfNeeded();
       recipientEl.evaluating = (async () => {
-        this.view.errModule.debug(`evaluateRecipients.evaluat.recipient.email(${String(recipientEl.email)})`);
+        this.view.errModule.debug(`evaluateRecipients.evaluat.recipient.email(${recipientEl.email})`);
         this.view.errModule.debug(`evaluateRecipients.evaluating.recipient.status(${recipientEl.status})`);
         this.view.errModule.debug(`evaluateRecipients.evaluating: calling getUpToDatePubkeys`);
         const info = await this.view.storageModule.getUpToDatePubkeys(recipientEl.email);
diff --git a/extension/chrome/elements/pgp_block_modules/pgp-block-attachments-module.ts b/extension/chrome/elements/pgp_block_modules/pgp-block-attachments-module.ts
@@ -30,19 +30,19 @@ export class PgpBlockViewAttachmentsModule {
       const size = filesize(attachments[i].length);
 
       const htmlContent = `<b>${Xss.escape(nameVisible)}</b>&nbsp;&nbsp;&nbsp;${size}<span class="progress"><span class="percent"></span></span>`;
-      const attachment = $(`<a href="#" index="${Number(i)}">`);
+      const attachment = $(`<a href="#" index="${i}">`);
       attachment.attr('title', name);
       Xss.sanitizeAppend(attachment, htmlContent);
       if (isEncrypted) {
         attachment.addClass('preview-attachment');
         attachment.attr('data-test', 'preview-attachment');
         attachment.append(
-          `<button class="download-attachment" data-test="download-attachment-${Number(i)}" index="${Number(
+          `<button class="download-attachment" data-test="download-attachment-${i}" index="${
             i
-          )}" title="DOWNLOAD"><img src="/img/svgs/download-link-green.svg"></button>`
+          }" title="DOWNLOAD"><img src="/img/svgs/download-link-green.svg"></button>`
         ); // xss-escaped
       } else {
-        attachment.attr('data-test', `download-attachment-${Number(i)}`);
+        attachment.attr('data-test', `download-attachment-${i}`);
         attachment.addClass('download-attachment');
       }
       $('#attachments').append(attachment); // xss-escaped
diff --git a/extension/js/common/api/email-provider/gmail/gmail.ts b/extension/js/common/api/email-provider/gmail/gmail.ts
@@ -51,7 +51,7 @@ export class Gmail extends EmailProviderApi implements EmailProviderInterface {
       method: 'GET',
       data: {
         labelIds: labelId !== 'ALL' ? labelId : undefined,
-        includeSpamTrash: Boolean(labelId === 'SPAM' || labelId === 'TRASH'),
+        includeSpamTrash: labelId === 'SPAM' || labelId === 'TRASH',
         // pageToken: page_token,
         // q,
         // maxResults
diff --git a/extension/js/common/api/shared/api-error.ts b/extension/js/common/api/shared/api-error.ts
@@ -101,7 +101,7 @@ export class AjaxErr extends ApiCallErr {
       const redactedPayload = AjaxErr.redactSensitiveData(Catch.stringify(req.data).substring(0, 1000));
       stack += `\n\nresponseText(0, 1000):\n${redactedRes}\n\npayload(0, 1000):\n${redactedPayload}`;
     }
-    const message = `${String(xhr.statusText || '(no status text)')}: ${String(xhr.status || -1)} when ${ApiCallErr.describeApiAction(req)} -> ${
+    const message = `${xhr.statusText || '(no status text)'}: ${xhr.status || -1} when ${ApiCallErr.describeApiAction(req)} -> ${
       resMsg || '(no standard err msg)'
     }`;
     return new AjaxErr(message, stack, status, CatchHelper.censoredUrl(req.url), responseText, xhr.statusText || '(no status text)', resMsg, resDetails);
diff --git a/extension/js/common/browser/browser.ts b/extension/js/common/browser/browser.ts
@@ -10,7 +10,7 @@ import { BgUtils } from '../../service_worker/bgutils.js';
 
 export class Browser {
   public static objUrlCreate = (content: Uint8Array | string) => {
-    return URL.createObjectURL(new Blob([content], { type: 'application/octet-stream' }));
+    return URL.createObjectURL(new Blob([content as BlobPart], { type: 'application/octet-stream' }));
   };
 
   public static saveToDownloads = (attachment: Attachment) => {
diff --git a/extension/js/common/core/crypto/smime/smime-key.ts b/extension/js/common/core/crypto/smime/smime-key.ts
@@ -107,7 +107,7 @@ export class SmimeKey {
       }
       p7.addRecipient(certificate);
     }
-    p7.content = this.forge.util.createBuffer(input);
+    p7.content = this.forge.util.createBuffer(new Uint8Array(input));
     p7.encrypt();
     let data: Uint8Array;
     if (armor) {
@@ -151,7 +151,7 @@ export class SmimeKey {
         type: this.forge.pki.oids.messageDigest
       }] */
     });
-    p7.content = this.forge.util.createBuffer(data);
+    p7.content = this.forge.util.createBuffer(new Uint8Array(data));
     p7.sign();
     return SmimeKey.messageToDer(p7);
   }
@@ -413,7 +413,7 @@ export class SmimeKey {
     if (!eku) {
       return false;
     }
-    return !!(eku as { emailProtection: boolean }).emailProtection;
+    return (eku as { emailProtection: boolean }).emailProtection;
   }
 
   private static dateToNumber(date: Date): undefined | number {
diff --git a/extension/js/common/core/mime.ts b/extension/js/common/core/mime.ts
@@ -192,7 +192,7 @@ export class Mime {
     const parser = new MimeParser();
     const leafNodes: { [key: string]: MimeParserNode } = {};
     parser.onbody = (node: MimeParserNode) => {
-      const path = String(node.path.join('.'));
+      const path = node.path.join('.');
       if (typeof leafNodes[path] === 'undefined') {
         leafNodes[path] = node;
       }
@@ -343,7 +343,7 @@ export class Mime {
     let from: string | undefined;
     const getHdrValAsArr = (hdr: MimeContentHeader) =>
       typeof hdr === 'string' ? ([hdr].map(h => Str.parseEmail(h).email).filter(e => !!e) as string[]) : hdr.map(h => h.address);
-    const getHdrValAsStr = (hdr: MimeContentHeader) => Str.parseEmail((Array.isArray(hdr) ? hdr[0]?.address : String(hdr || '')) || '').email;
+    const getHdrValAsStr = (hdr: MimeContentHeader) => Str.parseEmail((Array.isArray(hdr) ? hdr[0]?.address : hdr || '') || '').email;
     for (const hdrName of headersNames) {
       const header = parsedMimeMsg.headers[hdrName];
       if (header) {
@@ -427,13 +427,13 @@ export class Mime {
     if (node.headers['content-disposition']?.[0]) {
       const header = node.headers['content-disposition'][0];
       if (header.params?.filename) {
-        return String(header.params.filename);
+        return header.params.filename;
       }
     }
     if (node.headers['content-type']?.[0]) {
       const header = node.headers['content-type'][0];
       if (header.params?.name) {
-        return String(header.params.name);
+        return header.params.name;
       }
     }
     return;
diff --git a/extension/js/common/message-renderer.ts b/extension/js/common/message-renderer.ts
@@ -740,6 +740,7 @@ export class MessageRenderer {
         signerEmail,
         verificationPubs,
         result.content,
+        // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-conversion
         !!result.isEncrypted,
         result.signature,
         renderModule,
diff --git a/extension/js/common/platform/catch.ts b/extension/js/common/platform/catch.ts
@@ -258,7 +258,7 @@ export class Catch {
       thrown.stack += Catch.formattedStackBlock('Catch.reportErr calling stack', CatchHelper.stackTrace());
       if (thrown.hasOwnProperty('workerStack')) {
         // https://github.com/openpgpjs/openpgpjs/issues/656#event-1498323188
-        thrown.stack += Catch.formattedStackBlock('openpgp.js worker stack', String((thrown as Error & { workerStack: string }).workerStack));
+        thrown.stack += Catch.formattedStackBlock('openpgp.js worker stack', (thrown as Error & { workerStack: string }).workerStack);
       }
     }
     const exception = Catch.formExceptionFromThrown(thrown);
diff --git a/extension/js/common/platform/store/encryption-key-store.ts b/extension/js/common/platform/store/encryption-key-store.ts
@@ -4,17 +4,24 @@ import { storageGet, storageSet } from '../../browser/chrome.js';
 import { Buf } from '../../core/buf.js';
 import { secureRandomBytes } from '../util.js';
 
+const toArrayBufferStrict = (view: ArrayBufferView): ArrayBuffer => {
+  const ab = new ArrayBuffer(view.byteLength);
+  new Uint8Array(ab).set(new Uint8Array(view.buffer, view.byteOffset, view.byteLength));
+  return ab;
+};
+
 export class EncryptionKeyStore {
   private static key = 'cryptup_cryptokey' as const;
 
   public static async get(length?: number): Promise<ArrayBuffer> {
     const storageObj = await storageGet('local', [EncryptionKeyStore.key]);
     const value = storageObj[EncryptionKeyStore.key];
     if (!length || typeof value !== 'undefined') {
-      return Buf.fromBase64Str(value as string);
+      const buf = Buf.fromBase64Str(value as string); // Buf extends Uint8Array
+      return toArrayBufferStrict(buf);
     }
     const newKey = secureRandomBytes(length);
     await storageSet('local', { [EncryptionKeyStore.key]: new Buf(newKey).toBase64Str() });
-    return newKey;
+    return toArrayBufferStrict(newKey);
   }
 }
diff --git a/extension/js/common/ui/backup-ui/backup-ui.ts b/extension/js/common/ui/backup-ui/backup-ui.ts
@@ -147,7 +147,7 @@ export class BackupUi {
   private renderPrvKeysBackupSelection = async (kinfos: KeyInfoWithIdentity[]) => {
     for (const ki of kinfos) {
       // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-      const email = Xss.escape(String(ki.emails![0]));
+      const email = Xss.escape(ki.emails![0]);
       const dom = `
       <div class="mb-20">
         <div class="details">
diff --git a/extension/js/common/xss-safe-factory.ts b/extension/js/common/xss-safe-factory.ts
@@ -171,8 +171,8 @@ export class XssSafeFactory {
     const params: UrlParams = {
       isReplyBox: true,
       frameId: `frame_${Str.sloppyRandom(10)}`,
-      skipClickPrompt: Boolean(skipClickPrompt),
-      ignoreDraft: Boolean(ignoreDraft),
+      skipClickPrompt,
+      ignoreDraft,
       replyMsgId: convoParams.replyMsgId,
       draftId: convoParams.draftId,
       removeAfterClose: convoParams.removeAfterClose,
diff --git a/extension/js/content_scripts/webmail/generic/setup-webmail-content-script.ts b/extension/js/content_scripts/webmail/generic/setup-webmail-content-script.ts
@@ -463,7 +463,7 @@ export const contentScriptSetupIfVacant = async (webmailSpecific: WebmailSpecifi
       await webmailSpecific.start(acctEmail, clientConfiguration, inject, notifications, factory, relayManager);
     } catch (e) {
       if (e instanceof TabIdRequiredError) {
-        console.error(`FlowCrypt cannot start: ${String(e.message)}`);
+        console.error(`FlowCrypt cannot start: ${e.message}`);
       } else if (e instanceof Error && e.message === 'Extension context invalidated.') {
         console.info(`FlowCrypt cannot start: extension context invalidated. Destroying.`);
         win.destroy();
diff --git a/extension/js/service_worker/bg-handlers.ts b/extension/js/service_worker/bg-handlers.ts
@@ -175,7 +175,7 @@ export class BgHandlers {
     if (tab.id) {
       const message = await messenger.messageDisplay.getDisplayedMessage(tab.id);
       if (message?.id) {
-        return await messenger.messages.getFull(Number(message.id));
+        return await messenger.messages.getFull(message.id);
       }
     }
     return;
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json

Original file line number	Diff line number	Diff line change
`@@ -98,6 +98,7 @@ const commonConfig = {`
`98`	`98`	`'@typescript-eslint/consistent-type-definitions': 'off',`
`99`	`99`	`'@typescript-eslint/dot-notation': 'error',`
`100`	`100`	`'@typescript-eslint/explicit-function-return-type': 'off',`
	`101`	`+ '@typescript-eslint/no-unnecessary-type-conversion': 'warn',`
`101`	`102`	`'@typescript-eslint/explicit-member-accessibility': [`
`102`	`103`	`'error',`
`103`	`104`	`{`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ View.run(`
`69`	`69`	await Ui.modal.warning(`some keys could not be processed due to errors:\n${errs.map(e => `-> ${e.message}\n`).join('')}`);
`70`	`70`	`}`
`71`	`71`	`$('.copy_from_email').val('');`
`72`		`- $('.pubkey').val(String(KeyUtil.armor(keys[0])));`
	`72`	`+ $('.pubkey').val(KeyUtil.armor(keys[0]));`
`73`	`73`	`$('.action_ok').trigger('click');`
`74`	`74`	`} else if (errs.length) {`
`75`	`75`	await Ui.modal.error(`error processing public keys:\n${errs.map(e => `-> ${e.message}\n`).join('')}`);
Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ export class ComposeQuoteModule extends ViewModule<ComposeView> {`
`211`	`211`	`}`
`212`	`212`	`const text = this.messageToReplyOrForward.text;`
`213`	`213`	`const from = Str.parseEmail(this.messageToReplyOrForward.headers.from \|\| '').email;`
`214`		`- const date = new Date(String(this.messageToReplyOrForward.headers.date));`
	`214`	`+ const date = new Date(this.messageToReplyOrForward.headers.date);`
`215`	`215`	`const dateStr = Str.fromDate(date).replace(' ', ' at ');`
`216`	`216`	`const rtl = new RegExp('[' + Str.rtlChars + ']').exec(text);`
`217`	`217`	const dirAttr = `dir="${rtl ? 'rtl' : 'ltr'}"`;
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ export class AjaxErr extends ApiCallErr {`
`101`	`101`	`const redactedPayload = AjaxErr.redactSensitiveData(Catch.stringify(req.data).substring(0, 1000));`
`102`	`102`	stack += `\n\nresponseText(0, 1000):\n${redactedRes}\n\npayload(0, 1000):\n${redactedPayload}`;
`103`	`103`	`}`
`104`		- const message = `${String(xhr.statusText \|\| '(no status text)')}: ${String(xhr.status \|\| -1)} when ${ApiCallErr.describeApiAction(req)} -> ${
	`104`	+ const message = `${xhr.statusText \|\| '(no status text)'}: ${xhr.status \|\| -1} when ${ApiCallErr.describeApiAction(req)} -> ${
`105`	`105`	`resMsg \|\| '(no standard err msg)'`
`106`	`106`	}`;
`107`	`107`	`return new AjaxErr(message, stack, status, CatchHelper.censoredUrl(req.url), responseText, xhr.statusText \|\| '(no status text)', resMsg, resDetails);`
Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ export class SmimeKey {`
`107`	`107`	`}`
`108`	`108`	`p7.addRecipient(certificate);`
`109`	`109`	`}`
`110`		`- p7.content = this.forge.util.createBuffer(input);`
	`110`	`+ p7.content = this.forge.util.createBuffer(new Uint8Array(input));`
`111`	`111`	`p7.encrypt();`
`112`	`112`	`let data: Uint8Array;`
`113`	`113`	`if (armor) {`
`@@ -151,7 +151,7 @@ export class SmimeKey {`
`151`	`151`	`type: this.forge.pki.oids.messageDigest`
`152`	`152`	`}] */`
`153`	`153`	`});`
`154`		`- p7.content = this.forge.util.createBuffer(data);`
	`154`	`+ p7.content = this.forge.util.createBuffer(new Uint8Array(data));`
`155`	`155`	`p7.sign();`
`156`	`156`	`return SmimeKey.messageToDer(p7);`
`157`	`157`	`}`
`@@ -413,7 +413,7 @@ export class SmimeKey {`
`413`	`413`	`if (!eku) {`
`414`	`414`	`return false;`
`415`	`415`	`}`
`416`		`- return !!(eku as { emailProtection: boolean }).emailProtection;`
	`416`	`+ return (eku as { emailProtection: boolean }).emailProtection;`
`417`	`417`	`}`
`418`	`418`
`419`	`419`	`private static dateToNumber(date: Date): undefined \| number {`