chore: remove deprecation comment from getToolsDir, add changelog entry for #470

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))

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");
 }

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) {