Merge pull request #113 from railwayapp/jr/pro-3941-fix-remixreact-router-run-runtime-detection

fix remix metadata detection

core/__snapshots__/TestGenerateBuildPlanForExamples_node-remix_1.snap.json

@@ -8,6 +8,10 @@
    "directory": "/root/.npm",
    "type": "shared"
   },
+  "remix": {
+   "directory": ".cache",
+   "type": "shared"
+  },
   "vite": {
    "directory": "node_modules/.vite",
    "type": "shared"
@@ -133,6 +137,7 @@
   {
    "caches": [
     "node-modules",
+    "remix",
     "vite"
    ],
    "commands": [

core/providers/node/node.go

@@ -484,5 +484,5 @@ func (p *NodeProvider) isNuxt() bool {
 }
 
 func (p *NodeProvider) isRemix() bool {
-	return p.hasDependency("remix") && p.hasDependency("@remix-run/node")
+	return p.hasDependency("@remix-run/node")
 }

core/providers/node/spa.go

@@ -41,6 +41,10 @@ func (p *NodeProvider) isSPA(ctx *generate.GenerateContext) bool {
 }
 
 func (p *NodeProvider) getSPAFramework(ctx *generate.GenerateContext) string {
+	if !p.isSPA(ctx) {
+		return ""
+	}
+
 	if p.isVite(ctx) {
 		return "vite"
 	} else if p.isAstro(ctx) {