Merge pull request #1353 from opencomponents/compile-client-on-dev

always compile client on dev

package.json

@@ -104,7 +104,7 @@
     "multer": "^1.4.3",
     "nice-cache": "^0.0.5",
     "oc-client": "^4.0.1",
-    "oc-client-browser": "^1.7.4",
+    "oc-client-browser": "^1.7.5",
     "oc-empty-response-handler": "^1.0.2",
     "oc-get-unix-utc-timestamp": "^1.0.6",
     "oc-s3-storage-adapter": "^2.1.1",
@@ -128,4 +128,4 @@
     "universalify": "^2.0.0",
     "yargs": "^17.7.2"
   }
-}
+}

src/cli/facade/dev.ts

@@ -169,6 +169,7 @@ const dev = ({ local, logger }: { logger: Logger; local: Local }) =>
         baseUrl,
         prefix: opts.prefix || '',
         dependencies: dependencies.modules,
+        compileClient: true,
         discovery: true,
         env: { name: 'local' },
         fallbackRegistryUrl,

src/registry/routes/static-redirector.ts

@@ -15,10 +15,16 @@ export default function staticRedirector(repository: Repository) {
 
     if (req.route.path === clientPath) {
       if (res.conf.local) {
-        filePath = path.join(
-          __dirname,
-          '../../components/oc-client/_package/src/oc-client.js'
-        );
+        if (res.conf.compiledClient) {
+          res.type('application/javascript');
+          res.send(res.conf.compiledClient.dev);
+          return;
+        } else {
+          filePath = path.join(
+            __dirname,
+            '../../components/oc-client/_package/src/oc-client.js'
+          );
+        }
       } else {
         if (res.conf.compiledClient) {
           res.type('application/javascript');

src/types.ts

@@ -143,7 +143,7 @@ export type PublishAuthConfig =
 
 export interface Config {
   baseUrl: string;
-  compiledClient?: { code: string; map: string };
+  compiledClient?: { code: string; map: string; dev: string };
   baseUrlFunc?: (opts: { host?: string; secure: boolean }) => string;
   beforePublish: (req: Request, res: Response, next: NextFunction) => void;
   customHeadersToSkipOnWeakVersion: string[];