Release 5.6.0

.codegraph/.gitignore

@@ -0,0 +1,5 @@
+# CodeGraph data files — local to each machine, not for committing.
+# Ignore everything in .codegraph/ except this file itself, so transient
+# files (the database, daemon.pid, sockets, logs) never show up in git.
+*
+!.gitignore

.eslintignore

@@ -1,3 +1,11 @@
 node_modules
 build
 __tests__/**/*.json
+# Subpath facades generated by scripts/generate-resolver-facades.js
+/helpers.js
+/helpers.d.ts
+/constants.js
+/constants.d.ts
+/models.d.ts
+/publicReportingAPI.js
+/publicReportingAPI.d.ts

.eslintrc

@@ -21,7 +21,10 @@
   },
   "ignorePatterns": ["__tests__/**/*"],
   "rules": {
-    "valid-jsdoc": ["error", { "requireReturn": false }],
+    "valid-jsdoc": 0,
+    "import/extensions": 0,
+    "import/no-unresolved": 0,
+    "camelcase": 0,
     "consistent-return": 0,
     "@typescript-eslint/no-plusplus": 0,
     "prettier/prettier": 2,
@@ -36,6 +39,12 @@
     "@typescript-eslint/ban-ts-comment": 0,
     "@typescript-eslint/no-var-requires": 0,
     "@typescript-eslint/no-floating-promises": 2,
-    "max-classes-per-file": 0
+    "max-classes-per-file": 0,
+    "no-shadow": 0,
+    "@typescript-eslint/no-shadow": 2,
+    "no-console": 0,
+    "no-unused-vars": 0,
+    "@typescript-eslint/no-unused-vars": 2,
+    "@typescript-eslint/no-explicit-any": 2
   }
 }

.github/workflows/CI-pipeline.yml

@@ -47,6 +47,9 @@ jobs:
       - name: Install dependencies
         run: npm install
 
+      - name: Audit for vulnerable prod dependencies
+        run: npm audit --audit-level=high --omit=dev
+
       - name: Build the source code
         run: npm run build
 

.github/workflows/publish.yml

@@ -51,6 +51,8 @@ jobs:
           registry-url: 'https://registry.npmjs.org'
       - name: Install dependencies
         run: npm install
+      - name: Build
+        run: npm run build
       - name: Publish to NPM
         run: |
           npm config list

.gitignore

@@ -6,3 +6,12 @@ coverage.lcov
 coverage/
 .npmrc
 build
+
+# Subpath facades generated by scripts/generate-resolver-facades.js
+/helpers.js
+/helpers.d.ts
+/constants.js
+/constants.d.ts
+/models.d.ts
+/publicReportingAPI.js
+/publicReportingAPI.d.ts

CHANGELOG.md

