This extension is interesting. However, looks a bit hacky.
- I'd like to have something like action=amphtml on the same URL to get the AMP page dynamically, instead of a static file being generated on the server.
- This would allow any page to be rendered as AMP, not only those with the <amp/> tag.
- Maybe this could be done implementing a new parser (or similar, although it sounds a bit insane :) ), and save the parsed content in cache. It may improve cacheability and automatic purge when the page is purged (because of template or image change, etc).
- I think something like this is done in Extension:MobileFrontend.
- Think of a way for other extensions to hook and add their own amp markup for their own tags.