Add SwiftLint

Author: ivan-magda

.swiftlint.yml

@@ -0,0 +1,39 @@
+opt_in_rules:
+  # Performance
+  - first_where
+  - last_where
+  - sorted_first_last
+  - contains_over_filter_count
+  - empty_count
+
+  # Modern Swift style
+  - shorthand_optional_binding
+  - implicit_return
+  - toggle_bool
+
+  # Safety
+  - fatal_error_message
+
+  # Code cleanliness
+  - modifier_order
+  - redundant_type_annotation
+  - convenience_type
+
+  # Formatting
+  - indentation_width
+
+excluded:
+  - .build
+  - .swiftpm
+
+indentation_width:
+  indentation_width: 2
+
+identifier_name:
+  min_length:
+    warning: 2
+  excluded:
+    - i
+    - id
+    - x
+    - y

Package.swift

@@ -8,7 +8,7 @@ let package = Package(
   platforms: [.macOS(.v10_15)],
   products: [
     .executable(name: "claude", targets: ["cli"]),
-    .library(name: "Core", targets: ["Core"]),
+    .library(name: "Core", targets: ["Core"])
   ],
   dependencies: [
     .package(url: "https://github.com/swift-server/async-http-client.git", from: "1.32.0")
@@ -30,6 +30,6 @@ let package = Package(
       name: "CoreTests",
       dependencies: ["Core"],
       path: "Tests/CoreTests"
-    ),
+    )
   ]
 )

Sources/Core/Agent.swift

@@ -103,10 +103,10 @@ extension Agent {
       "properties": .object([
         "command": .object([
           "type": "string",
-          "description": "The shell command to execute",
+          "description": "The shell command to execute"
         ])
       ]),
-      "required": .array(["command"]),
+      "required": .array(["command"])
     ])
   )
 

Sources/Core/ShellExecutor.swift

