@arsedizioni/ars-utils 22.5.30 → 22.5.33

package/README.md

@@ -27,7 +27,7 @@ di vita proprio vive nel suo entry point. Importare sempre dal path specifico.
 | L0.5 | `ui.shell` | `ShellService` (`info`, `error`, `busy*`, `wait`, `BusyTimer`), `ShellMessageComponent`, `ShellBusyComponent` | zero Material e zero CDK: e' quello che shell, interceptor, guard e app initializer possono permettersi sul percorso di boot |
 | L1 | `ui` | `FlexLayoutModule`, `MediaObserver`, `UIService`, tipi dei dialog | kernel: deve restare magro, e non importa nulla da Material |
 | L1 | `ui.paginator` | `PaginatorIntl` | isola `@angular/material/paginator`, che si porta dietro `mat-select` e `mat-option` |
-| L1.5 | `ui.dialogs` | `DialogService` (estende `ShellService`) + confirm, delete, toast, `open` | tutto cio' che chiede qualcosa all'utente, piu' il toast |
+| L1.5 | `ui.dialogs` | `DialogService` (estende `ShellService`) + confirm, delete, toast, `open`, e le aperture pigre dei dialog pesanti | tutto cio' che chiede qualcosa all'utente, piu' il toast |
 | L2 | `ui.dialogs.auth` | credentials, reset-password, recover-password, prompt-otp, `OtpInputComponent`, `PasswordStrengthComponent` | solo route di login/account |
 | L2 | `ui.dialogs.prompt` | prompt, prompt-date, prompt-time | porta `@angular/material/datepicker` |
 | L2 | `ui.dialogs.select` | select, select-tree, send-to | porta `mat-tree`, `mat-list`, `mat-chips` |
@@ -35,7 +35,6 @@ di vita proprio vive nel suo entry point. Importare sempre dal path specifico.
 | L2 | `ui.controls` | button-selector, button-toggle, chips-selector, tree-picker, calendar-empty-header | |
 | L2 | `ui.filters` | `FilterBarComponent` + modello `Filters` | |
 | L2 | `ui.navigation` | `NavigationBarComponent` + definizioni | shell applicativa |
-| L3 | `ui.application` | `ApplicationDialogService` | facade: carica i dialog pesanti con `import()` |
 
 ### Specifici ARS
 
@@ -69,24 +68,28 @@ ui.dialogs      <-- DialogService extends ShellService: dipende da `ui` E da `ui
  |         |                  |              |          |            |
  |   ui.dialogs.auth   ui.dialogs.prompt  ui.files   ui.help    ui.oauth
  |                                                   ui.notifications (+ support.common)
-ui.application  --- import() --->  ui.dialogs.{auth,prompt,select}, ui.files
- |
 clipper.ui  (+ clipper.common, ui.controls, ui.files, ui.dialogs.select)
+
+`ui.dialogs` --- import() --->  ui.dialogs.{auth,prompt,select}, ui.files
+Gli archi dinamici puntano all'insu' e sono l'unica ragione per cui quei quattro entry point
+restano fuori dal bundle iniziale. Regge finche' NESSUNO di loro importa `DialogService`: per
+`error()` e `busy()` iniettano `ShellService`.
 ```
 
 `evolution.common` e `support.common` dipendono solo da `core`.
 
 ### Regole da rispettare
 
-1. Il grafo tra entry point deve restare un DAG. Un entry point di livello L2 non puo'
-   importare `ui.application`, ne' un entry point puo' importare se stesso per package path
-   (ng-packagr lo segnala come `circular dependency on itself`).
+1. Il grafo **statico** tra entry point deve restare un DAG: `ui.dialogs` raggiunge i dialog
+   pesanti solo con `import()`, e quei quattro entry point non devono importare `DialogService`
+   in nessun modo — ng-packagr rifiuta la build con `circular dependency` al primo che lo fa.
+   Per messaggi e attesa iniettano `ShellService`, che sta sotto di loro nel grafo.
 2. I tipi condivisi fra il facade e i dialog pesanti stanno in `ui` (`dialog.definitions.ts`).
    Se un tipo servisse a entrambi e vivesse nel dialog, l'`import()` del facade tornerebbe statico.
 3. Il facade usa `import type` per i tipi dei componenti e `import()` per i valori: e' quello
    che tiene i dialog fuori dal bundle iniziale.
 4. Dopo una modifica, verificare che gli `import()` sopravvivano alla build:
-   `grep -o "import('@arsedizioni[^']*')" dist/ars-utils/fesm2022/*ui.application.mjs`
+   `grep -o "import('@arsedizioni[^']*')" dist/ars-utils/fesm2022/*ui.dialogs.mjs` deve stampare i nove.
 5. `ui.shell` non importa nulla da `@angular/material` ne' da `@angular/cdk`, e nemmeno da `ui`.
    E' l'unica ragione per cui esiste: se ci finisce dentro un solo simbolo di Material, la shell
    torna a pagare l'intero stack sul percorso di boot. Da verificare dopo ogni build:
