Skip to content

Slai Agent Runtime:Fix export output paths containing Unicode characters - #6582

Merged
JohnMcPMS merged 1 commit into
microsoft:masterfrom
huong8373tt-beep:fix/export-emoji-output-path
Oct 8, 2026
Merged

JohnMcPMS merged 1 commit into
microsoft:masterfrom
huong8373tt-beep:fix/export-emoji-output-path

Conversation

@huong8373tt-beep

@huong8373tt-beep Slai.life (huong8373tt-beep) commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Convert the UTF-8 --output argument to UTF-16 before constructing the export output path.
  • Add regression coverage for exporting to an existing directory whose name contains an emoji.

Fixes #5813.

Validation

  • Reproduced with WinGet 1.29.380 on Windows 11 build 26200: winget export --source winget --output <existing Unicode/emoji directory>\packages.json --disable-interactivity failed with 0x80070003, while the same command succeeded for an ASCII-only sibling directory.
  • Built AppInstallerCLITests (Release|x64) locally.
  • Ran ExportFlow_ExportAll_UnicodeOutputPath: passed (3 assertions).
  • Ran ExportFlow_ExportAll*: passed (41 assertions in 4 test cases).

AI assistance disclosure

This issue was discovered while Slai Agent Runtime was operating in a Windows environment. The developer authorized this contribution after the issue was reproduced and the fix was validated. Slai Agent Runtime assisted with reproduction, implementation, and test execution; the account owner is responsible for this submission.

Microsoft Reviewers: Open in CodeFlow

@huong8373tt-beep
Slai.life (huong8373tt-beep) requested a review from a team as a code owner October 7, 2026 17:44
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@huong8373tt-beep Slai.life (huong8373tt-beep) changed the title Fix export output paths containing Unicode characters Slai Agent Runtime:Fix export output paths containing Unicode characters Oct 7, 2026
@JohnMcPMS

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@JohnMcPMS
JohnMcPMS merged commit abd5744 into microsoft:master Oct 8, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

winget export not writing file if path contains emoji

2 participants