Sidetree DID Docs are not DID Core Spec Compliant without update or transformation.

[OR13]

const transformArrays = [
  "assertionMethod",
  "authentication",
  "capabilityDelegation",
  "capabilityInvocation"
];

const transformObjects = ["publicKey", "keyAgreement"];

module.exports = didDocument => {
  const convertedDidDocument = { ...didDocument };

  transformObjects.forEach(property => {
    convertedDidDocument[property].forEach(value => {
      if (value.id[0] === "#") {
        value.id = convertedDidDocument.id + value.id;
      }

      if (!value.controller) {
        value.controller = convertedDidDocument.id;
      }
    });
  });

  transformArrays.forEach(property => {
    convertedDidDocument[property] = convertedDidDocument[property].map(
      value => {
        if (value[0] === "#") {
          return convertedDidDocument.id + value;
        }
        return value;
      }
    );
  });

  return convertedDidDocument;
};

[OR13]

We should add this transform to the library, since its likely to be needed until a better solution is available.

[gjgd]

Since #138 the controller property is added and we have this helper function for adding dids to fragids so I think we can close this

[OR13]

We should leave this open until fragids are valid according to the did core spec, or we don't use them.

[OR13]

Linking the sidetree core issue: decentralized-identity/sidetree#368

[OR13]

Since #138 the controller property is added and we have this helper function for adding dids to fragids so I think we can close this

This helper method is excellent though.