Return undefined instead of null in get-product-id files

[jeffdaley]

Changes the get-product-id files to return undefined instead of null as a fallback. This better matches (and thus passes more Glint checks for) receiving component types, which tend to be more efficient as value?: string rather than value?: string | null