plugin: Fixes for CI

Author: nhooyr

ci/dev/test.sh

@@ -6,7 +6,7 @@ main() {
 
   cd test/test-plugin
   make -s out/index.js
-  cd $OLDPWD
+  cd "$OLDPWD"
   mocha -r ts-node/register ./test/*.test.ts "$@"
 }
 

src/node/plugin.ts

@@ -54,14 +54,14 @@ export class PluginAPI {
    */
   public async applications(): Promise<Application[]> {
     const apps = new Array<Application>()
-    for (const [_, p] of this.plugins) {
+    for (const [, p] of this.plugins) {
       const pluginApps = await p.applications()
 
       // Add plugin key to each app.
       apps.push(
         ...pluginApps.map((app) => {
-          app = {...app, path: path.join(p.routerPath, app.path || "")}
-          app = {...app, iconPath: path.join(app.path || "", app.iconPath)}
+          app = { ...app, path: path.join(p.routerPath, app.path || "") }
+          app = { ...app, iconPath: path.join(app.path || "", app.iconPath) }
           return {
             ...app,
             plugin: {
@@ -85,7 +85,7 @@ export class PluginAPI {
    * mount mounts all plugin routers onto r.
    */
   public mount(r: express.Router): void {
-    for (const [_, p] of this.plugins) {
+    for (const [, p] of this.plugins) {
       r.use(`/${p.name}`, p.router())
     }
   }
@@ -142,7 +142,7 @@ export class PluginAPI {
         encoding: "utf8",
       })
       const packageJSON: PackageJSON = JSON.parse(str)
-      for (const [_, p] of this.plugins) {
+      for (const [, p] of this.plugins) {
         if (p.name === packageJSON.name) {
           this.logger.warn(
             `ignoring duplicate plugin ${q(p.name)} at ${q(dir)}, using previously loaded ${q(p.modulePath)}`,

src/node/routes/apps.ts

@@ -12,6 +12,6 @@ export function router(papi: PluginAPI): express.Router {
   router.get("/", async (req, res) => {
     res.json(await papi.applications())
   })
-  
+
   return router
 }

src/node/routes/index.ts

@@ -16,14 +16,14 @@ import { PluginAPI } from "../plugin"
 import { getMediaMime, paths } from "../util"
 import { WebsocketRequest } from "../wsRouter"
 import * as domainProxy from "./domainProxy"
+import * as apps from "./apps"
 import * as health from "./health"
 import * as login from "./login"
 import * as proxy from "./pathProxy"
 // static is a reserved keyword.
 import * as _static from "./static"
 import * as update from "./update"
 import * as vscode from "./vscode"
-import * as apps from "./apps"
 
 declare global {
   // eslint-disable-next-line @typescript-eslint/no-namespace