omnibase_infra 0.3.0__py3-none-any.whl → 0.3.1__py3-none-any.whl

omnibase_infra/__init__.py

@@ -76,7 +76,7 @@ See Also
 - Runtime kernel: omnibase_infra.runtime.service_kernel
 """
 
-__version__ = "0.3.0"
+__version__ = "0.3.1"
 
 from . import (
     enums,

omnibase_infra/runtime/db/postgres_repository_runtime.py

@@ -125,7 +125,10 @@ logger = logging.getLogger(__name__)
 #
 # =============================================================================
 _ORDER_BY_PATTERN = re.compile(r"\bORDER\s+BY\b", re.IGNORECASE)
-_LIMIT_PATTERN = re.compile(r"\bLIMIT\s+(\d+)\b", re.IGNORECASE)
+# Pattern to detect numeric LIMIT for validation (e.g., LIMIT 100)
+_LIMIT_NUMERIC_PATTERN = re.compile(r"\bLIMIT\s+(\d+)\b", re.IGNORECASE)
+# Pattern to detect parameterized LIMIT (e.g., LIMIT $1) - cannot validate at build time
+_LIMIT_PARAM_PATTERN = re.compile(r"\bLIMIT\s+\$\d+\b", re.IGNORECASE)
 
 
 class PostgresRepositoryRuntime:
@@ -392,6 +395,11 @@ class PostgresRepositoryRuntime:
             - If no ORDER BY and PK declared: inject ORDER BY {pk}
             - If no ORDER BY and no PK: HARD ERROR
 
+        When injecting ORDER BY, the clause is inserted BEFORE any existing
+        LIMIT clause to produce valid SQL. For example:
+            - Input:  "SELECT * FROM users LIMIT $1"
+            - Output: "SELECT * FROM users ORDER BY id LIMIT $1"
+
         Args:
             sql: The SQL query to potentially modify.
             op_name: Operation name for error context.
@@ -423,7 +431,21 @@ class PostgresRepositoryRuntime:
 
         # Inject ORDER BY using configured order or just PK
         order_by = self._config.default_order_by or pk_column
-        return f"{sql.rstrip().rstrip(';')} ORDER BY {order_by}"
+
+        # Check if LIMIT exists - ORDER BY must be inserted BEFORE LIMIT
+        param_match = _LIMIT_PARAM_PATTERN.search(sql)
+        numeric_match = _LIMIT_NUMERIC_PATTERN.search(sql)
+        limit_match = param_match or numeric_match
+
+        if limit_match:
+            # Insert ORDER BY before LIMIT
+            limit_start = limit_match.start()
+            return (
+                f"{sql[:limit_start].rstrip()} ORDER BY {order_by} {sql[limit_start:]}"
+            )
+        else:
+            # No LIMIT, append at end
+            return f"{sql.rstrip().rstrip(';')} ORDER BY {order_by}"
 
     def _inject_limit(
         self,
@@ -434,9 +456,10 @@ class PostgresRepositoryRuntime:
         """Inject or validate LIMIT clause for multi-row results.
 
         Rules:
