✨ Add debug flag for analysis (#1601)

- Requested for improving analysis experience when debugging Signed-off-by: ibolton336 <ibolton@redhat.com>
konveyor · Dec 11, 2023 · e99029b · e99029b
1 parent 21b7afd
commit e99029b
Show file tree

Hide file tree

Showing 6 changed files with 55 additions and 6 deletions.
diff --git a/client/public/locales/en/translation.json b/client/public/locales/en/translation.json
@@ -495,6 +495,7 @@
     },
     "terms": {
       "advanced": "Advanced",
+      "advancedAnalysisDetails": "Enhanced analysis details",
       "analysisMode": "Analysis mode",
       "configureAnalysis": "Configure analysis",
       "customRules": "Custom rules",
@@ -519,6 +520,9 @@
     "title": {
       "advancedOptions": "Advanced options",
       "review": "Review analysis details"
+    },
+    "tooltip": {
+      "advancedAnalysisDetails": "Enable enhanced analysis details to get more information about the analysis."
     }
   }
 }
diff --git a/client/src/app/api/models.ts b/client/src/app/api/models.ts
@@ -324,6 +324,7 @@ export interface TaskData {
   tagger: {
     enabled: boolean;
   };
+  verbosity: number;
   mode: {
     binary: boolean;
     withDeps: boolean;

diff --git a/client/src/app/pages/applications/analysis-wizard/analysis-wizard.tsx b/client/src/app/pages/applications/analysis-wizard/analysis-wizard.tsx
@@ -53,6 +53,7 @@ const defaultTaskData: TaskData = {
   tagger: {
     enabled: true,
   },
+  verbosity: 0,
   mode: {
     binary: false,
     withDeps: false,
@@ -178,6 +179,7 @@ export const AnalysisWizard: React.FC<IAnalysisWizard> = ({
       branch: "",
       rootPath: "",
       autoTaggingEnabled: true,
+      advancedAnalysisEnabled: false,
     },
     resolver: yupResolver(allFieldsSchema),
     mode: "all",
@@ -227,6 +229,7 @@ export const AnalysisWizard: React.FC<IAnalysisWizard> = ({
       tasks: analyzableApplications.map((app: Application) => initTask(app)),
       data: {
         ...defaultTaskData,
+        verbosity: fieldValues.advancedAnalysisEnabled ? 1 : 0,
         tagger: {
           enabled: fieldValues.autoTaggingEnabled,
         },

diff --git a/client/src/app/pages/applications/analysis-wizard/review.tsx b/client/src/app/pages/applications/analysis-wizard/review.tsx
@@ -51,6 +51,7 @@ export const Review: React.FC<IReview> = ({ applications, mode }) => {
     customRulesFiles,
     excludedRulesTags,
     autoTaggingEnabled,
+    advancedAnalysisEnabled,
   } = watch();
 
   const hasIncludedPackages = withKnownLibs.includes("select");
@@ -192,11 +193,6 @@ export const Review: React.FC<IReview> = ({ applications, mode }) => {
             </List>
           </DescriptionListDescription>
         </DescriptionListGroup>
-        <DescriptionListGroup>
-          <DescriptionListTerm>
-            {t("wizard.terms.transactionReport")}
-          </DescriptionListTerm>
-        </DescriptionListGroup>{" "}
         <DescriptionListGroup>
           <DescriptionListTerm>
             {t("wizard.terms.autoTagging")}
@@ -207,6 +203,16 @@ export const Review: React.FC<IReview> = ({ applications, mode }) => {
               : t("wizard.terms.disabled")}
           </DescriptionListDescription>
         </DescriptionListGroup>
+        <DescriptionListGroup>
+          <DescriptionListTerm>
+            {t("wizard.terms.advancedAnalysisDetails")}
+          </DescriptionListTerm>
+          <DescriptionListDescription id="advanced-analysis-details">
+            {advancedAnalysisEnabled
+              ? t("wizard.terms.enabled")
+              : t("wizard.terms.disabled")}
+          </DescriptionListDescription>
+        </DescriptionListGroup>
       </DescriptionList>
     </>
   );

diff --git a/client/src/app/pages/applications/analysis-wizard/schema.ts b/client/src/app/pages/applications/analysis-wizard/schema.ts
@@ -157,6 +157,7 @@ const useCustomRulesStepSchema = (): yup.SchemaOf<CustomRulesStepValues> => {
 export interface OptionsStepValues {
   excludedRulesTags: string[];
   autoTaggingEnabled: boolean;
+  advancedAnalysisEnabled: boolean;
   selectedSourceLabels: TargetLabel[];
 }
 
@@ -165,6 +166,7 @@ const useOptionsStepSchema = (): yup.SchemaOf<OptionsStepValues> => {
   return yup.object({
     excludedRulesTags: yup.array().of(yup.string().defined()),
     autoTaggingEnabled: yup.bool().defined(),
+    advancedAnalysisEnabled: yup.bool().defined(),
     selectedSourceLabels: yup.array().of(
       yup.object().shape({
         name: yup.string().defined(),

diff --git a/client/src/app/pages/applications/analysis-wizard/set-options.tsx b/client/src/app/pages/applications/analysis-wizard/set-options.tsx
@@ -1,10 +1,13 @@
 import * as React from "react";
 import {
   Checkbox,
+  Flex,
+  FlexItem,
   Form,
   Text,
   TextContent,
   Title,
+  Tooltip,
 } from "@patternfly/react-core";
 import {
   SelectVariant,
@@ -25,14 +28,20 @@ import { getParsedLabel } from "@app/utils/rules-utils";
 import { DEFAULT_SELECT_MAX_HEIGHT } from "@app/Constants";
 import { useFetchTargets } from "@app/queries/targets";
 import defaultSources from "./sources";
+import { QuestionCircleIcon } from "@patternfly/react-icons";
 
 export const SetOptions: React.FC = () => {
   const { t } = useTranslation();
 
   const { watch, control, setValue } =
     useFormContext<AnalysisWizardFormValues>();
 
-  const { formLabels, excludedRulesTags, autoTaggingEnabled } = watch();
+  const {
+    formLabels,
+    excludedRulesTags,
+    autoTaggingEnabled,
+    advancedAnalysisEnabled,
+  } = watch();
 
   const [isSelectTargetsOpen, setSelectTargetsOpen] = React.useState(false);
   const [isSelectSourcesOpen, setSelectSourcesOpen] = React.useState(false);
@@ -254,6 +263,30 @@ export const SetOptions: React.FC = () => {
         id="enable-auto-tagging-checkbox"
         name="autoTaggingEnabled"
       />
+      <Flex>
+        <FlexItem>
+          <Checkbox
+            className={spacing.mtMd}
+            label={t("wizard.composed.enable", {
+              what: t("wizard.terms.advancedAnalysisDetails").toLowerCase(),
+            })}
+            isChecked={advancedAnalysisEnabled}
+            onChange={() =>
+              setValue("advancedAnalysisEnabled", !advancedAnalysisEnabled)
+            }
+            id="enable-advanced-analysis-details-checkbox"
+            name="advancedAnalysisDetailsEnabled"
+          />
+        </FlexItem>
+        <FlexItem>
+          <Tooltip
+            position="right"
+            content={t("wizard.tooltip.advancedAnalysisDetails")}
+          >
+            <QuestionCircleIcon className={spacing.mlSm} />
+          </Tooltip>
+        </FlexItem>
+      </Flex>
     </Form>
   );
 };