In general, you should never put leading or trailing spaces around the passage name due to Twine 2 considering them significant while SugarCube does not. In this case, adding a trailing space is completely unnecessary, so I'm unsure why it was even suggested.
Also. If you're going to use a variable, as in this case, then it's also generally easier to bypass Twine 2's automatic passage creation feature by using the separate argument version of <<link>>, rather than using the link markup and then having to play delete-the-passage every time.