-
Notifications
You must be signed in to change notification settings - Fork 324
Parse #Playground macro expansions and include them in CodeLens request #2340
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
award999 marked this conversation as resolved.
Show resolved
Hide resolved
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,101 @@ | ||
| //===----------------------------------------------------------------------===// | ||
| // | ||
| // This source file is part of the Swift.org open source project | ||
| // | ||
| // Copyright (c) 2014 - 2025 Apple Inc. and the Swift project authors | ||
| // Licensed under Apache License v2.0 with Runtime Library Exception | ||
| // | ||
| // See https://swift.org/LICENSE.txt for license information | ||
| // See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors | ||
| // | ||
| //===----------------------------------------------------------------------===// | ||
|
|
||
| internal import BuildServerIntegration | ||
| import Foundation | ||
| @_spi(SourceKitLSP) import LanguageServerProtocol | ||
| import SKLogging | ||
| import SourceKitLSP | ||
| import SwiftParser | ||
| import SwiftSyntax | ||
|
|
||
| // MARK: - SwiftPlaygroundsScanner | ||
|
|
||
| final class SwiftPlaygroundsScanner: SyntaxVisitor { | ||
| /// The base ID to use to generate IDs for any playgrounds found in this file. | ||
| private let baseID: String | ||
|
|
||
| /// The snapshot of the document for which we are getting playgrounds. | ||
| private let snapshot: DocumentSnapshot | ||
|
|
||
| /// Accumulating the result in here. | ||
| private var result: [TextDocumentPlayground] = [] | ||
|
|
||
| /// Keep track of if "Playgrounds" has been imported | ||
| private var isPlaygroundImported: Bool = false | ||
|
|
||
| private init(baseID: String, snapshot: DocumentSnapshot) { | ||
| self.baseID = baseID | ||
| self.snapshot = snapshot | ||
| super.init(viewMode: .sourceAccurate) | ||
| } | ||
|
|
||
award999 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| /// Designated entry point for `SwiftPlaygroundsScanner`. | ||
| static func findDocumentPlaygrounds( | ||
| in node: some SyntaxProtocol, | ||
| workspace: Workspace, | ||
| snapshot: DocumentSnapshot | ||
| ) async -> [TextDocumentPlayground] { | ||
| guard let canonicalTarget = await workspace.buildServerManager.canonicalTarget(for: snapshot.uri), | ||
| let moduleName = await workspace.buildServerManager.moduleName(for: snapshot.uri, in: canonicalTarget), | ||
| let baseName = snapshot.uri.fileURL?.lastPathComponent | ||
| else { | ||
| return [] | ||
| } | ||
| let visitor = SwiftPlaygroundsScanner(baseID: "\(moduleName)/\(baseName)", snapshot: snapshot) | ||
| visitor.walk(node) | ||
| return visitor.isPlaygroundImported ? visitor.result : [] | ||
| } | ||
|
|
||
| /// Add a playground location with the given parameters to the `result` array. | ||
| private func record( | ||
| id: String, | ||
| label: String?, | ||
| range: Range<AbsolutePosition> | ||
| ) { | ||
| let positionRange = snapshot.absolutePositionRange(of: range) | ||
|
|
||
| result.append( | ||
| TextDocumentPlayground( | ||
| id: id, | ||
| label: label, | ||
| range: positionRange, | ||
| ) | ||
| ) | ||
| } | ||
|
|
||
| override func visit(_ node: ImportPathComponentSyntax) -> SyntaxVisitorContinueKind { | ||
| if node.name.text == "Playgrounds" { | ||
| isPlaygroundImported = true | ||
| } | ||
| return .skipChildren | ||
| } | ||
|
|
||
| override func visit(_ node: MacroExpansionExprSyntax) -> SyntaxVisitorContinueKind { | ||
award999 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| guard node.macroName.text == "Playground" else { | ||
| return .skipChildren | ||
| } | ||
|
|
||
| let startPosition = snapshot.position(of: node.positionAfterSkippingLeadingTrivia) | ||
| let stringLiteral = node.arguments.first?.expression.as(StringLiteralExprSyntax.self) | ||
| let playgroundLabel = stringLiteral?.representedLiteralValue | ||
| let playgroundID = "\(baseID):\(startPosition.line + 1):\(startPosition.utf16index + 1)" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we know if /* 🧑🧑🧒🧒 */ #Playground { print("blub") }There are functions on
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @chrismiles doing something like in the example above compiles, and if it was named i.e. then
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. My bet would be that print("/* 🧑🧑🧒🧒 */ ".utf8.count)
print("/* 🧑🧑🧒🧒 */ ".utf16.count) |
||
|
|
||
| record( | ||
| id: playgroundID, | ||
| label: playgroundLabel, | ||
| range: node.trimmedRange | ||
| ) | ||
|
|
||
| return .skipChildren | ||
| } | ||
| } | ||
Uh oh!
There was an error while loading. Please reload this page.