Fix new module verification CI and TODOs in pom.xml (#6257)

Author: zoewangg

.github/workflows/new-module-verification.yml

@@ -129,23 +129,23 @@ jobs:
               fi
               
               # 2. Check if added to tests-coverage-reporting pom.xml
-              if ! grep -q "<module>.*$MODULE_NAME</module>" test/tests-coverage-reporting/pom.xml 2>/dev/null; then
+              if ! grep -q "<dependency>.*$MODULE_NAME</dependency>" test/tests-coverage-reporting/pom.xml 2>/dev/null; then
                 echo "::error::Module $MODULE_NAME is not added to tests-coverage-reporting pom.xml"
                 HAS_ERRORS=1
               else
                 echo "✅ Module is added to tests-coverage-reporting"
               fi
               
               # 3. Check if added to aws-sdk-java pom.xml
-              if ! grep -q "<module>.*$MODULE_NAME</module>" aws-sdk-java/pom.xml 2>/dev/null; then
+              if ! grep -q "<dependency>.*$MODULE_NAME</dependency>" aws-sdk-java/pom.xml 2>/dev/null; then
                 echo "::error::Module $MODULE_NAME is not added to aws-sdk-java pom.xml"
                 HAS_ERRORS=1
               else
                 echo "✅ Module is added to aws-sdk-java pom.xml"
               fi
               
               # 4. Check if added to architecture-tests pom.xml
-              if ! grep -q "<module>.*$MODULE_NAME</module>" test/architecture-tests/pom.xml 2>/dev/null; then
+              if ! grep -q "<dependency>.*$MODULE_NAME</dependency>" test/architecture-tests/pom.xml 2>/dev/null; then
                 echo "::error::Module $MODULE_NAME is not added to architecture-tests pom.xml"
                 HAS_ERRORS=1
               else

pom.xml

@@ -684,10 +684,6 @@
                             <exclude>*.internal.*</exclude>
                             <exclude>software.amazon.awssdk.thirdparty.*</exclude>
                             <exclude>software.amazon.awssdk.regions.*</exclude>
-                            <!-- TODO revert - Temporarily disable because system setting USER_REGION was removed -->
-                            <exclude>software.amazon.awssdk.utils.JavaSystemSetting</exclude>
-                            <!-- TODO remove once S3ObjectSize is change is merged -->
-                            <exclude>software.amazon.awssdk.services.s3.model.CompleteMultipartUploadRequest</exclude>
                         </excludes>
 
                         <ignoreMissingOldVersion>true</ignoreMissingOldVersion>