Fix missing type in manifest and mkdir call in manifest builder

Author: leeyi45

lib/buildtools/src/build/modules/__tests__/building.test.ts

@@ -1,7 +1,10 @@
 import fs from 'fs/promises';
+import { outDir } from '@sourceacademy/modules-repotools/getGitRoot';
+import type { ResolvedBundle } from '@sourceacademy/modules-repotools/types';
 import { beforeEach, expect, test, vi } from 'vitest';
 import { testMocksDir } from '../../../__tests__/fixtures.js';
 import { buildBundle, buildTab } from '../index.js';
+import { buildManifest } from '../manifest.js';
 
 const written: string[] = [];
 vi.spyOn(fs, 'open').mockResolvedValue({
@@ -98,3 +101,39 @@ test('build tab', async () => {
   expect(tab.body(0)).toEqual(0);
   expect(tab.toSpawn()).toEqual(true);
 });
+
+test('build manifest', async () => {
+  const sampleManifests: Record<string, ResolvedBundle> = {
+    bundle0: {
+      type: 'bundle',
+      name: 'bundle0',
+      directory: './bundle0',
+      manifest: {
+        requires: 1
+      }
+    },
+    bundle1: {
+      type: 'bundle',
+      name: 'bundle1',
+      directory: './bundle1',
+      manifest: {
+        version: '1.0.0'
+      }
+    }
+  };
+
+  await buildManifest(sampleManifests, outDir);
+
+  expect(fs.mkdir).toHaveBeenCalledExactlyOnceWith(outDir);
+  expect(fs.writeFile).toHaveBeenCalledExactlyOnceWith(
+    `${outDir}/modules.json`,
+    JSON.stringify({
+      bundle0: {
+        requires: 1
+      },
+      bundle1: {
+        version: '1.0.0'
+      }
+    })
+  );
+});

lib/buildtools/src/build/modules/manifest.ts

@@ -11,6 +11,8 @@ export async function buildManifest(bundles: Record<string, ResolvedBundle>, out
     [name]: manifest
   }), {});
 
+  await fs.mkdir(outDir);
+
   const outpath = `${outDir}/modules.json`;
   await fs.writeFile(outpath, JSON.stringify(finalManifest, null, 2));
 

lib/repotools/src/types.ts

@@ -7,7 +7,8 @@ export const severity = {
 export type Severity = (typeof severity)[keyof typeof severity];
 
 export interface BundleManifest {
-  version?: string;
+  requires?: number;
+  version?: string
   tabs?: string[];
 }