fix(component): use setTimeout instead of requestAnimationFrame for server and zoneless

requestAnimationFrame is not available in server-side contexts and will cause errors.
Using setTimeout ensures compatibility across all environments.

Author: JelleBruisten

modules/component/spec/core/tick-scheduler.spec.ts

@@ -4,18 +4,24 @@ import {
   TestBed,
   tick,
 } from '@angular/core/testing';
-import { ApplicationRef, NgZone } from '@angular/core';
+import { ApplicationRef, NgZone, PLATFORM_ID } from '@angular/core';
 import {
-  AnimationFrameTickScheduler,
+  ZonelessTickScheduler,
   NoopTickScheduler,
   TickScheduler,
 } from '../../src/core/tick-scheduler';
 import { ngZoneMock, noopNgZoneMock } from '../fixtures/fixtures';
 
 describe('TickScheduler', () => {
-  function setup(ngZone: unknown) {
+  function setup(ngZone: unknown, server: boolean = false) {
     TestBed.configureTestingModule({
-      providers: [{ provide: NgZone, useValue: ngZone }],
+      providers: [
+        { provide: NgZone, useValue: ngZone },
+        {
+          provide: PLATFORM_ID,
+          useValue: server ? 'server' : 'browser',
+        },
+      ],
     });
     const tickScheduler = TestBed.inject(TickScheduler);
     const appRef = TestBed.inject(ApplicationRef);
@@ -31,16 +37,16 @@ describe('TickScheduler', () => {
     });
   });
 
-  describe('when NgZone is not provided', () => {
+  describe('when NgZone is not provided and running in server context', () => {
     // `fakeAsync` uses 16ms as `requestAnimationFrame` delay
     const animationFrameDelay = 16;
 
-    it('should initialize AnimationFrameTickScheduler', () => {
+    it('should initialize ZonelessTickScheduler', () => {
       const { tickScheduler } = setup(noopNgZoneMock);
-      expect(tickScheduler instanceof AnimationFrameTickScheduler).toBe(true);
+      expect(tickScheduler instanceof ZonelessTickScheduler).toBe(true);
     });
 
-    it('should schedule tick using the animationFrameScheduler', fakeAsync(() => {
+    it('should schedule tick using the ZonelessTickScheduler', fakeAsync(() => {
       const { tickScheduler, appRef } = setup(noopNgZoneMock);
 
       tickScheduler.schedule();
@@ -87,4 +93,56 @@ describe('TickScheduler', () => {
       expect(appRef.tick).toHaveBeenCalledTimes(3);
     }));
   });
+
+  describe('when NgZone is not provided', () => {
+    it('should initialize ZonelessTickScheduler', () => {
+      const { tickScheduler } = setup(noopNgZoneMock, true);
+      expect(tickScheduler instanceof ZonelessTickScheduler).toBe(true);
+    });
+
+    it('should schedule tick using the ZonelessTickScheduler', fakeAsync(() => {
+      const { tickScheduler, appRef } = setup(noopNgZoneMock, true);
+
+      tickScheduler.schedule();
+
+      expect(appRef.tick).toHaveBeenCalledTimes(0);
+      tick();
+      expect(appRef.tick).toHaveBeenCalledTimes(1);
+    }));
+
+    it('should coalesce multiple synchronous schedule calls', fakeAsync(() => {
+      const { tickScheduler, appRef } = setup(noopNgZoneMock, true);
+
+      tickScheduler.schedule();
+      tickScheduler.schedule();
+      tickScheduler.schedule();
+
+      tick();
+      expect(appRef.tick).toHaveBeenCalledTimes(1);
+    }));
+
+    it('should coalesce multiple schedule calls that are queued to the microtask queue', fakeAsync(() => {
+      const { tickScheduler, appRef } = setup(noopNgZoneMock, true);
+
+      queueMicrotask(() => tickScheduler.schedule());
+      queueMicrotask(() => tickScheduler.schedule());
+      queueMicrotask(() => tickScheduler.schedule());
+
+      flushMicrotasks();
+      expect(appRef.tick).toHaveBeenCalledTimes(0);
+      tick();
+      expect(appRef.tick).toHaveBeenCalledTimes(1);
+    }));
+
+    it('should schedule multiple ticks for multiple asynchronous schedule calls', fakeAsync(() => {
+      const { tickScheduler, appRef } = setup(noopNgZoneMock, true);
+
+      setTimeout(() => tickScheduler.schedule(), 100);
+      setTimeout(() => tickScheduler.schedule(), 200);
+      setTimeout(() => tickScheduler.schedule(), 300);
+
+      tick(300);
+      expect(appRef.tick).toHaveBeenCalledTimes(3);
+    }));
+  });
 });

modules/component/src/core/tick-scheduler.ts

@@ -1,13 +1,20 @@
-import { ApplicationRef, inject, Injectable, NgZone } from '@angular/core';
+import {
+  ApplicationRef,
+  inject,
+  Injectable,
+  NgZone,
+  PLATFORM_ID,
+} from '@angular/core';
 import { isNgZone } from './zone-helpers';
+import { isPlatformServer } from '@angular/common';
 
 @Injectable({
   providedIn: 'root',
   useFactory: () => {
     const zone = inject(NgZone);
     return isNgZone(zone)
       ? new NoopTickScheduler()
-      : inject(AnimationFrameTickScheduler);
+      : inject(ZonelessTickScheduler);
   },
 })
 export abstract class TickScheduler {
@@ -17,17 +24,19 @@ export abstract class TickScheduler {
 @Injectable({
   providedIn: 'root',
 })
-export class AnimationFrameTickScheduler extends TickScheduler {
+export class ZonelessTickScheduler extends TickScheduler {
   private isScheduled = false;
-
-  constructor(private readonly appRef: ApplicationRef) {
-    super();
-  }
+  private readonly platformId = inject(PLATFORM_ID);
+  private readonly isServer = isPlatformServer(this.platformId);
+  private readonly appRef = inject(ApplicationRef);
+  private readonly scheduleFn = this.isServer
+    ? setTimeout
+    : requestAnimationFrame;
 
   schedule(): void {
     if (!this.isScheduled) {
       this.isScheduled = true;
-      requestAnimationFrame(() => {
+      this.scheduleFn(() => {
         this.appRef.tick();
         this.isScheduled = false;
       });