codeql icon indicating copy to clipboard operation
codeql copied to clipboard

Ruby: add support for extracting overlay databases

Open nickrolfe opened this issue 7 months ago • 2 comments

There will need to be followup changes on the QL side before querying will work, but this is enough to be able to build an overlay database for Ruby.

nickrolfe avatar Jun 06 '25 11:06 nickrolfe

@aibaars do you have capacity to review this?

nickrolfe avatar Jun 12 '25 15:06 nickrolfe

I've force-pushed a couple of the suggested fixes. Regarding the others:

  • prefix.dbscheme: since this will change the QL-for-QL dbscheme, I'll do it as a quick follow-up PR. That way, if any issues arise, it can be reverted without affecting Ruby overlay support.
  • similarly, I'll add the Rust path-transformer support as a separate PR.
  • for implementing full path-transformer support, I don't think it would be an enormous amount of work to port either the existing Java or C++ implementations, along with their unit tests, but it's probably enough work that we discussed within our team and decided not to do it (since it's not on the critical path for overlay support).

nickrolfe avatar Jun 19 '25 15:06 nickrolfe