piral-cli 0.15.0-alpha.4029 → 0.15.0-alpha.4041

package/src/commands.ts

@@ -410,6 +410,9 @@ const allCommands: Array<ToolCommand<any>> = [
         .string('target')
         .describe('target', 'Sets the target directory or file of bundling.')
         .default('target', apps.debugPiletDefaults.target)
+        .string('public-url')
+        .describe('public-url', 'Sets the public URL (path) of the application.')
+        .default('public-url', apps.debugPiletDefaults.publicUrl)
         .number('port')
         .describe('port', 'Sets the port of the local development server.')
         .default('port', apps.debugPiletDefaults.port)
@@ -448,6 +451,7 @@ const allCommands: Array<ToolCommand<any>> = [
       return apps.debugPilet(args.base as string, {
         entry: args.source as string,
         target: args.target as string,
+        publicUrl: args['public-url'] as string,
         port: args.port as number,
         hmr: args.hmr as boolean,
         bundlerName: args.bundler as string,
@@ -480,6 +484,9 @@ const allCommands: Array<ToolCommand<any>> = [
         .string('target')
         .describe('target', 'Sets the target file of bundling.')
         .default('target', apps.buildPiletDefaults.target)
+        .string('public-url')
+        .describe('public-url', 'Sets the public URL (path) of the application.')
+        .default('public-url', apps.buildPiletDefaults.publicUrl)
         .number('log-level')
         .describe('log-level', 'Sets the log level to use (1-5).')
         .default('log-level', apps.buildPiletDefaults.logLevel)
@@ -523,6 +530,7 @@ const allCommands: Array<ToolCommand<any>> = [
       return apps.buildPilet(args.base as string, {
         entry: args.source as string,
         target: args.target as string,
+        publicUrl: args['public-url'] as string,
         minify: args.minify as boolean,
         contentHash: args['content-hash'] as boolean,
         bundlerName: args.bundler as string,

package/src/common/importmap.ts

@@ -7,6 +7,7 @@ import { SharedDependency } from '../types';
 
 interface Importmap {
   imports: Record<string, string>;
+  inherit: Array<string>;
 }
 
 function tryResolve(baseDir: string, name: string) {
@@ -53,6 +54,7 @@ function getLocalDependencyVersion(
 async function resolveImportmap(dir: string, importmap: Importmap) {
   const dependencies: Array<SharedDependency> = [];
   const sharedImports = importmap?.imports;
+  const inheritedImports = importmap?.inherit;
 
   if (typeof sharedImports === 'object' && sharedImports) {
     for (const depName of Object.keys(sharedImports)) {
@@ -96,7 +98,9 @@ async function resolveImportmap(dir: string, importmap: Importmap) {
 
         if (exists) {
           const isDirectory = await checkIsDirectory(entry);
-          const packageJson = isDirectory ? resolve(entry, 'package.json') : await findFile(dirname(entry), 'package.json');
+          const packageJson = isDirectory
+            ? resolve(entry, 'package.json')
+            : await findFile(dirname(entry), 'package.json');
           const packageJsonExists = await checkExists(packageJson);
 
           if (packageJsonExists) {
@@ -131,6 +135,26 @@ async function resolveImportmap(dir: string, importmap: Importmap) {
     }
   }
 
+  if (Array.isArray(inheritedImports)) {
+    for (const inheritedImport of inheritedImports) {
+      const packageJson = tryResolve(dir, `${inheritedImport}/package.json`);
+
+      if (packageJson) {
+        const packageDir = dirname(packageJson);
+        const packageDetails = require(packageJson);
+        const otherDependencies = await readImportmap(packageDir, packageDetails);
+
+        for (const dependency of otherDependencies) {
+          const entry = dependencies.find((dep) => dep.name === dependency.name);
+
+          if (!entry) {
+            dependencies.push(dependency);
+          }
+        }
+      }
+    }
+  }
+
   return dependencies;
 }
 

package/src/common/package.test.ts

@@ -29,7 +29,7 @@ describe('CLI package module', () => {
   it('getPiletsInfo returns pilets information about provided piralInfo', () => {
     const emptyPiletsInfo = {
       files: [],
-      externals: [],
+      template: 'default',
       scripts: {},
       validators: {},
       devDependencies: {},
@@ -45,7 +45,7 @@ describe('CLI package module', () => {
     const piralInfo = {
       pilets: {
         files: ['foo.tgz', 'foo2.tgz'],
-        externals: [],
+        template: 'default',
         scripts: {},
         validators: {},
         devDependencies: {},

package/src/common/package.ts

@@ -150,8 +150,10 @@ function findPackage(pck: string | Array<string>, baseDir: string) {
     if (path) {
       log('generalDebug_0003', `Following the app package in "${path}" ...`);
       const appPackage = require(path);
+      const root = dirname(path);
       const relPath = appPackage && appPackage.app;
-      appPackage.app = relPath && resolve(dirname(path), relPath);
+      appPackage.app = relPath && resolve(root, relPath);
+      appPackage.root = root;
       return appPackage;
     }
   }
@@ -192,6 +194,10 @@ export function getPiralPackage(
       start: 'piral debug',
       build: 'piral build',
     },
+    importmap: {
+      imports: {},
+      inherit: ['piral-base', framework !== 'piral-base' && 'piral-core'].filter(Boolean),
+    },
     pilets: getPiletsInfo({}),
     dependencies,
     devDependencies,
@@ -345,8 +351,8 @@ export async function copyPiralFiles(
 export function getPiletsInfo(piralInfo: any): PiletsInfo {
   const {
     files = [],
-    externals = [],
     scripts = {},
+    template = 'default',
     validators = {},
     devDependencies = {},
     preScaffold = '',
@@ -358,8 +364,8 @@ export function getPiletsInfo(piralInfo: any): PiletsInfo {
 
   return {
     files,
-    externals,
     scripts,
+    template,
     validators,
     devDependencies,
     preScaffold,
@@ -442,6 +448,21 @@ export async function findPackageVersion(rootPath: string, packageName: string):
   }
 }
 
+export async function retrieveExternals(root: string, packageInfo: any) {
+  const sharedDependencies = await readImportmap(root, packageInfo);
+
+  if (sharedDependencies.length === 0) {
+    const allDeps = {
+      ...packageInfo.devDependencies,
+      ...packageInfo.dependencies,
+    };
+    const deps = packageInfo.pilets?.externals;
+    return makeExternals(allDeps, deps);
+  }
+
+  return sharedDependencies.map((m) => m.name);
+}
+
 export async function retrievePiletsInfo(entryFile: string) {
   const exists = await checkExists(entryFile);
 
@@ -456,12 +477,9 @@ export async function retrievePiletsInfo(entryFile: string) {
   }
 
   const packageInfo = require(packageJson);
-  const allDeps = {
-    ...packageInfo.devDependencies,
-    ...packageInfo.dependencies,
-  };
   const info = getPiletsInfo(packageInfo);
-  const externals = makeExternals(allDeps, info.externals);
+  const root = dirname(packageJson);
+  const externals = await retrieveExternals(root, packageInfo);
 
   return {
     ...info,
@@ -475,7 +493,7 @@ export async function retrievePiletsInfo(entryFile: string) {
     },
     scripts: packageInfo.scripts,
     ignored: checkArrayOrUndefined(packageInfo, 'preservedDependencies'),
-    root: dirname(packageJson),
+    root,
   };
 }
 
@@ -599,9 +617,10 @@ export async function retrievePiletData(target: string, app?: string) {
     app || (piletPackage.piral && piletPackage.piral.name) || Object.keys(piletPackage.devDependencies),
     target,
   );
-  const appFile: string = appPackage && appPackage.app;
+  const appFile: string = appPackage?.app;
+  const appRoot: string = appPackage?.root;
 
-  if (!appFile) {
+  if (!appFile || !appRoot) {
     fail('appInstanceInvalid_0011');
   }
 
@@ -616,6 +635,7 @@ export async function retrievePiletData(target: string, app?: string) {
     ignored: checkArrayOrUndefined(piletPackage, 'preservedDependencies'),
     importmap,
     appFile,
+    appRoot,
     piletPackage,
     appPackage,
     emulator,

package/src/injectors/pilet.test.ts

@@ -4,6 +4,8 @@ import PiletInjector from './pilet';
 
 const optionsMock = {
   pilets: [],
+  publicUrl: '/',
+  meta: 'debug-meta.json',
   api: '',
   app: '',
   active: true,
@@ -83,6 +85,8 @@ describe('Piral-CLI piral injector', () => {
     // Arrange
     const optionsMock = {
       pilets: [],
+      meta: 'debug-meta.json',
+      publicUrl: '/',
       api: 'http://someFakeApi:1234',
       app: '',
       active: true,
@@ -110,6 +114,8 @@ describe('Piral-CLI piral injector', () => {
     // Arrange
     const optionsMock = {
       pilets: [],
+      meta: 'debug-meta.json',
+      publicUrl: '/',
       api: 'http://someFakeApi:1234',
       app: '',
       active: true,

package/src/injectors/pilet.ts

@@ -10,6 +10,7 @@ import { axios, mime } from '../external';
 import { Bundler } from '../types';
 
 const { host } = config;
+const indexPath = '/index.html';
 
 interface Pilet {
   bundler: Bundler;
@@ -19,6 +20,7 @@ interface Pilet {
 
 export interface PiletInjectorConfig extends KrasInjectorConfig {
   pilets: Array<Pilet>;
+  publicUrl: string;
   meta: string;
   api: string;
   app: string;
@@ -210,26 +212,30 @@ export default class PiletInjector implements KrasInjector {
   }
 
   handle(req: KrasRequest): KrasResponse {
-    const { app, api } = this.config;
-    const path = req.url.substring(1).split('?')[0];
+    const { app, api, publicUrl } = this.config;
 
     if (!req.target) {
-      const target = join(app, path);
+      if (req.url.startsWith(publicUrl)) {
+        const path = req.url.substring(publicUrl.length).split('?')[0];
+        const target = join(app, path);
 
-      if (existsSync(target) && statSync(target).isFile()) {
-        if (req.url === '/index.html') {
-          return this.sendIndexFile(target, req.url);
+        if (existsSync(target) && statSync(target).isFile()) {
+          if (req.url === indexPath) {
+            return this.sendIndexFile(target, req.url);
+          } else {
+            return this.sendFile(target, req.url);
+          }
+        } else if (req.url !== indexPath) {
+          return this.handle({
+            ...req,
+            url: indexPath,
+          });
         }
-        return this.sendFile(target, req.url);
-      } else if (req.url !== '/index.html') {
-        return this.handle({
-          ...req,
-          url: '/index.html',
-        });
-      } else {
-        return undefined;
       }
+
+      return undefined;
     } else if (req.target === api) {
+      const path = req.url.substring(1).split('?')[0];
       return this.sendResponse(path, req.url);
     }
   }

package/src/rules/pilet-has-externals-as-peers.ts

@@ -1,5 +1,5 @@
 import { PiletRuleContext } from '../types';
-import { getPiletsInfo, getSourceFiles, isValidDependency } from '../common';
+import { retrieveExternals, getSourceFiles, isValidDependency } from '../common';
 
 export type Options = 'ignore' | 'active' | 'only-used';
 
@@ -21,7 +21,7 @@ Received: Missing "${missingNames.join('", "')}".
  */
 export default async function (context: PiletRuleContext, options: Options = 'ignore') {
   if (options !== 'ignore') {
-    const { externals } = getPiletsInfo(context.data.appPackage);
+    const externals = await retrieveExternals(context.data.appRoot, context.data.appPackage);
     const markedPeerDependencies = Object.keys(context.peerDependencies);
     const markedPeerModules = context.peerModules;
     const missingExternals = externals

package/src/types/common.ts

@@ -55,7 +55,8 @@ export interface TemplateFileLocation {
 
 export interface PiletsInfo {
   files: Array<string | TemplateFileLocation>;
-  externals: Array<string>;
+  template: string;
+  externals?: Array<string>;
   devDependencies: Record<string, string | true>;
   scripts: Record<string, string>;
   validators: Record<string, any>;
@@ -106,6 +107,7 @@ export interface PiletRuleContext extends RuleContext {
 }
 
 export interface PiralData {
+  appRoot: string;
   appFile: string;
   appPackage: any;
   piletPackage: any;

package/src/types/public.ts

@@ -80,6 +80,7 @@ export interface DebugPiralParameters extends BaseBundleParameters {
 export interface WatchPiralParameters extends BaseBundleParameters {
   piral: string;
   externals: Array<string>;
+  publicUrl: string;
   entryFiles: string;
   logLevel: LogLevels;
 }