pyopenapi-gen 0.14.2__py3-none-any.whl → 0.15.0__py3-none-any.whl

pyopenapi_gen/__init__.py

@@ -43,7 +43,7 @@ __all__ = [
 ]
 
 # Semantic version of the generator core – automatically managed by semantic-release.
-__version__: str = "0.10.2"
+__version__: str = "0.14.3"
 
 
 # ---------------------------------------------------------------------------

pyopenapi_gen/context/render_context.py

@@ -153,10 +153,25 @@ class RenderContext:
             self.core_package_name + "."
         )
         if is_target_in_core_pkg_namespace:
+            # For root-level sibling packages (core and output package both at top level),
+            # we MUST use absolute imports because Python doesn't support relative imports
+            # that go beyond the top-level package.
+            #
+            # Example structure that requires absolute imports:
+            #   output/              # NOT a package (no __init__.py)
+            #   ├── core/            # top-level package
+            #   └── businessapi/     # top-level package
+            #
+            # In this case, businessapi/client.py cannot use "from ..core import X"
+            # because that would try to go UP from businessapi to output/, which is not a package.
+            #
+            # Solution: Always use absolute imports for core when it's a root-level sibling.
+
+            # Always use absolute imports for core imports
             if name:
                 self.import_collector.add_import(module=logical_module, name=name)
             else:
-                self.import_collector.add_plain_import(module=logical_module)  # Core plain import
+                self.import_collector.add_plain_import(module=logical_module)
             return
 
         # 3. Stdlib/Builtin?
@@ -628,3 +643,84 @@ class RenderContext:
 
         except ValueError:  # If current_file is not under overall_project_root
             return None
+
+    def get_core_import_path(self, submodule: str) -> str:
+        """
+        Calculate the correct import path to a core package submodule from the current file.
+
+        Handles both sibling core packages (core/, custom_core/) and external packages (api_sdks.my_core).
+
+        Args:
+            submodule: The submodule within core to import from (e.g., "schemas", "http_transport", "exceptions")
+
+        Returns:
+            The correct import path string (e.g., "...core.schemas", "..core.http_transport", "api_sdks.my_core.schemas")
+
+        Examples:
+            From businessapi/models/user.py → core/schemas.py returns "...core.schemas"
+            From businessapi/client.py → core/http_transport.py returns "..core.http_transport"
+            From businessapi/endpoints/auth.py → core/exceptions.py returns "...core.exceptions"
+            External core package (api_sdks.my_core) returns "api_sdks.my_core.schemas"
+        """
+        # 1. Check if core_package_name contains dots or is already a relative import
+        if "." in self.core_package_name and not self.core_package_name.startswith("."):
+            # External package (e.g., "api_sdks.my_core") - use absolute import
+            return f"{self.core_package_name}.{submodule}"
+
+        if self.core_package_name.startswith(".."):
+            # Already a relative import path
+            return f"{self.core_package_name}.{submodule}"
+
+        # 2. Local core package (sibling) - calculate relative path
+        return self._calculate_relative_core_path(submodule)
+
+    def _calculate_relative_core_path(self, submodule: str) -> str:
+        """Calculate relative import path to sibling core package."""
+
+        if not self.current_file or not self.package_root_for_generated_code or not self.overall_project_root:
+            # Fallback to absolute import if context not fully set
+            logger.warning(
+                f"Cannot calculate relative core path: context not fully set. "
+                f"current_file={self.current_file}, package_root={self.package_root_for_generated_code}, "
+                f"project_root={self.overall_project_root}. Using absolute import."
+            )
+            return f"{self.core_package_name}.{submodule}"
+
+        try:
+            # 1. Get current file's directory
+            current_file_abs = Path(self.current_file).resolve()
+            current_dir_abs = current_file_abs.parent
+
+            # 2. Determine core package location (sibling to output package)
+            project_root_abs = Path(self.overall_project_root).resolve()
+
+            # Core is sibling to the output package
+            core_abs = project_root_abs / self.core_package_name
+            target_abs = core_abs / submodule.replace(".", os.sep)
+
+            # 3. Calculate relative path from current directory to target
+            relative_path = os.path.relpath(target_abs, start=current_dir_abs)
+
+            # 4. Convert filesystem path to Python import path
+            # e.g., "../../core/schemas" → "...core.schemas"
+            path_parts = relative_path.split(os.sep)
+
+            dots = 0
+            module_parts = []
+
+            for part in path_parts:
+                if part == "..":
+                    dots += 1
+                elif part != ".":
+                    module_parts.append(part)
+
+            # Prefix with dots (add one more for Python relative imports)
+            prefix = "." * (dots + 1)
+            module_path = ".".join(module_parts)
+
+            return f"{prefix}{module_path}"
+
+        except Exception as e:
+            # Fallback to absolute import on any error
+            logger.warning(f"Failed to calculate relative core path: {e}. Using absolute import.")
+            return f"{self.core_package_name}.{submodule}"

