fix: WP-2469 apply cli magic to unpublished templates (#799)

Author: deloreyj

astro-blog-starter-template/worker-configuration.d.ts

@@ -1,6 +1,6 @@
 /* eslint-disable */
 // Generated by Wrangler by running `wrangler types` (hash: 187132f48ddf0f604882ba8213fe386f)
-// Runtime types generated with workerd@1.20250617.0 2025-04-01 nodejs_compat
+// Runtime types generated with workerd@1.20250617.0 2025-10-08 nodejs_compat
 declare namespace Cloudflare {
 	interface Env {
 		ASSETS: Fetcher;
@@ -374,6 +374,8 @@ declare abstract class Navigator {
     sendBeacon(url: string, body?: (ReadableStream | string | (ArrayBuffer | ArrayBufferView) | Blob | FormData | URLSearchParams | URLSearchParams)): boolean;
     readonly userAgent: string;
     readonly hardwareConcurrency: number;
+    readonly language: string;
+    readonly languages: string[];
 }
 /**
 * The Workers runtime supports a subset of the Performance API, used to measure timing and performance,
@@ -2217,6 +2219,7 @@ declare class URLPattern {
     get pathname(): string;
     get search(): string;
     get hash(): string;
+    get hasRegExpGroups(): boolean;
     test(input?: (string | URLPatternInit), baseURL?: string): boolean;
     exec(input?: (string | URLPatternInit), baseURL?: string): URLPatternResult | null;
 }

astro-blog-starter-template/wrangler.json

@@ -1,6 +1,6 @@
 {
   "name": "astro-blog-starter-template",
-  "compatibility_date": "2025-04-01",
+  "compatibility_date": "2025-10-08",
   "compatibility_flags": ["nodejs_compat"],
   "main": "./dist/_worker.js/index.js",
   "assets": {

chanfana-openapi-template/worker-configuration.d.ts

@@ -1,6 +1,6 @@
 /* eslint-disable */
 // Generated by Wrangler by running `wrangler types` (hash: 751a7ef0204e37564547937fa13c0dba)
-// Runtime types generated with workerd@1.20250617.0 2025-04-01 
+// Runtime types generated with workerd@1.20250617.0 2025-10-08 
 declare namespace Cloudflare {
 	interface Env {
 		DB: D1Database;
@@ -374,6 +374,8 @@ declare abstract class Navigator {
     sendBeacon(url: string, body?: (ReadableStream | string | (ArrayBuffer | ArrayBufferView) | Blob | FormData | URLSearchParams | URLSearchParams)): boolean;
     readonly userAgent: string;
     readonly hardwareConcurrency: number;
+    readonly language: string;
+    readonly languages: string[];
 }
 /**
 * The Workers runtime supports a subset of the Performance API, used to measure timing and performance,
@@ -2217,6 +2219,7 @@ declare class URLPattern {
     get pathname(): string;
     get search(): string;
     get hash(): string;
+    get hasRegExpGroups(): boolean;
     test(input?: (string | URLPatternInit), baseURL?: string): boolean;
     exec(input?: (string | URLPatternInit), baseURL?: string): URLPatternResult | null;
 }

chanfana-openapi-template/wrangler.jsonc

@@ -1,5 +1,5 @@
 {
-  "compatibility_date": "2025-04-01",
+  "compatibility_date": "2025-10-08",
   "main": "src/index.ts",
   "name": "chanfana-openapi-template",
   "upload_source_maps": true,

cli/src/deployLiveDemos.ts

@@ -1,6 +1,6 @@
 import "zx/globals";
 import subprocess from "node:child_process";
-import { getTemplates } from "./util";
+import { getPublishedTemplates } from "./util";
 
 export type DeployLiveDemosConfig = {
   templateDirectory: string;
@@ -29,7 +29,7 @@ function runCommand(command: string, cwd: string) {
 export default async function deployLiveDemos({
   templateDirectory,
 }: DeployLiveDemosConfig) {
-  const templates = getTemplates(templateDirectory);
+  const templates = getPublishedTemplates(templateDirectory);
   await Promise.all(
     templates.map(({ path: templatePath }) => {
       runCommand("npm install", templatePath);

cli/src/lint.ts

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
 import {
-  getTemplates,
+  getAllTemplates,
   readJson,
   readJsonC,
   readToml,
@@ -19,7 +19,7 @@ export type LintConfig = {
 const integrationsPlatCategories = ["starter", "storage", "ai"];
 
 export function lint(config: LintConfig) {
-  const templates = getTemplates(config.templateDirectory);
+  const templates = getAllTemplates(config.templateDirectory);
   const results = templates.flatMap((template) =>
     lintTemplate(template, config.fix),
   );
@@ -42,7 +42,7 @@ const CHECKS = {
   "package.json": [lintPackageJson],
   ".gitignore": [lintGitIgnore],
 };
-const TARGET_COMPATIBILITY_DATE = "2025-04-01";
+const TARGET_COMPATIBILITY_DATE = "2025-10-08";
 const DASH_CONTENT_START_MARKER = "<!-- dash-content-start -->";
 const DASH_CONTENT_END_MARKER = "<!-- dash-content-end -->";
 
@@ -290,6 +290,7 @@ function lintPackageJson(
       icon_urls?: string[];
       preview_image_url?: string;
       preview_icon_url?: string;
+      publish?: boolean;
     };
   };
 
@@ -323,13 +324,16 @@ function lintPackageJson(
           );
       });
     }
-    // Ensure a preview image URL is set
-    if (!pkg.cloudflare.preview_image_url) {
-      problems.push('"cloudflare.preview_image_url" must be defined');
-    }
-    // Ensure preview_icon_url is set
-    if (!pkg.cloudflare.preview_icon_url) {
-      problems.push('"cloudflare.preview_icon_url" must be defined');
+    // Only require preview URLs for published templates
+    if (pkg.cloudflare.publish === true) {
+      // Ensure a preview image URL is set
+      if (!pkg.cloudflare.preview_image_url) {
+        problems.push('"cloudflare.preview_image_url" must be defined');
+      }
+      // Ensure preview_icon_url is set
+      if (!pkg.cloudflare.preview_icon_url) {
+        problems.push('"cloudflare.preview_icon_url" must be defined');
+      }
     }
   }
 

cli/src/npm.ts

@@ -3,7 +3,7 @@ import { z } from "zod";
 import { createHash } from "node:crypto";
 import path from "node:path";
 import MarkdownError from "./MarkdownError";
-import { getTemplates } from "./util";
+import { getAllTemplates } from "./util";
 
 export type NpmLockfilesConfig = {
   templateDirectory: string;
@@ -19,7 +19,7 @@ export async function generateNpmLockfiles({
 }: NpmLockfilesConfig): Promise<void> {
   const repoRoot = path.resolve(templateDirectory);
   const config = await new TemplatesConfig(repoRoot).load();
-  const templates = getTemplates(templateDirectory);
+  const templates = getAllTemplates(templateDirectory);
 
   for (const { name } of templates) {
     echo(chalk.blue(`Updating template: ${chalk.grey(name)}`));
@@ -58,7 +58,7 @@ export async function lintNpmLockfiles({
 }: NpmLockfilesConfig): Promise<void> {
   const repoRoot = path.resolve(templateDirectory);
   const config = await new TemplatesConfig(repoRoot).load();
-  const templates = getTemplates(templateDirectory);
+  const templates = getAllTemplates(templateDirectory);
 
   const errors: string[] = [];
   for (const { name } of templates) {

cli/src/upload.ts

@@ -1,7 +1,7 @@
 import {
   collectTemplateFiles,
   fetchWithRetries,
-  getTemplates,
+  getPublishedTemplates,
   handleCloudflareResponse,
   SeedRepo,
 } from "./util";
@@ -20,7 +20,7 @@ export type UploadConfig = {
 };
 
 export async function upload(config: UploadConfig) {
-  const templates = getTemplates(config.templateDirectory);
+  const templates = getPublishedTemplates(config.templateDirectory);
   const formData = templates.reduce((body, template) => {
     const files = collectTemplateFiles(template.path, !!config.seedRepo);
     console.info(`Uploading ${template.path}:`);

cli/src/util.ts

@@ -41,13 +41,13 @@ export const ALLOWED_DIRECTORIES = [
   "test-results",
 ];
 
-export function getTemplates(templateDirectory: string): Template[] {
+export function getAllTemplates(templateDirectory: string): Template[] {
   if (path.basename(templateDirectory).endsWith(TEMPLATE_DIRECTORY_SUFFIX)) {
     // If the specified path is a template directory, just return that.
     const templatePath = templateDirectory;
     const packageJsonPath = path.join(templatePath, "package.json");
 
-    if (!isDashTemplate(packageJsonPath)) {
+    if (!fs.existsSync(packageJsonPath)) {
       return [];
     }
 
@@ -81,20 +81,33 @@ export function getTemplates(templateDirectory: string): Template[] {
     .filter((name) =>
       fs.statSync(path.join(templateDirectory, name)).isDirectory(),
     )
+    .filter((name) => name.endsWith(TEMPLATE_DIRECTORY_SUFFIX))
     .filter((name) => {
       const packageJsonPath = path.join(
         templateDirectory,
         name,
         "package.json",
       );
-      return isDashTemplate(packageJsonPath);
+      return fs.existsSync(packageJsonPath);
     })
     .map((name) => ({
       name,
       path: path.join(templateDirectory, name),
     }));
 }
 
+export function getPublishedTemplates(templateDirectory: string): Template[] {
+  return getAllTemplates(templateDirectory).filter((template) => {
+    const packageJsonPath = path.join(template.path, "package.json");
+    return isDashTemplate(packageJsonPath);
+  });
+}
+
+// Deprecated: Use getAllTemplates() or getPublishedTemplates() instead
+export function getTemplates(templateDirectory: string): Template[] {
+  return getPublishedTemplates(templateDirectory);
+}
+
 export function collectTemplateFiles(
   templatePath: string,
   onlySeedRepoFiles?: boolean,

cli/src/validateD2CButtons.ts

@@ -1,6 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
-import { getTemplates } from "./util";
+import { getAllTemplates } from "./util";
 import MarkdownError from "./MarkdownError";
 
 export type ValidateD2CButtonsConfig = {
@@ -10,7 +10,7 @@ export type ValidateD2CButtonsConfig = {
 export async function validateD2CButtons({
   templateDirectory,
 }: ValidateD2CButtonsConfig) {
-  const templates = getTemplates(templateDirectory);
+  const templates = getAllTemplates(templateDirectory);
   const successes: string[] = [];
   const errors: string[] = [];
   let numBadStatuses = 0;