Merge branch 'release/23.0' into merge/release-22.9.1-into-release-23.0

Author: itsmeichigo

.buildkite/commands/build-for-testing.sh

@@ -18,5 +18,5 @@ echo "--- :hammer_and_wrench: Building"
 bundle exec fastlane build_for_testing
 
 echo "--- :arrow_up: Upload Build Products"
-tar -cf build-products.tar DerivedData/Build/Products/
+tar -cf build-products.tar DerivedData/Build/Products/ DerivedData/Index.noindex/DataStore/
 buildkite-agent artifact upload build-products.tar

.buildkite/commands/run-periphery.sh

@@ -0,0 +1,44 @@
+#!/bin/bash -eu
+
+if .buildkite/commands/should-skip-job.sh --job-type validation; then
+  exit 0
+fi
+
+### Prepare
+
+echo '--- 📦 Downloading Build Artifacts'
+buildkite-agent artifact download build-products.tar .
+tar -xf build-products.tar
+
+### Run the Tests
+
+PERIPHERY_OUTPUT_FILE="periphery-errors.txt"
+
+echo '+++ :zombie: Detecting unused code'
+set +e
+# This is the script in the root.
+./Scripts/Periphery/setup-and-run-periphery.sh \
+  --strict --quiet --skip-build --index-store-path 'DerivedData/Index.noindex/DataStore/' \
+  --write-results "$PERIPHERY_OUTPUT_FILE"
+TESTS_EXIT_STATUS=$?
+set -e
+
+# Handle the result of the periphery scan
+if [[ "$TESTS_EXIT_STATUS" -ne 0 ]]; then
+  echo '😱 Unused code detected!'
+  echo ''
+  echo '💡 You can run Periphery locally by calling `setup-and-run-periphery.sh` from `./Scripts/Periphery/` in the repo.'
+  echo ''
+  echo 'If you think there is a false positive violation, please check the known issues of Periphery at https://github.com/peripheryapp/periphery/issues.'
+  echo 'If you think a violation is valid but it should be surpressed for any reason, please apply the `// periphery: ignore - {your-reason-here}` comment.'
+  echo ''
+  # Add the periphery errors as a Buildkite annotation
+  (echo "### Periphery found unused code"; echo ''; echo '```'; cat "$PERIPHERY_OUTPUT_FILE"; echo ''; echo '```') \
+    | buildkite-agent annotate --context periphery --style error
+else
+  echo '😊 No unused code found.'
+  buildkite-agent annotate --context periphery --style success 'No unused code found by Periphery :tada:'
+fi
+
+echo '--- 🚦 Report Exit code'
+exit $TESTS_EXIT_STATUS

.buildkite/pipeline.yml

@@ -79,6 +79,14 @@ steps:
         agents:
           queue: linter
 
+      - label: "🧟 Detect unused code"
+        command: ".buildkite/commands/run-periphery.sh"
+        depends_on: build
+        plugins: [$CI_TOOLKIT]
+        notify:
+          - github_commit_status:
+              context: Periphery - Detect unused code
+
       - label: 🧹 Lint Translations
         command: gplint /workdir/WooCommerce/Resources/AppStoreStrings.pot
         plugins:

.gitignore

@@ -102,3 +102,6 @@ WooCommerce/WooCommerceScreenshots/Mocks/vendor/
 Modules/.build
 Modules/.swiftpm/configuration/registries.json
 Modules/.swiftpm/xcode/package.xcworkspace/contents.xcworkspacedata
+
+# Periphery install
+vendor/Periphery/

.periphery.yml

