feat(vue-renderer): improvements

[pi0]

Types of changes

• Bug fix (a non-breaking change which fixes an issue)
• New feature (a non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

• refactor renderRoute
• new hooks

New Hooks

Shared:

• vue-renderer:context: Final chance to extend context. (Setting modern, spa, url, etc)

SSR:

• vue-renderer:ssr:prepareContext: Inject to the context before renderToString
• vue-renderer:ssr:context: Extend rendered context. Ultimate possibilities for SSR hacks!
• vue-renderer:ssr:csp: Extend generated CSP string hashes
• vue-renderer:ssr:templateParams: Customize HTML template params

SPA:

• vue-renderer:spa:templateParams: Cutomize HTML template params for SPA

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly. (PR: #)
• I have added tests to cover my changes (if not applicable, please state why)
• All new and existing tests are passing.

[pimlie]

Could we extract all those hook identifiers into some object? That way IDE's can possibly autocomplete them and the chance of errors (by nuxt users) due to small typos is much smaller. Also there is no overview now anywhere with a full list of all available hooks.

I would imagine something like:

export const hookIds = {
  build: {
    done: 'build:done',
  },
  vue-renderer: { // <- dash might be an issue
    ssr: {
      context: 'vue-rendere:ssr:content'
  }
 ...
}

Then if someone wants to use a hook they can either use the text itself (but that'd be mainly for backwards compatiblity) but the preferred way would be to import the nuxtHooks object and just do nuxt.hook(hookIds.build.done, () => {})

-- edit --
another possible way is to remove the need for identifiers, just call nuxt.hooks.build.done(() => {})

[codecov-io]

Codecov Report

Merging #4722 into dev will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##              dev   #4722      +/-   ##
=========================================
+ Coverage   92.79%   92.9%   +0.11%     
=========================================
  Files          73      73              
  Lines        2429    2439      +10     
  Branches      596     599       +3     
=========================================
+ Hits         2254    2266      +12     
+ Misses        160     158       -2     
  Partials       15      15

Impacted Files	Coverage Δ
packages/core/src/nuxt.js	100% <ø> (ø)	⬆️
packages/vue-renderer/src/renderer.js	95.87% <100%> (+1.31%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5220cf...cc10e50. Read the comment docs.

[codecov-io]

Codecov Report

Merging #4722 into dev will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #4722      +/-   ##
==========================================
+ Coverage   92.28%   92.31%   +0.02%     
==========================================
  Files          74       74              
  Lines        2450     2459       +9     
  Branches      603      606       +3     
==========================================
+ Hits         2261     2270       +9     
  Misses        169      169              
  Partials       20       20

Impacted Files	Coverage Δ
packages/core/src/nuxt.js	100% <ø> (ø)	⬆️
packages/vue-renderer/src/renderer.js	94.81% <100%> (+0.25%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 995911d...c78c4a2. Read the comment docs.