Conversation
…process-23-0C39vfltMe fix: semgrep-detect-child-process
- Replace `console.log(err)` with `console.error(err)` in catch blocks in domLineBreaksComputer.ts and sharedWebContentExtractorService.ts - Remove redundant `console.log` that duplicates `logService.info()` in remote.ts - Fix JSDoc typo "Returns of" -> "Returns if" in offsetRange.ts Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This api is very strange. Reducing where it's exposed because it really should not exist
This is how we should be registering slash commands. Not hard coding targets. It would be a good exercise to apply when clauses to the rest of these in the future.
…riptions (#303630) * refactor: rename image carousel configuration and update related descriptions also make default true * update title * fix: update image carousel title and description to "Images Preview"
* Fix missing global flag in sanitizeId regex The regex in sanitizeId was missing the 'g' flag, so only the first occurrence of '.' or '/' was replaced with '_'. Since settings IDs contain multiple dots (e.g. 'editor.font.size'), this meant subsequent dots were left in the sanitized ID. * Add regression test for sanitizeId global replacement Export sanitizeId and add a test verifying that all occurrences of '.' and '/' are replaced in generated tree element IDs, not just the first. --------- Co-authored-by: Shehab Sherif <shehabsherif0@users.noreply.github.com>
Make sure content part disposes of refs
Try to clean up inProgress handling for chat sessions
Remove unused `IChatSessionDto.id` field
Fix code quality issues: error logging and JSDoc typo
Replace child_process.exec with execFile to prevent potential command injection
These types are generally expected to be immutable
Use unknown instead of any
* Enable starting sessions on remote agent hosts in sessions app Co-authored-by: Copilot <copilot@github.com> * Fix Co-authored-by: Copilot <copilot@github.com> * fix Co-authored-by: Copilot <copilot@github.com> * fix Co-authored-by: Copilot <copilot@github.com> * fix test Co-authored-by: Copilot <copilot@github.com> --------- Co-authored-by: Copilot <copilot@github.com>
When chat.customizationsMenu.enabled is true (default): - Gear icon directly opens the AI Customizations editor - Tool Sets, Chat Settings, and Show Agent Debug Logs move to the '...' context menu When the setting is false, the original gear dropdown behavior is preserved.
…303637) * Handle unsandboxed terminal confirmation actions * test: fix unsandboxed terminal confirmation actions
Mark many chat session related types as readonly
* Don't localize markdown icon syntax Co-authored-by: Copilot <copilot@github.com> * Add eslint rule for localized markdown icons --------- Co-authored-by: Copilot <copilot@github.com>
* Handle unsandboxed terminal confirmation actions * test: fix unsandboxed terminal confirmation actions * Fixing confirmation window issues outside sandbox
* carousel: improve image loading perf * resolve comments
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.
11 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 : )