fix(lombok): Refine Lombok annotations

Author: HossamSaberr

pom.xml

@@ -30,14 +30,16 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
-                <version>3.11.0</version> <configuration>
+                <version>3.11.0</version>
+                <configuration>
                     <source>${maven.compiler.source}</source>
                     <target>${maven.compiler.target}</target>
                     <annotationProcessorPaths>
                         <path>
                             <groupId>org.projectlombok</groupId>
                             <artifactId>lombok</artifactId>
-                            <version>1.18.32</version> </path>
+                            <version>1.18.32</version>
+                        </path>
                     </annotationProcessorPaths>
                 </configuration>
             </plugin>

src/main/java/com/mycmd/ShellContext.java

@@ -2,14 +2,18 @@
 
 import lombok.Getter;
 import lombok.Setter;
+import lombok.NonNull;
 import java.io.*;
 import java.util.*;
 import java.time.Instant;
 
 @Getter 
-@Setter 
 public class ShellContext {
+    
+    @Setter
+    @NonNull 
     private File currentDir;
+    
     private List<String> history;
     private Map<String, String> aliases;
     private static final String ALIAS_FILE = ".mycmd_aliases";
@@ -30,12 +34,30 @@ public ShellContext() {
 
     public void addToHistory(String command) {
         history.add(command);
-        commandHistory.add(command); // Add to command history
+        commandHistory.add(command); 
         if (history.size() > MAX_HISTORY) {
             history.remove(0);
         }
     }
 
+    /** * OVERRIDES Lombok's generated getter to return a DEFENSIVE COPY 
+     * to prevent external code from modifying the shell directly.
+     */
+    @Override
+    public List<String> getHistory() {
+        return new ArrayList<>(history);
+    }
+
+    @Override
+    public Map<String, String> getAliases() {
+        return new HashMap<>(aliases);
+    }
+    
+    @Override
+    public Map<String, String> getEnvVars() {
+        return new HashMap<>(envVars);
+    }
+
 
     public void clearHistory() {
         history.clear();
@@ -51,11 +73,27 @@ public void removeAlias(String name) {
         aliases.remove(name);
         saveAliases();
     }
+    
+    public String getAlias(String name) {
+        return aliases.get(name);
+    }
 
     public boolean hasAlias(String name) {
         return aliases.containsKey(name);
     }
 
+    // Environmental variable accessors
+    public void setEnvVar(String key, String value) {
+        envVars.put(key, value);
+    }
+
+    public String getEnvVar(String key) {
+        return envVars.get(key);
+    }
+
+
+    // --- Private Persistence Methods ---
+
     private void loadAliases() {
         File aliasFile = new File(System.getProperty("user.home"), ALIAS_FILE);
         if (!aliasFile.exists()) {
@@ -109,5 +147,4 @@ public File resolvePath(String path) {
             return new File(currentDir, path);
         }
     }
-
-}
+}