Skip to content

feat(common): add the css_rebuild manifest opt-in#1664

Merged
cmorman89 merged 3 commits into
release/1.4from
fix-plugin-css-mode
Jul 9, 2026
Merged

feat(common): add the css_rebuild manifest opt-in#1664
cmorman89 merged 3 commits into
release/1.4from
fix-plugin-css-mode

Conversation

@cmorman89

@cmorman89 cmorman89 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds one additive, default-off field to @vertesia/common:

  • AppUIConfig.css_rebuild (default false) — a manifest opt-in. When true, the host modifies the app's css at load time to attempt to fix broken or missing styles. Only takes effect in css isolation mode.

Type-only; no runtime behavior lives in this package. The host that consumes it ships in the Vertesia web app off this same maintenance line. The build-tool fix that makes future builds correct at the source ships separately on the main line (#1699).

Verification

  • @vertesia/common builds; lint clean. The field is optional and default-off — no existing consumer changes.

Test Plan

  • Build @vertesia/common
  • Confirm the field is optional and default-off (no behavior change for existing manifests)

🤖 Generated with Claude Code

https://claude.ai/code/session_01JpAEfftYG74jVAFhZ87EcD

@cmorman89
cmorman89 requested a review from a team as a code owner July 6, 2026 19:41
@cmorman89 cmorman89 self-assigned this Jul 7, 2026
@cmorman89 cmorman89 added the backport/main Trigger: forward-port this merged PR to main label Jul 7, 2026
AppCssMeta is the capability metadata a plugin module exports next to its
inline css (the isolation modes the build's payloads support). AppUIConfig
gains legacy_css_compat: when set, a host may rebuild a legacy build's css
from its stylesheet at load time. Both are additive and default-off.

The host that consumes these types ships on this maintenance line; the
plugin-builder change that emits cssMeta ships on the main line.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JpAEfftYG74jVAFhZ87EcD
@cmorman89
cmorman89 force-pushed the fix-plugin-css-mode branch from 014afaf to e460eb7 Compare July 9, 2026 16:13
cmorman89 and others added 2 commits July 9, 2026 12:31
Scope the manifest contract to shipped behavior: the host decides css
handling from the manifest alone, so the module-side capability marker
is not consumed anywhere. Reword the legacy_css_compat doc for the
operator editing a manifest rather than the host internals.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JpAEfftYG74jVAFhZ87EcD
Name the flag by what it does — the host rebuilds the app's css from
its stylesheet at load time — instead of by a build-generation story.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JpAEfftYG74jVAFhZ87EcD
@cmorman89 cmorman89 changed the title fix(plugin-builder): keep inlined plugin CSS intact for css isolation feat(common): add the css_rebuild manifest opt-in Jul 9, 2026
@cmorman89
cmorman89 merged commit 666deda into release/1.4 Jul 9, 2026
10 checks passed
@cmorman89
cmorman89 deleted the fix-plugin-css-mode branch July 9, 2026 21:35
@vertesia-release-bot

Copy link
Copy Markdown
Contributor

✅ Opened a backport to main: #1703

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/main Trigger: forward-port this merged PR to main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants