Repository navigation
build(deps): upgrade every dependency to its latest release - #139
Conversation
…rows A newer vitest cannot call an arrow function as a constructor and types a class mock's implementation as either a constructable or a this-typed function returning nothing. Mocks of classes now use function implementations, and a small helper copies a test double's members onto the constructed instance, which replaces the casts that pretended a double was a full instance. The helper lives outside the published build.
TypeScript 7 ships no importable compiler API, so typescript-eslint, typedoc and knip cannot load it. typescript is aliased to Microsoft's typescript6 compatibility package for them and the native compiler is installed beside it, so tsc builds and type-checks with the native binary.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Vitest now sets an environment variable whose config value is undefined to the string undefined, and turbo's strict environment mode withholds the CI matrix variables, so on Linux the helper read the case sensitivity as the string undefined and reported a case-insensitive filesystem. The config forwards only variables that are set, and the test task declares them to turbo so the per-OS settings reach the tests and take part in the cache key.
|
@codex security review |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
🎉 This PR is included in version 1.41.4 🎉 The release is available on:
Installationnpm install -g markmv@1.41.4Security & Supply Chain✅ OIDC Trusted Publishing: published from CI with no long-lived npm token Test CoverageThis release includes comprehensive test coverage reports. View coverage details in the release assets. Your semantic-release bot 📦🚀 |
1 participant
Every dependency is at its latest release and pnpm outdated reports nothing. The majors that needed work: vitest 5 cannot call an arrow function as a constructor, so class mocks use function implementations and a small helper (src/test-support, excluded from the published build) copies a test double onto the constructed instance in place of the casts that were there. TypeScript 7 ships no importable compiler API: ESLint could not even load eslint.config.ts under it. typescript is therefore aliased to Microsoft's @typescript/typescript6 compatibility package for typescript-eslint, typedoc and knip, and the native compiler is installed as @typescript/native so tsc builds and type-checks with the 7.0 binary; this is the setup documented in the TypeScript 7 announcement. Removing the alias once typescript-eslint supports 7.1 is tracked in #135. The vitest 5 upgrade also fixes the coverage run, which crashed on main inside a transitive glob dependency after the audit's brace-expansion override.