Skip to content

Commit dc30b81

Browse files
committed
don't conflate installing workers-types and generating types
1 parent 9e14fbe commit dc30b81

File tree

8 files changed

+13
-6
lines changed

8 files changed

+13
-6
lines changed

packages/create-cloudflare/src/__tests__/workers.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ describe("updateTsConfig", () => {
2525
ctx = createTestContext();
2626
ctx.args.ts = true;
2727
ctx.template.installWorkersTypes = false;
28+
ctx.template.skipWranglerTypegen = false;
2829

2930
vi.mocked(existsSync).mockImplementation(() => true);
3031
vi.mocked(getLatestTypesEntrypoint).mockReturnValue(mockCompatDate);

packages/create-cloudflare/src/cli.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,9 @@ const configure = async (ctx: C3Context) => {
170170
if (ctx.template.installWorkersTypes) {
171171
await installWorkersTypes(ctx);
172172
}
173-
await generateWorkersTypes(ctx);
173+
if (ctx.template.skipWranglerTypegen) {
174+
await generateWorkersTypes(ctx);
175+
}
174176

175177
await offerGit(ctx);
176178
await gitCommit(ctx);

packages/create-cloudflare/src/templates.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,8 +150,11 @@ export type TemplateConfig = {
150150

151151
bindings?: Record<string, unknown>;
152152

153-
/** Don't generate types, use @cloudflare/workers-types instead. Usually because the framework is pinned to Wrangler 3. Default false. */
153+
/** Default false. For frameworks that are pinned to Wrangler 3. */
154154
installWorkersTypes?: boolean;
155+
156+
/** Default false. To accomodate SSG frameworks etc. */
157+
skipWranglerTypegen?: boolean;
155158
};
156159

157160
type CopyFiles = (StaticFileMap | VariantInfo) & {

packages/create-cloudflare/src/workers.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ export async function updateTsConfig(ctx: C3Context) {
120120
}
121121

122122
/**
123+
* TODO: delete if/when qwik and remix move to wrangler v4
123124
* Installs the latest version of the `@cloudflare/workers-types` package
124125
* and updates the .tsconfig file to use the latest entrypoint version.
125126
*/

packages/create-cloudflare/templates/docusaurus/pages/c3.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,6 @@ const config: TemplateConfig = {
2727
devScript: "preview",
2828
deployScript: "deploy",
2929
previewScript: "preview",
30-
installWorkersTypes: true,
30+
skipWranglerTypegen: false,
3131
};
3232
export default config;

packages/create-cloudflare/templates/docusaurus/workers/c3.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ const config: TemplateConfig = {
2929
devScript: "start",
3030
deployScript: "deploy",
3131
previewScript: "preview",
32-
installWorkersTypes: true,
32+
skipWranglerTypegen: true,
3333
};
3434
export default config;

packages/create-cloudflare/templates/gatsby/pages/c3.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,6 @@ const config: TemplateConfig = {
4747
devScript: "develop",
4848
deployScript: "deploy",
4949
previewScript: "preview",
50-
installWorkersTypes: true,
50+
skipWranglerTypegen: true,
5151
};
5252
export default config;

packages/create-cloudflare/templates/gatsby/workers/c3.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,6 @@ const config: TemplateConfig = {
4949
devScript: "develop",
5050
deployScript: "deploy",
5151
previewScript: "preview",
52-
installWorkersTypes: true,
52+
skipWranglerTypegen: true,
5353
};
5454
export default config;

0 commit comments

Comments
 (0)