Reorganize fileLoader, make it lock-free via SyncMaps

[jakebailey]

I'm currently modifying this to work on JSX stuff; sending this refactor in early.

I'm going to add more stuff to the file loader for JSX, so converting its return into a struct will make a future PR simpler.

Additionally, there's no real need to use full locks here; sync maps are sufficient for all uses, so we can do everything in the loader lock-free (more or less). Actually, we don't need any locks or sync maps at all outside the one used to dedupe tasks.

[Copilot]

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

internal/compiler/program.go:154

• The code still references p.files after the change to use p.processedFiles. Update the iteration to use p.processedFiles.files.

for _, file := range p.files {

[jakebailey]

Bug in the lint rule, nice.

EDIT: I can't fix it without golang/go#73278 or a bunch of finagling to fix up the CFG, so I'll just rename the var to make the lint rule happy.

[jakebailey]

Realized that none of this even needs to be locked; we can just put the info onto the tasks, which we collect later anyway.