Fix things certain dependencies not being specified correctly

leeyi45 · leeyi45 · commit ecf67f5ffe43 · 2025-08-04T02:08:47.000+08:00
diff --git a/.github/workflows/workflow-again.yml b/.github/workflows/workflow-again.yml
@@ -10,7 +10,7 @@ on:
 
 jobs:
   find-packages:
-    name: Get information about the packages within the repository
+    name: Get packages within the repository
     runs-on: ubuntu-latest
 
     steps:
@@ -38,7 +38,7 @@ jobs:
       libs: ${{ steps.info.outputs.libs }}
 
   tabs:
-    name: Do work with the tabs
+    name: Tabs
     runs-on: ubuntu-latest
     needs: find-packages
     strategy:
@@ -65,11 +65,16 @@ jobs:
       - name: Build Tab
         run: yarn workspaces foreach -A --include ${{ matrix.tabInfo.name }} run build
 
+      - name: Test Tab
+        if: matrix.tabInfo.changes == 'true' && matrix.tabInfo.needsPlaywright
+        run: |
+          cd ${{ matrix.tabInfo.directory }}
+          yarn playwright install --with-deps
+          yarn test
+
       - name: Run Auxillary Tasks
         if: matrix.tabInfo.changes == 'true'
         run: |
           cd ${{ matrix.tabInfo.directory }}
           yarn tsc
           yarn lint
-          yarn playwright install --with-deps
-          yarn test
diff --git a/src/bundles/ar/package.json b/src/bundles/ar/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/binary_tree/package.json b/src/bundles/binary_tree/package.json
@@ -18,4 +18,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/communication/package.json b/src/bundles/communication/package.json
@@ -23,4 +23,4 @@
     "lint": "buildtools lint .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/copy_gc/package.json b/src/bundles/copy_gc/package.json
@@ -18,4 +18,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/csg/package.json b/src/bundles/csg/package.json
@@ -6,6 +6,7 @@
     "@jscad/modeling": "patch:@jscad/modeling@npm%3A2.9.6#~/.yarn/patches/@jscad-modeling-npm-2.9.6-71e6aa3a91.patch",
     "@jscad/regl-renderer": "^2.6.1",
     "@jscad/stl-serializer": "2.1.11",
+    "@sourceacademy/modules-lib": "workspace:^",
     "save-file": "^2.3.1"
   },
   "exports": {
@@ -24,4 +25,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/curve/package.json b/src/bundles/curve/package.json
@@ -24,4 +24,4 @@
     "tsc": "buildtools tsc .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/game/package.json b/src/bundles/game/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/mark_sweep/package.json b/src/bundles/mark_sweep/package.json
@@ -18,4 +18,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/midi/package.json b/src/bundles/midi/package.json
@@ -19,4 +19,4 @@
     ".": "./dist/index.js",
     "./*": "./dist/*.js"
   }
-}
+}
diff --git a/src/bundles/nbody/package.json b/src/bundles/nbody/package.json
@@ -27,4 +27,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/painter/package.json b/src/bundles/painter/package.json
@@ -3,6 +3,7 @@
   "version": "1.0.0",
   "private": true,
   "dependencies": {
+    "@sourceacademy/modules-lib": "workspace:^",
     "plotly.js-dist": "^3.0.0"
   },
   "devDependencies": {
@@ -22,4 +23,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/physics_2d/package.json b/src/bundles/physics_2d/package.json
@@ -21,4 +21,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/pix_n_flix/package.json b/src/bundles/pix_n_flix/package.json
@@ -18,4 +18,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/plotly/package.json b/src/bundles/plotly/package.json
@@ -25,4 +25,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/remote_execution/package.json b/src/bundles/remote_execution/package.json
@@ -21,4 +21,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/repeat/package.json b/src/bundles/repeat/package.json
@@ -18,4 +18,4 @@
     "lint": "buildtools lint .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/repl/package.json b/src/bundles/repl/package.json
@@ -21,4 +21,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/robot_simulation/package.json b/src/bundles/robot_simulation/package.json
@@ -3,6 +3,7 @@
   "version": "1.0.0",
   "private": true,
   "dependencies": {
+    "@sourceacademy/modules-lib": "workspace:^",
     "three": "^0.175.0"
   },
   "devDependencies": {
@@ -24,4 +25,4 @@
     "lint": "buildtools lint .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/rune/package.json b/src/bundles/rune/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/rune_in_words/package.json b/src/bundles/rune_in_words/package.json
@@ -18,4 +18,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/scrabble/package.json b/src/bundles/scrabble/package.json
@@ -18,4 +18,4 @@
     "lint": "buildtools lint .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/sound/package.json b/src/bundles/sound/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/sound_matrix/package.json b/src/bundles/sound_matrix/package.json
@@ -21,4 +21,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/stereo_sound/package.json b/src/bundles/stereo_sound/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/unittest/package.json b/src/bundles/unittest/package.json
@@ -21,4 +21,4 @@
     "lint": "buildtools lint .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/unity_academy/package.json b/src/bundles/unity_academy/package.json
@@ -26,4 +26,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/bundles/wasm/package.json b/src/bundles/wasm/package.json
@@ -22,4 +22,4 @@
     "test": "buildtools test --project .",
     "postinstall": "yarn tsc"
   }
-}
+}
diff --git a/src/tabs/ArcadeTwod/package.json b/src/tabs/ArcadeTwod/package.json
@@ -6,6 +6,7 @@
     "@blueprintjs/core": "^5.10.2",
     "@blueprintjs/icons": "^5.9.0",
     "@sourceacademy/modules-lib": "workspace:^",
