@bleedingdev/modern-js-runtime-extensions 3.9.0-ultramodern.6 → 3.9.0-ultramodern.7

package/package.json

@@ -11,7 +11,7 @@
     "directory": "packages/runtime/runtime-extensions"
   },
   "license": "MIT",
-  "version": "3.9.0-ultramodern.6",
+  "version": "3.9.0-ultramodern.7",
   "engines": {
     "node": ">=26.7.0"
   },
@@ -274,7 +274,7 @@
   },
   "devDependencies": {
     "@loadable/component": "5.16.7",
-    "@modern-js/utils": "npm:@bleedingdev/modern-js-utils@3.9.0-ultramodern.6",
+    "@modern-js/utils": "npm:@bleedingdev/modern-js-utils@3.9.0-ultramodern.7",
     "@rsbuild/core": "2.2.3",
     "@rslib/core": "1.0.0",
     "@scripts/rstest-config": "2.66.0",
@@ -303,8 +303,8 @@
     }
   },
   "dependencies": {
-    "@modern-js/create-request": "npm:@bleedingdev/modern-js-create-request@3.9.0-ultramodern.6",
-    "@modern-js/runtime-utils": "npm:@bleedingdev/modern-js-runtime-utils@3.9.0-ultramodern.6",
+    "@modern-js/create-request": "npm:@bleedingdev/modern-js-create-request@3.9.0-ultramodern.7",
+    "@modern-js/runtime-utils": "npm:@bleedingdev/modern-js-runtime-utils@3.9.0-ultramodern.7",
     "react-helmet-async": "3.0.0"
   },
   "scripts": {

package/tests/buildIdentity.test.ts

@@ -28,25 +28,6 @@ const readers = {
   sourceRevision: () => 'compiled-revision',
 };
 
-it('applies injected identity to every surface without mutating the artifact', () => {
-  const resolved = resolveUltramodernBuildArtifact(artifact, readers);
-  for (const record of [
-    resolved.deliveryUnit,
-    resolved.surfaces.api,
-    resolved.surfaces.ui,
-  ]) {
-    expect(record).toMatchObject({
-      build: 'compiled-build',
-      buildMarker: 'compiled-build',
-      sourceRevision: 'compiled-revision',
-    });
-  }
-  expect(resolved.deliveryUnit.unitId).toBe('shop/catalog');
-  expect(resolved.surfaces.api.surface).toBe('api');
-  expect(resolved.surfaces.ui.surface).toBe('ui');
-  expect(artifact.deliveryUnit.buildMarker).toBe('original');
-});
-
 it('falls back only for missing compiler references and preserves each supplied value', () => {
   expect(resolveUltramodernBuildArtifact(artifact)).toEqual(artifact);
 

package/tests/contextExtensions.test.ts

@@ -1,50 +1,6 @@
 import { createRuntimeContextExtension } from '../src/contextExtensions';
 
 describe('runtime context extensions', () => {
-  it('stores and retrieves typed values per context object', () => {
-    const extension = createRuntimeContextExtension<{ value: number }>(
-      'test:isolation-a',
-    );
-    const contextA = {};
-    const contextB = {};
-
-    extension.set(contextA, { value: 1 });
-    extension.set(contextB, { value: 2 });
-
-    expect(extension.get(contextA)).toEqual({ value: 1 });
-    expect(extension.get(contextB)).toEqual({ value: 2 });
-
-    extension.remove(contextA);
-    expect(extension.get(contextA)).toBeUndefined();
-    expect(extension.get(contextB)).toEqual({ value: 2 });
-  });
-
-  it('isolates extensions with different ids on the same context', () => {
-    const first = createRuntimeContextExtension<string>('test:first');
-    const second = createRuntimeContextExtension<string>('test:second');
-    const context = {};
-
-    first.set(context, 'one');
-    second.set(context, 'two');
-
-    expect(first.get(context)).toBe('one');
-    expect(second.get(context)).toBe('two');
-  });
-
-  it('shares state between extensions created with the same id', () => {
-    const a = createRuntimeContextExtension<string>('test:shared');
-    const b = createRuntimeContextExtension<string>('test:shared');
-    const context = {};
-
-    a.set(context, 'value');
-    expect(b.get(context)).toBe('value');
-  });
-
-  it('returns undefined for contexts without any extensions', () => {
-    const extension = createRuntimeContextExtension<string>('test:empty');
-    expect(extension.get({})).toBeUndefined();
-  });
-
   it('does not leak into string-key enumeration of the context', () => {
     const extension = createRuntimeContextExtension<string>('test:hidden');
     const context = {};

package/tests/headRuntime.test.tsx

@@ -146,41 +146,6 @@ describe('server Helmet collection', () => {
     expect(meta).toContain('content="inner"');
     expect(meta).not.toContain('content="outer"');
     expect(meta).toContain('content="modernjs"');
-    // outer instance tags come first, like react-helmet-async
-    expect(meta.indexOf('keywords')).toBeLessThan(meta.indexOf('description'));
-  });
-
-  it('keeps duplicate metas within a single Helmet instance', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <Helmet>
-        <meta name="author" content="a" />
-        <meta name="author" content="b" />
-      </Helmet>,
-    );
-
-    const meta = getHelmetData(context)!.meta.toString();
-    expect(meta).toContain('content="a"');
-    expect(meta).toContain('content="b"');
-  });
-
-  it('normalizes charSet when deduping and drops meta without primary attribute', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <>
-        <Helmet>
-          <meta charSet="utf-8" />
-          <meta content="orphan" />
-        </Helmet>
-        <Helmet meta={[{ charSet: 'utf-8' } as any]} />
-      </>,
-    );
-
-    const meta = getHelmetData(context)!.meta.toString();
-    expect(meta.match(/charset/g)).toHaveLength(1);
-    expect(meta).not.toContain('orphan');
   });
 
   it('applies the innermost title and the innermost titleTemplate', () => {
@@ -202,71 +167,6 @@ describe('server Helmet collection', () => {
     );
   });
 
-  it('falls back to defaultTitle without applying the template', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <Helmet defaultTitle="Default" titleTemplate="%s | Site" />,
-    );
-
-    expect(getHelmetData(context)!.title.toString()).toBe(
-      '<title data-rh="true">Default</title>',
-    );
-  });
-
-  it('maps <html>/<body> children to html/body attributes', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <Helmet>
-        <html lang="en" />
-        <body className="dark" />
-      </Helmet>,
-    );
-
-    const helmet = getHelmetData(context)!;
-    expect(helmet.htmlAttributes.toString()).toBe('lang="en"');
-    expect(helmet.bodyAttributes.toString()).toBe('class="dark"');
-  });
-
-  it('keeps one canonical link but all distinct stylesheets', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <>
-        <Helmet>
-          <link rel="canonical" href="https://outer.example/" />
-          <link rel="stylesheet" href="/a.css" />
-          <link rel="stylesheet" href="/b.css" />
-        </Helmet>
-        <Helmet>
-          <link rel="canonical" href="https://inner.example/" />
-        </Helmet>
-      </>,
-    );
-
-    const link = getHelmetData(context)!.link.toString();
-    expect(link).toContain('https://inner.example/');
-    expect(link).not.toContain('https://outer.example/');
-    expect(link).toContain('/a.css');
-    expect(link).toContain('/b.css');
-  });
-
-  it('does not serialize script innerHTML as an attribute', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <Helmet>
-        <script type="application/ld+json">{'{"@context":"a"}'}</script>
-      </Helmet>,
-    );
-
-    const script = getHelmetData(context)!.script.toString();
-    expect(script).toBe(
-      '<script data-rh="true" type="application/ld+json">{"@context":"a"}</script>',
-    );
-  });
-
   it('publishes prioritized SEO tags separately with component parity', () => {
     const context = createServerContext();
     renderWithContext(
@@ -292,29 +192,6 @@ describe('server Helmet collection', () => {
     expect(helmet.link.toString()).not.toContain('https://example.com/page');
     expect(helmet.script.toString()).toContain('/ordinary.js');
     expect(helmet.script.toString()).not.toContain('application/ld+json');
-
-    const priority = helmet.priority.toComponent();
-    expect(priority.map(element => element.type)).toEqual([
-      'meta',
-      'link',
-      'script',
-    ]);
-    expect(priority[0].props).toMatchObject({
-      'data-rh': true,
-      content: 'priority description',
-      name: 'description',
-    });
-    expect(priority[2].props).toMatchObject({
-      'data-rh': true,
-      dangerouslySetInnerHTML: { __html: '{"name":"page"}' },
-      type: 'application/ld+json',
-    });
-    expect(helmet.meta.toComponent()[0].props).toMatchObject({
-      content: 'ordinary keywords',
-      name: 'keywords',
-    });
-    expect(helmet.title.toComponent()[0].props.children).toBe('Priority Page');
-    expect(helmet.htmlAttributes.toComponent()).toEqual({ className: 'app' });
   });
 
   it('is idempotent when React replays the tree (streaming SSR retry)', () => {
@@ -349,21 +226,4 @@ describe('server Helmet collection', () => {
     expect(second.htmlAttributes.toString()).toBe(firstSnapshot.htmlAttributes);
     expect(second.meta.toString().match(/description/g)).toHaveLength(1);
   });
-
-  it('keeps the innermost base tag only', () => {
-    const context = createServerContext();
-    renderWithContext(
-      context,
-      <>
-        <Helmet base={{ href: 'https://outer.example/' } as any} />
-        <Helmet>
-          <base href="https://inner.example/" />
-        </Helmet>
-      </>,
-    );
-
-    const base = getHelmetData(context)!.base.toString();
-    expect(base).toContain('https://inner.example/');
-    expect(base).not.toContain('https://outer.example/');
-  });
 });

package/tests/rendererHead.test.ts

@@ -5,7 +5,6 @@ import {
   collectHeadRecord,
   completeHeadRender,
   createHeadChunkProcessor,
-  createWebHeadMarkerStripper,
 } from '../src';
 import { createNodeHeadMarkerStripper, pipeNodeHeadStream } from '../src/node';
 
@@ -45,16 +44,6 @@ describe('renderer head transactions', () => {
     expect(completeHeadRender(context, userTemplate)).toBe(userTemplate);
   });
 
-  it('does not retain ordinary streaming content', () => {
-    const context = {};
-    beginHeadRender(context);
-    const processor = createHeadChunkProcessor(context);
-    const content = 'x'.repeat(128 * 1024);
-
-    expect(processor.push(content)).toBe(content);
-    expect(processor.finish()).toBe('');
-  });
-
   it('strips markers split across chunks without corrupting unicode', () => {
     const sampleContext = {};
     beginHeadRender(sampleContext);
@@ -101,36 +90,6 @@ describe('renderer head transactions', () => {
     expect(published).toEqual(['previous']);
   });
 
-  it('strips split markers from a web byte stream', async () => {
-    const context = {};
-    let published: string[] = [];
-    beginHeadRender(context);
-    const props = collectHeadRecord(
-      context,
-      () => 'web',
-      records => {
-        published = records;
-      },
-    )!;
-    const html = `č${marker(props)}尾`;
-    const encoder = new TextEncoder();
-    const encoded = encoder.encode(html);
-    const source = new ReadableStream<Uint8Array>({
-      start(controller) {
-        controller.enqueue(encoded.slice(0, 1));
-        controller.enqueue(encoded.slice(1, 17));
-        controller.enqueue(encoded.slice(17));
-        controller.close();
-      },
-    });
-
-    const output = await new Response(
-      source.pipeThrough(createWebHeadMarkerStripper(context)),
-    ).text();
-    expect(output).toBe('č尾');
-    expect(published).toEqual(['web']);
-  });
-
   it('strips split markers from a Node stream', async () => {
     const context = {};
     let published: string[] = [];