chore: release prep for v1.25.11 (#162)

* chore: release prep for v1.25.11

* feat: added "repair" script to package.json

Author: SunDevil311

CHANGELOG.md

@@ -22,6 +22,27 @@ This project attempts to follow [Keep a Changelog](https://keepachangelog.com/en
 
 ---
 
+## [1.25.11] - 2025-11-12
+
+### Added
+
+- `gotoDesktop(page, path)` and `gotoMobile(page, path)` helper functions to streamline viewport + navigation setup.
+- `clickAndWaitForNavigation(page, locator, options)` utility for safe SPA or full-page navigation detection with optional URL pattern matching.
+- `DEBUG_LOGS` flag in `helpers.js` to allow toggling of console logs for test diagnostics.
+- Navigation debug logs to `getVisibleNav()` to indicate which navigation region was detected (when debugging is enabled).
+
+### Changed
+
+- Refactored all E2E tests to use `gotoDesktop()` and `gotoMobile()` for consistency and DRY principles.
+- Replaced brittle direct `waitForNavigation()` usages with `clickAndWaitForNavigation()` helper.
+- Updated mobile and desktop tests to improve consistency across specs and improve visibility assertions.
+
+### Removed
+
+- Legacy direct `setViewportSize()` and `page.goto()` calls from individual test blocks (now handled via `goto*()` helpers).
+
+---
+
 ## [1.25.10] - 2025-11-12
 
 ### Changed
@@ -1855,7 +1876,8 @@ This enables analytics filtering and CSP hardening for the audit environment.
 
 <!-- Link references -->
 
-[Unreleased]: https://github.com/netwk-pro/netwk-pro.github.io/compare/v1.25.10...HEAD
+[Unreleased]: https://github.com/netwk-pro/netwk-pro.github.io/compare/v1.25.11...HEAD
+[1.25.11]: https://github.com/netwk-pro/netwk-pro.github.io/releases/tag/v1.25.11
 [1.25.10]: https://github.com/netwk-pro/netwk-pro.github.io/releases/tag/v1.25.10
 [1.25.9]: https://github.com/netwk-pro/netwk-pro.github.io/releases/tag/v1.25.9
 [1.25.8]: https://github.com/netwk-pro/netwk-pro.github.io/releases/tag/v1.25.8

cspell.json

@@ -14,6 +14,7 @@
     "CCPA",
     "CPRA",
     "cryptomator",
+    "domcontentloaded",
     "dont",
     "elif",
     "Embedder",
@@ -28,6 +29,7 @@
     "homescreen",
     "HREFTOP",
     "HSTS",
+    "interactable",
     "Izzy",
     "Keybase",
     "keypair",
@@ -40,6 +42,7 @@
     "lightningcss",
     "linksheet",
     "linktree",
+    "LOCALAPPDATA",
     "lsheet",
     "Mailvelope",
     "Maricopa",

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@networkpro/web",
   "private": false,
-  "version": "1.25.10",
+  "version": "1.25.11",
   "description": "Locking Down Networks, Unlocking Confidence™ | Security, Networking, Privacy — Network Pro Strategies",
   "keywords": [
     "advisory",
@@ -54,6 +54,7 @@
     "verify": "npm run checkout",
     "delete": "rm -rf .svelte-kit node_modules package-lock.json",
     "clean": "npm run delete && npm cache clean --force && npm install",
+    "repair": "npm run delete && npx playwright uninstall && rm -rf %LOCALAPPDATA%/ms-playwright && npm install && npx playwright install",
     "upgrade": "ncu -u --format group --color",
     "check:updates": "ncu --format group --color",
     "test": "npm run test:all",

package.json

@@ -11,15 +11,16 @@ This file is part of Network Pro.
  * @description Runs Playwright E2E tests with desktop and root route assertions.
  * @module tests/e2e
  * @author Scott Lopez
- * @updated 2025-10-21
+ * @updated 2025-11-12
  */
 
 import { expect, test } from '@playwright/test';
 import {
+  clickAndWaitForNavigation,
   getFooter,
   getVisibleNav,
-  setDesktopView,
-  setMobileView,
+  gotoDesktop,
+  gotoMobile,
 } from './shared/helpers.js';
 
 // Root route should display nav bar and about link
@@ -29,9 +30,7 @@ test.describe('Desktop Tests', () => {
   test("should display the navigation bar and 'about' link", async ({
     page,
   }) => {
-    await setDesktopView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+    await gotoDesktop(page);
 
     const nav = await getVisibleNav(page);
 
@@ -42,9 +41,7 @@ test.describe('Desktop Tests', () => {
 
   // Root route should display the footer properly
   test('should display the footer correctly', async ({ page }) => {
-    await setDesktopView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+    await gotoDesktop(page);
 
     const footer = getFooter(page);
     await expect(footer).toBeVisible();
@@ -54,44 +51,35 @@ test.describe('Desktop Tests', () => {
   test("should render the 'about' page with footer visible", async ({
     page,
   }) => {
-    await setDesktopView(page);
-    await page.goto('/about');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+    await gotoDesktop(page, '/about');
 
     const footer = getFooter(page);
     await expect(footer).toBeVisible();
   });
 
   // Root route should have a clickable "about" link
   test("should ensure the 'about' link is clickable", async ({ page }) => {
-    await setDesktopView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+    await gotoDesktop(page);
 
     const nav = await getVisibleNav(page);
     const aboutLink = nav.getByRole('link', { name: 'about' });
-    await expect(aboutLink).toBeVisible();
-    await aboutLink.click();
 
-    // safer wait pattern with load state
-    await Promise.all([
-      page.waitForLoadState('load'),
-      page.waitForURL('**/about', { timeout: 60000 }),
-    ]);
+    await clickAndWaitForNavigation(page, aboutLink, {
+      urlPattern: /\/about/,
+      timeout: 60000,
+    });
+
     await expect(page).toHaveURL(/\/about/);
   });
 }); // End Desktop Tests
 
 // Root route should display headings properly on mobile
 test.describe('Mobile Tests', () => {
   test('should display main content correctly on mobile', async ({ page }) => {
-    await setMobileView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60000 });
+    await gotoMobile(page);
 
-    const mainHeading = page.locator('h1, h2');
-    await expect(mainHeading).toBeVisible();
+    // Ensure main headline is present on mobile
+    const h2 = page.locator('h2.index-title2');
+    await expect(h2).toContainText(/security/i);
   });
 });
-
-// cspell:ignore domcontentloaded

tests/e2e/app.spec.js

@@ -11,37 +11,36 @@ This file is part of Network Pro.
  * @description Runs Playwright E2E tests with mobile assertions.
  * @module tests/e2e
  * @author Scott Lopez
- * @updated 2025-10-21
+ * @updated 2025-11-12
  */
 
 import { expect, test } from '@playwright/test';
 import {
   clickAndWaitForNavigation,
   getFooter,
   getVisibleNav,
-  setMobileView,
+  gotoMobile,
 } from './shared/helpers.js';
 
 // ---------------------------------------------------------------------------
 // 📱 Mobile viewport smoke tests
 // ---------------------------------------------------------------------------
 
 test.describe('Mobile Tests', () => {
-  // Increase timeout for all mobile tests
   test.setTimeout(90_000);
 
+  test.beforeEach(({ browserName }) => {
+    if (browserName === 'webkit')
+      test.skip('Skipping WebKit: manual validation only');
+  });
+
   // -------------------------------------------------------------------------
   // 🧱 Test 1: Main description text
   // -------------------------------------------------------------------------
   test('should display the main description text on mobile', async ({
     page,
-    browserName,
   }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setMobileView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60_000 });
+    await gotoMobile(page);
 
     const description = page.locator(
       'div.index-title1:has-text("Locking Down Networks")',
@@ -52,60 +51,39 @@ test.describe('Mobile Tests', () => {
   // -------------------------------------------------------------------------
   // 🧱 Test 2: Main content
   // -------------------------------------------------------------------------
-  test('should display main content correctly on mobile', async ({
-    page,
-    browserName,
-  }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setMobileView(page);
-    await page.goto('/');
-    await page.waitForLoadState('domcontentloaded', { timeout: 60_000 });
+  test('should display main content correctly on mobile', async ({ page }) => {
+    await gotoMobile(page);
 
-    const mainHeading = page.locator('h1, h2');
-    await expect(mainHeading).toBeVisible();
+    const mainHeading = page.locator('h2.index-title2');
+    await expect(mainHeading).toContainText(/security/i);
   });
 
   // -------------------------------------------------------------------------
   // 🧱 Test 3: "About" link navigation
   // -------------------------------------------------------------------------
   test("should ensure the 'about' link is clickable on mobile", async ({
     page,
-    browserName,
   }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setMobileView(page);
-    await page.goto('/', { waitUntil: 'networkidle' });
+    await gotoMobile(page);
 
     const nav = await getVisibleNav(page);
-
-    // Ensure the link exists before interacting
-    await page.waitForSelector('a[href="/about"]', { timeout: 10_000 });
     const aboutLink = nav.getByRole('link', { name: 'about' });
 
-    // Use the safe navigation helper
     await clickAndWaitForNavigation(page, aboutLink, {
       urlPattern: /\/about/,
       timeout: 60_000,
     });
+
+    await expect(page).toHaveURL(/\/about/);
   });
 
   // -------------------------------------------------------------------------
   // 🧱 Test 4: Footer presence on /about
   // -------------------------------------------------------------------------
-  test('should display the footer on /about (mobile)', async ({
-    page,
-    browserName,
-  }) => {
-    if (browserName === 'webkit') test.skip();
-
-    await setMobileView(page);
-    await page.goto('/about');
+  test('should display the footer on /about (mobile)', async ({ page }) => {
+    await gotoMobile(page, '/about');
 
     const footer = getFooter(page);
     await expect(footer).toBeVisible();
   });
 });
-
-// cspell:ignore domcontentloaded networkidle