You could use `as_xml_add_text_node` here, but that's optional :-)

`g_autofree gchar *bandwidth_str = as_xml_get_prop_value (node, "bandwidth_mbitps");` (a bit nicer to read)

I know it's not your code, but what about `g_free (g_steal_pointer (&priv->version))` or `as_relation_set_version (relation, NULL)`?

What is the value in case an application *can* use the internet to enhance the experience, but otherwise does not need to and would also work perfectly fine offline? (e.g. a media player fetching cover art for songs from the web, but which would play them perfectly find just as well w/o internet connection) Would that be maybe: ```xml <supports> <internet>always</internet> </supports> ```