[drakeo338]

Pull Request OpenBB

Description

Fixes #7799.

_safe_segment in the CLI router generator returned names like 24hr or class unchanged, so the generated router contained async def 24hr( and unimportable module names. Names that start with a digit now get a leading underscore (_24hr), as the Fetcher, POST and test generators already do, and Python keywords get a trailing one (class_), as pydantic_gen does for field names.

• Summary of the change/ bug fix.
• Link # issue, if applicable.

How has this been tested?

Two new tests in cli/tests/test_codegen_router_gen.py fail on the unfixed source and pass with the fix; the file runs 21 passed. Other checkboxes are left unticked because I did not run them.

Checklist

• I ensure I have self-reviewed my code.
• Update our tests following these guidelines.

I used an AI coding assistant (Claude Code) for this change and reviewed and tested it myself.

[dlee-bq]

Thanks for this, @drakeo338.

It fixes #7799. One change before merge: for Python keywords, please append the underscore (class → class_) rather than prefixing it. That matches how the generator already handles keywords in field names (pydantic_gen.safe_field_name / query_field_name give from_), and it keeps generated commands from looking private on the obb surface (obb.x.ticker._class).

Keep the _ prefix for names that start with a digit (_24hr), as you have it.

[drakeo338]

Done in 45dd415: keywords now get a trailing underscore (class_), digit-leading names keep the _ prefix (_24hr), and the tests cover both

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}