- fixed the jenkins comments by removed the duplicated some fields (id, description) already provided by BaseStandardCredentials.

Author: Susenthiran28

src/main/java/io/jenkins/plugins/vigilnz/build/SecurityCheckBuilder.java

@@ -178,7 +178,7 @@ public ListBoxModel doFillCredentialsIdItems(@AncestorInPath Item project) {
                     project,
                     ACL.SYSTEM,  // Use actual user authentication, not ACL.SYSTEM
                     Collections.emptyList())) {
-                String label = c.getTokenId().isEmpty() ? c.getTokenDescription() : c.getTokenId();
+                String label = c.getId().isEmpty() ? c.getDescription() : c.getId();
                 if (label == null || label.isEmpty()) {
                     label = c.getId();
                 }

src/main/java/io/jenkins/plugins/vigilnz/credentials/TokenCredentials.java

@@ -18,51 +18,28 @@ public class TokenCredentials extends BaseStandardCredentials {
 
     private final Secret token;
 
-    @SuppressWarnings("lgtm[jenkins/plaintext-storage]")
-    private final String tokenId;
-
-    @SuppressWarnings("lgtm[jenkins/plaintext-storage]")
-    private final String tokenDescription;
-
     @DataBoundConstructor
     public TokenCredentials(
             CredentialsScope scope,
-            String tokenId,
-            String tokenDescription,
+            String id,
+            String description,
             Secret token
     ) {
-        super(scope, tokenId, tokenDescription);
-        // If tokenId is null or empty, use empty string (Jenkins will handle ID generation)
-        // This prevents errors when updating credentials that were created without an ID
-        String idToUse = (tokenId == null || tokenId.trim().isEmpty()) ? "" : tokenId;
+        super(scope, id, description);
 
         if (token == null || Secret.toString(token).isEmpty()) {
             throw new IllegalArgumentException("Token is required");
         }
-        if (tokenId != null && tokenId.contains(" ")) {
-            throw new IllegalArgumentException("Token ID must not contain spaces");
-        }
+//        if (id != null && id.contains(" ")) {
+//            throw new IllegalArgumentException("Token ID must not contain spaces");
+//        }
         this.token = token;
-        this.tokenId = idToUse;
-        this.tokenDescription = tokenDescription;
     }
 
     public Secret getToken() {
         return token;
     }
 
-    public String getTokenId() {
-        // If tokenId was never set by user, return the actual Jenkins-generated ID
-        if (tokenId == null || tokenId.trim().isEmpty()) {
-            return getId();
-        }
-        return tokenId;
-    }
-
-    public String getTokenDescription() {
-        return tokenDescription;
-    }
-
 
     // Descriptor for Jenkins UI
     @Symbol("vigilnzToken")

src/main/resources/io/jenkins/plugins/vigilnz/credentials/TokenCredentials/config.jelly

@@ -15,13 +15,13 @@
         </script>
     </f:entry>
 
-    <f:entry field="tokenId" title="ID">
+    <f:entry field="id" title="ID">
         <j:choose>
             <!-- Existing credential: show read-only with actual Jenkins ID -->
             <j:when test="${instance != null}">
                 <!-- Get the actual ID to display and submit -->
                 <j:set var="idValue"
-                       value="${instance.tokenId != null and instance.tokenId != '' ? instance.tokenId : instance.id}"/>
+                       value="${instance.id != null and instance.id != '' ? instance.id : instance.id}"/>
                 <!-- Display as read-only -->
                 <f:readOnlyTextbox value="${idValue}"/>
 
@@ -33,7 +33,7 @@
         </j:choose>
     </f:entry>
 
-    <f:entry title="Description" field="tokenDescription">
+    <f:entry title="Description" field="description">
         <f:textbox/>
     </f:entry>
 </j:jelly>