Switch to named server config exports

Author: jhildenbiddle

jest.config.js

@@ -1,6 +1,6 @@
-import serverConfigs from './server.configs.js';
+import { testConfig } from './server.configs.js';
 
-const { hostname, port } = serverConfigs.test;
+const { hostname, port } = testConfig;
 const TEST_HOST = `http://${hostname}:${port}`;
 const sharedConfig = {
   errorOnDeprecated: true,

playwright.config.js

@@ -1,7 +1,7 @@
 import { devices } from '@playwright/test';
-import serverConfigs from './server.configs.js';
+import { testConfig } from './server.configs.js';
 
-const { hostname, port } = serverConfigs.test;
+const { hostname, port } = testConfig;
 const TEST_HOST = `http://${hostname}:${port}`;
 
 process.env.TEST_HOST = TEST_HOST;

server.configs.js

@@ -6,7 +6,7 @@ const __filename = url.fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
 // Production (CDN URLs, watch disabled)
-const prod = {
+export const prodConfig = {
   hostname: '127.0.0.1',
   notify: false,
   open: false,
@@ -19,13 +19,13 @@ const prod = {
 };
 
 // Development (local URLs, watch enabled)
-const dev = {
-  ...prod,
+export const devConfig = {
+  ...prodConfig,
   files: ['CHANGELOG.md', 'docs/**/*', 'lib/**/*'],
   port: 3000,
   rewriteRules,
   server: {
-    ...prod.server,
+    ...prodConfig.server,
     routes: {
       '/changelog.md': path.resolve(__dirname, 'CHANGELOG.md'),
       '/lib': path.resolve(__dirname, 'lib'),
@@ -36,11 +36,11 @@ const dev = {
 };
 
 // Test (local URLs, watch disabled)
-const test = {
-  ...dev,
+export const testConfig = {
+  ...devConfig,
   port: 4000,
   server: {
-    ...dev.server,
+    ...devConfig.server,
     middleware: [
       // Blank page required for test environment
       {
@@ -56,9 +56,3 @@ const test = {
   snippet: false,
   watch: false,
 };
-
-export default {
-  dev,
-  prod,
-  test,
-};

server.js

@@ -1,15 +1,13 @@
 import { create } from 'browser-sync';
-import serverConfigs from './server.configs.js';
+import { devConfig, prodConfig } from './server.configs.js';
 
 const bsServer = create();
 const args = process.argv.slice(2);
-const configName =
-  Object.keys(serverConfigs).find(name => args.includes(`--${name}`)) || 'prod';
-const settings = serverConfigs[configName];
-const isWatch = Boolean(settings.files) && settings.watch !== false;
+const config = args.includes('--dev') ? devConfig : prodConfig;
+const isWatch = Boolean(config.files) && config.watch !== false;
 const urlType = configName === 'prod' ? 'CDN' : 'local';
 
 // prettier-ignore
 console.log(`\nStarting ${configName} server (${urlType} URLs, watch: ${isWatch})\n`);
 
-bsServer.init(settings);
+bsServer.init(config);

test/config/server.js

@@ -1,13 +1,13 @@
 import * as process from 'node:process';
 import { create } from 'browser-sync';
-import serverConfigs from '../../server.configs.js';
+import { testConfig } from '../../server.configs.js';
 
 const bsServer = create();
 
 export async function startServer() {
   // Wait for server to start
   return new Promise(resolve => {
-    const settings = serverConfigs.test;
+    const settings = testConfig;
 
     console.log('\n');