Skip to content

Conversation

@totalimmersion
Copy link
Contributor

Summary

Add class and response field for the new predefined_filter property returned in QueryChannels responses when a predefined filter is used.

Changes

  • Added ParsedPredefinedFilterResponse static class containing:
    • name: The predefined filter name
    • filter: The interpolated filter map
    • sort: Optional interpolated sort list
  • Updated ChannelListResponse to include predefinedFilter field

Related

🤖 Generated with Claude Code

Add class and response field for the new `predefined_filter` property
returned in QueryChannels responses when a predefined filter is used.

- Added ParsedPredefinedFilterResponse static class
- Updated ChannelListResponse to include predefinedFilter field
- Added unit tests for JSON deserialization

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@totalimmersion totalimmersion force-pushed the feature/CHA-2354-predefined-filter-response branch from 2b7d26b to d8f4e6b Compare February 1, 2026 20:45
totalimmersion and others added 8 commits February 3, 2026 09:22
- Remove unused ParsedPredefinedFilterResponse import
- Fix text block formatting (opening """ on separate line)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The project uses Java 11 which doesn't support text blocks.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use the same visibility and deserialization settings as DefaultClient.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Extend BasicTest like other tests in the project to ensure proper
test discovery and integration with the test infrastructure.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Verify that the new model classes have the expected fields and getters.
Avoid JSON deserialization tests which caused CI issues.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.

3 participants