feat(sdk-utilities,logger,oidc,sdk-request-middleware,sdk-types): Added

packages

Ported over logger, oidc, and abstracted request middleware into own
packages. As a result we created a shared types and and a utilities
directory. Updated davinci-client to use these packages. Added copyright

 Co-authored-by: AJ Ancheta <ajancheta@pingidentity.com>
 Co-authored-by: Justin Lowery <justin.lowery@pingidentity.com>

Author: ancheetah

.changeset/config.json

@@ -8,7 +8,6 @@
   ],
   "commit": false,
   "fixed": [["@forgerock/*"]],
-  "privatePackages": false,
   "linked": [],
   "access": "public",
   "baseBranch": "main",

.changeset/new-otters-warn.md

@@ -0,0 +1,9 @@
+---
+'@forgerock/sdk-request-middleware': minor
+'@forgerock/davinci-client': minor
+'@forgerock/sdk-logger': minor
+'@forgerock/sdk-utilities': minor
+'@forgerock/sdk-oidc': minor
+---
+
+created effects type packages, logger, oidc, and request middleware

.gitignore

@@ -18,6 +18,11 @@ packages/**/coverage/
 node_modules/
 tests/**/app/index.js*
 tests/**/ie11/ie-bundle.js*
+# Out-TSC directories (TypeScript output)
+**/out-tsc/*
+out-tsc/
+**/out-tsc
+**/*.out-tsc
 .swc
 .vite
 

CHANGELOG.md

@@ -1,3 +1,4 @@
 # ForgeRock Javascript SDK Changelogs
 
 [ForgeRock DaVinci Client package](./packages/davinci-client/CHANGELOG.md)
+[ForgeRock SDK Utilities package](./packages/sdk-utilities/CHANGELOG.md)

e2e/davinci-app/main.ts

@@ -8,7 +8,6 @@ import './style.css';
 
 import { Config, FRUser, TokenManager } from '@forgerock/javascript-sdk';
 import { davinci } from '@forgerock/davinci-client';
-
 import type { DaVinciConfig, RequestMiddleware } from '@forgerock/davinci-client/types';
 
 import textComponent from './components/text.js';
@@ -27,7 +26,7 @@ const searchParams = new URLSearchParams(qs);
 const config: DaVinciConfig =
   serverConfigs[searchParams.get('clientId') || '724ec718-c41c-4d51-98b0-84a583f450f9'];
 
-const requestMiddleware: RequestMiddleware[] = [
+const requestMiddleware: RequestMiddleware<'DAVINCI_NEXT' | 'DAVINCI_START'>[] = [
   (fetchArgs, action, next) => {
     if (action.type === 'DAVINCI_START') {
       fetchArgs.url.searchParams.set('start', 'true');
@@ -55,7 +54,9 @@ const urlParams = new URLSearchParams(window.location.search);
   if (continueToken) {
     resumed = await davinciClient.resume({ continueToken });
   } else {
-    await Config.setAsync(config);
+    // the current davinci-config has a slightly
+    // different middleware type than the old legacy config
+    await Config.setAsync(config as any);
   }
   function renderComplete() {
     const clientInfo = davinciClient.getClient();

e2e/davinci-app/package.json

@@ -5,7 +5,7 @@
   "type": "module",
   "private": true,
   "nx": {
-    "tags": ["scope:app"]
+    "tags": ["scope:e2e"]
   },
   "scripts": {
     "build": "pnpm nx nxBuild",

eslint.config.mjs

@@ -15,7 +15,12 @@ const compat = new FlatCompat({
 
 export default [
   {
-    ignores: ['**/dist', '**/vite.config.*.timestamp*', '**/vitest.config.*.timestamp*'],
+    ignores: [
+      '**/dist',
+      '**/vite.config.*.timestamp*',
+      '**/vitest.config.*.timestamp*',
+      '**/out-tsc',
+    ],
   },
   ...compat.extends('plugin:@typescript-eslint/recommended', 'plugin:prettier/recommended'),
   {
@@ -57,6 +62,7 @@ export default [
   {
     files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
     rules: {
+      'import/extensions': [2, 'ignorePackages'],
       '@nx/enforce-module-boundaries': [
         'warn',
         {
@@ -65,14 +71,30 @@ export default [
           depConstraints: [
             {
               sourceTag: 'scope:e2e',
-              onlyDependOnLibsWithTags: ['scope:app'],
+              onlyDependOnLibsWithTags: ['scope:package'],
             },
             {
               sourceTag: 'scope:package',
-              onlyDependOnLibsWithTags: [],
+              onlyDependOnLibsWithTags: [
+                'scope:sdk-utilities',
+                'scope:sdk-effects',
+                'scope:sdk-types',
+              ],
+            },
+            {
+              sourceTag: 'scope:sdk-utilities',
+              onlyDependOnLibsWithTags: ['scope:sdk-types'],
+            },
+            {
+              sourceTag: 'scope:sdk-effects',
+              onlyDependOnLibsWithTags: ['scope:sdk-utilities', 'scope:sdk-types'],
+            },
+            {
+              sourceTag: 'scope:sdk-config',
+              onlyDependOnLibsWithTags: ['scope:sdk-utilities', 'scope:sdk-types'],
             },
             {
-              sourceTag: 'scope:types',
+              sourceTag: 'scope:sdk-types',
               onlyDependOnLibsWithTags: [],
             },
           ],
@@ -133,7 +155,7 @@ export default [
     },
   },
   {
-    ignores: ['dist/*', '**/**/tsconfig.spec.vitest-temp.json'],
+    ignores: ['**/*.md', 'dist/*', '**/**/tsconfig.spec.vitest-temp.json'],
   },
   packageJson.configs.recommended,
   {

nx.json

@@ -25,7 +25,8 @@
       "!{projectRoot}/test-setup.[jt]s",
       "noEslintConfig",
       "noMarkdown",
-      "workspaceRootIgnores"
+      "workspaceRootIgnores",
+      "!{projectRoot}/**/?(*.)+(spec|test).[jt]s?(x)?(.snap)"
     ],
     "noMarkdown": ["!{projectRoot}/*.md", "!{projectRoot}/**/*.md"],
     "noTests": [
@@ -69,7 +70,7 @@
     },
     "test": {
       "inputs": ["default", "^default", "noMarkdown", "^noMarkdown"],
-      "dependsOn": ["^test"],
+      "dependsOn": ["^test", "^build", "^build", "^build", "^build"],
       "outputs": ["{projectRoot}/coverage"],
       "cache": true
     },
@@ -152,6 +153,20 @@
         "typecheckTargetName": "typecheck"
       },
       "include": ["packages/**/**/*", "e2e/**/**/*"]
+    },
+    {
+      "plugin": "@nx/vite/plugin",
+      "options": {
+        "buildTargetName": "vite:build",
+        "testTargetName": "vite:test",
+        "serveTargetName": "vite:serve",
+        "devTargetName": "vite:dev",
+        "previewTargetName": "vite:preview",
+        "serveStaticTargetName": "vite:serve-static",
+        "typecheckTargetName": "vite:typecheck",
+        "buildDepsTargetName": "vite:build-deps",
+        "watchDepsTargetName": "vite:watch-deps"
+      }
     }
   ],
   "parallel": 1,
@@ -161,7 +176,6 @@
   },
   "generators": {
     "@nx/js:library": {
-      "publishable": true,
       "outDir": "{projectRoot}/dist",
       "bundler": "tsc",
       "linter": "eslint",

package.json

@@ -1,19 +1,26 @@
 {
   "name": "ping-javascript-sdk",
+  "version": "0.0.0",
   "private": true,
   "description": "Ping JavaScript SDK",
-  "packageManager": "pnpm@9.15.9+sha512.68046141893c66fad01c079231128e9afb89ef87e2691d69e4d40eee228988295fd4682181bae55b58418c3a253bde65a505ec7c5f9403ece5cc3cd37dcf2531",
-  "engines": {
-    "node": "^20 || ^22",
-    "pnpm": "9.15.9"
+  "homepage": "https://github.com/ForgeRock/ping-javascript-sdk#readme",
+  "bugs": {
+    "url": "https://github.com/ForgeRock/ping-javascript-sdk/issues"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/ForgeRock/ping-javascript-sdk.git"
   },
+  "author": "ForgeRock",
   "scripts": {
     "build": "nx affected --target=build",
-    "clean": "shx rm -rf ./{coverage,dist,docs,node_modules,tmp}/ ./{packages,e2e}/*/{dist,node_modules}/ && git clean -fX -e \"!.env*,nx-cloud.env\"",
+    "changeset": "changeset",
     "ci:release": "pnpm publish -r --no-git-checks && changeset tag",
     "ci:version": "changeset version && pnpm install --no-frozen-lockfile && pnpm format",
-    "changeset": "changeset",
+    "circular-dep-check": "madge --circular .",
+    "clean": "shx rm -rf ./{coverage,dist,docs,node_modules,tmp}/ ./{packages,e2e}/*/{dist,node_modules}/ && git clean -fX -e \"!.env*,nx-cloud.env\"",
     "commit": "git cz",
+    "commitlint": "commitlint --edit",
     "create-package": "nx g @nx/js:library",
     "docs": "nx affected --target=typedoc",
     "format": "pnpm nx format:write",
@@ -24,9 +31,8 @@
     "serve": "nx serve",
     "test": "CI=true nx affected:test",
     "test:e2e": "CI=true nx affected:e2e",
-    "watch": "nx watch-deps",
     "verdaccio": "nx local-registry",
-    "commitlint": "commitlint --edit"
+    "watch": "nx watch-deps"
   },
   "lint-staged": {
     "*": [
@@ -36,15 +42,12 @@
       "git add"
     ]
   },
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/ForgeRock/ping-javascript-sdk.git"
-  },
-  "author": "ForgeRock",
-  "bugs": {
-    "url": "https://github.com/ForgeRock/ping-javascript-sdk/issues"
+  "config": {
+    "commitizen": {
+      "path": "./node_modules/cz-conventional-changelog"
+    }
   },
-  "homepage": "https://github.com/ForgeRock/ping-javascript-sdk#readme",
+  "dependencies": {},
   "devDependencies": {
     "@changesets/changelog-github": "^0.5.0",
     "@changesets/cli": "^2.27.9",
@@ -80,7 +83,7 @@
     "@types/node": "22.14.1",
     "@typescript-eslint/typescript-estree": "8.23.0",
     "@typescript-eslint/utils": "^8.13.0",
-    "@vitest/coverage-v8": "3.0.4",
+    "@vitest/coverage-v8": "^3.0.5",
     "@vitest/ui": "3.0.4",
     "conventional-changelog-conventionalcommits": "^8.0.0",
     "cz-conventional-changelog": "^3.3.0",
@@ -97,6 +100,7 @@
     "jsdom": "26.0.0",
     "jsonc-eslint-parser": "^2.1.0",
     "lint-staged": "^15.0.0",
+    "madge": "8.0.0",
     "npm-cli-login": "^1.0.0",
     "nx": "20.5.0",
     "playwright": "^1.47.2",
@@ -119,10 +123,10 @@
     "vitest": "3.0.5",
     "vitest-canvas-mock": "^0.3.3"
   },
-  "config": {
-    "commitizen": {
-      "path": "./node_modules/cz-conventional-changelog"
-    }
+  "packageManager": "pnpm@9.15.9+sha512.68046141893c66fad01c079231128e9afb89ef87e2691d69e4d40eee228988295fd4682181bae55b58418c3a253bde65a505ec7c5f9403ece5cc3cd37dcf2531",
+  "engines": {
+    "node": "^20 || ^22",
+    "pnpm": "9.15.9"
   },
   "nx": {
     "includedScripts": []

packages/davinci-client/package.json

@@ -22,7 +22,9 @@
     "test:watch": "pnpm nx nxTest --watch"
   },
   "dependencies": {
-    "@forgerock/javascript-sdk": "4.7.0",
+    "@forgerock/sdk-oidc": "workspace:*",
+    "@forgerock/sdk-request-middleware": "workspace:*",
+    "@forgerock/sdk-types": "workspace:*",
     "@reduxjs/toolkit": "catalog:",
     "immer": "catalog:"
   },