Skip to content

chore(pi-fff): migrate from @mariozechner to @earendil-works packages#459

Merged
dmtrKovalenko merged 2 commits into
dmtrKovalenko:mainfrom
SamuelLHuber:update-pi-fff-deps
May 8, 2026
Merged

chore(pi-fff): migrate from @mariozechner to @earendil-works packages#459
dmtrKovalenko merged 2 commits into
dmtrKovalenko:mainfrom
SamuelLHuber:update-pi-fff-deps

Conversation

@SamuelLHuber
Copy link
Copy Markdown
Collaborator

Update deprecated package names in peerDependencies and imports:

  • @mariozechner/pi-coding-agent → @earendil-works/pi-coding-agent
  • @mariozechner/pi-tui → @earendil-works/pi-tui

Copilot AI review requested due to automatic review settings May 8, 2026 09:44
Copy link
Copy Markdown
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

Migrates the pi-fff extension off deprecated @mariozechner/* package names to the new @earendil-works/* equivalents, aligning imports and peer dependency declarations with the renamed upstream packages.

Changes:

  • Updated pi-fff TypeScript imports to reference @earendil-works/pi-coding-agent and @earendil-works/pi-tui.
  • Updated pi-fff peerDependencies to the new package scope.
  • Bumped the Neovim help doc “Last change” date.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/pi-fff/src/index.ts Switches imports to @earendil-works/* packages.
packages/pi-fff/package.json Switches peer dependency package names to @earendil-works/*.
doc/fff.nvim.txt Updates documentation “Last change” header date.

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

Comment thread packages/pi-fff/package.json
Update deprecated package names in peerDependencies and imports:
- @mariozechner/pi-coding-agent → @earendil-works/pi-coding-agent
- @mariozechner/pi-tui → @earendil-works/pi-tui
@SamuelLHuber SamuelLHuber force-pushed the update-pi-fff-deps branch from 610e920 to b506da4 Compare May 8, 2026 09:48
@SamuelLHuber
Copy link
Copy Markdown
Collaborator Author

@dmtrKovalenko let me know if you're fine with me merging these on my own or I shall ping you on these.

Pi has changed their repo from https://github.com/badlogic/pi-mono/ to https://github.com/earendil-works/pi/ and the npm packages too as Mario joins Earendil Works. so I've updated the dependencies.

@dmtrKovalenko
Copy link
Copy Markdown
Owner

I think it's always better to get a review just in case. I would love my changes to be reviewed by someone there is just no one here who will do that for rust code 😅

@dmtrKovalenko dmtrKovalenko merged commit 8cb1f47 into dmtrKovalenko:main May 8, 2026
@SamuelLHuber
Copy link
Copy Markdown
Collaborator Author

perfekt, then I intuitively did it right.

yeah I am not proficient in rust. there is agents, but they are like a catch low hanging fruits and ping pong rather then a proper review unless one spends the time to set their environment up to level up imo (for these kinds of libraries like fff, it's different for basic business app level code)

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.

3 participants