ogar-render-askama: render_python — Python import-shape emit PoC (SDK + package)

[AdaWorldAPI]

What

A Python emit back-end for the same ruff → OGAR harvest that render_osm
renders to Rust. Proves the pull-back side of the transpiler is
language-pluggable: OSM (and odoo) come out of the one substrate in Python
import shape, just like Rust.

Emits

python/osm/ — the package

• models.py — one @dataclass per THINK class; associations → typed id
  fields (belongs_to/has_one → Optional[int], has_many/habtm →
  List[int]); each grounded model carries its CLASS_ID.
• controllers/<name>.py — the DO arm as a faithful mirror of
  app/controllers/, one module per source controller by its verbatim (snake)
  name: osm.controllers.nodes.show(inp), standalone, not methods. Mirrors the
  Rust controllers module — no singularisation.
• __init__.py — re-exports each controller at the package root
  (osm.nodes ≡ osm.controllers.nodes) + CLASS_IDS.

python/ogar_sdk.py — the substrate-pull SDK (Python mirror of Rust
lance_graph_contract::ogar_codebook):

>>> from ogar_sdk import class_id, render_classid
>>> hex(class_id("osm_node"))
'0xf01'
>>> hex(render_classid("account_move", 0x0002))   # odoo, canon-high
'0x2020002'

Ships OSM (0x0F) and odoo/commerce (0x02) — the pull is domain-agnostic.

Verified

• render_python builds clean; py_compile green on package + SDK.
• Functional smoke: osm.controllers.nodes.show and re-exported
  osm.nodes.show both callable; class_id('osm_node')=0xf01;
  render_classid('account_move',0x0002)=0x2020002; Node.CLASS_ID=0xf01;
  osm.nodes.show({}) → NotImplementedError('port Api::NodesController#show').

Depends on

Stacks on #153 (the delete-the-singulariser / faithful-controllers change),
so the container module names are verbatim (nodes, searches, capabilities
— not searche, capabilitie). Merge #153 first.

Follow-up

Input is dict (the Rails params bag); typed params remain the ruff
param-harvest brick (same gap as the Rust DO arm).

🤖 Generated with Claude Code

https://claude.ai/code/session_01EYvNjD8M8LMNYbRy3gq2FP

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 89f6b18. Configure here.}

[cursor]

Colliding pyify drops actions

Medium Severity

When two distinct is_a rail keys in the same part_of module normalize to the same Python name via pyify, the emitter skips later entries with continue and never merges their controller sources into the surviving stub. Rails predicates ending in ? are especially at risk because snake adds a trailing underscore that trim_matches('_') then removes, so foo? and foo can share one function name and one tile vanishes from the package.

Additional Locations (1)

• crates/ogar-render-askama/examples/render_python.rs#L60-L61

 

^{Reviewed by Cursor Bugbot for commit 89f6b18. Configure here.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 89f6b1855f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Mark generated CLASS_ID as a ClassVar

For grounded classes this emits CLASS_ID as a dataclass instance field, so it becomes part of __init__, repr, equality, and positional construction; for example the generated Node(5) sets CLASS_ID=5 and leaves changeset unset. Since this value is intended to be stable per-class identity, generate it as ClassVar[int] (and import ClassVar) so model instances cannot silently override or serialize it as mutable record data.

Useful? React with 👍 / 👎.