fix: correctly allow empty ignoredHostnames array to exclude localhost (fixes #30)

Author: johannschopplich

README.md

@@ -51,6 +51,9 @@ export default defineNuxtConfig({
 })
 ```
 
+> [!TIP]
+> To allow tracking events on localhost, set the `ignoredHostnames` option to an empty array.
+
 ### Runtime Config
 
 Alternatively, leveraging [automatically replaced public runtime config values](https://nuxt.com/docs/api/configuration/nuxt-config#runtimeconfig) by matching environment variables at runtime, set your desired option in your project's `.env` file:

src/module.ts

@@ -105,7 +105,7 @@ export default defineNuxtModule<ModuleOptions>({
     enabled: true,
     hashMode: false,
     domain: '',
-    ignoredHostnames: [],
+    ignoredHostnames: undefined,
     ignoreSubDomains: false,
     trackLocalhost: false,
     apiHost: 'https://plausible.io',
@@ -117,14 +117,12 @@ export default defineNuxtModule<ModuleOptions>({
     const logger = useLogger('plausible')
     const { resolve } = createResolver(import.meta.url)
 
+    // Set default hostnames if `ignoredHostnames` is not set
+    options.ignoredHostnames ??= [...DEFAULT_HOSTNAMES]
+
     // Dedupe `ignoredHostnames` items
     options.ignoredHostnames = Array.from(new Set(options.ignoredHostnames))
 
-    // Add default hostnames if `ignoredHostnames` is empty
-    if (options.ignoredHostnames.length === 0) {
-      options.ignoredHostnames = DEFAULT_HOSTNAMES
-    }
-
     // Migrate `trackLocalhost` to `ignoredHostnames`
     if (options.trackLocalhost) {
       logger.warn('The `trackLocalhost` option has been deprecated. Please use `ignoredHostnames` instead.')