Skip to content

refactor: refactor setAgenticMode function to use named parameters for better readability

Rinku Crequested to merge
gitlab-community/gitlab-org/gitlab:fix/setAgenticMode-refactor into master

What does this MR do and why?

Refactor setAgenticMode function to use named parameters for better readability

References

issue: #560518

Screenshots or screen recordings

Before After 
export const setAgenticMode = (
    agenticMode = true, 
    saveCookie = false, 
    isEmbedded = false
) => {
  if (!isEmbedded) {
    duoChatGlobalState.isShown = !agenticMode;
    duoChatGlobalState.isAgenticChatShown = agenticMode;
  }

  if (saveCookie) {
    saveDuoAgenticModePreference(agenticMode);
  }
};
 
export const setAgenticMode = ({
  agenticMode = true,
  saveCookie = false,
  isEmbedded = false,
} = {}) => {
  if (!isEmbedded) {
    duoChatGlobalState.isShown = !agenticMode;
    duoChatGlobalState.isAgenticChatShown = agenticMode;
  }

  if (saveCookie) {
    saveDuoAgenticModePreference(agenticMode);
  }
};

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Rinku C

Merge request reports