feat: refactor complete parser

[sfc-gh-twhite]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors the parser by extracting the _parse_snowflake_response logic into a shared helper function in the utils module for consolidation and reuse. Key changes include:

• Moving _parse_snowflake_response from planner and gateway modules to tools/utils.py.
• Updating references in planner.py and gateway.py to use the new helper function.
• Removing duplicate implementations of the response parser.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
agent_gateway/tools/utils.py	Introduces a shared _parse_snowflake_response parser
agent_gateway/gateway/planner.py	Replaces self._parse_snowflake_response with the imported version
agent_gateway/gateway/gateway.py	Replaces self._parse_snowflake_response with the imported version and removes custom error handling

Comments suppressed due to low confidence (1)

agent_gateway/gateway/gateway.py:100

• [nitpick] The removal of the custom error handling for missing keys in the JSON response may lead to less informative errors; consider integrating similar error handling in the new shared parser for clearer diagnostic messages.

def _parse_snowflake_response(self, data_str):

[Copilot]

Consider wrapping json.loads(data_str) in a try/except block to handle malformed JSON input and avoid uncaught exceptions.

Suggested change

	json_list = [i["data"] for i in json.loads(data_str)]
	try:
	json_list = [i["data"] for i in json.loads(data_str)]
	except json.JSONDecodeError:
	# Handle the error (e.g., log it, return an empty list, etc.)
	json_list = []