Well the API that we're interested in is actually against `Attachment` - defining `html?` on `AttachmentData` and then delegating through to it feels really strange when you know it's simply returning false in all cases. It makes sense that `pdf?` is defined on `AttachmentData` because it needs to know details about the file itself.