@pega/dx-component-builder-sdk 8.23.14 → 8.23.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pega/dx-component-builder-sdk",
-  "version": "8.23.14",
+  "version": "8.23.16",
   "description": "Utility for building custom UI components",
   "bin": "./bin/index.js",
   "scripts": {

package/src/cli.js

@@ -59,7 +59,7 @@ const parseArgumentsIntoOptions = rawArgs => {
     task: args._[0],
     git: true, // args['--git'] || false
     runInstall: true, // args['--install'] || false
-    skipBundle: false,
+    skipBundle: args['--skipBundle'] || false,
     params: rawArgs
   };
 };

package/src/tasks/build/helper.js

@@ -53,7 +53,7 @@ export const getBuildComponentQuestions = async (category) => {
   const defaultPegaConfig = await getPegaServerConfig();
 
   if (components.length == 0) {
-    console.log(chalk.bold.yellow(`No components to build`));
+    console.log(chalk.redBright("No components to build"));
     process.exit(1);
   }
 

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

@@ -47,7 +47,7 @@ export const getBuildComponentQuestions = async (category) => {
   const defaultPegaConfig = await getPegaServerConfig();
 
   if (components.length == 0) {
-    console.log(chalk.bold.yellow(`No components to build`));
+    console.log(chalk.redBright("No components to build"));
     process.exit(1);
   }
 

package/src/tasks/create/templates/SDK-React/field/Decimal/demo.stories.jsx.mustache

@@ -26,7 +26,8 @@ export const Base{{COMPONENT_CLASS_NAME}} = () => {
           return {
             updateFieldValue: (propName, theValue) => {
               setValue(theValue);
-            }
+            },
+            triggerFieldChange: () => {/* nothing */}
           };
         },
         getStateProps: () => {

package/src/tasks/create/templates/SDK-React/field/Decimal/index.tsx.mustache

@@ -41,7 +41,7 @@ export default function {{COMPONENT_CLASS_NAME}}(props: any) {
   const propName = pConn.getStateProps().value;
   const helperTextToDisplay = validatemessage || helperText;
 
-  const [decValue, setDecimalvalue] = useState();
+  const [decValue, setDecimalvalue] = useState("");
   const [theCurrDec, setCurrDec] = useState(".");
   const [theCurrSep, setCurrSep] = useState(",");
 

package/src/tasks/create/templates/SDK-React/field/Decimal-Currency/index.tsx.mustache

@@ -49,7 +49,7 @@ export default function {{COMPONENT_CLASS_NAME}}(props: any) {
     'data-test-id': testId
   };
 
-  const [currValue, setCurrValue] = useState();
+  const [currValue, setCurrValue] = useState("");
   const [theCurrSym, setCurrSym] = useState("$");
   const [theCurrDec, setCurrDec] = useState(".");
   const [theCurrSep, setCurrSep] = useState(",");

package/src/tasks/create/templates/SDK-React/template/PAGE/demo.stories.jsx.mustache

@@ -49,6 +49,7 @@ export const Base{{COMPONENT_CLASS_NAME}} = () => {
          if(config.type === 'AppAnnouncement') {
               return (
                 <AppAnnouncement
+                  key='announcement'
                   header='Announcements'
                   description={configProps.description}
                   image={configProps.image.replace(/ /g, '+')}

package/src/tasks/create/templates/SDK-React/widget/PAGE/index.tsx.mustache

@@ -62,16 +62,16 @@ export default function {{COMPONENT_CLASS_NAME}}(props) {
 }
 
 {{COMPONENT_CLASS_NAME}}.defaultProps = {
-  header: PropTypes.string,
-  description: PropTypes.string,
-  datasource: PropTypes.instanceOf(Object),
-  whatsnewlink: PropTypes.string
-};
-
-{{COMPONENT_CLASS_NAME}}.propTypes = {
   header: "",
   description: "",
   // image: "",
   datasource: [],
   whatsnewlink: ""
 };
+
+{{COMPONENT_CLASS_NAME}}.propTypes = {
+  header: PropTypes.string,
+  description: PropTypes.string,
+  datasource: PropTypes.instanceOf(Object),
+  whatsnewlink: PropTypes.string
+};

package/src/tasks/publish/helper.js

@@ -55,6 +55,12 @@ export const getBuildCategory = async (options) => {
 
 export const getPublishComponentQuestions = async (category) => {
   const components = await getComponentsObj(true, category);
+
+  if (components.length == 0) {
+    console.log(chalk.redBright("No components to publish"));
+    process.exit(1);
+  }
+
   const defaultPegaConfig = await getPegaServerConfig();
 
   return [
@@ -241,7 +247,16 @@ export const publishComponentToServer = async data => {
             /* TODO - Handle refresh_token */
           } else if (status == 200) {
             console.log(chalk.bold.green(`Success : ${respData.message}`));
-          } else {
+          }
+          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 {
             throw new Error(`${respData.message}`);
           }
         })

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

@@ -54,8 +54,8 @@ export const getBuildCategory = async (options) => {
   ];
 };
 
-export const getPublishComponentQuestions = async () => {
-  const components = await getComponentsObj();
+export const getPublishComponentQuestions = async (category) => {
+
   const defaultPegaConfig = await getPegaServerConfig();
 
   return [
@@ -238,7 +238,16 @@ export const publishComponentToServer = async data => {
             /* TODO - Handle refresh_token */
           } else if (status == 200) {
             console.log(chalk.bold.green(`Success : ${respData.message}`));
-          } else {
+          }
+          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 {
             throw new Error(`${respData.message}`);
           }
         })

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

@@ -154,6 +154,13 @@ export default async options => {
   }
 
   const components = await getComponentsObj(false, category);
+
+  // end if find no componets
+  if (components.length == 0) {
+    console.log(chalk.redBright("No components to publish"));
+    process.exit(1);
+  }
+
   const questions = await getPublishComponentQuestions(category);
   const answers = await inquirer.prompt(questions);
   const {sourceMap, devBuild } = answers;
@@ -180,6 +187,7 @@ export default async options => {
 
   }
   else {
+
     for (const component of components) {
       const componentKey = component.value;
       const componentKeyName = component.name;

package/src/tasks/validator/index.js

@@ -9,7 +9,7 @@ import {
   COMPONENT_SCHEMA,
   CATEGORY_CUSTOM,
   CATEGORY_OVERRIDE } from '../../constants.js';
-  
+
 import {
   getComponentDirectoryPath,
   getComponents,
@@ -34,7 +34,7 @@ const allowedOverrideSubtypes = [
     ...COMPONENT_SCHEMA.subtype.field.map(({ value }) => value),
     ...COMPONENT_SCHEMA.subtype.template.map(({ value }) => value),
     ...COMPONENT_SCHEMA.subtype.widget.map(({ value }) => value)
-  ]), "ACTION", "DATA_CAPTURE", "SUMMARY", "CASEVIEW", "DATAVIEW", "LIST"
+  ]), "ACTION", "DATA_CAPTURE", "SUMMARY", "CASEVIEW", "DATAVIEW", "LIST", "TAB"
 ];
 
 const schema = {
@@ -312,7 +312,7 @@ const overrideSchema = {
 };
 
 const validateSchema = ajv.compile(schema);
-const validateOverideSchema = ajv.compile(overrideSchema);
+const validateOverrideSchema = ajv.compile(overrideSchema);
 
 
 
@@ -352,7 +352,7 @@ export default async (option, category = CATEGORY_CUSTOM) => {
     }
     else if (category === CATEGORY_OVERRIDE) {
       // for now, if now file just end
-      console.log(`${chalk.bold.yellow('Config file not availabe for: ' + componentToValidate)}`);
+      console.log(`${chalk.bold.yellow('Config file not available for: ' + componentToValidate)}`);
       return;
     }
   }
@@ -360,7 +360,7 @@ export default async (option, category = CATEGORY_CUSTOM) => {
   //const data = JSON.parse(fs.readFileSync(path.resolve(configFile), 'utf8'));
   const data = JSON.parse(configData);
 
-  const valid = (category === CATEGORY_OVERRIDE) ? validateOverideSchema(data) : validateSchema(data);
+  const valid = (category === CATEGORY_OVERRIDE) ? validateOverrideSchema(data) : validateSchema(data);
 
   if (valid) {
 
@@ -383,6 +383,12 @@ export default async (option, category = CATEGORY_CUSTOM) => {
 
     console.log(`${chalk.bold.green(componentToValidate)} schema is valid`);
   } else {
-    throw new Error(`Invalid: ${ajv.errorsText(validateSchema.errors)}`);
+    if (category === CATEGORY_OVERRIDE) {
+      throw new Error(`Invalid: ${ajv.errorsText(validateOverrideSchema.errors)}`);
+    }
+    else {
+      throw new Error(`Invalid: ${ajv.errorsText(validateSchema.errors)}`);
+    }
+
   }
 };

package/src/util.js

@@ -120,6 +120,8 @@ export const deleteServerComponent = async componentKey => {
   try {
     const OauthData = fs.readFileSync(TOKEN_PATH, 'utf8');
 
+    let status = 500;
+
     if (OauthData) {
       const {
         access_token: accessToken,
@@ -133,7 +135,10 @@ export const deleteServerComponent = async componentKey => {
           Authorization: `${tokenType} ${accessToken}`
         }
       })
-        .then(response => response.text())
+        .then(response => {
+          status = response.status;
+          return response.text();
+        })
         .then(resp => {
           const respData = JSON.parse(resp);
           if (respData.status == 200) {
@@ -142,7 +147,21 @@ export const deleteServerComponent = async componentKey => {
             throw new Error(`${respData.message}`);
           }
         })
-        .catch(e => Promise.reject(`${chalk.bold.red(e)}`));
+        .catch(e => {
+          if (status === 403) {
+            // eslint-disable-next-line prefer-promise-reject-errors
+            Promise.reject(
+              `${chalk.bold.red(
+                'Error forbidden: User does not have privileges to Delete.'
+              )}`
+            );
+          }
+          else {
+            // eslint-disable-next-line prefer-promise-reject-errors
+            Promise.reject(`${chalk.bold.red(e)}`)
+          }
+
+        });
     }
   } catch (error) {
     console.log(`\n${chalk.bold.red(error)}`);