console messaging for config-manager by devintrivir · Pull Request #100 · trivir/frodo-cli

devintrivir · 2026-04-23T21:06:00Z

No description provided.

phalestrivir

Missing some indicators, and a few wording things that I mentioned, but mostly looks good. Keep in mind that since this PR was created there might've been some new commands added, so you will want to make sure to rebase and get indicators added to those as well. For future PRs though I'll make sure we include indicators for each command that we add.

phalestrivir · 2026-07-17T15:21:55Z

 export async function configManagerImportCookieDomains(): Promise<boolean> {
  try {
    const getFile = getFilePath('cookie-domains/cookie-domains.json');
    const readFile = fs.readFileSync(getFile, 'utf-8');
    const importData = JSON.parse(readFile);
    await updateCookieDomains(importData);
    return true;
  } catch (error) {
-    printError(error, `Error importing custom domains`);
+    printError(error, `Error importing cookie domains`);
  }


We should add indicator here as well

phalestrivir · 2026-07-17T15:23:29Z

 export async function configManagerImportEmailProvider(): Promise<boolean> {
  try {
    const filePath = getFilePath('email-provider');
    const fileData = fs.readFileSync(
      `${filePath}/external.email.json`,
      'utf-8'
    );
    let importData = JSON.parse(fileData);
    importData = { idm: { [importData._id]: importData } };
    await config.importConfigEntities(importData);
    return true;
  } catch (error) {
    printError(error);
  }
  return false;
 }


We should add indicator here as well

phalestrivir · 2026-07-17T15:24:32Z

 export async function configManagerImportEmailTemplates(
  templateName?: string
 ): Promise<boolean> {
  try {
    const templateDir = getFilePath('email-templates');
    const templateFiles = fs.readdirSync(templateDir);
    const importTemplateData = { emailTemplate: {} };

    for (const templateFile of templateFiles) {
      const templateJsonPath = getFilePath(
        `email-templates/${templateFile}/${templateFile}.json`
      );
      const readTemplate = fs.readFileSync(templateJsonPath, 'utf8');
      const importData = JSON.parse(readTemplate);
      const id = importData._id;

      if (templateName && id !== `emailTemplate/${templateName}`) {
        continue;
      }

      if (importData.html && typeof importData.html === 'object') {
        for (const locale in importData.html) {
          const htmlFilePath = getFilePath(
            `email-templates/${templateFile}/${importData.html[locale].file}`
          );
          importData.html[locale] = fs.readFileSync(htmlFilePath, 'utf8');
        }
      }
      if (importData.message && typeof importData.message === 'object') {
        for (const locale in importData.message) {
          const messageFilePath = getFilePath(
            `email-templates/${templateFile}/${importData.message[locale].file}`
          );
          importData.message[locale] = fs.readFileSync(messageFilePath, 'utf8');
        }
      }
      if (importData.styles && typeof importData.styles === 'object') {
        const stylesFilePath = getFilePath(
          `email-templates/${templateFile}/${importData.styles.file}`
        );
        importData.styles = fs.readFileSync(stylesFilePath, 'utf8');
      }
      importTemplateData.emailTemplate[id.replace('emailTemplate/', '')] =
        importData;
    }
    await importEmailTemplates(importTemplateData);
    return true;
  } catch (error) {
    printError(error, `Error importing email templates to files`);
  }
  return false;
 }


We should have progress indicator here as well

phalestrivir · 2026-07-17T15:25:00Z

 export async function configManagerImportEndpoints(
  endpointName?: string
 ): Promise<boolean> {
  try {
    const endpointsDir = getFilePath(`endpoints`);
    const endpointsFiles = fs.readdirSync(endpointsDir);
    const importEndpointData = { idm: {} };
    for (const endpointsFile of endpointsFiles) {
      const jsonFilePath = getFilePath(
        `endpoints/${endpointsFile}/${endpointsFile}.json`
      );
      const readJsonEndpoint = fs.readFileSync(jsonFilePath, 'utf8') as any;
      const importData = JSON.parse(readJsonEndpoint) as any;
      const id = importData._id;

      if (endpointName && id !== `endpoint/${endpointName}`) {
        continue;
      }
      if (importData.file) {
        const scriptPath = getFilePath(
          `endpoints/${endpointsFile}/${importData.file}`
        );
        importData.source = fs.readFileSync(scriptPath, 'utf8');
        delete importData.file;
      }

      importEndpointData.idm[id] = importData;
    }
    await importConfigEntities(importEndpointData);
    return true;
  } catch (error) {
-    printError(error, `Error importing config entity endpoints`);
+    printError(error, `Error importing endpoints`);
  }
  return false;
 }


We should add indicator here as well

phalestrivir · 2026-07-17T15:27:42Z

 export async function configManagerImportInternalRoles(
  internalRoleName?: string
 ): Promise<boolean> {
  try {
    const internalRolesDir = getFilePath('internal-roles');
    const internalRolesFiles = fs.readdirSync(internalRolesDir);
    const importData = { internalRole: {} };
    for (const internalRolesFile of internalRolesFiles) {
      const filePath = getFilePath(`internal-roles/${internalRolesFile}`);
      const readFile = fs.readFileSync(filePath, 'utf-8');
      const roleData = JSON.parse(readFile);
      if (internalRoleName && roleData.name !== internalRoleName) {
        continue;
      }
      importData.internalRole[roleData._id] = roleData;
    }
    await importInternalRoles(importData);
    return true;
  } catch (error) {
-    printError(error, `Error importing internal roles to files`);
+    printError(error, `Error importing internal roles`);
  }
  return false;
 }
