feat: add id prop handling to default doc builder #67
+1
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
✨ Proposal
This PR adds support for the
id
property in thedefault_doc_builder
. Although optional, theid
is a highly relevant field for many use cases, including unique document identification, tracking, and integration with external systems.✅ Expected Behavior
id
is retrieved, it will be used by thedefault_doc_builder
.id
is absent at the time of index creation, the behavior remains unchanged — the system will continue to function exactly as it did before.🛠️ Motivation
Supporting a pre-filled
id
reduces the need for custom logic elsewhere in the application and enables more consistent document handling when a specific identifier is required.🔄 Compatibility
This is a non-breaking change. The
id
remains optional, ensuring full backward compatibility with existing implementations.