experience report: issuer.js

[dckc]

comparing issuers.js vs. the grammar here (via a scala implementation), I ran across

• 32 import './types';

• new Error(...) which, I gather, can be rewritten as Error(...) with no observable change
• new Set() (as discussed earlier in https://github.com/Agoric/Jessie/issues/29 etc.)

• 33 export { makeIssuerKit }; as well as module level function declarations

To conform to the current grammar we'd have to change...

function makeIssuerKit(...) { ... }
harden(makeIssuerKit);
export { makeIssuerKit };

to more like...

export const makeIssuerKit = harden((...) => {...});

cc @erights @michaelfig

[erights]

For all four of these, I think we should change the definition of Jessie to allow them. I'll comment on each of the issues you link to.

[erights]

No, after reading https://github.com/Agoric/Jessie/issues/29 I changed my mind. It is correct. We should implement and use that std library so that we don't have unnecessarily unhardened objects, like instances of Set running around loose.