Skip to content

Conversation

@plemarquand
Copy link
Contributor

@plemarquand plemarquand commented Dec 1, 2025

The disableSwiftPMIntegration setting prevented Package.swift files from activating the extension, but if the extension is activated via compile_commands or bsp integration we still attempt package operations like swift package describe when setting up a FolderContext.

If SwiftPM integration has been disabled, return an empty package representation with no targets or dependencies so that the extension still activates quickly and provides language features from sourcekit-lsp without attempting to use SwiftPM.

Issue: #1925

Tasks

  • Required tests have been written
  • Documentation has been updated
  • Added an entry to CHANGELOG.md if applicable

@plemarquand plemarquand force-pushed the disable-swiftpm-deeper-integration branch 2 times, most recently from ec10151 to 0597942 Compare December 1, 2025 16:48
@plemarquand plemarquand marked this pull request as ready for review December 3, 2025 02:25
The `disableSwiftPMIntegration` setting prevented `Package.swift` files
from activating the extension, but if the extension is activated via
compile_commands or bsp integration we still attempt package operations
like `swift package describe` when setting up a `FolderContext`.

If SwiftPM integration has been disabled, return an empty package
representation with no targets or dependencies so that the extension
still activates quickly and provides language features from
sourcekit-lsp without attempting to use SwiftPM.

Issue: swiftlang#1925
@plemarquand plemarquand force-pushed the disable-swiftpm-deeper-integration branch from 0597942 to 1801b2d Compare December 3, 2025 02:27
@plemarquand plemarquand merged commit 3bfae1a into swiftlang:main Dec 4, 2025
25 checks passed
@plemarquand plemarquand deleted the disable-swiftpm-deeper-integration branch December 4, 2025 00:37
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