RequiredOptions.ts duplicates PickRequired.ts

[pixelzoom]

RequiredOption.ts looks like it duplicates PickRequired.ts. We had a long discussion about PickRequired.ts (and its name) in https://github.com/phetsims/phet-io/issues/1843, and decided on PickRequired.ts and PickOptional.ts.

Recommended to delete RequiredOption.ts.

[pixelzoom]

We discussed this in 3/3/2022 dev meeting, and they are in fact different. I updated the doc in RequiredOption.ts to clarify.

But the name is still misleading. It returns the type of an option, and it doesn't even need to be a required option (as demonstrated by the example in the doc.) Should it be renamed to something like TypeOfOption or OptionType?

[pixelzoom]

It looks like there's another problem with RequiredOption.ts. You can list multiple option names and it doesn't complain. Example:

type SelfOptions = {
  name: string
  age: number
};

// No complaint about this?
something: RequiredOption<SelfOptions, 'name' | 'age'>;