Repository navigation
fix(update): show update notices to the person only - #56
Conversation
The update notice no longer enters the model's context. Codex hooks print systemMessage instead of additionalContext, the Claude Code mod shows only its toast, and Pi / Oh My Pi show a UI notification at session start. The notice text is now plain information for the person. Refs #44
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Thanks for the quick fix, and for taking the report seriously. I also owe you an apology for #44. I'd seen an agent update LCU on its own in a Claude Code context that had never used LCU, and for a moment I took it as something LCU's components were deliberately doing. I filed the issue in the heat of that moment. After I calmed down and looked into it, that wasn't what was happening, and I should have checked before opening the issue. Sorry for the noise and for the wrong framing. |
It was definitely a wrong choice to get it into the context of an agent that didn't use lcu in the first place, and the fact that your agent updated it without notice is as issue as well, and makes sense considering that bad pattern. So it's normal that it was confusing to you. So, still, thanks for reporting it, that's definitely useful, better file it in the heat of that moment than never! You don't owe me an apology and I don't think that's noise. Really appreciate you taking the time to explain, thanks! |
Refs #44. Thanks @Suzukaa-a for flagging that the update notice read as prompt injection.
The update notice no longer enters the model's context in any adapter:
lcu/update.mjs): plain information for the person ("LCU X is available (installed: Y). Runlcu update, then restart the agents that use LCU."), no instructions to an agent.lcu update --notice --hookprints a top-levelsystemMessage(a warning shown to the user) instead ofhookSpecificOutput.additionalContext. The hook definitions are unchanged, so their trust hash is too.lcu-approve0.6.0): toast only; the hook result is returned unchanged.session_starthandler callsctx.ui.notifyin interactive sessions (3 s cap, errors ignored).INSTALLATION.mdandADAPTERS.mdupdate-notice sections. Hermes and DeepSeek Harness keeplcu status/doctor/update --check.The once-a-day account-wide cooldown and
LCU_NO_UPDATE_CHECK=1are unchanged.Tests
node --test tests/node/*.test.mjs: 446 pass, 0 failadaptersnpm test: 212 pass, 8 skipped, 0 fail (includes new Pi notice test)claude plugin teston the mod: 49/49tests/run.sh, e2eNot verified live
The Codex
systemMessagewarning and the Pi/OMP notification have not been observed in a live session. Headlessclaude -pruns still consume the daily announcement without anyone seeing it.

Confidence Score: 5/5
The PR appears safe to merge; no actionable defects were found.Summary
Moves update notices out of model context and onto user-facing surfaces.
Known limitation acknowledged by
0xpolarzero: headlessclaude -pruns still consume the daily announcement without showing it. The PR also states that Codex warnings and Pi/Oh My Pi notifications have not been observed live.Diagram
%%{init: {'theme': 'neutral'}}%% flowchart LR Cache[Cached update notice] --> Cooldown[Shared announcement cooldown] Cooldown --> Codex[Codex systemMessage] Cooldown --> Claude[Claude toast] Cooldown --> Pi[Pi or Oh My Pi notification] Codex --> Person[Person using LCU] Claude --> Person Pi --> PersonReviews (1) · Last reviewed commit: "fix(update): show update notices to the ..." · Reviewed by Greptile