[JPrevost]

Return nil instead of raising an error when a record has no identifiers, and add a regression test.

This branch should return record with null isbns, production will return error for this query:

{
  recordId(id: "alma:9935045653306761") {
    title
    isbns
  }
}

https://mitlibraries.atlassian.net/browse/TIMX-746 Sentry: TIMDEX-6Q

Developer

• No changes to view

Code Reviewer

• The commit message is clear and follows our guidelines
  (not just this pull request message)
• There are appropriate tests covering any new functionality
• The documentation has been updated or is unnecessary
• The changes have been verified
• New dependencies are appropriate or there were no changes

Requires database migrations?

NO

Includes new or updated dependencies?

NO

[Copilot]

🟢 Approval recommended

The change safely handles missing identifier data and adds regression coverage for the absent-key case, with no active review issues identified.

No open issues 🎉

What changed in this PR

Updates deprecated identifier field resolvers so records without an identifiers key return null instead of raising an error, with regression coverage for the missing-key case.

Changes:

• Safely handles missing identifier data in deprecated identifier accessors.
• Adds a regression test verifying isbns returns nil when identifiers are absent.

File	Description
test/​graphql/​types/​record_type_test.rb	Adds regression coverage verifying that isbns returns nil when the record lacks an identifiers key.
app/​graphql/​types/​record_type.rb	Makes deprecated identifier field resolution safe when identifier data is missing.

_{Review effort: Lite}