Repository navigation
chore(deps): bump the npm-minor-and-patch group across 1 directory with 7 updates - #344
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
| "clsx": "^2.1.1", | ||
| "docx-preview": "0.3.7", | ||
| "jszip": "^3.10.1", | ||
| "docx-preview": "0.4.1", |
There was a problem hiding this comment.
Preview changes lack browser coverage
docx-preview moves to a release that changes how it creates HTML, but this PR adds no browser test for the preview. prepareSandboxDoc warns that safe previews depend on the renderer moving newly created elements into the iframe in the same job. The existing smoke test only checks source text and cannot catch a change to that timing.
Add a browser test that renders a crafted document with the new version, checks that no handler runs in the parent page, and confirms the preview appears.
Prompt To Fix With AI
This is a comment left during a code review.
Path: src/agent_env/explorer/ui/package.json
Line: 23
Comment:
**Preview changes lack browser coverage**
`docx-preview` moves to a release that changes how it creates HTML, but this PR adds no browser test for the preview. `prepareSandboxDoc` warns that safe previews depend on the renderer moving newly created elements into the iframe in the same job. The existing smoke test only checks source text and cannot catch a change to that timing.
Add a browser test that renders a crafted document with the new version, checks that no handler runs in the parent page, and confirms the preview appears.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
…th 7 updates Bumps the npm-minor-and-patch group with 7 updates in the /src/agent_env/explorer/ui directory: | Package | From | To | | --- | --- | --- | | [@dagrejs/dagre](https://github.com/dagrejs/dagre) | `3.1.1` | `3.1.3` | | [@radix-ui/react-accordion](https://github.com/radix-ui/primitives/tree/HEAD/packages/react/accordion) | `1.2.20` | `1.3.1` | | [@xyflow/react](https://github.com/xyflow/xyflow/tree/HEAD/packages/react) | `12.11.3` | `12.12.0` | | [docx-preview](https://github.com/VolodymyrBaydalka/docxjs) | `0.3.7` | `0.4.1` | | [jszip](https://github.com/Stuk/jszip) | `3.10.1` | `3.10.2` | | [autoprefixer](https://github.com/postcss/autoprefixer) | `10.5.4` | `10.6.1` | | [postcss](https://github.com/postcss/postcss) | `8.5.28` | `8.5.31` | Updates `@dagrejs/dagre` from 3.1.1 to 3.1.3 - [Release notes](https://github.com/dagrejs/dagre/releases) - [Changelog](https://github.com/dagrejs/dagre/blob/master/changelog.md) - [Commits](dagrejs/dagre@v3.1.1...v3.1.3) Updates `@radix-ui/react-accordion` from 1.2.20 to 1.3.1 - [Changelog](https://github.com/radix-ui/primitives/blob/main/packages/react/accordion/CHANGELOG.md) - [Commits](https://github.com/radix-ui/primitives/commits/HEAD/packages/react/accordion) Updates `@xyflow/react` from 12.11.3 to 12.12.0 - [Release notes](https://github.com/xyflow/xyflow/releases) - [Changelog](https://github.com/xyflow/xyflow/blob/main/packages/react/CHANGELOG.md) - [Commits](https://github.com/xyflow/xyflow/commits/@xyflow/react@12.12.0/packages/react) Updates `docx-preview` from 0.3.7 to 0.4.1 - [Release notes](https://github.com/VolodymyrBaydalka/docxjs/releases) - [Commits](https://github.com/VolodymyrBaydalka/docxjs/commits) Updates `jszip` from 3.10.1 to 3.10.2 - [Changelog](https://github.com/Stuk/jszip/blob/main/CHANGES.md) - [Commits](Stuk/jszip@v3.10.1...v3.10.2) Updates `autoprefixer` from 10.5.4 to 10.6.1 - [Release notes](https://github.com/postcss/autoprefixer/releases) - [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md) - [Commits](postcss/autoprefixer@10.5.4...10.6.1) Updates `postcss` from 8.5.28 to 8.5.31 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.5.28...8.5.31) --- updated-dependencies: - dependency-name: "@dagrejs/dagre" dependency-version: 3.1.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: npm-minor-and-patch - dependency-name: "@radix-ui/react-accordion" dependency-version: 1.3.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: npm-minor-and-patch - dependency-name: "@xyflow/react" dependency-version: 12.12.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: npm-minor-and-patch - dependency-name: autoprefixer dependency-version: 10.6.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm-minor-and-patch - dependency-name: docx-preview dependency-version: 0.4.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: npm-minor-and-patch - dependency-name: jszip dependency-version: 3.10.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: npm-minor-and-patch - dependency-name: postcss dependency-version: 8.5.31 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor-and-patch ... Signed-off-by: dependabot[bot] <support@github.com>
4f6c935 to
0468565
Compare
Bumps the npm-minor-and-patch group with 7 updates in the /src/agent_env/explorer/ui directory:
3.1.13.1.31.2.201.3.112.11.312.12.00.3.70.4.13.10.13.10.210.5.410.6.18.5.288.5.31Updates
@dagrejs/dagrefrom 3.1.1 to 3.1.3Changelog
Sourced from @dagrejs/dagre's changelog.
... (truncated)
Commits
a73acbfBumping for releasea037580Merge pull request #527 from dagrejs/check-release-notes17757d9Adding a check on release notes to the release744d6e1Bump version and set as pre-releasea5ea31cBuilding for release89b75d9Merge pull request #526 from dagrejs/pre-commit09913acchore: add .pre-commit-config.yaml to run testsdaa9324Merge pull request #524 from dajiaohuang/fix/zero-width-edge-intersections-20...77c6330Merge branch 'master' into fix/zero-width-edge-intersections-20261015a4cfd26chore: revert dist changes and move tests to layout-test.tsUpdates
@radix-ui/react-accordionfrom 1.2.20 to 1.3.1Changelog
Sourced from @radix-ui/react-accordion's changelog.
Commits
Updates
@xyflow/reactfrom 12.11.3 to 12.12.0Release notes
Sourced from @xyflow/react's releases.
Changelog
Sourced from @xyflow/react's changelog.
... (truncated)
Commits
8716946chore(packages): bump9fd41fdchore(packages): bump19f9c2afix(react/store): reset defaultEdgeOptions2acefb5fix(react/store): reset functions #59918f0a5d6chore(packages): bump286816cchore(packages): bump3319ef4fix(react): keep minimap bounds at the viewport when all nodes are hiddenb7186f3fix(react): minimap still works after reactflow remounts #5971d8edc77Merge pull request #5955 from Jian-Zhang08/fix/minimap-onnodeclick-conditiona...59b1a60Simplify node click handling in MiniMap componentUpdates
docx-previewfrom 0.3.7 to 0.4.1Release notes
Sourced from docx-preview's releases.
Commits
Updates
jszipfrom 3.10.1 to 3.10.2Changelog
Sourced from jszip's changelog.
Commits
020fa33Release preparation for 3.10.2 (#974)d38fda5Fix cross-realm binary type detection in getTypeOf (#578)e682f9aFix grunt build rebundling existing dist via browser field (#971)5adef36Add missing types for JSZip.defaults (#927)41a1342Fix blob support in Node.js 18 and up (#955)643714aMove all continuous integration to GitHub Actions (#942)418c11fUpgrade Playwright to the latest version (#940)2ceb998Create FUNDING.ymlMaintainer changes
This version was pushed to npm by jkoops, a new releaser for jszip since your current version.
Updates
autoprefixerfrom 10.5.4 to 10.6.1Release notes
Sourced from autoprefixer's releases.
Changelog
Sourced from autoprefixer's changelog.
Commits
c99614dRelease 10.6.1 versionf1d08cbUpdate dependenciescf1ebcbFix grid gap inheritance from row-gap and column-gap longhands (#1557)e1212dcRelease 10.6.0 versiondac3c5fAdd support for prefixing initial-letter (#1556)b031690Release 10.5.6 versionda65c36Add warning when Autoprefixer can cover CSS for old Safari604f8d9Update dependencies54608caFix old WebKit gradients with color hints (#1554)7e81fffRelease 10.5.5 versionUpdates
postcssfrom 8.5.28 to 8.5.31Release notes
Sourced from postcss's releases.
Changelog
Sourced from postcss's changelog.
Commits
d190857Release 8.5.31 version4ef6d8cUpdate dependencies6abd674Call the builder with every node when the output is escaped (#2175)f1aca7eRelease 8.5.30 version9d83445Update dependencies6c9bc73Fix invalid source map mapping on escaping before new line (#2169)1fd8f88Add missed changes in changeloga7a1f34Release 8.5.29 versionb55a36dUpdate dependencies7647a81Escape </style> in output of non-Root nodes (#2163)

Confidence Score: 5/5
The PR appears safe to merge, with the earlier browser-test suggestion still outstanding.Fix with agent prompt
Summary
Updates seven dependencies in the explorer UI and refreshes their locked dependencies.
Reviews (2) · Last reviewed commit: "chore(deps): bump the npm-minor-and-patc..." · Reviewed by Greptile