PromucFlow_constructor/app/client/src/sagas/PostEvaluationSagas.ts

408 lines
13 KiB
TypeScript
Raw Normal View History

import {
ENTITY_TYPE,
Log,
PLATFORM_ERROR,
Severity,
} from "entities/AppsmithConsole";
import { DataTree, UnEvalTree } from "entities/DataTree/dataTreeFactory";
2021-07-20 10:02:56 +00:00
import {
DataTreeDiff,
DataTreeDiffEvent,
getDataTreeWithoutPrivateWidgets,
2021-07-20 10:02:56 +00:00
getEntityNameAndPropertyPath,
isAction,
isJSAction,
2021-07-20 10:02:56 +00:00
isWidget,
} from "@appsmith/workers/Evaluation/evaluationUtils";
2021-07-20 10:02:56 +00:00
import {
EvalError,
EvalErrorTypes,
EvaluationError,
getEvalErrorPath,
getEvalValuePath,
} from "utils/DynamicBindingUtils";
import { find, get, some } from "lodash";
import LOG_TYPE from "entities/AppsmithConsole/logtype";
2021-07-20 10:02:56 +00:00
import { put, select } from "redux-saga/effects";
import { AnyReduxAction } from "@appsmith/constants/ReduxActionConstants";
feat: Renamed design system package (#19854) ## Description This PR includes changes for renaming design system package. Since we are building new package for the refactored design system components, the old package is renaming to design-system-old. Fixes #19536 ## Type of change - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) ## How Has This Been Tested? - Manual - Jest - Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-23 03:50:47 +00:00
import { Toaster, Variant } from "design-system-old";
import AppsmithConsole from "utils/AppsmithConsole";
2021-07-20 10:02:56 +00:00
import * as Sentry from "@sentry/react";
import AnalyticsUtil from "utils/AnalyticsUtil";
2021-07-20 10:02:56 +00:00
import {
createMessage,
ERROR_EVAL_ERROR_GENERIC,
2021-11-10 14:43:21 +00:00
JS_OBJECT_BODY_INVALID,
VALUE_IS_INVALID,
JS_EXECUTION_FAILURE,
refactor: admin settings (#9906) * refactor admin settings feature * separated save-restart bar to separate component * created new CE dir to facilitate code split * created separate ee dir and exporting everything we have in ce file. * little mod * minor fix * splitting settings types config * using object literals for category types instead of enums * CE: support use of component for each category * minor style fix * authentication page UI changes implemented * github signup doc url added back * removed comments * routing updates * made subcategories listing in left pane optional * added muted saml to auth listing * added breadcrumbs and enabled button * created separate component for auth page and auth config * added callout and disconnect components * updated breadcrumbs component * minor updates to common components * updated warning callout and added icon * ce: test cases fixed * updated test file name * warning banner callout added on auth page * updated callout banner for form login * CE: Split config files * CE: moved the window declaration in EE file as its dependency will be updated in EE * CE: Splitting ApiConstants and SocialLogin constants * CE: split login page * CE: moved getSocialLoginButtonProps func to EE file as it's dependencies will be updated in EE * added key icon * CE: created a factory class to share social auths list * Minor style fix for social btns * Updated the third party auth styles * Small fixes to styling * ce: splitting forms constants * breadcrumbs implemented for all pages in admin settings * Settings breadcrumbs separated * splitted settings breadcrumbs between ce and ee * renamed default import * minor style fix * added login form config. * updated login/signup pages to use form login disabled config * removed common functionality outside * implemented breadcrumb component from scratch without using blueprint * removed unwanted code * Small style update * updated breadcrumb categories file name and breadcrumb icon * added cypress tests for admin settings auth page * added comments * update locator for upgrade button * added link for intercom on upgrade button * removed unnecessary file * minor style fix * style fix for auth option cards * split messages constant * fixed imports for message constants splitting. * added message constants * updated unit test cases * fixed messages import in cypress index * fixed messages import again, cypress fails to read re-exported objs. * added OIDC auth method on authentication page * updated import statements from ee to @appsmith * removed dead code * updated read more link UI * PR comments fixes * some UI fixes * used color and fonts from theme * fixed some imports * fixed some imports * removed warning imports * updated OIDC logo and auth method desc copies * css changes * css changes * css changes * updated cypress test for breadcrumb * moved callout component to ads as calloutv2 * UI changes for form fields * updated css for spacing between form fields * added sub-text on auth pages * added active class for breadcrumb item * added config for disable signup toggle and fixed UI issues of restart banner * fixed admin settings page bugs * assigned true as default state for signup * fixed messages import statements * updated code for PR comments related suggestions * reverted file path change in cypress support * updated cypress test * updated cypress test Co-authored-by: Ankita Kinger <ankita@appsmith.com>
2022-02-11 18:08:46 +00:00
} from "@appsmith/constants/messages";
2021-07-20 10:02:56 +00:00
import log from "loglevel";
import { AppState } from "@appsmith/reducers";
2021-07-20 10:02:56 +00:00
import { getAppMode } from "selectors/applicationSelectors";
import { APP_MODE } from "entities/App";
2021-07-20 10:02:56 +00:00
import { dataTreeTypeDefCreator } from "utils/autocomplete/dataTreeTypeDefCreator";
import CodemirrorTernService from "utils/autocomplete/CodemirrorTernService";
import { selectFeatureFlags } from "selectors/usersSelectors";
import FeatureFlags from "entities/FeatureFlags";
import { JSAction } from "entities/JSCollection";
import { isWidgetPropertyNamePath } from "utils/widgetEvalUtils";
2021-07-20 10:02:56 +00:00
const getDebuggerErrors = (state: AppState) => state.ui.debugger.errors;
function logLatestEvalPropertyErrors(
currentDebuggerErrors: Record<string, Log>,
2021-07-20 10:02:56 +00:00
dataTree: DataTree,
evaluationOrder: Array<string>,
) {
const errorsToAdd = [];
const errorsToDelete = [];
const updatedDebuggerErrors: Record<string, Log> = {
2021-07-20 10:02:56 +00:00
...currentDebuggerErrors,
};
for (const evaluatedPath of evaluationOrder) {
const { entityName, propertyPath } = getEntityNameAndPropertyPath(
evaluatedPath,
);
const entity = dataTree[entityName];
if (isWidget(entity) || isAction(entity) || isJSAction(entity)) {
if (entity.logBlackList && propertyPath in entity.logBlackList) {
2021-07-20 10:02:56 +00:00
continue;
}
const allEvalErrors: EvaluationError[] = get(
2021-07-20 10:02:56 +00:00
entity,
getEvalErrorPath(evaluatedPath, {
fullPath: false,
isPopulated: false,
}),
2021-07-20 10:02:56 +00:00
[],
);
const evaluatedValue = get(
2021-07-20 10:02:56 +00:00
entity,
getEvalValuePath(evaluatedPath, {
isPopulated: false,
fullPath: false,
}),
2021-07-20 10:02:56 +00:00
);
const evalErrors: EvaluationError[] = [];
const evalWarnings: EvaluationError[] = [];
for (const err of allEvalErrors) {
if (err.severity === Severity.WARNING) {
evalWarnings.push(err);
}
if (err.severity === Severity.ERROR) {
evalErrors.push(err);
}
}
2021-07-20 10:02:56 +00:00
const idField = isWidget(entity) ? entity.widgetId : entity.actionId;
fix: jsObject updates (#19303) ## Description This is a hotfix change for the issue created due to a data tree split where JSObject has `name` as a variable and a function is modified to change behavior from sync to async or vice versa. Fixes https://github.com/appsmithorg/appsmith/issues/19310 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - Manual - Cypress ### Test Plan - [ ] https://github.com/appsmithorg/TestSmith/issues/2129 ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [x] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-03 07:49:33 +00:00
const nameField = isWidget(entity) ? entity.widgetName : entityName;
2021-07-20 10:02:56 +00:00
const entityType = isWidget(entity)
? ENTITY_TYPE.WIDGET
: isAction(entity)
? ENTITY_TYPE.ACTION
: ENTITY_TYPE.JSACTION;
const debuggerKeys = [
{
key: `${idField}-${propertyPath}`,
errors: evalErrors,
},
{
key: `${idField}-${propertyPath}-warning`,
errors: evalWarnings,
isWarning: true,
},
];
2021-07-20 10:02:56 +00:00
for (const { errors, isWarning, key: debuggerKey } of debuggerKeys) {
// if dataTree has error but debugger does not -> add
// if debugger has error and data tree has error -> update error
// if debugger has error but data tree does not -> remove
// if debugger or data tree does not have an error -> no change
if (errors.length) {
// TODO Rank and set the most critical error
// const error = evalErrors[0];
// Reformatting eval errors here to a format usable by the debugger
const errorMessages = errors.map((e) => {
// Error format required for the debugger
return {
message: e.errorMessage,
type: e.errorType,
};
});
2021-07-20 10:02:56 +00:00
const analyticsData = isWidget(entity)
? {
widgetType: entity.type,
}
: {};
const logPropertyPath = !isJSAction(entity)
? propertyPath
: entityName;
// Add or update
if (
!isJSAction(entity) ||
(isJSAction(entity) && propertyPath === "body")
) {
errorsToAdd.push({
payload: {
id: debuggerKey,
logType: isWarning
? LOG_TYPE.EVAL_WARNING
: LOG_TYPE.EVAL_ERROR,
// Unless the intention is to change the message shown in the debugger please do not
// change the text shown here
text: isJSAction(entity)
? createMessage(JS_OBJECT_BODY_INVALID)
: createMessage(VALUE_IS_INVALID, propertyPath),
messages: errorMessages,
source: {
id: idField,
name: nameField,
type: entityType,
propertyPath: logPropertyPath,
},
state: {
[logPropertyPath]: evaluatedValue,
},
analytics: analyticsData,
},
severity: isWarning ? Severity.WARNING : Severity.ERROR,
});
}
} else if (debuggerKey in updatedDebuggerErrors) {
errorsToDelete.push({ id: debuggerKey });
}
2021-07-20 10:02:56 +00:00
}
}
}
// Add and delete errors from debugger
AppsmithConsole.addErrors(errorsToAdd);
AppsmithConsole.deleteErrors(errorsToDelete);
2021-07-20 10:02:56 +00:00
}
export function* evalErrorHandler(
errors: EvalError[],
dataTree?: DataTree,
evaluationOrder?: Array<string>,
): any {
if (dataTree && evaluationOrder) {
const currentDebuggerErrors: Record<string, Log> = yield select(
2021-07-20 10:02:56 +00:00
getDebuggerErrors,
);
// Update latest errors to the debugger
logLatestEvalPropertyErrors(
2021-07-20 10:02:56 +00:00
currentDebuggerErrors,
dataTree,
evaluationOrder,
);
}
errors.forEach((error) => {
switch (error.type) {
case EvalErrorTypes.CYCLICAL_DEPENDENCY_ERROR: {
if (error.context) {
// Add more info about node for the toast
const { dependencyMap, diffs, entityType, node } = error.context;
2021-07-20 10:02:56 +00:00
Toaster.show({
text: `${error.message} Node was: ${node}`,
variant: Variant.danger,
});
AppsmithConsole.error({
text: `${error.message} Node was: ${node}`,
});
// Send the generic error message to sentry for better grouping
Sentry.captureException(new Error(error.message), {
tags: {
node,
entityType,
},
extra: {
dependencyMap,
diffs,
},
2021-07-20 10:02:56 +00:00
// Level is warning because it could be a user error
level: Sentry.Severity.Warning,
});
// Log an analytics event for cyclical dep errors
AnalyticsUtil.logEvent("CYCLICAL_DEPENDENCY_ERROR", {
node,
entityType,
// Level is warning because it could be a user error
level: Sentry.Severity.Warning,
});
}
break;
}
case EvalErrorTypes.EVAL_TREE_ERROR: {
Toaster.show({
text: createMessage(ERROR_EVAL_ERROR_GENERIC),
variant: Variant.danger,
});
break;
}
case EvalErrorTypes.BAD_UNEVAL_TREE_ERROR: {
Sentry.captureException(error);
break;
}
case EvalErrorTypes.EVAL_PROPERTY_ERROR: {
log.debug(error);
break;
}
case EvalErrorTypes.CLONE_ERROR: {
Sentry.captureException(new Error(error.message), {
extra: {
request: error.context,
},
});
break;
}
case EvalErrorTypes.PARSE_JS_ERROR: {
Toaster.show({
text: `${error.message} at: ${error.context?.entity.name}`,
variant: Variant.danger,
});
AppsmithConsole.error({
text: `${error.message} at: ${error.context?.propertyPath}`,
});
break;
}
case EvalErrorTypes.EXTRACT_DEPENDENCY_ERROR: {
Sentry.captureException(new Error(error.message), {
extra: error.context,
});
break;
}
2021-07-20 10:02:56 +00:00
default: {
log.debug(error);
}
}
});
}
export function* logSuccessfulBindings(
unEvalTree: UnEvalTree,
2021-07-20 10:02:56 +00:00
dataTree: DataTree,
evaluationOrder: string[],
isCreateFirstTree: boolean,
2021-07-20 10:02:56 +00:00
) {
const appMode: APP_MODE | undefined = yield select(getAppMode);
2021-07-20 10:02:56 +00:00
if (appMode === APP_MODE.PUBLISHED) return;
if (!evaluationOrder) return;
if (isCreateFirstTree) {
// we only aim to log binding success which were added by user
// for first evaluation, bindings are not added by user hence skipping it.
return;
}
2021-07-20 10:02:56 +00:00
evaluationOrder.forEach((evaluatedPath) => {
const { entityName, propertyPath } = getEntityNameAndPropertyPath(
evaluatedPath,
);
const entity = dataTree[entityName];
if (isAction(entity) || isWidget(entity)) {
const unevalValue = get(unEvalTree, evaluatedPath);
2021-07-20 10:02:56 +00:00
const entityType = isAction(entity) ? entity.pluginType : entity.type;
const isABinding = find(entity.dynamicBindingPathList, {
2021-07-20 10:02:56 +00:00
key: propertyPath,
});
2021-07-20 10:02:56 +00:00
const logBlackList = entity.logBlackList;
const errors: EvaluationError[] = get(
2021-07-20 10:02:56 +00:00
dataTree,
getEvalErrorPath(evaluatedPath),
[],
) as EvaluationError[];
const hasErrors = errors.length > 0;
2021-07-20 10:02:56 +00:00
if (isABinding && !hasErrors && !(propertyPath in logBlackList)) {
AnalyticsUtil.logEvent("BINDING_SUCCESS", {
unevalValue,
entityType,
propertyPath,
});
}
}
});
}
export function* postEvalActionDispatcher(actions: Array<AnyReduxAction>) {
2021-07-20 10:02:56 +00:00
for (const action of actions) {
yield put(action);
}
}
// We update the data tree definition after every eval so that autocomplete
// is accurate
2021-07-20 10:02:56 +00:00
export function* updateTernDefinitions(
dataTree: DataTree,
updates: DataTreeDiff[],
isCreateFirstTree: boolean,
fix: Show JS Function data in autocompletion hints (#19811) ## Description This PR adds JS function data to autocompletion hints Fixes #15909 <img width="278" alt="Screenshot 2023-01-16 at 20 35 55" src="https://user-images.githubusercontent.com/46670083/212754461-68844350-5d23-4b50-af1f-675b7719dc49.png"> ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-17 07:12:16 +00:00
jsData: Record<string, unknown> = {},
2021-07-20 10:02:56 +00:00
) {
const shouldUpdate: boolean =
isCreateFirstTree ||
some(updates, (update) => {
if (update.event === DataTreeDiffEvent.NEW) return true;
if (update.event === DataTreeDiffEvent.DELETE) return true;
if (update.event === DataTreeDiffEvent.EDIT) return false;
const { entityName } = getEntityNameAndPropertyPath(
update.payload.propertyPath,
);
const entity = dataTree[entityName];
if (!entity || !isWidget(entity)) return false;
return isWidgetPropertyNamePath(entity, update.payload.propertyPath);
2021-07-20 10:02:56 +00:00
});
if (!shouldUpdate) return;
const start = performance.now();
// remove private widgets from dataTree used for autocompletion
const treeWithoutPrivateWidgets = getDataTreeWithoutPrivateWidgets(dataTree);
const featureFlags: FeatureFlags = yield select(selectFeatureFlags);
const { def, entityInfo } = dataTreeTypeDefCreator(
treeWithoutPrivateWidgets,
!!featureFlags.JS_EDITOR,
fix: Show JS Function data in autocompletion hints (#19811) ## Description This PR adds JS function data to autocompletion hints Fixes #15909 <img width="278" alt="Screenshot 2023-01-16 at 20 35 55" src="https://user-images.githubusercontent.com/46670083/212754461-68844350-5d23-4b50-af1f-675b7719dc49.png"> ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-17 07:12:16 +00:00
jsData,
);
CodemirrorTernService.updateDef("DATA_TREE", def, entityInfo);
const end = performance.now();
log.debug("Tern", { updates });
log.debug("Tern definitions updated took ", (end - start).toFixed(2));
2021-07-20 10:02:56 +00:00
}
export function* handleJSFunctionExecutionErrorLog(
collectionId: string,
collectionName: string,
action: JSAction,
errors: any[],
) {
errors.length
? AppsmithConsole.addErrors([
{
payload: {
id: `${collectionId}-${action.id}`,
logType: LOG_TYPE.EVAL_ERROR,
text: `${createMessage(JS_EXECUTION_FAILURE)}: ${collectionName}.${
action.name
}`,
messages: errors.map((error) => ({
message: error.errorMessage || error.message,
type: PLATFORM_ERROR.JS_FUNCTION_EXECUTION,
subType: error.errorType,
})),
source: {
id: action.id,
name: `${collectionName}.${action.name}`,
type: ENTITY_TYPE.JSACTION,
propertyPath: `${collectionName}.${action.name}`,
},
},
},
])
: AppsmithConsole.deleteErrors([{ id: `${collectionId}-${action.id}` }]);
}