fix: fix a couple of templates

Author: AaravMalani

lib/templates/template.ts

@@ -61,7 +61,7 @@ export async function getConfig(): Promise<Config> {
     pluginName = await input({
       message: "What is the name of the plugin class? (use PascalCase)",
       required: false,
-      default: generateDefaultPluginName(name, location),
+      default: generateDefaultPluginName(name, location, type),
       validate: validatePluginName,
     });
   } else {
@@ -97,8 +97,8 @@ export async function generateTemplate(config: Config) {
   packageJSONContent.description = config.description;
   if (type === "external") {
     // For external plugins, the main entry point is at index.js, instead of the default index.cjs
-    packageJSONContent.exports["."].import = "./dist/index.js";
-    packageJSONContent.module = "./dist/index.js";
+    packageJSONContent.exports["."].require = "./dist/index.js";
+    packageJSONContent.main = "./dist/index.js";
   }
   await fs.writeFile(packageJSON, JSON.stringify(packageJSONContent, null, 2));
 
@@ -113,7 +113,10 @@ export async function generateTemplate(config: Config) {
   if (pluginName) {
     for await (const file of fs.glob(`${bundlePath.replace(pathlib.sep, "/")}/**/*.{ts,tsx}`)) {
       const content = await fs.readFile(file, "utf-8");
-      const updatedContent = content.replace(/PluginName/g, pluginName);
+      let updatedContent = content.replace(/PluginName/g, pluginName);
+      if (type === "external") {
+        updatedContent = updatedContent.replace(/abstract /g, "");
+      }
       await fs.writeFile(file, updatedContent);
     }
   }

lib/templates/utils.ts

@@ -15,14 +15,15 @@ export function validatePluginName(value: string) {
   return true;
 }
 
-export function generateDefaultPluginName(name: string, location: string) {
-  return (
-    name
-      .split("-")
-      .map(word => word.charAt(0).toUpperCase() + word.slice(1))
-      .join("") +
-    location.charAt(0).toUpperCase() +
-    location.slice(1) +
-    "Plugin"
-  );
+export function generateDefaultPluginName(name: string, location: string, type: string) {
+  return type == "installable"
+    ? "Base"
+    : "" +
+        name
+          .split("-")
+          .map(word => word.charAt(0).toUpperCase() + word.slice(1))
+          .join("") +
+        location.charAt(0).toUpperCase() +
+        location.slice(1) +
+        "Plugin";
 }

lib/templates/web/rollup.config.mjs

@@ -6,7 +6,7 @@ import typescript from "@rollup/plugin-typescript";
  * @type {import('rollup').RollupOptions}
  */
 export default {
-  input: "src/index.ts",
+  input: "src/index.tsx",
   output: [
     {
       file: "dist/index.cjs",

lib/templates/web/src/__tests__/web.test.ts

@@ -0,0 +1,6 @@
+import { PluginName } from "..";
+import { test, expect } from "vitest";
+
+test("should have a valid channel id", () => {
+  expect(PluginName.channelAttach).toEqual(["__channel_id"]);
+});

lib/templates/web/src/index.tsx

@@ -6,7 +6,7 @@ import {
 } from "@sourceacademy/conductor/conduit";
 import type { ITabService, Tab } from "@sourceacademy/common-tabs";
 
-export abstract class TestPlugin implements IPlugin {
+export abstract class PluginName implements IPlugin {
   readonly id: string = "__plugin_id";
   static readonly channelAttach = ["__channel_id"];
   private readonly __testChannel: IChannel<string>;
@@ -32,4 +32,4 @@ export abstract class TestPlugin implements IPlugin {
     tabService.showTab(tab.id);
   }
 }
-checkIsPluginClass(TestPlugin);
+checkIsPluginClass(PluginName);