@elisra-devops/docgen-data-provider 1.140.0 → 1.141.0

package/bin/utils/logger.js.map

@@ -1 +1 @@
-{"version":3,"file":"logger.js","sourceRoot":"","sources":["../../src/utils/logger.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;;AACb,mCAAmC;AAEnC,IAAI,MAAsB,CAAC;AAE3B,MAAM,SAAS,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,GAAG,IAAI,CAAC,SAAS,MAAM,IAAI,CAAC,KAAK,KAAK,IAAI,CAAC,OAAO,EAAE,CAAC,CAAC;AACxG,uCAAuC;AACvC,uDAAuD;AAEvD,oCAAoC;AACpC,0CAA0C;AAC1C,sBAAsB;AACtB,oBAAoB;AACpB,sCAAsC;AACtC,4DAA4D;AAC5D,0BAA0B;AAC1B,6BAA6B;AAC7B,YAAY;AACZ,sCAAsC;AACtC,yDAAyD;AACzD,6BAA6B;AAC7B,YAAY;AACZ,+EAA+E;AAC/E,SAAS;AACT,QAAQ;AACR,WAAW;AACX,MAAM,GAAG,OAAO,CAAC,YAAY,CAAC;IAC5B,MAAM,EAAE,OAAO,CAAC,MAAM,CAAC,SAAS,EAAE;IAClC,KAAK,EAAE,OAAO;IACd,UAAU,EAAE;QACV,IAAI,OAAO,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC;QACrE,6DAA6D;KAC9D;CACF,CAAC,CAAC;AACH,IAAI;AAEJ,kBAAe,MAAM,CAAC"}
+{"version":3,"file":"logger.js","sourceRoot":"","sources":["../../src/utils/logger.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;;;AACb,mCAAmC;AACnC,yBAAyB;AACzB,6BAA6B;AAC7B,+CAA+C;AAC/C,6CAA+C;AAC/C,uCAAwD;AACxD,IAAI,MAAsB,CAAC;AAE3B,4FAA4F;AAC5F,2FAA2F;AAC3F,gCAAgC;AACnB,QAAA,cAAc,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;IACpD,MAAM,KAAK,GAAG,4BAAe,CAAC,QAAQ,EAAE,CAAC;IACzC,IAAI,KAAK,aAAL,KAAK,uBAAL,KAAK,CAAE,KAAK;QAAG,IAAgC,CAAC,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC;IACxE,uDAAuD;IACvD,IAAI,KAAK,aAAL,KAAK,uBAAL,KAAK,CAAE,OAAO;QAAG,IAAgC,CAAC,OAAO,GAAG,KAAK,CAAC,OAAO,CAAC;IAC9E,wFAAwF;IACxF,IAAI,KAAK,aAAL,KAAK,uBAAL,KAAK,CAAE,OAAO;QAAG,IAAgC,CAAC,OAAO,GAAG,KAAK,CAAC,OAAO,CAAC;IAC9E,OAAO,IAAI,CAAC;AACd,CAAC,CAAC,CAAC;AAEH,yFAAyF;AACzF,6EAA6E;AAC7E,sFAAsF;AACtF,oFAAoF;AACpF,yFAAyF;AACzF,wDAAwD;AACxD,6FAA6F;AAC7F,4FAA4F;AAC5F,8FAA8F;AAC9F,MAAM,aAAa,GAAG,kFAAkF,CAAC;AAEzG,0FAA0F;AAC1F,4FAA4F;AAC5F,8FAA8F;AAC9F,6FAA6F;AAC7F,6FAA6F;AAC7F,4EAA4E;AAC5E,SAAS,QAAQ,CAAC,GAA4B,EAAE,GAAW;IACzD,IAAI,CAAC;QACH,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;IACvC,CAAC;IAAC,WAAM,CAAC;QACP,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,CAAC;IACvB,CAAC;AACH,CAAC;AACD,SAAS,WAAW,CAAC,KAAc,EAAE,KAAK,GAAG,CAAC,EAAE,OAAO,IAAI,OAAO,EAAU;IAC1E,IAAI,KAAK,GAAG,CAAC,IAAI,KAAK,KAAK,IAAI,IAAI,OAAO,KAAK,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC3E,IAAI,IAAI,CAAC,GAAG,CAAC,KAAe,CAAC;QAAE,OAAO,YAAY,CAAC;IACnD,IAAI,CAAC,GAAG,CAAC,KAAe,CAAC,CAAC;IAC1B,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,EAAE,KAAK,GAAG,CAAC,EAAE,IAAI,CAAC,CAAC,CAAC;IACnF,MAAM,GAAG,GAA4B,EAAE,CAAC;IACxC,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,KAAgC,CAAC,EAAE,CAAC;QAChE,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAgC,EAAE,GAAG,CAAC,CAAC;QAC7D,IAAI,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC;YACb,GAAG,CAAC,GAAG,CAAC,GAAG,uBAAuB,CAAC;YACnC,SAAS;QACX,CAAC;QACD,GAAG,CAAC,GAAG,CAAC,GAAG,aAAa,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,WAAW,CAAC,IAAI,CAAC,KAAK,EAAE,KAAK,GAAG,CAAC,EAAE,IAAI,CAAC,CAAC;IAC/F,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AACY,QAAA,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;IAC5C,MAAM,MAAM,GAAG,IAA+B,CAAC;IAC/C,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;QACtC,2FAA2F;QAC3F,0FAA0F;QAC1F,2FAA2F;QAC3F,2FAA2F;QAC3F,0FAA0F;QAC1F,8EAA8E;QAC9E,IAAI,GAAG,KAAK,OAAO,IAAI,GAAG,KAAK,WAAW;YAAE,SAAS;QACrD,MAAM,IAAI,GAAG,QAAQ,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QACnC,MAAM,KAAK,GAAG,CAAC,IAAI,CAAC,EAAE;YACpB,CAAC,CAAC,uBAAuB;YACzB,CAAC,CAAC,2FAA2F;gBAC3F,wFAAwF;gBACxF,aAAa,CAAC,IAAI,CAAC,GAAG,CAAC;oBACvB,CAAC,CAAC,YAAY;oBACd,CAAC,CAAC,WAAW,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC9B,IAAI,CAAC;YACH,MAAM,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC;QACtB,CAAC;QAAC,WAAM,CAAC;YACP,wFAAwF;YACxF,8EAA8E;YAC9E,MAAM,CAAC,cAAc,CAAC,MAAM,EAAE,GAAG,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;QACtG,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC,CAAC,CAAC;AAEH,2FAA2F;AAC3F,2FAA2F;AAC3F,mDAAmD;AACnD,SAAS,cAAc;IACrB,MAAM,UAAU,GAAG;QACjB,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,iBAAiB,CAAC;QAC1C,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,oBAAoB,CAAC;QAC7C,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,cAAc,CAAC;KAC5C,CAAC;IACF,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;QACnC,IAAI,CAAC;YACH,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC;gBAAE,SAAS;YACxC,OAAO,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,OAAO,IAAI,SAAS,CAAC;QAC7E,CAAC;QAAC,WAAM,CAAC;YACP,qBAAqB;QACvB,CAAC;IACH,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,0FAA0F;AAC1F,4FAA4F;AAC5F,6FAA6F;AAC7F,gFAAgF;AAChF,MAAM,iBAAiB,GAAG,CAAC,KAAc,EAAU,EAAE,CACnD,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AAE/D,MAAM,UAAU,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,GAAG,IAAI,CAAC,SAAS,MAAM,IAAI,CAAC,KAAK,KAAK,iBAAiB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;AAE5H,uFAAuF;AACvF,MAAM,eAAe,GAAG,IAAI,CAAC;AAC7B,MAAM,aAAa,GAAG,IAAI,CAAC;AAC3B,SAAS,KAAK,CAAC,KAAc,EAAE,GAAW;IACxC,IAAI,OAAO,KAAK,KAAK,QAAQ;QAAE,OAAO,SAAS,CAAC;IAChD,OAAO,KAAK,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC;AAC1D,CAAC;AAED,0FAA0F;AAC1F,2FAA2F;AAC3F,yFAAyF;AACzF,2FAA2F;AAC3F,gFAAgF;AAChF,wFAAwF;AACxF,2FAA2F;AAC3F,wFAAwF;AACxF,iEAAiE;AACjE,MAAM,2BAA2B,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,2BAA2B,IAAI,MAAM,CAAC,CAAC,WAAW,EAAE,KAAK,OAAO,CAAC;AAClH,2FAA2F;AAC3F,0FAA0F;AAC1F,4BAA4B;AAC5B,MAAa,oBAAqB,SAAQ,SAAS;IACjD,GAAG,CAAC,IAA6B,EAAE,QAAoB;;QACrD,YAAY,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC,CAAC;QAC9C,IAAI,CAAC;YACH,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC;YACzB,MAAM,aAAa,GAAG,KAAK,KAAK,MAAM,IAAI,KAAK,KAAK,OAAO,CAAC;YAC5D,uFAAuF;YACvF,wFAAwF;YACxF,qFAAqF;YACrF,qFAAqF;YACrF,sDAAsD;YACtD,MAAM,WAAW,GAAG,MAAA,4BAAe,CAAC,QAAQ,EAAE,0CAAE,WAAW,CAAC;YAC5D,MAAM,qBAAqB,GACzB,CAAC,KAAK,KAAK,OAAO,IAAI,KAAK,KAAK,MAAM,CAAC,IAAI,CAAC,WAAW,KAAK,SAAS,IAAI,WAAW,KAAK,mBAAmB,CAAC,CAAC;YAChH,IAAI,2BAA2B,IAAI,CAAC,aAAa,IAAI,qBAAqB,CAAC,EAAE,CAAC;gBAC5E,mFAAmF;gBACnF,qFAAqF;gBACrF,sFAAsF;gBACtF,mFAAmF;gBACnF,yEAAyE;gBACzE,uFAAuF;gBACvF,MAAM,MAAM,GAAG,OAAO,IAAI,CAAC,KAAK,KAAK,QAAQ,CAAC;gBAC9C,MAAM,OAAO,GAAG,MAAA,KAAK,CAAC,IAAI,CAAC,OAAO,EAAE,eAAe,CAAC,mCAAI,EAAE,CAAC;gBAC3D,MAAM,KAAK,GAAoB;oBAC7B,EAAE,EAAE,OAAO,IAAI,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;oBAClF,KAAK,EAAE,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC;oBACzB,OAAO,EAAE,MAAM,CAAC,MAAA,IAAI,CAAC,OAAO,mCAAI,qCAAqC,CAAC;oBACtE,OAAO,EAAE,MAAM,CAAC,MAAA,IAAI,CAAC,OAAO,mCAAI,SAAS,CAAC;oBAC1C,KAAK,EAAE,OAAO,IAAI,CAAC,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS;oBAC9D,OAAO,EAAE,OAAO,IAAI,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS;oBACpE,IAAI,EAAE,OAAO,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS;oBAC3D,kBAAkB,EAAE,OAAO,IAAI,CAAC,kBAAkB,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC,CAAC,SAAS;oBACrG,mBAAmB,EAAE,OAAO,IAAI,CAAC,mBAAmB,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,CAAC,SAAS;oBACxG,OAAO,EAAE,OAAO,IAAI,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS;oBACpE,MAAM,EAAE,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS;oBACjE,OAAO;oBACP,GAAG,EAAE,MAAM;wBACT,CAAC,CAAC;4BACE,OAAO;4BACP,IAAI,EAAE,OAAO,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS;4BAC3D,KAAK,EAAE,KAAK,CAAC,IAAI,CAAC,KAAK,EAAE,aAAa,CAAC;yBACxC;wBACH,CAAC,CAAC,SAAS;oBACb,aAAa,EAAE,CAAC,aAAa,IAAI,WAAW,KAAK,mBAAmB,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS;iBACxF,CAAC;gBACF,MAAA,IAAA,oBAAU,GAAE,0CAAE,IAAI,CAAC,KAAK,CAAC,CAAC;YAC5B,CAAC;QACH,CAAC;QAAC,OAAO,CAAC,EAAE,CAAC;YACX,sCAAsC;YACtC,OAAO,CAAC,KAAK,CAAC,2CAA2C,EAAE,CAAC,CAAC,CAAC;QAChE,CAAC;QACD,QAAQ,EAAE,CAAC;IACb,CAAC;CACF;AArDD,oDAqDC;AAED,yFAAyF;AACzF,yFAAyF;AACzF,wDAAwD;AACxD,MAAM,OAAO,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,UAAU,IAAI,MAAM,CAAC,CAAC,WAAW,EAAE,KAAK,MAAM,CAAC;AAE5E,0FAA0F;AAC1F,0FAA0F;AAC1F,4FAA4F;AAC5F,wFAAwF;AACxF,6FAA6F;AAC7F,0FAA0F;AAC1F,+EAA+E;AAC/E,MAAM,iBAAiB,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,IAAI,MAAM,CAAC;AAE1D,MAAM,GAAG,OAAO,CAAC,YAAY,CAAC;IAC5B,KAAK,EAAE,OAAO;IACd,WAAW,EAAE,EAAE,OAAO,EAAE,qCAAqC,EAAE,OAAO,EAAE,cAAc,EAAE,EAAE;IAC1F,MAAM,EAAE,OAAO;QACb,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CACpB,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,EACtC,OAAO,CAAC,MAAM,CAAC,SAAS,EAAE,EAC1B,IAAA,sBAAc,GAAE,EAChB,IAAA,cAAM,GAAE,EACR,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,EACtB,OAAO,CAAC,MAAM,CAAC,IAAI,EAAE,CACtB;QACH,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CACpB,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,EACtC,OAAO,CAAC,MAAM,CAAC,SAAS,EAAE,EAC1B,IAAA,sBAAc,GAAE,EAChB,IAAA,cAAM,GAAE,EACR,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,EACtB,UAAU,CACX;IACL,0FAA0F;IAC1F,6FAA6F;IAC7F,mEAAmE;IACnE,UAAU,EAAE,CAAC,IAAI,OAAO,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,KAAK,EAAE,iBAAiB,EAAE,CAAC,EAAE,IAAI,oBAAoB,EAAE,CAAC;CACvG,CAAC,CAAC;AAEH,kBAAe,MAAM,CAAC"}

package/bin/utils/runContext.d.ts

@@ -0,0 +1,8 @@
+import { AsyncLocalStorage } from 'async_hooks';
+export interface RunContext {
+    runId: string;
+    captureMode?: 'verbose' | 'retain-on-failure';
+    docType?: string;
+    project?: string;
+}
+export declare const runContextStore: AsyncLocalStorage<RunContext>;

package/bin/utils/runContext.js

@@ -0,0 +1,12 @@
+'use strict';
+var _a;
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.runContextStore = void 0;
+const async_hooks_1 = require("async_hooks");
+// Symbol.for uses the global symbol registry, so every duplicated copy of this file across
+// the DocGen packages — hoisted or nested at any depth by npm — converges on the same
+// AsyncLocalStorage instance. Keying by module identity instead would silently split into
+// two stores and runId would go missing with no visible error.
+const KEY = Symbol.for('elisradevops.docgen.runContext');
+exports.runContextStore = (_a = globalThis[KEY]) !== null && _a !== void 0 ? _a : (globalThis[KEY] = new async_hooks_1.AsyncLocalStorage());
+//# sourceMappingURL=runContext.js.map

package/bin/utils/runContext.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"runContext.js","sourceRoot":"","sources":["../../src/utils/runContext.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;;;;AACb,6CAAgD;AAehD,2FAA2F;AAC3F,sFAAsF;AACtF,0FAA0F;AAC1F,+DAA+D;AAC/D,MAAM,GAAG,GAAG,MAAM,CAAC,GAAG,CAAC,gCAAgC,CAAC,CAAC;AAE5C,QAAA,eAAe,GAC1B,MAAE,UAAsC,CAAC,GAAG,CAA+C,mCAC3F,CAAE,UAAsC,CAAC,GAAG,CAAC,GAAG,IAAI,+BAAiB,EAAc,CAAC,CAAC"}

package/eslint.config.mjs

@@ -0,0 +1,40 @@
+import tseslint from 'typescript-eslint';
+import globals from 'globals';
+
+// Phase 4 regression guard: bans the two log-record-corrupting patterns the observability plan
+// found and fixed by hand across all four backend repos. Deliberately does NOT extend
+// js.configs.recommended/tseslint's recommended rule sets — this repo has never had a linter,
+// and pulling in a general rule set now would surface a large, unrelated pre-existing backlog
+// (unused vars, etc.) as new lint failures. Scope is exactly the one regression this guards
+// against. Keep this file identical in shape across docgen-api-gate / docgen-content-control /
+// docgen-data-provider-package / docgen-dg-skins-package — see each repo's CLAUDE.md "keep
+// these files in sync" note.
+export default tseslint.config(
+  {
+    ignores: ['bin/**', 'coverage/**', '**/test/**', '**/tests/**'],
+  },
+  {
+    files: ['src/**/*.ts'],
+    extends: [tseslint.configs.base],
+    languageOptions: {
+      globals: globals.node,
+    },
+    rules: {
+      'no-restricted-syntax': [
+        'error',
+        {
+          selector:
+            "CallExpression[callee.object.name='logger'] > :matches(TemplateLiteral, BinaryExpression[operator='+']).arguments:first-child MemberExpression[property.name='stack']",
+          message:
+            'Do not embed .stack in the log message string — errors({stack:true}) attaches it automatically when the Error is passed as an argument. Pass the Error itself as a positional/meta argument instead.',
+        },
+        {
+          selector:
+            "CallExpression[callee.object.name='logger'] CallExpression[callee.object.name='JSON'][callee.property.name='stringify'] > Identifier.arguments:first-child[name=/^(e|err|error|ex)$/]",
+          message:
+            "Don't JSON.stringify an Error — message/stack are non-enumerable and it serializes to {}. Pass the Error object directly as a log argument.",
+        },
+      ],
+    },
+  }
+);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elisra-devops/docgen-data-provider",
-  "version": "1.140.0",
+  "version": "1.141.0",
   "description": "A document generator data provider, aimed to retrive data from azure devops",
   "repository": {
     "type": "git",
@@ -11,9 +11,11 @@
   "types": "./bin/index.d.ts",
   "scripts": {
     "test": "jest --coverage --runInBand --forceExit",
-    "build": "tsc"
+    "build": "tsc",
+    "lint": "eslint ."
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.5",
     "@types/babel__generator": "^7.6.8",
     "@types/babel__template": "^7.4.4",
     "@types/caseless": "^0.12.5",
@@ -24,11 +26,14 @@
     "@types/yargs-parser": "^21.0.3",
     "baseline-browser-mapping": "^2.9.0",
     "dotenv": "^16.4.5",
+    "eslint": "^9.39.5",
+    "globals": "^15.15.0",
     "jest": "^29.7.0",
     "jest-junit": "^16.0.0",
     "npm": "^10.8.2",
     "ts-jest": "^29.2.3",
-    "typescript": "^5.5.4"
+    "typescript": "^5.5.4",
+    "typescript-eslint": "^8.71.0"
   },
   "dependencies": {
     "@types/xml2js": "^0.4.5",

package/src/helpers/tfs.ts

@@ -248,7 +248,7 @@ export class TFSServices {
 
         // Log error if needed
         if (printError) {
-          this.logDetailedError(e, url);
+          this.logDetailedError(e, url, attempts);
         }
 
         throw e;
@@ -286,23 +286,27 @@ export class TFSServices {
   /**
    * Log detailed error information
    */
-  private static logDetailedError(error: any, url: string): void {
-    if (error.response) {
-      logger.error(`Error for ${url} - ${error.message}`);
-      logger.error(`Status: ${error.response.status}`);
-
-      if (error.response.data) {
-        if (typeof error.response.data === 'string') {
-          logger.error(`Response: ${error.response.data.substring(0, 200)}`);
-        } else {
-          const dataMessage =
-            error.response.data.message || JSON.stringify(error.response.data).substring(0, 200);
-          logger.error(`Response: ${dataMessage}`);
+  private static logDetailedError(error: any, url: string, attempt?: number): void {
+    let responseExcerpt: string | undefined;
+    if (error.response?.data) {
+      if (typeof error.response.data === 'string') {
+        responseExcerpt = error.response.data.substring(0, 200);
+      } else {
+        try {
+          responseExcerpt = error.response.data.message || JSON.stringify(error.response.data).substring(0, 200);
+        } catch {
+          responseExcerpt = '[unserializable response data]';
         }
       }
-    } else {
-      logger.error(`Error for ${url} - ${error.message}`);
     }
+    logger.error('ADO request failed', {
+      message: error.message,
+      stack: error.stack,
+      url,
+      status: error.response?.status,
+      responseExcerpt,
+      attempt,
+    });
   }
 
   private static getErrorMessage(error: any): string {

package/src/modules/JfrogDataProvider.ts

@@ -19,7 +19,7 @@ export default class JfrogDataProvider {
     if (!serviceConnectionResponse?.url) {
       throw new Error(`Service connection ${connectionId} returned no URL — identity may lack service-endpoint read permission`);
     }
-    logger.debug(`service connection url ${JSON.stringify(serviceConnectionResponse.url)}`);
+    logger.debug(`service connection url ${serviceConnectionResponse.url}`);
     return serviceConnectionResponse.url;
   }
 

package/src/modules/PipelinesDataProvider.ts

@@ -1252,7 +1252,7 @@ export default class PipelinesDataProvider {
       },
       parameters: parameter, //'{"Test":"123"}'
     };
-    logger.info(JSON.stringify(data));
+    logger.info('Triggering build', data);
     let url = `${this.orgUrl}${projectName}/_apis/build/builds?api-version=5.0`;
     let res = await TFSServices.postRequest(url, this.token, 'post', data, null);
     return res;

package/src/modules/ResultDataProvider.ts

@@ -259,10 +259,22 @@ export default class ResultDataProvider {
 
       return { combinedResults, openPcrToTestCaseTraceMap, testCaseToOpenPcrTraceMap };
     } catch (error: any) {
-      logger.error(`Error during getCombinedResultsSummary: ${error.message}`);
+      // One event, one record — this used to be two separate calls (message, then
+      // conditionally the response body), which interleave with other requests' output
+      // under concurrency and become two unrelated entries in the dashboard's error grouping.
+      let responseSummary: string | undefined;
       if (error.response) {
-        logger.error(`Response Data: ${JSON.stringify(error.response.data)}`);
+        // Truncated to match the bound already used for this in tfs.ts — an untruncated ADO
+        // response body isn't a credential, but it can still be large/sensitive work-item content
+        // and doesn't need to go into logs unbounded. JSON.stringify itself (not just the
+        // output) can throw on a circular value, so that has to be guarded too, not just capped.
+        try {
+          responseSummary = JSON.stringify(error.response.data).substring(0, 200);
+        } catch {
+          responseSummary = '[unserializable response data]';
+        }
       }
+      logger.error(`Error during getCombinedResultsSummary: ${error.message}`, { responseSummary });
       // Ensure the error is rethrown to propagate it correctly
       throw new Error(error.message || 'Unknown error occurred during getCombinedResultsSummary');
     }
@@ -3261,8 +3273,7 @@ export default class ResultDataProvider {
       );
       return detailedPoints;
     } catch (err: any) {
-      logger.error(`Error during fetching Cross Test Points: ${err.message}`);
-      logger.error(`Error stack: ${err.stack}`);
+      logger.error('Error during fetching Cross Test Points', err);
       return [];
     }
   }
@@ -3903,10 +3914,7 @@ export default class ResultDataProvider {
         relatedCRs,
       };
     } catch (error: any) {
-      logger.error(`Error while fetching run result: ${error.message}`);
-      if (isTestReporter) {
-        logger.error(`Error stack: ${error.stack}`);
-      }
+      logger.error('Error while fetching run result', error);
       return null;
     }
   }
@@ -4595,7 +4603,7 @@ export default class ResultDataProvider {
           ? testCase.workItem.workItemFields
           : [];
         if (testCaseWorkItemFields.length === 0) {
-          logger.warn(`Could not fetch the steps from WI ${JSON.stringify(testCase.workItem.id)}`);
+          logger.warn(`Could not fetch the steps from WI ${testCase.workItem.id}`);
           if (!isTestReporter) {
             continue;
           }
@@ -5058,8 +5066,7 @@ export default class ResultDataProvider {
         ? createResponseObject(resultData, testSuiteId, point, ...additionalArgs)
         : null;
     } catch (error: any) {
-      logger.error(`Error occurred for point ${point.testCaseId}: ${error.message}`);
-      logger.error(`Stack trace: ${error.stack}`);
+      logger.error(`Error occurred for point ${point.testCaseId}`, error);
       return null;
     }
   }
@@ -5153,8 +5160,7 @@ export default class ResultDataProvider {
           summarizedItemMap.set(wi.id, mappedItem);
         }
       } catch (error: any) {
-        logger.error(`Error occurred while fetching linked work items: ${error.message}`);
-        logger.error(`Error Stack: ${error.stack}`);
+        logger.error('Error occurred while fetching linked work items', error);
       }
     }
 
@@ -5826,8 +5832,7 @@ export default class ResultDataProvider {
           }
           return resultDataResponse;
         } catch (err: any) {
-          logger.error(`Error occurred while fetching result data: ${err.message}`);
-          logger.error(`Error stack: ${err.stack}`);
+          logger.error('Error occurred while fetching result data', err);
           return null;
         }
       },

package/src/modules/TicketsDataProvider.ts

@@ -783,11 +783,9 @@ export default class TicketsDataProvider {
         : null;
     } catch (err: any) {
       logger.error(
-        `Error occurred while constructing the customer query list ${err.message} with query ${JSON.stringify(
-          rootQuery,
-        )}`,
+        `Error occurred while constructing the customer query list with query ${JSON.stringify(rootQuery)}`,
+        err,
       );
-      logger.error(`Error stack ${err.message}`);
       return null;
     }
   }
@@ -2697,8 +2695,10 @@ export default class TicketsDataProvider {
         const subRes = await postBatch(currentIds);
         res.push(...subRes.value);
       } catch (error) {
-        logger.error(`error populating workitems array`);
-        logger.error(JSON.stringify(error));
+        // JSON.stringify(error) on an AxiosError calls its own .toJSON(), which serializes
+        // config.auth.password / config.headers.Authorization — the PAT, in plaintext. Pass
+        // the error as meta instead and let the logger's redact format scrub it.
+        logger.error(`error populating workitems array`, error);
         return [];
       }
     }
@@ -2708,8 +2708,10 @@ export default class TicketsDataProvider {
         const subRes = await postBatch(currentIds);
         res.push(...subRes.value);
       } catch (error) {
-        logger.error(`error populating workitems array`);
-        logger.error(JSON.stringify(error));
+        // JSON.stringify(error) on an AxiosError calls its own .toJSON(), which serializes
+        // config.auth.password / config.headers.Authorization — the PAT, in plaintext. Pass
+        // the error as meta instead and let the logger's redact format scrub it.
+        logger.error(`error populating workitems array`, error);
         return [];
       }
     }
