From 5cfc1f256167281f7cebb02d6d141fca886ac614 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 21:22:52 +0100
Subject: [PATCH 01/12] Introduce ChangeDetection

Instead to jump between ways how to perform "change detection"
(decide should existing target file be overwritten), make it
a configurable strategy.

Before 3.4.0 it was timestamp, post 3.4.0 it was content, but
in any case users were left short.

Fixes
https://github.com/apache/maven-resources-plugin/issues/453
---
 pom.xml                                       |  5 +-
 .../shared/filtering/ChangeDetection.java     | 43 +++++++++
 .../filtering/DefaultMavenFileFilter.java     | 75 ++++++++++++----
 .../DefaultMavenResourcesFiltering.java       |  4 +-
 .../shared/filtering/FilteringUtils.java      | 87 ++++++++++++++++++-
 .../shared/filtering/MavenFileFilter.java     | 51 ++++++++++-
 .../filtering/MavenResourcesExecution.java    | 35 ++++++--
 7 files changed, 269 insertions(+), 31 deletions(-)
 create mode 100644 src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java

diff --git a/pom.xml b/pom.xml
index 5d1e6d72..21b86f92 100644
--- a/pom.xml
+++ b/pom.xml
@@ -63,7 +63,6 @@
   <properties>
     <mavenVersion>3.9.12</mavenVersion>
     <slf4jVersion>1.7.36</slf4jVersion>
-    <plexusBuildApiVersion>0.0.7</plexusBuildApiVersion>
     <version.plexus-utils>3.6.0</version.plexus-utils>
     <project.build.outputTimestamp>2024-08-28T15:27:52Z</project.build.outputTimestamp>
     <!-- don't fail check for some rules that are too hard to enforce (could even be told broken for some)
@@ -83,9 +82,9 @@
       <version>${slf4jVersion}</version>
     </dependency>
     <dependency>
-      <groupId>org.sonatype.plexus</groupId>
+      <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-build-api</artifactId>
-      <version>${plexusBuildApiVersion}</version>
+      <version>1.2.0</version>
     </dependency>
     <dependency>
       <groupId>org.apache.maven</groupId>
diff --git a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
new file mode 100644
index 00000000..f4c832ba
--- /dev/null
+++ b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
@@ -0,0 +1,43 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.shared.filtering;
+
+/**
+ * Change detection strategies: to decide whether an existing target file has changed and needs to be overwritten or not.
+ *
+ * @since 3.5.0
+ */
+public enum ChangeDetection {
+    /**
+     * Only consider the timestamp of the file to determine if it has changed. This was default before 3.4.0.
+     */
+    TIMESTAMP,
+    /**
+     * Consider the content of the file to determine if it has changed. This is the default since 3.4.0.
+     */
+    CONTENT,
+    /**
+     * Combine timestamp and content change detection.
+     */
+    COMBINED,
+    /**
+     * Disable change detection; always overwrite.
+     */
+    ALWAYS;
+}
diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
index c6b03549..479d649c 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
@@ -56,12 +56,37 @@ public void copyFile(
             String encoding,
             MavenSession mavenSession)
             throws MavenFilteringException {
+        copyFile(
+                from,
+                to,
+                filtering,
+                mavenProject,
+                filters,
+                escapedBackslashesInFilePath,
+                encoding,
+                mavenSession,
+                ChangeDetection.CONTENT);
+    }
+
+    @Override
+    public void copyFile(
+            File from,
+            File to,
+            boolean filtering,
+            MavenProject mavenProject,
+            List<String> filters,
+            boolean escapedBackslashesInFilePath,
+            String encoding,
+            MavenSession mavenSession,
+            ChangeDetection changeDetection)
+            throws MavenFilteringException {
         MavenResourcesExecution mre = new MavenResourcesExecution();
         mre.setMavenProject(mavenProject);
         mre.setFileFilters(filters);
         mre.setEscapeWindowsPaths(escapedBackslashesInFilePath);
         mre.setMavenSession(mavenSession);
         mre.setInjectProjectBuildFilters(true);
+        mre.setChangeDetection(changeDetection);
 
         List<FilterWrapper> filterWrappers = getDefaultFilterWrappers(mre);
         copyFile(from, to, filtering, filterWrappers, encoding);
@@ -82,18 +107,50 @@ public void copyFile(MavenFileFilterRequest mavenFileFilterRequest) throws Maven
     @Override
     public void copyFile(File from, File to, boolean filtering, List<FilterWrapper> filterWrappers, String encoding)
             throws MavenFilteringException {
+        copyFile(from, to, filtering, filterWrappers, encoding, ChangeDetection.CONTENT);
+    }
+
+    @Override
+    @Deprecated
+    public void copyFile(
+            File from,
+            File to,
+            boolean filtering,
+            List<FilterWrapper> filterWrappers,
+            String encoding,
+            boolean overwrite)
+            throws MavenFilteringException {
+        // overwrite forced to false to preserve backward comp
+        copyFile(
+                from,
+                to,
+                filtering,
+                filterWrappers,
+                encoding,
+                overwrite ? ChangeDetection.ALWAYS : ChangeDetection.CONTENT);
+    }
+
+    @Override
+    public void copyFile(
+            File from,
+            File to,
+            boolean filtering,
+            List<FilterWrapper> filterWrappers,
+            String encoding,
+            ChangeDetection changeDetection)
+            throws MavenFilteringException {
         try {
             if (filtering) {
                 if (getLogger().isDebugEnabled()) {
                     getLogger().debug("filtering " + from.getPath() + " to " + to.getPath());
                 }
                 FilterWrapper[] array = filterWrappers.toArray(new FilterWrapper[0]);
-                FilteringUtils.copyFile(from, to, encoding, array, false);
+                FilteringUtils.copyFile(from, to, encoding, array, changeDetection);
             } else {
                 if (getLogger().isDebugEnabled()) {
                     getLogger().debug("copy " + from.getPath() + " to " + to.getPath());
                 }
-                FilteringUtils.copyFile(from, to, encoding, new FilterWrapper[0], false);
+                FilteringUtils.copyFile(from, to, encoding, new FilterWrapper[0], changeDetection);
             }
 
             buildContext.refresh(to);
@@ -104,18 +161,4 @@ public void copyFile(File from, File to, boolean filtering, List<FilterWrapper>
                     e);
         }
     }
-
-    @Override
-    @Deprecated
-    public void copyFile(
-            File from,
-            File to,
-            boolean filtering,
-            List<FilterWrapper> filterWrappers,
-            String encoding,
-            boolean overwrite)
-            throws MavenFilteringException {
-        // overwrite forced to false to preserve backward comp
-        copyFile(from, to, filtering, filterWrappers, encoding);
-    }
 }
diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
index 0c8688f8..dd62d124 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
@@ -280,7 +280,7 @@ private File getTargetFile(File file) throws MavenFilteringException {
                 File destinationFile = getDestinationFile(outputDirectory, targetPath, name, mavenResourcesExecution);
 
                 if (mavenResourcesExecution.isFlatten() && destinationFile.exists()) {
-                    if (mavenResourcesExecution.isOverwrite()) {
+                    if (mavenResourcesExecution.getChangeDetection() == ChangeDetection.ALWAYS) {
                         LOGGER.warn("existing file " + destinationFile.getName() + " will be overwritten by " + name);
                     } else {
                         throw new MavenFilteringException("existing file " + destinationFile.getName()
@@ -303,7 +303,7 @@ private File getTargetFile(File file) throws MavenFilteringException {
                         resource.isFiltering() && filteredExt,
                         mavenResourcesExecution.getFilterWrappers(),
                         encoding,
-                        mavenResourcesExecution.isOverwrite());
+                        mavenResourcesExecution.getChangeDetection());
             }
 
             // deal with deleted source files
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index b68e3cd6..4c3b5f7c 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -308,10 +308,59 @@ static boolean isEmpty(final String string) {
      * @throws IOException if an IO error occurs during copying or filtering
      */
     public static void copyFile(File from, File to, String encoding, FilterWrapper[] wrappers) throws IOException {
-        setReadWritePermissions(to);
+        copyFile(from, to, encoding, wrappers, ChangeDetection.CONTENT);
+    }
+
+    /**
+     * <b>If wrappers is null or empty, the file will be copy only if to.lastModified() &lt; from.lastModified() or if
+     * overwrite is true</b>.
+     *
+     * @param from the file to copy
+     * @param to the destination file
+     * @param encoding the file output encoding (only if wrappers is not empty)
+     * @param wrappers array of {@link FilterWrapper}
+     * @throws IOException if an IO error occurs during copying or filtering
+     */
+    public static void copyFile(
+            File from, File to, String encoding, FilterWrapper[] wrappers, ChangeDetection changeDetection)
+            throws IOException {
+        if (!to.isFile()) {
+            changeDetection = ChangeDetection.ALWAYS;
+        }
+        boolean needsCopy = false;
+        boolean unconditionally = false;
+        switch (changeDetection) {
+            case ALWAYS:
+                needsCopy = true;
+                unconditionally = true;
+                break;
+            case TIMESTAMP:
+                needsCopy = to.lastModified() < from.lastModified();
+                unconditionally = true;
+                break;
+            case CONTENT:
+                needsCopy = true;
+                break;
+            case COMBINED:
+                needsCopy = to.lastModified() < from.lastModified();
+                break;
+            default:
+                throw new IllegalArgumentException("Unsupported change detection mode: " + changeDetection);
+        }
+        if (needsCopy) {
+            if (unconditionally) {
+                copyUnconditionally(from, to, encoding, wrappers);
+            } else {
+                copyIfContentsChanged(from, to, encoding, wrappers);
+            }
+        }
+    }
 
+    public static boolean copyUnconditionally(File from, File to, String encoding, FilterWrapper[] wrappers)
+            throws IOException {
+        setReadWritePermissions(to);
         if (wrappers == null || wrappers.length == 0) {
-            try (OutputStream os = new CachingOutputStream(to.toPath())) {
+            try (OutputStream os = Files.newOutputStream(to.toPath())) {
                 Files.copy(from.toPath(), os);
             }
         } else {
@@ -321,7 +370,7 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
                 for (FilterWrapper wrapper : wrappers) {
                     wrapped = wrapper.getReader(wrapped);
                 }
-                try (Writer writer = new CachingWriter(to.toPath(), charset)) {
+                try (Writer writer = Files.newBufferedWriter(to.toPath(), charset)) {
                     char[] buffer = new char[COPY_BUFFER_LENGTH];
                     int nRead;
                     while ((nRead = wrapped.read(buffer, 0, COPY_BUFFER_LENGTH)) >= 0) {
@@ -330,8 +379,38 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
                 }
             }
         }
+        copyFilePermissions(from, to);
+        return true;
+    }
 
+    public static boolean copyIfContentsChanged(File from, File to, String encoding, FilterWrapper[] wrappers)
+            throws IOException {
+        setReadWritePermissions(to);
+        boolean copied = false;
+        if (wrappers == null || wrappers.length == 0) {
+            try (CachingOutputStream os = new CachingOutputStream(to.toPath())) {
+                Files.copy(from.toPath(), os);
+                copied = os.isModified();
+            }
+        } else {
+            Charset charset = charset(encoding);
+            try (Reader fileReader = Files.newBufferedReader(from.toPath(), charset)) {
+                Reader wrapped = fileReader;
+                for (FilterWrapper wrapper : wrappers) {
+                    wrapped = wrapper.getReader(wrapped);
+                }
+                try (CachingWriter writer = new CachingWriter(to.toPath(), charset)) {
+                    char[] buffer = new char[COPY_BUFFER_LENGTH];
+                    int nRead;
+                    while ((nRead = wrapped.read(buffer, 0, COPY_BUFFER_LENGTH)) >= 0) {
+                        writer.write(buffer, 0, nRead);
+                    }
+                    copied = writer.isModified();
+                }
+            }
+        }
         copyFilePermissions(from, to);
+        return copied;
     }
 
     /**
@@ -349,7 +428,7 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
     @Deprecated
     public static void copyFile(File from, File to, String encoding, FilterWrapper[] wrappers, boolean overwrite)
             throws IOException {
-        copyFile(from, to, encoding, wrappers);
+        copyFile(from, to, encoding, wrappers, overwrite ? ChangeDetection.ALWAYS : ChangeDetection.CONTENT);
     }
 
     /**
diff --git a/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java b/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
index a68db1d1..7badf117 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
@@ -42,6 +42,7 @@ public interface MavenFileFilter extends DefaultFilterInfo {
      * @param encoding The encoding which is used during the filtering process.
      * @throws MavenFilteringException in case of failure.
      * @see DefaultFilterInfo#getDefaultFilterWrappers(MavenProject, List, boolean,MavenSession, MavenResourcesExecution)
+     * @deprecated Use copyFile(File, File, boolean, MavenProject, List<String>, boolean, String, ChangeDetection) instead.
      */
     void copyFile(
             File from,
@@ -54,6 +55,33 @@ void copyFile(
             MavenSession mavenSession)
             throws MavenFilteringException;
 
+    /**
+     * Will copy a file with some filtering using defaultFilterWrappers.
+     *
+     * @param from file to copy/filter
+     * @param to destination file
+     * @param filtering enable or not filtering
+     * @param mavenProject {@link MavenProject}
+     * @param mavenSession {@link MavenSession}
+     * @param escapedBackslashesInFilePath escape backslashes in file path.
+     * @param filters {@link List} of String which are path to a Property file
+     * @param encoding The encoding which is used during the filtering process.
+     * @throws MavenFilteringException in case of failure.
+     * @see DefaultFilterInfo#getDefaultFilterWrappers(MavenProject, List, boolean,MavenSession, MavenResourcesExecution)
+     * @since 3.5.0
+     */
+    void copyFile(
+            File from,
+            File to,
+            boolean filtering,
+            MavenProject mavenProject,
+            List<String> filters,
+            boolean escapedBackslashesInFilePath,
+            String encoding,
+            MavenSession mavenSession,
+            ChangeDetection changeDetection)
+            throws MavenFilteringException;
+
     /**
      * @param mavenFileFilterRequest the request
      * @throws MavenFilteringException in case of failure.
@@ -68,7 +96,9 @@ void copyFile(
      * @param filterWrappers {@link List} of FileUtils.FilterWrapper
      * @param encoding The encoding used during the filtering.
      * @throws MavenFilteringException In case of an error.
+     * @deprecated use {@link #copyFile(File, File, boolean, List, String, ChangeDetection)} instead
      */
+    @Deprecated
     void copyFile(File from, File to, boolean filtering, List<FilterWrapper> filterWrappers, String encoding)
             throws MavenFilteringException;
 
@@ -81,7 +111,7 @@ void copyFile(File from, File to, boolean filtering, List<FilterWrapper> filterW
      * @param overwrite unused
      * @throws MavenFilteringException In case of an error.
      * @since 1.0-beta-2
-     * @deprecated use {@link #copyFile(File, File, boolean, List, String)} instead
+     * @deprecated use {@link #copyFile(File, File, boolean, List, String, ChangeDetection)} instead
      */
     @Deprecated
     void copyFile(
@@ -92,4 +122,23 @@ void copyFile(
             String encoding,
             boolean overwrite)
             throws MavenFilteringException;
+
+    /**
+     * @param from The source file
+     * @param to The destination file
+     * @param filtering true to apply filtering
+     * @param filterWrappers The filters to be applied.
+     * @param encoding The encoding to use
+     * @param changeDetection The change detection mode to use to determine if the file should be copied/filtered.
+     * @throws MavenFilteringException In case of an error.
+     * @since 3.5.0
+     */
+    void copyFile(
+            File from,
+            File to,
+            boolean filtering,
+            List<FilterWrapper> filterWrappers,
+            String encoding,
+            ChangeDetection changeDetection)
+            throws MavenFilteringException;
 }
diff --git a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
index 460aeeaf..4e271a0b 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
@@ -29,6 +29,8 @@
 import org.codehaus.plexus.interpolation.StringSearchInterpolator;
 import org.codehaus.plexus.interpolation.ValueSource;
 
+import static java.util.Objects.requireNonNull;
+
 /**
  * A bean to configure a resources filtering execution.
  *
@@ -91,7 +93,7 @@ public class MavenResourcesExecution extends AbstractMavenFilteringRequest {
      *
      * @since 1.0-beta-2
      */
-    private boolean overwrite = false;
+    private ChangeDetection changeDetection = ChangeDetection.CONTENT;
 
     /**
      * Copy any empty directories included in the Resources.
@@ -352,11 +354,13 @@ public void setAddDefaultExcludes(boolean addDefaultExcludes) {
     /**
      * Overwrite existing files even if the destination files are newer.
      *
-     * @return {@link #overwrite}
+     * @return {@code true} if operation always overwrites.
      * @since 1.0-beta-2
+     * @deprecated Use #getChangeDetection() instead.
      */
+    @Deprecated
     public boolean isOverwrite() {
-        return overwrite;
+        return changeDetection == ChangeDetection.ALWAYS;
     }
 
     /**
@@ -364,9 +368,30 @@ public boolean isOverwrite() {
      *
      * @param overwrite overwrite true or false.
      * @since 1.0-beta-2
+     * @deprecated Use #setChangeDetection(ChangeDetection) instead.
      */
+    @Deprecated
     public void setOverwrite(boolean overwrite) {
-        this.overwrite = overwrite;
+        this.changeDetection = overwrite ? ChangeDetection.ALWAYS : ChangeDetection.CONTENT;
+    }
+
+    /**
+     * Change detection strategy to determine whether an existing file should be overwritten.
+     *
+     * @since 3.5.0
+     */
+    public ChangeDetection getChangeDetection() {
+        return changeDetection;
+    }
+
+    /**
+     * Sets the change detection strategy to determine whether an existing file should be overwritten.
+     *
+     * @param changeDetection the change detection strategy to use, must not be {@code null}.
+     * @since 3.5.0
+     */
+    public void setChangeDetection(ChangeDetection changeDetection) {
+        this.changeDetection = requireNonNull(changeDetection);
     }
 
     /**
@@ -440,7 +465,7 @@ public MavenResourcesExecution copyOf() {
         mre.setMavenSession(this.getMavenSession());
         mre.setNonFilteredFileExtensions(copyList(this.getNonFilteredFileExtensions()));
         mre.setOutputDirectory(this.getOutputDirectory());
-        mre.setOverwrite(this.isOverwrite());
+        mre.setChangeDetection(this.getChangeDetection());
         mre.setProjectStartExpressions(copyList(this.getProjectStartExpressions()));
         mre.setResources(copyList(this.getResources()));
         mre.setResourcesBaseDirectory(this.getResourcesBaseDirectory());

From 6202237410fa7c6535e4ad9de78887f36db043c6 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 21:30:41 +0100
Subject: [PATCH 02/12] Fix javadoc

---
 .../java/org/apache/maven/shared/filtering/MavenFileFilter.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java b/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
index 7badf117..272e5ec5 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java
@@ -42,7 +42,7 @@ public interface MavenFileFilter extends DefaultFilterInfo {
      * @param encoding The encoding which is used during the filtering process.
      * @throws MavenFilteringException in case of failure.
      * @see DefaultFilterInfo#getDefaultFilterWrappers(MavenProject, List, boolean,MavenSession, MavenResourcesExecution)
-     * @deprecated Use copyFile(File, File, boolean, MavenProject, List<String>, boolean, String, ChangeDetection) instead.
+     * @deprecated Use {@link #copyFile(File, File, boolean, MavenProject, List, boolean, String, MavenSession, ChangeDetection)} instead.
      */
     void copyFile(
             File from,

From 487941440152b10bc2c46344cf9ffe7995d79a70 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 21:37:27 +0100
Subject: [PATCH 03/12] Tidy up

---
 .../maven/shared/filtering/FilteringUtils.java      | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 4c3b5f7c..d9bb2936 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -320,8 +320,9 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
      * @param encoding the file output encoding (only if wrappers is not empty)
      * @param wrappers array of {@link FilterWrapper}
      * @throws IOException if an IO error occurs during copying or filtering
+     * @return {@code true} if the file was copied.
      */
-    public static void copyFile(
+    public static boolean copyFile(
             File from, File to, String encoding, FilterWrapper[] wrappers, ChangeDetection changeDetection)
             throws IOException {
         if (!to.isFile()) {
@@ -347,16 +348,18 @@ public static void copyFile(
             default:
                 throw new IllegalArgumentException("Unsupported change detection mode: " + changeDetection);
         }
+        boolean copied = false;
         if (needsCopy) {
             if (unconditionally) {
-                copyUnconditionally(from, to, encoding, wrappers);
+                copied = copyUnconditionally(from, to, encoding, wrappers);
             } else {
-                copyIfContentsChanged(from, to, encoding, wrappers);
+                copied = copyIfContentsChanged(from, to, encoding, wrappers);
             }
         }
+        return copied;
     }
 
-    public static boolean copyUnconditionally(File from, File to, String encoding, FilterWrapper[] wrappers)
+    private static boolean copyUnconditionally(File from, File to, String encoding, FilterWrapper[] wrappers)
             throws IOException {
         setReadWritePermissions(to);
         if (wrappers == null || wrappers.length == 0) {
@@ -383,7 +386,7 @@ public static boolean copyUnconditionally(File from, File to, String encoding, F
         return true;
     }
 
-    public static boolean copyIfContentsChanged(File from, File to, String encoding, FilterWrapper[] wrappers)
+    private static boolean copyIfContentsChanged(File from, File to, String encoding, FilterWrapper[] wrappers)
             throws IOException {
         setReadWritePermissions(to);
         boolean copied = false;

From 1f2ab7f54b930c2973a117c8e4aa0c33cbbf3234 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 21:39:11 +0100
Subject: [PATCH 04/12] Javadoc

---
 .../maven/shared/filtering/MavenResourcesExecution.java       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
index 4e271a0b..1e3a4436 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
@@ -89,9 +89,9 @@ public class MavenResourcesExecution extends AbstractMavenFilteringRequest {
     private boolean addDefaultExcludes = true;
 
     /**
-     * Overwrite existing files even if the destination files are newer. <code>false</code> by default.
+     * Change detection strategy to determine whether an existing file should be overwritten.
      *
-     * @since 1.0-beta-2
+     * @since 3.5.0
      */
     private ChangeDetection changeDetection = ChangeDetection.CONTENT;
 

From 18d3aa95ed038c3926646ff722cafb10797809a2 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 21:48:00 +0100
Subject: [PATCH 05/12] Rename enum

---
 .../java/org/apache/maven/shared/filtering/ChangeDetection.java | 2 +-
 .../java/org/apache/maven/shared/filtering/FilteringUtils.java  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
index f4c832ba..2b2baa64 100644
--- a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
+++ b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
@@ -35,7 +35,7 @@ public enum ChangeDetection {
     /**
      * Combine timestamp and content change detection.
      */
-    COMBINED,
+    TIMESTAMP_AND_CONTENT,
     /**
      * Disable change detection; always overwrite.
      */
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index d9bb2936..285ad8ab 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -342,7 +342,7 @@ public static boolean copyFile(
             case CONTENT:
                 needsCopy = true;
                 break;
-            case COMBINED:
+            case TIMESTAMP_AND_CONTENT:
                 needsCopy = to.lastModified() < from.lastModified();
                 break;
             default:

From beadc12625cef1a032aebe9d1c0aa05db5dbf626 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 22:01:16 +0100
Subject: [PATCH 06/12] Drop unwanted deps

---
 pom.xml                                         | 17 ++++++-----------
 .../DefaultMavenResourcesFiltering.java         | 10 +++++-----
 .../maven/shared/filtering/FilteringUtils.java  |  9 +++++++--
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/pom.xml b/pom.xml
index 21b86f92..e86189c1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -113,16 +113,6 @@
       <artifactId>plexus-interpolation</artifactId>
       <version>1.29</version>
     </dependency>
-    <dependency>
-      <groupId>commons-io</groupId>
-      <artifactId>commons-io</artifactId>
-      <version>2.21.0</version>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.commons</groupId>
-      <artifactId>commons-lang3</artifactId>
-      <version>3.20.0</version>
-    </dependency>
 
     <dependency>
       <groupId>org.mockito</groupId>
@@ -163,7 +153,12 @@
       <artifactId>org.eclipse.sisu.inject</artifactId>
       <scope>test</scope>
     </dependency>
-
+    <dependency>
+      <groupId>commons-io</groupId>
+      <artifactId>commons-io</artifactId>
+      <version>2.21.0</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
   <build>
diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
index dd62d124..47c7171c 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
@@ -34,12 +34,12 @@
 import java.util.List;
 import java.util.Locale;
 
-import org.apache.commons.io.FilenameUtils;
-import org.apache.commons.io.IOUtils;
-import org.apache.commons.lang3.StringUtils;
 import org.apache.maven.model.Resource;
 import org.codehaus.plexus.util.DirectoryScanner;
+import org.codehaus.plexus.util.FileUtils;
+import org.codehaus.plexus.util.IOUtil;
 import org.codehaus.plexus.util.Scanner;
+import org.codehaus.plexus.util.StringUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.sonatype.plexus.build.incremental.BuildContext;
@@ -93,7 +93,7 @@ public boolean filteredFileExtension(String fileName, List<String> userNonFilter
     }
 
     private static String getExtension(String fileName) {
-        String rawExt = FilenameUtils.getExtension(fileName);
+        String rawExt = FileUtils.getExtension(fileName);
         return rawExt == null ? null : rawExt.toLowerCase(Locale.ROOT);
     }
 
@@ -507,7 +507,7 @@ private String filterFileName(String name, List<FilterWrapper> wrappers) throws
         }
 
         try (StringWriter writer = new StringWriter()) {
-            IOUtils.copy(reader, writer);
+            IOUtil.copy(reader, writer);
             String filteredFilename = writer.toString();
 
             if (LOGGER.isDebugEnabled()) {
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 285ad8ab..293041ac 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -32,7 +32,6 @@
 import java.util.StringTokenizer;
 import java.util.regex.Pattern;
 
-import org.apache.commons.lang3.SystemUtils;
 import org.codehaus.plexus.util.io.CachingOutputStream;
 import org.codehaus.plexus.util.io.CachingWriter;
 
@@ -41,6 +40,12 @@
  * @author Dennis Lundberg
  */
 public final class FilteringUtils {
+    /**
+     * Logic borrowed from Commons-Lang3: we really need only this, to decide is it Windows.
+     * It is nonsense to import whole commons-lang3 just for this test.
+     */
+    private static final boolean IS_WINDOWS =
+            System.getProperty("os.name", "unknown").startsWith("Windows");
     /**
      * The number of bytes in a kilobyte.
      */
@@ -115,7 +120,7 @@ public static File resolveFile(final File baseFile, String filename) {
         }
 
         // deal with absolute files
-        if (filenm.startsWith(File.separator) || (SystemUtils.IS_OS_WINDOWS && filenm.indexOf(":") > 0)) {
+        if (filenm.startsWith(File.separator) || (IS_WINDOWS && filenm.indexOf(":") > 0)) {
             File file = new File(filenm);
 
             try {

From 2fe1de167367608184b6c941dbdf67a04eda9bba Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Thu, 19 Feb 2026 22:04:44 +0100
Subject: [PATCH 07/12] Drop and tidy up deps

---
 pom.xml | 30 ++++++++++++++++--------------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/pom.xml b/pom.xml
index e86189c1..cbb9e129 100644
--- a/pom.xml
+++ b/pom.xml
@@ -71,11 +71,6 @@
   </properties>
 
   <dependencies>
-    <dependency>
-      <groupId>javax.inject</groupId>
-      <artifactId>javax.inject</artifactId>
-      <version>1</version>
-    </dependency>
     <dependency>
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
@@ -86,6 +81,22 @@
       <artifactId>plexus-build-api</artifactId>
       <version>1.2.0</version>
     </dependency>
+    <dependency>
+      <groupId>org.codehaus.plexus</groupId>
+      <artifactId>plexus-utils</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.codehaus.plexus</groupId>
+      <artifactId>plexus-interpolation</artifactId>
+      <version>1.29</version>
+    </dependency>
+
+    <dependency>
+      <groupId>javax.inject</groupId>
+      <artifactId>javax.inject</artifactId>
+      <version>1</version>
+      <scope>provided</scope>
+    </dependency>
     <dependency>
       <groupId>org.apache.maven</groupId>
       <artifactId>maven-core</artifactId>
@@ -104,15 +115,6 @@
       <version>${mavenVersion}</version>
       <scope>provided</scope>
     </dependency>
-    <dependency>
-      <groupId>org.codehaus.plexus</groupId>
-      <artifactId>plexus-utils</artifactId>
-    </dependency>
-    <dependency>
-      <groupId>org.codehaus.plexus</groupId>
-      <artifactId>plexus-interpolation</artifactId>
-      <version>1.29</version>
-    </dependency>
 
     <dependency>
       <groupId>org.mockito</groupId>

From a1f51f1b04df4344747ef7751fe283d96b3dbf60 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Fri, 20 Feb 2026 12:33:28 +0100
Subject: [PATCH 08/12] Add NEVER strategy

It simply copied if target does not exists, and stands off
if target exists.
---
 .../maven/shared/filtering/ChangeDetection.java      | 12 ++++++++----
 .../maven/shared/filtering/FilteringUtils.java       |  4 ++++
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
index 2b2baa64..7d0bc9cf 100644
--- a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
+++ b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
@@ -19,17 +19,17 @@
 package org.apache.maven.shared.filtering;
 
 /**
- * Change detection strategies: to decide whether an existing target file has changed and needs to be overwritten or not.
+ * Change detection strategies: to decide whether an <strong>existing target file</strong> needs to be overwritten or not.
  *
  * @since 3.5.0
  */
 public enum ChangeDetection {
     /**
-     * Only consider the timestamp of the file to determine if it has changed. This was default before 3.4.0.
+     * Only consider the file timestamp to determine change. This was default before 3.4.0.
      */
     TIMESTAMP,
     /**
-     * Consider the content of the file to determine if it has changed. This is the default since 3.4.0.
+     * Only consider the content of the file to determine change. This is the default since 3.4.0.
      */
     CONTENT,
     /**
@@ -39,5 +39,9 @@ public enum ChangeDetection {
     /**
      * Disable change detection; always overwrite.
      */
-    ALWAYS;
+    ALWAYS,
+    /**
+     * Disable change detection; never overwrite.
+     */
+    NEVER;
 }
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 293041ac..89e0dae7 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -336,6 +336,10 @@ public static boolean copyFile(
         boolean needsCopy = false;
         boolean unconditionally = false;
         switch (changeDetection) {
+            case NEVER:
+                needsCopy = !to.isFile();
+                unconditionally = true;
+                break;
             case ALWAYS:
                 needsCopy = true;
                 unconditionally = true;

From e403227befd2178ada8765e13f30a093b0207eca Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Fri, 20 Feb 2026 12:47:18 +0100
Subject: [PATCH 09/12] Reshuffle

---
 .../AbstractMavenFilteringRequest.java        | 28 ++++++++++++++++
 .../filtering/DefaultMavenFileFilter.java     |  7 ++--
 .../shared/filtering/FilteringUtils.java      |  4 ++-
 .../filtering/MavenResourcesExecution.java    | 32 ++-----------------
 4 files changed, 38 insertions(+), 33 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/AbstractMavenFilteringRequest.java b/src/main/java/org/apache/maven/shared/filtering/AbstractMavenFilteringRequest.java
index 6daebd89..e94c9d4f 100644
--- a/src/main/java/org/apache/maven/shared/filtering/AbstractMavenFilteringRequest.java
+++ b/src/main/java/org/apache/maven/shared/filtering/AbstractMavenFilteringRequest.java
@@ -28,6 +28,8 @@
 import org.apache.maven.project.MavenProject;
 import org.codehaus.plexus.interpolation.Interpolator;
 
+import static java.util.Objects.requireNonNull;
+
 /**
  * @since 1.0-beta-3
  */
@@ -83,6 +85,13 @@ public class AbstractMavenFilteringRequest {
 
     private Consumer<Interpolator> interpolatorCustomizer;
 
+    /**
+     * Change detection strategy to determine whether an existing file should be overwritten.
+     *
+     * @since 3.5.0
+     */
+    private ChangeDetection changeDetection = ChangeDetection.CONTENT;
+
     /**
      * Create instance.
      */
@@ -360,4 +369,23 @@ public Consumer<Interpolator> getInterpolatorCustomizer() {
     public void setInterpolatorCustomizer(Consumer<Interpolator> interpolatorCustomizer) {
         this.interpolatorCustomizer = interpolatorCustomizer;
     }
+
+    /**
+     * Change detection strategy to determine whether an existing file should be overwritten.
+     *
+     * @since 3.5.0
+     */
+    public ChangeDetection getChangeDetection() {
+        return changeDetection;
+    }
+
+    /**
+     * Sets the change detection strategy to determine whether an existing file should be overwritten.
+     *
+     * @param changeDetection the change detection strategy to use, must not be {@code null}.
+     * @since 3.5.0
+     */
+    public void setChangeDetection(ChangeDetection changeDetection) {
+        this.changeDetection = requireNonNull(changeDetection);
+    }
 }
diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
index 479d649c..499befaf 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
@@ -45,6 +45,7 @@ public DefaultMavenFileFilter(BuildContext buildContext) {
         this.buildContext = requireNonNull(buildContext);
     }
 
+    @Deprecated
     @Override
     public void copyFile(
             File from,
@@ -89,7 +90,7 @@ public void copyFile(
         mre.setChangeDetection(changeDetection);
 
         List<FilterWrapper> filterWrappers = getDefaultFilterWrappers(mre);
-        copyFile(from, to, filtering, filterWrappers, encoding);
+        copyFile(from, to, filtering, filterWrappers, encoding, changeDetection);
     }
 
     @Override
@@ -101,9 +102,11 @@ public void copyFile(MavenFileFilterRequest mavenFileFilterRequest) throws Maven
                 mavenFileFilterRequest.getTo(),
                 mavenFileFilterRequest.isFiltering(),
                 filterWrappers,
-                mavenFileFilterRequest.getEncoding());
+                mavenFileFilterRequest.getEncoding(),
+                mavenFileFilterRequest.getChangeDetection());
     }
 
+    @Deprecated
     @Override
     public void copyFile(File from, File to, boolean filtering, List<FilterWrapper> filterWrappers, String encoding)
             throws MavenFilteringException {
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 89e0dae7..efc08aca 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -311,7 +311,9 @@ static boolean isEmpty(final String string) {
      * @param encoding the file output encoding (only if wrappers is not empty)
      * @param wrappers array of {@link FilterWrapper}
      * @throws IOException if an IO error occurs during copying or filtering
+     * @deprecated Use {@link #copyFile(File, File, String, FilterWrapper[], ChangeDetection)} instead.
      */
+    @Deprecated
     public static void copyFile(File from, File to, String encoding, FilterWrapper[] wrappers) throws IOException {
         copyFile(from, to, encoding, wrappers, ChangeDetection.CONTENT);
     }
@@ -435,7 +437,7 @@ private static boolean copyIfContentsChanged(File from, File to, String encoding
      * @param wrappers array of {@link FilterWrapper}
      * @param overwrite unused
      * @throws IOException if an IO error occurs during copying or filtering
-     * @deprecated use {@link #copyFile(File, File, String, FilterWrapper[])} instead
+     * @deprecated use {@link #copyFile(File, File, String, FilterWrapper[], ChangeDetection)} instead
      */
     @Deprecated
     public static void copyFile(File from, File to, String encoding, FilterWrapper[] wrappers, boolean overwrite)
diff --git a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
index 1e3a4436..8cd052ae 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MavenResourcesExecution.java
@@ -29,8 +29,6 @@
 import org.codehaus.plexus.interpolation.StringSearchInterpolator;
 import org.codehaus.plexus.interpolation.ValueSource;
 
-import static java.util.Objects.requireNonNull;
-
 /**
  * A bean to configure a resources filtering execution.
  *
@@ -88,13 +86,6 @@ public class MavenResourcesExecution extends AbstractMavenFilteringRequest {
      */
     private boolean addDefaultExcludes = true;
 
-    /**
-     * Change detection strategy to determine whether an existing file should be overwritten.
-     *
-     * @since 3.5.0
-     */
-    private ChangeDetection changeDetection = ChangeDetection.CONTENT;
-
     /**
      * Copy any empty directories included in the Resources.
      *
@@ -360,7 +351,7 @@ public void setAddDefaultExcludes(boolean addDefaultExcludes) {
      */
     @Deprecated
     public boolean isOverwrite() {
-        return changeDetection == ChangeDetection.ALWAYS;
+        return getChangeDetection() == ChangeDetection.ALWAYS;
     }
 
     /**
@@ -372,26 +363,7 @@ public boolean isOverwrite() {
      */
     @Deprecated
     public void setOverwrite(boolean overwrite) {
-        this.changeDetection = overwrite ? ChangeDetection.ALWAYS : ChangeDetection.CONTENT;
-    }
-
-    /**
-     * Change detection strategy to determine whether an existing file should be overwritten.
-     *
-     * @since 3.5.0
-     */
-    public ChangeDetection getChangeDetection() {
-        return changeDetection;
-    }
-
-    /**
-     * Sets the change detection strategy to determine whether an existing file should be overwritten.
-     *
-     * @param changeDetection the change detection strategy to use, must not be {@code null}.
-     * @since 3.5.0
-     */
-    public void setChangeDetection(ChangeDetection changeDetection) {
-        this.changeDetection = requireNonNull(changeDetection);
+        setChangeDetection(overwrite ? ChangeDetection.ALWAYS : ChangeDetection.CONTENT);
     }
 
     /**

From df9415736b69851432abef117f4751283a11b5eb Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Fri, 20 Feb 2026 12:49:00 +0100
Subject: [PATCH 10/12] Clearer javadoc

---
 .../apache/maven/shared/filtering/ChangeDetection.java | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
index 7d0bc9cf..98ef9a92 100644
--- a/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
+++ b/src/main/java/org/apache/maven/shared/filtering/ChangeDetection.java
@@ -25,23 +25,23 @@
  */
 public enum ChangeDetection {
     /**
-     * Only consider the file timestamp to determine change. This was default before 3.4.0.
+     * Only consider the file timestamp to determine is overwrite of existing file needed. This was default before 3.4.0.
      */
     TIMESTAMP,
     /**
-     * Only consider the content of the file to determine change. This is the default since 3.4.0.
+     * Only consider the content of the file to determine is overwrite of existing file needed. This is the default since 3.4.0.
      */
     CONTENT,
     /**
-     * Combine timestamp and content change detection.
+     * Combine timestamp and content change detection for existing file.
      */
     TIMESTAMP_AND_CONTENT,
     /**
-     * Disable change detection; always overwrite.
+     * Disable change detection; always overwrite existing file.
      */
     ALWAYS,
     /**
-     * Disable change detection; never overwrite.
+     * Disable change detection; never overwrite existing file.
      */
     NEVER;
 }

From 4c6eaa7734a07167fde2b2897abe527c536b1e39 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Fri, 20 Feb 2026 12:52:44 +0100
Subject: [PATCH 11/12] Tidy up

---
 .../apache/maven/shared/filtering/DefaultMavenFileFilter.java  | 1 -
 .../java/org/apache/maven/shared/filtering/FilteringUtils.java | 3 ---
 2 files changed, 4 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
index 499befaf..3a22ca2f 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenFileFilter.java
@@ -123,7 +123,6 @@ public void copyFile(
             String encoding,
             boolean overwrite)
             throws MavenFilteringException {
-        // overwrite forced to false to preserve backward comp
         copyFile(
                 from,
                 to,
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index efc08aca..165f7b56 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -332,9 +332,6 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
     public static boolean copyFile(
             File from, File to, String encoding, FilterWrapper[] wrappers, ChangeDetection changeDetection)
             throws IOException {
-        if (!to.isFile()) {
-            changeDetection = ChangeDetection.ALWAYS;
-        }
         boolean needsCopy = false;
         boolean unconditionally = false;
         switch (changeDetection) {

From 2c9091218a69ca0bfa0c702e3b1c5226c9325af4 Mon Sep 17 00:00:00 2001
From: Tamas Cservenak <tamas@cservenak.net>
Date: Fri, 20 Feb 2026 13:24:18 +0100
Subject: [PATCH 12/12] Fix javadoc

---
 .../org/apache/maven/shared/filtering/FilteringUtils.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 165f7b56..74be18a0 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -319,13 +319,13 @@ public static void copyFile(File from, File to, String encoding, FilterWrapper[]
     }
 
     /**
-     * <b>If wrappers is null or empty, the file will be copy only if to.lastModified() &lt; from.lastModified() or if
-     * overwrite is true</b>.
+     * Copies a file from the source to the destination, applying the specified filters if provided.
      *
      * @param from the file to copy
      * @param to the destination file
      * @param encoding the file output encoding (only if wrappers is not empty)
      * @param wrappers array of {@link FilterWrapper}
+     * @param changeDetection the strategy to apply if to is existing file
      * @throws IOException if an IO error occurs during copying or filtering
      * @return {@code true} if the file was copied.
      */