refactor(devtools): drop explicit OnPush change detection strategy

Drop explicit `ChangeDetectionStrategy.OnPush` from the apps since it's now the default.

Author: hawkgs

devtools/projects/ng-devtools/src/lib/devtools-tabs/devtools-tabs.component.ts

@@ -6,15 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {
-  ChangeDetectionStrategy,
-  Component,
-  computed,
-  inject,
-  input,
-  output,
-  signal,
-} from '@angular/core';
+import {Component, computed, inject, input, output, signal} from '@angular/core';
 import {MatIcon} from '@angular/material/icon';
 import {MatMenu, MatMenuItem, MatMenuTrigger} from '@angular/material/menu';
 import {MatSlideToggle} from '@angular/material/slide-toggle';
@@ -67,7 +59,6 @@ type Tab = 'Components' | 'Profiler' | 'Router Tree' | 'Injector Tree' | 'Transf
     ButtonComponent,
   ],
   providers: [TabUpdate],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class DevToolsTabsComponent {
   public readonly frameManager = inject(FrameManager);

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/directive-explorer.component.ts

@@ -15,7 +15,6 @@ import {
   output,
   signal,
   viewChild,
-  ChangeDetectionStrategy,
   computed,
   DestroyRef,
 } from '@angular/core';
@@ -37,7 +36,7 @@ import {FrameManager} from '../../application-services/frame_manager';
 import {BreadcrumbsComponent} from './directive-forest/breadcrumbs/breadcrumbs.component';
 import {FlatNode} from './directive-forest/component-data-source';
 import {DirectiveForestComponent} from './directive-forest/directive-forest.component';
-import {findNodeByPosition, IndexedNode, indexForest} from './directive-forest/index-forest';
+import {IndexedNode} from './directive-forest/index-forest';
 import {constructPathOfKeysToPropertyValue} from './property-resolver/directive-property-resolver';
 import {ElementPropertyResolver} from './property-resolver/element-property-resolver';
 import {FlatNode as PropertyFlatNode} from '../../shared/object-tree-explorer/object-tree-types';
@@ -99,7 +98,6 @@ const sameDirectives = (a: IndexedNode, b: IndexedNode) => {
     SignalGraphPaneComponent,
     ResponsiveSplitDirective,
   ],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class DirectiveExplorerComponent {
   readonly showCommentNodes = input(false);

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/directive-forest/breadcrumbs/breadcrumbs.component.ts

@@ -7,7 +7,6 @@
  */
 
 import {
-  ChangeDetectionStrategy,
   Component,
   computed,
   effect,
@@ -26,7 +25,6 @@ import {MatIcon} from '@angular/material/icon';
   templateUrl: './breadcrumbs.component.html',
   styleUrls: ['./breadcrumbs.component.scss'],
   imports: [MatIcon],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class BreadcrumbsComponent {
   readonly parents = input.required<FlatNode[]>();

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/directive-forest/directive-forest.component.ts

@@ -14,7 +14,6 @@ import {
 import {FlatTreeControl} from '@angular/cdk/tree';
 import {
   afterRenderEffect,
-  ChangeDetectionStrategy,
   Component,
   computed,
   DestroyRef,
@@ -45,7 +44,6 @@ const RESIZE_OBSERVER_DEBOUNCE = 250; // ms
   selector: 'ng-directive-forest',
   templateUrl: './directive-forest.component.html',
   styleUrls: ['./directive-forest.component.scss'],
-  changeDetection: ChangeDetectionStrategy.OnPush,
   imports: [
     FilterComponent,
     CdkVirtualScrollViewport,

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/directive-forest/filter/filter.component.ts

@@ -6,14 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {
-  ChangeDetectionStrategy,
-  Component,
-  ElementRef,
-  input,
-  output,
-  viewChild,
-} from '@angular/core';
+import {Component, ElementRef, input, output, viewChild} from '@angular/core';
 import {MatIcon} from '@angular/material/icon';
 import {MatTooltip} from '@angular/material/tooltip';
 
@@ -52,7 +45,6 @@ const genericSearchGenerator: FilterFnGenerator = (filter: string) => {
   templateUrl: './filter.component.html',
   styleUrls: ['./filter.component.scss'],
   imports: [MatIcon, MatTooltip],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class FilterComponent {
   protected readonly input = viewChild.required<ElementRef>('input');

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/directive-forest/tree-node/tree-node.component.ts

@@ -8,7 +8,6 @@
 
 import {
   afterRenderEffect,
-  ChangeDetectionStrategy,
   Component,
   computed,
   ElementRef,
@@ -39,7 +38,6 @@ export type NodeTextMatch = {
   selector: 'ng-tree-node',
   templateUrl: './tree-node.component.html',
   styleUrls: ['./tree-node.component.scss'],
-  changeDetection: ChangeDetectionStrategy.OnPush,
   imports: [MatIcon, MatTooltip],
   host: {
     '[style.padding-left]': 'paddingLeft()',

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/property-pane/defer-view/defer-view.component.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {ChangeDetectionStrategy, Component, computed, input} from '@angular/core';
+import {Component, computed, input} from '@angular/core';
 import {MatToolbar} from '@angular/material/toolbar';
 import {DeferBlock} from '../../../../../../../protocol';
 
@@ -15,7 +15,6 @@ import {DeferBlock} from '../../../../../../../protocol';
   selector: 'ng-defer-view',
   styleUrls: ['./defer-view.component.scss', '../styles/view-tab.scss'],
   imports: [MatToolbar],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class DeferViewComponent {
   readonly defer = input.required<NonNullable<DeferBlock>>();

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/property-pane/for-loop-view/for-loop-view.component.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {ChangeDetectionStrategy, Component, computed, input} from '@angular/core';
+import {Component, computed, input} from '@angular/core';
 import {MatExpansionModule} from '@angular/material/expansion';
 import {ForLoopBlock} from '../../../../../../../protocol';
 import {ObjectTreeExplorerComponent} from '../../../../shared/object-tree-explorer/object-tree-explorer.component';
@@ -19,7 +19,6 @@ import {MatToolbar} from '@angular/material/toolbar';
   selector: 'ng-for-loop-view',
   styleUrls: ['./for-loop-view.component.scss', '../styles/view-tab.scss'],
   imports: [MatExpansionModule, ObjectTreeExplorerComponent, MatToolbar],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class ForLoopViewComponent {
   protected readonly forLoop = input.required<NonNullable<ForLoopBlock>>();

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/property-pane/property-pane-header/component-metadata/component-metadata.component.ts

@@ -6,14 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {
-  ChangeDetectionStrategy,
-  Component,
-  ɵFramework as Framework,
-  computed,
-  inject,
-  input,
-} from '@angular/core';
+import {Component, ɵFramework as Framework, computed, inject, input} from '@angular/core';
 
 import {
   AcxDirectiveMetadata,
@@ -29,7 +22,6 @@ import {ElementPropertyResolver} from '../../../property-resolver/element-proper
   templateUrl: './component-metadata.component.html',
   styleUrls: ['./component-metadata.component.scss'],
   imports: [DocsRefButtonComponent],
-  changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class ComponentMetadataComponent {
   readonly currentSelectedComponent = input.required<ComponentType>();

devtools/projects/ng-devtools/src/lib/devtools-tabs/directive-explorer/property-pane/property-pane-header/property-pane-header.component.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {ChangeDetectionStrategy, Component, input, output, signal, inject} from '@angular/core';
+import {Component, input, output, signal, inject} from '@angular/core';
 import {MatExpansionModule} from '@angular/material/expansion';
 import {MatIcon} from '@angular/material/icon';
 
@@ -19,7 +19,6 @@ import {SUPPORTED_APIS} from '../../../../application-providers/supported_apis';
   templateUrl: './property-pane-header.component.html',
   selector: 'ng-property-pane-header',
   styleUrls: ['./property-pane-header.component.scss'],
-  changeDetection: ChangeDetectionStrategy.OnPush,
   imports: [MatExpansionModule, MatIcon, ComponentMetadataComponent, ButtonComponent],
 })
 export class PropertyPaneHeaderComponent {