@@ -26,7 +26,7 @@ public struct ShellResult: Sendable {
 
 public struct ShellExecutor: Sendable {
   private static let dangerousPatterns = [
-    "rm -rf /", "sudo", "shutdown", "reboot", "> /dev/",
+    "rm -rf /", "sudo", "shutdown", "reboot", "> /dev/"
   ]
 
   public let workingDirectory: String

Sources/cli/SwiftClaudeCode.swift

@@ -2,7 +2,7 @@ import Core
 import Foundation
 
 @main
-struct SwiftClaudeCode {
+enum SwiftClaudeCode {
   static func main() async throws {
     print("\(ANSIColor.bold)swift-claude-code\(ANSIColor.reset) v\(Agent.version)")
     print("\(ANSIColor.dim)A Claude Code-like agent built from scratch in Swift\(ANSIColor.reset)")

Tests/CoreTests/APIModelsTests.swift

@@ -16,10 +16,10 @@ struct ContentBlockTests {
   @Test func decodesToolUse() throws {
     let json = """
       {
-          "type": "tool_use",
-          "id": "toolu_123",
-          "name": "bash",
-          "input": {"command": "ls"}
+        "type": "tool_use",
+        "id": "toolu_123",
+        "name": "bash",
+        "input": {"command": "ls"}
       }
       """
     let block = try JSONDecoder().decode(ContentBlock.self, from: Data(json.utf8))
@@ -36,10 +36,10 @@ struct ContentBlockTests {
   @Test func decodesToolResult() throws {
     let json = """
       {
-          "type": "tool_result",
-          "tool_use_id": "toolu_789",
-          "content": "output text",
-          "is_error": true
+        "type": "tool_result",
+        "tool_use_id": "toolu_789",
+        "content": "output text",
+        "is_error": true
       }
       """
     let block = try JSONDecoder().decode(ContentBlock.self, from: Data(json.utf8))
@@ -64,15 +64,15 @@ struct ContentBlockTests {
     let blocks: [ContentBlock] = [
       .text("Hello"),
       .toolUse(id: "t1", name: "bash", input: .object(["command": "ls"])),
-      .text("World"),
+      .text("World")
     ]
     #expect(blocks.textContent == "Hello\nWorld")
   }
 
   @Test func textContentEmptyWhenNoTextBlocks() {
     let blocks: [ContentBlock] = [
       .toolUse(id: "t1", name: "bash", input: .object(["command": "ls"])),
-      .toolResult(toolUseId: "t1", content: "output", isError: false),
+      .toolResult(toolUseId: "t1", content: "output", isError: false)
     ]
     #expect(blocks.textContent == "")
   }
@@ -93,7 +93,7 @@ struct MessageTests {
       role: .assistant,
       content: [
         .text("Let me run that."),
-        .toolUse(id: "toolu_abc", name: "bash", input: .object(["command": "pwd"])),
+        .toolUse(id: "toolu_abc", name: "bash", input: .object(["command": "pwd"]))
       ]
     )
 
@@ -113,7 +113,7 @@ struct StopReasonTests {
     arguments: [
       (#""end_turn""#, StopReason.endTurn),
       (#""tool_use""#, StopReason.toolUse),
-      (#""max_tokens""#, StopReason.maxTokens),
+      (#""max_tokens""#, StopReason.maxTokens)
     ]
   )
   func decodes(json: String, expected: StopReason) throws {
@@ -141,7 +141,7 @@ struct APIRequestTests {
             "properties": .object([
               "command": .object(["type": "string"])
             ]),
-            "required": .array(["command"]),
+            "required": .array(["command"])
           ])
         )
       ]
@@ -162,17 +162,17 @@ struct APIResponseTests {
   @Test func decodes() throws {
     let json = """
       {
-          "id": "msg_abc123",
-          "type": "message",
-          "role": "assistant",
-          "content": [
-              {"type": "text", "text": "Here are the files."}
-          ],
-          "stop_reason": "end_turn",
-          "usage": {
-              "input_tokens": 100,
-              "output_tokens": 50
-          }
+        "id": "msg_abc123",
+        "type": "message",
+        "role": "assistant",
+        "content": [
+          {"type": "text", "text": "Here are the files."}
+        ],
+        "stop_reason": "end_turn",
+        "usage": {
+          "input_tokens": 100,
+          "output_tokens": 50
+        }
       }
       """
     let response = try JSONDecoder().decode(APIResponse.self, from: Data(json.utf8))
@@ -190,11 +190,11 @@ struct APIErrorTests {
   @Test func decodesErrorResponse() throws {
     let json = """
       {
-          "type": "error",
-          "error": {
-              "type": "invalid_request_error",
-              "message": "max_tokens must be positive"
-          }
+        "type": "error",
+        "error": {
+          "type": "invalid_request_error",
+          "message": "max_tokens must be positive"
+        }
       }
       """
     let response = try JSONDecoder().decode(APIErrorResponse.self, from: Data(json.utf8))

Tests/CoreTests/AgentTests.swift

@@ -104,12 +104,12 @@ struct AgentLoopTests {
       makeResponse(
         content: [
           .text("Let me check."),
-          .toolUse(id: "t1", name: "bash", input: .object(["command": "echo hi"])),
+          .toolUse(id: "t1", name: "bash", input: .object(["command": "echo hi"]))
         ],
         stopReason: .toolUse
       ),
       // Second response: final answer after seeing tool result
-      makeResponse(content: [.text("done")]),
+      makeResponse(content: [.text("done")])
     ]
     let (agent, _) = makeAgent(mock: mock)
 

Tests/CoreTests/JSONValueTests.swift

@@ -25,7 +25,7 @@ struct JSONValueTests {
       "verbose": .bool(true),
       "count": .int(3),
       "ratio": .double(1.5),
-      "nothing": .null,
+      "nothing": .null
     ])
     let data = try JSONEncoder().encode(original)
     let decoded = try JSONDecoder().decode(JSONValue.self, from: data)