perf(*): remove unnecessary optional attributes

packages/abc/avatar-list/avatar-list.component.ts

@@ -38,7 +38,7 @@ import { AvatarListItemComponent } from './avatar-list-item.component';
 })
 export class AvatarListComponent implements AfterViewInit, OnChanges {
   private readonly cdr = inject(ChangeDetectorRef);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly destroy$ = inject(DestroyRef);
 
   private inited = false;
@@ -79,8 +79,8 @@ export class AvatarListComponent implements AfterViewInit, OnChanges {
   }
 
   ngAfterViewInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/error-collect/error-collect.component.ts

@@ -42,7 +42,7 @@ export class ErrorCollectComponent implements OnInit {
   private readonly el: HTMLElement = inject(ElementRef).nativeElement;
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly doc = inject(DOCUMENT);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly platform = inject(Platform);
   private readonly destroy$ = inject(DestroyRef);
 
@@ -83,8 +83,8 @@ export class ErrorCollectComponent implements OnInit {
   }
 
   private install(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/exception/exception.component.ts

@@ -43,7 +43,7 @@ export class ExceptionComponent implements OnInit {
 
   private readonly i18n = inject(DelonLocaleService);
   private readonly dom = inject(DomSanitizer);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly destroy$ = inject(DestroyRef);
 
@@ -116,8 +116,8 @@ export class ExceptionComponent implements OnInit {
   }
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/global-footer/global-footer.component.ts

@@ -39,7 +39,7 @@ export class GlobalFooterComponent implements OnInit {
   private readonly router = inject(Router);
   private readonly win = inject(WINDOW);
   private readonly dom = inject(DomSanitizer);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly destroy$ = inject(DestroyRef);
 
@@ -74,8 +74,8 @@ export class GlobalFooterComponent implements OnInit {
   }
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/loading/loading.service.ts

@@ -13,7 +13,7 @@ import { LoadingShowOptions } from './loading.types';
 export class LoadingService implements OnDestroy {
   private readonly overlay = inject(Overlay);
   private readonly configSrv = inject(AlainConfigService);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
 
   private _overlayRef?: OverlayRef;
   private compRef: ComponentRef<LoadingDefaultComponent> | null = null;
@@ -55,7 +55,7 @@ export class LoadingService implements OnDestroy {
       backdropClass: 'loading-backdrop'
     });
     this.compRef = this._overlayRef.attach(new ComponentPortal(LoadingDefaultComponent));
-    const dir = this.configSrv.get('loading')!.direction || this.directionality?.value;
+    const dir = this.configSrv.get('loading')!.direction || this.directionality.value;
     if (this.instance != null) {
       this.instance!!.options = this.opt;
       this.instance!!.dir = dir;

packages/abc/onboarding/onboarding.service.ts

@@ -28,7 +28,7 @@ export class OnboardingService implements OnDestroy {
   private readonly doc = inject(DOCUMENT);
   private readonly configSrv = inject(AlainConfigService);
   private readonly keyStoreSrv = inject(ONBOARDING_STORE_TOKEN);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
 
   private compRef!: ComponentRef<OnboardingComponent>;
   private op$!: Subscription;
@@ -116,7 +116,7 @@ export class OnboardingService implements OnDestroy {
       ...this.i18n.getData('onboarding'),
       ...items[this.active]
     } as OnboardingItem;
-    const dir = this.configSrv.get('onboarding')!.direction || this.directionality?.value;
+    const dir = this.configSrv.get('onboarding')!.direction || this.directionality.value;
     Object.assign(this.compRef.instance, { item, config: this.config, active: this.active, max: items.length, dir });
     const pipes = [
       switchMap(() => (item.url ? this.router.navigateByUrl(item.url) : of(true))),

packages/abc/page-header/page-header.component.ts

@@ -63,10 +63,10 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
   private readonly router = inject(Router);
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly menuSrv = inject(MenuService);
-  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN, { optional: true });
-  private readonly titleSrv = inject(TitleService, { optional: true });
+  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN);
+  private readonly titleSrv = inject(TitleService);
   private readonly reuseSrv = inject(ReuseTabService, { optional: true });
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly destroy$ = inject(DestroyRef);
 
   @ViewChild('conTpl', { static: false }) private conTpl!: ElementRef;
@@ -140,7 +140,7 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
 
     const obsList: Array<Observable<NzSafeAny>> = [this.router.events.pipe(filter(ev => ev instanceof NavigationEnd))];
     if (this.menuSrv != null) obsList.push(this.menuSrv.change);
-    if (this.i18nSrv != null) obsList.push(this.i18nSrv.change);
+    obsList.push(this.i18nSrv.change);
     merge(...obsList)
       .pipe(
         takeUntilDestroyed(),
@@ -163,13 +163,13 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
     this.menus.forEach(item => {
       if (typeof item.hideInBreadcrumb !== 'undefined' && item.hideInBreadcrumb) return;
       let title = item.text;
-      if (item.i18n && this.i18nSrv) title = this.i18nSrv.fanyi(item.i18n);
+      if (item.i18n) title = this.i18nSrv.fanyi(item.i18n);
       paths.push({ title, link: (item.link && [item.link]) as string[] });
     });
     // add home
     if (this.home) {
       paths.splice(0, 0, {
-        title: (this.homeI18n && this.i18nSrv && this.i18nSrv.fanyi(this.homeI18n)) || this.home,
+        title: (this.homeI18n && this.i18nSrv.fanyi(this.homeI18n)) || this.home,
         link: [this.homeLink!]
       });
     }
@@ -180,16 +180,14 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
     if (this._title == null && this._titleTpl == null && this.autoTitle && this.menus.length > 0) {
       const item = this.menus[this.menus.length - 1];
       let title = item.text;
-      if (item.i18n && this.i18nSrv) {
+      if (item.i18n) {
         title = this.i18nSrv.fanyi(item.i18n);
       }
       this._titleVal = title!;
     }
 
     if (this._titleVal && this.syncTitle) {
-      if (this.titleSrv) {
-        this.titleSrv.setTitle(this._titleVal);
-      }
+      this.titleSrv.setTitle(this._titleVal);
       if (!this.inited && this.reuseSrv) {
         this.reuseSrv.title = this._titleVal;
       }
@@ -207,8 +205,8 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
   }
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/page-header/page-header.spec.ts

@@ -352,10 +352,6 @@ describe('abc: page-header', () => {
       beforeEach(() => {
         genModule({
           providers: [
-            {
-              provide: TitleService,
-              useFactory: () => null
-            },
             {
               provide: ReuseTabService,
               useFactory: () => null

packages/abc/result/result.component.ts

@@ -31,7 +31,7 @@ import { NzIconDirective } from 'ng-zorro-antd/icon';
 })
 export class ResultComponent implements OnInit {
   private readonly cdr = inject(ChangeDetectorRef);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly destroy$ = inject(DestroyRef);
 
   _type = '';
@@ -58,8 +58,8 @@ export class ResultComponent implements OnInit {
   dir?: Direction = 'ltr';
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/abc/reuse-tab/reuse-tab.component.ts

@@ -79,10 +79,10 @@ export class ReuseTabComponent implements OnInit, OnChanges {
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly router = inject(Router);
   private readonly route = inject(ActivatedRoute);
-  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN, { optional: true });
+  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN);
   private readonly doc = inject(DOCUMENT);
   private readonly platform = inject(Platform);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly stateKey = inject(REUSE_TAB_STORAGE_KEY);
   private readonly stateSrv = inject(REUSE_TAB_STORAGE_STATE);
 
@@ -124,7 +124,7 @@ export class ReuseTabComponent implements OnInit, OnChanges {
   // #endregion
 
   private genTit(title: ReuseTitle): string {
-    return title.i18n && this.i18nSrv ? this.i18nSrv.fanyi(title.i18n) : title.text!;
+    return title.i18n ? this.i18nSrv.fanyi(title.i18n) : title.text!;
   }
 
   private get curUrl(): string {
@@ -302,8 +302,8 @@ export class ReuseTabComponent implements OnInit, OnChanges {
   // #endregion
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });
@@ -327,7 +327,7 @@ export class ReuseTabComponent implements OnInit, OnChanges {
       this.genList(res);
     });
 
-    this.i18nSrv?.change
+    this.i18nSrv.change
       .pipe(
         filter(() => this.srv.inited),
         takeUntilDestroyed(this.destroy$),

packages/abc/st/st-export.ts

@@ -9,7 +9,7 @@ import { _STColumn } from './st.types';
 
 @Injectable()
 export class STExport {
-  private readonly xlsxSrv = inject(XlsxService, { optional: true });
+  private readonly xlsxSrv = inject(XlsxService);
 
   private _stGet(item: NzSafeAny, col: STColumn, index: number, colIndex: number): NzSafeAny {
     const ret: { [key: string]: NzSafeAny } = { t: 's', v: '' };
@@ -81,13 +81,6 @@ export class STExport {
   }
 
   async export(opt: STExportOptions): Promise<XlsxExportResult> {
-    if (this.xlsxSrv == null) {
-      if (typeof ngDevMode === 'undefined' || ngDevMode) {
-        console.warn(`XlsxService service not found`);
-      }
-      return Promise.reject();
-    }
-
     const sheets = this.genSheet(opt);
     return this.xlsxSrv.export({
       sheets,

packages/abc/st/st.component.ts

@@ -94,7 +94,7 @@ import type { _STColumn, _STDataValue, _STHeader, _STTdNotify, _STTdNotifyType }
   encapsulation: ViewEncapsulation.None
 })
 export class STComponent implements AfterViewInit, OnChanges {
-  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN, { optional: true });
+  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN);
   private readonly el: HTMLElement = inject(ElementRef).nativeElement;
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly doc = inject(DOCUMENT);
@@ -254,7 +254,7 @@ export class STComponent implements AfterViewInit, OnChanges {
       }
     });
 
-    this.i18nSrv?.change
+    this.i18nSrv.change
       .pipe(
         takeUntilDestroyed(),
         filter(() => this._columns.length > 0)

packages/abc/tag-select/tag-select.component.ts

@@ -36,7 +36,7 @@ import { NzIconDirective } from 'ng-zorro-antd/icon';
 })
 export class TagSelectComponent implements OnInit {
   private readonly i18n = inject(DelonLocaleService);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly destroy$ = inject(DestroyRef);
   locale: LocaleData = {};
@@ -48,8 +48,8 @@ export class TagSelectComponent implements OnInit {
   @Output() readonly change = new EventEmitter<boolean>();
 
   ngOnInit(): void {
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });

packages/form/src/sf.component.ts

@@ -19,7 +19,7 @@ import {
 } from '@angular/core';
 import { takeUntilDestroyed } from '@angular/core/rxjs-interop';
 import { DomSanitizer } from '@angular/platform-browser';
-import { merge, Observable, filter } from 'rxjs';
+import { merge, filter } from 'rxjs';
 
 import { ACLService } from '@delon/acl';
 import { ALAIN_I18N_TOKEN, DelonLocaleService, LocaleData } from '@delon/theme';
@@ -82,8 +82,8 @@ export class SFComponent implements OnInit, OnChanges, OnDestroy {
   private readonly dom = inject(DomSanitizer);
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly localeSrv = inject(DelonLocaleService);
-  private readonly aclSrv = inject(ACLService, { optional: true });
-  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN, { optional: true });
+  private readonly aclSrv = inject(ACLService);
+  private readonly i18nSrv = inject(ALAIN_I18N_TOKEN);
   private readonly platform = inject(Platform);
 
   private _renders = new Map<string, TemplateRef<void>>();
@@ -317,22 +317,16 @@ export class SFComponent implements OnInit, OnChanges, OnDestroy {
         this.cdr.markForCheck();
       }
     });
-    const refSchemas: Array<Observable<NzSafeAny> | null> = [
-      this.aclSrv ? this.aclSrv.change : null,
-      this.i18nSrv ? this.i18nSrv.change : null
-    ].filter(o => o != null);
-    if (refSchemas.length > 0) {
-      merge(...(refSchemas as Array<Observable<NzSafeAny>>))
-        .pipe(
-          filter(() => this._inited),
-          takeUntilDestroyed()
-        )
-        .subscribe(() => this.refreshSchema());
-    }
+    merge(this.aclSrv.change, this.i18nSrv.change)
+      .pipe(
+        filter(() => this._inited),
+        takeUntilDestroyed()
+      )
+      .subscribe(() => this.refreshSchema());
   }
 
   protected fanyi(key: string): string {
-    return (this.i18nSrv ? this.i18nSrv.fanyi(key) : '') || key;
+    return this.i18nSrv.fanyi(key) || key;
   }
 
   private inheritUI(ui: SFUISchemaItemRun): void {

packages/theme/layout-default/layout-nav.component.ts

@@ -56,7 +56,7 @@ export class LayoutDefaultNavComponent implements OnInit, OnDestroy {
   private readonly cdr = inject(ChangeDetectorRef);
   private readonly ngZone = inject(NgZone);
   private readonly sanitizer = inject(DomSanitizer);
-  private readonly directionality = inject(Directionality, { optional: true });
+  private readonly directionality = inject(Directionality);
 
   private bodyEl!: HTMLBodyElement;
   private destroy$ = inject(DestroyRef);
@@ -259,8 +259,8 @@ export class LayoutDefaultNavComponent implements OnInit, OnDestroy {
       .subscribe(() => this.clearFloating());
     this.underPad();
 
-    this.dir = this.directionality?.value;
-    this.directionality?.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
+    this.dir = this.directionality.value;
+    this.directionality.change.pipe(takeUntilDestroyed(this.destroy$)).subscribe(direction => {
       this.dir = direction;
       this.cdr.detectChanges();
     });