pyopenapi_gen/core/writers/python_construct_renderer.py

@@ -183,21 +183,8 @@ class PythonConstructRenderer:
         context.add_import("dataclasses", "dataclass")
 
         # Always use self-contained BaseSchema for client independence with automatic field mapping
-        # Use the core package from context - could be relative (..core) or absolute (api_sdks.my_core)
-        if context.core_package_name.startswith(".."):
-            # Already a relative import
-            core_import_path = f"{context.core_package_name}.schemas"
-        elif "." in context.core_package_name:
-            # External core package with dots (e.g., api_sdks.my_core) - use absolute import
-            core_import_path = f"{context.core_package_name}.schemas"
-        elif context.core_package_name == "core":
-            # Default relative core package
-            core_import_path = "..core.schemas"
-        else:
-            # Simple external core package name (e.g., shared_core_pkg) - use absolute import
-            core_import_path = f"{context.core_package_name}.schemas"
-
-        context.add_import(core_import_path, "BaseSchema")
+        # Use absolute core import path - add_import will handle it correctly
+        context.add_import(f"{context.core_package_name}.schemas", "BaseSchema")
 
         # Add __all__ export
         writer.write_line(f'__all__ = ["{class_name}"]')

pyopenapi_gen/types/resolvers/schema_resolver.py

@@ -201,10 +201,24 @@ class OpenAPISchemaResolver(SchemaTypeResolver):
             # If anything goes wrong with attribute access, assume not a self-reference
             current_file = None
 
-        # Only treat as self-reference if we have a real file path that matches
-        if current_file and isinstance(current_file, str) and current_file.endswith(f"{module_stem}.py"):
+        # Only treat as self-reference if the EXACT filename matches (not just a suffix)
+        # Bug fix: Use basename comparison to avoid false positives with similar names
+        # e.g., "vector_index_with_embedding_response_data.py" should NOT match "embedding_response_data.py"
+        is_self_import = False
+        if current_file and isinstance(current_file, str):
+            import os
+
+            current_filename = os.path.basename(current_file)
+            expected_filename = f"{module_stem}.py"
+            is_self_import = current_filename == expected_filename
+
+        if is_self_import:
             # This is a self-import (importing from the same file), so skip the import
             # and mark as forward reference to require quoting
+            logger.debug(
+                f"Self-import detected: current_file={current_file}, module_stem={module_stem}, "
+                f"class_name={class_name}, returning forward ref WITHOUT import"
+            )
             return ResolvedType(python_type=class_name or "Any", is_optional=not required, is_forward_ref=True)
 
         # Use the render context's relative path calculation to determine proper import path
@@ -226,6 +240,10 @@ class OpenAPISchemaResolver(SchemaTypeResolver):
                 # If relative path calculation fails, fall back to the default
                 logger.debug(f"Failed to calculate relative path for {module_stem}, using default: {e}")
 
