chore: release prep for v1.17.0

Author: SunDevil311

.github/workflows/meta-check.yml

@@ -0,0 +1,34 @@
+# .github/workflows/meta-check.yml
+#
+# Copyright © 2025 Network Pro Strategies (Network Pro™)
+# SPDX-License-Identifier: CC-BY-4.0 OR GPL-3.0-or-later
+# This file is part of Network Pro
+
+name: Metadata Check
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  meta:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v5
+        with:
+          node-version: 24
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run meta tests
+        run: npx vitest run tests/meta/meta.test.js --reporter=verbose

.github/workflows/playwright.yml

@@ -12,9 +12,9 @@ on:
     branches: [master]
   workflow_dispatch:
 
-# ✅ Least-privilege access
+# Required for actions/checkout
 permissions:
-  contents: read # Required for actions/checkout
+  contents: read
 
 jobs:
   test:

CHANGELOG.md

@@ -35,8 +35,20 @@ This project attempts to follow [Keep a Changelog](https://keepachangelog.com/en
   - Documented use of both `report-uri` (legacy) and `report-to` (modern, recommended).
   - Added example headers including `Report-To` definition.
 
+### Added
+
+- New `meta-check.yml` GitHub Actions workflow to validate `<title>` and `<meta>` descriptions using Vitest.
+  - Runs separately from Playwright to avoid hydration timing issues.
+  - Ensures SEO metadata is tested in CI without blocking other jobs.
+- New `meta.test.js` file in `tests/unit/meta` for testing in CI.
+- New `meta.test.js` file in `tests/unit/server` for local testing.
+
 ### Changed
 
+- Cleaned up Playwright E2E tests:
+  - Removed brittle `toHaveTitle` assertions causing CI failures.
+  - Standardized footer checks to use helper function.
+  - Improved test readability and consistency in `app.spec.js` and `mobile.spec.js`.
 - Updated generator metadata in `app.html` to reflect `SvelteKit 2.42.1`.
 - Updated Node.js version in `.node-version` and `.nvmrc` to `24.8.0`.
 - Updated CSP endpoint section and footer in `README.md`.
@@ -71,8 +83,8 @@ This project attempts to follow [Keep a Changelog](https://keepachangelog.com/en
 
 ### Removed
 
-- Deleted `src/routes/example.svx`, which was unused and unneeded
-- Removed `mdsvex` from package.json, as it is unlikely to be used
+- Deleted `src/routes/example.svx`, which was unused and unneeded.
+- Removed `mdsvex` from package.json, as it is unlikely to be used.
 
 ### Notes
 

tests/e2e/app.spec.js

@@ -11,7 +11,7 @@ This file is part of Network Pro.
  * @description Runs Playwright E2E tests with desktop and root route assertions.
  * @module tests/e2e
  * @author SunDevil311
- * @updated 2025-05-29
+ * @updated 2025-09-17
  */
 
 import { expect, test } from '@playwright/test';
@@ -22,28 +22,14 @@ import {
   setMobileView,
 } from './shared/helpers.js';
 
-// Root route should load successfully with the correct title
+// Root route should display nav bar and about link
 test.describe('Desktop Tests', () => {
-  test('should load successfully with the correct title', async ({
-    page,
-    browserName,
-  }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setDesktopView(page);
-    await page.goto('/');
-    await page.waitForLoadState('load', { timeout: 60000 });
-    await expect(page).toHaveTitle(/Security, Networking, Privacy/, {
-      timeout: 60000,
-    });
-  });
-
-  // Root route should display nav bar and about link
   test("should display the navigation bar and 'about' link", async ({
     page,
   }) => {
     await setDesktopView(page);
     await page.goto('/');
+    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
 
     const nav = await getVisibleNav(page);
 
@@ -56,8 +42,9 @@ test.describe('Desktop Tests', () => {
   test('should display the footer correctly', async ({ page }) => {
     await setDesktopView(page);
     await page.goto('/');
+    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
 
-    const footer = page.locator('footer');
+    const footer = getFooter(page);
     await expect(footer).toBeVisible();
   });
 
@@ -67,6 +54,7 @@ test.describe('Desktop Tests', () => {
   }) => {
     await setDesktopView(page);
     await page.goto('/about');
+    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
 
     const footer = getFooter(page);
     await expect(footer).toBeVisible();
@@ -76,40 +64,29 @@ test.describe('Desktop Tests', () => {
   test("should ensure the 'about' link is clickable", async ({ page }) => {
     await setDesktopView(page);
     await page.goto('/');
+    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
 
     const nav = await getVisibleNav(page);
 
     const aboutLink = nav.getByRole('link', { name: 'about' });
-    await expect(aboutLink).toBeVisible({ timeout: 60000 });
+    await expect(aboutLink).toBeVisible();
     await aboutLink.click();
 
     await page.waitForURL('/about', { timeout: 60000 });
     await expect(page).toHaveURL(/\/about/);
   });
 }); // End Desktop Tests
 
-// Root route should load successfully with the correct title (mobile)
+// Root route should display headings properly on mobile
 test.describe('Mobile Tests', () => {
-  test('should load successfully with the correct title on mobile', async ({
-    page,
-    browserName,
-  }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setMobileView(page);
-    await page.goto('/');
-    await page.waitForLoadState('load', { timeout: 60000 });
-    await expect(page).toHaveTitle(/Security, Networking, Privacy/, {
-      timeout: 60000,
-    });
-  });
-
-  // Root route should display headings properly on mobile
   test('should display main content correctly on mobile', async ({ page }) => {
     await setMobileView(page);
     await page.goto('/');
+    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
 
     const mainHeading = page.locator('h1, h2');
     await expect(mainHeading).toBeVisible();
   });
 });
+
+// cspell:ignore domcontentloaded

tests/e2e/mobile.spec.js

@@ -11,7 +11,7 @@ This file is part of Network Pro.
  * @description Runs Playwright E2E tests with mobile assertions.
  * @module tests/e2e
  * @author SunDevil311
- * @updated 2025-05-29
+ * @updated 2025-09-17
  */
 
 import { expect, test } from '@playwright/test';
@@ -60,10 +60,10 @@ test.describe('Mobile Tests', () => {
 
     const nav = await getVisibleNav(page);
     const aboutLink = nav.getByRole('link', { name: 'about' });
-    await expect(aboutLink).toBeVisible({ timeout: 60000 });
+    await expect(aboutLink).toBeVisible();
 
     await aboutLink.click();
-    await expect(page).toHaveURL(/\/about/);
+    await expect(page).toHaveURL(/\/about/, { timeout: 60000 });
   });
 
   test('should display the footer on /about (mobile)', async ({

tests/unit/meta/meta.test.js

@@ -0,0 +1,40 @@
+/* ==========================================================================
+tests/unit/meta/meta.test.js
+
+Copyright © 2025 Network Pro Strategies (Network Pro™)
+SPDX-License-Identifier: CC-BY-4.0 OR GPL-3.0-or-later
+This file is part of Network Pro.
+========================================================================== */
+
+/**
+ * @file meta.test.js
+ * @description Checks for correct metadata population in CI
+ * @module tests/unit/meta
+ * @author SunDevil311
+ * @updated 2025-09-17
+ */
+
+import { describe, expect, it } from 'vitest';
+import { load } from '../../../src/routes/+layout.js';
+
+const routes = [
+  {
+    path: '/',
+    title: /Security, Networking, Privacy/,
+    description: /Network Pro/,
+  },
+  // add more paths as needed:
+  // { path: '/about', title: /About/, description: /Network Pro/ }
+];
+
+describe('Meta checks', () => {
+  for (const route of routes) {
+    it(`should return correct meta for ${route.path}`, () => {
+      const mockUrl = new URL(`http://localhost${route.path}`);
+      const { meta } = load({ url: mockUrl });
+
+      expect(meta.title).toMatch(route.title);
+      expect(meta.description).toMatch(route.description);
+    });
+  }
+});

tests/unit/server/internal/auditCoverage.test.js

@@ -10,9 +10,9 @@ This file is part of Network Pro.
  * @file auditCoverage.test.js
  * @description Scans all .js files in src/ and scripts/ for matching unit
  * tests
- * @module tests/internal
+ * @module tests/unit/server/internal
  * @author SunDevil311
- * @updated 2025-07-01
+ * @updated 2025-09-17
  */
 
 import fs from 'fs';

tests/unit/server/meta.test.js

@@ -0,0 +1,30 @@
+/* ==========================================================================
+tests/unit/server/meta.test.js
+
+Copyright © 2025 Network Pro Strategies (Network Pro™)
+SPDX-License-Identifier: CC-BY-4.0 OR GPL-3.0-or-later
+This file is part of Network Pro.
+========================================================================== */
+
+/**
+ * @file meta.test.js
+ * @description Checks for correct metadata population
+ * @module tests/unit/server
+ * @author SunDevil311
+ * @updated 2025-09-17
+ */
+
+import { describe, expect, it } from 'vitest';
+import { load } from '../../../src/routes/+layout.js'; // adjust path if needed
+
+describe('Meta info', () => {
+  it('should have correct title and description', () => {
+    // Mock a URL similar to what SvelteKit provides
+    const mockUrl = new URL('http://localhost/');
+
+    const { meta } = load({ url: mockUrl });
+
+    expect(meta.title).toMatch(/Security, Networking, Privacy/);
+    expect(meta.description).toMatch(/Network Pro/);
+  });
+});