+    "phaser": "^3.54.0",
     "react": "^18.3.1",
     "react-dom": "^18.3.1"
   },
diff --git a/yarn.lock b/yarn.lock
@@ -3120,6 +3120,7 @@ __metadata:
     "@jscad/regl-renderer": "npm:^2.6.1"
     "@jscad/stl-serializer": "npm:2.1.11"
     "@sourceacademy/modules-buildtools": "workspace:^"
+    "@sourceacademy/modules-lib": "workspace:^"
     save-file: "npm:^2.3.1"
     typescript: "npm:^5.8.2"
   languageName: unknown
@@ -3186,6 +3187,7 @@ __metadata:
   resolution: "@sourceacademy/bundle-painter@workspace:src/bundles/painter"
   dependencies:
     "@sourceacademy/modules-buildtools": "workspace:^"
+    "@sourceacademy/modules-lib": "workspace:^"
     "@types/plotly.js": "npm:^3.0.0"
     plotly.js-dist: "npm:^3.0.0"
     typescript: "npm:^5.8.2"
@@ -3259,6 +3261,7 @@ __metadata:
   resolution: "@sourceacademy/bundle-robot_simulation@workspace:src/bundles/robot_simulation"
   dependencies:
     "@sourceacademy/modules-buildtools": "workspace:^"
+    "@sourceacademy/modules-lib": "workspace:^"
     "@types/three": "npm:^0.175.0"
     three: "npm:^0.175.0"
     typescript: "npm:^5.8.2"
@@ -3588,6 +3591,7 @@ __metadata:
     "@sourceacademy/modules-lib": "workspace:^"
     "@types/react": "npm:^18.3.1"
     "@types/react-dom": "npm:^18.3.1"
+    phaser: "npm:^3.54.0"
     react: "npm:^18.3.1"
     react-dom: "npm:^18.3.1"
   languageName: unknown

Original file line number	Diff line number	Diff line change
`@@ -22,4 +22,4 @@`
`22`	`22`	`"test": "buildtools test --project .",`
`23`	`23`	`"postinstall": "yarn tsc"`
`24`	`24`	`}`
`25`		`-}`
	`25`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -18,4 +18,4 @@`
`18`	`18`	`"test": "buildtools test --project .",`
`19`	`19`	`"postinstall": "yarn tsc"`
`20`	`20`	`}`
`21`		`-}`
	`21`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -23,4 +23,4 @@`
`23`	`23`	`"lint": "buildtools lint .",`
`24`	`24`	`"postinstall": "yarn tsc"`
`25`	`25`	`}`
`26`		`-}`
	`26`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -24,4 +24,4 @@`
`24`	`24`	`"tsc": "buildtools tsc .",`
`25`	`25`	`"postinstall": "yarn tsc"`
`26`	`26`	`}`
`27`		`-}`
	`27`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -19,4 +19,4 @@`
`19`	`19`	`".": "./dist/index.js",`
`20`	`20`	`"./": "./dist/.js"`
`21`	`21`	`}`
`22`		`-}`
	`22`	`+}`