Skip to content

Upgrades; fix for current RS ana handling - #5

Merged
blms merged 6 commits into
mainfrom
blms/upgrade
Aug 13, 2026
Merged

Upgrades; fix for current RS ana handling#5
blms merged 6 commits into
mainfrom
blms/upgrade

Conversation

@blms

@blms blms commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

In this PR

NER annotations began appearing like this in the current RS frontend:

Screenshot 2026-08-11 at 11 41 44 AM

This is because of this change: recogito/recogito-client@55491cff

This PR:

  • updates the NER plugin to match the client's new functionality
  • upgrades Astro and the recogito SDK
  • bumps the plugin version number to 0.5.0

Questions

Does it make sense to shim the plugin like this, or would it be better to make a modification directly in @recogito/standoff-converter?

@rsimon

rsimon commented Aug 12, 2026

Copy link
Copy Markdown
Member

Ah - I remember. Yes, we decided to change the serialization format.

Does it make sense to shim the plugin like this, or would it be better to make a modification directly in @recogito/standoff-converter?

My sense: yes :-)

So this should definitely be fied in the standoff-converter crosswalk, since we're now clearly out of sync.

At the same time, I wonder if it makes sense to also keep the shim - at least temporary - as a precaution? Do people still have documents around in the old format? The hoisting as such doesn't break anything and could safely stay in there for a while. If we keep it, we should probably make a note in the code, e.g. just pointing to this discussion here?

@rsimon rsimon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @blms,

just a note to say I made two additions:

  • Upgraded the standoff-converter package dependency to the latest version I just published to the npm registry
  • Upgraded the uuid package to the latest version. (The old version caused security warnings; and the new version is compatible - tested in several of my projects.)

@blms

blms commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Looks great, thank you!

@blms
blms merged commit 098b2d6 into main Aug 13, 2026
@blms
blms deleted the blms/upgrade branch August 13, 2026 14:18
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