Skip to content

fix: Add %appdata% resolution support for debugger config#391

Merged
JakeShirley merged 2 commits into
Mojang:mainfrom
ThomasOrs:Thomas/ResolveAppData
Jun 2, 2026
Merged

fix: Add %appdata% resolution support for debugger config#391
JakeShirley merged 2 commits into
Mojang:mainfrom
ThomasOrs:Thomas/ResolveAppData

Conversation

@ThomasOrs
Copy link
Copy Markdown
Contributor

  • Adds a resolution for %appdata% in launch.json to the appdata/roaming path which GDK lives under.
  • Updates readme to point to the GDK development_behavior_packs folder instead of UWP

Comment thread src/session.ts
Comment on lines 426 to 430
//if the value is a string and starts with %localappdata%, replace it with the actual path to AppData\Local
const value = args[key as keyof IAttachRequestArguments];
if (typeof value === 'string' && value.toLowerCase().startsWith('%localappdata%')) {
(args as any)[key] = path.join(localAppDataDir, value.substring('%localappdata%'.length));
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the %localappdata% remain for legacy UWP support?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably doesn't hurt

@ThomasOrs ThomasOrs changed the title feat: Adds %appdata% resolution support for debugger config feat: Add %appdata% resolution support for debugger config Jun 2, 2026
@JakeShirley JakeShirley merged commit b5c3502 into Mojang:main Jun 2, 2026
3 checks passed
@JakeShirley JakeShirley changed the title feat: Add %appdata% resolution support for debugger config fix: Add %appdata% resolution support for debugger config Jun 2, 2026
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.

2 participants