Skip to content

Comments

deps: upgrade jsonpathly to 3.0.0#782

Merged
paulRbr merged 1 commit intobump-sh:mainfrom
jamietanna:fix/jsonpath
Jan 30, 2026
Merged

deps: upgrade jsonpathly to 3.0.0#782
paulRbr merged 1 commit intobump-sh:mainfrom
jamietanna:fix/jsonpath

Conversation

@jamietanna
Copy link
Contributor

@jamietanna jamietanna commented Jan 29, 2026

As part of 0, it was noted that the upstream library we use for JSON
Path expressions isn't fully RFC 9535 compliant.

With help from Claude Sonnet 4.5 to help diagnose a change in the public
API for jsonpathly, where a single quoted path is returned.

Also:

  • chore: update lockfile after npm install

@jamietanna

This comment was marked as outdated.

@jamietanna jamietanna marked this pull request as ready for review January 29, 2026 17:19
@paulRbr paulRbr changed the title fix(deps): upgrade jsonpathly to 3.0.0 deps: upgrade jsonpathly to 3.0.0 Jan 29, 2026
@paulRbr paulRbr enabled auto-merge January 29, 2026 19:25
@paulRbr paulRbr self-assigned this Jan 29, 2026
@paulRbr paulRbr self-requested a review January 29, 2026 19:25
@paulRbr paulRbr assigned paulRbr and unassigned paulRbr Jan 29, 2026
@paulRbr
Copy link
Member

paulRbr commented Jan 29, 2026

Thanks for the PR @jamietanna. I've adapted the change you made in the core/overlay lib and it looks good to go!

🚀

As part of [0], it was noted that the upstream library we use for JSON
Path expressions isn't fully RFC 9535 compliant.

Jammie took help from Claude Sonnet 4.5 to help diagnose a change in
the public API for jsonpathly, where a single quoted path is returned.

Paulr rewrote the code for better readability (and replace only first
and last single quotes instead of all quotes)

Co-authored-by: Paul B. <paulr@bump.sh>

[0]: atamano/jsonpathly#12
@paulRbr paulRbr merged commit 478d949 into bump-sh:main Jan 30, 2026
9 checks passed
@jamietanna
Copy link
Contributor Author

jamietanna commented Jan 30, 2026

Thanks very much 💜 do you know when this would be able to get released?

Looks like it's out in https://github.com/bump-sh/cli/releases/tag/v2.9.11 🚀 Thanks!

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