portapack 0.3.0 → 0.3.2

package/src/utils/slugify.ts

@@ -18,44 +18,43 @@
  * @returns A safe, lowercase slug string.
  */
 export function slugify(url: string): string {
-    if (!url || typeof url !== 'string') return 'index';
+  if (!url || typeof url !== 'string') return 'index';
 
-    let cleaned = url.trim();
-    let pathAndSearch = '';
+  let cleaned = url.trim();
+  let pathAndSearch = '';
 
-    try {
-        const urlObj = new URL(url, 'https://placeholder.base');
-        pathAndSearch = (urlObj.pathname ?? '') + (urlObj.search ?? '');
-    } catch {
-        pathAndSearch = cleaned.split('#')[0]; // Remove fragment
-    }
+  try {
+    const urlObj = new URL(url, 'https://placeholder.base');
+    pathAndSearch = (urlObj.pathname ?? '') + (urlObj.search ?? '');
+  } catch {
+    pathAndSearch = cleaned.split('#')[0]; // Remove fragment
+  }
 
-    // Decode URI components AFTER parsing from URL to handle %20 etc.
-    try {
-        cleaned = decodeURIComponent(pathAndSearch);
-    } catch (e) {
-        cleaned = pathAndSearch; // Proceed if decoding fails
-    }
+  // Decode URI components AFTER parsing from URL to handle %20 etc.
+  try {
+    cleaned = decodeURIComponent(pathAndSearch);
+  } catch (e) {
+    cleaned = pathAndSearch; // Proceed if decoding fails
+  }
 
-    cleaned = cleaned
-        // Remove common web extensions FIRST
-        .replace(/\.(html?|php|aspx?|jsp)$/i, '')
-        // Replace path separators and common separators/spaces with a hyphen
-        .replace(/[\s/?=&\\]+/g, '-') // Target spaces, /, ?, =, &, \
-        // Remove any remaining characters that are not alphanumeric, hyphen, underscore, or period
-        .replace(/[^\w._-]+/g, '')    // Allow word chars, '.', '_', '-'
-        // Collapse consecutive hyphens
-        .replace(/-+/g, '-')
-        // Trim leading/trailing hyphens
-        .replace(/^-+|-+$/g, '')
-        // Convert to lowercase
-        .toLowerCase();
+  cleaned = cleaned
+    // Remove common web extensions FIRST
+    .replace(/\.(html?|php|aspx?|jsp)$/i, '')
+    // Replace path separators and common separators/spaces with a hyphen
+    .replace(/[\s/?=&\\]+/g, '-') // Target spaces, /, ?, =, &, \
+    // Remove any remaining characters that are not alphanumeric, hyphen, underscore, or period
+    .replace(/[^\w._-]+/g, '') // Allow word chars, '.', '_', '-'
+    // Collapse consecutive hyphens
+    .replace(/-+/g, '-')
+    // Trim leading/trailing hyphens
+    .replace(/^-+|-+$/g, '')
+    // Convert to lowercase
+    .toLowerCase();
 
-    // Return 'index' if the process results in an empty string
-    return cleaned || 'index';
+  // Return 'index' if the process results in an empty string
+  return cleaned || 'index';
 }
 
-
 /**
  * Converts a URL or path string into a clean slug suitable for use as an HTML ID.
  * Note: This implementation might be very similar or identical to slugify depending on exact needs.
@@ -65,6 +64,6 @@ export function slugify(url: string): string {
  * @returns A safe, lowercase slug string (e.g. "products-item-1", "search-q-test-page-2")
  */
 export function sanitizeSlug(rawUrl: string): string {
-    // Re-use the improved slugify logic for consistency
-    return slugify(rawUrl);
-}
+  // Re-use the improved slugify logic for consistency
+  return slugify(rawUrl);
+}

package/tests/unit/cli/cli-entry.test.ts

@@ -1,4 +1,3 @@
-// tests/unit/cli/cli-entry.test.ts
 import type { CLIResult } from '../../../src/types';
 import { jest, describe, it, expect, beforeEach, afterEach } from '@jest/globals';
 
@@ -7,9 +6,9 @@ import { jest, describe, it, expect, beforeEach, afterEach } from '@jest/globals
 const mockRunCliFn = jest.fn<() => Promise<CLIResult>>();
 
 jest.mock('../../../src/cli/cli', () => ({
-    __esModule: true,
-    runCli: mockRunCliFn,
-    main: mockRunCliFn, // Mock both exports for safety
+  __esModule: true,
+  runCli: mockRunCliFn,
+  main: mockRunCliFn, // Mock both exports for safety
 }));
 
 // Spy on process methods IF startCLI interacts with them directly
