Test: exercise PRowl reviewer

[tusharshah21]

Demo PR with intentional issues to verify the action end-to-end.

[github-actions]

[SECURITY] The code is vulnerable to SQL injection because it directly concatenates user input (orderId) into the SQL query string.

Suggested change

	const sql = "SELECT * FROM orders WHERE id = " + orderId;
	const sql = "SELECT * FROM orders WHERE id = ?";

[github-actions]

[BUG] The code assumes that 'cart.items' is not empty, which can lead to an error if 'cart.items' is an empty array. Accessing the first element without checking can cause an 'undefined' error.

Suggested change

	return cart.items[0].price;
	if (cart.items.length > 0) {
	return cart.items[0].price;
	} else {
	return 0; // or handle the empty case appropriately
	}

[github-actions]

[PERFORMANCE] The nested loop results in an O(n^2) time complexity, which is inefficient for checking duplicates. Using a Set to track seen elements reduces the complexity to O(n).

Suggested change

	for (let j = 0; j < arr.length; j++) {
	function hasDuplicate(arr) {
	const seen = new Set();
	for (let i = 0; i < arr.length; i++) {
	if (seen.has(arr[i])) {
	return true;
	}
	seen.add(arr[i]);
	}
	return false;
	}