Fix PR #147 review comments

- Comment out .unsafeFlags in Package.swift to restore SPM dependency compatibility
- Update README SPM version reference from 0.0.3 to 0.0.4
- Fix devcontainer.json extension indentation (3-space → 4-space)
- Add trailing newlines to .periphery.yml and .vscode/launch.json
- Remove .vscode from .gitignore
- Fix multiline_arguments_brackets violations in SyntaxDocTests/Settings.swift

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: leogdion

.devcontainer/swift-6.1/devcontainer.json

@@ -28,7 +28,7 @@
             },
             // Add the IDs of extensions you want installed when the container is created.
             "extensions": [
-               "swiftlang.swift-vscode"
+                "swiftlang.swift-vscode"
             ]
         }
     },

.gitignore

@@ -152,7 +152,6 @@ node_modules/
 .env
 # Editor directories and files
 .idea
-.vscode
 *.suo
 *.ntvs*
 *.njsproj

.periphery.yml

@@ -1,2 +1,2 @@
 retain_public: true
-retain_codable_properties: true
+retain_codable_properties: true

.vscode/launch.json

@@ -21,4 +21,4 @@
             "configuration": "release"
         }
     ]
-}
+}

Package.swift

@@ -60,20 +60,20 @@ let swiftSettings: [SwiftSetting] = [
   .enableExperimentalFeature("WarnUnsafeReflection"),
 
   // Enhanced compiler checking
-  .unsafeFlags([
-    // Enable concurrency warnings
-    "-warn-concurrency",
-    // Enable actor data race checks
-    "-enable-actor-data-race-checks",
-    // Complete strict concurrency checking
-    "-strict-concurrency=complete",
-    // Enable testing support
-    "-enable-testing",
-    // Warn about functions with >100 lines
-    "-Xfrontend", "-warn-long-function-bodies=100",
-    // Warn about slow type checking expressions
-    "-Xfrontend", "-warn-long-expression-type-checking=100"
-  ])
+  // .unsafeFlags([
+  //   // Enable concurrency warnings
+  //   "-warn-concurrency",
+  //   // Enable actor data race checks
+  //   "-enable-actor-data-race-checks",
+  //   // Complete strict concurrency checking
+  //   "-strict-concurrency=complete",
+  //   // Enable testing support
+  //   "-enable-testing",
+  //   // Warn about functions with >100 lines
+  //   "-Xfrontend", "-warn-long-function-bodies=100",
+  //   // Warn about slow type checking expressions
+  //   "-Xfrontend", "-warn-long-expression-type-checking=100"
+  // ])
 ]
 
 // swiftlint:disable:next explicit_top_level_acl explicit_acl

README.md

@@ -73,7 +73,7 @@ Add SyntaxKit to your project using Swift Package Manager:
 ```swift
 // Package.swift
 dependencies: [
-    .package(url: "https://github.com/brightdigit/SyntaxKit.git", from: "0.0.3")
+    .package(url: "https://github.com/brightdigit/SyntaxKit.git", from: "0.0.4")
 ]
 ```
 

Tests/SyntaxDocTests/Settings.swift

@@ -22,8 +22,8 @@ internal enum Settings {
     // a flat sibling of the test binary. Strategy 1a always runs first, so a macOS project
     // root containing Sources/ will never reach this check.
     if FileManager.default.fileExists(
-      atPath: workingDir.appendingPathComponent("Documentation.docc").path)
-    {
+      atPath: workingDir.appendingPathComponent("Documentation.docc").path
+    ) {
       return workingDir
     }
 
@@ -33,8 +33,8 @@ internal enum Settings {
       .deletingLastPathComponent()  // Tests
       .deletingLastPathComponent()  // Project root
     if FileManager.default.fileExists(
-      atPath: sourceRelative.appendingPathComponent("Sources").path)
-    {
+      atPath: sourceRelative.appendingPathComponent("Sources").path
+    ) {
       return sourceRelative
     }