fix(loader-overlay)!: errors following a full review of the component (#DS-5482)

[artembelik]

What

A full review of loader-overlay, in the same shape as the 20.3.0 component reviews.

text and caption lied about being required

@Input() text: string; // no initializer
@Input() caption: string;

The automated signal migration skipped both — it saw them read inside @if blocks and would not risk the narrowing. They are input<string>() now and report string | undefined, which is what they always held. Same pathology as KbqSplitButton.disabled in the first review wave: the call sites that were already wrong now fail to compile.

transparent was backwards as a valueless attribute

<kbq-loader-overlay transparent> passed the empty string, which is falsy, so the attribute rendered the filled background — the opposite of how it reads. With booleanAttribute it means true, and [transparent]="'false'" means false.

Closed internals

Everything the template uses to choose between a projected slot and an input is protected or private now: isEmpty, isExternalIndicator, isExternalText, isExternalCaption, spinnerSize, and the three content queries (which are signal queries as well). What the overlay renders is the contract, not how it decides.

The three getters became computed, so isEmpty now tracks a [text] that arrives later instead of relying on whoever happened to run change detection.

Migration

loader-overlay-signals runs from ng update @koobiq/components@20. It rewrites text and caption reads to calls — on receivers typed KbqLoaderOverlay and through template reference variables on <kbq-loader-overlay>, in external and inline templates — and reports the rest. size, transparent and card were already signals in 20.2.0 and are not touched.

Documented in docs/guides/migration.{en,ru}.md, section 18.

Testing

• loader-overlay.component.spec.ts: 5 → 8 tests. The new ones pin the valueless transparent attribute, isEmpty reacting to a late [text], and the unbound inputs reporting undefined. The existing class snapshot is unchanged.
• loader-overlay-signals/index.spec.ts: 13 tests — auto-fix, idempotence, receiver scoping, template refs, warnings, and the --fix=false path.
• Full packages/components (4996 tests) and packages/schematics (447 tests) suites pass.
• check-api is in sync.

No e2e screenshots were regenerated: every e2e case binds [transparent]="false" explicitly, so nothing it renders changed.

BREAKING CHANGE

🤖 Generated with Claude Code

[github-actions]

Visit the preview URL for this PR (updated for commit 47fd675):

https://koobiq-next--prs-1982-bsqilact.web.app

_{(expires Sat, 05 Sep 2026 15:05:29 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c}