Change workflow to return package names instead

Author: leeyi45

.github/actions/info/dist.js

@@ -12,7 +12,11 @@ async function findPackages(directory, maxDepth) {
       const fullPath = pathlib.join(currentDir, item.name);
       if (item.isFile()) {
         if (item.name === "package.json") {
-          yield fullPath;
+          try {
+            const { default: { name } } = await import(fullPath, { with: { type: "json" } });
+            if (name) yield name;
+          } catch {
+          }
         }
         continue;
       }

.github/actions/info/index.ts

@@ -16,7 +16,10 @@ async function findPackages(directory: string, maxDepth?: number) {
       const fullPath = pathlib.join(currentDir, item.name);
       if (item.isFile()) {
         if (item.name === 'package.json') {
-          yield fullPath;
+          try {
+            const { default: { name } } = await import(fullPath, { with: { type: 'json' }});
+            if (name) yield name;
+          } catch {}
         }
         continue;
       }

.github/workflows/new-workflow.yml

@@ -16,6 +16,9 @@ jobs:
       matrix:
         type: ['lib', 'bundles', 'tabs']
 
+    outputs:
+      packages: ${{ steps.find.outputs.packages }}
+
     steps:
       - name: Check out source code
         uses: actions/checkout@v4
@@ -37,6 +40,3 @@ jobs:
         uses: ./.github/actions/info
         with:
           type: ${{ matrix.type }}
-
-      - name: Print
-        run: echo ${{ steps.find.outputs.packages }}

eslint.config.js

@@ -137,6 +137,8 @@ export default tseslint.config(
       'jsdoc/check-alignment': 'warn',
       'jsdoc/require-asterisk-prefix': 'warn',
 
+      'no-empty': ['error', { allowEmptyCatch: true }],
+
       '@sourceacademy/region-comment': 'error',
 
       '@stylistic/brace-style': ['warn', '1tbs', { allowSingleLine: true }],
@@ -277,7 +279,6 @@ export default tseslint.config(
     rules: {
       'func-style': 'off',
       'import/extensions': ['error', 'never', { json: 'always' }],
-      'no-empty': ['error', { allowEmptyCatch: true }],
       'no-constant-condition': 'off', // Was 'error',
       'no-fallthrough': 'off',