@@ -25,69 +24,72 @@ let stderrSpy: jest.SpiedFunction<typeof process.stderr.write>;
 import { startCLI } from '../../../src/cli/cli-entry';
 
 describe('CLI Entry Point Function (startCLI)', () => {
-    const originalArgv = [...process.argv]; // Clone original argv
-
-    beforeEach(() => {
-        jest.clearAllMocks();
-
-        // Reset argv for each test (startCLI uses process.argv internally)
-        process.argv = ['node', '/path/to/cli-entry.js', 'default-arg'];
-
-        // Default mock implementation for the dependency (runCli)
-        mockRunCliFn.mockResolvedValue({ exitCode: 0, stdout: 'Default Success', stderr: '' });
-
-        // Spies (optional for testing startCLI directly, but good practice)
-        exitSpy = jest.spyOn(process, 'exit').mockImplementation(() => undefined as never);
-        stdoutSpy = jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
-        stderrSpy = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
-    });
-
-    afterEach(() => {
-        process.argv = originalArgv; // Restore original argv
-        // jest.restoreAllMocks(); // Usually covered by clearAllMocks
-    });
-
-    it('should call the underlying runCli function with process.argv', async () => {
-        const testArgs = ['node', 'cli-entry.js', 'input.file', '-o', 'output.file'];
-        process.argv = testArgs; // Set specific argv for this test
-
-        await startCLI(); // Execute the function exported from cli-entry
-
-        expect(mockRunCliFn).toHaveBeenCalledTimes(1);
-        // Verify runCli received the arguments startCLI got from process.argv
-        expect(mockRunCliFn).toHaveBeenCalledWith(testArgs);
-        // startCLI itself doesn't call process.exit or write, the surrounding block does
-        expect(exitSpy).not.toHaveBeenCalled();
-        expect(stdoutSpy).not.toHaveBeenCalled();
-        expect(stderrSpy).not.toHaveBeenCalled();
-    });
-
-    it('should return the result object from runCli', async () => {
-        const expectedResult: CLIResult = { exitCode: 2, stdout: 'Programmatic output', stderr: 'Some warning' };
-        mockRunCliFn.mockResolvedValue(expectedResult); // Configure mock return
-        process.argv = ['node', 'cli.js', 'input.html']; // Set argv for this call
-
-        // Call startCLI and check its return value
-        const result = await startCLI();
-
-        expect(result).toEqual(expectedResult); // Verify return value
-        expect(mockRunCliFn).toHaveBeenCalledTimes(1);
-        expect(mockRunCliFn).toHaveBeenCalledWith(process.argv); // Check args passed
-        expect(exitSpy).not.toHaveBeenCalled(); // startCLI doesn't exit
-    });
-
-    it('should return the rejected promise if runCli rejects', async () => {
-         const testArgs = ['node', 'cli.js', 'crash'];
-         process.argv = testArgs;
-         const testError = new Error('Unhandled crash');
-         mockRunCliFn.mockRejectedValue(testError); // Configure mock rejection
-
-         // Expect startCLI itself to reject when runCli rejects
-         await expect(startCLI()).rejects.toThrow(testError);
-
-         expect(mockRunCliFn).toHaveBeenCalledTimes(1);
-         expect(mockRunCliFn).toHaveBeenCalledWith(testArgs);
-         expect(exitSpy).not.toHaveBeenCalled(); // No exit from startCLI
-    });
-
-});
+  const originalArgv = [...process.argv]; // Clone original argv
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+
+    // Reset argv for each test (startCLI uses process.argv internally)
+    process.argv = ['node', '/path/to/cli-entry.js', 'default-arg'];
+
+    // Default mock implementation for the dependency (runCli)
+    mockRunCliFn.mockResolvedValue({ exitCode: 0, stdout: 'Default Success', stderr: '' });
+
+    // Spies (optional for testing startCLI directly, but good practice)
+    exitSpy = jest.spyOn(process, 'exit').mockImplementation(() => undefined as never);
+    stdoutSpy = jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    stderrSpy = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
+  });
+
+  afterEach(() => {
+    process.argv = originalArgv; // Restore original argv
+    // jest.restoreAllMocks(); // Usually covered by clearAllMocks
+  });
+
+  it('should call the underlying runCli function with process.argv', async () => {
+    const testArgs = ['node', 'cli-entry.js', 'input.file', '-o', 'output.file'];
+    process.argv = testArgs; // Set specific argv for this test
+
+    await startCLI(); // Execute the function exported from cli-entry
+
+    expect(mockRunCliFn).toHaveBeenCalledTimes(1);
+    // Verify runCli received the arguments startCLI got from process.argv
+    expect(mockRunCliFn).toHaveBeenCalledWith(testArgs);
+    // startCLI itself doesn't call process.exit or write, the surrounding block does
+    expect(exitSpy).not.toHaveBeenCalled();
+    expect(stdoutSpy).not.toHaveBeenCalled();
+    expect(stderrSpy).not.toHaveBeenCalled();
+  });
+
+  it('should return the result object from runCli', async () => {
+    const expectedResult: CLIResult = {
+      exitCode: 2,
+      stdout: 'Programmatic output',
+      stderr: 'Some warning',
+    };
+    mockRunCliFn.mockResolvedValue(expectedResult); // Configure mock return
+    process.argv = ['node', 'cli.js', 'input.html']; // Set argv for this call
+
+    // Call startCLI and check its return value
+    const result = await startCLI();
+
+    expect(result).toEqual(expectedResult); // Verify return value
+    expect(mockRunCliFn).toHaveBeenCalledTimes(1);
+    expect(mockRunCliFn).toHaveBeenCalledWith(process.argv); // Check args passed
+    expect(exitSpy).not.toHaveBeenCalled(); // startCLI doesn't exit
+  });
+
+  it('should return the rejected promise if runCli rejects', async () => {
+    const testArgs = ['node', 'cli.js', 'crash'];
+    process.argv = testArgs;
+    const testError = new Error('Unhandled crash');
+    mockRunCliFn.mockRejectedValue(testError); // Configure mock rejection
+
+    // Expect startCLI itself to reject when runCli rejects
+    await expect(startCLI()).rejects.toThrow(testError);
+
+    expect(mockRunCliFn).toHaveBeenCalledTimes(1);
+    expect(mockRunCliFn).toHaveBeenCalledWith(testArgs);
+    expect(exitSpy).not.toHaveBeenCalled(); // No exit from startCLI
+  });
+});