Skip to content

docs: update PostGIS skill with withinDistance, measurements, transformations, aggregates#66

Open
pyramation wants to merge 1 commit intomainfrom
devin/1774052161-update-postgis-skills
Open

docs: update PostGIS skill with withinDistance, measurements, transformations, aggregates#66
pyramation wants to merge 1 commit intomainfrom
devin/1774052161-update-postgis-skills

Conversation

@pyramation
Copy link
Contributor

Summary

Updates search-postgis.md to document new PostGIS features added in constructive-io/constructive#864. Documentation-only change — no code or scripts modified.

New sections added:

  • withinDistance (ST_DWithin) — compound input operator with point + distance fields, with both GraphQL and SDK examples
  • orderingEquals operator — was implemented but missing from docs
  • Type-specific fields table — documents available subfields per geometry type (Point, LineString, Polygon, Multi*, Collection)
  • Measurement fieldsarea, length, perimeter (geodesic, computed client-side)
  • Transformation fieldscentroid, bbox, numPoints
  • Aggregate functions — ST_Extent, ST_Union, ST_Collect, ST_ConvexHull
  • Cross-references — links to pgpm Docker skill and codegen ORM patterns (no duplicated content)

Review & Testing Checklist for Human

  • Verify withinDistance input shape matches the actual WithinDistanceInput GraphQL type — doc shows { point: GeoJSON, distance: Float }. Check against within-distance-operator.ts in the constructive repo.
  • Verify measurement field availability — doc claims area/perimeter on Polygon, length on LineString. Confirm against measurement-fields.ts.
  • Check cross-reference relative paths../../pgpm/SKILL.md, ../../pgpm/references/docker.md, and ./codegen-orm-patterns.md should resolve correctly from skills/constructive-graphql/references/.

Notes

  • The aggregate functions section is intentionally brief since aggregates depend on the Graphile aggregates system being enabled, which is a separate configuration concern.
  • Geography Point fields use longitude/latitude instead of x/y — this is documented but worth double-checking against the inflection plugin.

Link to Devin session: https://app.devin.ai/sessions/d0a20ee2e5194f4e8c342c6bc12fea07
Requested by: @pyramation

…rmations, aggregates

- Add withinDistance (ST_DWithin) compound input operator with examples
- Add orderingEquals operator (was missing from docs)
- Add type-specific fields table (Point, LineString, Polygon, Multi*, Collection)
- Add measurement fields: area, length, perimeter (geodesic)
- Add transformation fields: centroid, bbox, numPoints
- Add aggregate functions: ST_Extent, ST_Union, ST_Collect, ST_ConvexHull
- Add SDK example for withinDistance query
- Add cross-references to pgpm Docker skill and codegen ORM patterns
- Update 'When to Use' section with operator recommendations
@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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