Skip to content

Commit beb5c1a

Browse files
authored
fix(script): build javascript-algoliasearch after other clients (#223)
1 parent ac441c9 commit beb5c1a

File tree

1 file changed

+16
-2
lines changed

1 file changed

+16
-2
lines changed

scripts/buildClients.ts

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,20 @@ async function buildAllClients(
8686
}
8787

8888
export async function buildClients(
89-
generators: Generator[],
89+
allGenerators: Generator[],
9090
verbose: boolean
9191
): Promise<void> {
92-
const langs = [...new Set(generators.map((gen) => gen.language))];
92+
const langs = [...new Set(allGenerators.map((gen) => gen.language))];
93+
94+
// We exclude `javascript-algoliasearch` from the build batch because it
95+
// is made of built generated clients and can cause race issue when executed
96+
// together.
97+
const jsAlgoliasearch = allGenerators.find(
98+
(gen) => gen.key === 'javascript-algoliasearch'
99+
);
100+
const generators = allGenerators.filter(
101+
(gen) => gen.key !== 'javascript-algoliasearch'
102+
);
93103

94104
await Promise.all([
95105
Promise.all(
@@ -103,4 +113,8 @@ export async function buildClients(
103113
.map((lang) => buildAllClients(lang, verbose))
104114
),
105115
]);
116+
117+
if (jsAlgoliasearch) {
118+
await buildPerClient(jsAlgoliasearch, verbose);
119+
}
106120
}

0 commit comments

Comments
 (0)