fix: remove @android/ssr from dev and peerDeps

Author: pieh

package-lock.json

@@ -73,7 +73,6 @@
     "node": ">=18.13.0"
   },
   "devDependencies": {
-    "@angular/ssr": "^19.0.0-rc.1",
     "@opentelemetry/api": "~1.8.0",
     "@netlify/eslint-config-node": "^7.0.1",
     "@types/node": "^18.19.0",
@@ -88,14 +87,6 @@
     "fs-extra": "^11.1.1",
     "semver": "^7.5.4"
   },
-  "peerDependencies": {
-    "@angular/ssr": "^19.0.0-rc.1"
-  },
-  "peerDependenciesMeta": {
-    "@angular/ssr": {
-      "optional": true
-    }
-  },
   "jest": {
     "testMatch": [
       "**/test/**/*.js"

package.json

@@ -1,7 +1 @@
-// import { CommonEngine } from '@angular/ssr/node'
-
-// ideally figure out how to use the real CommonEngine type from the Angular package
-// as currently it causes following error (due to repo setup?)
-// ✘ [ERROR] TS2345: Argument of type 'import("/Users/misiek/dev/angular-runtime/tests/fixtures/angular-19-common-engine/node_modules/@angular/ssr/node/index").CommonEngine' is not assignable to parameter of type 'import("/Users/misiek/dev/angular-runtime/node_modules/@angular/ssr/node/index").CommonEngine'.
-//  Types have separate declarations of a private property 'options'. [plugin angular-compiler]
-export declare function render(commonEngine: any): Promise<Response>
+export declare function render(commonEngine: unknown): Promise<Response>

src/common-engine.d.ts

@@ -1,9 +1,7 @@
-/* eslint-disable no-underscore-dangle */
 import assert from 'node:assert'
-import { Module } from 'node:module'
 import { join } from 'node:path'
 import { join as posixJoin } from 'node:path/posix'
-import { test, describe, before, after } from 'node:test'
+import { test, describe } from 'node:test'
 import { fileURLToPath } from 'node:url'
 
 import build from '@netlify/build'
@@ -98,21 +96,6 @@ test('Angular 19 using App Engine (Developer Preview)', async () => {
 })
 
 describe('Angular version validation', () => {
-  let originalNodeModulePaths
-  before(() => {
-    // Node.js automatically add all parent dirs to lookups, so this test will find @angular/core from main project dev dependencies
-    // even if test fixture itself doesn't contain it.
-
-    // to workaround this we just allow lookup in test fixture dir inspired by https://stackoverflow.com/questions/32455431/prevent-require-from-looking-up-modules-in-the-parent-directory
-
-    originalNodeModulePaths = Module._nodeModulePaths
-    Module._nodeModulePaths = (...args) => originalNodeModulePaths.apply(Module, args).slice(0, 1)
-  })
-
-  after(() => {
-    Module._nodeModulePaths = originalNodeModulePaths
-  })
-
   test('checks version for angular 19', async () => {
     const result = validateAngularVersion(
       await getAngularVersion(fileURLToPath(new URL('fixtures/angular-19-common-engine', import.meta.url))),
@@ -141,4 +124,3 @@ describe('Angular version validation', () => {
     assert.strictEqual(result, false)
   })
 })
-/* eslint-enable no-underscore-dangle */