Test that removing the preinstall script from the root package makes installations work better

Author: leeyi45

.github/workflows/pull-request.yml

@@ -42,7 +42,7 @@ jobs:
       playwright: ${{ steps.filter.outputs.devserver || steps.filter.outputs.libraries || steps.filter.outputs.tabs }}
 
   test:
-    name: Verify all tests pass and build success
+    name: Verify that all tests, linting, type-checking and building succeeds.
     runs-on: ubuntu-latest
 
     # execute after the paths-filter job is done
@@ -62,8 +62,8 @@ jobs:
             node-version: 22
             cache: yarn
 
-        - name: Setup Repotools
-          run: yarn workspaces focus @sourceacademy/modules-repotools && yarn workspaces foreach -A --include "@sourceacademy/modules-repotools" run build
+        # - name: Setup Repotools
+        #   run: yarn workspaces focus @sourceacademy/modules-repotools && yarn workspaces foreach -A --include "@sourceacademy/modules-repotools" run build
 
         - name: Install dependencies
           run: yarn install --immutable

package.json

@@ -35,7 +35,6 @@
     "lint:all": "buildtools lintglobal && yarn lint:modules",
     "lint:inspect": "yarn dlx eslint --inspect-config",
     "lint:modules": "yarn workspaces foreach -j 5 -pW --from \"./src/{bundles,tabs}\" run lint",
-    "preinstall": "yarn build:libs",
     "prepare": "husky",
     "run:bundles": "yarn workspaces foreach -ptW --from \"./src/bundles/*\" run",
     "serve": "http-server --cors=* -c-1 -p 8022 ./build",

vitest.config.js

@@ -3,6 +3,11 @@
 import pathlib from 'path';
 import { coverageConfigDefaults, defineConfig } from 'vitest/config';
 
+const coverageReporters = ['html', 'text'];
+if (process.env.GITHUB_ACTIONS) {
+  coverageReporters.push('github-actions');
+}
+
 export default defineConfig({
   resolve: {
     alias: [{
@@ -20,7 +25,7 @@ export default defineConfig({
     clearMocks: true,
     coverage: {
       provider: 'v8',
-      reporter: ['html', 'text'],
+      reporter: coverageReporters,
       exclude: [
         ...coverageConfigDefaults.exclude,
         './build/**',