fix: make header names python-safe

[saurabhkoshatwar]

Fixes #580

Summary

• Allow Toolbox tools to use parameter/header names that are not valid Python identifiers (e.g. X-Application-ID).
• Introduce a Python‑safe field name mapping so function signatures and Pydantic models stay valid while preserving the original parameter names over the wire.

Details

• Sanitize schema parameter names into Python‑safe field names and use those in inspect.Parameter and Pydantic models.
• Maintain a mapping from Python‑safe names back to original schema names in ToolboxTool so requests still use the exact header/parameter names defined in tools.yaml.

[anubhav756]

/gcbrun

[anubhav756]

Thanks for the PR @saurabhkoshatwar !

Could you please take a look at the integration test failure?

I'm also thinking if we could add any more integration/unit tests to verify this change. For instance to see if there could still be any validation failures while actually using hyphenated parameters?

[anubhav756]

/gcbrun

[anubhav756]

Thanks for the PR @saurabhkoshatwar !

Could you please take a look at the integration test failure?

I'm also thinking if we could add any more integration/unit tests to verify this change. For instance to see if there could still be any validation failures while actually using hyphenated parameters?

@saurabhkoshatwar Bumping this up in case it got missed :)

[anubhav756]

/gcbrun

[saurabhkoshatwar]

@anubhav756 could you please rerun the tests? Thanks!

[anubhav756]

/gcbrun

[anubhav756]

/gcbrun

[saurabhkoshatwar]

@anubhav756, please rerun /gcbrun

[anubhav756]

/gcbrun

[anubhav756]

@anubhav756, please rerun /gcbrun

@saurabhkoshatwar Done!

[Deeven-Seru]

Hi @saurabhkoshatwar , @anubhav756 I was digging into this and I think we might have a validation order issue.

The current approach builds the Pydantic models with safe names (like user_id), but the actual tool invocation still passes the original hyphenated keys (like user-id). Since there’s no alias mapping, model_validate() is going to fail at runtime because it can’t find the underscored fields it’s looking for.

I tested this locally and it breaks as soon as you try to actually call the tool. I think it will be better if we use Pydantic's Field(alias=...) so the model knows how to map the original names to our safe identifiers.

Also, it would be good to add an E2E test case for this—the current unit tests only check that the model can be created, not that it actually validates hyphenated data during a real call.