garvit-arora commented on PR #74419: URL: https://github.com/apache/airflow/pull/74419#issuecomment-6111862399
Hi @dhanusharer — zozo123 asked us to consolidate the overlapping Genie hook work into one PR. Your original hook commit is preserved as the first contribution in #74517. I propose using this earlier PR (#74419) as the landing PR and carrying over the additional safeguards from commit `0b9c37c5` in #74517: bounded structured results and IDs, no retry of ambiguous POSTs, direct hook use without Common AI, the standard `ManagedAgentToolset` example with `max_retries=0`, and focused tests and docs. The commit is based on your hook and retains your attribution. Would you be willing to incorporate those changes here? I can help resolve conflicts or supply a focused patch. Once the work is consolidated and preserved in one PR, I will close #74517. The real Databricks workspace run and passing targeted pytest are still outstanding, so I am not asking for merge yet. --- Drafted-by: Codex (GPT-6) (no human review before posting) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
