@pega/custom-dx-components 0.23.35 → 0.23.37

package/src/tasks/create/templates/Launchpad/Widget/PAGE-CASE/demo.stories.tsx.mustache

@@ -1,17 +1,20 @@
 /* eslint-disable react/jsx-no-useless-fragment */
 // @ts-nocheck
-import { withKnobs } from '@storybook/addon-knobs';
+import type { Meta, StoryObj } from '@storybook/react';
 
 import {{COMPONENT_CLASS_NAME}} from './index';
 
 import { configProps, operatorDetails } from './mock';
 
-export default {
+const meta: Meta<typeof {{COMPONENT_CLASS_NAME}}> = {
   title: '{{COMPONENT_CLASS_NAME}}',
-  decorators: [withKnobs],
-  component: {{COMPONENT_CLASS_NAME}}
+  component: {{COMPONENT_CLASS_NAME}},
+  excludeStories: /.*Data$/
 };
 
+export default meta;
+type Story = StoryObj<typeof {{COMPONENT_CLASS_NAME}}>;
+
 if (!window.PCore) {
   window.PCore = {};
 }
@@ -34,7 +37,7 @@ window.PCore.getUserApi = () => {
   };
 };
 
-export const Base{{COMPONENT_CLASS_NAME}} = () => {
+export const Base{{COMPONENT_CLASS_NAME}}: Story = args => {
 
   const props = {
     label: configProps.label,

package/src/tasks/create/templates/Launchpad/Widget/PAGE-CASE/index.tsx.mustache

@@ -1,6 +1,5 @@
 import { Fragment } from 'react';
-import { DateTimeDisplay, Card, CardHeader, CardContent, Flex } from '@pega/cosmos-react-core';
-import PropTypes from 'prop-types';
+import { DateTimeDisplay, Card, CardHeader, CardContent, Flex, withConfiguration } from '@pega/cosmos-react-core';
 import type { PConnFieldProps } from './PConnProps';
 
 // includes in bundle
@@ -21,12 +20,12 @@ interface {{COMPONENT_CLASS_NAME}}Props extends PConnFieldProps {
 
 // props passed in combination of props from property panel (config.json) and run time props from Constellation
 // any default values in config.pros should be set in defaultProps at bottom of this file
-export default function {{COMPONENT_CLASS_NAME}}(props: {{COMPONENT_CLASS_NAME}}Props) {
+function {{COMPONENT_CLASS_NAME}}(props: {{COMPONENT_CLASS_NAME}}Props) {
 
   const {
     getPConnect,
-    title,
-    label,
+    title = 'Create operator',
+    label = 'Create operator',
     createLabel,
     updateLabel
   } = props;
@@ -48,8 +47,22 @@ export default function {{COMPONENT_CLASS_NAME}}(props: {{COMPONENT_CLASS_NAME}}
       <CardContent>
 {{! pick new delimiters for mustache }} {{=<% %>=}}
       <Flex container={{ direction: 'row'}}>
-      <Operator label={_label} name={user} id={user} getPConnect={getPConnect} value={undefined} validatemessage='' hideLabel={false} readOnly={false} required={false} disabled={false} externalUser={undefined} />
-<%={{ }}=%>     {{! revert delimiters for mustache }}
+           <Operator
+              label={_label}
+              name={user}
+              id={user}
+              getPConnect={getPConnect}
+              value={undefined}
+              validatemessage=''
+              hideLabel={false}
+              readOnly={false}
+              required={false}
+              disabled={false}
+              externalUser={undefined}
+              testId=''
+              helperText=''
+              metaObj={undefined}
+            /><%={{ }}=%>     {{! revert delimiters for mustache }}
       {dateTimeValue && (
         <Fragment>
           {' '}
@@ -67,20 +80,8 @@ export default function {{COMPONENT_CLASS_NAME}}(props: {{COMPONENT_CLASS_NAME}}
 
 }
 
-{{COMPONENT_CLASS_NAME}}.defaultProps = {
-  "label": "Create operator",
-  "title": "Create operator",
-  createLabel: null,
-  updateLabel: null
-};
-
-{{COMPONENT_CLASS_NAME}}.propTypes = {
-  getPConnect: PropTypes.func.isRequired,
-  label: PropTypes.string,
-  title: PropTypes.string,
-  createLabel: PropTypes.string,
-  updateLabel: PropTypes.string
-};
+
+export default withConfiguration({{COMPONENT_CLASS_NAME}});
 
 // as objects are there in props, shallow comparision fails & re-rendering of comp happens even with
 // same key value pairs in obj. hence using custom comparison function on when to re-render

package/src/tasks/create/templates/Launchpad/Widget/PAGE-CASE/styles.ts.mustache

@@ -1,15 +1,8 @@
-
-// utilizing theming, comment out, if want individual style
-import styled from 'styled-components';
-import { Configuration } from '@pega/cosmos-react-core';
-
-export default styled(Configuration)``;
-
 // individual style, comment out above, and uncomment here and add styles
-// import styled, { css } from 'styled-components';
-//
-// export default styled.div(() => {
-//  return css`
-//    margin: 0px 0;
-//  `;
-// });
+import styled, { css } from 'styled-components';
+
+export default styled.div(() => {
+  return css`
+    margin: 0px 0;
+  `;
+});

package/src/tasks/delete-all/helper.js

@@ -143,9 +143,26 @@ export const deleteServerComponents = async (sourceOfComponents, answers) => {
     ({library, organization} = lpAnswers);
   }
 
-  const spinner = ora('Fetching components from server').start();
-  const data = await getComponentsFromServer(library, organization);
-  spinner.stop();
+
+  let spinner;
+  let data;
+  try {
+    spinner = ora('Fetching components from server').start();
+    data = await getComponentsFromServer(library, organization);
+    spinner.stop();
+  }
+  catch (error) {
+    spinner.stop();
+    if (error.indexOf("ECONNREFUSED") >= 0) {
+      console.log(`\n${chalk.bold.red('Error occurred in authentication. Please regenerate using authenticate')}`);
+    }
+    else {
+      console.log(`${chalk.bold.red(error)}`);
+    }
+
+    // spinner.stop();
+    process.exit(1);
+  }
 
 
   // const serverComponents = isLaunchpad ? data : data.pxResults;
@@ -185,7 +202,13 @@ export const deleteServerComponents = async (sourceOfComponents, answers) => {
         const componentsList = serverComponents.map((component) => `${component.name}~|~`);
         for (const componentToDelete of componentsList) {
           // eslint-disable-next-line no-await-in-loop
-          await deleteLaunchpadServerComponent(componentToDelete);
+          try {
+            await deleteLaunchpadServerComponent(componentToDelete);
+          }
+          catch (err) {
+            console.log(chalk.bold.red(err.toString()));
+            process.exit(1);
+          }
         }
       }
     } else {
@@ -253,7 +276,16 @@ export const deleteServerComponents = async (sourceOfComponents, answers) => {
           const [name, rulesetName, rulesetVersion] = componentToDelete.split('~|~');
           if (rulesetName === selectedComp.rulesetName && rulesetVersion === selectedComp.rulesetVersion) {
             // eslint-disable-next-line no-await-in-loop
-            await deleteInfinityServerComponent(componentToDelete);
+
+            try {
+              await deleteInfinityServerComponent(componentToDelete);
+            }
+            catch (err) {
+              console.log(chalk.bold.red(err.toString()));
+              process.exit(1);
+            }
+
+
           }
         }
       }

package/src/tasks/init/template/package.json.mustache

@@ -28,11 +28,11 @@
     "lint:styles": "stylelint \"./**/*.@(js|jsx|ts|tsx)\"",
     "lint:format": "prettier -c .",
     "lint:spelling": "cspell --no-progress **/*",
-    "test": "echo 'Running functional and unit tests will DELETE existing local components, proceed (y/n)?' && read proceed && if test \"$proceed\" = \"y\"; then npm run test:full; fi ",
-    "test:full": "npm run test:functional && npm run test:unit",
-    "test:functional": "npm run test:functional:local --prefix node_modules/@pega/custom-dx-components",
     "test-coverage": "node --input-type=module  --experimental-vm-modules node_modules/jest/bin/jest.js --runInBand node_modules/@pega/custom-dx-component --ci --testResultsProcessor='jest-junit' --coverage",
-    "test:unit": "npm run test:unit:local --prefix node_modules/@pega/custom-dx-components"
+    "_internal-test": "echo 'Running functional and unit tests will DELETE existing local components, proceed (y/n)?' && read proceed && if test \"$proceed\" = \"y\"; then npm run internal-test-full; fi ",
+    "_internal-test-full": "npm run _internal-test-functional && npm run _internal-test-unit",
+    "_internal-test-functional": "npm run test:functional:local --prefix node_modules/@pega/custom-dx-components",
+    "_internal-test-unit": "npm run test:unit:local --prefix node_modules/@pega/custom-dx-components"
   },
     "scriptsComments": {
     "build": "tsc -b tsconfig.build.json",
@@ -69,8 +69,8 @@
   },
   "devDependencies": {
     "@pega/configs": "^0.6.0",
-    "@pega/custom-dx-components": "0.23.35",
-    "@pega/pcore-pconnect-typedefs": "2.0.0",
+    "@pega/custom-dx-components": "0.23.37",
+    "@pega/pcore-pconnect-typedefs": "~2.1.1",
     "@storybook/addon-actions": "^7.6.7",
     "@storybook/addon-essentials": "^7.6.7",
     "@storybook/addon-knobs": "^7.0.2",

package/src/tasks/init/template/tasks.config.json

@@ -34,5 +34,6 @@
     "subtype": "",
     "description": "",
     "icon": ""
-  }
+  },
+  "usePromotedWebPack": false
 }

package/src/tasks/linter/index.js

@@ -15,12 +15,15 @@ export const lintComponent = async (dirToLint) => {
   const formatter = await eslint.loadFormatter("stylish");
   const resultText = formatter.format(results);
 
-  // 4. Output it.
-  // console.log(`lint resultText: ${resultText}`);
-  console.log(resultText);
-
   // Indicate linting is complete
   console.log(chalk.green(` linting complete...`));
+
+  if (resultText.length > 0 && resultText.indexOf("error") >= 0) {
+    throw new Error( "Lint:\n" + resultText );
+  }
+  else {
+    console.log(resultText);
+  }
 };
 
 // inpsired by https://eslint.org/docs/latest/developer-guide/nodejs-api

package/src/tasks/list/helper.js

@@ -102,6 +102,15 @@ async function fetchComponentsFromInfinity(apiUrl, headers) {
     })
       .then((response) => {
         status = response.status;
+        if (status === 404){
+          throw new Error('404: Server resource not found');
+        }
+        else if (status === 405){
+          throw new Error('405: Server method not allowed');
+        }
+        else if (status === 408){
+          throw new Error('408: Server timed out');
+        }
         return response.json();
       })
       .then((resp) => {
@@ -122,6 +131,7 @@ async function fetchComponentsFromInfinity(apiUrl, headers) {
 }
 
 async function fetchComponentsFromLaunchpad(apiUrl, headers) {
+
   const serverConfig = await getPegaServerConfig();
   // constructing urls for all type of components
   const urls = COMPONENT_SCHEMA.type.map((type) => apiUrl.replace('{type}', type.value));

package/src/tasks/list/index.js

@@ -1,5 +1,6 @@
 import inquirer from 'inquirer';
 import ora from 'ora';
+import chalk from 'chalk';
 
 import { showVersion, getPegaServerConfig } from '../../util.js';
 
@@ -35,9 +36,25 @@ const getList = async (sourceOfComponents, answers) => {
 
 
       const spinner = ora('Fetching components from server\n').start();
-      const data = await getComponentsFromServer(library, organization);
-      spinner.stop();
-      await displayServerComponents(data, answers);
+
+      try {
+        const data = await getComponentsFromServer(library, organization);
+        spinner.stop();
+        await displayServerComponents(data, answers);
+      }
+      catch (error) {
+
+        if (error.indexOf("ECONNREFUSED") >= 0) {
+          console.log(`\n${chalk.bold.red('Error occurred in authentication. Please regenerate using authenticate')}`);
+        }
+        else {
+          console.log(`${chalk.bold.red(error)}`);
+        }
+
+        // spinner.stop();
+        process.exit(1);
+      }
+
 
       break;
     }

package/src/tasks/promote-webpack/index.js

@@ -9,7 +9,8 @@ import inquirer from 'inquirer';
 import {
   convertIntoPascalCase,
   checkPathAccess,
-  showVersion
+  showVersion,
+  updateUseWebPackPromotion
 } from '../../util.js';
 
 import {
@@ -91,36 +92,12 @@ export const copyWebPack = async () => {
 
 }
 
-export const updateBuilder = async () => {
 
-  const builderIndexDir = fileURLToPath(new URL(BUILDER_INDEX_FILE_PATH, import.meta.url));
-
-  let indexFileJS = fs.readFileSync(builderIndexDir, { encoding: 'utf8' });
-
-  if (indexFileJS.length > 0) {
-    // comment out original webpack reference
-
-    // if not commented out, comment out
-    if (indexFileJS.indexOf("// import configFactory from './webpack.config.js';") < 0) {
-      indexFileJS = indexFileJS.replaceAll("import configFactory from './webpack.config.js';", "// import configFactory from './webpack.config.js';");
-    }
-
-    // uncomment out promoted webpack reference
-    indexFileJS = indexFileJS.replaceAll("// import configFactory from '../../../../../../../webpack.config.js';", "import configFactory from '../../../../../../../webpack.config.js';");
-
-    fs.writeFileSync(builderIndexDir, indexFileJS, { encoding: 'utf8' });
-
-    console.log(chalk.bold.green(`Bundler now using ${redirectFile}.`));
-    console.log(chalk.bold.green(`Utilize this file for webpack modifications.`));
-  }
-
-
-}
 
 export default async options => {
   showVersion();
 
   await copyWebPack();
-  await updateBuilder();
+  await updateUseWebPackPromotion(true);
 
 };

package/src/tasks/publish/helper.js

@@ -94,7 +94,7 @@ export const zipComponent = async componentKey => {
 
   const configContent = Buffer.from(fs.readFileSync(`${srcDirectory}/config.json`)).toString();
 
-  fs.rmdir('dist', { recursive: true }, err => {
+  fs.rm('dist', { recursive: true }, err => {
     if (err) {
       throw err;
     }

package/src/tasks/publish/index.js

@@ -9,7 +9,7 @@ import { getComponentDirectoryPath, showVersion } from '../../util.js';
 
 import { getPublishComponentQuestions, publishComponentToServer, zipComponent } from './helper.js';
 
-export default async (options) => {
+export default async options => {
   showVersion();
 
   let componentKey;
@@ -19,13 +19,15 @@ export default async (options) => {
 
   let doFetch = true;
 
-  if (options.params.length >= 8) {
+  if (options.params.length >= 7) {
     componentKey = options.params[3];
     const rulesetName = options.params[4];
     const rulesetVersion = options.params[5];
     const dBuild = options.params[6];
     devBuild = dBuild === 'Y' || dBuild === 'y' || dBuild === true || dBuild === 'true';
-    doFetch = false;
+
+    // eslint-disable-next-line no-unneeded-ternary,no-nested-ternary
+    doFetch = options.params.length >= 7 ? (options.params[7] === 'noFetch' ? false : true) : true;
 
     content = { componentKey, rulesetName, rulesetVersion, devBuild };
   } else {
@@ -79,7 +81,7 @@ export default async (options) => {
     }
   );
 
-  await tasks.run().catch((err) => {
+  await tasks.run().catch(err => {
     console.log(chalk.bold.red(err.toString()));
   });
   return true;

package/src/tasks/publish-all/helper.js

@@ -159,55 +159,67 @@ export const publishComponentToServer = async (data, doFetch) => {
           headers.Authorization = `${tokenType} ${accessToken}`;
         }
 
-        fetch(url, {
+
+        const response = await fetch(url, {
           method: 'POST',
           agent: getHttpsAgent(defaultPegaConfig),
-          headers,
+          headers: headers,
           body: JSON.stringify(apiBody)
-        })
-          .then(response => {
-            status = response.status;
-            return response.text();
-          })
-          .then(resp => {
-            const respData = isLaunchpad ? resp : JSON.parse(resp);
-            if (status === 401) {
-              throw new Error(
-                'Error occurred in authentication. Please regenerate using authenticate'
-              );
-              // console.log(accessTokenUri, refreshToken);
-              /* TODO - Handle refresh_token */
-            } else if (status === 200 || status === 201) {
-              console.log(
+        });
+
+        status = response.status;
+        if (!response.ok) {
+          if (status === 401) {
+            throw new Error(
+              'Error occurred in authentication. Please regenerate using authenticate'
+            );
+            // console.log(accessTokenUri, refreshToken);
+            /* TODO - Handle refresh_token */
+          }
+          else if (status === 404){
+            throw new Error('404: Server resource not found');
+          }
+          else if (status === 405){
+            throw new Error('405: Server method not allowed');
+          }
+          else if (status === 408){
+            throw new Error('408: Server timed out');
+          }
+          else if (response.status === 403) {
+            throw new Error(
+              'Error forbidden: User does not have privileges to Publish.'
+            );
+          }
+        }
+
+        try {
+          const respData = isLaunchpad ? await response.text() : await response.json();
+          console.log(
                 chalk.bold.green(`Success : ${isLaunchpad ? respData : respData.message}`)
+          );
+
+          if (status === 500) {
+            status = 999;
+            throw new Error(respData.message);
+          }
+        }
+        catch (err) {
+          if (status === 500) {
+              throw new Error(
+               `Error occurred in authentication. Please regenerate using authenticate`
               );
-            } else {
-              throw new Error(`${respData.message}`);
-            }
-          })
-          .catch(e => {
-            if (e.code === 'ECONNREFUSED') {
-              // eslint-disable-next-line prefer-promise-reject-errors
-              Promise.reject(
-                `${chalk.bold.red(
-                  'Error occurred in authentication. Please regenerate using authenticate'
-                )}`
-              );
-            } else if (status === 403) {
-              // eslint-disable-next-line prefer-promise-reject-errors
-              Promise.reject(
-                `${chalk.bold.red('Error forbidden: User does not have privileges to Publish.')}`
-              );
-            } else {
-              // eslint-disable-next-line prefer-promise-reject-errors
-              Promise.reject(`${chalk.bold.red(e)}`);
-            }
-          });
+          }
+          else if (status === 999) {
+            throw new Error (err);
+          }
+        }
+
       } else {
         throw new Error(`Error occurred in authentication. Please regenerate using authenticate`);
       }
     } catch (err) {
-      throw new Error(`Error occurred in authentication. Please regenerate using authenticate`);
+      // throw new Error(`Error occurred in authentication. Please regenerate using authenticate`);
+      throw new Error(err);
     }
   }
 };

package/src/tasks/publish-all/index.js

@@ -16,8 +16,8 @@ import {
 
 export function getCustomTasks(componentKey, sourceMap, devBuild, content, options) {
   const sDevBuild = devBuild ? '(dev build)' : '';
-  // eslint-disable-next-line no-unneeded-ternary
-  const doFetch = options.params.length >= 7 ? false : true;
+  // eslint-disable no-unneeded-ternary,no-nested-ternary
+  const doFetch = options.params.length >= 7 ? (options.params[6] === 'noFetch' ? false : true) : true;
   return new Listr(
     [
       {
@@ -26,13 +26,6 @@ export function getCustomTasks(componentKey, sourceMap, devBuild, content, optio
           await validate(componentKey);
         }
       },
-      {
-        title: `Bundle Component ${sDevBuild}`,
-        task: async () => {
-          await bundleComponent(componentKey, sourceMap, devBuild);
-        },
-        skip: () => (options.skipBundle ? 'Skipped bundling component' : undefined)
-      },
       {
         title: 'Lint component',
         task: async () => {
@@ -41,6 +34,13 @@ export function getCustomTasks(componentKey, sourceMap, devBuild, content, optio
           await lintComponent(targetDirectory);
         }
       },
+      {
+        title: `Bundle Component ${sDevBuild}`,
+        task: async () => {
+          await bundleComponent(componentKey, sourceMap, devBuild);
+        },
+        skip: () => (options.skipBundle ? 'Skipped bundling component' : undefined)
+      },
       {
         title: 'Zip Component',
         task: async () => {
@@ -55,9 +55,7 @@ export function getCustomTasks(componentKey, sourceMap, devBuild, content, optio
         }
       }
     ],
-    {
-      exitOnError: true
-    }
+    { concurrent: false, exitOnError: true }
   );
 }
 
@@ -76,7 +74,7 @@ export default async options => {
   let devBuild;
   let content;
 
-  if (options.params.length >= 7) {
+  if (options.params.length >= 6) {
     const rulesetName = options.params[3];
     const rulesetVersion = options.params[4];
     const dBuild = options.params[5];
@@ -94,16 +92,17 @@ export default async options => {
   }
 
   // eslint-disable-next-line no-shadow
-  for (const componentKey of components) {
+  for await (const componentKey of components) {
     console.log(chalk.bold.green(`Publishing: ${componentKey}`));
 
     const myCustomTasks = getCustomTasks(componentKey, sourceMap, devBuild, content, options);
 
-    // eslint-disable-next-line no-await-in-loop
-    await myCustomTasks.run().catch(err => {
+    try {
+      await myCustomTasks.run();
+    } catch (err) {
       console.log(chalk.bold.red(err.toString()));
       process.exit(1);
-    });
+    }
   }
 
   // once done, remove leftover directories

package/src/tasks/revert-webpack/index.js

@@ -8,7 +8,8 @@ import { up } from 'inquirer/lib/utils/readline.js';
 
 import {
   convertIntoPascalCase,
-  showVersion
+  showVersion,
+  updateUseWebPackPromotion
 } from '../../util.js';
 import {
   WEBPACK_CONFIG_JS_FILENAME,
@@ -37,33 +38,12 @@ export const renamePromotedWebPack = async () => {
 
 }
 
-export const revertBuilder = async () => {
 
-  const builderIndexDir = fileURLToPath(new URL(BUILDER_INDEX_FILE_PATH, import.meta.url));
-
-  let indexFileJS = fs.readFileSync(builderIndexDir, { encoding: 'utf8' });
-
-  if (indexFileJS.length > 0) {
-    // uncomment out original webpack reference
-    indexFileJS = indexFileJS.replaceAll("// import configFactory from './webpack.config.js';", "import configFactory from './webpack.config.js';");
-
-    // comment out promoted webpack reference
-    if (indexFileJS.indexOf("// import configFactory from '../../../../../../../webpack.config.js';") < 0) {
-      indexFileJS = indexFileJS.replaceAll("import configFactory from '../../../../../../../webpack.config.js';", "// import configFactory from '../../../../../../../webpack.config.js';");
-    }
-
-    fs.writeFileSync(builderIndexDir, indexFileJS, { encoding: 'utf8' });
-
-    console.log(chalk.bold.green(`Bundler reverted to using internal webpack.config.js.`));
-  }
-
-
-}
 
 export default async options => {
   showVersion();
 
   await renamePromotedWebPack();
-  await revertBuilder();
+  await updateUseWebPackPromotion(false);
 
 };