Yes that part was left completely unspecified and I think the first to implement it "somehow" were Amber and Nostrudel, and then I followed with nsec.app and nostr-login. The PR you mentioned should indeed clarify this part.