Skip to content

EPD-2055: read revision overrides correctly - #358

Merged
josorio-autoblocks merged 9 commits into
mainfrom
jose/epd-2055-update-sdks-to-read-revision-overrides
Jun 25, 2025
Merged

EPD-2055: read revision overrides correctly#358
josorio-autoblocks merged 9 commits into
mainfrom
jose/epd-2055-update-sdks-to-read-revision-overrides

Conversation

@josorio-autoblocks

@josorio-autoblocks josorio-autoblocks commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

Greptile Summary

Enhanced prompt revision override functionality in the Python SDK to support both V1 and V2 prompt managers, adding validation and testing context support.

  • Improved AutoblocksPromptManager in autoblocks/_impl/prompts/v2/manager.py with version compatibility checks and support for unified/legacy revision override formats
  • Added comprehensive test suite in tests/autoblocks/test_prompts_v2_manager.py covering revision validation, fallback mechanisms, and override scenarios
  • Disabled prompt refreshing in test environments to ensure consistent behavior during testing
  • Added revision usage tracking for test environments to improve debugging capabilities

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, 3 comments
Edit PR Review Bot Settings | Greptile

Comment thread autoblocks/_impl/prompts/v2/manager.py
Comment thread autoblocks/_impl/prompts/v2/manager.py
Comment thread tests/autoblocks/test_prompts_v2_manager.py
@josorio-autoblocks

Copy link
Copy Markdown
Contributor Author

@adamnolte tested here https://github.com/autoblocksai/python-demo-app/actions/runs/15815588563/job/44573772697 in the Run template logging tests section

Comment thread autoblocks/_impl/prompts/v2/manager.py Outdated
adamnolte
adamnolte previously approved these changes Jun 23, 2025
@josorio-autoblocks
josorio-autoblocks merged commit 2be1ba9 into main Jun 25, 2025
13 of 18 checks passed
@josorio-autoblocks
josorio-autoblocks deleted the jose/epd-2055-update-sdks-to-read-revision-overrides branch June 25, 2025 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants