parseArgs duplicates collected + aliased arguments

[j0sh]

parseArgs duplicates collected + aliased arguments

[iuioiua]

This issue belongs in https://github.com/denoland/deno_std

[kt3k]

This seems to have started from #4189

The example test case passes with [email protected] (and fails with [email protected]):

import { parseArgs } from "https://deno.land/[email protected]/cli/parse_args.ts";
import { assertEquals } from "https://deno.land/[email protected]/assert/mod.ts";

Deno.test("collect with alias", () => {
  const args = ["--header", "abc", "--header", "def"];
  const parsed = parseArgs(args, {
    collect: ["header"],
    alias: {
      H: "header",
    },
  });
  assertEquals(parsed.header, ["abc", "def"])
})

I'd suggest we should revert #4189

[timreichen]

I think I found the bug. The aliasMap Set for key contains key itself. On that example header contains header as an alias.

Map(2) {
  "H" => Set(1) { "header" },
  "header" => Set(2) { "H", "header" }
}

This is fixed by filtering out the key.

before:

const set = new Set([key, ...aliases]);
aliases.forEach((alias) => aliasMap.set(alias, set));

after:

aliases.forEach((alias) => aliasMap.set(alias, new Set([key, ...aliases.filter((it) => it !== alias)])));