Fix Node.js' code coverage/code coverage job

[bcoe]

• Version: 15.x.x
• Platform: All
• Subsystem: test

The problem(s)

1. new file paths are breaking the coverage reporting

File paths for Node.js' own files now have this form node:/internal/cluster/shared_handle:

• there's now anode: prefix.
• there's no longer a .js prefix.

I will happily fix this problem in the test reporter, but could someone point me towards where these changes were made (@watilde, @guybedford?) -- I want to make sure I link to the change in the reporter codebase.

2. nightly coverage is broken again

Coverage has been broken for several nights in a row. I believe this is due to an unrelated CI/CD upgrade?

Suggested fix

1. let's update c8 to support the new path format (I've tested that this fix is pretty trivial).
2. let's move code coverage to GitHub Actions (It seems like we need to fix Jenkins once every few months, and I can't see a reason why not to use actions).

CC: @nodejs/testing

[aduh95]

could someone point me towards where these changes were made

I believe this has been introduced by #35498.

[bcoe]

@aduh95 thank you! (sorry for looping in the wrong folks, I assumed it was potentially the new URL parser).

[joyeecheung]

Have we ever tried adding c8 to https://github.com/nodejs/citgm/ ? (I searched for it and I don't think so?)

[MylesBorins]

we should add it, it isn't included right now afaik. For reference this is the lookup table

https://github.com/nodejs/citgm/blob/main/lib/lookup.json

[bcoe]

Have we ever tried adding c8 to https://github.com/nodejs/citgm/ ? (I searched for it and I don't think so?)

@joyeecheung @MylesBorins not a bad idea, but it wouldn't have actually caught this, because it only breaks on Node.js' own coverage -- which now has different file paths (c8 would continue working for userland.)