@patcherly/nodejs-connector 2.2.3 → 2.2.5

package/README.md

@@ -21,6 +21,20 @@ npx patcherly login
 patcherly login
 ```
 
+## Test ingest
+
+After pairing, verify the pipeline without waiting for a real exception. In your [Patcherly dashboard](https://app.patcherly.com/targets), open **Targets → your target → Test Mode** (30-minute window per target), then run:
+
+```bash
+npx patcherly send-test
+```
+
+The sample is tagged `is_test_sample` — it does not affect metrics or notifications. See [Verify ingest with send-test](https://help.patcherly.com/connectors/nodejs/#verify-ingest-end-to-end-with-patcherly-send-test) in the help center.
+
+## Security
+
+OAuth pairing and per-token **HMAC signing**; fix payloads verified before apply. See [Connectors overview](https://help.patcherly.com/connectors/overview/), [Prompt injection protection](https://help.patcherly.com/security/prompt-injection-protection.md), and [Custom sanitiser patterns](https://help.patcherly.com/security/custom-sanitizer-patterns.md). Node.js-specific detail: [Node.js connector — security](https://help.patcherly.com/connectors/nodejs/#hmac-signing).
+
 ## Documentation
 
 - [Node.js connector guide](https://help.patcherly.com/connectors/nodejs/)

package/lib/ingest_severity.js

@@ -1,6 +1,6 @@
 /**
- * AUTO-GENERATED from config/settings_schema.yaml error_type_configurations — do not edit by hand.
- * Shared log-line → error_type → severity inference for connector ingest payloads.
+ * AUTO-GENERATED from config/settings_schema.yaml + log_ingest_skip_patterns.yaml — do not edit by hand.
+ * Shared log-line → error_type → severity inference and pre-ingest noise filtering.
  */
 'use strict';
 
@@ -22,6 +22,58 @@ const DEFAULT_ERROR_TYPE_SEVERITIES = {
   "warning": "Low",
 };
 
+const LOG_INGEST_KEEP_PATTERNS = [
+  "PHP\\s+Fatal",
+  "PHP\\s+Parse\\s+error",
+  "PHP\\s+Recoverable\\s+fatal",
+  "^Fatal error:",
+  "Maximum execution time|Allowed memory size",
+  "Uncaught\\s+\\S*(Error|Exception)",
+  "^Traceback\\s",
+  "^\\w+(Error|Exception):",
+  "^Exception:",
+  "^Error:\\s",
+  "Unhandled\\s+(Promise\\s+)?Rejection",
+  "uncaughtException|uncaught exception",
+  "Assertion failed",
+  "File\\s+[\"']",
+  "^\\s*#\\d+\\s+",
+  "^\\s+at\\s+",
+  "\"level\"\\s*:\\s*\"(error|fatal|critical)\"",
+  "\"level\"\\s*:\\s*50\\b",
+  "\"severity\"\\s*:\\s*\"(ERROR|CRITICAL|FATAL)\"",
+].map((p) => new RegExp(p, 'i'));
+const LOG_INGEST_SKIP_PHP_PREFIX = new RegExp("^PHP\\s+(Notice|Deprecated|Warning|Strict\\s+standards|Info)\\s*:", 'i');
+const LOG_INGEST_SKIP_PATTERNS = [
+  "^\\(node:\\d+\\)\\s+\\[DEP\\d+\\]",
+  "DeprecationWarning:",
+  "^UserWarning:",
+  "^\\[info\\]",
+  "^INFO:",
+  "^DEBUG:",
+].map((p) => new RegExp(p, 'i'));
+const LOG_INGEST_SKIP_SUBSTRINGS = ["auditor:scan", "\"kind\":\"installed-plugin\""];
+const LOG_INGEST_FAILURE_SIGNAL = new RegExp("\\b(error|exception|traceback|fatal|critical|panic|failed|failure|rejection|errno|segfault)\\b|^\\w+(Error|Exception):", 'i');
+
+function shouldSkipLogLineForIngest(logLine) {
+  const line = String(logLine || '').trim();
+  if (!line) return true;
+  for (const re of LOG_INGEST_KEEP_PATTERNS) {
+    if (re.test(line)) return false;
+  }
+  if (LOG_INGEST_SKIP_PHP_PREFIX && LOG_INGEST_SKIP_PHP_PREFIX.test(line)) return true;
+  for (const re of LOG_INGEST_SKIP_PATTERNS) {
+    if (re.test(line)) return true;
+  }
+  const lower = line.toLowerCase();
+  for (const sub of LOG_INGEST_SKIP_SUBSTRINGS) {
+    if (sub && lower.includes(String(sub).toLowerCase())) return true;
+  }
+  if (!LOG_INGEST_FAILURE_SIGNAL.test(line)) return true;
+  return false;
+}
+
+
 function inferErrorTypeFromLogLine(logLine) {
   const line = String(logLine || '').toLowerCase();
   if (line.includes('parse error')) return 'parse';
@@ -48,6 +100,7 @@ function buildIngestSeverityFields(logLine) {
 
 module.exports = {
   DEFAULT_ERROR_TYPE_SEVERITIES,
+  shouldSkipLogLineForIngest,
   inferErrorTypeFromLogLine,
   severityForErrorType,
   buildIngestSeverityFields,

package/node_agent.js

@@ -20,7 +20,7 @@ const path = require('path');
 const { execFile } = require('child_process');
 const util = require('util');
 const execFileAsync = util.promisify(execFile);
-const { buildIngestSeverityFields } = require('./lib/ingest_severity.js');
+const { buildIngestSeverityFields, shouldSkipLogLineForIngest } = require('./lib/ingest_severity.js');
 const apiPaths = require('./lib/api_paths.js');
 const { namedPaths, appPath } = apiPaths;
 
@@ -193,7 +193,7 @@ const { DEFAULT_API_URL, getConfiguredServerUrl, isExplicitApiBaseConfigured } =
  * update-release-latest.yml workflow so the value baked into every released tarball matches
  * the GitHub release tag. Reported to the API on every context upload.
  */
-const PATCHERLY_CONNECTOR_VERSION = '2.2.3';
+const PATCHERLY_CONNECTOR_VERSION = '2.2.5';
 let CENTRAL_SERVER_URL = getConfiguredServerUrl();
 const IDS_PATH = process.env.PATCHERLY_IDS_PATH || path.join(__dirname, 'patcherly_ids.json');
 const QUEUE_PATH = process.env.PATCHERLY_QUEUE_PATH || path.join(__dirname, 'patcherly_queue.jsonl');
@@ -1004,7 +1004,7 @@ function extractErrorContext(logData) {
     const lines = logData.split('\n');
     const events = [];
     let current = [];
-    const startOrCont = /^(Traceback\s|File\s+["']|Exception:|Error:\s|PHP\s+Fatal|PHP\s+Warning|^\s+at\s+|\s*#\d+\s+)/i;
+    const startOrCont = /^(Traceback\s|File\s+["']|Exception:|Error:\s|PHP\s+Fatal|^\s+at\s+|\s*#\d+\s+)/i;
     const errorWord = /\b(error|exception|traceback|fatal)\b/i;
     // Python exception type line (e.g. "ValueError: bad") — treat as continuation when in a block
     const pythonExceptionLine = /^\w+(?:Error|Exception):\s/i;
@@ -1033,7 +1033,10 @@ function extractErrorContext(logData) {
     }
     flush();
     if (events.length === 0) {
-        const errorLines = lines.filter(l => /error/i.test(l));
+        const errorLines = lines.filter(l =>
+            /\b(error|exception|traceback|fatal|critical|failed|failure|rejection)\b/i.test(l)
+            || /^\s*\w+(Error|Exception):/i.test(l)
+        );
         if (errorLines.length) events.push(errorLines.join('\n'));
     }
     return events;
@@ -1261,6 +1264,10 @@ async function processError(errorContext) {
         // ingest (errorContext is string or object; server expects log_line string)
         const logLine = typeof errorContext === 'string' ? errorContext : JSON.stringify(errorContext);
         const logLineSanitized = sanitizeLogLineForIngest(logLine);
+        if (shouldSkipLogLineForIngest(logLineSanitized)) {
+            console.log('Non-error log noise skipped.');
+            return;
+        }
         const payload = { log_line: logLineSanitized, source: 'log_monitor' };
         Object.assign(payload, buildIngestSeverityFields(logLineSanitized));
         if (TENANT_ID && TARGET_ID){ payload.tenant_id = TENANT_ID; payload.target_id = TARGET_ID; }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patcherly/nodejs-connector",
-  "version": "2.2.3",
+  "version": "2.2.5",
   "description": "Patcherly detects bugs in real time in your app.  It creates a customized AI patch, and once you approve it, backs up your code, fixes it & tests the patch for you. If anything is off, it rolls back the changes automatically, or you can always roll it back in a click. Website: https://patcherly.com · Docs: https://help.patcherly.com/connectors/nodejs/",
   "license": "MIT",
   "homepage": "https://patcherly.com",