Skip to content

add optional id to plugins#113

Merged
jdesrosiers merged 1 commit intohyperjump-io:mainfrom
srivastava-diya:main
Apr 24, 2026
Merged

add optional id to plugins#113
jdesrosiers merged 1 commit intohyperjump-io:mainfrom
srivastava-diya:main

Conversation

@srivastava-diya
Copy link
Copy Markdown
Contributor

Description

This PR makes EvaluationPlugin instances JSON-serializable by introducing an optional id identifier.

Solves : #110

Changes Made

  1. Types & Base Plugins:
    • Updated the EvaluationPlugin interface in lib/experimental.d.ts to include an optional id?: string property.
    • Added id properties to the BasicOutputPlugin, DetailedOutputPlugin, and AnnotationsPlugin class declarations.
    • Updated README.md to reflect the new id field in the EvaluationPlugin documentation.
  2. Keyword-Specific Plugins:
    • Added an id property to the plugins defined in the dynamicRef, unevaluatedProperties, and unevaluatedItems keyword handlers.
    • Added an id property to the draft-2020-12/dynamicRef keyword handler.

@jdesrosiers jdesrosiers merged commit 50dd7df into hyperjump-io:main Apr 24, 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.

2 participants