Skip to content

Rename fullTextSearch composite filter to unifiedSearch in skills docs#82

Merged
pyramation merged 1 commit intomainfrom
devin/1775105890-rename-fullTextSearch-to-unifiedSearch
Apr 2, 2026
Merged

Rename fullTextSearch composite filter to unifiedSearch in skills docs#82
pyramation merged 1 commit intomainfrom
devin/1775105890-rename-fullTextSearch-to-unifiedSearch

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

@pyramation pyramation commented Apr 2, 2026

Summary

Renames all references to the composite GraphQL filter field from fullTextSearch to unifiedSearch across the skills documentation. This disambiguates the composite multi-algorithm filter (which fans out to tsvector + BM25 + trgm simultaneously) from the tsvector-specific fullTextSearchTsv per-adapter filter.

Companion to constructive-io/constructive#956 which renames the actual plugin code, codegen, and tests.

What changed (13 files):

  • graphile-search/SKILL.md — field name, config option (enableFullTextSearchenableUnifiedSearch), description
  • graphile-search/references/ — adapter flag descriptions (supportsTextSearch comment), codegen SDK query examples, adapter interface JSDoc
  • constructive-graphql/SKILL.md — quick start examples, search strategy table, reference table
  • constructive-graphql/references/ — search-composite, search-trigram, search-postgis, search-rag
  • constructive-ai/references/rag-pipeline.md — hybrid search example

What was intentionally left unchanged:

  • fullTextSearchTsv (tsvector adapter filter field name)
  • db.fullTextSearch.create() / .update() / .delete() / .findMany() (metaschema table ORM operations in search-tsvector.md)
  • filterPrefix: 'fullText' (tsvector adapter config)
  • fullTextScalarName (GraphQL scalar config)

Review & Testing Checklist for Human

  • Verify no composite fullTextSearch references were missed: Run grep -r 'fullTextSearch' .agents/skills/ | grep -v 'fullTextSearchTsv' | grep -v 'db\.fullTextSearch' | grep -v 'fullTextSearch\.create' | grep -v 'fullTextSearch\.update' | grep -v 'fullTextSearch\.delete' | grep -v 'fullTextSearch\.findMany' | grep -v 'filterPrefix' | grep -v 'fullTextScalarName' | grep -v 'fullTextBodyTsv' — should return zero results
  • Verify no tsvector/metaschema references were accidentally renamed: Spot-check search-tsvector.md — all db.fullTextSearch.* ORM calls and fullTextSearchTsv filter usages should remain unchanged
  • Merge after constructive#956: This PR should land after (or simultaneously with) the code PR to avoid docs referencing a field name that doesn't exist yet

Notes

  • This is a docs-only change — no code, no tests, no runtime impact.
  • Minor cosmetic: enableUnifiedSearch: true, in graphile-search/SKILL.md line 79 has slightly different trailing whitespace alignment than the surrounding config lines. Non-functional.

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


Open with Devin

Update all references to the composite GraphQL filter field from
fullTextSearch to unifiedSearch across graphile-search, constructive-graphql,
and constructive-ai skills documentation.

TSVector-specific references (fullTextSearchTsv, filterPrefix, etc.) and
metaschema table references (db.fullTextSearch.create(), full_text_search
table) are intentionally left unchanged.

Companion to constructive-io/constructive#956.
@devin-ai-integration
Copy link
Copy Markdown

🤖 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

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@pyramation pyramation merged commit c8223b0 into main Apr 2, 2026
2 checks passed
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