@@ -125,14 +128,14 @@ significherebbe far dipendere tre entry point headless da uno di UI, cioe' rompe
 La conseguenza pratica per chi scrive l'applicazione:
 
 - shell, interceptor, guard, `provideAppInitializer` -> `ShellService` di `ui.shell`
-- componenti di rotta -> `DialogService` di `ui.dialogs`, e `ApplicationDialogService` di `ui.application` per i dialog pesanti
+- componenti di rotta -> `DialogService` di `ui.dialogs`, che apre anche i dialog pesanti
 
 I tre servizi sono una catena, non tre servizi diversi:
 
 ```
 ShellService                info, error, busy*, wait, busyTimer     ui.shell
-  └ DialogService           + open, confirm, delete, toast          ui.dialogs
-      └ ApplicationDialogService  + select, prompt, sendTo, ...     ui.application
+  └ DialogService           + open, confirm, delete, toast,         ui.dialogs
+                              select, prompt, sendTo, ... (pigri)
 ```
 
 Quindi chi inietta `DialogService` ha tutto quello che aveva prima: `info`, `error` e `busy` non
@@ -147,7 +150,7 @@ iniettato e nient'altro.
 
 - Nessun barrel radice: `@arsedizioni/ars-utils` esporta solo `ARS_UTILS_VERSION`.
 - `ui` non contiene piu' i dialog: `DialogService` sta in `ui.dialogs`.
-- `ui.application` resta il solo facade e non riesporta piu' componenti: i controlli sono in
+- `ui.application` non esiste piu': i nove metodi che apriva stanno in `DialogService`. I controlli sono in
   `ui.controls`, i file in `ui.files`, i filtri in `ui.filters`, la navigazione in `ui.navigation`.
 - `SystemUtils.markdownToHtml` non esiste piu': usare `MarkdownUtils.toHtml` da `core.markdown`.
 - `SelectableModel` espone `first` e `selectedAny` al posto di `selectedFirst` e `selectedAll`.

package/fesm2022/arsedizioni-ars-utils-clipper.ui.mjs

@@ -50,10 +50,9 @@ import { switchMap, EMPTY, finalize, Subject } from 'rxjs';
 import { debounceTime, finalize as finalize$1 } from 'rxjs/operators';
 import { startOfToday, startOfDay, subDays, endOfDay, startOfMonth, endOfMonth } from 'date-fns';
 import { Clipboard } from '@angular/cdk/clipboard';
-import { ApplicationDialogService } from '@arsedizioni/ars-utils/ui.application';
+import { DialogService } from '@arsedizioni/ars-utils/ui.dialogs';
 import { SendToDialogComponent } from '@arsedizioni/ars-utils/ui.dialogs.select';
 import saveAs from 'file-saver';
-import { DialogService } from '@arsedizioni/ars-utils/ui.dialogs';
 import { MAT_DIALOG_DATA, MatDialogRef, MatDialogClose, MatDialogContent, MatDialogTitle, MatDialogActions } from '@angular/material/dialog';
 import { Router } from '@angular/router';
 import { ShellService } from '@arsedizioni/ars-utils/ui.shell';
@@ -510,7 +509,7 @@ class ClipperDocumentManager {
         this.clipboard = inject(Clipboard);
         this.clipperService = inject(ClipperService);
         this.broadcastService = inject(BroadcastService);
-        this.dialogService = inject(ApplicationDialogService);
+        this.dialogService = inject(DialogService);
         this.canHandleTooManyResults = signal(false, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "canHandleTooManyResults" }] : /* istanbul ignore next */ []));
         this.canPopulateContacts = signal(false, /* @ts-ignore */
@@ -997,7 +996,7 @@ class ClipperDocumentViewerService {
         }
         // NB: `import()` here does NOT make the viewer lazy. ng-packagr flattens every file of an
         // entry point into one FESM module, so a dynamic import of a sibling is inlined at build time
-        // (the cross-entry-point ones in `ui.application` are the ones that survive). It stays as an
+        // (the cross-entry-point ones, like those in `DialogService`, are the ones that survive). It stays as an
         // import() only because it keeps the module graph honest; to load the viewer on demand for
         // real it would have to live in an entry point of its own.
         Promise.resolve().then(function () { return document_component; }).then(({ ClipperDocumentComponent }) => {
@@ -1920,7 +1919,7 @@ class ClipperBrowserComponent extends ClipperSearchResultManager {
     constructor() {
         super();
         /** Error text of a field: the message declared in the schema, or the shared fallback. */
-        this.getFieldErrorMessage = SignalsUtils.getFieldErrorMessage;
+        this.getErrorMessage = SignalsUtils.getFieldErrorMessage;
         this.filterByNumber = viewChild('filterByNumber', /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "filterByNumber" }] : /* istanbul ignore next */ []));
         this.filterByText = viewChild('filterByText', /* @ts-ignore */