@@ -2843,8 +2845,9 @@ export default class TicketsDataProvider {
       );
       return attachmentList;
     } catch (e) {
-      logger.error(`error fetching attachments for work item ${id}`);
-      logger.error(`${JSON.stringify(e)}`);
+      // JSON.stringify(e) on an Error returns "{}" — own enumerable props only, message/stack
+      // are non-enumerable — so this logged literally nothing useful. Pass it as meta instead.
+      logger.error(`error fetching attachments for work item ${id}`, e);
       return [];
     }
   }
@@ -3161,11 +3164,9 @@ export default class TicketsDataProvider {
       return { tree1: tree1Node, tree2: tree2Node };
     } catch (err: any) {
       logger.error(
-        `Error occurred while constructing the query list ${err.message} with query ${JSON.stringify(
-          rootQuery,
-        )}`,
+        `Error occurred while constructing the query list with query ${JSON.stringify(rootQuery)}`,
+        err,
       );
-      logger.error(`Error stack ${err.message}`);
     }
   }
 

package/src/tests/modules/JfrogDataProvider.test.ts

@@ -36,7 +36,9 @@ describe('JfrogDataProvider', () => {
         `${mockOrgUrl}${mockTeamProject}/_apis/serviceendpoint/endpoints/${mockConnectionId}?api-version=7.1`,
         mockTfsToken
       );
-      expect(logger.debug).toHaveBeenCalledWith(`service connection url "${mockResponse.url}"`);
+      // The log used JSON.stringify(url), which wraps a plain string in quotes for no reason —
+      // fixed to interpolate the string directly (see docgen-log-redaction / Phase 2).
+      expect(logger.debug).toHaveBeenCalledWith(`service connection url ${mockResponse.url}`);
       expect(result).toBe(mockResponse.url);
     });
 

