@yahoo/uds-create-loader 2.45.0 → 3.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yahoo/uds-create-loader",
-  "version": "2.45.0",
+  "version": "3.0.1",
   "type": "module",
   "private": false,
   "files": [
@@ -38,7 +38,7 @@
     "test:update": "bun test src/ --update-snapshots"
   },
   "dependencies": {
-    "@yahoo/uds-create-config": "2.45.0",
+    "@yahoo/uds-create-config": "3.0.1",
     "@babel/parser": "^7.29.0",
     "@babel/traverse": "^7.29.0",
     "@babel/types": "^7.29.0",
@@ -62,5 +62,6 @@
     "webpack": {
       "optional": true
     }
-  }
+  },
+  "udsSchemaVersion": 20260910212453
 }

package/src/loader/artifacts.ts

@@ -104,7 +104,10 @@ export function getLoaderProjection(
   const cached = projectionCache.get(config);
   if (cached) return cached;
 
-  const components = (config.components ?? {}) as Record<string, unknown>;
+  // Items are grouped BY KIND in a serialized config; a top-level `components` map is the shape the
+  // old artifact had. Read the old way it answered `{}`, so `componentNames` was empty and the
+  // loader's pre-filter concluded no file referenced a UDS component — it transformed nothing at all.
+  const components = config.items?.component ?? {};
   const componentNames = new Set<string>(Object.keys(components));
 
   const projection: LoaderProjection = { componentNames };

package/src/loader.test.ts

@@ -1,22 +1,18 @@
 /**
  * Unit tests for the trimmed `transformLoader`.
  *
- * Fixtures use the public authoring API (`defineComponent`,
- * `defineStyleProp`, `uds.register*`) and call `Config.toJSON()` so the
- * loader sees exactly the serialized-config shape it would in production.
- * Avoids coupling tests to internal serialized layer / marker shape —
- * those shapes can evolve without touching this file.
+ * Fixtures are built through the public entity API (`System.register`,
+ * `Component.item`, `StyleProperty.item`) and serialized with `toJSON()`, so the
+ * loader sees exactly the shape it would in production. Avoids coupling tests to
+ * the internal serialized layer / prop shape — those can evolve without touching
+ * this file.
  */
 
 import { describe, expect, it } from 'bun:test';
 import path from 'node:path';
 import { parse } from '@babel/parser';
 import type { SerializedConfig } from '@yahoo/uds-create-config';
-import {
-  Config,
-  defineComponent,
-  defineStyleProp,
-} from '@yahoo/uds-create-config';
+import { Component, StyleProperty, System } from '@yahoo/uds-create-config';
 import transformLoader from './loader';
 import { seedUdsConfig } from './loader/artifacts';
 
@@ -47,35 +43,33 @@ function runLoader(
 // Shared config factories — authoring-API based
 // ---------------------------------------------------------------------------
 
-const bg = defineStyleProp({
-  cssProperty: 'background-color',
-  classPrefix: 'bg',
-  values: [],
-});
-
-const padding = defineStyleProp({
-  cssProperty: 'padding',
-  classPrefix: 'p',
-  values: [],
-});
-
 /** Single-layer primitive `<Box>` with two style-prop bindings. Lowerable. */
 function makeBoxConfig(): SerializedConfig {
-  const Box = defineComponent('div').config(({ styleProp }) => ({
-    props: {
-      bg: styleProp('bg'),
-      padding: styleProp('padding'),
-    },
-  }));
-  const config = new Config()
-    .registerStyleProps({ bg, padding })
-    .registerComponents({ Box });
-  return config.toJSON();
+  return new System()
+    .register({
+      styleProps: {
+        bg: StyleProperty.item({ properties: ['background-color'] }),
+        padding: StyleProperty.item({ properties: ['padding'] }),
+      },
+      components: {
+        Box: Component.item({
+          layers: { root: 'div' },
+          props: {
+            bg: { type: 'styleProperty', value: StyleProperty.ref('bg') },
+            padding: {
+              type: 'styleProperty',
+              value: StyleProperty.ref('padding'),
+            },
+          },
+        }),
+      },
+    })
+    .toJSON() as SerializedConfig;
 }
 
 /** Empty config — no registered components. */
 function makeEmptyConfig(): SerializedConfig {
-  return new Config().toJSON();
+  return new System().toJSON() as SerializedConfig;
 }
 
 describe('loader — new-API import rewrite', () => {
@@ -265,32 +259,14 @@ export function Demo() {
     expect(output).toBe(source);
   });
 
-  it('skips the default registry location under componentsDir', () => {
-    const config = {
-      ...makeBoxConfig(),
-      buildOptions: { componentsDir: 'src/uds' },
-    };
-    const source = `
-import { Box } from '@/uds/components/box';
-
-export function Demo() {
-  return <Box bg="brand">hi</Box>;
-}
-`;
-    const output = runLoader(source, config, {
-      resourcePath: path.join(
-        process.cwd(),
-        'src/uds/registry/aaaaaaaa-1111-4111-8111-111111111111@1.0.0/src/uds/components/demo.tsx',
-      ),
-    });
-
-    expect(output).toBe(source);
-  });
-
-  it('skips an explicit buildOptions.registryDir', () => {
+  it('skips an explicit build.registryDir', () => {
+    const box = makeBoxConfig();
     const config = {
-      ...makeBoxConfig(),
-      buildOptions: { registryDir: 'vendor/linked' },
+      ...box,
+      items: {
+        ...box.items,
+        settings: { settings: { build: { registryDir: 'vendor/linked' } } },
+      },
     };
     const source = `
 import { Box } from '@/uds/components/box';

package/src/loader.ts

@@ -70,9 +70,19 @@ const transformLoader: LoaderDefinitionFunction = function (source) {
   const config = loadUdsConfig(dataPath, isDev);
   if (!config) return source;
 
-  // Configurable registry location (`buildOptions.registryDir`, defaulting
-  // to `<componentsDir>/registry`) — same skip rationale as above.
-  const registryRel = resolveRegistryDir(config.buildOptions);
+  // Configurable registry location (`build.registryDir`) — same skip rationale as above. The
+  // settings are one item of the `settings` kind in a serialized config, keyed by the kind.
+  const registryRel = resolveRegistryDir(
+    (
+      config as {
+        items?: {
+          settings?: {
+            settings?: { build?: Parameters<typeof resolveRegistryDir>[0] };
+          };
+        };
+      }
+    ).items?.settings?.settings?.build,
+  );
   if (resourcePath.replace(/\\/g, '/').includes(`/${registryRel}/`)) {
     return source;
   }

package/src/next.test.ts

@@ -6,12 +6,12 @@ import { withUds } from './next';
 // Repo rule: test artifacts under the package's dist/.test-tmp.
 const TMP = path.join(import.meta.dir, '../dist/.test-tmp/next-registry-deps');
 
-const REGISTRY_REL = 'src/uds/registry';
+const REGISTRY_REL = '.uds-registry';
 
 function writeFixture(args: {
   registryDeps?: Record<string, string>;
   consumerHas?: string[];
-  buildOptions?: { componentsDir?: string; registryDir?: string };
+  build?: { registryDir?: string };
 }): string {
   const root = path.join(TMP, `proj-${Math.random().toString(36).slice(2, 8)}`);
 
@@ -19,12 +19,15 @@ function writeFixture(args: {
   writeFileSync(
     path.join(root, '.uds/config.json'),
     JSON.stringify({
-      buildOptions: args.buildOptions ?? { componentsDir: 'src/uds' },
+      items: { settings: { settings: { build: args.build ?? {} } } },
     }),
   );
 
   if (args.registryDeps) {
-    const registryDir = path.join(root, REGISTRY_REL);
+    const registryDir = path.join(
+      root,
+      args.build?.registryDir ?? REGISTRY_REL,
+    );
     mkdirSync(path.join(registryDir, 'node_modules'), { recursive: true });
     writeFileSync(
       path.join(registryDir, 'package.json'),
@@ -141,10 +144,10 @@ describe('withUds — linked-registry dependency resolution', () => {
     ).toBeUndefined();
   });
 
-  it('honors a custom buildOptions.registryDir', () => {
+  it('honors a custom build.registryDir', () => {
     const root = writeFixture({
       registryDeps: { clsx: '^2.0.0' },
-      buildOptions: { registryDir: 'src/uds/registry' },
+      build: { registryDir: 'src/uds/registry' },
     });
 
     const config = applyIn(root, () => withUds({}));

package/src/next.ts

@@ -27,9 +27,7 @@ function resolveLoaderPath(): string {
 function readRegistryDeps(
   outputDir: string,
 ): { registryDirRel: string; nodeModules: string; deps: string[] } | null {
-  let buildOptions:
-    | { componentsDir?: string; registryDir?: string }
-    | undefined;
+  let buildOptions: { registryDir?: string } | undefined;
   try {
     buildOptions = (
       JSON.parse(
@@ -37,8 +35,10 @@ function readRegistryDeps(
           path.resolve(process.cwd(), outputDir, 'config.json'),
           'utf-8',
         ),
-      ) as { buildOptions?: typeof buildOptions }
-    ).buildOptions;
+      ) as {
+        items?: { settings?: { settings?: { build?: typeof buildOptions } } };
+      }
+    ).items?.settings?.settings?.build;
   } catch {
     return null;
   }

package/dist/inspect-transform.js

@@ -1,63 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-import { pathToFileURL } from 'node:url';
-import { runLoaderTransform } from './transform-runner';
-function readFlag(flag) {
-    const index = process.argv.indexOf(flag);
-    if (index === -1)
-        return null;
-    const value = process.argv[index + 1];
-    if (!value || value.startsWith('-'))
-        return null;
-    return value;
-}
-function printUsage() {
-    console.error([
-        'Usage:',
-        '  bun run src/inspect-transform.ts --file <path-to-tsx> --config <path-to-uds.config.ts>',
-        '',
-        'Example:',
-        '  bun run --filter @yahoo/uds-create-loader inspect:transform -- --file packages/components/src/accordion.tsx --config apps/studio/uds.config.ts',
-    ].join('\n'));
-    process.exit(1);
-}
-async function loadUdsConfig(configPath) {
-    const resolvedPath = path.resolve(configPath);
-    const configUrl = pathToFileURL(resolvedPath).href;
-    const module = (await import(configUrl));
-    if (module.default !== undefined) {
-        return module.default;
-    }
-    if (module.udsConfig !== undefined) {
-        return module.udsConfig;
-    }
-    return module;
-}
-async function main() {
-    const sourcePath = readFlag('--file');
-    const configPath = readFlag('--config');
-    if (!sourcePath || !configPath) {
-        printUsage();
-    }
-    const sourceAbsolutePath = path.resolve(sourcePath);
-    if (!fs.existsSync(sourceAbsolutePath)) {
-        throw new Error(`Source file not found: ${sourceAbsolutePath}`);
-    }
-    const rawConfig = await loadUdsConfig(configPath);
-    const config = rawConfig.toJSON();
-    const source = fs.readFileSync(sourceAbsolutePath, 'utf8');
-    const result = runLoaderTransform(source, {
-        config,
-        resourcePath: sourceAbsolutePath,
-        isDev: false,
-    });
-    if ('error' in result) {
-        throw new Error(result.error);
-    }
-    process.stdout.write(result.output);
-}
-void main().catch((error) => {
-    const message = error instanceof Error ? error.message : String(error);
-    console.error(`[inspect-transform] ${message}`);
-    process.exit(1);
-});