♻️ Improved repository list argument merging for multiple entity types (#438)

tdrive/backend/node/src/cli/cmds/search_cmds/index_all.ts

@@ -15,6 +15,7 @@ import { EntityTarget, SearchServiceAPI } from "../../../core/platform/services/
 import CompanyUser, { TYPE as CompanyUserTYPE } from "../../../services/user/entities/company_user";
 import runWithPlatform from "../../lib/run_with_platform";
 import SearchRepository from "src/core/platform/services/search/repository";
+import parseYargsCommaSeparatedStringArray from "../../utils/yargs_comma_array";
 
 type Options = {
   spinner: ora.Ora;
@@ -180,22 +181,12 @@ const command: yargs.CommandModule<unknown, unknown> = {
     },
   },
   handler: async argv => {
-    const repositoryArg = argv[repositoryArgumentName];
+    const repositories = parseYargsCommaSeparatedStringArray(
-    const repositories =
+      argv[repositoryArgumentName] as string /* ignore typechecker */,
-      typeof repositoryArg === "string" ? [repositoryArg] : (repositoryArg as [string]);
+    );
-
-    function* eachOnlyOnce<T>(list: Iterable<T>): Iterable<T> {
-      const seen = new Set<T>();
-      for (const item of list) {
-        if (seen.has(item)) continue;
-        seen.add(item);
-        yield item;
-      }
-    }
-
     runWithPlatform("Re-index", async ({ spinner, platform }) => {
       try {
-        for (const repositoryName of eachOnlyOnce(repositories))
+        for (const repositoryName of repositories)
           await RepositoryNameToCTOR.get(repositoryName)(platform, {
             spinner,
             repairEntities: !!argv.repairEntities,

tdrive/backend/node/src/cli/utils/yargs_comma_array.ts

@@ -0,0 +1,14 @@
+import _ from "lodash";
+
+/** Take potentially repeated string yarg arguments and make into garanteed string array; comma separated and whitespace trimed */
+export default function parseYargsCommaSeparatedStringArray(
+  args: undefined | string | string[],
+): string[] {
+  return _.uniq(
+    (typeof args === "string" ? [args] : ((args || []) as string[]))
+      .join(",")
+      .split(",")
+      .map(x => (x || "").trim())
+      .filter(x => x),
+  );
+}