+        logger.debug(
+            f"Adding import for named schema: module={import_module}, name={class_name}, "
+            f"current_file={current_file}, module_stem={module_stem}"
+        )
         context.add_import(import_module, class_name or "Any")
 
         return ResolvedType(

pyopenapi_gen/types/services/type_service.py

@@ -144,6 +144,10 @@ class UnifiedTypeService:
 
         # Quote forward references BEFORE adding | None so we get: "DataSource" | None not "DataSource | None"
         if resolved.is_forward_ref and not python_type.startswith('"'):
+            logger.debug(
+                f'Quoting forward ref: {python_type} -> "{python_type}" '
+                f"(is_forward_ref={resolved.is_forward_ref}, needs_import={resolved.needs_import})"
+            )
             python_type = f'"{python_type}"'
 
         # Add modern | None syntax if needed

pyopenapi_gen/visit/model/dataclass_generator.py

@@ -80,7 +80,8 @@ class DataclassGenerator:
         context.add_import("dataclasses", "dataclass")
         context.add_import("dataclasses", "field")
         context.add_import("typing", "Any")
-        context.add_import("..core.schemas", "BaseSchema")
+        # Use absolute core import path - add_import will handle it correctly
+        context.add_import(f"{context.core_package_name}.schemas", "BaseSchema")
 
         description = schema.description or "Generic JSON value object that preserves arbitrary data."
 

{pyopenapi_gen-0.14.2.dist-info → pyopenapi_gen-0.15.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyopenapi-gen
-Version: 0.14.2
+Version: 0.15.0
 Summary: Modern, async-first Python client generator for OpenAPI specifications with advanced cycle detection and unified type resolution
 Project-URL: Homepage, https://github.com/your-org/pyopenapi-gen
 Project-URL: Documentation, https://github.com/your-org/pyopenapi-gen/blob/main/README.md

{pyopenapi_gen-0.14.2.dist-info → pyopenapi_gen-0.15.0.dist-info}/RECORD

@@ -1,4 +1,4 @@
-pyopenapi_gen/__init__.py,sha256=jU5GYbWa4MBDeDzOzwUHl_8eLGtotxRGLj42d2yskUo,3017
+pyopenapi_gen/__init__.py,sha256=AUYdbyW8dbrYxo2DvttZkFO79T5QHsS96P8z4TU8y6U,3017
 pyopenapi_gen/__main__.py,sha256=4-SCaCNhBd7rtyRK58uoDbdl93J0KhUeajP_b0CPpLE,110
 pyopenapi_gen/cli.py,sha256=9T_XF3-ih_JlM_BOkmHft-HoMCGOqL5UrnAHBJ0fb5w,2320
 pyopenapi_gen/http_types.py,sha256=EMMYZBt8PNVZKPFu77TQija-JI-nOKyXvpiQP9-VSWE,467
@@ -7,7 +7,7 @@ pyopenapi_gen/py.typed,sha256=Nqnn8clbgv-5l0PgxcTOldg8mkMKrFn4TvPL-rYUUGg,1
 pyopenapi_gen/context/CLAUDE.md,sha256=eUPvSY2ADQK21i52bWfzyBcDPVvvepErMiQrq6ndwlU,9004
 pyopenapi_gen/context/file_manager.py,sha256=vpbRByO5SH6COdjb6C-pXkdSIRu7QFqrXxi69VLKBnM,1691
 pyopenapi_gen/context/import_collector.py,sha256=5WzQ5fXxARHJFZmYZ_GjPG5YjjzRLZTdN7jO2cbbk88,15225
-pyopenapi_gen/context/render_context.py,sha256=07oINhHWi78qhcf-FNSOtojUrshmw0ZOTEgPpzBptbU,30271
+pyopenapi_gen/context/render_context.py,sha256=fAq0u3rDMZ0_OEHegrRUd98ySxs390XJbdypqQX8bwI,34894
 pyopenapi_gen/core/CLAUDE.md,sha256=bz48K-PSrhxCq5ScmiLiU9kfpVVzSWRKOA9RdKk_pbg,6482
 pyopenapi_gen/core/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 pyopenapi_gen/core/exceptions.py,sha256=HYFiYdmzsZUl46vB8M3B6Vpp6m8iqjUcKDWdL4yEKHo,498
@@ -64,7 +64,7 @@ pyopenapi_gen/core/parsing/transformers/inline_object_promoter.py,sha256=d9DZ1CV
 pyopenapi_gen/core/writers/code_writer.py,sha256=5rWMPRdj5dWu3NRER6do4NEJEIa9qL68Hc7zhJ8jSdg,4763
 pyopenapi_gen/core/writers/documentation_writer.py,sha256=ZnXJsRc9y9bUMsQR7g0qoBJZbBo2MCxHH_5KgxrPtyA,8645
 pyopenapi_gen/core/writers/line_writer.py,sha256=-K2FaMtQa6hdzGZLcjQrT2ItEmfE-kquTCrn6R3I_QA,7743
-pyopenapi_gen/core/writers/python_construct_renderer.py,sha256=aRLSiqUkf5Cp72BbJP7nVTmwPPrYlk8W66175joKAg0,12766
+pyopenapi_gen/core/writers/python_construct_renderer.py,sha256=mJ8iCDQAJWg32ZaDI4U7eWRzym3bwzOrJ2VKAXOZds0,12066
 pyopenapi_gen/core_package_template/README.md,sha256=8YP-MS0KxphRbCGBf7kV3dYIFLU9piOJ3IMm3K_0hcI,1488
 pyopenapi_gen/emit/models_emitter.py,sha256=MBRq71UjtlZHrpf9QhDN0wXk_X-oGeGd6TAq3RKG7ko,7180
 pyopenapi_gen/emitters/CLAUDE.md,sha256=iZYEZq1a1h033rxuh97cMpsKUElv72ysvTm3-QQUvrs,9323
@@ -98,9 +98,9 @@ pyopenapi_gen/types/contracts/types.py,sha256=k3tCjMu6M1zcVXF6jzr-Q9S6LXaMEMoKoO
 pyopenapi_gen/types/resolvers/__init__.py,sha256=_5kA49RvyOTyXgt0GbbOfHJcdQw2zHxvU9af8GGyNWc,295
 pyopenapi_gen/types/resolvers/reference_resolver.py,sha256=ue6gw665Wo07POuAg5820A6AXiCVyQY-unfIzGjxrSY,2034
 pyopenapi_gen/types/resolvers/response_resolver.py,sha256=_W6-Z_SBQ8tDMd_VqveO5AiFs7Od7eAuqevpUIDCT5o,6481
-pyopenapi_gen/types/resolvers/schema_resolver.py,sha256=tWNWvCYkXyZ40bzy4ShQJPX2vb_FVgDcrc7tKnWTTwo,22784
+pyopenapi_gen/types/resolvers/schema_resolver.py,sha256=PWw77fPh8SG4i1YD45-cOmjmrA1HQkIz1u75AILlvM0,23617
 pyopenapi_gen/types/services/__init__.py,sha256=inSUKmY_Vnuym6tC-AhvjCTj16GbkfxCGLESRr_uQPE,123
-pyopenapi_gen/types/services/type_service.py,sha256=N_UCyTdt8DRO7FdM7tQT_0G8UJ3OrzqSxTEqEkyrOLg,6784
+pyopenapi_gen/types/services/type_service.py,sha256=3IH0GBLeC-ruokBmhvC8iR5GT5tmFg1n6tcLZYa5TSk,6998
 pyopenapi_gen/types/strategies/__init__.py,sha256=bju8_KEPNIow1-woMO-zJCgK_E0M6JnFq0NFsK1R4Ss,171
 pyopenapi_gen/types/strategies/response_strategy.py,sha256=TmXX2YDQB0cQQ7BvRCGSJM6Iu1p0MMJioCi-_8HPCAg,7341
 pyopenapi_gen/visit/CLAUDE.md,sha256=Rq2e4S74TXv0ua2ZcCrO6cwCCccf3Yph44oVdj1yFPY,8297
@@ -122,11 +122,11 @@ pyopenapi_gen/visit/endpoint/processors/import_analyzer.py,sha256=ou5pl3S6PXvHrh
 pyopenapi_gen/visit/endpoint/processors/parameter_processor.py,sha256=E0VoygkhU8iCsvH0U-tA6ZZg2Nm3rfr4e17vxqQLH7c,7666
 pyopenapi_gen/visit/model/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 pyopenapi_gen/visit/model/alias_generator.py,sha256=wEMHipPA1_CFxvQ6CS9j4qgXK93seI1bI_tFJvIrb70,3563
-pyopenapi_gen/visit/model/dataclass_generator.py,sha256=z0kKH_V2PJyiI9snmryDpAzRiOsQJtk1jrijCnEQQwg,14326
+pyopenapi_gen/visit/model/dataclass_generator.py,sha256=L794s2yyYUO__akGd120NJMV7g2xqiPfQyZo8CduIVM,14426
 pyopenapi_gen/visit/model/enum_generator.py,sha256=AXqKUFuWUUjUF_6_HqBKY8vB5GYu35Pb2C2WPFrOw1k,10061
 pyopenapi_gen/visit/model/model_visitor.py,sha256=TC6pbxpQiy5FWhmQpfllLuXA3ImTYNMcrazkOFZCIyo,9470
-pyopenapi_gen-0.14.2.dist-info/METADATA,sha256=ERW_JKw-KdBCYXTQsG1sSXDYV9ILr9GgnKEvzfgHHNY,14025
-pyopenapi_gen-0.14.2.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-pyopenapi_gen-0.14.2.dist-info/entry_points.txt,sha256=gxSlNiwom50T3OEZnlocA6qRjGdV0bn6hN_Xr-Ub5wA,56
-pyopenapi_gen-0.14.2.dist-info/licenses/LICENSE,sha256=UFAyTWKa4w10-QerlJaHJeep7G2gcwpf-JmvI2dS2Gc,1088
-pyopenapi_gen-0.14.2.dist-info/RECORD,,
+pyopenapi_gen-0.15.0.dist-info/METADATA,sha256=iRFEkYUsm_-6q06MRBl03b1y3J7Utw1xqiZ7NF6aFSQ,14025
+pyopenapi_gen-0.15.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+pyopenapi_gen-0.15.0.dist-info/entry_points.txt,sha256=gxSlNiwom50T3OEZnlocA6qRjGdV0bn6hN_Xr-Ub5wA,56
+pyopenapi_gen-0.15.0.dist-info/licenses/LICENSE,sha256=UFAyTWKa4w10-QerlJaHJeep7G2gcwpf-JmvI2dS2Gc,1088
+pyopenapi_gen-0.15.0.dist-info/RECORD,,