Looks like it doesn’t add id which is required by the spec:
https://github.com/nostrability/nostrability/issues/324
But honestly we should remove that requirement since id should be recalculated anyways on rumors. What do you think nostr:npub1gcxzte5zlkncx26j68ez60fzkvtkm9e0vrwdcvsjakxf9mu9qewq...