npm: v1.2.0+ not published — v1.1.1 has broken tool execute signatures

[pbaille]

Hi! The fix in commit 7a383b2 ("fix(extensions): update tool execute signatures") from Feb 2 corrected the swapped signal/onUpdate arguments in uv.ts (and other extensions), but the latest version on npm is still 1.1.1 which has the broken code.

This causes pi to crash with:

• signal.addEventListener is not a function
• TypeError: onUpdate is not a function at bash.js:155

whenever the bash tool is invoked, because the registered tool's execute receives (id, params, signal, onUpdate, ctx) from pi but the v1.1.1 code destructures it as (id, params, onUpdate, _ctx, signal) — swapping the callback and the abort signal.

The fix already exists in the repo (tag v1.2.0 and beyond). Could you publish a new version to npm?

Thanks!

[christophsturm]

would be great to get this fixed because the stacktrace that pi outputs in that case does not even point to any extension. (pi should probably be fixed to handle this more gracefully, but its also on vacation)

[goofansu]

Tags can be installed like the following. It installs the package in ~/.pi/agent/git/ directory, and you can verify by pi list.

pi install git:github.com/mitsuhiko/agent-stuff@v1.2.0

[christophsturm]

perfect! is there also a recommended way to install from a local git repo?

[goofansu]

perfect! is there also a recommended way to install from a local git repo?

Yeah. Just pi install --help and you can see the examples.