Refactor fastify-vite internals and improve code quality (#388)

- Rename Vite class to FastifyViteDecoration for clarity
- Rename this.config to this.runtimeConfig to distinguish from Vite config
- Refactor mode setup functions to accept context object instead of using
  `this` binding, improving testability and type safety
- Extract shared FastifyViteDecorationPriorToSetup interface and
  hasIterableRoutes helper to new support.ts module
- Move loadBundle and loadEntries to module scope in production mode
- Simplify development.ts by importing vite at module level

Code quality improvements:

- Remove unused variables and function parameters across codebase
- Add typescript/consistent-type-definitions eslint rule
- Convert type aliases to interfaces where appropriate
- Use type-only imports where applicable
- Extend oxlint with additional plugins (eslint, oxc, typescript, unicorn)

Testing:

- Enable production mode tests
- Add vitest globalSetup to build test fixtures before running tests

Author: onlywei

.oxlintrc.json

@@ -3,7 +3,7 @@
   "categories": {
     "correctness": "error"
   },
-  "plugins": ["import"],
+  "plugins": ["eslint", "import", "oxc", "typescript", "unicorn"],
   "rules": {
     "import/extensions": [
       "error",
@@ -12,6 +12,7 @@
         "ignorePackages": true
       }
     ],
-    "no-unused-vars": "warn"
+    "no-unused-vars": "warn",
+    "typescript/consistent-type-definitions": "error"
   }
 }

examples/middie-compat/server.js

@@ -25,7 +25,7 @@ export async function mainMiddieFirst(dev) {
     return reply.html()
   })
 
-  server.get('/middleware-test', (req, reply) => {
+  server.get('/middleware-test', (req) => {
     return { customMiddleware: req.raw.customMiddleware ?? false }
   })
 
@@ -56,7 +56,7 @@ export async function mainViteFirst(dev) {
     return reply.html()
   })
 
-  server.get('/middleware-test', (req, reply) => {
+  server.get('/middleware-test', (req) => {
     return { customMiddleware: req.raw.customMiddleware ?? false }
   })
 

examples/react-next-mini/renderer.js

@@ -11,7 +11,7 @@ export default {
   createRoute,
 }
 
-function createRoute({ handler, errorHandler, route }, scope, config) {
+function createRoute({ handler, errorHandler, route }, scope) {
   if (route.getServerSideProps) {
     // If getServerSideProps is provided, register JSON endpoint for it
     scope.get(`/json${route.path}`, async (req, reply) => {
@@ -27,7 +27,7 @@ function createRoute({ handler, errorHandler, route }, scope, config) {
     // If getServerSideProps is provided,
     // make sure it runs before the SSR route handler
     ...(route.getServerSideProps && {
-      async preHandler(req, reply) {
+      async preHandler(req) {
         req.serverSideProps = await route.getServerSideProps({
           req,
           fetchJSON: scope.fetchJSON,

examples/react-streaming/client/views/index.jsx

@@ -3,7 +3,7 @@ import { Link } from 'react-router'
 import { useAtom } from 'jotai'
 import { todoList } from '../state.js'
 
-export default function Index(props) {
+export default function Index() {
   const [state, updateState] = useAtom(todoList)
   const [input, setInput] = useState(null)
   const addItem = async () => {

examples/react-streaming/renderer.js

@@ -22,7 +22,7 @@ export default {
 }
 
 // The return value of this function gets registered as reply.html()
-async function createHtmlFunction(source, scope, config) {
+async function createHtmlFunction(source) {
   const [headSource, footer] = source.split('<!-- element -->')
   const headTemplate = await createHtmlTemplateFunction(headSource)
   return function ({ stream, data }) {

examples/vue-hydration/renderer.js

@@ -12,7 +12,7 @@ async function foobarHook(req) {
   req.server.log.info(`Hello from ${req.url} and foobarHook`)
 }
 
-function createRoute({ handler, errorHandler, route }, scope, config) {
+function createRoute({ handler, errorHandler, route }, scope) {
   scope.route({
     url: route.path,
     method: 'GET',

examples/vue-hydration/server.js

@@ -2,7 +2,7 @@ import Fastify from 'fastify'
 import FastifyVite from '@fastify/vite'
 import renderer from './renderer.js'
 
-export async function main(dev) {
+export async function main() {
   const server = Fastify({
     logger: {
       transport: {

examples/vue-next-mini/renderer.js

@@ -16,7 +16,7 @@ export function prepareServer(server) {
   server.log.info('prepareServer() hook picked up from configuration!')
 }
 
-async function createRoute({ handler, errorHandler, route }, scope, config) {
+async function createRoute({ handler, errorHandler, route }, scope) {
   if (route.configure) {
     await route.configure(scope)
   }
@@ -35,7 +35,7 @@ async function createRoute({ handler, errorHandler, route }, scope, config) {
     // If getServerSideProps is provided,
     // make sure it runs before the SSR route handler
     ...(route.getServerSideProps && {
-      async preHandler(req, reply) {
+      async preHandler(req) {
         req.serverSideProps = await route.getServerSideProps({
           req,
           ky: scope.ky,

examples/vue-streaming/renderer.js

@@ -21,7 +21,7 @@ export default {
 }
 
 // The return value of this function gets registered as reply.html()
-async function createHtmlFunction(source, scope, config) {
+async function createHtmlFunction(source) {
   const [headSource, footer] = source.split('<!-- element -->')
   const headTemplate = await createHtmlTemplateFunction(headSource)
   return function ({ stream, data }) {

examples/vue-vanilla-ts/globals.d.ts

@@ -1,5 +1,5 @@
 declare module '*.vue' {
-  import { DefineComponent } from 'vue'
+  import type { DefineComponent } from 'vue'
   const component: DefineComponent<{}, {}, any>
   export default component
 }