Skip to content

Conversation

@daveey
Copy link
Contributor

@daveey daveey commented Jan 17, 2026

Add CollectiveFilter that checks if target belongs to a specific collective.
Used by events to select objects by their collective membership.

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Copy link
Contributor Author

daveey commented Jan 17, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • add-to-merge-queue - adds this PR to the back of the merge queue
  • add-to-merge-queue-as-hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@blacksmith-sh

This comment has been minimized.

@daveey daveey marked this pull request as ready for review January 17, 2026 04:08
@daveey daveey requested a review from sasmith January 17, 2026 04:08
@datadog-official

This comment has been minimized.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 89bfaf4216

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 89bfaf4 to cdb5d2d Compare January 17, 2026 04:30
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch 2 times, most recently from 9386e8d to c025c5f Compare January 17, 2026 04:37
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from cdb5d2d to 162ef91 Compare January 17, 2026 04:37
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch from c025c5f to 76dc6a7 Compare January 17, 2026 04:42
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 162ef91 to 2ea8636 Compare January 17, 2026 04:42
@daveey daveey changed the base branch from 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply to graphite-base/4956 January 17, 2026 04:46
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 2ea8636 to 3155039 Compare January 17, 2026 04:49
@daveey daveey force-pushed the graphite-base/4956 branch from 76dc6a7 to f2eb41e Compare January 17, 2026 04:49
@daveey daveey changed the base branch from graphite-base/4956 to 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply January 17, 2026 04:49
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 3155039 to cc254e0 Compare January 17, 2026 05:07
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch from f2eb41e to af6dca4 Compare January 17, 2026 05:07
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from cc254e0 to 75cd391 Compare January 17, 2026 05:17
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch from af6dca4 to 1c95e70 Compare January 17, 2026 05:17
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 75cd391 to e04bec5 Compare January 17, 2026 05:19
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch 2 times, most recently from 2fdbe93 to b0f9602 Compare January 17, 2026 05:25
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from e04bec5 to 094c395 Compare January 17, 2026 05:25
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch from b0f9602 to 481ff9d Compare January 17, 2026 05:43
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 094c395 to fd2d361 Compare January 17, 2026 05:43
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply branch from 481ff9d to 93ddb90 Compare January 17, 2026 07:04
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from fd2d361 to 54f81b3 Compare January 17, 2026 07:04
}

// Check if the collective name matches
return coll->name == _config.collective_name;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

check by id

@daveey daveey changed the base branch from 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply to graphite-base/4956 January 18, 2026 05:18
…bership

Add CollectiveFilter that checks if target belongs to a specific collective.
Used by events to select objects by their collective membership.

Python:
- CollectiveFilterConfig with filter_type="collective" and collective field
- BelongsToCollective() helper function
- Added to AnyFilter union

C++:
- CollectiveFilterConfig struct in handler_config.hpp
- CollectiveFilter class in filter.hpp
- Added to FilterConfig variant

Tests:
- test_collective_filter.py with creation, helper, serialization tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@daveey daveey force-pushed the graphite-base/4956 branch from 93ddb90 to 8adb6df Compare January 18, 2026 05:34
@daveey daveey force-pushed the 01-16-feat_mettagrid_add_collectivefilter_for_filtering_by_collective_membership_add_collectivefilter_that_checks_if_target_belongs_to_a_specific_collective._used_by_events_to_select_objects_by_their_collective_membership._co-authored branch from 54f81b3 to 83c898c Compare January 18, 2026 05:34
@daveey daveey changed the base branch from graphite-base/4956 to 01-16-feat_mettagrid_add_objecttypefilter_for_filtering_by_object_type_add_objecttypefilter_that_checks_if_target_has_a_specific_tag._used_by_events_to_select_which_objects_to_apply_mutations_to._co-authored-by_claude_opus_4.5_noreply January 18, 2026 05:34
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