opa,opa-react: tweak packages, adjust workflow

Remove test files according to publint suggestion.

Signed-off-by: Stephan Renatus <[email protected]>

.changeset/tasty-eels-travel.md

@@ -0,0 +1,8 @@
+---
+"@styra/opa-react": patch
+---
+
+tweak package (publint)
+
+1. remove test files from package
+2. package.json: move "browser" field to "exports"

.github/workflows/pull-request.yaml

@@ -63,6 +63,8 @@ jobs:
         run: npx -w ${{ matrix.pkg }} --if-present typedoc
       - name: are the types wrong?
         run: npx -w ${{ matrix.pkg }} attw --pack
+      - name: publint
+        run: npx -w ${{ matrix.pkg }} publint run --strict
       - name: jsr publish dry-run
         run: npx -w ${{ matrix.pkg }} jsr publish --dry-run
         if: matrix.pkg == 'packages/opa'

package.json

@@ -5,7 +5,10 @@
     "docs": "typedoc"
   },
   "private": true,
-  "workspaces": ["./packages/opa", "./packages/opa-react"],
+  "workspaces": [
+    "./packages/opa",
+    "./packages/opa-react"
+  ],
   "type": "module",
   "engineStrict": true,
   "engines": {
@@ -16,6 +19,7 @@
     "@arethetypeswrong/cli": "^0.15.3",
     "@changesets/cli": "^2.27.7",
     "jsr": "^0.12.4",
+    "publint": "^0.2.8",
     "typedoc": "^0.25.13",
     "typedoc-plugin-extras": "^3.0.0",
     "typedoc-plugin-replace-text": "^3.3.0"

packages/opa-react/.npmignore

@@ -1,4 +1,5 @@
 /*
-!/**/*.ts
-!/**/*.js
+!/**/*.ts*
+!/**/*.js*
 !/**/*.map
+/tests/**/*.ts*

packages/opa-react/package.json

@@ -20,12 +20,12 @@
   "license": "Apache-2.0",
   "type": "module",
   "sideEffects": false,
-  "browser": "./dist/cjs/index.js",
   "main": "./dist/cjs/index.js",
   "exports": {
     ".": {
       "import": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js"
+      "require": "./dist/cjs/index.js",
+      "browser": "./dist/cjs/index.js"
     }
   },
   "devDependencies": {

packages/opa/.npmignore

@@ -3,6 +3,7 @@
 !/**/*.ts
 !/**/*.js
 !/**/*.map
+/tests/**/*.ts
 
 /.eslintrc.js
 /cjs