fix: display the leash active notification in status bar for pi - #2
fix: display the leash active notification in status bar for pi#2bge-kernel-panic wants to merge 1 commit into
Conversation
Extensions such as ponytail interfere with the leash message and it's super important to view the leash status icon to make sure it's actually started before using the agent.
There was a problem hiding this comment.
Pull request overview
This PR updates the Pi extension to make Leash’s “active” signal visible in the status bar (to avoid interference from other extensions that may hide notifications), and increments the package version.
Changes:
- Set a Pi status-bar entry on
session_startindicating “🔒 Leash active”. - Bump
@sailthru/leashversion from1.0.17to1.0.18. - Add an
allowScriptssection inpackage.json.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/pi/leash.ts | Adds ctx.ui.setStatus(...) on session start to show Leash as active in the status bar. |
| package.json | Version bump and introduces allowScripts configuration. |
| dist/pi/leash.js | Updates compiled Pi bundle to include the new status-bar call. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "allowScripts": { | ||
| "esbuild@0.27.7": true, | ||
| "msgpackr-extract@3.0.4": true, | ||
| "sharp@0.34.5": true | ||
| } |
There was a problem hiding this comment.
it's needed because those are build scripts and otherwise the build fails to 100% run. Since the version is locked and I checked those packages for known vulnerabilities (note I looked for any mention online, did not inspect the code), I beleive this is 100% safe.
|
@bpediredla-mg Small QoL for Pi agent users (i.e. me right now ;) ) |
Extensions such as ponytail interfere with the leash message and it's super important to view the leash status icon to make sure it's actually started before using the agent.