@cueloop/extension-api 0.1.0-alpha.101 → 0.1.0-alpha.103

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/extension-api",
-  "version": "0.1.0-alpha.101",
+  "version": "0.1.0-alpha.103",
   "description": "The typed contract for extending cueloop: renderers, commands, keybindings, exporters",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -26,7 +26,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/schema": "0.1.0-alpha.101",
+    "@cueloop/schema": "0.1.0-alpha.103",
     "valibot": "^1.5.0"
   },
   "peerDependencies": {

package/src/installed-packages.ts

@@ -45,14 +45,19 @@ function readPackageJson(path: string): v.InferOutput<typeof PackageJsonSchema>
 
 function packageEntry(root: string, entry: string | undefined): string | undefined {
   if (entry === undefined) return undefined;
+
   if (!entry.startsWith("./")) throw new Error("entry must start with ./");
+
   const absolute = resolve(root, entry);
   const lexicalWithin = relative(root, absolute);
 
   if (isAbsolute(lexicalWithin) || lexicalWithin === ".." || lexicalWithin.startsWith(`..${sep}`))
     throw new Error("entry escapes package root");
+
   if (!existsSync(absolute)) throw new Error(`entry missing: ${entry}`);
+
   if (!statSync(absolute).isFile()) throw new Error(`entry is not a file: ${entry}`);
+
   const within = relative(realpathSync(root), realpathSync(absolute));
 
   if (isAbsolute(within) || within === ".." || within.startsWith(`..${sep}`))
@@ -70,6 +75,7 @@ export function discoverInstalledExtensionPackages(
   const rootManifest = join(installRoot, "package.json");
 
   if (!existsSync(rootManifest)) return { packages, errors };
+
   let dependencies: Record<string, string>;
 
   try {
@@ -85,10 +91,12 @@ export function discoverInstalledExtensionPackages(
       const manifest = readPackageJson(join(root, "package.json")).cueloop;
 
       if (!manifest) throw new Error("package has no cueloop manifest");
+
       const clientEntry = packageEntry(root, manifest.client);
       const daemonEntry = packageEntry(root, manifest.daemon);
 
       if (!clientEntry && !daemonEntry) throw new Error("manifest declares no entry point");
+
       packages.push({ name, root, clientEntry, daemonEntry });
     } catch (error) {
       errors.push(`Extension package ${name}: ${String(error)}`);

package/src/registry.ts

@@ -42,14 +42,17 @@ export class Registry {
       registerVcsAdapter: (adapter) => {
         if (adapter.id === "git" || adapter.id === "jj")
           throw new Error(`Invalid or duplicate VCS adapter ID: ${adapter.id}`);
+
         const parsed = v.safeParse(VcsAdapterSchema, adapter);
 
         if (!parsed.success) throw new Error(`Invalid VCS adapter contract: ${adapter.id}`);
+
         if (
           record.vcsAdapters.has(adapter.id) ||
           this.extensions.some((extension) => extension.vcsAdapters.has(adapter.id))
         )
           throw new Error(`Invalid or duplicate VCS adapter ID: ${adapter.id}`);
+
         record.vcsAdapters.set(adapter.id, adapter);
       },
     };