@markwharton/pwa-core 1.0.1 → 1.2.0

package/dist/auth/apiKey.d.ts

@@ -1,3 +1,4 @@
+import { Result } from '../types';
 /**
  * API Key utilities for machine-to-machine authentication
  */
@@ -16,8 +17,10 @@ export declare function extractApiKey(request: {
 export declare function hashApiKey(apiKey: string): string;
 /**
  * Validate API key against stored hash
+ *
+ * @returns Result indicating success or failure with error message
  */
-export declare function validateApiKey(apiKey: string, storedHash: string): boolean;
+export declare function validateApiKey(apiKey: string, storedHash: string): Result<void>;
 /**
  * Generate a new API key (random 32-byte hex string)
  */

package/dist/auth/apiKey.js

@@ -5,6 +5,7 @@ exports.hashApiKey = hashApiKey;
 exports.validateApiKey = validateApiKey;
 exports.generateApiKey = generateApiKey;
 const crypto_1 = require("crypto");
+const types_1 = require("../types");
 /**
  * API Key utilities for machine-to-machine authentication
  */
@@ -23,10 +24,15 @@ function hashApiKey(apiKey) {
 }
 /**
  * Validate API key against stored hash
+ *
+ * @returns Result indicating success or failure with error message
  */
 function validateApiKey(apiKey, storedHash) {
     const keyHash = hashApiKey(apiKey);
-    return keyHash === storedHash;
+    if (keyHash === storedHash) {
+        return (0, types_1.okVoid)();
+    }
+    return (0, types_1.err)('Invalid API key');
 }
 /**
  * Generate a new API key (random 32-byte hex string)

package/dist/auth/token.d.ts

@@ -1,3 +1,4 @@
+import { Result } from '../types';
 /**
  * Initialize JWT secret - call once at startup
  * Fails fast if secret is missing or too short
@@ -14,8 +15,10 @@ export declare function extractToken(authHeader: string | null): string | null;
 /**
  * Validate and decode a JWT token
  * Generic type allows project-specific payload shapes
+ *
+ * @returns Result with decoded payload or error message
  */
-export declare function validateToken<T extends object>(token: string): T | null;
+export declare function validateToken<T extends object>(token: string): Result<T>;
 /**
  * Generate a JWT token with custom payload
  */

package/dist/auth/token.js

@@ -10,6 +10,7 @@ exports.validateToken = validateToken;
 exports.generateToken = generateToken;
 exports.generateLongLivedToken = generateLongLivedToken;
 const jsonwebtoken_1 = __importDefault(require("jsonwebtoken"));
+const types_1 = require("../types");
 /**
  * JWT token utilities - works with any payload structure
  * Use BaseJwtPayload or extend it for type safety
@@ -46,17 +47,20 @@ function extractToken(authHeader) {
 /**
  * Validate and decode a JWT token
  * Generic type allows project-specific payload shapes
+ *
+ * @returns Result with decoded payload or error message
  */
 function validateToken(token) {
     try {
         const payload = jsonwebtoken_1.default.verify(token, getJwtSecret());
         if (typeof payload === 'object' && payload !== null) {
-            return payload;
+            return (0, types_1.ok)(payload);
         }
-        return null;
+        return (0, types_1.err)('Invalid token payload');
     }
-    catch {
-        return null;
+    catch (error) {
+        const message = error instanceof Error ? error.message : 'Token validation failed';
+        return (0, types_1.err)(message);
     }
 }
 /**

package/dist/client/api.d.ts

@@ -32,5 +32,6 @@ export declare function apiPatch<T>(url: string, body?: unknown): Promise<T>;
 export declare function apiDelete<T>(url: string): Promise<T>;
 /**
  * Alternative: Response wrapper style (like financial-tracker's authJsonFetch)
+ * Preserves actual HTTP status code on success
  */
 export declare function apiCallSafe<T>(url: string, options?: RequestInit): Promise<ApiResponse<T>>;

package/dist/client/api.js

@@ -101,16 +101,42 @@ async function apiDelete(url) {
 }
 /**
  * Alternative: Response wrapper style (like financial-tracker's authJsonFetch)
+ * Preserves actual HTTP status code on success
  */
 async function apiCallSafe(url, options = {}) {
-    try {
-        const data = await apiCall(url, options);
-        return { ok: true, status: 200, data };
+    const token = getToken?.();
+    const headers = {
+        'Content-Type': 'application/json',
+        ...options.headers
+    };
+    if (token) {
+        headers['Authorization'] = `Bearer ${token}`;
     }
-    catch (error) {
-        if (error instanceof apiError_1.ApiError) {
-            return { ok: false, status: error.status, error: error.message };
+    try {
+        const response = await fetch(url, {
+            ...options,
+            headers
+        });
+        if (!response.ok) {
+            if (response.status === 401 && onUnauthorized) {
+                onUnauthorized();
+            }
+            let errorMessage = 'Request failed';
+            try {
+                const errorData = (await response.json());
+                errorMessage = errorData.error || errorMessage;
+            }
+            catch {
+                // Ignore JSON parse errors
+            }
+            return { ok: false, status: response.status, error: errorMessage };
         }
+        // Handle empty responses
+        const text = await response.text();
+        const data = text ? JSON.parse(text) : undefined;
+        return { ok: true, status: response.status, data };
+    }
+    catch {
         return { ok: false, status: 0, error: 'Network error' };
     }
 }

package/dist/index.d.ts

@@ -1,3 +1,4 @@
+export * from './types';
 export * from './auth';
 export * from './http';
 export * from './storage';

package/dist/index.js

@@ -15,6 +15,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 // Main entry point - re-export everything
+__exportStar(require("./types"), exports);
 __exportStar(require("./auth"), exports);
 __exportStar(require("./http"), exports);
 __exportStar(require("./storage"), exports);

package/dist/types.d.ts

@@ -0,0 +1,35 @@
+/**
+ * Shared types for pwa-core
+ */
+/**
+ * Standard result type for operations that can fail.
+ * Used consistently across all modules (auth, push, client).
+ *
+ * @example
+ * // Success
+ * { ok: true, data: user }
+ *
+ * // Failure
+ * { ok: false, error: 'Token expired' }
+ *
+ * // With status code (HTTP/push operations)
+ * { ok: false, error: 'Subscription expired', statusCode: 410 }
+ */
+export interface Result<T> {
+    ok: boolean;
+    data?: T;
+    error?: string;
+    statusCode?: number;
+}
+/**
+ * Helper to create a success result
+ */
+export declare function ok<T>(data: T): Result<T>;
+/**
+ * Helper to create a success result with no data
+ */
+export declare function okVoid(): Result<void>;
+/**
+ * Helper to create a failure result
+ */
+export declare function err<T>(error: string, statusCode?: number): Result<T>;

package/dist/types.js

@@ -0,0 +1,28 @@
+"use strict";
+/**
+ * Shared types for pwa-core
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ok = ok;
+exports.okVoid = okVoid;
+exports.err = err;
+/**
+ * Helper to create a success result
+ */
+function ok(data) {
+    return { ok: true, data };
+}
+/**
+ * Helper to create a success result with no data
+ */
+function okVoid() {
+    return { ok: true };
+}
+/**
+ * Helper to create a failure result
+ */
+function err(error, statusCode) {
+    return statusCode !== undefined
+        ? { ok: false, error, statusCode }
+        : { ok: false, error };
+}

package/package.json

@@ -1,16 +1,36 @@
 {
   "name": "@markwharton/pwa-core",
-  "version": "1.0.1",
+  "version": "1.2.0",
   "description": "Shared patterns for Azure PWA projects",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "exports": {
     ".": "./dist/index.js",
+    "./types": "./dist/types.js",
     "./auth": "./dist/auth/index.js",
     "./http": "./dist/http/index.js",
     "./storage": "./dist/storage/index.js",
     "./client": "./dist/client/index.js"
   },
+  "typesVersions": {
+    "*": {
+      "types": [
+        "dist/types.d.ts"
+      ],
+      "auth": [
+        "dist/auth/index.d.ts"
+      ],
+      "http": [
+        "dist/http/index.d.ts"
+      ],
+      "storage": [
+        "dist/storage/index.d.ts"
+      ],
+      "client": [
+        "dist/client/index.d.ts"
+      ]
+    }
+  },
   "scripts": {
     "build": "tsc",
     "clean": "rm -rf dist"
@@ -34,7 +54,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/MarkWharton/pwa-packages.git",
+    "url": "git+https://github.com/MarkWharton/pwa-packages.git",
     "directory": "packages/core"
   },
   "author": "Mark Wharton",

package/dist/storage/helpers.d.ts

@@ -1,8 +0,0 @@
-/**
- * Generate a unique row key from an identifier string
- * Uses SHA-256 hash for consistent, URL-safe keys
- *
- * @param identifier - The string to hash (e.g., subscription endpoint, user ID)
- * @returns A 32-character hex string suitable for Azure Table Storage row keys
- */
-export declare function generateRowKey(identifier: string): string;

package/dist/storage/helpers.js

@@ -1,14 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.generateRowKey = generateRowKey;
-const crypto_1 = require("crypto");
-/**
- * Generate a unique row key from an identifier string
- * Uses SHA-256 hash for consistent, URL-safe keys
- *
- * @param identifier - The string to hash (e.g., subscription endpoint, user ID)
- * @returns A 32-character hex string suitable for Azure Table Storage row keys
- */
-function generateRowKey(identifier) {
-    return (0, crypto_1.createHash)('sha256').update(identifier).digest('hex').substring(0, 32);
-}