Created by: camdencheek
Okay, this is the last step to break the cyclical import issues. This makes job.Map operate on the Job interface with a new MapChildren() method rather than unwrapping into every possible job type.
Stacked on https://github.com/sourcegraph/sourcegraph/pull/38353
This is nice for a couple of reasons:
job/jobutil imports every package that defines a job, which causes a hotbed of circular import issues.Map without ever importing internal/search/job/jobutil.Unit tests. All existing tests passed, and I checked that there were a number that exercised current mapping behavior.