fix(skip-list): fixed the handling and passing of the skip list

- the skip list was ignored for secondary entry points, and thus libraries were brought back in (libs/native-federation-core/src/lib/config/share-utils.ts)
- fixed a few typos

Author: Alain Sollberger

.gitignore

@@ -16,6 +16,7 @@ node_modules
 *.launch
 .settings/
 *.sublime-workspace
+.tool-versions
 
 # IDE - VSCode
 .vscode/*
@@ -46,6 +47,5 @@ migrations.json
 # Angular
 .angular
 
-.nx
 .cursor/rules/nx-rules.mdc
 .github/instructions/nx.instructions.md

libs/mf/rspack.ts

@@ -1,6 +1,6 @@
 export * from './src/rspack';
 export {
-  DEFAULT_SECONARIES_SKIP_LIST,
+  DEFAULT_SECONDARIES_SKIP_LIST,
   DEFAULT_SKIP_LIST,
   SharedMappings,
   findRootTsConfigJson,

libs/mf/src/schematics/init-webpack/schematic.ts

@@ -146,7 +146,7 @@ function nxBuildersAvailable(tree: Tree): boolean {
   return true;
 }
 
-function infereNxBuilderNames(tree: Tree): { dev: string; prod: string } {
+function inferNxBuilderNames(tree: Tree): { dev: string; prod: string } {
   const dep = getPackageJsonDependency(tree, '@nx/angular');
 
   const useDevServer =
@@ -334,7 +334,7 @@ export default function config(options: MfSchematicSchema): Rule {
       options.nxBuilders = nxBuildersAvailable(tree); // tree.exists('nx.json');
     }
 
-    const nxBuilderNames = infereNxBuilderNames(tree);
+    const nxBuilderNames = inferNxBuilderNames(tree);
 
     if (options.nxBuilders) {
       console.log('Using Nx builders!');
@@ -354,7 +354,7 @@ export default function config(options: MfSchematicSchema): Rule {
 
     if (!projectConfig?.architect?.build || !projectConfig?.architect?.serve) {
       throw new Error(
-        `The project doen't have a build or serve target in angular.json!`
+        `The project doesn't have a build or serve target in angular.json!`
       );
     }
 

libs/mf/src/utils/share-utils.ts

@@ -15,7 +15,7 @@ export const DEFAULT_SKIP_LIST = [
   'zone.js',
 ];
 
-export const DEFAULT_SECONARIES_SKIP_LIST = [
+export const DEFAULT_SECONDARIES_SKIP_LIST = [
   '@angular/router/upgrade',
   '@angular/common/upgrade',
 ];
@@ -135,7 +135,7 @@ function getSecondaries(
   packagePath: string,
   key: string,
   shareObject: SharedConfig,
-  exclude = [...DEFAULT_SECONARIES_SKIP_LIST]
+  exclude = [...DEFAULT_SECONDARIES_SKIP_LIST]
 ): Record<string, SharedConfig> {
   if (typeof includeSecondaries === 'object') {
     if (Array.isArray(includeSecondaries.skip)) {
@@ -217,7 +217,7 @@ function readConfiguredSecondaries(
 
 export function shareAll(
   config: CustomSharedConfig = {},
-  skip: string[] = [...DEFAULT_SKIP_LIST, ...DEFAULT_SECONARIES_SKIP_LIST],
+  skip: string[] = [...DEFAULT_SKIP_LIST, ...DEFAULT_SECONDARIES_SKIP_LIST],
   packageJsonPath = ''
 ): Config {
   if (!packageJsonPath) {
@@ -247,7 +247,7 @@ export function setInferVersion(infer: boolean): void {
 export function share(
   shareObjects: Config,
   packageJsonPath = '',
-  skip: string[] = DEFAULT_SECONARIES_SKIP_LIST
+  skip: string[] = DEFAULT_SECONDARIES_SKIP_LIST
 ): Config {
   if (!packageJsonPath) {
     packageJsonPath = cwd();

libs/mf/src/utils/with-mf-plugin.ts

@@ -1,5 +1,5 @@
 import {
-  DEFAULT_SECONARIES_SKIP_LIST,
+  DEFAULT_SECONDARIES_SKIP_LIST,
   DEFAULT_SKIP_LIST,
   findRootTsConfigJson,
   shareAll,
@@ -15,7 +15,7 @@ export function withModuleFederationPlugin(config: unknown) {
 
   const skip = [
     ...DEFAULT_SKIP_LIST,
-    ...DEFAULT_SECONARIES_SKIP_LIST,
+    ...DEFAULT_SECONDARIES_SKIP_LIST,
     ...(config['skip'] || []),
   ];
 

libs/native-federation-core/src/lib/config/share-utils.ts

@@ -17,7 +17,6 @@ import {
 } from '../utils/package-info';
 import { getConfigContext } from './configuration-context';
 import { logger } from '../utils/logger';
-import { resolveGlobSync } from '../utils/resolve-glob';
 
 import {
   KeyValuePair,
@@ -26,7 +25,7 @@ import {
 
 let inferVersion = false;
 
-export const DEFAULT_SECONARIES_SKIP_LIST = [
+export const DEFAULT_SECONDARIES_SKIP_LIST = [
   '@angular/router/upgrade',
   '@angular/common/upgrade',
 ];
@@ -110,7 +109,8 @@ function _findSecondaries(
   libPath: string,
   excludes: string[],
   shareObject: SharedConfig,
-  acc: Record<string, SharedConfig>
+  acc: Record<string, SharedConfig>,
+  preparedSkipList: PreparedSkipList
 ): void {
   const files = fs.readdirSync(libPath);
 
@@ -136,27 +136,29 @@ function _findSecondaries(
       acc[secondaryLibName] = { ...shareObject };
     }
 
-    _findSecondaries(s, excludes, shareObject, acc);
+    _findSecondaries(s, excludes, shareObject, acc, preparedSkipList);
   }
 }
 
 function findSecondaries(
   libPath: string,
   excludes: string[],
-  shareObject: SharedConfig
+  shareObject: SharedConfig,
+  preparedSkipList: PreparedSkipList
 ): Record<string, SharedConfig> {
   const acc = {} as Record<string, SharedConfig>;
-  _findSecondaries(libPath, excludes, shareObject, acc);
+  _findSecondaries(libPath, excludes, shareObject, acc, preparedSkipList);
   return acc;
 }
 
 function getSecondaries(
   includeSecondaries: IncludeSecondariesOptions,
   libPath: string,
   key: string,
-  shareObject: SharedConfig
+  shareObject: SharedConfig,
+  preparedSkipList: PreparedSkipList
 ): Record<string, SharedConfig> | null {
-  let exclude = [...DEFAULT_SECONARIES_SKIP_LIST];
+  let exclude = [...DEFAULT_SECONDARIES_SKIP_LIST];
 
   if (typeof includeSecondaries === 'object') {
     if (Array.isArray(includeSecondaries.skip)) {
@@ -176,22 +178,29 @@ function getSecondaries(
     key,
     libPath,
     exclude,
-    shareObject
+    shareObject,
+    preparedSkipList
   );
   if (configured) {
     return configured;
   }
 
   // Fallback: Search folders
-  const secondaries = findSecondaries(libPath, exclude, shareObject);
+  const secondaries = findSecondaries(
+    libPath,
+    exclude,
+    shareObject,
+    preparedSkipList
+  );
   return secondaries;
 }
 
 function readConfiguredSecondaries(
   parent: string,
   libPath: string,
   exclude: string[],
-  shareObject: SharedConfig
+  shareObject: SharedConfig,
+  preparedSkipList: PreparedSkipList
 ): Record<string, SharedConfig> | null {
   const libPackageJson = path.join(libPath, 'package.json');
 
@@ -232,7 +241,7 @@ function readConfiguredSecondaries(
       continue;
     }
 
-    if (isInSkipList(secondaryName, PREPARED_DEFAULT_SKIP_LIST)) {
+    if (isInSkipList(secondaryName, preparedSkipList)) {
       continue;
     }
 
@@ -350,10 +359,9 @@ export function shareAll(
 
   const versionMaps = getVersionMaps(projectPath, projectPath);
   const share: Record<string, unknown> = {};
+  const preparedSkipList = prepareSkipList(skip);
 
   for (const versions of versionMaps) {
-    const preparedSkipList = prepareSkipList(skip);
-
     for (const key in versions) {
       if (isInSkipList(key, preparedSkipList)) {
         continue;
@@ -561,7 +569,8 @@ export function share(
         includeSecondaries,
         libPath,
         key,
-        shareObject
+        shareObject,
+        preparedSkipList
       );
       if (secondaries) {
         addSecondaries(secondaries, result);

libs/native-federation-core/src/lib/core/bundle-shared.ts

@@ -37,7 +37,7 @@ export async function bundleShared(
 
   fs.mkdirSync(cachePath, { recursive: true });
 
-  const inferedPackageInfos = Object.keys(sharedBundles)
+  const inferredPackageInfos = Object.keys(sharedBundles)
     .filter((packageName) => !sharedBundles[packageName].packageInfo)
     .map((packageName) => getPackageInfo(packageName, folder))
     .filter((pi) => !!pi) as PackageInfo[];
@@ -49,7 +49,7 @@ export async function bundleShared(
       ...sharedBundles[packageName].packageInfo,
     })) as PackageInfo[];
 
-  const packageInfos = [...inferedPackageInfos, ...configuredPackageInfos];
+  const packageInfos = [...inferredPackageInfos, ...configuredPackageInfos];
 
   const configState =
     'BUNDLER_CHUNKS' + // TODO: Replace this with lib version

libs/native-federation-esbuild/src/lib/adapter.ts

@@ -44,7 +44,7 @@ export function createEsBuildAdapter(config: EsBuildAdapterConfig) {
 
     for (const entryPoint of entryPoints) {
       const isPkg = entryPoint.fileName.includes('node_modules');
-      const pkgName = isPkg ? inferePkgName(entryPoint.fileName) : '';
+      const pkgName = isPkg ? inferPkgName(entryPoint.fileName) : '';
       const tmpFolder = `node_modules/.tmp/${pkgName}`;
 
       if (isPkg) {
@@ -170,7 +170,7 @@ async function prepareNodePackage(
   });
 }
 
-function inferePkgName(entryPoint: string) {
+function inferPkgName(entryPoint: string) {
   return entryPoint
     .replace(/.*?node_modules/g, '')
     .replace(/[^A-Za-z0-9.]/g, '_');

libs/native-federation-runtime/src/lib/load-remote-module.ts

@@ -88,7 +88,7 @@ function getRemoteNameByOptions(options: LoadRemoteModuleOptions) {
     remoteName = getRemoteNameByBaseUrl(baseUrl);
   } else {
     throw new Error(
-      'unexpcted arguments: Please pass remoteName or remoteEntry'
+      'unexpected arguments: Please pass remoteName or remoteEntry'
     );
   }
 

libs/native-federation/src/builders/build/builder.ts

@@ -199,7 +199,7 @@ export async function* runBuilder(
   const fedOptions: FederationOptions = {
     workspaceRoot: context.workspaceRoot,
     outputPath: browserOutputPath,
-    federationConfig: infereConfigPath(options.tsConfig),
+    federationConfig: inferConfigPath(options.tsConfig),
     tsConfig: options.tsConfig,
     verbose: options.verbose,
     watch: false, // options.watch,
@@ -454,7 +454,7 @@ function getLocaleFilter(
   return localize;
 }
 
-function infereConfigPath(tsConfig: string): string {
+function inferConfigPath(tsConfig: string): string {
   const relProjectPath = path.dirname(tsConfig);
   const relConfigPath = path.join(relProjectPath, 'federation.config.js');