Fix incorrect inspect property usage in scope detection (#301472)
* Fix incorrect inspect property usage in scope detection
In findAutoConfigurationTarget() (themeConfiguration.ts), change
settings.userRemote to settings.userRemoteValue. The former is an
IInspectValue<T> object that can be defined when only language
overrides exist, causing the function to incorrectly return
USER_REMOTE when no actual remote value is set.
In commandLineAutoApprover.ts, change configInspectValue.workspaceFolder
to configInspectValue.workspaceFolderValue. The former is an
IInspectValue<T> object whose keys are {value, override, overrides},
so checkTarget() never finds tool-pattern keys on it, making
workspace-folder-scoped auto-approve rules undetectable.
Both properties were introduced in bb18007f.
* fix: use plain command name in test config to match regex behavior
The test config 'git*' gets escaped to regex /^git\*\b/ which only matches
the literal string 'git*', not 'git status'. Changed to 'git' which produces
/^git\b/ and correctly matches 'git status' via word boundary.
---------
Co-authored-by: Shehab Sherif <shehabsherif0@users.noreply.github.com> S
Shehab Sherif committed
44a5af29376aa91b18985bd1772d9f34db19dbb3
Parent: adc6348
Committed by GitHub <noreply@github.com>
on 5/5/2026, 3:02:31 AM