@pylonsync/sdk 0.11.5 → 0.12.0

package/dist/index.d.ts

@@ -684,6 +684,56 @@ export interface AppManifest {
      *  `pylon deploy` refuses to ship when one is missing from the
      *  project's secrets. */
     requiredEnv?: ManifestRequiredEnv[];
+    /** Production build settings (`pylon build`). */
+    build?: BuildConfig;
+}
+/**
+ * Production build settings for `pylon build`. Dev (`pylon dev`) ignores the
+ * browser settings; they apply to the artifact and to `pylon start` builds.
+ *
+ * ```ts
+ * buildManifest({
+ *   // ...
+ *   build: {
+ *     target: ["chrome >= 90", "safari >= 14", "firefox >= 90"],
+ *     polyfill: "usage",
+ *     server: { external: ["sharp"] },
+ *   },
+ * });
+ * ```
+ */
+export interface BuildConfig {
+    /** Browser target for client JavaScript: an ECMAScript edition
+     *  (`"es2018"` to `"es2022"`) or browserslist queries. Unset: modern
+     *  output, no lowering. Needs `@swc/core` and `browserslist` in the app's
+     *  devDependencies. The client runtime needs ES modules and dynamic
+     *  `import()`, so browsers older than Chrome 63, Firefox 67, and Safari
+     *  11.1 cannot run it at any target. */
+    target?: string | string[];
+    /** core-js polyfills for `target`. `"usage"` adds the features the bundle
+     *  uses; `"entry"` adds every feature the target lacks. Needs `core-js` in
+     *  the app's dependencies. Default `false`. */
+    polyfill?: false | "usage" | "entry";
+    css?: {
+        /** CSS target (prefixes, nesting, color syntax), same forms as
+         *  `target`. Default: `target`. Needs `lightningcss` and `browserslist`. */
+        target?: string | string[];
+    };
+    /** Emit source maps for the client and server bundles. Default `false`. */
+    sourcemap?: boolean;
+    server?: {
+        /** Bundle npm packages into the server output. `false` ships every
+         *  dependency in `server/node_modules`. Default `true`. */
+        bundle?: boolean;
+        /** Packages to leave out of the server bundle and ship in
+         *  `server/node_modules` with their dependencies: native modules and
+         *  packages that read their own files at run time. */
+        external?: string[];
+    };
+    /** Project files or directories to copy into the artifact, for files the
+     *  app reads from disk at run time (content/, data/). `public/` and image
+     *  files under `app/` are always copied. */
+    include?: string[];
 }
 /** One environment variable the app declares it needs. */
 export interface ManifestRequiredEnv {
@@ -1121,6 +1171,8 @@ export declare function buildManifest(options: {
      *  ```
      */
     requiredEnv?: ManifestRequiredEnv[];
+    /** Production build settings for `pylon build`. See `BuildConfig`. */
+    build?: BuildConfig;
     /** Set by `discoverFunctions()` (spread its result into this call).
      *  When true, the framework's AgentRun/AgentMessage entities and
      *  their owner-scoping policies are appended to the manifest —

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.11.5",
+  "version": "0.12.0",
   "type": "module",
   "main": "./src/index.ts",
   "types": "./dist/index.d.ts",

package/src/build-config.test.ts

@@ -0,0 +1,47 @@
+/**
+ * `buildManifest({ build })`: the production build block reaches the
+ * manifest as written, and a value `pylon build` cannot use fails when app.ts
+ * runs, not at build time.
+ */
+import { expect, test } from "bun:test";
+import { buildManifest, entity, field } from "./index";
+
+const base = {
+  name: "t",
+  version: "0",
+  entities: [entity("Doc", { title: field.string() })],
+  routes: [],
+};
+
+test("the build block is copied into the manifest", () => {
+  const build = {
+    target: ["chrome >= 90", "safari >= 14"],
+    polyfill: "usage" as const,
+    css: { target: "es2019" },
+    server: { external: ["sharp"] },
+    include: ["content"],
+  };
+  expect(buildManifest({ ...base, build }).build).toEqual(build);
+});
+
+test("no build block, or an empty one, adds nothing", () => {
+  expect("build" in buildManifest(base)).toBe(false);
+  expect("build" in buildManifest({ ...base, build: {} })).toBe(false);
+});
+
+test("invalid values throw with the field name", () => {
+  const bad: Array<[unknown, string]> = [
+    [{ target: [] }, "build.target"],
+    [{ target: [""] }, "build.target"],
+    [{ polyfill: "all" }, "build.polyfill"],
+    [{ polyfill: "usage" }, "polyfill needs build.target"],
+    [{ css: { target: 5 } }, "build.css.target"],
+    [{ sourcemap: "yes" }, "build.sourcemap"],
+    [{ server: { bundle: "no" } }, "build.server.bundle"],
+    [{ server: { external: [""] } }, "build.server.external"],
+    [{ include: "content" }, "build.include"],
+  ];
+  for (const [build, msg] of bad) {
+    expect(() => buildManifest({ ...base, build: build as any })).toThrow(msg);
+  }
+});

package/src/index.ts

@@ -873,6 +873,95 @@ export interface AppManifest {
    *  `pylon deploy` refuses to ship when one is missing from the
    *  project's secrets. */
   requiredEnv?: ManifestRequiredEnv[];
+  /** Production build settings (`pylon build`). */
+  build?: BuildConfig;
+}
+
+/**
+ * Production build settings for `pylon build`. Dev (`pylon dev`) ignores the
+ * browser settings; they apply to the artifact and to `pylon start` builds.
+ *
+ * ```ts
+ * buildManifest({
+ *   // ...
+ *   build: {
+ *     target: ["chrome >= 90", "safari >= 14", "firefox >= 90"],
+ *     polyfill: "usage",
+ *     server: { external: ["sharp"] },
+ *   },
+ * });
+ * ```
+ */
+export interface BuildConfig {
+  /** Browser target for client JavaScript: an ECMAScript edition
+   *  (`"es2018"` to `"es2022"`) or browserslist queries. Unset: modern
+   *  output, no lowering. Needs `@swc/core` and `browserslist` in the app's
+   *  devDependencies. The client runtime needs ES modules and dynamic
+   *  `import()`, so browsers older than Chrome 63, Firefox 67, and Safari
+   *  11.1 cannot run it at any target. */
+  target?: string | string[];
+  /** core-js polyfills for `target`. `"usage"` adds the features the bundle
+   *  uses; `"entry"` adds every feature the target lacks. Needs `core-js` in
+   *  the app's dependencies. Default `false`. */
+  polyfill?: false | "usage" | "entry";
+  css?: {
+    /** CSS target (prefixes, nesting, color syntax), same forms as
+     *  `target`. Default: `target`. Needs `lightningcss` and `browserslist`. */
+    target?: string | string[];
+  };
+  /** Emit source maps for the client and server bundles. Default `false`. */
+  sourcemap?: boolean;
+  server?: {
+    /** Bundle npm packages into the server output. `false` ships every
+     *  dependency in `server/node_modules`. Default `true`. */
+    bundle?: boolean;
+    /** Packages to leave out of the server bundle and ship in
+     *  `server/node_modules` with their dependencies: native modules and
+     *  packages that read their own files at run time. */
+    external?: string[];
+  };
+  /** Project files or directories to copy into the artifact, for files the
+   *  app reads from disk at run time (content/, data/). `public/` and image
+   *  files under `app/` are always copied. */
+  include?: string[];
+}
+
+/** Throws when `build` has a value `pylon build` cannot use. */
+function validateBuildConfig(build: BuildConfig): void {
+  const fail = (msg: string): never => {
+    throw new Error(`buildManifest: build.${msg}`);
+  };
+  const checkTarget = (t: unknown, field: string) => {
+    if (t === undefined) return;
+    const list = Array.isArray(t) ? t : [t];
+    if (list.length === 0 || list.some((x) => typeof x !== "string" || x.trim() === "")) {
+      fail(`${field} must be a non-empty string or a list of non-empty strings`);
+    }
+  };
+  checkTarget(build.target, "target");
+  checkTarget(build.css?.target, "css.target");
+  const pf = build.polyfill;
+  if (pf !== undefined && pf !== false && pf !== "usage" && pf !== "entry") {
+    fail(`polyfill must be false, "usage", or "entry"`);
+  }
+  if (pf && build.target === undefined) {
+    fail(`polyfill needs build.target (the browsers to polyfill for)`);
+  }
+  if (build.sourcemap !== undefined && typeof build.sourcemap !== "boolean") {
+    fail(`sourcemap must be a boolean`);
+  }
+  if (build.server?.bundle !== undefined && typeof build.server.bundle !== "boolean") {
+    fail(`server.bundle must be a boolean`);
+  }
+  for (const [field, list] of [
+    ["server.external", build.server?.external],
+    ["include", build.include],
+  ] as const) {
+    if (list === undefined) continue;
+    if (!Array.isArray(list) || list.some((x) => typeof x !== "string" || x === "")) {
+      fail(`${field} must be a list of non-empty strings`);
+    }
+  }
 }
 
 /** One environment variable the app declares it needs. */
@@ -1166,11 +1255,19 @@ export async function discoverAppRoutes(opts?: {
 
   // Resolve the first existing `<base>.{tsx,ts,jsx,js}` in `dir` and
   // return it as a cwd-relative, extension-less module path (or null).
+  // Component paths always use "/", on Windows too: they are manifest keys,
+  // client-bundle import specifiers, and the input to the SSR runtime's
+  // boundary walk, all of which split on "/".
   const findModule = (dir: string, base: string): string | null => {
     const hit = [`${base}.tsx`, `${base}.ts`, `${base}.jsx`, `${base}.js`]
       .map((n: string) => path.join(dir, n))
       .find((p: string) => fs.existsSync(p));
-    return hit ? path.relative(cwd, hit).replace(/\.(tsx?|jsx?)$/, "") : null;
+    return hit
+      ? path
+          .relative(cwd, hit)
+          .replace(/\.(tsx?|jsx?)$/, "")
+          .replace(/\\/g, "/")
+      : null;
   };
 
   function walk(dir: string, segments: string[], layouts: string[]): void {
@@ -2142,6 +2239,8 @@ export function buildManifest(options: {
    *  ```
    */
   requiredEnv?: ManifestRequiredEnv[];
+  /** Production build settings for `pylon build`. See `BuildConfig`. */
+  build?: BuildConfig;
   /** Set by `discoverFunctions()` (spread its result into this call).
    *  When true, the framework's AgentRun/AgentMessage entities and
    *  their owner-scoping policies are appended to the manifest —
@@ -2162,6 +2261,7 @@ export function buildManifest(options: {
   // can `.policies(policy({ allowRead: "..." }))` without one. Auto-
   // derive from the entity + a counter so two attached policies
   // don't collide.
+  if (options.build) validateBuildConfig(options.build);
   const attached: PolicyDefinition[] = [];
   for (const ent of options.entities) {
     const extracted = extractAttachedPolicies(ent);
@@ -2225,6 +2325,9 @@ export function buildManifest(options: {
     ...(options.requiredEnv && options.requiredEnv.length > 0
       ? { requiredEnv: options.requiredEnv }
       : {}),
+    ...(options.build && Object.keys(options.build).length > 0
+      ? { build: options.build }
+      : {}),
   };
 }