feat: Add getPacks method to retrieve installed CodeQL packs

[GeekMasher]

This pull request introduces enhancements to the CodeQLService class, focusing on improving the handling of CodeQL packs and query pack selection. The changes include the addition of a new method to retrieve installed CodeQL packs, refactoring of the query pack selection logic to leverage this new method, and improved error handling and logging.

Enhancements to CodeQL pack handling:

• Added getPacks method: This new method retrieves a list of installed CodeQL packs using the CodeQL CLI, with detailed logging and error handling. It parses the CLI output to extract pack names and ensures no duplicates are included.

Refactoring of query pack selection:

• Refactored findQueryPack method: The method was converted to an asynchronous function that now uses the getPacks method to identify the appropriate query pack for a given language. It prioritizes language-specific packs and falls back to a default pack if none are found. Enhanced logging provides better visibility into the selection process.

• Updated usage of findQueryPack: The call to findQueryPack in the runAnalysis method was updated to support its new asynchronous implementation.

[Copilot]

Pull Request Overview

This PR enhances the CodeQL service by adding a new getPacks method to retrieve installed CodeQL packs and refactoring the query pack selection logic to use this new method instead of filesystem-based pack discovery.

• Added getPacks method that uses CodeQL CLI to retrieve installed packs with JSON parsing
• Refactored findQueryPack method to be async and use the new getPacks method for pack discovery
• Updated runAnalysis method to handle the async findQueryPack call

[Copilot]

Use 'const' instead of 'var' for block-scoped variable declaration. Since 'packs' is reassigned through array methods rather than variable reassignment, it can be declared as const.

Suggested change

	var packs: string[] = [];
	const packs: string[] = [];

[Copilot]

Using Array.includes() for duplicate checking has O(n) complexity for each insertion, resulting in O(n²) overall complexity. Consider using a Set for O(1) lookups and convert to array at the end.

Suggested change

	if (!packs.includes(packName)) {
	packs.push(packName);
	if (!packs.has(packName)) {
	packs.add(packName);