chore: add editorconfig-checker (#4312)

* chore(deps): add editorconfig-checker to devDependencies

* style: apply editorconfig-checker suggestions

* chore: unset charset and newline settings in `.editorconfig` for `.lockb` and `.pxm` files

* chore: add editorconfig-checker script to `package.json`

* test: update static response assertions to match optional newline

Author: 3w36zj6

.editorconfig

@@ -14,3 +14,8 @@ trim_trailing_whitespace = true
 indent_style = space
 indent_size = 2
 trim_trailing_whitespace = false
+
+[*.{lockb,pxm}]
+charset = unset
+insert_final_newline = unset
+end_of_line = unset

benchmarks/deno/.gitignore

@@ -1 +1 @@
-*.sqlite
+*.sqlite

benchmarks/http-server/.gitignore

@@ -1,2 +1,2 @@
 .benchmark-temp/
-benchmark-results.md
+benchmark-results.md

benchmarks/utils/.gitignore

@@ -1,2 +1,2 @@
 yarn.lock
-bun.lockb
+bun.lockb

benchmarks/webapp/.gitignore

@@ -1 +1 @@
-yarn.lock
+yarn.lock

bun.lock

@@ -14,6 +14,7 @@
         "@vitest/coverage-v8": "^3.0.5",
         "arg": "^5.0.2",
         "bun-types": "^1.1.39",
+        "editorconfig-checker": "^6.1.0",
         "esbuild": "^0.15.18",
         "eslint": "^9.10.0",
         "glob": "^11.0.0",
@@ -660,6 +661,8 @@
 
     "eastasianwidth": ["[email protected]", "", {}, "sha512-I88TYZWc9XiYHRQ4/3c5rjjfgkjhLyW2luGIheGERbNQ6OY7yTybanSpDXZa8y7VUP9YmDcYa+eyq4ca7iLqWA=="],
 
+    "editorconfig-checker": ["[email protected]", "", { "bin": { "ec": "dist/index.js", "editorconfig-checker": "dist/index.js" } }, "sha512-i25NwXlcWhadiO62AFUauucFPi9LwasoMRqoM8KkG3rJp6bx09mL/fDQpNMz2f6Nx/7EPVIZHK5/tIdztWzflA=="],
+
     "elegant-spinner": ["[email protected]", "", {}, "sha512-B+ZM+RXvRqQaAmkMlO/oSe5nMUOaUnyfGYCEHoR8wrXsZR2mA0XVibsxV1bvTwxdRWah1PkQqso2EzhILGHtEQ=="],
 
     "emoji-regex": ["[email protected]", "", {}, "sha512-MSjYzcWNOA0ewAHpz0MxpYFvwg6yjy1NG3xteoqz644VCo/RPgnr1/GGt+ic3iJTzQ8Eu3TdM14SawnVUmGE6A=="],

docs/MIGRATION.md

@@ -268,10 +268,10 @@ You do not have to use Body parse middleware to parse request body. Use `c.req.p
 
 ```ts
 // Parse Request body
- app.post('', (c) => {
-   const body = c.req.parseBody()
-   ...
- })
+app.post('', (c) => {
+  const body = c.req.parseBody()
+  ...
+})
 ```
 
 ### GraphQL Server middleware is obsolete

package.json

@@ -24,6 +24,7 @@
     "lint:fix": "eslint src runtime-tests build perf-measures benchmarks --fix",
     "format": "prettier --check --cache \"src/**/*.{js,ts,tsx}\" \"runtime-tests/**/*.{js,ts,tsx}\" \"build/**/*.{js,ts,tsx}\" \"perf-measures/**/*.{js,ts,tsx}\" \"benchmarks/**/*.{js,ts,tsx}\"",
     "format:fix": "prettier --write --cache --cache-strategy metadata \"src/**/*.{js,ts,tsx}\" \"runtime-tests/**/*.{js,ts,tsx}\" \"build/**/*.{js,ts,tsx}\" \"perf-measures/**/*.{js,ts,tsx}\" \"benchmarks/**/*.{js,ts,tsx}\"",
+    "editorconfig-checker": "editorconfig-checker",
     "copy:package.cjs.json": "cp ./package.cjs.json ./dist/cjs/package.json && cp ./package.cjs.json ./dist/types/package.json",
     "build": "bun run --shell bun remove-dist && bun ./build/build.ts && bun run copy:package.cjs.json",
     "postbuild": "publint",
@@ -665,6 +666,7 @@
     "@vitest/coverage-v8": "^3.0.5",
     "arg": "^5.0.2",
     "bun-types": "^1.1.39",
+    "editorconfig-checker": "^6.1.0",
     "esbuild": "^0.15.18",
     "eslint": "^9.10.0",
     "glob": "^11.0.0",
@@ -685,4 +687,4 @@
   "engines": {
     "node": ">=16.9.0"
   }
-}
+}

runtime-tests/bun/.static/plain.txt

@@ -1 +1 @@
-Bun!!
+Bun!!

runtime-tests/bun/index.test.tsx

@@ -132,21 +132,21 @@ describe('Serve Static Middleware', () => {
   it('Should return 200 response - /static/plain.txt', async () => {
     const res = await app.request(new Request('http://localhost/static/plain.txt'))
     expect(res.status).toBe(200)
-    expect(await res.text()).toBe('Bun!')
+    expect(await res.text()).toMatch(/^Bun!(\r?\n)?$/)
     expect(onNotFound).not.toHaveBeenCalled()
   })
 
   it('Should return 200 response - /static/download', async () => {
     const res = await app.request(new Request('http://localhost/static/download'))
     expect(res.status).toBe(200)
-    expect(await res.text()).toBe('download')
+    expect(await res.text()).toMatch(/^download(\r?\n)?$/)
     expect(onNotFound).not.toHaveBeenCalled()
   })
 
   it('Should return 200 response - /dot-static/plain.txt', async () => {
     const res = await app.request(new Request('http://localhost/dot-static/plain.txt'))
     expect(res.status).toBe(200)
-    expect(await res.text()).toBe('Bun!!')
+    expect(await res.text()).toMatch(/^Bun!!(\r?\n)?$/)
   })
 
   it('Should return 200 response - /static/helloworld', async () => {
@@ -164,7 +164,7 @@ describe('Serve Static Middleware', () => {
   it('Should return 200 response - /static-absolute-root/plain.txt', async () => {
     const res = await app.request('http://localhost/static-absolute-root/plain.txt')
     expect(res.status).toBe(200)
-    expect(await res.text()).toBe('Bun!')
+    expect(await res.text()).toMatch(/^Bun!(\r?\n)?$/)
     expect(onNotFound).not.toHaveBeenCalled()
   })
 })