Fix: #32 Recursion Issue

[atalhens]

On observing the way, the agent executes the question received from the user: https://github.com/NVIDIA/AgentIQ/blob/develop/src/aiq/agent/react_agent/agent.py#L108.

Looks like there is no sanity check if the input query received from the user side is valid or not.

Edit added to check is the input is like "", " " [empty or blank spaces].

Output will be shown as No human input provided.

Closes #32

[mdemoret-nv]

@sean-javiya-nvidia Can you review these changes?

[sean-javiya-nvidia]

Awesome! Thanks for quickly addressing @atalhens!

The last suggestion I have, could we add a unit test for the added improvement?

we could maybe add a unit test here, maybe something like this

async def test_no_input(mock_react_graph):
    response = await mock_react_graph.ainvoke(ReActGraphState(messages=[HumanMessage(',')]))
    response = ReActGraphState(**response)
    response = response.messages[-1]  # pylint: disable=unsubscriptable-object
    assert isinstance(response, AIMessage)
    assert response.content == NO_INPUT_ERROR_MESSAGE

that way, we can ensure any future contributions or changes don't accidentally change the desired behavior

[sean-javiya-nvidia]

commented above

[atalhens]

Added unit test, Thanks for your suggestion @sean-javiya-nvidia!
Unit test check for HumanMesaage('')

Let me know if it addresses your concern :)

[mdemoret-nv]

@atalhens Thanks for your contribution!