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

[artembelik]

What

The audit pass over the seven components that were already signal-based. Six of them needed no change — actions-panel, dynamic-translation, overflow-items, resizer, skeleton and content-panel are already compliant on signal inputs, booleanAttribute / numberAttribute, _IdGenerator, protected for template members, readonly on injections, and @docs-private. This PR is the one component where the audit found something.

Five inputs never got a coercion transform

readonly verticalBreakpoint = input(400, { transform: numberAttribute });
readonly minWidth = input<number | undefined>();          // no transform
readonly wide = input(false);                              // no transform
readonly resizable = input(false, { transform: booleanAttribute });
readonly vertical = input<boolean | null>(null);           // no transform

So <kbq-dl wide> and <kbq-dl vertical> did nothing — a valueless attribute passes the empty string, which is falsy — while <kbq-dl resizable> on the line above worked. That inconsistency inside one class is the whole finding.

vertical is tri-state, so booleanAttribute alone was wrong

null means "decide from verticalBreakpoint", and booleanAttribute(null) is false. It uses a transform that passes null through and coerces everything else, so the auto mode survives and <kbq-dl vertical> means true.

The numeric ones

minWidth, dtMinWidth and ddMinWidth report number | undefined — what an unbound description list always held. A static attribute used to reach the layout arithmetic as a string, which coerced in the <= comparison but not in Math.max.

Migration

dl-attribute-coercion runs from ng update @koobiq/components@20. Warn-only: whether markup relied on a valueless wide being ignored is a decision the call site owns.

It reads templates in both .html files and inline .ts templates, since <kbq-dl> is written in either.

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

Testing

• dl.component.spec.ts: 34 → 39 tests. The valueless wide and vertical attributes, null surviving as the vertical default, the coerced numeric attributes, and the unbound optional widths.
• dl-attribute-coercion/index.spec.ts: 6 tests, including one that a bound [wide] is not reported.
• Full packages/components (5166 tests) and packages/schematics (488 tests) suites pass.
• check-api is in sync.

BREAKING CHANGE

🤖 Generated with Claude Code

[github-actions]

Visit the preview URL for this PR (updated for commit 29a0448):

https://koobiq-next--prs-1995-dy1hd4pv.web.app

_{(expires Mon, 07 Sep 2026 08:57:57 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c}