fix: reset deletes only sdk files instead of the whole folder

.github/workflows/build.yml

@@ -3,6 +3,7 @@ on:
   push:
     branches:
       - main
+      - 3.2.5_patch
   pull_request:
     paths-ignore:
       - "**/*.md"

.github/workflows/lint.yml

@@ -3,6 +3,7 @@ on:
   push:
     branches:
       - main
+      - 3.2.5_patch
   pull_request:
     paths-ignore:
       - "**/*.md"
@@ -13,4 +14,6 @@ jobs:
       - uses: actions/checkout@v4
 
       - name: Run lints
-        run: make lint
+        run: |
+          brew install swiftlint
+          make lint

.github/workflows/test.yml

@@ -3,6 +3,7 @@ on:
   push:
     branches:
       - main
+      - 3.2.5_patch
   pull_request:
     paths-ignore:
       - "**/*.md"

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## Next
 
+## 3.2.5 - 2024-05-14
+
+- fix: `reset` deletes only sdk files instead of the whole folder [#132](https://github.com/PostHog/posthog-ios/pull/132)
+
 ## 3.2.4 - 2024-03-12
 
 - `maxQueueSize` wasn't respected when capturing events [#116](https://github.com/PostHog/posthog-ios/pull/116)

PostHog.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name             = "PostHog"
-  s.version          = "3.2.4"
+  s.version          = "3.2.5"
   s.summary          = "The hassle-free way to add posthog to your iOS app."
 
   s.description      = <<-DESC

PostHog/PostHogStorage.swift

@@ -20,6 +20,7 @@ func applicationSupportDirectoryURL() -> URL {
 }
 
 class PostHogStorage {
+    // when adding or removing items here, make sure to update the reset method
     enum StorageKey: String {
         case distinctId = "posthog.distinctId"
         case anonymousId = "posthog.anonymousId"
@@ -125,8 +126,16 @@ class PostHogStorage {
     }
 
     public func reset() {
-        deleteSafely(appFolderUrl)
-        createDirectoryAtURLIfNeeded(url: appFolderUrl)
+        // sadly the StorageKey.allCases does not work here
+        deleteSafely(url(forKey: .distinctId))
+        deleteSafely(url(forKey: .anonymousId))
+        // .queue not needed since it'll be deleted by the queue.clear()
+        deleteSafely(url(forKey: .oldQeueue))
+        deleteSafely(url(forKey: .enabledFeatureFlags))
+        deleteSafely(url(forKey: .enabledFeatureFlagPayloads))
+        deleteSafely(url(forKey: .groups))
+        deleteSafely(url(forKey: .registerProperties))
+        deleteSafely(url(forKey: .optOut))
     }
 
     public func remove(key: StorageKey) {

PostHog/PostHogVersion.swift

@@ -9,7 +9,7 @@ import Foundation
 
 // if you change this, make sure to also change it in the podspec and check if the script scripts/bump-version.sh still works
 // This property is internal only
-public var postHogVersion = "3.2.4"
+public var postHogVersion = "3.2.5"
 
 // This property is internal only
 public var postHogSdkName = "posthog-ios"

PostHogExample/AppDelegate.swift

@@ -24,6 +24,7 @@ class AppDelegate: NSObject, UIApplicationDelegate {
         PostHogSDK.shared.setup(config)
 //        PostHogSDK.shared.debug()
 //        PostHogSDK.shared.capture("App started!")
+//        PostHogSDK.shared.reset()
 
         let defaultCenter = NotificationCenter.default
 

PostHogTests/PostHogSDKTest.swift

@@ -694,6 +694,18 @@ class PostHogSDKTest: QuickSpec {
             sut.reset()
             sut.close()
         }
+
+        it("reset deletes posthog files but not other folders") {
+            let appFolder = applicationSupportDirectoryURL()
+            expect(FileManager.default.fileExists(atPath: appFolder.path)) == false
+
+            let sut = self.getSut()
+
+            sut.reset()
+            sut.close()
+
+            expect(FileManager.default.fileExists(atPath: appFolder.path)) == true
+        }
     }
 }