feat: simplify local hostname to <slug>.lvh.me

src/lib/hostname.ts

@@ -15,8 +15,9 @@ export function sanitizeHostname(raw: string): string {
 }
 
 export function buildProjectHostname(projectSlug: string, subdomain?: string): string {
-  const machine = getMachineHostname();
-  const base = `${projectSlug}.${machine}.lvh.me`;
+  // lvh.me always resolves to 127.0.0.1, so the project slug alone is enough
+  // for a unique, easy-to-type local hostname — no machine segment needed.
+  const base = `${projectSlug}.lvh.me`;
   if (subdomain) {
     return `${subdomain}.${base}`;
   }

tests/lib/hostname.test.ts

@@ -1,14 +1,6 @@
-import os from 'node:os';
-import {describe, expect, it, vi} from 'vitest';
+import {describe, expect, it} from 'vitest';
 import {buildProjectHostname, sanitizeHostname} from '../../src/lib/hostname.js';
 
-vi.mock('node:os', () => ({
-  default: {hostname: vi.fn()},
-  hostname: vi.fn(),
-}));
-
-const mockHostname = vi.mocked(os.hostname);
-
 describe('sanitizeHostname', () => {
   it('lowercases and strips .local suffix', () => {
     expect(sanitizeHostname('Rasmus-MacBook.local')).toBe('rasmus-macbook');
@@ -28,15 +20,13 @@ describe('sanitizeHostname', () => {
 });
 
 describe('buildProjectHostname', () => {
-  it('combines project slug with machine hostname', () => {
-    mockHostname.mockReturnValue('rasmus-macbook.local');
-    expect(buildProjectHostname('my-theme')).toBe('my-theme.rasmus-macbook.lvh.me');
+  it('uses the project slug directly under lvh.me', () => {
+    expect(buildProjectHostname('my-theme')).toBe('my-theme.lvh.me');
   });
 
-  it('builds phpmyadmin subdomain', () => {
-    mockHostname.mockReturnValue('rasmus-macbook.local');
+  it('builds a phpmyadmin subdomain', () => {
     expect(buildProjectHostname('my-theme', 'phpmyadmin')).toBe(
-      'phpmyadmin.my-theme.rasmus-macbook.lvh.me',
+      'phpmyadmin.my-theme.lvh.me',
     );
   });
 });

tests/lib/status.test.ts

@@ -10,13 +10,6 @@ import {
 } from '../../src/lib/status.js';
 import type {ProjectConfig} from '../../src/types/config.js';
 
-// buildProjectHostname embeds the machine hostname; pin it so URL assertions
-// are deterministic regardless of the host the test runs on.
-vi.mock('node:os', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('node:os')>();
-  return {...actual, default: {...actual, hostname: () => 'testbox'}};
-});
-
 const PROJECT: ProjectConfig = {
   project_id: 'proj-123',
   name: 'acme',
@@ -78,9 +71,9 @@ describe('getProjectStatus', () => {
       {name: 'proj-123-phpmyadmin', running: true},
     ]);
     expect(status.urls).toEqual({
-      site: 'http://acme.testbox.lvh.me:5477',
-      admin: 'http://acme.testbox.lvh.me:5477/wp-admin',
-      phpmyadmin: 'http://phpmyadmin.acme.testbox.lvh.me:5477',
+      site: 'http://acme.lvh.me:5477',
+      admin: 'http://acme.lvh.me:5477/wp-admin',
+      phpmyadmin: 'http://phpmyadmin.acme.lvh.me:5477',
     });
   });