Add @deprecated annotations and move formatLogMessage to Message.toString()

Co-authored-by: laeubi <[email protected]>

Author: Copilot

src/main/java/org/codehaus/plexus/build/BuildContext.java

@@ -211,6 +211,7 @@ public interface BuildContext {
      * @param message a {@link java.lang.String} object.
      * @deprecated Use {@link org.codehaus.plexus.build.messages.Messages} API instead
      */
+    @Deprecated
     void addMessage(File file, int line, int column, String message, int severity, Throwable cause);
 
     /**
@@ -221,6 +222,7 @@ public interface BuildContext {
      * @param file a {@link java.io.File} object.
      * @deprecated Use {@link org.codehaus.plexus.build.messages.Messages#clear(java.nio.file.Path)} instead
      */
+    @Deprecated
     void removeMessages(File file);
 
     /**

src/main/java/org/codehaus/plexus/build/messages/DefaultMessages.java

@@ -86,7 +86,7 @@ public MessageBuilder build(MessageType type, Path path) {
      */
     private void handleMessage(Message message) {
         // Log the message
-        String logMessage = formatLogMessage(message);
+        String logMessage = message.toString();
 
         switch (message.getType()) {
             case ERROR:
@@ -113,44 +113,6 @@ private void handleMessage(Message message) {
         }
     }
 
-    /**
-     * Formats a message for logging.
-     *
-     * @param message the message to format
-     * @return the formatted log message
-     */
-    private String formatLogMessage(Message message) {
-        StringBuilder sb = new StringBuilder();
-
-        if (message.getPath() != null) {
-            sb.append(message.getPath().toAbsolutePath());
-        }
-
-        if (message.getLine() > 0 && message.getColumn() > 0) {
-            sb.append(" [");
-            sb.append(message.getLine());
-            sb.append(':').append(message.getColumn());
-            sb.append("]");
-        } else if (message.getLine() > 0) {
-            sb.append(" [");
-            sb.append(message.getLine());
-            sb.append("]");
-        } else if (message.getColumn() > 0) {
-            sb.append(" [:");
-            sb.append(message.getColumn());
-            sb.append("]");
-        }
-
-        if (message.getMessage() != null) {
-            if (sb.length() > 0) {
-                sb.append(": ");
-            }
-            sb.append(message.getMessage());
-        }
-
-        return sb.toString();
-    }
-
     /**
      * Maps a MessageType to a BuildContext severity level.
      *

src/main/java/org/codehaus/plexus/build/messages/Message.java

@@ -84,4 +84,43 @@ public String getMessage() {
     public Throwable getCause() {
         return cause;
     }
+
+    /**
+     * Returns a string representation of this message.
+     * The format is: path [line:column]: message
+     *
+     * @return the formatted message string
+     */
+    @Override
+    public String toString() {
+        StringBuilder sb = new StringBuilder();
+
+        if (path != null) {
+            sb.append(path.toAbsolutePath());
+        }
+
+        if (line > 0 && column > 0) {
+            sb.append(" [");
+            sb.append(line);
+            sb.append(':').append(column);
+            sb.append("]");
+        } else if (line > 0) {
+            sb.append(" [");
+            sb.append(line);
+            sb.append("]");
+        } else if (column > 0) {
+            sb.append(" [:");
+            sb.append(column);
+            sb.append("]");
+        }
+
+        if (message != null) {
+            if (sb.length() > 0) {
+                sb.append(": ");
+            }
+            sb.append(message);
+        }
+
+        return sb.toString();
+    }
 }