Skip to content

fix(gamma): preserve market group item title#119

Merged
cesarenaldi merged 1 commit into
mainfrom
feature/dev-316-preserve-groupitemtitle
Jun 23, 2026
Merged

fix(gamma): preserve market group item title#119
cesarenaldi merged 1 commit into
mainfrom
feature/dev-316-preserve-groupitemtitle

Conversation

@cesarenaldi

@cesarenaldi cesarenaldi commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add group_item_title to the Market model
  • preserve groupItemTitle when parsing market responses
  • cover the field in the existing gamma model test

Verification

  • uv run pytest tests/unit/test_gamma_models.py
  • uv run ruff check
  • uv run pyright

Note

Low Risk
Optional string field on a Pydantic model with a unit test; no auth, trading, or behavioral logic changes.

Overview
Gamma Market parsing now keeps groupItemTitle from API payloads as group_item_title, matching how grouped markets label individual legs.

The new field is on the model (with a groupItemTitle alias) and is copied in _normalize_market when flattening raw responses. A unit test asserts the value survives Market.parse_response.

Reviewed by Cursor Bugbot for commit 264ef60. Bugbot is set up for automated code reviews on this repo. Configure here.

@cesarenaldi cesarenaldi merged commit e4b8185 into main Jun 23, 2026
7 checks passed
@cesarenaldi cesarenaldi deleted the feature/dev-316-preserve-groupitemtitle branch June 23, 2026 13:17
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.

1 participant