smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(coding-agent/tools): suppressed repeated bash fixup notices per session

Browse Source 
- Tracked emission state so the bash fixup notice is shown at most once per BashTool instance.
This commit is contained in:
can1357
2026-05-15 04:52:05 +02:00
parent 4c494aaa3a
commit 995ba8e51e
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/src/tools/bash.ts
+6 -2
View File
@@ -246,6 +246,7 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
readonly #asyncEnabled: boolean;
readonly #autoBackgroundEnabled: boolean;
readonly #autoBackgroundThresholdMs: number;
#bashFixupNoticeEmitted = false;
constructor(private readonly session: ToolSession) {
this.#asyncEnabled = this.session.settings.get("async.enabled");
@@ -574,8 +575,11 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
const pendingNotices: string[] = [];
const timeoutClampNotice = formatTimeoutClampNotice(requestedTimeoutSec, timeoutSec);
if (timeoutClampNotice) pendingNotices.push(timeoutClampNotice);
const bashFixupNotice = formatBashFixupNotice(bashFixups);
if (bashFixupNotice) pendingNotices.push(bashFixupNotice);
const bashFixupNotice = this.#bashFixupNoticeEmitted ? undefined : formatBashFixupNotice(bashFixups);
if (bashFixupNotice) {
pendingNotices.push(bashFixupNotice);
this.#bashFixupNoticeEmitted = true;
}
if (asyncRequested) {
if (!AsyncJobManager.instance()) {