Missing of keyword in protocol action rules leads to confusing error

[diehuxx]

Background:

When a protocol definition has a recipient or author rule, the of keyword is required. But the error that the JSON schema throws is difficult for people to understand. /descriptor/definition/structure/album/$actions/0/who: must be equal to one of the allowed values

Task

We should either alter the JSON schema to throw a friendlier message, or add validation in plain javascript.

JSON Schema: https://github.com/TBD54566975/dwn-sdk-js/blob/main/json-schemas/interface-methods/protocol-rule-set.json#L24-L74

Protocol Definition javascript validation: https://github.com/TBD54566975/dwn-sdk-js/blob/main/src/interfaces/protocols-configure.ts#L44

// correct
[{
  who: 'anyone',
  can: 'read'
},
{
  who: 'recipient',
  of: 'foo',
  can: 'read'
},
{
  who: 'author',
  of: 'foo',
  can: 'read'
}]

// incorrect
[{
  who: 'recipient',
  // missing `of`
  can: 'read'
},
{
  who: 'author',
  // missing `of`
  can: 'read'
},
{
  who: 'anyone',
  of: 'foo', // should not have `of`
  can: 'read'
}]

Picking Up This Issue:

• If you'd like to work on this, please comment "picking this up" below, and I'll assign the issue to you

Questions:

• If you need assistance or clarification, feel free to comment below.
• Feeling overwhelmed? Join our #hack-together channel and collaborate with a buddy.
• New to open-source? Check out our Hacktober #getting-started channel.
• Leaderboard: Check your ranking after finishing this task.

Resources:

• Creating a Pull Request: If you're new to GitHub and unsure how to create a pull request, follow this step-by-step guide.

Remember, communication is key! If you have any questions or face any challenges, we're here to help so please don't hesitate to reach out.

Good Luck! 🍁

[flothjl]

picking this up.

[EbonyLouis]

@flothjl It's been assigned to you! 🎉

Kudos for submitting your PR so quickly! 🚀