Skip to content

Commit

Permalink
fix(react): Update error message for invalid remote name
Browse files Browse the repository at this point in the history
  • Loading branch information
ndcunningham committed Oct 20, 2023
1 parent 1f0a54e commit 58ee114
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 3 deletions.
9 changes: 8 additions & 1 deletion packages/angular/src/utils/mf/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,14 @@ export function getFunctionDeterminateRemoteUrl(isServer: boolean = false) {
const remoteEntry = isServer ? 'server/remoteEntry.js' : 'remoteEntry.mjs';

return function (remote: string) {
const remoteConfiguration = readCachedProjectConfiguration(remote);
let remoteConfiguration = null;
try {
remoteConfiguration = readCachedProjectConfiguration(remote);
} catch (e) {
throw new Error(
`Cannot find remote "${remote}". Check that the remote name is correct in your module federation config file.\n`
);
}
const serveTarget = remoteConfiguration?.targets?.[target];

if (!serveTarget) {
Expand Down
6 changes: 5 additions & 1 deletion packages/nx/src/project-graph/project-graph.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,11 @@ export function readCachedProjectConfiguration(
): ProjectConfiguration {
const graph = readCachedProjectGraph();
const node = graph.nodes[projectName];
return node.data;
try {
return node.data;
} catch (e) {
throw new Error(`Cannot find project: '${projectName}' in your workspace.`);
}
}

/**
Expand Down
9 changes: 8 additions & 1 deletion packages/react/src/module-federation/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,14 @@ export function getFunctionDeterminateRemoteUrl(isServer: boolean = false) {
const remoteEntry = isServer ? 'server/remoteEntry.js' : 'remoteEntry.js';

return function (remote: string) {
const remoteConfiguration = readCachedProjectConfiguration(remote);
let remoteConfiguration = null;
try {
remoteConfiguration = readCachedProjectConfiguration(remote);
} catch (e) {
throw new Error(
`Cannot find remote: "${remote}". Check that the remote name is correct in your module federation config file.\n`
);
}
const serveTarget = remoteConfiguration?.targets?.[target];

if (!serveTarget) {
Expand Down

0 comments on commit 58ee114

Please sign in to comment.