@constructive-io/knative-job-fn 0.2.4 → 0.2.6

package/CHANGELOG.md

@@ -3,10 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [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
+
 ## 0.2.4 (2025-12-18)
 
 **Note:** Version bump only for package @constructive-io/knative-job-fn
 
-## [0.2.3](https://github.com/constructive-io/jobs/compare/@launchql/knative-job-fn@0.2.2...@launchql/knative-job-fn@0.2.3) (2025-12-17)
+## [0.2.3](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-fn@0.2.2...@constructive-io/knative-job-fn@0.2.3) (2025-12-17)
 
-**Note:** Version bump only for package @launchql/knative-job-fn
+**Note:** Version bump only for package @constructive-io/knative-job-fn

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/knative-job-fn",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "knative job fn",
   "author": "Constructive <developers@constructive.io>",
   "homepage": "https://github.com/constructive-io/jobs/tree/master/packages/knative-job-fn#readme",
@@ -18,7 +18,7 @@
     "url": "https://github.com/constructive-io/jobs"
   },
   "scripts": {
-    "test": "jest",
+    "test": "jest --passWithNoTests",
     "test:watch": "jest --watch",
     "test:debug": "node --inspect node_modules/.bin/jest --runInBand",
     "build": "tsc -p tsconfig.json",
@@ -31,5 +31,5 @@
     "body-parser": "1.19.0",
     "express": "4.17.1"
   },
-  "gitHead": "86d74dc4fce9051df0d2b5bcc163607aba42f009"
+  "gitHead": "fc182ff9e4c4745a3e86eda6d58e3b0061f36564"
 }

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);