Skip to content

chore: update eigen query map#7416

Open
artsyit wants to merge 1 commit intomainfrom
update-eigen-query-map
Open

chore: update eigen query map#7416
artsyit wants to merge 1 commit intomainfrom
update-eigen-query-map

Conversation

@artsyit
Copy link
Copy Markdown
Collaborator

@artsyit artsyit commented Feb 3, 2026

Greetings human 🤖 this PR was automatically created as part of eigen's deploy process

@artsyit artsyit self-assigned this Feb 3, 2026
@artsyit artsyit force-pushed the update-eigen-query-map branch 3 times, most recently from 8ccc958 to 0f2495e Compare February 5, 2026 15:40
@artsyit artsyit force-pushed the update-eigen-query-map branch 3 times, most recently from 3a588cf to b65e2a6 Compare February 12, 2026 16:32
@github-actions github-actions Bot deleted a comment from claude Bot Feb 12, 2026
@artsyit artsyit force-pushed the update-eigen-query-map branch 2 times, most recently from a0b51c1 to 006c44a Compare February 12, 2026 18:37
@github-actions github-actions Bot deleted a comment from claude Bot Feb 12, 2026
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Feb 12, 2026

Code Review

Summary

This PR automatically updates the Eigen query map with 4 new GraphQL queries and reorders 1 existing query. The changes appear to be part of an automated deployment process.

Issues Found

🟢 No blocking issues found

Areas Reviewed

  • Architecture & Design: The changes follow the established pattern for persisted queries in the query map
  • Security: No security concerns - these are read-only GraphQL queries
  • Performance: All new queries use appropriate field selections and follow GraphQL best practices
  • Testing: The query map file is valid JSON and all queries appear to be syntactically valid GraphQL
  • ID Field Convention: All new queries correctly use internalID for database IDs as per CLAUDE.md guidelines

Details

The PR adds 4 new queries:

  1. ArtistInsightsQuery (773bba4b6157a184f04ae274c5a72986): Fetches artist insights including auction results
  2. ArtistBelowTheFoldQuery (9c58e7deba3ec034d11d6e37e25dbe0a): Simple cacheable query for artist ID
  3. ArtistAboutQuery (cf5166cab7deba1eb1d4cd2f614484f3): Comprehensive artist information including related artists, shows, and articles
  4. ConversationDetails_Test_Query (a2ee275fb31c2903fa68a14b8ba0dcf3): Test query variant of ConversationDetailsQuery

The existing ConversationDetailsQuery was moved but its content remains unchanged.

All queries follow the repository's conventions for ID fields, using internalID for database identifiers and id for Relay global IDs.

This automated update from Eigen looks good to merge.

@mzikherman mzikherman force-pushed the update-eigen-query-map branch from 006c44a to 9045730 Compare February 12, 2026 21:26
@artsyit artsyit force-pushed the update-eigen-query-map branch 16 times, most recently from 3005a1d to 61ef50b Compare February 20, 2026 12:05
@artsyit artsyit force-pushed the update-eigen-query-map branch 17 times, most recently from b56ed95 to d1ccbb3 Compare March 3, 2026 18:06
@artsyit artsyit force-pushed the update-eigen-query-map branch 12 times, most recently from 351b52d to 38b6f38 Compare March 10, 2026 16:47
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