Bug: Record merging syntax with switch statement adds a semicolon instead of a comma

[ByronBecker]

Example case:

let person_1 = {
  name = "John";
  ageGroup = #minor";
};

let minor_permissions = { ... };
let adult_permissions = { ... };

let person_1_plus_permissions = person_1 and (
  switch(person1.ageGroup) {
    case (#minor) { minorPermissions };
    case (#adult) { adultPermissions };
  },  // prettier puts a `;` here instead of a `,`, which throws a syntax error
);

[rvanasa]

Strange!

It looks like there is an extraneous quotation mark in ageGroup = #minor"; (at least in the above example). Fixing that causes the code to format as expected for me (playground link).

The most likely underlying cause is that Prettier is falling back to the JavaScript formatter, which can happen if there is a plugin installation issue.

[q-uint]

(can probably be simplified)

type AgeGroup = {
  #minor;
  #adult;
};

type Person = {
  name : Text;
  ageGroup : AgeGroup;
};

type Permissions = {
  some : Text;
};

type PPerson = Person and Permissions;

let person_1 : Person = {
  name = "John";
  ageGroup = #minor;
};

func ageGroup(p : Person) : AgeGroup {
  return p.ageGroup;
};

func pperson(p : Person) : PPerson {
  {
    p and (
      switch (ageGroup(p)) {
        case (#minor) { { some = "M" } };
        case (#adult) { { some = "A" } };
      }, // ADDED? OK
    ): // ADDED? NOK
  };
};

Playground: link

[rvanasa]

Okay, thanks! I see what you're saying and will look into fixing this.

[rvanasa]

Fixed in v0.2.4!