Since #1784 ,

Just from reading the code, that happens to work as expected because it is not using the primary return value of dir2, but that’s extremely non-obvious and very likely to be broken during future maintenance. To the extent the cod is only looking for the “homeDir”, I think that should be modeled explicitly and not use the id value at all. Alternatively, if the id value mattered, then this needs to actually be implemented.

Cc: @giuseppe