site<- and horizons<- SPC methods have always joined on all matching column names. (i.e. merge is called with no by argument)
It would be good to optionally support a by, or a by.x and by.y, argument on the left hand side to these to allow for differences in column name between existing slot data and incoming data, as well as to allow folks to be very explicit about join conditions where there might be unintended conflicts or similar