github / github/copilot-cli

Extension onPostToolUse modifiedResult is not applied to the model's conversation context

Aperta
#3,361 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub
area:plugins
Lingua principale
Shell
Stelle
11.2k
Fork
1.9k
Merge medio
14h 16m
PR unite (30g)
6

Descrizione

## Summary

When a Copilot CLI extension returns `modifiedResult` from `onPostToolUse`, the modified text appears in the TUI tool-result preview line but is **not** applied to the model's conversation context. The model sees and responds with the original unmodified tool result.

## Reproduction

Minimal single-file extension (`.github/extensions/repro/extension.mjs`):

```js
import { joinSession } from "@github/copilot-sdk/extension";

joinSession({
tools: [
{
name: "say_hello",
description: "Returns a greeting message",
parameters: { type: "object", properties: {} },
async handler() {
return "ORIGINAL: The tool returned this text.";
},
},
],
hooks: {
async onPostToolUse(input) {
if (input.toolName !== "say_hello") return {};
return {
modifiedResult: {
textResultForLlm: "MODIFIED: The hook replaced the result with this text.",
resultType: "success",
},
};
},
},
});
```

### Steps

1. Place the above in a git repo under `.github/extensions/repro/extension.mjs`
2. Run `copilot --allow-all` from the repo root
3. Ask: "call say_hello and tell me exactly what it returned"

### Expected

The model reports: `MODIFIED: The hook replaced the result with this text.`

### Actual

The TUI tool-result preview correctly shows:
```
● say_hello
└ MODIFIED: The hook replaced the result with this text.
```

But the model responds with: `ORIGINAL: The tool returned this text.`

## Observations

- The hook **does** fire — stderr logging in the hook confirms execution.
- The TUI preview line shows the **modified** text.
- The model responds with the **original** text.
- This suggests the hook's `modifiedResult` is applied to the display layer but not to the model's message history.

## Impact

Any extension relying on `onPostToolUse` + `modifiedResult` for result transformation (sanitization, redaction, enrichment) is silently broken. The hook appears to work from the TUI, but the model operates on original data.

## Environment

- Copilot CLI 1.0.49-1
- Windows 11

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia con la riproduzione in .github/extensions/repro/extension.mjs e segui la gestione di modifiedResult da parte di onPostToolUse, dall’hook passando per l’anteprima del risultato dello strumento fino al contesto della conversazione del modello. Verifica dove il risultato originale entra nella cronologia dei messaggi. Il lavoro è completato quando il modello riporta il testo modificato mentre l’anteprima della TUI rimane corretta.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
cli
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.