Skip to content

chore: update dependencies#2623

Merged
matyasf merged 6 commits into
masterfrom
update_deps
Jul 7, 2026
Merged

chore: update dependencies#2623
matyasf merged 6 commits into
masterfrom
update_deps

Conversation

@matyasf

@matyasf matyasf commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Update most dependencies to their latest versions.

  • Bump vitest from ^3.2.2 to ^4.1.9 across all packages. Adjust vitest.setup.ts and tsconfig files for the new major versions.
  • Drop the deprecated @lerna/project dependency and read the project version
    directly by walking up to the nearest lerna.json.
  • Drop the deprecated @lerna/package dependency and implement it directly
  • Delete pkg-utils and move its code to ui-scripts/lib/utils/pkg-utils
  • Convert the root scripts/*.js to ESM (.mjs) so they can consume the ESM-only package, and update these

I suggest to look at the PR per commit, its divided into distinct parts

to test:

  • smoke test docs app

Update most dependencies to their latest versions.
@matyasf matyasf self-assigned this Jul 3, 2026
Comment on lines +30 to +31
// getOffsetParents returns only transformed or non-statically-positioned
// ancestors, plus the document body (always included as the last entry).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This change is here because Jest was updated

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-07 11:40 UTC

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Visual regression report

No changes.

Status Count
Unchanged 32
Changed 0
New 0
Removed 0

📊 View full report

Baselines come from the visual-baselines branch. They refresh on every merge to master.

github-actions Bot pushed a commit that referenced this pull request Jul 3, 2026
import { Project } from '@lerna/project'
const syncRootPackageVersion = async (useProjectVersion: boolean) => {
const project = new Project(process.cwd())
const syncRootPackageVersion = async () => {

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I've removed the deprecated lerna/project package and added code that has the same behaviour

matyasf and others added 3 commits July 3, 2026 20:15
Bump vitest from ^3.2.2 to ^4.1.9 across all packages and @types/jest from
^29.5.14 to ^30.0.0. Adjust vitest.setup.ts and tsconfig files for the new
major versions.
…on reader

Drop the deprecated @lerna/project dependency and read the project version
directly by walking up to the nearest lerna.json. Removes the associated
deprecated transitive deps from the lockfile.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…h local Package class

Drop the deprecated @lerna/package dependency and implement a minimal local
Package class in pkg-utils exposing name/location/version/serialize. Update
ui-scripts to use the version getter/setter instead of the old get/set API.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Comment thread scripts/bootstrap.mjs

mark('Deleting build artifacts')
execSync(path.resolve('scripts/clean.js'), opts)
execSync(path.resolve('scripts/clean.mjs'), opts)
"@babel/runtime": "^7.29.7",
"@babel/standalone": "^7.29.3",
"codesandbox": "^2.2.3",
"codesandbox-import-utils": "^2.2.3",

@matyasf matyasf Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This package has wayyy less transitive dependencies

import type { PackageJson, NormalizeOptions } from 'read-package-up'
import childProcess from 'child_process'

export class Package {

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is copied from Lerna's source code, they are no longer exporting this class

Comment on lines +150 to +159
settings: {
// The rule resolves import paths via eslint-module-utils. Without
// TypeScript extensions the default node resolver can't resolve .ts/.tsx
// files, resolve.default() returns undefined and the rule silently bails.
"import/resolver": {
node: {
extensions: [".js", ".jsx", ".ts", ".tsx", ".json"]
}
}
},

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

While doing some refactoring here to remove pkg-utils it turned out that this plugin was not working at all :/

I really dont like ESLint its slow and full of ways where it fails silently

…te deps

Finalize the merge of @instructure/pkg-utils into ui-scripts: move its package
utilities into lib/utils/pkg-utils/package-utils.ts, add getPackages() there,
and repoint all imports (bump, deprecate, publish, publish-private, tag, npm,
addNewExportsEntiresToPackageJSONs) at the local module. Drop the package from
the tsconfig references and the eslint node-package list.

Convert the root scripts/*.js to ESM (.mjs) so they can consume the ESM-only
chalk 5 and read-package-up, and update dependencies: chalk 5, which 7,
read-package-up 12, @vitejs/plugin-react, and codesandbox-import-utils.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
})
})

it('should close when Esc key is pressed', async () => {

@matyasf matyasf Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This test was randomly failing because its async and does not have an await.

This seems to be a systematic issue with Cypress tests, I'll fix them in a later PR

@git-nandor git-nandor requested a review from HerrTopi July 7, 2026 08:37
@matyasf matyasf merged commit bf18f72 into master Jul 7, 2026
10 of 12 checks passed
@matyasf matyasf deleted the update_deps branch July 7, 2026 11:39
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.

4 participants