chore: finish biome migration (#21)

Author: erikbrinkman

.gitignore

@@ -3,7 +3,6 @@
 coverage
 node_modules
 *.tsbuildinfo
-.eslintcache
 
 # Artifacts #
 #############

eslint.config.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-console */
-
 import { mkdir } from "node:fs/promises";
 import { argv } from "bun";
 import yargs from "yargs";

prettier.config.js

@@ -6,7 +6,6 @@ const encoder = new TextEncoder();
 const decoder = new TextDecoder();
 
 // from https://en.wikipedia.org/wiki/MIME#Multipart_messages
-// eslint-disable-next-line spellcheck/spell-checker
 const example = `MIME-Version: 1.0
 Subject: =?iso-8859-1?Q?=A1Hola,_se=F1or!?=
 Content-Type: multipart/mixed; boundary=frontier
@@ -39,7 +38,6 @@ Content-Type: multipart/related;
 ------MultipartBoundary--NYswbLinUCqE8KaJecg8DEV6giqFeyGLtHeT0qLB4h------
 `;
 
-// eslint-disable-next-line @typescript-eslint/require-await
 async function* stringToStream(
   file: string,
 ): AsyncIterableIterator<Uint8Array> {
@@ -62,7 +60,6 @@ describe("parseMhtml()", () => {
     const expectedHeaders = [
       {
         "MIME-Version": "1.0",
-        // eslint-disable-next-line spellcheck/spell-checker
         Subject: "¡Hola, señor!",
         "Content-Type": "multipart/mixed; boundary=frontier",
       },
@@ -86,7 +83,6 @@ describe("parseMhtml()", () => {
   });
 
   test("other headers", async () => {
-    // eslint-disable-next-line spellcheck/spell-checker
     const content = `MIME-Version: 1.0
 From: this is a wrapped: header
   with an extra delimiter in: both sections
@@ -125,15 +121,13 @@ This is a message with multiple parts in MIME format.
     const headers = files.map(({ headers }) => Object.fromEntries(headers));
     const expectedHeaders = [
       {
-        // eslint-disable-next-line spellcheck/spell-checker
         "Content-Type": `multipart/related;type="text/html";boundary="----MultipartBoundary--NYswbLinUCqE8KaJecg8DEV6giqFeyGLtHeT0qLB4h----"`,
         Date: "Sat, 16 Apr 2022 17:48:31 -0000",
         From: "<Saved by Blink>",
         "MIME-Version": "1.0",
         "Snapshot-Content-Location":
           "https://www.newyorker.com/culture/cultural-comment/what-the-twilight-zone-reveals-about-todays-prestige-tv",
         Subject:
-          // eslint-disable-next-line spellcheck/spell-checker
           "What “The Twilight Zone” Reveals About Today’s Prestige TV | The New Yorker",
       },
     ];
@@ -144,12 +138,10 @@ This is a message with multiple parts in MIME format.
   });
 
   test("fails non-ascii in q-encoding", async () => {
-    // eslint-disable-next-line spellcheck/spell-checker
     const content = `MIME-Version: 1.0
 Subject: =?iso-8859-1?Q?=A1Hola,\xffse=F1or!?=
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "got non-ascii character when decoding q-quoted word",
     );
@@ -237,7 +229,6 @@ This is a message with multiple parts in MIME format.
 Subject: =?utf-8?Q?a=ZZb?=
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "got invalid hex escape when decoding q-quoted word",
     );
@@ -246,7 +237,6 @@ Subject: =?utf-8?Q?a=ZZb?=
   test("fails without empty header delimiter", async () => {
     const content = `MIME-Version: 1.0`;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "didn't find an empty line to signify the end of header parsing",
     );
@@ -258,7 +248,6 @@ invalid header
 
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "header line didn't have key-value delimiter",
     );
@@ -269,7 +258,6 @@ invalid header
 
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "first headers didn't contain a content type",
     );
@@ -281,7 +269,6 @@ Content-Type: text/plain; boundary=frontier
 
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "first content type header didn't contain",
     );
@@ -293,7 +280,6 @@ Content-Type: multipart/mixed
 
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "first content type header didn't contain",
     );
@@ -306,7 +292,6 @@ Content-Type: multipart/mixed; boundary=frontier
 
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "unhandled encoding type: unknown",
     );
@@ -319,7 +304,6 @@ Content-Type: multipart/mixed; boundary=frontier
 This is a message with multiple parts in MIME format.
 `;
     const parser = parseMhtml(stringToStream(content));
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(consume(parser)).rejects.toThrow(
       "stream didn't end with the appropriate termination boundary",
     );

src/cli.ts

@@ -86,7 +86,6 @@ async function parseHeaders(
   let key = "";
   let val = "";
   for (;;) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
     const { done, value } = await iter.next();
     if (done) {
       throw new Error(
@@ -143,7 +142,6 @@ const defaultDecoders = new Map<string, Decoder>([
   ["binary", decodeBinary],
 ]);
 
-// eslint-disable-next-line spellcheck/spell-checker
 /**
  * extract the boundary condition from a multipart header
  *
@@ -187,7 +185,6 @@ function getBoundary(headers: MhtmlHeaders): [Uint8Array, Uint8Array] {
 
 /** options for mhtml parsing */
 export interface ParseOptions {
-  // eslint-disable-next-line spellcheck/spell-checker
   /**
    * custom decoders keyed by (lowercased) Content-Transfer-Encoding
    *
@@ -234,7 +231,6 @@ export async function* parseMhtml(
   let bound = null;
   let cont = true;
 
-  // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
   while (cont) {
     // parse out headers and get encoding for content
     const headers = await parseHeaders(lines);
@@ -255,7 +251,6 @@ export async function* parseMhtml(
         [Symbol.asyncIterator]() {
           return {
             async next(): Promise<IteratorResult<Uint8Array>> {
-              // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
               const { done, value } = await lines.next();
               if (done) {
                 throw new Error(

src/index.spec.ts

@@ -12,7 +12,6 @@ import {
 const encoder = new TextEncoder();
 const decoder = new TextDecoder();
 
-// eslint-disable-next-line @typescript-eslint/require-await
 async function* toAsyncIterable<T>(items: Iterable<T>): AsyncIterable<T> {
   for (const item of items) {
     yield item;
@@ -148,7 +147,6 @@ test("collect()", async () => {
 
 describe("decodeQuotedPrintable()", () => {
   test("success", async () => {
-    // eslint-disable-next-line spellcheck/spell-checker
     const input = ["key=3Dvalue", "this line continues =", "on the next line"];
     const res = decoder.decode(
       await collect(
@@ -163,7 +161,6 @@ describe("decodeQuotedPrintable()", () => {
   });
 
   test("normalizes to a custom separator when given one", async () => {
-    // eslint-disable-next-line spellcheck/spell-checker
     const input = ["key=3Dvalue", "this line continues =", "on the next line"];
     const res = decoder.decode(
       await collect(
@@ -183,7 +180,6 @@ describe("decodeQuotedPrintable()", () => {
     const decoded = decodeQuotedPrintable(
       toAsyncIterable(input.map((l) => encoder.encode(l))),
     );
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(
       (async () => {
         for await (const _ of decoded) {
@@ -198,7 +194,6 @@ describe("decodeQuotedPrintable()", () => {
     const decoded = decodeQuotedPrintable(
       toAsyncIterable(input.map((l) => encoder.encode(l))),
     );
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(
       (async () => {
         for await (const _ of decoded) {
@@ -213,7 +208,6 @@ describe("decodeQuotedPrintable()", () => {
     const decoded = decodeQuotedPrintable(
       toAsyncIterable(input.map((l) => encoder.encode(l))),
     );
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression,@typescript-eslint/await-thenable
     await expect(
       (async () => {
         for await (const _ of decoded) {