From 7f0b57e26514c4bfe0e90af7baa6826a42a45cd8 Mon Sep 17 00:00:00 2001
From: Sylvain Lesage <severo@rednegra.net>
Date: Wed, 16 Apr 2025 21:29:43 +0200
Subject: [PATCH] types must be the first element (#74)

* types must be the first element. Spotted by publint.dev

* Package test for exports

* Test package.json for string exports

---------

Co-authored-by: Kenny Daniel <platypii@gmail.com>
---
 package.json         |  8 ++++----
 test/package.test.js | 11 +++++++++++
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/package.json b/package.json
index d6e6cb1..9288ecb 100644
--- a/package.json
+++ b/package.json
@@ -30,12 +30,12 @@
   "types": "types/hyparquet.d.ts",
   "exports": {
     ".": {
-      "import": "./src/hyparquet.js",
-      "types": "./types/hyparquet.d.ts"
+      "types": "./types/hyparquet.d.ts",
+      "import": "./src/hyparquet.js"
     },
     "./src/*.js": {
-      "import": "./src/*.js",
-      "types": "./types/*.d.ts"
+      "types": "./types/*.d.ts",
+      "import": "./src/*.js"
     }
   },
   "scripts": {
diff --git a/test/package.test.js b/test/package.test.js
index b0d1a14..9731021 100644
--- a/test/package.test.js
+++ b/test/package.test.js
@@ -21,4 +21,15 @@ describe('package.json', () => {
     expect('dependencies' in packageJson).toBe(false)
     expect('peerDependencies' in packageJson).toBe(false)
   })
+  it('should have exports with types first', () => {
+    const { exports } = packageJson
+    expect(exports).toBeDefined()
+    for (const [, exportObj] of Object.entries(exports)) {
+      if (typeof exportObj === 'object') {
+        expect(Object.keys(exportObj)).toEqual(['types', 'import'])
+      } else {
+        expect(typeof exportObj).toBe('string')
+      }
+    }
+  })
 })