Save path should utilize ds.Commit.ID, ds.Name, and ds.Peername instead of a ref

[b5]

we've run into this most recently in #1911, and have known about this issue for a while. We could clean up TODOs like this one:

qri/base/save.go

Lines 133 to 135 in d451f6f

	// TODO(dustmop): Remove the dependence on the ds having an assigned Name. It is only
	// needed for updating the refstore. Either pass in the reference needed to update the refstore,
	// or move the refstore update out of this function.

Now that we have the collection subsystem to manage references independent of the save path