feat(vue-app): context.beforeSerialize method

packages/types/app/index.d.ts

@@ -72,6 +72,7 @@ export interface Context {
     redirected: boolean
     next: NextFunction
     beforeRenderFns: Array<() => any>
+    beforeRenderHooks: Array<() => any>
     fetchCounters: Record<string, number>
     nuxt: {
       layout: string
@@ -87,6 +88,7 @@ export interface Context {
   error(params: NuxtError): void
   nuxtState: NuxtState
   beforeNuxtRender(fn: (params: { Components: VueRouter['getMatchedComponents'], nuxtState: NuxtState }) => void): void
+  beforeRender(fn: (params: { Components: VueRouter['getMatchedComponents'], nuxtState: NuxtState }) => void): void
   enablePreview?: (previewData?: Record<string, any>) => void
   $preview?: Record<string, any>
 }

packages/vue-app/template/index.js

@@ -183,6 +183,7 @@ async function createApp(ssrContext, config = {}) {
     req: ssrContext ? ssrContext.req : undefined,
     res: ssrContext ? ssrContext.res : undefined,
     beforeRenderFns: ssrContext ? ssrContext.beforeRenderFns : undefined,
+    beforeRenderHooks: ssrContext ? ssrContext.beforeRenderHooks : undefined,
     ssrContext
   })
 

packages/vue-app/template/server.js

@@ -83,8 +83,10 @@ export default async (ssrContext) => {
   // Create ssrContext.next for simulate next() of beforeEach() when wanted to redirect
   ssrContext.redirected = false
   ssrContext.next = createNext(ssrContext)
-  // Used for beforeNuxtRender({ Components, nuxtState })
+  // deprecated: Used for beforeNuxtRender({ Components, nuxtState })
   ssrContext.beforeRenderFns = []
+  // for beforeRender({ Components, nuxtState })
+  ssrContext.beforeRenderHooks = []
   // Nuxt object (window.{{globals.context}}, defaults to window.__NUXT__)
   ssrContext.nuxt = { <% if (features.layouts) { %>layout: 'default', <% } %>data: [], <% if (features.fetch) { %>fetch: {}, <% } %>error: null<%= (store ? ', state: null' : '') %>, serverRendered: true, routePath: '' }
   <% if (features.fetch) { %>
@@ -118,18 +120,23 @@ export default async (ssrContext) => {
   <% } %>
 
   const beforeRender = async () => {
-    // Call beforeNuxtRender() methods
+    // Deprecated: Call beforeNuxtRender() methods
     await Promise.all(ssrContext.beforeRenderFns.map(fn => promisify(fn, { Components, nuxtState: ssrContext.nuxt })))
-    <% if (store) { %>
+
     ssrContext.rendered = () => {
+      // Call beforeRender() hooks
+      ssrContext.beforeRenderHooks.forEach(fn => fn({ Components, nuxtState: ssrContext.nuxt }))
+
+      <% if (store) { %>
       // Add the state from the vuex store
       ssrContext.nuxt.state = store.state
+      <% } %>
+
       <% if (isFullStatic && store) { %>
       // Stop recording store mutations
       ssrContext.unsetMutationObserver()
       <% } %>
     }
-    <% } %>
   }
 
   const renderErrorPage = async () => {

packages/vue-app/template/utils.js

@@ -258,7 +258,11 @@ export async function setContext (app, context) {
       }
     }
     if (process.server) {
-      app.context.beforeNuxtRender = fn => context.beforeRenderFns.push(fn)
+      app.context.beforeNuxtRender = fn => {
+        console.warn('`context.beforeNuxtRender(fn)` is deprecated in favour of `context.beforeRender(fn)`. The function updating the `nuxtState` has to be synchronous. Learn more on https://nuxtjs.org/docs/2.x/internals-glossary/context#beforenuxtrender')
+        context.beforeRenderFns.push(fn)
+      }
+      app.context.beforeRender = fn => context.beforeRenderHooks.push(fn)
     }
     if (process.client) {
       app.context.nuxtState = window.<%= globals.context %>

test/dev/basic.ssr.test.js

@@ -201,10 +201,16 @@ describe('basic ssr', () => {
     })
   })
 
-  test('/special-state -> check window.__NUXT__.test = true', async () => {
-    const window = await nuxt.server.renderAndGetWindow(url('/special-state'))
+  test('/before-nuxt-render -> check window.__NUXT__.beforeNuxtRender = true', async () => {
+    const window = await nuxt.server.renderAndGetWindow(url('/before-nuxt-render'))
     expect(window.document.title).toBe('Nuxt')
-    expect(window.__NUXT__.test).toBe(true)
+    expect(window.__NUXT__.beforeNuxtRender).toBe(true)
+  })
+
+  test('/before-render -> check window.__NUXT__.beforeRender = true', async () => {
+    const window = await nuxt.server.renderAndGetWindow(url('/before-render'))
+    expect(window.document.title).toBe('Nuxt')
+    expect(window.__NUXT__.beforeRender).toBe(true)
   })
 
   test('/error', async () => {

test/fixtures/basic/pages/special-state.vue → test/fixtures/basic/pages/before-nuxt-render.vue

@@ -7,7 +7,7 @@ export default {
   middleware ({ beforeNuxtRender }) {
     if (process.server) {
       beforeNuxtRender(({ nuxtState }) => {
-        nuxtState.test = true
+        nuxtState.beforeNuxtRender = true
       })
     }
   }

test/fixtures/basic/pages/before-render.vue

@@ -0,0 +1,26 @@
+<template>
+  <div>
+    <h1>Special state in `window.__NUXT__`</h1>
+    <client-only><pre>{{ nuxtState }}</pre></client-only>
+  </div>
+</template>
+
+<script>
+export default {
+  data () {
+    return {
+      nuxtState: null
+    }
+  },
+  fetch () {
+    if (process.server) {
+      this.$root.context.beforeRender(({ nuxtState }) => {
+        nuxtState.beforeRender = true
+      })
+    }
+  },
+  beforeMount () {
+    this.nuxtState = window.__NUXT__
+  }
+}
+</script>