package/src/tests/modules/ResultDataProvider.test.ts

@@ -6027,8 +6027,12 @@ describe('ResultDataProvider', () => {
         resultDataProvider.getCombinedResultsSummary(mockTestPlanId, mockProjectName)
       ).rejects.toThrow('boom');
 
-      expect(logger.error).toHaveBeenCalledWith('Error during getCombinedResultsSummary: boom');
-      expect(logger.error).toHaveBeenCalledWith('Response Data: {"detail":"bad"}');
+      // Was two separate calls (message, then the response body) — consolidated into one
+      // record with the response summary as meta (docgen-log-redaction / Phase 2/4).
+      expect(logger.error).toHaveBeenCalledWith('Error during getCombinedResultsSummary: boom', {
+        responseSummary: '{"detail":"bad"}',
+      });
+      expect(logger.error).toHaveBeenCalledTimes(1);
     });
   });
 

package/src/tests/utils/logger.test.ts

@@ -0,0 +1,270 @@
+import * as winston from 'winston';
+import Transport = require('winston-transport');
+import { redact, DiagnosticsTransport } from '../../utils/logger';
+import { installLogSink, LogSink, DiagnosticEvent } from '../../utils/logSink';
+import { runContextStore } from '../../utils/runContext';
+
+const applyRedact = (info: Record<string, unknown>) => (redact() as any).transform({ ...info });
+
+describe('logger redact format', () => {
+  test('scrubs a top-level token/password/secret regardless of key casing', () => {
+    const out = applyRedact({ level: 'info', message: 'x', token: 'abc', Password: 'p', SECRET: 's' });
+    expect(out.token).toBe('[REDACTED]');
+    expect(out.Password).toBe('[REDACTED]');
+    expect(out.SECRET).toBe('[REDACTED]');
+  });
+
+  test('scrubs nested config.auth.password and config.headers.Authorization — the AxiosError.toJSON() leak', () => {
+    // This is the exact shape that was leaking via logger.error(JSON.stringify(error))
+    // in TicketsDataProvider.ts on ADO batch work-item failures.
+    const axiosLikeError = {
+      message: 'Request failed',
+      config: {
+        auth: { username: '', password: 'super-secret-pat' },
+        headers: { Authorization: 'Bearer abc.def.ghi' },
+      },
+    };
+    const out: any = applyRedact({ level: 'error', message: 'error populating workitems array', err: axiosLikeError });
+    expect(out.err.config.auth.password).toBe('[REDACTED]');
+    expect(out.err.config.headers.Authorization).toBe('[REDACTED]');
+    expect(out.err.message).toBe('Request failed');
+  });
+
+  test('leaves non-sensitive fields untouched', () => {
+    const out = applyRedact({ level: 'info', message: 'ok', project: 'Cube-ADCS', docType: 'SVD' });
+    expect(out.project).toBe('Cube-ADCS');
+    expect(out.docType).toBe('SVD');
+  });
+
+  test('never throws on a circular object', () => {
+    const circular: any = { name: 'x' };
+    circular.self = circular;
+    expect(() => applyRedact({ level: 'info', message: 'x', circular })).not.toThrow();
+  });
+
+  test('never throws and passes info through on null/undefined meta', () => {
+    expect(() => applyRedact({ level: 'info', message: 'x', meta: null })).not.toThrow();
+    expect(() => applyRedact({ level: 'info', message: 'x', meta: undefined })).not.toThrow();
+  });
+});
+
+// A capturing transport + the exact JSON format chain logger.ts uses (errors -> timestamp ->
+// redact -> splat -> json), so these tests exercise the real pipeline, not just redact() alone.
+class CaptureTransport extends Transport {
+  lines: Record<string, unknown>[] = [];
+  log(info: Record<string, unknown>, callback: () => void) {
+    this.lines.push(JSON.parse((info as any)[Symbol.for('message')] ?? JSON.stringify(info)));
+    callback();
+  }
+}
+function makeTestLogger() {
+  const capture = new CaptureTransport();
+  const logger = winston.createLogger({
+    level: 'silly',
+    format: winston.format.combine(
+      winston.format.errors({ stack: true }),
+      winston.format.timestamp(),
+      redact(),
+      winston.format.splat(),
+      winston.format.json()
+    ),
+    transports: [capture],
+  });
+  return { logger, capture };
+}
+
+describe('logger pipeline — errors({stack:true})', () => {
+  test('logger.error(err) (single-arg Error) still produces a non-empty message and a stack', () => {
+    // This is the regression errors({stack:true}) exists to prevent: without it, winston's
+    // single-arg path makes `info` *be* the Error, and message/stack are non-enumerable, so
+    // json() would emit {"level":"error","timestamp":"…"} — content-free.
+    const { logger, capture } = makeTestLogger();
+    logger.error(new Error('boom'));
+    expect(capture.lines).toHaveLength(1);
+    expect(capture.lines[0].message).toBe('boom');
+    expect(typeof capture.lines[0].stack).toBe('string');
+    expect((capture.lines[0].stack as string).length).toBeGreaterThan(0);
+  });
+});
+
+describe('logger pipeline — hostile inputs never throw', () => {
+  test.each([
+    ['null', null],
+    ['undefined', undefined],
+    ['NaN', NaN],
+    ['a circular object', (() => { const c: any = { a: 1 }; c.self = c; return c; })()],
+    ['an object with a throwing getter', { get boom() { throw new Error('nope'); } }],
+    ['a BigInt', BigInt(9007199254740993)],
+    ['a Symbol', Symbol('x')],
+    ['an Error with no message', new Error()],
+    ['a 10MB string', 'x'.repeat(10 * 1024 * 1024)],
+  ])('logger.error(%s) does not throw', (_label, value) => {
+    const { logger } = makeTestLogger();
+    expect(() => logger.error(value as any)).not.toThrow();
+  });
+});
+
+describe('DiagnosticsTransport (Phase 6a)', () => {
+  function makeDiagnosticsLogger() {
+    const capture = new CaptureTransport();
+    const logger = winston.createLogger({
+      level: 'silly',
+      format: winston.format.combine(
+        winston.format.errors({ stack: true }),
+        winston.format.timestamp(),
+        redact(),
+        winston.format.splat(),
+        winston.format.json()
+      ),
+      transports: [capture, new DiagnosticsTransport()],
+    });
+    return { logger, capture };
+  }
+
+  test('pushes warn/error records to whatever sink is installed', () => {
+    const events: DiagnosticEvent[] = [];
+    const sink: LogSink = { push: (e) => events.push(e) };
+    installLogSink(sink);
+    const { logger } = makeDiagnosticsLogger();
+    logger.warn('a stable warning');
+    logger.error('a stable error', Object.assign(new Error('boom'), { code: 'ECONN' }));
+    expect(events).toHaveLength(2);
+    expect(events[0].level).toBe('warn');
+    expect(events[1].err?.code).toBe('ECONN');
+    expect(events[1].err?.stack).toEqual(expect.any(String));
+  });
+
+  test('does not push info/debug records', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    logger.info('not interesting to the dashboard');
+    expect(events).toHaveLength(0);
+  });
+
+  test('a throwing sink cannot break the log call', () => {
+    installLogSink({
+      push: () => {
+        throw new Error('sink is down');
+      },
+    });
+    const { logger } = makeDiagnosticsLogger();
+    expect(() => logger.error('still must not throw')).not.toThrow();
+  });
+
+  test.each([
+    ['null', null],
+    ['undefined', undefined],
+    ['NaN', NaN],
+    ['a circular object', (() => { const c: any = { a: 1 }; c.self = c; return c; })()],
+    ['an object with a throwing getter', { get boom() { throw new Error('nope'); } }],
+    ['a BigInt', BigInt(9007199254740993)],
+    ['a Symbol', Symbol('x')],
+    ['an Error with no message', new Error()],
+    ['a 10MB string', 'x'.repeat(10 * 1024 * 1024)],
+  ])('logger.error(%s) with a sink installed does not throw', (_label, value) => {
+    installLogSink({ push: () => undefined });
+    const { logger } = makeDiagnosticsLogger();
+    expect(() => logger.error(value as any)).not.toThrow();
+  });
+});
+
+describe('textFormat safety (regression)', () => {
+  // The hostile-input matrix above exercises makeTestLogger()'s json() pipeline — it never
+  // touched the real singleton logger's text-format branch (LOG_FORMAT's default), which is
+  // where a Symbol message crashed: a template literal's implicit ToString throws on a Symbol,
+  // and this formatter sits with no try/catch around it.
+  test('logger.error(Symbol) does not throw through the real default-exported logger', () => {
+    // eslint-disable-next-line @typescript-eslint/no-var-requires
+    const logger = require('../../utils/logger').default;
+    expect(() => logger.error(Symbol('x'))).not.toThrow();
+  });
+});
+
+describe('DiagnosticsTransport capture policy (Phase 6b)', () => {
+  function makeDiagnosticsLogger() {
+    const capture = new CaptureTransport();
+    const logger = winston.createLogger({
+      level: 'silly',
+      format: winston.format.combine(
+        winston.format.errors({ stack: true }),
+        winston.format.timestamp(),
+        redact(),
+        winston.format.splat(),
+        winston.format.json()
+      ),
+      transports: [capture, new DiagnosticsTransport()],
+    });
+    return { logger, capture };
+  }
+
+  test('normal mode (no captureMode) does not persist debug/info', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'run-1' }, () => {
+      logger.debug('a debug line');
+      logger.info('an info line');
+    });
+    expect(events).toHaveLength(0);
+  });
+
+  test('verbose mode persists debug and info', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'run-2', captureMode: 'verbose' }, () => {
+      logger.debug('a debug line');
+      logger.info('an info line');
+    });
+    expect(events).toHaveLength(2);
+    expect(events[0].retainPending).toBeUndefined();
+  });
+
+  test('warn/error persist regardless of capture mode', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'run-3' }, () => {
+      logger.warn('a warning');
+      logger.error('an error');
+    });
+    expect(events).toHaveLength(2);
+  });
+
+  test('retain-on-failure persists debug/info tagged retainPending: true', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'run-4', captureMode: 'retain-on-failure' }, () => {
+      logger.debug('a debug line');
+    });
+    expect(events).toHaveLength(1);
+    expect(events[0].retainPending).toBe(true);
+  });
+
+  test('retain-on-failure does not tag warn/error as retainPending', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'run-5', captureMode: 'retain-on-failure' }, () => {
+      logger.error('an error');
+    });
+    expect(events).toHaveLength(1);
+    expect(events[0].retainPending).toBeUndefined();
+  });
+
+  test('a concurrent normal-mode run is unaffected by a sibling verbose run', () => {
+    const events: DiagnosticEvent[] = [];
+    installLogSink({ push: (e) => events.push(e) });
+    const { logger } = makeDiagnosticsLogger();
+    runContextStore.run({ runId: 'verbose-run', captureMode: 'verbose' }, () => {
+      logger.debug('verbose debug');
+    });
+    runContextStore.run({ runId: 'normal-run' }, () => {
+      logger.debug('normal debug');
+    });
+    expect(events).toHaveLength(1);
+    expect(events[0].message).toBe('verbose debug');
+  });
+});