Skip to content

web: Lit Development Mode, performance fixes.#19825

Merged
GirlBossRush merged 1 commit intomainfrom
lit-refinements
Jan 30, 2026
Merged

web: Lit Development Mode, performance fixes.#19825
GirlBossRush merged 1 commit intomainfrom
lit-refinements

Conversation

@GirlBossRush
Copy link
Contributor

Details

This PR enables Lit's development mode, adding additional console logs to track down non-idiomatic Lit.

Additionally, a small number of fixes for some common warnings are included such as:

  • The router outlet component having too frequent of calls to requestUpdate.
  • The activeTheme property triggering requestUpdate for elements without shadow roots.

@GirlBossRush GirlBossRush requested a review from a team as a code owner January 29, 2026 02:06
@GirlBossRush GirlBossRush self-assigned this Jan 29, 2026
@GirlBossRush GirlBossRush added area:frontend Features or issues related to the browser, TypeScript, Node.js, etc area:docs Features or issues related to Docusaurus labels Jan 29, 2026
@GirlBossRush GirlBossRush added this to the Release 2025.12.2 milestone Jan 29, 2026
@netlify
Copy link

netlify bot commented Jan 29, 2026

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit fe7eb87
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/697ac619d70dff000824e89b
😎 Deploy Preview https://deploy-preview-19825--authentik-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Jan 29, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 602f997
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/697ac09ae441290008038041
😎 Deploy Preview https://deploy-preview-19825--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment on lines 48 to +57
@@ -54,7 +54,7 @@
* a dark sidebar on light themes. Only minor changes are needed for the dark mode.
* @see PatternFly's nav.css for reference:
*/
:host([theme="light"]) .pf-c-nav {
:host(:not([theme="dark"])) .pf-c-nav {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAICT this was the last remaining check for a light theme. The path to removing all the theme attribute style overrides will be awhile, but this marks one fewer dependency

Comment on lines +77 to +79
public get activeWhen(): RegExp[] {
return this.activeMatchers;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes Lit warning about missing getter.

Comment on lines +81 to +88
public override connectedCallback(): void {
super.connectedCallback();
this.synchronize();
}

public override disconnectedCallback(): void {
super.disconnectedCallback();
cancelAnimationFrame(this.#scrollAnimationFrame);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes Lit warning about chained updates

@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.23%. Comparing base (5834f43) to head (fe7eb87).
⚠️ Report is 22 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19825      +/-   ##
==========================================
- Coverage   93.25%   93.23%   -0.03%     
==========================================
  Files         965      965              
  Lines       53184    53184              
==========================================
- Hits        49598    49585      -13     
- Misses       3586     3599      +13     
Flag Coverage Δ
conformance 38.15% <ø> (+<0.01%) ⬆️
e2e 44.16% <ø> (-0.01%) ⬇️
integration 22.97% <ø> (+<0.01%) ⬆️
unit 91.39% <ø> (-0.02%) ⬇️
unit-migrate 91.39% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment on lines +159 to +160
"lit-element": "^4.2.2",
"lit-html": "^3.3.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix to hoist Lit into a single version

https://lit.dev/docs/tools/development/#multiple-lit-versions

@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2026

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-d084446c58484e127048c736ce20e2d018b62bb8
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-d084446c58484e127048c736ce20e2d018b62bb8

Afterwards, run the upgrade commands from the latest release notes.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in authentik Core Jan 29, 2026
@GirlBossRush GirlBossRush merged commit e56ab5a into main Jan 30, 2026
102 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in authentik Core Jan 30, 2026
@GirlBossRush GirlBossRush deleted the lit-refinements branch January 30, 2026 00:56
kensternberg-authentik added a commit that referenced this pull request Jan 31, 2026
* main: (52 commits)
  website: QL Search keyboard interactions docs, examples. (#16259)
  website/integrations: immich: add signing algorithm (#19187)
  website/docs: endpoint devices: add version command (#19767)
  common: introduce common (#19852)
  web: bump @sentry/browser from 10.37.0 to 10.38.0 in /web in the sentry group across 1 directory (#19871)
  core: bump debugpy from 1.8.19 to 1.8.20 (#19872)
  ci: bump actions/cache from 5.0.2 to 5.0.3 (#19873)
  web: bump chromedriver from 144.0.1 to 145.0.0 in /web (#19874)
  web: Captcha Refinements, Part 2  (#19757)
  root: assign cherry-pick PRs to original author (#19858)
  web: Lit Development Mode, performance fixes. (#19825)
  web: Fix development theme overrides (#19826)
  website/docs: add tip for recovering from accidental main branch work (#19865)
  web: bump API Client version (#19857)
  rbac: clean up roles and permissions (#19588)
  web: bump API Client version (#19851)
  website/docs: add more info to entra id scim doc (#19849)
  sources/oauth: Fix an issue where wechat may crash duing login. (#18973)
  providers/scim: fix email validation mismatch (#19848)
  providers/scim: modify user- and group syncing behavior (#13947)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Features or issues related to Docusaurus area:frontend Features or issues related to the browser, TypeScript, Node.js, etc

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants