-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Context
PR #234's /simplify review added Constants.MCPScope.user and updated the files changed by that PR. However, there are remaining usages:
MCP scope strings
ConfigurationDiscovery.swift:159—scope: "user"ConfigurationDiscovery.swift:170—scope: "local"Component.swift:131— doc comment references"local","project","user"as bare strings
Settings file name
ProjectSyncStrategy.swift:154— hardcoded"settings.local.json"in error message (should usescope.settingsPath.lastPathComponent)
Proposed Fix
- Use
Constants.MCPScope.user/Constants.MCPScope.localinConfigurationDiscovery.swift - Replace hardcoded settings file name with
scope.settingsPath.lastPathComponent - Consider adding
Constants.MCPScope.projectandConstants.MCPScope.localif not already covered
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request