fix registered date bug

[bklaing2]

Purpose

closes: datacite/datacite#2512

Approach

Open Questions and Pre-Merge TODOs

Learning

Types of changes

• Bug fix (non-breaking change which fixes an issue)

• New feature (non-breaking change which adds functionality)

• Breaking change (fix or feature that would cause existing functionality to change)

Reviewer, please remember our guidelines:

• Be humble in the language and feedback you give, ask don't tell.
• Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
• Offer suggestions on how to improve code e.g. simplification or expanding clarity.
• Ensure you give reasons for the changes you are proposing.

[Copilot]

Copilot review overview

🟡 Changes recommended

The recursion guard permanently suppresses later URL synchronization on reused DOI instances.

Review effort: Balanced
Findings: 1

Open (1)

• Recursion guard permanently suppresses later URL synchronization · New

What changed in this PR

Fixes DOI registration dates being indexed before persistence.

Changes:

• Updates handle URLs before queuing DOI indexing.
• Reloads DOI data before Elasticsearch indexing.
• Adds regression coverage for registration ordering.

File	Description
app/​models/​doi.rb	Adds URL-update recursion guard.
app/​models/​concerns/​indexable.rb	Updates URL before enqueueing indexing.
app/​jobs/​index_job_doi_registration.rb	Reloads DOI before indexing.
spec/​concerns/​indexable_spec.rb	Tests registration ordering.
spec/​jobs/​index_job_doi_registration_spec.rb	Tests fresh registration data indexing.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.