Skip to content

Conversation

@akhuoa
Copy link
Collaborator

@akhuoa akhuoa commented Dec 9, 2025

Summary

This PR introduces a strict version check to prevent incompatibility issues with @abi-software/flatmapvuer. Currently, Yarn resolutions only output a warning during installation if versions do not match. These warnings are easily missed.

Changes

  • Added "@abi-software/flatmapvuer": "1.12.1" in resolutions.
  • Added a postinstall script (verify-resolutions.js) that converts resolution warnings into fatal errors.
  • Scope: While there are other resolution warnings in our project, this script is currently configured to only check @abi-software/flatmapvuer.
  • Extensibility: Additional packages can be easily added to the checklist by updating the PACKAGES_TO_CHECK variable in the script.

Note

If there is an updated version of @abi-software/flatmapvuer, we have to update its version in resolutions.

Related issue: https://github.com/nih-sparc/sparc-app-issues/issues/500

@akhuoa akhuoa requested a review from alan-wu December 9, 2025 21:59
@cypress
Copy link

cypress bot commented Dec 9, 2025

SPARC Vue3 Portal Testing    Run #839

Run Properties:  status check failed Failed #839  •  git commit fe2d0dedd2: Merge pull request #450 from akhuoa/refactor/unify-flatmapvuer
Project SPARC Vue3 Portal Testing
Branch Review main
Run status status check failed Failed #839
Run duration 53m 52s
Commit git commit fe2d0dedd2: Merge pull request #450 from akhuoa/refactor/unify-flatmapvuer
Committer Alan Wu
View all properties for this run ↗︎

Test results
Tests that failed  Failures 3
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 40
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 166
View all changes introduced in this branch ↗︎

Tests for review

Failed  datasets/imagegallery.cy.js • 2 failed tests

View Output

Test Artifacts
Dataset 385 > Viewer > Image Test Replay Screenshots
Dataset 226 > Viewer > Image Test Replay Screenshots
Failed  homepage.cy.js • 1 failed test

View Output

Test Artifacts
Homepage > Find by Test Replay Screenshots

Copy link
Contributor

@alan-wu alan-wu left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@alan-wu alan-wu merged commit fe2d0de into nih-sparc:main Dec 18, 2025
4 of 6 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