Skip to content

docs: preserve jsonpath exact static-eval dependency during setup - #103

Closed
MonaDevAI wants to merge 1 commit into
mainfrom
docs/setup-jsonpath-exact-dependency
Closed

MonaDevAI wants to merge 1 commit into
mainfrom
docs/setup-jsonpath-exact-dependency

Conversation

@MonaDevAI

Copy link
Copy Markdown
Contributor

Problem

Setup should preserve jsonpath's exact transitive dependency.

Change

Add a conditional setup note: repositories using jsonpath@1.3.0 must preserve static-eval@2.1.1 and review conflicting resolutions/overrides. No package manifests or runtime code changed.

Validation

  • Four documentation tests, docs check, formatting and diff check pass
  • Packaging/full validation not repeated for this documentation-only change
  • Documentation updated
  • No credentials or private data included

Risk

Version requirement follows the supplied request; independent registry verification was blocked by TLS failures. CI remains required before merge.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 7ecb4751-5093-4d69-919a-7bb8c124de11
Copilot AI balanced review requested due to automatic review settings November 4, 2026 18:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues block the documentation-only dependency-preservation guidance.

No open issues 🎉

What changed in this PR

This documentation-only PR adds dependency-preservation guidance to FixLab’s repository setup instructions. No package manifests or runtime code change.

Changes:

  • Instructs repositories using jsonpath@1.3.0 to preserve static-eval@2.1.1.
  • Requires reviewing conflicting resolutions or overrides before restoring dependencies, rather than silently substituting versions.
File Description
docs/​FIXLAB-SETUP.md Adds a conditional dependency-version note to the restore guidance.

Review effort: Balanced

@MonaDevAI

Copy link
Copy Markdown
Contributor Author

Closing: this dependency change was intended for FWWD, not the FixLab setup documentation.

@MonaDevAI MonaDevAI closed this Nov 4, 2026
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