From 22870ae0c2f3286dd70f06c6c9bbf48c237a52b0 Mon Sep 17 00:00:00 2001 From: Mario Zechner Date: Mon, 5 Jan 2026 20:26:48 +0100 Subject: [PATCH] chore: remove deprecation comment from getToolsDir, add changelog entry for #470 --- packages/coding-agent/CHANGELOG.md | 1 + packages/coding-agent/src/config.ts | 2 +- packages/coding-agent/src/migrations.ts | 6 +----- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/packages/coding-agent/CHANGELOG.md b/packages/coding-agent/CHANGELOG.md index 6353c4e9..34583998 100644 --- a/packages/coding-agent/CHANGELOG.md +++ b/packages/coding-agent/CHANGELOG.md @@ -14,6 +14,7 @@ ### Fixed +- Managed binaries (`fd`, `rg`) now stored in `~/.pi/agent/bin/` instead of `tools/`, eliminating false deprecation warnings ([#470](https://github.com/badlogic/pi-mono/pull/470) by [@mcinteerj](https://github.com/mcinteerj)) - Extensions defined in `settings.json` were not loaded ([#463](https://github.com/badlogic/pi-mono/pull/463) by [@melihmucuk](https://github.com/melihmucuk)) - OAuth refresh no longer logs users out when multiple pi instances are running ([#466](https://github.com/badlogic/pi-mono/pull/466) by [@Cursivez](https://github.com/Cursivez)) - Migration warnings now ignore `fd.exe` and `rg.exe` in `tools/` on Windows ([#458](https://github.com/badlogic/pi-mono/pull/458) by [@carlosgtrz](https://github.com/carlosgtrz)) diff --git a/packages/coding-agent/src/config.ts b/packages/coding-agent/src/config.ts index 0dd06a2b..b865350e 100644 --- a/packages/coding-agent/src/config.ts +++ b/packages/coding-agent/src/config.ts @@ -142,7 +142,7 @@ export function getSettingsPath(): string { return join(getAgentDir(), "settings.json"); } -/** Get path to tools directory (deprecated, use extensions/ for JS/TS tools) */ +/** Get path to tools directory */ export function getToolsDir(): string { return join(getAgentDir(), "tools"); } diff --git a/packages/coding-agent/src/migrations.ts b/packages/coding-agent/src/migrations.ts index 3febdd09..2721ed8d 100644 --- a/packages/coding-agent/src/migrations.ts +++ b/packages/coding-agent/src/migrations.ts @@ -216,11 +216,7 @@ function checkDeprecatedExtensionDirs(baseDir: string, label: string): string[] const customTools = entries.filter((e) => { const lower = e.toLowerCase(); return ( - lower !== "fd" && - lower !== "rg" && - lower !== "fd.exe" && - lower !== "rg.exe" && - !e.startsWith(".") // Ignore .DS_Store and other hidden files + lower !== "fd" && lower !== "rg" && lower !== "fd.exe" && lower !== "rg.exe" && !e.startsWith(".") // Ignore .DS_Store and other hidden files ); }); if (customTools.length > 0) {