@@ -1,7 +1,23 @@
+### Changed
+- The client has been migrated to TypeScript. It now ships with bundled type
+  definitions (no separate `@types` package required) and exposes constants,
+  models and helpers via subpath imports (e.g.
+  `@reportportal/client-javascript/constants`).
+- **Breaking Change** Deep `lib/**` imports published up to 5.5.x (e.g.
+  `require('@reportportal/client-javascript/lib/helpers')`) still resolve for Node,
+  TypeScript and bundlers via the `exports` map, but are no longer backed by physical files.
+  Tools that resolve imports by walking the filesystem instead of reading `exports` — most
+  notably `eslint-import-resolver-node`, the default resolver of `eslint-plugin-import` —
+  will report [`import/no-unresolved`](https://github.com/import-js/eslint-plugin-import/issues/1810) for these paths. Switch to the new subpath aliases
+  (`constants`, `models`, `helpers`, `publicReportingAPI`) instead.
+- **Breaking Change** Drop support of Node.js 14. The version [5.5.12](https://github.com/reportportal/client-javascript/releases/tag/v5.5.12) is the latest that supports it. The minimum supported Node.js version is now 16.0.0.
+### Added
+- `retry_of` property is now automatically included in the `startTestItem`
+  request payload when `retry: true` and a previous attempt exists in the
+  retry chain. This allows the ReportPortal backend to link retry chains
+  efficiently, improving query performance for large test runs.
 
 ## [5.5.12] - 2026-07-06
-
-## [Unreleased]
 ### Changed
 - Replaced the `uniqid` and `uuid` dependencies with the built-in `crypto.randomUUID()` for internal id generation, removing both external packages ([#210](https://github.com/reportportal/agent-js-playwright/issues/210)).
 - Bumped the minimum supported Node.js version to 14.17.0 (required by `crypto.randomUUID`).

DEV_GUIDE.md

@@ -0,0 +1,58 @@
+# Dev Guide
+
+Internal notes for contributors. This content is intentionally kept out of
+README.md, which is published to package registries.
+
+## Subpath facades
+
+`npm run build` compiles to `build/` and then runs `scripts/generate-resolver-facades.js`,
+which writes a one-line re-export at each public subpath alias — `helpers.js`,
+`constants.js`, `publicReportingAPI.js` — together with matching `.d.ts` files. `models` only
+gets a `.d.ts` facade: `src/models/**` is TypeScript types with no runtime value (confirmed —
+the compiled `build/models/*.js` files are all empty `__esModule` stubs), so
+`package.json#exports["./models"]` has no `import` / `require` condition. Node enforces that
+regardless of whether a `.js` file physically exists at the root, so writing one would just be
+misleading. If a consumer's TypeScript setup runs with `isolatedModules` and they import from
+`models` without `import type`, they'll now get `ERR_PACKAGE_PATH_NOT_EXPORTED` at runtime
+instead of a silently-empty object — that's intentional; there are no known consumers of this
+brand-new subpath yet, so this is the cheapest point to make the contract strict. If a genuine
+runtime value ever needs to live under `models`, move it to `constants/` (like `MERGE_TYPES`,
+which used to live here) instead of adding a runtime condition back.
+
+Supported package imports resolve through the `exports` / `typesVersions` maps in
+`package.json` straight to `build` for Node, TypeScript and bundlers; these files are never
+on that path. They exist only for tools that resolve imports by walking the filesystem
+instead of reading `exports`, chiefly `eslint-import-resolver-node` (the default resolver of
+`eslint-plugin-import`), which otherwise reports `import/no-unresolved` for these subpath
+imports and forces each consumer to configure an ignore.
+
+The alias list in the script is intentionally fixed and small — it does not mirror every
+internal module under `build`. Deep `lib/**` imports as published up to 5.5.x (e.g.
+`require('@reportportal/client-javascript/lib/rest')`) resolve via the `exports` map for
+Node, TypeScript and bundlers, but are **not** backed by a physical `lib/**` tree — a
+filesystem-based resolver hitting one of those undocumented deep paths still needs a local
+ignore. If you're bumping a first-party agent past this version and it imports
+`@reportportal/client-javascript/lib/**` directly, switch it to the matching short alias in
+the same PR rather than adding it here.
+
+Everything the script writes is gitignored and removed by `npm run clean`. It recomputes
+each facade's export style (`export *` vs `export =` vs default) from the real compiled
+module on every build, so it can't silently drift the way a hand-written file could. When
+you add a new top-level subpath to `exports`, add the matching entry to the `ALIASES` map in
+the script.
+
+## Code knowledge graph
+
+This repo carries a local **code knowledge graph** ([colbymchenry/codegraph](https://github.com/colbymchenry/codegraph))
+that the ReportPortal AI agents (and your own tooling) use to resolve symbols and
+references without scanning raw files.
+
+```bash
+npm run codegraph             # build it the first time, fast incremental sync after
+npm run codegraph -- --force  # rebuild from scratch
+```
+
+The graph lives in `.codegraph/codegraph.db` — it is **gitignored and local to your
+machine** (only `.codegraph/.gitignore` is committed). It is a pure derivative of the
+source, so regenerate it any time. The engine is fetched on demand via `npx`, so there
+is no added project dependency.

README.md

@@ -48,6 +48,28 @@ rpClient.checkConnect().then(() => {
 });
 ```
 
+## TypeScript
+
+Starting from version 5.6.0, the client has been migrated to TypeScript and ships with bundled type definitions. It works with both CommonJS and ES module projects:
+
+```typescript
+import RPClient from '@reportportal/client-javascript';
+
+const rpClient = new RPClient({
+    apiKey: 'reportportalApiKey',
+    endpoint: 'https://your-instance.com:8080/api/v1',
+    launch: 'LAUNCH_NAME',
+    project: 'PROJECT_NAME',
+});
+```
+
+Constants, models and helpers are available via subpath imports:
+
+```typescript
+import { RP_STATUSES } from '@reportportal/client-javascript/constants';
+import PublicReportingAPI from '@reportportal/client-javascript/publicReportingAPI';
+```
+
 ## Configuration
 
 When creating a client instance, you need to specify the following options.

VERSION

@@ -1 +1 @@
-5.5.12
+5.5.13-SNAPSHOT

__tests__/client-id.spec.js → __tests__/client-id.spec.ts

@@ -1,11 +1,14 @@
-const fs = require('fs');
-const util = require('util');
-const path = require('path');
-const { randomUUID } = require('crypto');
+import fs from 'fs';
+import util from 'util';
+import path from 'path';
+import { randomUUID } from 'crypto';
 
 const testHomeDir = path.join(__dirname, '__tmp__', 'rp-home');
 process.env.RP_CLIENT_JS_HOME = testHomeDir;
-const { getClientId } = require('../statistics/client-id');
+
+// A static `import` would be hoisted above the `process.env` assignment above, so the module
+// (which reads RP_CLIENT_JS_HOME at load time) is loaded lazily via a dynamic import instead.
+let getClientId: () => Promise<string>;
 
 const uuidv4Validation = /^[0-9A-F]{8}-[0-9A-F]{4}-4[0-9A-F]{3}-[89AB][0-9A-F]{3}-[0-9A-F]{12}$/i;
 const clientIdFile = path.join(testHomeDir, '.rp', 'rp.properties');
@@ -14,18 +17,21 @@ const unlink = util.promisify(fs.unlink);
 const readFile = util.promisify(fs.readFile);
 const writeFile = util.promisify(fs.writeFile);
 const removeTestHomeDir = () => fs.promises.rm(testHomeDir, { recursive: true, force: true });
-const unlinkFile = async (filePath) => {
+const unlinkFile = async (filePath: string) => {
   try {
     await unlink(filePath);
   } catch (error) {
-    if (error.code !== 'ENOENT') {
+    if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
       throw error;
     }
   }
 };
 
 describe('Client ID test suite', () => {
-  beforeAll(removeTestHomeDir);
+  beforeAll(async () => {
+    await removeTestHomeDir();
+    ({ getClientId } = await import('../src/statistics/client-id'));
+  });
   afterAll(removeTestHomeDir);
 
   it('getClientId should return the same client ID for two calls', async () => {

__tests__/config.spec.js → __tests__/config.spec.ts

@@ -1,8 +1,6 @@
-const { getClientConfig, getRequiredOption, getApiKey } = require('../lib/commons/config');
-const {
-  ReportPortalRequiredOptionError,
-  ReportPortalValidationError,
-} = require('../lib/commons/errors');
+import { getClientConfig, getRequiredOption, getApiKey } from '../src/commons/config';
+import { ReportPortalRequiredOptionError, ReportPortalValidationError } from '../src/commons/errors';
+import type { ReportPortalConfig } from '../src/models/config';
 
 describe('Config commons test suite', () => {
   describe('getRequiredOption', () => {
@@ -26,7 +24,7 @@ describe('Config commons test suite', () => {
     it('should throw ReportPortalRequiredOptionError in case of option not present in options', () => {
       let error;
       try {
-        getRequiredOption({ other: 1 }, 'project');
+        getRequiredOption({ other: 1 } as unknown as { project: unknown }, 'project');
       } catch (e) {
         error = e;
       }
@@ -73,7 +71,7 @@ describe('Config commons test suite', () => {
   describe('getClientConfig', () => {
     it('should print ReportPortalValidationError error to the console in case of options is not an object type', () => {
       jest.spyOn(console, 'dir').mockImplementation();
-      getClientConfig('options');
+      getClientConfig('options' as unknown as ReportPortalConfig);
 
       expect(console.dir).toHaveBeenCalledWith(
         new ReportPortalValidationError('`options` must be an object.'),
@@ -85,7 +83,7 @@ describe('Config commons test suite', () => {
       getClientConfig({
         apiKey: '123',
         project: 'prj',
-      });
+      } as unknown as ReportPortalConfig);
 
       expect(console.dir).toHaveBeenCalledWith(new ReportPortalRequiredOptionError('endpoint'));
     });
@@ -95,7 +93,7 @@ describe('Config commons test suite', () => {
       getClientConfig({
         apiKey: '123',
         endpoint: 'https://abc.com',
-      });
+      } as unknown as ReportPortalConfig);
 
       expect(console.dir).toHaveBeenCalledWith(new ReportPortalRequiredOptionError('project'));
     });
@@ -105,7 +103,7 @@ describe('Config commons test suite', () => {
       getClientConfig({
         project: 'prj',
         endpoint: 'https://abc.com',
-      });
+      } as unknown as ReportPortalConfig);
 
       expect(console.dir).toHaveBeenCalledWith(new ReportPortalRequiredOptionError('apiKey'));
     });

__tests__/helpers.spec.js → __tests__/helpers.spec.ts

@@ -1,8 +1,9 @@
-const os = require('os');
-const fs = require('fs');
-const glob = require('glob');
-const helpers = require('../lib/helpers');
-const pjson = require('../package.json');
+import os from 'os';
+import fs from 'fs';
+import * as glob from 'glob';
+import * as helpers from '../src/helpers';
+import type { TestItemParameter } from '../src/models/requests';
+import pjson from '../package.json';
 
 describe('Helpers', () => {
   describe('formatName', () => {
@@ -22,7 +23,7 @@ describe('Helpers', () => {
 
   describe('now', () => {
     it('returns milliseconds from unix time', () => {
-      expect(new Date() - helpers.now()).toBeLessThan(100); // less than 100 miliseconds difference
+      expect(Number(new Date()) - helpers.now()).toBeLessThan(100); // less than 100 miliseconds difference
     });
   });
 
@@ -46,11 +47,11 @@ describe('Helpers', () => {
     });
   });
 
-  describe('getSystemAttribute', () => {
+  describe('getSystemAttributes', () => {
     it('should return correct system attributes', () => {
       jest.spyOn(os, 'type').mockReturnValue('osType');
       jest.spyOn(os, 'arch').mockReturnValue('osArchitecture');
-      jest.spyOn(os, 'totalmem').mockReturnValue('1');
+      jest.spyOn(os, 'totalmem').mockReturnValue(1);
       const nodeVersion = process.version;
       const expectedAttr = [
         {
@@ -75,7 +76,7 @@ describe('Helpers', () => {
         },
       ];
 
-      const attr = helpers.getSystemAttribute();
+      const attr = helpers.getSystemAttributes();
 
       expect(attr).toEqual(expectedAttr);
     });
@@ -106,7 +107,7 @@ describe('Helpers', () => {
           key: 'keyThree',
           value: 'valueThree',
         },
-      ];
+      ] as TestItemParameter[];
 
       const testCaseId = helpers.generateTestCaseId('codeRef', parameters);