-            - If LIMIT > max_row_limit: HARD ERROR
+            - If parameterized LIMIT (e.g., $1): OK (no change, can't validate at build time)
+            - If numeric LIMIT > max_row_limit: HARD ERROR
+            - If numeric LIMIT <= max_row_limit: OK (no change)
             - If no LIMIT: inject LIMIT {max_row_limit}
-            - If LIMIT <= max_row_limit: OK (no change)
 
         Args:
             sql: The SQL query to potentially modify.
@@ -447,13 +470,18 @@ class PostgresRepositoryRuntime:
             SQL with LIMIT clause (injected or original).
 
         Raises:
-            RepositoryContractError: LIMIT exceeds max_row_limit.
+            RepositoryContractError: Numeric LIMIT exceeds max_row_limit.
         """
         max_limit = self._config.max_row_limit
-        limit_match = _LIMIT_PATTERN.search(sql)
 
+        # Check for parameterized LIMIT (e.g., LIMIT $1) - can't validate at build time
+        if _LIMIT_PARAM_PATTERN.search(sql):
+            return sql
+
+        # Check for numeric LIMIT (e.g., LIMIT 100) - can validate
+        limit_match = _LIMIT_NUMERIC_PATTERN.search(sql)
         if limit_match:
-            # Existing LIMIT - validate it
+            # Existing numeric LIMIT - validate it
             existing_limit = int(limit_match.group(1))
             if existing_limit > max_limit:
                 raise RepositoryContractError(

{omnibase_infra-0.3.0.dist-info → omnibase_infra-0.3.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: omnibase_infra
-Version: 0.3.0
+Version: 0.3.1
 Summary: ONEX Infrastructure - Service integration and database infrastructure tools
 License: MIT
 License-File: LICENSE

{omnibase_infra-0.3.0.dist-info → omnibase_infra-0.3.1.dist-info}/RECORD

@@ -1,4 +1,4 @@
-omnibase_infra/__init__.py,sha256=GaHfYUvSgKlKfO2_asrsWp0245bbZP91s5O__WZPkHI,3853
+omnibase_infra/__init__.py,sha256=cUEvzmpkZWPGbrKvpI3JpMQ4HjR_YEF_9gOZePu-AQQ,3853
 omnibase_infra/adapters/adapter_onex_tool_execution.py,sha256=gKiRTr3pc7-DNAS9_bSmLWssOr1FDCiciwE16cQF3Ro,15377
 omnibase_infra/capabilities/__init__.py,sha256=O6qQjWNwe0ZQt5hpGMMLx4CgagdpxBJzMGVpowi8eKY,462
 omnibase_infra/capabilities/capability_inference_rules.py,sha256=2VDLhvBlDOYyi1eCQGSNB8eXf2JQ4cG9Xftsb3P2xm0,8166
@@ -639,7 +639,7 @@ omnibase_infra/runtime/contract_loaders/operation_bindings_loader.py,sha256=TaOs
 omnibase_infra/runtime/db/__init__.py,sha256=WZLFf7XIXAefkUIiA1sg3Npw120P9zaqK6862mq7wgM,2380
 omnibase_infra/runtime/db/models/__init__.py,sha256=ENQ696584GQHSvbVN85PgEo3Agij5Oo10Vz7-FoVYBE,1377
 omnibase_infra/runtime/db/models/model_repository_runtime_config.py,sha256=4X231FUSe8UywlFIhb4cuyEBk2G0FiJP7-X5-f8Zop0,7826
-omnibase_infra/runtime/db/postgres_repository_runtime.py,sha256=74M8qR6We7DitPwstrFosJiiWXyjBpeVsfn4IXxuo2Q,20297
+omnibase_infra/runtime/db/postgres_repository_runtime.py,sha256=cijrpfrK1vwt6RqqHIi1PTfqPxfQ3IQT82dDx4eGQAE,21673
 omnibase_infra/runtime/dispatch_context_enforcer.py,sha256=44McRbtsqkGTiz7dQTOtV4AmisYOlVKKhI_1n5Jwziw,16620
 omnibase_infra/runtime/emit_daemon/__init__.py,sha256=wnc8rcQCu0vO10y8uqgoEFUsBbYQsJJiKYrJoLgEhsU,3012
 omnibase_infra/runtime/emit_daemon/cli.py,sha256=KP0h3f_cfmNcH-lsVl8rWc-m4Xlhuc0yN5VnNtvNM68,26117
@@ -884,8 +884,8 @@ omnibase_infra/validation/validator_routing_coverage.py,sha256=51vYEi5NhU6wqohk2
 omnibase_infra/validation/validator_runtime_shape.py,sha256=9Xl37_dNDinT7nr-BoJ9fG3O5cT1_j6N-C7bojwQi3g,39033
 omnibase_infra/validation/validator_security.py,sha256=Yn_kwxBtPzEj1GzrW0OafL-bR-FgoqdeFzseQqF2_B8,20622
 omnibase_infra/validation/validator_topic_category.py,sha256=1y6STpSNIJsE2AmX5f8PVzPuOfoQmedgocSbEqOdtrM,48798
-omnibase_infra-0.3.0.dist-info/METADATA,sha256=0N0aDXnWUfwLdAWoj6ivITqhPj681TUwj9salQS_Ens,7554
-omnibase_infra-0.3.0.dist-info/WHEEL,sha256=zp0Cn7JsFoX2ATtOhtaFYIiE2rmFAD4OcMhtUki8W3U,88
-omnibase_infra-0.3.0.dist-info/entry_points.txt,sha256=8jORmczvjPYrahcGRwxtuVqh8KIFOfb04GgnRnQfNsQ,166
-omnibase_infra-0.3.0.dist-info/licenses/LICENSE,sha256=CwYWppeN0neFHYdbbUdHbR4_PiyTYnmk6ufFWuVsL7I,1070
-omnibase_infra-0.3.0.dist-info/RECORD,,
+omnibase_infra-0.3.1.dist-info/METADATA,sha256=Y2H_CE7DKXZubx6YDg4IIwvUG1cYsMuB0uJ1sAb7CEI,7554
+omnibase_infra-0.3.1.dist-info/WHEEL,sha256=zp0Cn7JsFoX2ATtOhtaFYIiE2rmFAD4OcMhtUki8W3U,88
+omnibase_infra-0.3.1.dist-info/entry_points.txt,sha256=8jORmczvjPYrahcGRwxtuVqh8KIFOfb04GgnRnQfNsQ,166
+omnibase_infra-0.3.1.dist-info/licenses/LICENSE,sha256=CwYWppeN0neFHYdbbUdHbR4_PiyTYnmk6ufFWuVsL7I,1070
+omnibase_infra-0.3.1.dist-info/RECORD,,