@@ -0,0 +1,68 @@
+# This option tells Periphery to not report any Swift code as unused if it's accessible from Objective-C. This includes any class that inherits from NSObject, and any method or property marked with the @objc or @objcMembers attribute.
+#  Woocommerce-ios, is a mixed Swift and Objective-C project. Swift code is often called from Objective-C and vice-versa. Without this flag, Periphery would incorrectly flag Swift code that is only used by Objective-C parts of the app as "unused," which would be a false positive. Setting this to true is essential for getting accurate results in a mixed-language codebase.
+retain_objc_accessible: true
+
+# This option prevents Periphery from flagging unused parameters in protocol method implementations.
+# Often, a protocol will define a method with parameters that are needed by some, but not all, conforming types. If a specific implementation doesn't need to use a parameter, Swift still requires that the method signature matches the protocol exactly. Periphery would normally see this unused parameter and flag it. However, removing it would cause a compile error because the method signature would no longer match the protocol. This flag tells Periphery to ignore these cases, avoiding false positives.
+retain_unused_protocol_func_params: true
+
+# This option retains all code within a previews struct that conforms to SwiftUI's PreviewProvider protocol.
+# SwiftUI Previews are a powerful feature for UI development in Xcode. The code you write for previews is often not directly called from your main application target; it's used by Xcode's canvas. Periphery would see this and incorrectly mark the preview code as unused. This flag correctly tells Periphery to always keep preview provider code, which is the desired behavior.
+retain_swift_ui_previews: true
+
+# This option disables the check for public declarations that are only used within their own module.
+# Normally, Periphery would suggest changing public to internal or private in these cases to reduce the public API surface of a module.
+disable_redundant_public_analysis: false
+
+# By default, Periphery will scan all targets in your project, including test targets. 
+# This can lead to false positives, as test code is not part of the main application. 
+# This option tells Periphery to exclude all test targets from the scan, ensuring that only the application code is analyzed.
+exclude_tests: true
+
+schemes:
+- WooCommerce
+
+project: WooCommerce.xcworkspace
+
+# Used for local run by `periphery scan`
+# This will not conflict with CI build configuration because of build skipping by `--skip-build` and `--index-store-path`
+build_arguments:
+- -xcconfig
+- config/WooCommerce.debug.xcconfig
+
+# A baseline is used to manage the large number of warnings from a mature codebase.
+# It contains a snapshot of all "existing/acceptable" warnings. Periphery will ignore all
+# warnings in this file, allowing you to focus on new issues in pull requests.
+#
+# After fixing existing warnings, regenerate this file with:
+# periphery scan --write-baseline Scripts/Periphery/periphery_baseline.json
+baseline: Scripts/Periphery/periphery_baseline.json
+
+exclude_targets:
+- GenerateCredentials
+- WooCommerceScreenshots
+- WordPressAuthenticator
+- WordPressAuthenticatorTests
+- WooCommerceTests
+- WooCommerceUITests
+- NotificationExtension
+- StoreWidgetsExtension
+- WatchWidgetsExtension
+- "Woo Watch App"
+- Fakes
+- XcodeTarget_WooCommerceTests
+- XcodeTarget_WooCommerceUITests
+- XcodeTarget_WordPressAuthenticatorTests
+
+index_exclude:
+- Pods/*
+- vendor/**
+- BuildTools/.build/**
+- "**/Tests/*"
+- "**/Test*/*"
+- "docs/*"
+- "fastlane/*"
+- "config/*"
+- "**/*.xib"
+- "**/*.storyboard"
+- "**/*.generated.swift"

CHANGELOG.md

@@ -1,6 +1,9 @@
 <!--
 Contains editorialized release notes. Raw release notes should go into `RELEASE-NOTES.txt`.
 -->
+## 23.0
+Faster performance and smoother experience! This update brings improved product loading, better tax calculations, and enhanced address lookup with map support. We've added guided barcode scanning for POS users and strengthened shipping requirements for EU destinations. Plus, we've fixed stability issues for a more reliable experience.
+
 ## 22.9
 This update enhances Shipping Labels with improved accessibility, smarter address validation, and faster performance. We’ve also made the dedicated Point of Sale tab more widely available for quicker access, and POS orders are now filterable in your order list for better organization. Plus, we've optimized assets to reduce the app’s size.
 

Modules/Package.resolved

@@ -72,6 +72,7 @@ let package = Package(
         .package(url: "https://github.com/Alamofire/Alamofire", from: "5.2.0"),
         .package(url: "https://github.com/AliSoftware/OHHTTPStubs", from: "9.0.0"),
         .package(url: "https://github.com/apple/swift-algorithms.git", from: "1.2.0"),
+        .package(url: "https://github.com/apple/swift-async-algorithms", exact: "1.0.4"),
         .package(url: "https://github.com/Automattic/AutomatticAbout-swift.git", from: "1.1.5"),
         .package(url: "https://github.com/Automattic/Automattic-Tracks-iOS.git", from: "3.5.2"),
         .package(url: "https://github.com/Automattic/Gridicons-iOS", revision: "c904cb73e26e86463a78e1335c6f4fd54a9e9223"),
@@ -381,6 +382,7 @@ enum XcodeSupport {
                     .product(name: "Inject", package: "Inject"),
                     .product(name: "KeychainAccess", package: "KeychainAccess"),
                     .product(name: "Kingfisher", package: "Kingfisher"),
+                    .product(name: "AsyncAlgorithms", package: "swift-async-algorithms"),
                     .product(name: "ScrollViewSectionKit", package: "ScrollViewSectionKit"),
                     .product(name: "Shimmer", package: "SwiftUI-Shimmer"),
                     .product(name: "StripeTerminal", package: "stripe-terminal-ios"),

Modules/Package.swift

@@ -102,7 +102,9 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
         case .pointOfSaleOrdersi2:
             return true
         case .pointOfSaleBarcodeScanningi2:
-            return buildConfig == .localDeveloper || buildConfig == .alpha
+            return true
+        case .orderAddressMapSearch:
+            return true
         default:
             return true
         }

Modules/Sources/Experiments/DefaultFeatureFlagService.swift

@@ -210,4 +210,8 @@ public enum FeatureFlag: Int {
     /// Enables the Point of Sale Barcode Scanner set up flows, as part of i2
     ///
     case pointOfSaleBarcodeScanningi2
+
+    /// Enables the CTA to search for an address in the map in order details > shipping address.
+    ///
+    case orderAddressMapSearch
 }