fix: invalidate models components cache based on location (#10764)

Author: glowcloud

src/core/plugins/oas31/wrap-components/model.jsx

@@ -12,13 +12,22 @@ const ModelWrapper = createOnlyOAS31ComponentWrapper(
     const { getComponent, fn, getConfigs } = system
     const configs = getConfigs()
 
+    const pathname = window?.location?.pathname ?? "/"
+
+    if (
+      ModelWrapper.ModelWithJSONSchemaContext &&
+      ModelWrapper.pathname === pathname
+    ) {
+      return <ModelWrapper.ModelWithJSONSchemaContext {...props} />
+    }
+
     const Model = getComponent("OAS31Model")
     const withJSONSchemaSystemContext = getComponent(
       "withJSONSchema202012SystemContext"
     )
 
     // we cache the HOC as recreating it with every re-render is quite expensive
-    ModelWrapper.ModelWithJSONSchemaContext ??= withJSONSchemaSystemContext(
+    ModelWrapper.ModelWithJSONSchemaContext = withJSONSchemaSystemContext(
       Model,
       {
         config: {
@@ -36,9 +45,13 @@ const ModelWrapper = createOnlyOAS31ComponentWrapper(
         },
       }
     )
+    ModelWrapper.pathname = pathname
 
     return <ModelWrapper.ModelWithJSONSchemaContext {...props} />
   }
 )
 
+ModelWrapper.ModelWithJSONSchemaContext = null
+ModelWrapper.pathname = null
+
 export default ModelWrapper

src/core/plugins/oas31/wrap-components/models.jsx

@@ -10,7 +10,12 @@ const ModelsWrapper = createOnlyOAS31ComponentWrapper(({ getSystem }) => {
   const { getComponent, fn, getConfigs } = getSystem()
   const configs = getConfigs()
 
-  if (ModelsWrapper.ModelsWithJSONSchemaContext) {
+  const pathname = window?.location?.pathname ?? "/"
+
+  if (
+    ModelsWrapper.ModelsWithJSONSchemaContext &&
+    ModelsWrapper.pathname === pathname
+  ) {
     return <ModelsWrapper.ModelsWithJSONSchemaContext />
   }
 
@@ -20,7 +25,7 @@ const ModelsWrapper = createOnlyOAS31ComponentWrapper(({ getSystem }) => {
   )
 
   // we cache the HOC as recreating it with every re-render is quite expensive
-  ModelsWrapper.ModelsWithJSONSchemaContext ??= withJSONSchemaSystemContext(
+  ModelsWrapper.ModelsWithJSONSchemaContext = withJSONSchemaSystemContext(
     Models,
     {
       config: {
@@ -38,10 +43,12 @@ const ModelsWrapper = createOnlyOAS31ComponentWrapper(({ getSystem }) => {
       },
     }
   )
+  ModelsWrapper.pathname = pathname
 
   return <ModelsWrapper.ModelsWithJSONSchemaContext />
 })
 
 ModelsWrapper.ModelsWithJSONSchemaContext = null
+ModelsWrapper.pathname = null
 
 export default ModelsWrapper

src/core/plugins/oas32/wrap-components/model.jsx

@@ -12,13 +12,22 @@ const ModelWrapper = createOnlyOAS32ComponentWrapper(
     const { getComponent, fn, getConfigs } = system
     const configs = getConfigs()
 
+    const pathname = window?.location?.pathname ?? "/"
+
+    if (
+      ModelWrapper.ModelWithJSONSchemaContext &&
+      ModelWrapper.pathname === pathname
+    ) {
+      return <ModelWrapper.ModelWithJSONSchemaContext {...props} />
+    }
+
     const Model = getComponent("OAS31Model")
     const withJSONSchemaSystemContext = getComponent(
       "withJSONSchema202012SystemContext"
     )
 
     // we cache the HOC as recreating it with every re-render is quite expensive
-    ModelWrapper.ModelWithJSONSchemaContext ??= withJSONSchemaSystemContext(
+    ModelWrapper.ModelWithJSONSchemaContext = withJSONSchemaSystemContext(
       Model,
       {
         config: {
@@ -36,9 +45,13 @@ const ModelWrapper = createOnlyOAS32ComponentWrapper(
         },
       }
     )
+    ModelWrapper.pathname = pathname
 
     return <ModelWrapper.ModelWithJSONSchemaContext {...props} />
   }
 )
 
+ModelWrapper.ModelWithJSONSchemaContext = null
+ModelWrapper.pathname = null
+
 export default ModelWrapper

src/core/plugins/oas32/wrap-components/models.jsx

@@ -10,7 +10,12 @@ const ModelsWrapper = createOnlyOAS32ComponentWrapper(({ getSystem }) => {
   const { getComponent, fn, getConfigs } = getSystem()
   const configs = getConfigs()
 
-  if (ModelsWrapper.ModelsWithJSONSchemaContext) {
+  const pathname = window?.location?.pathname ?? "/"
+
+  if (
+    ModelsWrapper.ModelsWithJSONSchemaContext &&
+    ModelsWrapper.pathname === pathname
+  ) {
     return <ModelsWrapper.ModelsWithJSONSchemaContext />
   }
 
@@ -20,7 +25,7 @@ const ModelsWrapper = createOnlyOAS32ComponentWrapper(({ getSystem }) => {
   )
 
   // we cache the HOC as recreating it with every re-render is quite expensive
-  ModelsWrapper.ModelsWithJSONSchemaContext ??= withJSONSchemaSystemContext(
+  ModelsWrapper.ModelsWithJSONSchemaContext = withJSONSchemaSystemContext(
     Models,
     {
       config: {
@@ -38,10 +43,12 @@ const ModelsWrapper = createOnlyOAS32ComponentWrapper(({ getSystem }) => {
       },
     }
   )
+  ModelsWrapper.pathname = pathname
 
   return <ModelsWrapper.ModelsWithJSONSchemaContext />
 })
 
 ModelsWrapper.ModelsWithJSONSchemaContext = null
+ModelsWrapper.pathname = null
 
 export default ModelsWrapper