Merge pull request #19 from ls1intum/dev

Predixx · web-flow · commit bb4942eb956b · 2025-11-21T21:03:53.000+01:00
Release 0.2.6: Practice Mode
diff --git a/iris-thaumantias/CHANGELOG.md b/iris-thaumantias/CHANGELOG.md
@@ -2,6 +2,16 @@
 
 All notable changes to the Artemis VS Code extension will be documented in this file.
 
+## [0.2.6] - 2025-11-21
+
+### Added
+
+- **Practice Mode**: Added support for starting practice runs on programming exercises after the due date, including automatic repository detection and safety warnings.
+
+### Fixed
+
+- **Recently Cloned Notice**: Fixed an issue where the "Recently cloned" notice would persist even after the repository was successfully opened.
+
 ## [0.2.5] - 2025-11-07
 
 ### Added
diff --git a/iris-thaumantias/package.json b/iris-thaumantias/package.json
@@ -2,7 +2,7 @@
   "name": "iris-thaumantias",
   "displayName": "Artemis - TUM",
   "description": "Artemis brings interactive learning to life with instant, individual feedback on programming exercises, quizzes, modeling tasks, and more. This VS Code extension integrates Iris, an LLM-based virtual assistant that supports students with instant answers about exercises, lectures, and learning performance. Iris provides context-aware, personalized assistance for programming exercises, encouraging independent problem-solving through subtle hints and guiding questions instead of giving full solutions.",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "publisher": "aet-tum",
   "license": "MIT",
   "icon": "media/artemis-blue.png",
diff --git a/iris-thaumantias/src/api/artemisApi.ts b/iris-thaumantias/src/api/artemisApi.ts
@@ -207,6 +207,15 @@ export class ArtemisApiService {
         return response.json();
     }
 
+    // Start practice participation in an exercise
+    async startPracticeParticipation(exerciseId: number): Promise<any> {
+        const response = await this.makeRequest(
+            `/api/exercise/exercises/${exerciseId}/participations/practice`,
+            { method: 'POST' }
+        );
+        return response.json();
+    }
+
     // Authenticate user with username and password
     async authenticate(username: string, password: string, rememberMe: boolean = false): Promise<any> {
         const url = `${this.getServerUrl()}${CONFIG.API.ENDPOINTS.AUTHENTICATE}`;
diff --git a/iris-thaumantias/src/views/app/commands/repositoryCommands.ts b/iris-thaumantias/src/views/app/commands/repositoryCommands.ts
@@ -38,6 +38,7 @@ export class RepositoryCommandModule {
             saveGitCredentials: this.handleSaveGitCredentials,
             saveGitIdentity: this.handleSaveGitIdentity,
             requestGitIdentity: this.handleRequestGitIdentity,
+            startPractice: this.handleStartPractice,
         };
     }
 
