Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[bug] ssr-shims.js uses deprecated globalThis.litSsrCallConnectedCallback

Abierto
#3,158 0 comentarios 0 reacciones 1 asignado Ver en GitHub

@zeroedin ya está trabajando en esto.

Desde el 6/8/2026.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

bug

Summary

@patternfly/pfe-core/ssr-shims.js sets globalThis.litSsrCallConnectedCallback = true to opt elements into connectedCallback during Lit SSR. This global was deprecated in @lit-labs/ssr@4.x in favor of LitElementRenderer.renderOptions.

The result is a repeated console warning during every SSR render pass:

litSsrCallConnectedCallback is deprecated. Please use LitElementRenderer.renderOptions instead.

Current behavior

https://github.com/patternfly/patternfly-elements/blob/main/core/pfe-core/ssr-shims.ts

globalThis.litSsrCallConnectedCallback = true;

This triggers the deprecation path inside LitElementRenderer.connectedCallback() on every element instance rendered, flooding build logs with warnings.

Expected behavior

Replace the deprecated global with the new per-element configuration API:

import { LitElementRenderer } from '@lit-labs/ssr/lib/lit-element-renderer.js';

// Replicate the previous blanket opt-in for PFE/RHDS elements
LitElementRenderer.renderOptions.push(
  (element) =>
    element.localName.startsWith('pf-') || element.localName.startsWith('rh-')
      ? { connectedCallback: true }
      : undefined
);

Or, to preserve the exact previous behavior (all LitElements get connectedCallback called):

LitElementRenderer.renderOptions.push(() => ({ connectedCallback: true }));

Impact

  • The warning is cosmetic for now — @lit-labs/ssr still honors the global alongside the warning.
  • However, a future major version of @lit-labs/ssr will likely remove support for the global entirely, making this a future breaking change.

Environment

  • @patternfly/pfe-core: 5.0.8
  • @lit-labs/ssr: 4.1.0
Lenguaje dominante
TypeScript
Estrellas
394
Forks
107
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de patternfly/patternfly-elements

Todos los issues de patternfly/patternfly-elements

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.