Fix deps #289

ref DEV-1251

.eslintrc

@@ -10,8 +10,9 @@
     "expect": "readonly",
     "it": "readonly"
   },
-  "plugins": ["@typescript-eslint", "eslint-plugin-tsdoc"],
+  "plugins": ["@typescript-eslint", "eslint-plugin-tsdoc", "import"],
   "rules": {
+    "import/no-duplicates": ["error"],
     "@typescript-eslint/adjacent-overload-signatures": ["error"],
     "@typescript-eslint/array-type": ["off"],
     "@typescript-eslint/await-thenable": ["error"],
@@ -65,7 +66,6 @@
       }
     ],
     "@typescript-eslint/no-dupe-class-members": ["error"],
-    "@typescript-eslint/no-duplicate-imports": ["error"],
     "@typescript-eslint/no-dynamic-delete": ["off"],
     "@typescript-eslint/no-empty-function": ["off"],
     "@typescript-eslint/no-empty-interface": ["off"],
@@ -76,7 +76,6 @@
     "@typescript-eslint/no-extraneous-class": ["error"],
     "@typescript-eslint/no-floating-promises": ["error"],
     "@typescript-eslint/no-for-in-array": ["error"],
-    "@typescript-eslint/no-implicit-any-catch": ["error"],
     "@typescript-eslint/no-implied-eval": ["error"],
     "@typescript-eslint/no-inferrable-types": [
       "error",
@@ -97,7 +96,6 @@
     "@typescript-eslint/no-non-null-asserted-nullish-coalescing": ["error"],
     "@typescript-eslint/no-non-null-asserted-optional-chain": ["error"],
     "@typescript-eslint/no-non-null-assertion": ["off"],
-    "@typescript-eslint/no-parameter-properties": ["error"],
     "@typescript-eslint/no-redeclare": ["error"],
     "@typescript-eslint/no-require-imports": ["error"],
     "@typescript-eslint/no-restricted-imports": ["off"],
@@ -147,6 +145,7 @@
     "@typescript-eslint/non-nullable-type-assertion-style": ["error"],
     "@typescript-eslint/object-curly-spacing": ["off"],
     "@typescript-eslint/padding-line-between-statements": ["off"],
+    "@typescript-eslint/parameter-properties": ["error"],
     "@typescript-eslint/prefer-as-const": ["error"],
     "@typescript-eslint/prefer-enum-initializers": ["error"],
     "@typescript-eslint/prefer-for-of": ["off"],

.github/workflows/ci.yaml

@@ -19,6 +19,13 @@ jobs:
       with:
         node-version-file: "./.tool-versions"
     - run: npm ci
+    - run: npm ci
+      working-directory: ./website
+    - run: npm audit
+      if: ${{ !cancelled() }}
+    - run: npm audit
+      working-directory: ./website
+      if: ${{ !cancelled() }}
     - run: npm run format
       if: ${{ !cancelled() }}
     - run: npm run lint
@@ -29,6 +36,8 @@ jobs:
       if: ${{ !cancelled() }}
     - run: npm run bundle
       if: ${{ !cancelled() }}
+    - run: make docs
+      if: ${{ !cancelled() }}
 
   webapp-image:
     runs-on: ubuntu-22.04
@@ -43,6 +52,7 @@ jobs:
         username: ${{ secrets.QUAY_USERNAME }}
         password: ${{ secrets.QUAY_ROBOT_TOKEN }}
     - run: make push-image
+      if: ${{ github.repository == 'authgear/authgear-sdk-js' && github.event_name == 'push' }}
 
   # react_native_ios cannot be built on simulator due to WeChat SDK
 

Makefile

@@ -20,6 +20,7 @@ docs:
 		--name @authgear/web \
 		--entryPoints packages/authgear-web/index.d.ts \
 		--out ./temp/docs/web \
+		--plugin typedoc-plugin-markdown \
 		--namedAnchors \
 		--entryDocument index.md
 	npx typedoc \
@@ -28,6 +29,7 @@ docs:
 		--name @authgear/react-native \
 		--entryPoints packages/authgear-react-native/index.d.ts \
 		--out ./temp/docs/react-native \
+		--plugin typedoc-plugin-markdown \
 		--namedAnchors \
 		--entryDocument index.md
 	npx typedoc \
@@ -36,6 +38,7 @@ docs:
 		--name @authgear/capacitor \
 		--entryPoints packages/authgear-capacitor/index.d.ts \
 		--out ./temp/docs/capacitor \
+		--plugin typedoc-plugin-markdown \
 		--namedAnchors \
 		--entryDocument index.md
 	cp ./typedoc/index.md ./temp/docs/index.md
@@ -46,7 +49,7 @@ docs:
 	rm -rf ./website/docs
 	cp -R ./temp/docs/. ./website/docs
 	cp ./temp/sidebars.js ./website
-	(cd website && yarn build)
+	(cd website && npm run build)
 
 .PHONY: deploy-docs
 deploy-docs: docs