@@ -84,23 +85,67 @@ export class RepositoryCommandModule {
                         for (const exercise of exercises) {
                             const participations = exercise.studentParticipations || [];
 
-                            if (participations.length > 0 && participations[0].repositoryUri) {
-                                const exerciseRepoUrl = normalizeUrl(participations[0].repositoryUri);
-
-                                if (exerciseRepoUrl === normalizedRepoUrl) {
-                                    matchedExercise = {
-                                        id: exercise.id,
-                                        title: exercise.title
-                                    };
-                                    break;
+                            // Check all participations (including practice ones)
+                            for (const participation of participations) {
+                                if (participation.repositoryUri) {
+                                    const exerciseRepoUrl = normalizeUrl(participation.repositoryUri);
+
+                                    if (exerciseRepoUrl === normalizedRepoUrl) {
+                                        matchedExercise = {
+                                            id: exercise.id,
+                                            title: exercise.title
+                                        };
+                                        break;
+                                    }
                                 }
                             }
+                            
+                            if (matchedExercise) {
+                                break;
+                            }
                         }
 
                         if (matchedExercise) {
                             break;
                         }
                     }
+
+                    // Fallback: If no exact match found, check if it's a practice repo that matches a graded repo
+                    if (!matchedExercise && normalizedRepoUrl.includes('-practice-')) {
+                        // Try to construct the graded repo URL by removing '-practice'
+                        // Example: .../slug-practice-user -> .../slug-user
+                        const potentialGradedUrl = normalizedRepoUrl.replace('-practice-', '-');
+                        
+                        for (const courseData of coursesData.courses) {
+                            const exercises = courseData?.course?.exercises || courseData?.exercises || [];
+    
+                            for (const exercise of exercises) {
+                                const participations = exercise.studentParticipations || [];
+    
+                                for (const participation of participations) {
+                                    if (participation.repositoryUri) {
+                                        const exerciseRepoUrl = normalizeUrl(participation.repositoryUri);
+    
+                                        if (exerciseRepoUrl === potentialGradedUrl) {
+                                            matchedExercise = {
+                                                id: exercise.id,
+                                                title: exercise.title
+                                            };
+                                            break;
+                                        }
+                                    }
+                                }
+                                
+                                if (matchedExercise) {
+                                    break;
+                                }
+                            }
+    
+                            if (matchedExercise) {
+                                break;
+                            }
+                        }
+                    }
                 }
 
                 this.context.sendMessage({
@@ -156,6 +201,7 @@ export class RepositoryCommandModule {
             const workspaceFolder = vscode.workspace.workspaceFolders?.[0];
             let isConnected = false;
             let hasChanges = false;
+            let isGradedRepo = false;
 
             this.currentRepoContext = { expectedRepoUrl, exerciseId };
             this.currentWorkspacePath = workspaceFolder?.uri.fsPath;
@@ -194,6 +240,29 @@ export class RepositoryCommandModule {
                             console.warn('Failed to determine repository changes:', statusError);
                             hasChanges = false;
                         }
+                    } else {
+                        // Check if we are in the graded repository
+                        const coursesData = this.context.appStateManager.coursesData;
+                        if (coursesData?.courses) {
+                            for (const courseData of coursesData.courses) {
+                                const exercises = courseData?.course?.exercises || courseData?.exercises || [];
+                                const exercise = exercises.find((e: any) => e.id === exerciseId);
+                                
+                                if (exercise) {
+                                    const participations = exercise.studentParticipations || [];
+                                    // Find graded participation (not testRun)
+                                    const gradedParticipation = participations.find((p: any) => !p.testRun);
+                                    
+                                    if (gradedParticipation?.repositoryUri) {
+                                        const normalizedGraded = normalizeUrl(gradedParticipation.repositoryUri);
+                                        if (normalizedCurrent === normalizedGraded) {
+                                            isGradedRepo = true;
+                                        }
+                                    }
+                                    break;
+                                }
+                            }
+                        }
                     }
                 } catch (gitError: any) {
                     isConnected = false;
@@ -204,7 +273,8 @@ export class RepositoryCommandModule {
             this.context.sendMessage({
                 command: 'updateRepoStatus',
                 isConnected: isConnected,
-                hasChanges: hasChanges
+                hasChanges: hasChanges,
+                isGradedRepo: isGradedRepo
             });
         } catch (error: any) {
             console.error('Check repository status error:', error);
@@ -947,4 +1017,27 @@ export class RepositoryCommandModule {
             autoSaveEnabled: autoSave !== 'off'
         });
     }
+
+    private handleStartPractice = async (message: any): Promise<void> => {
+        const exerciseId: number = message.exerciseId;
+        const exerciseTitle: string = message.exerciseTitle;
+
+        try {
+            vscode.window.showInformationMessage('Starting practice mode...');
+            const participation = await this.context.artemisApi.startPracticeParticipation(exerciseId);
+
+            if (participation) {
+                vscode.window.showInformationMessage(
+                    `Successfully started practice mode for "${exerciseTitle}". You can now clone the practice repository.`
+                );
+
+                await this.context.actionHandler.openExerciseDetails(exerciseId);
+            }
+        } catch (error) {
+            console.error('Failed to start practice participation:', error);
+            vscode.window.showErrorMessage(
+                `Failed to start practice mode for "${exerciseTitle}": ${error instanceof Error ? error.message : 'Unknown error'}`
+            );
+        }
+    };
 }
diff --git a/iris-thaumantias/src/views/exerciseDetail/components/participationActionsComponent.ts b/iris-thaumantias/src/views/exerciseDetail/components/participationActionsComponent.ts
@@ -8,6 +8,8 @@ export interface ParticipationActionsData {
   exerciseTitle?: string;
   participationId?: number;
   uploadMessageIcon: string;
+  isPracticeAvailable?: boolean;
+  practiceParticipation?: any;
 }
 
 /**
@@ -26,10 +28,12 @@ export class ParticipationActionsComponent {
       exerciseType,
       uploadMessageIcon,
       participationId,
+      isPracticeAvailable,
+      practiceParticipation,
     } = data;
 
     const changeStatusHtml =
-      hasParticipation && isProgrammingExercise
+      (hasParticipation || practiceParticipation) && isProgrammingExercise
         ? `
       <div class="changes-status" id="changesStatus" data-state="checking">
         <span class="changes-status-indicator"></span>
@@ -42,7 +46,9 @@ export class ParticipationActionsComponent {
       hasParticipation,
       isProgrammingExercise,
       changeStatusHtml,
-      uploadMessageIcon
+      uploadMessageIcon,
+      isPracticeAvailable,
+      practiceParticipation
     );
 
     // Determine participation info based on exercise type
@@ -72,8 +78,22 @@ export class ParticipationActionsComponent {
     hasParticipation: boolean,
     isProgrammingExercise: boolean,
     changeStatusHtml: string,
-    uploadMessageIcon: string
+    uploadMessageIcon: string,
+    isPracticeAvailable?: boolean,
+    practiceParticipation?: any
   ): string {
+    if (isPracticeAvailable) {
+      return this._generatePracticeAvailableActions();
+    }
+
+    if (practiceParticipation) {
+      return this._generateProgrammingExerciseActions(
+        changeStatusHtml,
+        uploadMessageIcon,
+        true
+      );
+    }
+
     if (hasParticipation) {
       if (isProgrammingExercise) {
         return this._generateProgrammingExerciseActions(
@@ -92,15 +112,37 @@ export class ParticipationActionsComponent {
     }
   }
 
+  /**
+   * Generate actions for practice available state
+   */
+  private static _generatePracticeAvailableActions(): string {
+    return `
+      <div class="participation-actions not-participated">
+        <div class="action-button-row">
+          <button class="participate-btn" onclick="startPractice()">Practice</button>
+          <button class="participate-btn secondary" onclick="openExerciseInBrowser()">Open in browser</button>
+        </div>
+      </div>
+    `;
+  }
+
   /**
    * Generate actions for participated programming exercises
    */
   private static _generateProgrammingExerciseActions(
     changeStatusHtml: string,
-    uploadMessageIcon: string
+    uploadMessageIcon: string,
+    isPractice: boolean = false
   ): string {
+    const practiceLabel = isPractice 
+      ? `<div class="practice-mode-indicator">
+           <span class="codicon codicon-beaker"></span> Practice Mode
+         </div>` 
+      : '';
+
     return `
       <div class="participation-actions">
+        ${practiceLabel}
         ${changeStatusHtml}
         <div class="cloned-repo-notice" id="clonedRepoNotice" style="display: none;">
           <span id="clonedRepoMessage">Repository recently cloned.</span> 
diff --git a/iris-thaumantias/src/views/exerciseDetail/components/repositoryStatusScripts.ts b/iris-thaumantias/src/views/exerciseDetail/components/repositoryStatusScripts.ts
@@ -17,7 +17,17 @@ export class RepositoryStatusScripts {
     return `
       // Repository status checking
       window.checkRepositoryStatus = function(showChecking = false) {
-          const participation = exerciseData.exercise?.studentParticipations?.[0] || exerciseData.studentParticipations?.[0];
+          const ex = exerciseData.exercise || exerciseData;
+          const participations = ex.studentParticipations || [];
+          
+          // Check for practice participation first
+          let participation = participations.find(p => p.testRun);
+          
+          // If no practice participation, use the first one (graded)
+          if (!participation && participations.length > 0) {
+              participation = participations[0];
+          }
+          
           const repoUrl = participation?.repositoryUri;
           
           if (!repoUrl) {
diff --git a/iris-thaumantias/src/views/exerciseDetail/exercise-detail.css b/iris-thaumantias/src/views/exerciseDetail/exercise-detail.css
@@ -685,6 +685,7 @@
             border-radius: 50%;
             background: var(--theme-border);
             transition: background 0.15s ease-in-out, transform 0.15s ease-in-out;
+            flex-shrink: 0;
         }
 
         .changes-status[data-state='dirty'] {
@@ -1459,3 +1460,36 @@
             color: white;
             border-color: var(--theme-success);
         }
+
+/* Practice Mode Styles */
+.practice-mode-indicator {
+    display: flex;
+    align-items: center;
+    gap: 8px;
+    padding: 8px 12px;
+    background-color: var(--vscode-editorInfo-background, rgba(117, 190, 255, 0.1));
+    color: var(--vscode-editorInfo-foreground, #75beff);
+    border-radius: 4px;
+    margin-bottom: 12px;
+    font-weight: 600;
+    font-size: 13px;
+}
+
+.practice-mode-indicator .codicon {
+    font-size: 16px;
+}
+
+/* Wrong Repo Warning */
+.changes-status[data-state="wrong-repo"] {
+    color: var(--vscode-editorWarning-foreground, #cca700);
+    font-weight: 600;
+}
+
+.changes-status[data-state="wrong-repo"] .changes-status-indicator {
+    background-color: var(--vscode-editorWarning-foreground, #cca700);
+}
+
+.repo-status-icon.warning {
+    color: var(--vscode-editorWarning-foreground, #cca700);
+    border-color: var(--vscode-editorWarning-foreground, #cca700);
+}
diff --git a/iris-thaumantias/src/views/exerciseDetail/exerciseDetailView.ts b/iris-thaumantias/src/views/exerciseDetail/exerciseDetailView.ts
diff --git a/iris-thaumantias/src/views/utils/exerciseDataTransformer.ts b/iris-thaumantias/src/views/utils/exerciseDataTransformer.ts
