output hex if user input is hex

[ryanroe]

No description provided.

[Copilot]

Pull Request Overview

Adds hexadecimal support for sequential number generation when the user’s input starts with a 0x prefix.

• Detects hex input via an isHex flag
• Parses the starting value with base 16 and formats subsequent results back to hex
• Falls back to decimal parsing and output when no hex prefix is found

Comments suppressed due to low confidence (1)

extension.js:43

• There are no tests covering the new hexadecimal input/output logic; consider adding unit tests to verify hex parsing and formatting behavior.

				let isHex = start.startsWith('0x');

[Copilot]

This only checks for a lowercase '0x' prefix; consider normalizing the input (e.g., start.toLowerCase().startsWith('0x')) to also support uppercase '0X'.

Suggested change

	let isHex = start.startsWith('0x');
	let isHex = start.toLowerCase().startsWith('0x');

[Copilot]

Avoid using parseInt without an explicit radix on the decimal branch; specify parseInt(start, 10) to ensure consistent parsing.

Suggested change

	let result = isHex ? parseInt(start, 16) : parseInt(start);
	let result = isHex ? parseInt(start, 16) : parseInt(start, 10);

[Copilot]

Using eval for arithmetic operations can introduce security risks and performance overhead. Consider replacing it with direct arithmetic based on operator (e.g., a switch statement).

Suggested change

	result = eval(result + operator + parseInt(step));
	switch (operator) {
	case '+':
	result += parseInt(step);
	break;
	case '-':
	result -= parseInt(step);
	break;
	case '*':
	result *= parseInt(step);
	break;
	case '/':
	result /= parseInt(step);
	break;
	default:
	throw new Error('Unsupported operator: ' + operator);
	}

[Copilot]

Also specify a radix when calling parseInt(step) (e.g., parseInt(step, 10)) to prevent unintended parsing behavior.

Suggested change

	result = eval(result + operator + parseInt(step));
	result = eval(result + operator + parseInt(step, 10));