@constructive-io/knative-job-fn 0.2.5 → 0.2.7

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.2.7](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-fn@0.2.6...@constructive-io/knative-job-fn@0.2.7) (2025-12-25)
+
+**Note:** Version bump only for package @constructive-io/knative-job-fn
+
+## [0.2.6](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-fn@0.2.5...@constructive-io/knative-job-fn@0.2.6) (2025-12-24)
+
+**Note:** Version bump only for package @constructive-io/knative-job-fn
+
 ## [0.2.5](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-fn@0.2.4...@constructive-io/knative-job-fn@0.2.5) (2025-12-19)
 
 **Note:** Version bump only for package @constructive-io/knative-job-fn

package/dist/index.js

@@ -8,8 +8,46 @@ const body_parser_1 = __importDefault(require("body-parser"));
 const node_http_1 = __importDefault(require("node:http"));
 const node_https_1 = __importDefault(require("node:https"));
 const node_url_1 = require("node:url");
+function getHeaders(req) {
+    return {
+        'x-worker-id': req.get('X-Worker-Id'),
+        'x-job-id': req.get('X-Job-Id'),
+        'x-database-id': req.get('X-Database-Id'),
+        'x-callback-url': req.get('X-Callback-Url')
+    };
+}
 const app = (0, express_1.default)();
 app.use(body_parser_1.default.json());
+// Basic request logging for all incoming job invocations.
+app.use((req, res, next) => {
+    try {
+        // Log only the headers we care about plus a shallow body snapshot
+        const headers = getHeaders(req);
+        let body;
+        if (req.body && typeof req.body === 'object') {
+            // Only log top-level keys to avoid exposing sensitive body contents.
+            body = { keys: Object.keys(req.body) };
+        }
+        else if (typeof req.body === 'string') {
+            // For string bodies, log only the length.
+            body = { length: req.body.length };
+        }
+        else {
+            body = undefined;
+        }
+        // eslint-disable-next-line no-console
+        console.log('[knative-job-fn] Incoming job request', {
+            method: req.method,
+            path: req.originalUrl || req.url,
+            headers,
+            body
+        });
+    }
+    catch {
+        // best-effort logging; never block the request
+    }
+    next();
+});
 // Echo job headers back on responses for debugging/traceability.
 app.use((req, res, next) => {
     res.set({
@@ -122,6 +160,13 @@ app.use((req, res, next) => {
             if (res.locals.jobCallbackSent)
                 return;
             res.locals.jobCallbackSent = true;
+            // eslint-disable-next-line no-console
+            console.log('[knative-job-fn] Function completed', {
+                workerId: ctx.workerId,
+                jobId: ctx.jobId,
+                databaseId: ctx.databaseId,
+                statusCode: res.statusCode
+            });
             void sendJobCallback(ctx, 'success');
         });
     }
@@ -153,6 +198,33 @@ exports.default = {
                 // eslint-disable-next-line no-console
                 console.error('[knative-job-fn] Failed to send error callback', err);
             }
+            // Log the full error context for debugging.
+            try {
+                const headers = getHeaders(req);
+                // Some error types (e.g. GraphQL ClientError) expose response info.
+                const errorDetails = {
+                    message: error?.message,
+                    name: error?.name,
+                    stack: error?.stack
+                };
+                if (error?.response) {
+                    errorDetails.response = {
+                        status: error.response.status,
+                        statusText: error.response.statusText,
+                        errors: error.response.errors,
+                        data: error.response.data
+                    };
+                }
+                // eslint-disable-next-line no-console
+                console.error('[knative-job-fn] Function error', {
+                    headers,
+                    path: req.originalUrl || req.url,
+                    error: errorDetails
+                });
+            }
+            catch {
+                // never throw from the error logger
+            }
             res.status(200).json({ message: error.message });
         });
         app.listen(port, cb);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/knative-job-fn",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "knative job fn",
   "author": "Constructive <developers@constructive.io>",
   "homepage": "https://github.com/constructive-io/jobs/tree/master/packages/knative-job-fn#readme",
@@ -29,7 +29,7 @@
   },
   "dependencies": {
     "body-parser": "1.19.0",
-    "express": "4.17.1"
+    "express": "5.2.1"
   },
-  "gitHead": "22cfe32e994e26a6490e04e28bab26d1e7e6345c"
+  "gitHead": "976cc9e0e09201c7df40518a1797f4178fc21c2c"
 }

package/src/index.ts

@@ -13,10 +13,49 @@ type JobContext = {
   databaseId: string | undefined;
 };
 
+function getHeaders(req: any) {
+  return {
+    'x-worker-id': req.get('X-Worker-Id'),
+    'x-job-id': req.get('X-Job-Id'),
+    'x-database-id': req.get('X-Database-Id'),
+    'x-callback-url': req.get('X-Callback-Url')
+  };
+}
+
 const app: any = express();
 
 app.use(bodyParser.json());
 
+// Basic request logging for all incoming job invocations.
+app.use((req: any, res: any, next: any) => {
+  try {
+    // Log only the headers we care about plus a shallow body snapshot
+    const headers = getHeaders(req);
+
+    let body: any;
+    if (req.body && typeof req.body === 'object') {
+      // Only log top-level keys to avoid exposing sensitive body contents.
+      body = { keys: Object.keys(req.body) };
+    } else if (typeof req.body === 'string') {
+      // For string bodies, log only the length.
+      body = { length: req.body.length };
+    } else {
+      body = undefined;
+    }
+
+    // eslint-disable-next-line no-console
+    console.log('[knative-job-fn] Incoming job request', {
+      method: req.method,
+      path: req.originalUrl || req.url,
+      headers,
+      body
+    });
+  } catch {
+    // best-effort logging; never block the request
+  }
+  next();
+});
+
 // Echo job headers back on responses for debugging/traceability.
 app.use((req: any, res: any, next: any) => {
   res.set({
@@ -151,6 +190,13 @@ app.use((req: any, res: any, next: any) => {
       // If an error handler already sent a callback, skip.
       if (res.locals.jobCallbackSent) return;
       res.locals.jobCallbackSent = true;
+      // eslint-disable-next-line no-console
+      console.log('[knative-job-fn] Function completed', {
+        workerId: ctx.workerId,
+        jobId: ctx.jobId,
+        databaseId: ctx.databaseId,
+        statusCode: res.statusCode
+      });
       void sendJobCallback(ctx, 'success');
     });
   }
@@ -185,6 +231,36 @@ export default {
         console.error('[knative-job-fn] Failed to send error callback', err);
       }
 
+      // Log the full error context for debugging.
+      try {
+        const headers = getHeaders(req);
+
+        // Some error types (e.g. GraphQL ClientError) expose response info.
+        const errorDetails: any = {
+          message: error?.message,
+          name: error?.name,
+          stack: error?.stack
+        };
+
+        if (error?.response) {
+          errorDetails.response = {
+            status: error.response.status,
+            statusText: error.response.statusText,
+            errors: error.response.errors,
+            data: error.response.data
+          };
+        }
+
+        // eslint-disable-next-line no-console
+        console.error('[knative-job-fn] Function error', {
+          headers,
+          path: req.originalUrl || req.url,
+          error: errorDetails
+        });
+      } catch {
+        // never throw from the error logger
+      }
+
       res.status(200).json({ message: error.message });
     });
     app.listen(port, cb);