Skip to content

Conversation

@madmxg
Copy link
Contributor

@madmxg madmxg commented Nov 7, 2025

Description

When I started migrating my project to the new LTS version, I encountered an error when executing this codemod.

Validation

> npx codemod run @nodejs/fs-truncate-to-ftruncate
[1/2] 🔍 Resolving package from registry: https://app.codemod.com ...
Error: Registry error: Package not found: @nodejs/fs-truncate-to-ftruncate

Related Issues

No

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

When I started migrating my project to the new LTS version, I encountered an error when executing this code mod.

```bash
> npx codemod run @nodejs/fs-truncate-to-ftruncate
[1/2] 🔍 Resolving package from registry: https://app.codemod.com ...
Error: Registry error: Package not found: @nodejs/fs-truncate-to-ftruncate
```

Signed-off-by: Mikhail Garbuzov <madmxg@gmail.com>
Copilot AI review requested due to automatic review settings November 7, 2025 15:27
@madmxg madmxg requested a review from a team as a code owner November 7, 2025 15:27
@vercel
Copy link

vercel bot commented Nov 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Nov 7, 2025 4:25pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates a codemod command reference in the migration guide from v22 to v24, changing the codemod identifier from @nodejs/fs-truncate-to-ftruncate to @nodejs/fs-truncate-fd-deprecation to align with the GitHub repository directory name.

  • Updated the npx codemod command to use @nodejs/fs-truncate-fd-deprecation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@madmxg madmxg changed the title Fix codemod command for fs-truncate-fd-deprecation chore(Docs): Update codemod command for fs-truncate-fd-deprecation Nov 7, 2025
@madmxg madmxg changed the title chore(Docs): Update codemod command for fs-truncate-fd-deprecation chore(docs): Update codemod command for fs-truncate-fd-deprecation Nov 7, 2025
Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

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

LGMT !

fix link to codemod location

Co-authored-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Signed-off-by: Mikhail Garbuzov <madmxg@gmail.com>
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.72%. Comparing base (15921e1) to head (6a07c2e).
⚠️ Report is 13 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8338   +/-   ##
=======================================
  Coverage   76.71%   76.72%           
=======================================
  Files         118      118           
  Lines        9805     9822   +17     
  Branches      335      336    +1     
=======================================
+ Hits         7522     7536   +14     
- Misses       2281     2284    +3     
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@avivkeller avivkeller added this pull request to the merge queue Nov 12, 2025
Merged via the queue into nodejs:main with commit c49bc15 Nov 12, 2025
12 checks passed
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