ostruct-cli 0.8.2__py3-none-any.whl → 0.8.3__py3-none-any.whl

ostruct/cli/template_validation.py

@@ -50,6 +50,7 @@ Notes:
 """
 
 import logging
+import re
 from typing import (
     Any,
     Callable,
@@ -150,6 +151,45 @@ def find_loop_vars(nodes: List[Node]) -> Set[str]:
     return loop_vars
 
 
+def _extract_variable_name_from_jinja_error(error_message: str) -> str:
+    """Extract the actual variable name from a Jinja2 UndefinedError message.
+
+    Handles various Jinja2 error message formats:
+    - "'variable_name' is undefined"
+    - "'object_description' has no attribute 'property_name'"
+    - Other formats
+
+    Args:
+        error_message: The string representation of the Jinja2 UndefinedError
+
+    Returns:
+        The extracted variable name, or a sanitized version if parsing fails
+    """
+    # Pattern 1: Standard undefined variable: "'variable_name' is undefined"
+    match = re.match(r"'([^']+)' is undefined", error_message)
+    if match:
+        return match.group(1)
+
+    # Pattern 2: Attribute access on object: "'object' has no attribute 'property'"
+    # In this case, we want to extract just the property name, not the object description
+    match = re.match(r"'[^']+' has no attribute '([^']+)'", error_message)
+    if match:
+        property_name = match.group(1)
+        return property_name
+
+    # Pattern 3: Try to find any quoted identifier that looks like a variable name
+    # Look for quoted strings that contain only valid Python identifier characters
+    quoted_parts: List[str] = re.findall(r"'([^']+)'", error_message)
+    for part in quoted_parts:
+        # Check if this looks like a variable name (not a class name or description)
+        if re.match(r"^[a-zA-Z_][a-zA-Z0-9_]*$", part) and "." not in part:
+            return part
+
+    # Fallback: If we can't parse it properly, return a generic message
+    # This avoids exposing internal class names to users
+    return "unknown_variable"
+
+
 def validate_template_placeholders(
     template: str, template_context: Optional[Dict[str, Any]] = None
 ) -> None:
@@ -343,9 +383,7 @@ def validate_template_placeholders(
             env.from_string(template).render(validation_context)
         except jinja2.UndefinedError as e:
             # Convert Jinja2 undefined errors to TaskTemplateVariableError with helpful message
-            var_name = str(e).split("'")[
-                1
-            ]  # Extract variable name from error message
+            var_name = _extract_variable_name_from_jinja_error(str(e))
             error_msg = (
                 f"Missing required template variable: {var_name}\n"
                 f"Available variables: {', '.join(sorted(available_vars))}\n"

ostruct/cli/types.py

@@ -35,7 +35,6 @@ class CLIParams(TypedDict, total=False):
     no_progress: bool
     api_key: Optional[str]
     verbose: bool
-    debug_openai_stream: bool
     show_model_schema: bool
     debug_validation: bool
     temperature: Optional[float]
@@ -76,3 +75,17 @@ class CLIParams(TypedDict, total=False):
     tool_files: List[
         Tuple[str, str]
     ]  # List of (tool, path) tuples from --file-for
+    web_search: bool
+    debug: bool
+    show_templates: bool
+    debug_templates: bool
+    show_context: bool
+    show_context_detailed: bool
+    show_pre_optimization: bool
+    show_optimization_diff: bool
+    no_optimization: bool
+    show_optimization_steps: bool
+    optimization_step_detail: str
+    help_debug: bool
+    enabled_features: List[str]  # List of feature names to enable
+    disabled_features: List[str]  # List of feature names to disable

{ostruct_cli-0.8.2.dist-info → ostruct_cli-0.8.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: ostruct-cli
-Version: 0.8.2
+Version: 0.8.3
 Summary: CLI for OpenAI Structured Output with Multi-Tool Integration
 Author: Yaniv Golan
 Author-email: yaniv@golan.name
@@ -22,6 +22,7 @@ Requires-Dist: chardet (>=5.0.0,<6.0)
 Requires-Dist: click (>=8.1.7,<9.0)
 Requires-Dist: flake8 (>=6.0,<7.0) ; extra == "dev"
 Requires-Dist: flake8-pyproject (>=1.2.3,<2.0) ; extra == "dev"
+Requires-Dist: hypothesis (>=6.0.0,<7.0) ; extra == "dev"
 Requires-Dist: ijson (>=3.2.3,<4.0)
 Requires-Dist: jinja2 (>=3.1.2,<4.0)
 Requires-Dist: jsonschema (>=4.23.0,<5.0)
@@ -37,6 +38,7 @@ Requires-Dist: pygments (>=2.15.0,<3.0)
 Requires-Dist: pytest (>=8.3.4,<9.0) ; extra == "dev"
 Requires-Dist: pytest-asyncio (>=0.25.2,<1.0) ; extra == "dev"
 Requires-Dist: pytest-mock (>=3.14.0,<4.0) ; extra == "dev"
+Requires-Dist: pytest-rerunfailures (>=12.0,<13.0) ; extra == "dev"
 Requires-Dist: python-dotenv (>=1.0.1,<2.0) ; extra == "dev"
 Requires-Dist: pyyaml (>=6.0.2,<7.0)
 Requires-Dist: sphinx (>=7.0,<8.0) ; extra == "dev"
@@ -302,7 +304,7 @@ ostruct run analysis.j2 schema.json -fc data.csv
 ostruct run search.j2 schema.json -fs documentation.pdf
 
 # Web Search (real-time information)
-ostruct run research.j2 schema.json --web-search -V topic="latest AI developments"
+ostruct run research.j2 schema.json --enable-tool web-search -V topic="latest AI developments"
 
 # Multiple tools with one file
 ostruct run template.j2 schema.json --file-for code-interpreter shared.json --file-for file-search shared.json
@@ -370,6 +372,7 @@ tools:
   code_interpreter:
     auto_download: true
     output_directory: "./output"
+    download_strategy: "two_pass_sentinel"  # Enable reliable file downloads
 
 mcp:
   custom_server: "https://my-mcp-server.com"
@@ -384,6 +387,35 @@ Load custom configuration:
 ostruct --config my-config.yaml run template.j2 schema.json
 ```
 
+### Code Interpreter File Downloads
+
+**Important**: If you're using Code Interpreter with structured output (JSON schemas), you may need to enable the two-pass download strategy to ensure files are downloaded reliably.
+
+#### Option 1: CLI Flags (Recommended for one-off usage)
+
+```bash
+# Enable reliable file downloads for this run
+ostruct run template.j2 schema.json -fc data.csv --enable-feature ci-download-hack
+
+# Force single-pass mode (override config)
+ostruct run template.j2 schema.json -fc data.csv --disable-feature ci-download-hack
+```
+
+#### Option 2: Configuration File (Recommended for persistent settings)
+
+```yaml
+# ostruct.yaml
+tools:
+  code_interpreter:
+    download_strategy: "two_pass_sentinel"  # Enables reliable file downloads
+    auto_download: true
+    output_directory: "./downloads"
+```
+
+**Why this is needed**: OpenAI's structured output mode can prevent file download annotations from being generated. The two-pass strategy works around this by making two API calls: one to generate files (without structured output), then another to ensure schema compliance. For detailed technical information, see [docs/known-issues/2025-06-responses-ci-file-output.md](docs/known-issues/2025-06-responses-ci-file-output.md).
+
+**Performance**: The two-pass strategy approximately doubles token usage but ensures reliable file downloads when using structured output with Code Interpreter.
+
 ## Get Started Quickly
 
 🚀 **New to ostruct?** Follow our [step-by-step quickstart guide](https://ostruct.readthedocs.io/en/latest/user-guide/quickstart.html) featuring Juno the beagle for a hands-on introduction.
@@ -628,6 +660,60 @@ The registry file is stored at `~/.openai_structured/config/models.yml` and is a
 
 The update command uses HTTP conditional requests (If-Modified-Since headers) to check if the remote registry has changed before downloading, ensuring efficient updates.
 
+# Testing
+
+## Running Tests
+
+The test suite is divided into two categories:
+
+### Regular Tests (Default)
+
+```bash
+# Run all tests (skips live tests by default)
+pytest
+
+# Run specific test file
+pytest tests/test_config.py
+
+# Run with verbose output
+pytest -v
+```
+
+### Live Tests
+
+Live tests make real API calls to OpenAI and require a valid API key. They are skipped by default.
+
+```bash
+# Run only live tests (requires OPENAI_API_KEY)
+pytest -m live
+
+# Run all tests including live tests
+pytest -m "live or not live"
+
+# Run specific live test
+pytest tests/test_responses_annotations.py -m live
+```
+
+**Live tests include:**
+
+- Tests that make actual OpenAI API calls
+- Tests that run `ostruct` commands via subprocess
+- Tests that verify real API behavior and file downloads
+
+**Requirements for live tests:**
+
+- Valid `OPENAI_API_KEY` environment variable
+- Internet connection
+- May incur API costs
+
+## Test Markers
+
+- `@pytest.mark.live` - Tests that make real API calls or run actual commands
+- `@pytest.mark.no_fs` - Tests that need real filesystem (not pyfakefs)
+- `@pytest.mark.slow` - Performance/stress tests
+- `@pytest.mark.flaky` - Tests that may need reruns
+- `@pytest.mark.mock_openai` - Tests using mocked OpenAI client
+
 <!--
 MAINTAINER NOTE: After editing this README, please test GitHub rendering by:
 1. Creating a draft PR or pushing to a test branch

{ostruct_cli-0.8.2.dist-info → ostruct_cli-0.8.3.dist-info}/RECORD

@@ -3,31 +3,32 @@ ostruct/cli/__init__.py,sha256=e-DtWRviCr3fIAJH4cB4UAvles3-rqnhJaTOlBn9TKs,871
 ostruct/cli/base_errors.py,sha256=o-877bJJA8yJWISRPy0KyL6wDu1-_avddmQIfVePuFM,5989
 ostruct/cli/cache_manager.py,sha256=ej3KrRfkKKZ_lEp2JswjbJ5bW2ncsvna9NeJu81cqqs,5192
 ostruct/cli/cli.py,sha256=ZUkQ-iwk4KCKeetPMlvSDwxfXIe4h1-yP80SfJgvi_o,4098
-ostruct/cli/click_options.py,sha256=f2a6XTidy-RJgL64WAgTqRYpMPokY0O8Bb6aQgwMOtk,28298
-ostruct/cli/code_interpreter.py,sha256=sEq-zi0CfotRqEseA9pWTm-UU1fZTH_06WGvWPNwct8,7786
+ostruct/cli/click_options.py,sha256=FbRMfoFgg5icdgewPS8cePSYv6yTnxbRxyDvfFDdRaI,31845
+ostruct/cli/code_interpreter.py,sha256=lnnyEvUh2pGObN9ENpr-X4p0C0oIWiyG1CFQWW4akBQ,16726
 ostruct/cli/commands/__init__.py,sha256=3NHz-WZ9XqrnWWMksoV2MpYpHnjA-EO9lsrBOYeHcjY,723
 ostruct/cli/commands/list_models.py,sha256=yeuQpUGAmRr4uOHS7teuVHkC9dkqN0yKDOEw_n-ehi0,4662
 ostruct/cli/commands/quick_ref.py,sha256=zpD7kI3sfrcc9CiRkzWdASTNX5jAk3HFB7MilzlCsag,2004
-ostruct/cli/commands/run.py,sha256=1M5KYhXPLJLFVbfGJHRp88kO25CSN6TP1A0cVex1-7A,4370
+ostruct/cli/commands/run.py,sha256=Cm9Yuf0DLt5CqKfgAYubhQRcvLdK1vqIIuz_ynjfhQ4,6640
 ostruct/cli/commands/update_registry.py,sha256=7DQrPlCJScPVgE2HbFAM7UMap-EdYu58AQWfpI-H7Gw,2483
-ostruct/cli/config.py,sha256=keX5mUiyZ9_CrVHZFpxr7loBAetORRW-ZjNUoQFa_k8,9030
+ostruct/cli/config.py,sha256=7tKI1gWLpTISn5OorGWIx66N1J7XW2aq30hPNISZzQ0,9958
 ostruct/cli/cost_estimation.py,sha256=08hyE-kM5QYzj7y-KB3lMD_RxCMoM_Ve3-IQlSpJAo4,4483
-ostruct/cli/errors.py,sha256=WUa_LwwBMynrP9WVgS199ovir1XJFLZIRw5QoPRLVwY,25145
+ostruct/cli/errors.py,sha256=sVOM_ZKvrtnmT8_WR-DfX2crs6HmvmwrbNQlxm-5_Ew,24669
 ostruct/cli/exit_codes.py,sha256=gdvB1dpu0LalEUZrofpk5K6aTQ24n5AfkAK5umludHU,365
 ostruct/cli/explicit_file_processor.py,sha256=B6yUPbyn6MVd81GcyMVpORFwyaHFFESLwFixp2B6M5w,19767
 ostruct/cli/field_utils.py,sha256=bcRi1qQ0Ac2UCfrKSQ677_yu-VzaShm_zN7QLf98qc0,1939
-ostruct/cli/file_info.py,sha256=Vgx-unid2ODSYbJxzgC2ndEeh-DS6eADsrqG34_C820,16182
+ostruct/cli/file_info.py,sha256=s8AHPtyU3__2LRJCCIEPuODYM43is4Y2Q_VDhBOl3XU,17395
 ostruct/cli/file_list.py,sha256=alRAguq4tj1zH0_qlWaRoyHo1G1Xmxqu9Xd4QP-zYP0,20268
 ostruct/cli/file_search.py,sha256=N12mkji2ttvItLVPyAWE3KEfhTv8hV5IXPrQME2UFdE,15313
-ostruct/cli/file_utils.py,sha256=kLZKiZadkIiYvs3B_gngqf_PwMRMYIWw49i8GTGVbkk,23607
+ostruct/cli/file_utils.py,sha256=JZprQ-1LHQzI3eBfeCIS6VmxTa2fGUZHygGC8gcwpJM,24367
+ostruct/cli/json_extract.py,sha256=ZleIxat8g-JnA9VVqWgJaKxN7QzL25itQ8aP0Gb5e4Q,2650
 ostruct/cli/mcp_integration.py,sha256=EXIz_KbYC4srWZxuKVkINeaHgHvEYB9l1uhptpLNn5I,18774
-ostruct/cli/model_creation.py,sha256=l-FHxCaAmZ2akKszoRg8gf1KgJxeT10Pt8YX39bXeT4,23261
+ostruct/cli/model_creation.py,sha256=HGo8Qv7eBF8Co463IR7RcbTCQcaOvd_cBGuRodRCAa4,23261
 ostruct/cli/model_validation.py,sha256=j2az3q88-Ljm2cMMgZ8p_-gcp1vKQnWCknnw0y0YlAw,6675
 ostruct/cli/path_utils.py,sha256=j44q1OoLkqMErgK-qEuhuIZ1VyzqRIvNgxR1et9PoXA,4813
 ostruct/cli/progress.py,sha256=rj9nVEco5UeZORMbzd7mFJpFGJjbH9KbBFh5oTE5Anw,3415
 ostruct/cli/progress_reporting.py,sha256=MBjALM4pmPd_d9YuXqH162-tkC6DDKYmz-pJPSGLTfk,13669
 ostruct/cli/registry_updates.py,sha256=ohiHdlfrocvThpR_ZjMyqulDKFjRM1hIFKOlNzpaqHg,5138
-ostruct/cli/runner.py,sha256=pNw0TzdpDElzaUYxb8R-rzSPqa1MOzUM-hPaOvpETI4,54045
+ostruct/cli/runner.py,sha256=_xiT9_ywk5lNz3P1qFpqOcJGwXqAE4xQ8Zxc9p6x0Ko,65192
 ostruct/cli/schema_utils.py,sha256=9LnsjxEKg6RIfXQB3nS3pyDggm5n-4-thXf92897gJU,3590
 ostruct/cli/schema_validation.py,sha256=ohEuxJ0KF93qphj0JSZDnrxDn0C2ZU37g-U2JY03onM,8154
 ostruct/cli/security/__init__.py,sha256=CQpkCgTFYlA1p6atpQeNgIKtE4LZGUKt4EbytbGKpCs,846
@@ -41,6 +42,7 @@ ostruct/cli/security/security_manager.py,sha256=HFCqJ5kAhaZlFnPTEs6MKNM8JeE2q79d
 ostruct/cli/security/symlink_resolver.py,sha256=wtZdJ_T_0FOy6B1P5ty1odEXQk9vr8BzlWeAFD4huJE,16744
 ostruct/cli/security/types.py,sha256=15yuG_T4CXyAFFFdSWLjVS7ACmDGIPXhQpZ8awcDwCQ,2991
 ostruct/cli/security/windows_paths.py,sha256=qxC2H2kLwtmQ7YePYde3UrmOJcGnsLEebDLh242sUaI,13453
+ostruct/cli/sentinel.py,sha256=69faYPrhVJmEpYNLsCtf1HF96aan3APqXZdIjxBNZYo,798
 ostruct/cli/serialization.py,sha256=ec0UswDE2onwtZVUoZaMCsGv6zW_tSKdBng2qVo6Ucs,704
 ostruct/cli/services.py,sha256=nLYUbF3DDNuilh7j9q_atUOjTAWta7bxTS3G-zkveaA,21621
 ostruct/cli/template_debug.py,sha256=1tP3pTdvQwHcMneYnpWnS2Jr5EQijjyCSQDh5DPXvDE,24698
@@ -49,21 +51,21 @@ ostruct/cli/template_env.py,sha256=7ZcGKyqlkV-ZS2sSUvzyaLsnlvPmfCKUG0epIV8TD6o,1
 ostruct/cli/template_extensions.py,sha256=_lomtDGMGxMfpw05v_-daJ0JbhRm_r_-uEJlPAjbpkI,2699
 ostruct/cli/template_filters.py,sha256=S9ad8she0lXRr0PDQVkNrPWbZg7GnybptEXUMkQKOyo,20657
 ostruct/cli/template_io.py,sha256=yUWO-8rZnSdX97DTMSEX8fG9CP1ISsOhm2NZN3Fab9A,8821
-ostruct/cli/template_optimizer.py,sha256=_eQbHQP6fH5tbWLwXtLSLDYlGQ8hLpZKVgXsbcWF2xY,16394
-ostruct/cli/template_processor.py,sha256=AWtWEpJvCCFB1PzoDp6EovV4RaIMbY316cb-Iq5o48c,41726
-ostruct/cli/template_rendering.py,sha256=5i7gtFjW2QKVW3e_xK0Llrurn0I4e_gG0gzn2p9IFTc,15030
+ostruct/cli/template_optimizer.py,sha256=gwngZ5XrIkU5BrjFibWT0VRf9WrH6NS4SAlrUzme1C8,16684
+ostruct/cli/template_processor.py,sha256=hp9_ZQmVdN6JwCSQJoHB1mVq_p0sTZSojb7c4Y4Vjvs,47301
+ostruct/cli/template_rendering.py,sha256=41f6An7cpI04z2qyesmnXgPVMD1je3ePA6Xz8uuA06U,16705
 ostruct/cli/template_schema.py,sha256=ckH4rUZnEgfm_BHS9LnMGr8LtDxRmZ0C6UBVrSp8KTc,19604
 ostruct/cli/template_utils.py,sha256=MZdXXjL-x-IXX-5Y8GWopGNBkDE2ItLdCuCl0QWFR_U,14968
-ostruct/cli/template_validation.py,sha256=AXa2zmsws1j-0CTFlp7fMiZR43iNLnj4h467up2JdgU,12693
+ostruct/cli/template_validation.py,sha256=9L1JgM87NIV8JZUoDUT4dRjSDfBl9L8Yl2aVL_ahkmA,14294
 ostruct/cli/token_utils.py,sha256=r4KPEO3Sec18Q6mU0aClK6XGShvusgUggXEQgEPPlaA,1369
 ostruct/cli/token_validation.py,sha256=gmyPJ7B2gC_jSx_1wKZq87DEoFulj23X1XnVpO_aRNA,9930
-ostruct/cli/types.py,sha256=vCZFBUkeL1QUBM5tTSjEWf_5BUttlyC40kFfpNfTrrY,2474
+ostruct/cli/types.py,sha256=6nARJ4MF5HIank0t6KGU-PPHC0VpFh3R8fNJZBXwgbA,2903
 ostruct/cli/unattended_operation.py,sha256=kI95SSVJC_taxORXQYrce_qLEnuKc6edwn9tMOye-qs,9383
 ostruct/cli/utils.py,sha256=uY7c0NaINHWfnl77FcPE3TmYUXv3RqEeUTjrCMDij9A,922
 ostruct/cli/validators.py,sha256=lbxAUUVS5TPJ7HdYZ5yB7gUjJqfcClZCuh0oktoq0E0,15291
 ostruct/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ostruct_cli-0.8.2.dist-info/LICENSE,sha256=DmGAkaYzhrdzTB9Y2Rvfzd3mJiF9ZrTOhxN8t6wrfHA,1098
-ostruct_cli-0.8.2.dist-info/METADATA,sha256=-DJtx83jOezthaQNrOBZpoL_NEH9jbdgCIP1YXOSkGw,21976
-ostruct_cli-0.8.2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-ostruct_cli-0.8.2.dist-info/entry_points.txt,sha256=NFq9IuqHVTem0j9zKjV8C1si_zGcP1RL6Wbvt9fUDXw,48
-ostruct_cli-0.8.2.dist-info/RECORD,,
+ostruct_cli-0.8.3.dist-info/LICENSE,sha256=DmGAkaYzhrdzTB9Y2Rvfzd3mJiF9ZrTOhxN8t6wrfHA,1098
+ostruct_cli-0.8.3.dist-info/METADATA,sha256=mBsqO645XWKto3RgifluCxqkilYH0C14wWrm1Pa6P6I,24827
+ostruct_cli-0.8.3.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+ostruct_cli-0.8.3.dist-info/entry_points.txt,sha256=NFq9IuqHVTem0j9zKjV8C1si_zGcP1RL6Wbvt9fUDXw,48
+ostruct_cli-0.8.3.dist-info/RECORD,,