Merge pull request #47 from nikolainobadi/fix-publish-coordinator

Fix publish coordinator

Author: nikolainobadi

CLAUDE.md

@@ -136,6 +136,14 @@ The tool supports multiple build configurations:
 - This is a known issue with SwiftData in test environments and doesn't affect functionality
 - Tests must be run using `xcodebuild` instead of `swift test` due to SwiftData compatibility requirements
 
+## Test Development Guidelines
+
+### Data Type Verification
+- **Never guess at implementation**: When encountering a data type you haven't read into memory, always look up the file first
+- **Read before using**: Use the Read tool to examine struct/class/enum definitions before using them in tests
+- **Verify initializers**: Check actual initializer signatures rather than assuming parameters
+- **Example**: Before creating a `BuildConfig`, read the file to see its actual properties and initializer
+
 ## Code Style Preferences
 
 ### Extension Organization

Sources/NnexKit/Shared/NnexError.swift Sources/NnexKit/Error/NnexError.swiftSources/NnexKit/Shared/NnexError.swift renamed to Sources/NnexKit/Error/NnexError.swift

@@ -32,4 +32,5 @@ public enum NnexError: Error {
 
     case missingExecutable
     case selectionRequired
+    case uncommittedChanges
 }

…xKit/Shared/HomebrewFormulaService.swift …Kit/Formula/HomebrewFormulaService.swiftSources/NnexKit/Shared/HomebrewFormulaService.swift renamed to Sources/NnexKit/Formula/HomebrewFormulaService.swift Sources/NnexKit/Shared/HomebrewFormulaService.swift renamed to Sources/NnexKit/Formula/HomebrewFormulaService.swift

@@ -29,6 +29,7 @@ public extension AutoVersionHandler {
         }
 
         let fileContent = try fileSystem.readFile(at: mainFile)
+        
         return extractVersionFromContent(fileContent)
     }
 

…Shared/HomebrewFormulaStoreAdapter.swift …ormula/HomebrewFormulaStoreAdapter.swiftSources/NnexKit/Shared/HomebrewFormulaStoreAdapter.swift renamed to Sources/NnexKit/Formula/HomebrewFormulaStoreAdapter.swift Sources/NnexKit/Shared/HomebrewFormulaStoreAdapter.swift renamed to Sources/NnexKit/Formula/HomebrewFormulaStoreAdapter.swift

@@ -7,7 +7,8 @@
 
 public enum FormulaContentGenerator {
     public static func makeFormulaFileContent(
-        name: String,
+        formulaName: String,
+        installName: String,
         details: String,
         homepage: String,
         license: String,
@@ -17,7 +18,7 @@ public enum FormulaContentGenerator {
     ) -> String {
         let sanitizedVersion = sanitizeVersion(version)
         return """
-        class \(FormulaNameSanitizer.sanitizeFormulaName(name)) < Formula
+        class \(FormulaNameSanitizer.sanitizeFormulaName(formulaName)) < Formula
             desc "\(details)"
             homepage "\(homepage)"
             url "\(assetURL)"
@@ -26,18 +27,19 @@ public enum FormulaContentGenerator {
             license "\(license)"
 
             def install
-                bin.install "\(name)"
+                bin.install "\(installName)"
             end
 
             test do
-                system "#{bin}/\(name)", "--help"
+                system "#{bin}/\(installName)", "--help"
             end
         end
         """
     }
 
     public static func makeFormulaFileContent(
-        name: String,
+        formulaName: String,
+        installName: String,
         details: String,
         homepage: String,
         license: String,
@@ -53,7 +55,7 @@ public enum FormulaContentGenerator {
         if hasArm && hasIntel {
             let sanitizedVersion = sanitizeVersion(version)
             return """
-            class \(FormulaNameSanitizer.sanitizeFormulaName(name)) < Formula
+            class \(FormulaNameSanitizer.sanitizeFormulaName(formulaName)) < Formula
                 desc "\(details)"
                 homepage "\(homepage)"
                 version "\(sanitizedVersion)"
@@ -72,17 +74,18 @@ public enum FormulaContentGenerator {
                 end
 
                 def install
-                    bin.install "\(name)"
+                    bin.install "\(installName)"
                 end
 
                 test do
-                    system "#{bin}/\(name)", "--help"
+                    system "#{bin}/\(installName)", "--help"
                 end
             end
             """
         } else if hasArm {
             return makeFormulaFileContent(
-                name: name,
+                formulaName: formulaName,
+                installName: installName,
                 details: details,
                 homepage: homepage,
                 license: license,
@@ -92,7 +95,8 @@ public enum FormulaContentGenerator {
             )
         } else if hasIntel {
             return makeFormulaFileContent(
-                name: name,
+                formulaName: formulaName,
+                installName: installName,
                 details: details,
                 homepage: homepage,
                 license: license,
@@ -102,7 +106,8 @@ public enum FormulaContentGenerator {
             )
         } else {
             return makeFormulaFileContent(
-                name: name,
+                formulaName: formulaName,
+                installName: installName,
                 details: details,
                 homepage: homepage,
                 license: license,
@@ -118,6 +123,6 @@ public enum FormulaContentGenerator {
 // MARK: - Private Methods
 private extension FormulaContentGenerator {
     static func sanitizeVersion(_ version: String) -> String {
-        version.hasPrefix("v") ? String(version.dropFirst()) : version
+        return version.hasPrefix("v") ? String(version.dropFirst()) : version
     }
 }