Skip to content

[depthfirst-3687] Upgrade multiple packages in ci/requirements.txt - #1099

Open
depthfirst-app[bot] wants to merge 1 commit into
mainfrom
patcherai/ci-requirements.txt-multiple-upgrades-3687
Open

[depthfirst-3687] Upgrade multiple packages in ci/requirements.txt#1099
depthfirst-app[bot] wants to merge 1 commit into
mainfrom
patcherai/ci-requirements.txt-multiple-upgrades-3687

Conversation

@depthfirst-app

Copy link
Copy Markdown

Overview

Upgrade

  • Upgraded markdown-it-py from 2.0.1 to 2.2.0 in ci/requirements.txt.
  • Upgraded Pygments from 2.8.1 to 2.20.0 in ci/requirements.txt.

Changes

  • Renamed two generated MCP Plugin code-block artifact files:
    • plugins/moveworks-mcp-plugin/.codeblocks/block_3pythonblock_3.tsx
    • plugins/moveworks-mcp-plugin/.codeblocks/block_4pythonblock_4.tsx
  • Root cause: Pygments language guessing behavior changed between 2.8.1 and 2.20.0; the existing unlabeled TypeScript/TSX snippets are now detected as TSX instead of Python. The generated artifact contents are unchanged; only the filenames changed to match the new lexer detection result.

Backward Compatibility

  • markdown-it-py==2.2.0 supports Python >=3.7.
  • Pygments==2.20.0 supports Python >=3.9, so environments running these CI tools on Python 3.8 or older must upgrade Python before installing the new requirements.

Vulnerabilities Fixed

Warnings

  • Full discovered validation is currently blocked by pre-existing repository issues unrelated to this dependency upgrade:
    • python -m ci.validate_v3 requires yaml/PyYAML, which is not listed in ci/requirements.txt; with PyYAML installed separately, it fails on an existing missing connectors/moveworks-data-api/logo.png file.
    • python ci/copy_code_blocks.py still fails on baseline stale .codeblocks state outside the Pygments-induced MCP filename changes fixed in this PR.

Verified

  • ✅ Resolved/downloaded markdown-it-py==2.2.0.
  • ✅ Resolved/downloaded Pygments==2.20.0.
  • ✅ Installed ci/requirements.txt successfully with the upgraded pins.
  • ✅ Ran python -m compileall ci successfully.
  • ✅ Ran configured no-op setup/compile/test commands successfully.
  • ⚠️ Ran discovered validation commands and confirmed remaining failures are pre-existing/baseline issues unrelated to the upgraded packages.

Sign up for freeto 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.

0 participants