[pull] main from microsoft:main#1298
Merged
Merged
Conversation
* Update @vscode/codicons to version 0.0.46-13 in package.json and package-lock.json * Add 'developer-tools' icon to codicons library --------- Co-authored-by: mrleemurray <mrleemurray@users.noreply.github.com>
… provider (#317678) * feat: add languageDiagnosticsService option for nes/inline completion provider * test: add test cases
…igh/low eagerness levels (#315043) * Add Xtab275AggrTagHL prompting strategy Like Xtab275Aggressiveness but only emits the <|aggressive|> tag for high/low levels. When the level is medium, uses the plain xtab275 prompt without any aggressiveness tag. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Rename Xtab275AggrTagHL to Xtab275AggressivenessHighLow Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Refactor: extract getXtab275BasePostScript helper Eliminates 3x duplication of the xtab275 base postscript text across Xtab275, Xtab275Aggressiveness, and Xtab275AggressivenessHighLow cases. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add unit tests for Xtab275AggressivenessHighLow strategy Tests cover: - Medium level omits the aggressive tag - High level includes aggressive tag with 'high' - Low level includes aggressive tag with 'low' - High level still includes the xtab275 base postscript - Contrast test: Xtab275Aggressiveness includes tag for medium Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: trigger CI re-run --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…318148) Two related issues caused even tiny notebook cell outputs (e.g. 5 bytes) to be replaced with "Output is too large to be used as context". 1. Inverted size check in `RunNotebookCellOutput`: `getCharLimit` converts tokens to chars (×4), but it was applied to the byteLength side and compared against a token count. Compare byteLength against getCharLimit(tokenBudget / sizeLimitRatio) instead — the threshold was 16× too small. 2. Remove `ReadCellOutput` from `toolsCalledInParallel`. Tools in that set are invoked eagerly with a sentinel `{ tokenBudget: 1 }` sizing on the premise that they don't consume sizing info. `RunNotebookCellOutput` does — it gates output on `sizing.tokenBudget` — so the sentinel made every non-empty output trip the size check. Letting it use the normal lazy path gives it the endpoint's real prompt budget.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )