geometry relation support for data search

[ischneider]

• optional arg in filter creation
• add support in CLI filter command
• remove some superfluous respx.mock annotations

I considered adding this for the other CLI commands but this would have required significant reworking how geometry is handled, as we currently pass it along as a top-level item in the request and relation is only supported in a filter.

While it would be conceivable to unpack a user's filters and merge in a geometry filter, this felt beyond the scope of the issue.

In other words, with this MR, the only way to use relation via the CLI is to use planet data filter --geom=... --geom-relation=within | planet data search psscene --filter=-

Related Issue(s):

Closes #1204

Proposed Changes:

For inclusion in changelog (if applicable):

1. Add relation support for geometry filter in data

PR Checklist:

• This PR is as small and focused as possible
• If this PR includes proposed changes for inclusion in the changelog, the title of this PR summarizes those changes and is ready for inclusion in the Changelog.
• I have updated docstrings for function changes and docs in the 'docs' folder for user interface / behavior changes
• This PR does not break any examples or I have updated them

[Copilot]

Pull request overview

This PR adds support for specifying geometry relation types (intersects, contains, within, disjoint) when filtering data searches by geometry. Previously, only the default "intersects" relation was supported implicitly.

• Added optional relation parameter to geometry_filter() function with validation
• Added --geom-relation CLI option to the planet data filter command
• Removed superfluous @respx.mock decorators from tests that don't require HTTP mocking

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
planet/data_filter.py	Added optional relation parameter to geometry_filter() function with validation logic to ensure only valid relation types are accepted
planet/cli/data.py	Added --geom-relation CLI option and removed the geom_to_filter callback function, now calling geometry_filter() directly in the filter command
tests/integration/test_data_cli.py	Added test for geometry relation feature and removed unnecessary @respx.mock decorators from tests that don't make HTTP requests

Comments suppressed due to low confidence (1)

planet/data_filter.py:225

• The docstring should clarify that the relation parameter is optional and provide more context. Consider updating line 225 to: "The GeometryFilter can be used to search for items with a footprint geometry that has the specified spatial relationship (intersects, contains, within, or disjoint) with the specified geometry. The default relationship is intersects."

    The GeometryFilter can be used to search for items with a footprint
    geometry which intersects with the specified geometry.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The geometry_filter function is called unconditionally, but it will fail when geom is None (which happens when --geom is not specified). The removed geom_to_filter callback previously handled this by only calling geometry_filter when geom was not None. This should be changed to: geometry = data_filter.geometry_filter(geom, geom_relation) if geom else None

Suggested change

	geometry = data_filter.geometry_filter(geom, geom_relation)
	geometry = data_filter.geometry_filter(geom, geom_relation) if geom else None

[asonnenschein]

nit: I agree with co-pilot's feedback about this docstring https://github.com/planetlabs/planet-client-python/pull/1205/changes#r2662508698. Otherwise LGTM!