Skip to content

[pull] main from microsoft:main#1292

Merged
pull[bot] merged 15 commits into
code:mainfrom
microsoft:main
May 24, 2026
Merged

[pull] main from microsoft:main#1292
pull[bot] merged 15 commits into
code:mainfrom
microsoft:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented May 24, 2026

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 : )

zhichli and others added 15 commits May 22, 2026 11:06
…xtraction

VS Code's ToolName enum uses snake_case identifiers (create_file,
replace_string_in_file, apply_patch, insert_edit_into_file,
multi_replace_string_in_file, edit_notebook_file, read_file). The
initial CLI-derived FILE_TOOL_NAMES set only contained the Copilot
CLI/Claude camelCase variants, so tool.parameters.edit_type and
tool.parameters.file_path were never emitted on real VS Code tool
spans. Add the missing names and extend classifyEditType to handle
them. Verified end-to-end in Aspire on a trace where create_file now
correctly emits edit_type=create with the file_path gated by
captureContent.
- Move USAGE_REASONING_OUTPUT_TOKENS from GitHubCopilotAttr to GenAiAttr
  (its value is gen_ai.usage.reasoning.output_tokens, not a github.copilot.* key)
- Accept Anthropic-style mcp__server__tool double-underscore format in
  addition to mcp_server_tool single-underscore
- Add a test for the double-underscore variant
Chat: OTel enrichment for agent context
@pull pull Bot locked and limited conversation to collaborators May 24, 2026
@pull pull Bot added the ⤵️ pull label May 24, 2026
@pull pull Bot merged commit e638ea5 into code:main May 24, 2026
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant