Merge pull request #7382 from christianbeeznest/GH-7307

Assignment: Add AI task grader - refs #7307

Author: christianbeeznest

assets/vue/components/assignments/CorrectAndRateModal.vue

@@ -11,7 +11,6 @@
       @page="onPage"
       @sort="onSort"
     >
-      <!-- Full name -->
       <Column
         field="user.fullName"
         :header="t('Full name')"
@@ -126,13 +125,23 @@
               @click="saveCorrection(data)"
               type="primary"
             />
+            <BaseButton
+              v-if="canUseAiTaskGrader"
+              icon="robot"
+              size="normal"
+              only-icon
+              :label="t('AI grade')"
+              :class="actionBtnClass"
+              @click="openCorrectAndRate(data)"
+              type="black"
+            />
             <BaseButton
               icon="reply-all"
               size="normal"
               only-icon
               :label="t('Correct and rate')"
               :class="actionBtnClass"
-              @click="correctAndRate(data)"
+              @click="openCorrectAndRate(data)"
               type="success"
             />
             <BaseButton
@@ -212,7 +221,7 @@
 </template>
 
 <script setup>
-import { nextTick, watch, reactive, ref } from "vue"
+import { nextTick, watch, reactive, ref, computed, onMounted } from "vue"
 import { useI18n } from "vue-i18n"
 import Column from "primevue/column"
 import BaseButton from "../basecomponents/BaseButton.vue"
@@ -226,6 +235,11 @@ import { RESOURCE_LINK_DRAFT, RESOURCE_LINK_PUBLISHED } from "../../constants/en
 import EditStudentSubmissionForm from "./EditStudentSubmissionForm.vue"
 import CorrectAndRateModal from "./CorrectAndRateModal.vue"
 import MoveSubmissionModal from "./MoveSubmissionModal.vue"
+import { useCidReqStore } from "../../store/cidReq"
+import { storeToRefs } from "pinia"
+import { usePlatformConfig } from "../../store/platformConfig"
+import { useCourseSettings } from "../../store/courseSettingStore"
+import { useSecurityStore } from "../../store/securityStore"
 
 const props = defineProps({
   assignmentId: {
@@ -254,12 +268,54 @@ const editingItem = ref(null)
 const showCorrectAndRateDialog = ref(false)
 const correctingItem = ref(null)
 const showMoveDialog = ref(false)
-
-/**
- * Make action buttons visually bigger without relying on unknown "size" enums.
- * This keeps current BaseButton behavior and only increases hit area.
- */
 const actionBtnClass = "w-10 h-10 !p-2"
+const cidReqStore = useCidReqStore()
+const { course, session } = storeToRefs(cidReqStore)
+
+const platform = usePlatformConfig()
+const courseSettingsStore = useCourseSettings()
+const securityStore = useSecurityStore()
+
+async function loadCourseSettingsIfPossible() {
+  const courseId = course.value?.id
+  const sessionId = session.value?.id
+
+  if (!courseId) return
+
+  try {
+    await courseSettingsStore.loadCourseSettings(courseId, sessionId)
+  } catch (err) {
+    console.error("[Assignments] loadCourseSettings FAILED:", err)
+  }
+}
+
+onMounted(async () => {
+  await loadCourseSettingsIfPossible()
+})
+
+watch(
+  () => [course.value?.id, session.value?.id],
+  async () => {
+    await loadCourseSettingsIfPossible()
+  },
+)
+
+const aiHelpersEnabled = computed(() => {
+  const v = String(platform.getSetting("ai_helpers.enable_ai_helpers"))
+  return v === "true"
+})
+
+const taskGraderEnabled = computed(() => {
+  const v = courseSettingsStore?.getSetting?.("task_grader")
+  return String(v) === "true"
+})
+
+const canUseAiTaskGrader = computed(() => {
+  // Only teachers/admins and not in student view
+  const canEdit = !!(securityStore.isTeacher || securityStore.isCourseAdmin || securityStore.isAdmin)
+  const notStudentView = !platform.isStudentViewActive
+  return !!(canEdit && notStudentView && aiHelpersEnabled.value && taskGraderEnabled.value)
+})
 
 watch(
   loadParams,
@@ -270,14 +326,28 @@ watch(
   { deep: true, immediate: true },
 )
 
+function buildOrderFromSort() {
+  // PrimeVue multi sort meta => API expects { field: "asc|desc" }
+  const order = {}
+  const meta = Array.isArray(sortFields.value) ? sortFields.value : []
+  meta.forEach((s) => {
+    if (!s?.field) return
+    order[s.field] = s.order === 1 ? "asc" : "desc"
+  })
+  if (!Object.keys(order).length) {
+    order.sentDate = "desc"
+  }
+  return order
+}
+
 async function loadData() {
   loading.value = true
   try {
     const response = await cStudentPublicationService.getAssignmentDetailForTeacher({
       assignmentId: props.assignmentId,
       page: loadParams.page,
       itemsPerPage: loadParams.itemsPerPage,
-      order: { sentDate: "desc" },
+      order: buildOrderFromSort(),
     })
 
     submissions.value = response["hydra:member"]
@@ -295,13 +365,9 @@ function onPage(event) {
 }
 
 function onSort(event) {
-  Object.keys(loadParams)
-    .filter((key) => key.startsWith("order["))
-    .forEach((key) => delete loadParams[key])
-
-  event.multiSortMeta.forEach((sortItem) => {
-    loadParams[`order[${sortItem.field}]`] = sortItem.order === 1 ? "asc" : "desc"
-  })
+  if (event?.multiSortMeta) {
+    sortFields.value = event.multiSortMeta
+  }
 }
 
 /**
@@ -438,7 +504,7 @@ async function deleteSubmission(item) {
   }
 }
 
-function correctAndRate(item) {
+function openCorrectAndRate(item) {
   correctingItem.value = null
   nextTick(() => {
     correctingItem.value = item
@@ -447,11 +513,7 @@ function correctAndRate(item) {
 }
 
 function openCommentDialog(item) {
-  correctingItem.value = null
-  nextTick(() => {
-    correctingItem.value = item
-    showCorrectAndRateDialog.value = true
-  })
+  openCorrectAndRate(item)
 }
 
 function moveSubmission(item) {

assets/vue/components/assignments/TeacherSubmissionList.vue

@@ -46,6 +46,7 @@ async function getAssignmentMetadata(assignmentId, cid, sid = 0, gid = 0) {
 
 async function getAssignmentDetail({ assignmentId, page = 1, itemsPerPage = 10, order = {} }) {
   const params = {
+    ...buildCidParams(),
     page,
     itemsPerPage,
     ...Object.fromEntries(Object.entries(order).map(([key, val]) => [`order[${key}]`, val])),
@@ -56,6 +57,7 @@ async function getAssignmentDetail({ assignmentId, page = 1, itemsPerPage = 10,
 
 async function getAssignmentDetailForTeacher({ assignmentId, page = 1, itemsPerPage = 10, order = {} }) {
   const params = {
+    ...buildCidParams(),
     page,
     itemsPerPage,
     ...Object.fromEntries(Object.entries(order).map(([key, val]) => [`order[${key}]`, val])),
@@ -72,18 +74,23 @@ async function uploadStudentAssignment(formData, queryParams) {
 }
 
 async function getStudentProgress(queryParams = {}) {
-  const params = new URLSearchParams(queryParams).toString()
+  const merged = { ...buildCidParams(), ...queryParams }
+  const params = new URLSearchParams(merged).toString()
   const url = params ? `/assignments/progress?${params}` : `/assignments/progress`
   const response = await axios.get(url)
   return response.data
 }
 
 async function deleteAssignmentSubmission(submissionId) {
-  await axios.delete(`/assignments/submissions/${submissionId}`)
+  await axios.delete(`/assignments/submissions/${submissionId}`, {
+    params: buildCidParams(),
+  })
 }
 
 async function updateSubmission(id, data) {
-  await axios.patch(`/assignments/submissions/${id}/edit`, data)
+  await axios.patch(`/assignments/submissions/${id}/edit`, data, {
+    params: buildCidParams(),
+  })
 }
 
 async function uploadComment(submissionId, parentResourceNodeId, formData, sendMail = false) {
@@ -113,13 +120,17 @@ async function loadComments(submissionId) {
     })
     return response.data["hydra:member"] || []
   } catch (error) {
-    console.error("Failed to load comments", error)
+    console.error("[Assignments] Failed to load comments", error)
     return []
   }
 }
 
 async function moveSubmission(submissionId, newAssignmentId) {
-  const response = await axios.patch(`/assignments/submissions/${submissionId}/move`, { newAssignmentId })
+  const response = await axios.patch(
+    `/assignments/submissions/${submissionId}/move`,
+    { newAssignmentId },
+    { params: buildCidParams() },
+  )
   return response.data
 }
 
@@ -130,18 +141,19 @@ async function getUnsubmittedUsers(assignmentId) {
 }
 
 async function sendEmailToUnsubmitted(assignmentId, queryParams = {}) {
-  const params = new URLSearchParams(queryParams).toString()
+  const merged = { ...buildCidParams(), ...queryParams }
+  const params = new URLSearchParams(merged).toString()
   const response = await axios.post(`/assignments/${assignmentId}/unsubmitted-users/email?${params}`)
   return response.data
 }
 
 async function deleteAllCorrections(assignmentId, cid, sid = 0) {
-  const params = { cid, ...(sid && { sid }) }
+  const params = { ...buildCidParams(), cid, ...(sid && { sid }) }
   await axios.delete(`/assignments/${assignmentId}/corrections/delete`, { params })
 }
 
 async function exportAssignmentPdf(assignmentId, cid, sid = 0, gid = 0) {
-  const params = { cid, ...(sid && { sid }), ...(gid && { gid }) }
+  const params = { ...buildCidParams(), cid, ...(sid && { sid }), ...(gid && { gid }) }
   const response = await axios.get(`/assignments/${assignmentId}/export/pdf`, {
     params,
     responseType: "blob",
@@ -151,6 +163,7 @@ async function exportAssignmentPdf(assignmentId, cid, sid = 0, gid = 0) {
 
 async function downloadAssignments(assignmentId) {
   const response = await axios.get(`/assignments/${assignmentId}/download-package`, {
+    params: buildCidParams(),
     responseType: "blob",
   })
   return response.data
@@ -161,6 +174,7 @@ async function uploadCorrectionsPackage(assignmentId, file) {
   formData.append("file", file)
 
   const response = await axios.post(`/assignments/${assignmentId}/upload-corrections-package`, formData, {
+    params: buildCidParams(),
     headers: { "Content-Type": "multipart/form-data" },
   })
 
@@ -171,6 +185,36 @@ async function updateScore(iid, qualification) {
   return axios.put(`${ENTRYPOINT}c_student_publications/${iid}`, { qualification }, { params: buildCidParams() })
 }
 
+async function aiGradeSubmission(submissionId, payload = {}) {
+  const response = await axios.post(`/assignments/submissions/${submissionId}/ai-grade`, payload, {
+    headers: { "Content-Type": "application/json" },
+    params: buildCidParams(),
+  })
+  return response.data
+}
+
+async function getAiTextProviders() {
+  const { data } = await axios.get("/assignments/ai/text-providers")
+  return data
+}
+
+async function getAiTaskGraderDefaultPrompt(submissionId, params = {}) {
+  const { data } = await axios.get(`/assignments/submissions/${submissionId}/ai-task-grader-default-prompt`, {
+    params,
+  })
+  return data
+}
+
+async function aiTaskGrade(submissionId, payload) {
+  const { data } = await axios.post(`/assignments/submissions/${submissionId}/ai-task-grade`, payload)
+  return data
+}
+
+async function getAiTaskGradeCapabilities(submissionId) {
+  const { data } = await this.api.get(`/assignments/submissions/${submissionId}/ai-task-grade-capabilities`)
+  return data
+}
+
 export default {
   ...makeService("c_student_publications"),
   findStudentAssignments,
@@ -192,4 +236,9 @@ export default {
   uploadCorrectionsPackage,
   updateScore,
   updatePublication,
+  aiGradeSubmission,
+  getAiTextProviders,
+  getAiTaskGraderDefaultPrompt,
+  aiTaskGrade,
+  getAiTaskGradeCapabilities,
 }

assets/vue/services/cstudentpublication.js

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+/* For licensing terms, see /license.txt */
+
+namespace Chamilo\CoreBundle\AiProvider;
+
+interface AiDocumentProcessProviderInterface
+{
+    /**
+     * Process a document and return feedback text.
+     *
+     * @param string $prompt         Prompt that includes context and grading instructions
+     * @param string $toolName       Tag, e.g. 'task_grader'
+     * @param string $filename       Original file name, e.g. "submission.pdf"
+     * @param string $mimeType       e.g. "application/pdf"
+     * @param string $binaryContent  Raw file bytes (NOT base64)
+     * @param array  $options        Optional overrides (model, temperature, max_output_tokens, etc.)
+     */
+    public function processDocument(
+        string $prompt,
+        string $toolName,
+        string $filename,
+        string $mimeType,
+        string $binaryContent,
+        array $options = []
+    ): ?string;
+}

src/CoreBundle/AiProvider/AiDocumentProcessProviderInterface.php

@@ -163,6 +163,11 @@ public function __construct(
         }
     }
 
+    public function create(string $provider, string $serviceType = 'text'): object
+    {
+        return $this->getProvider($provider, $serviceType);
+    }
+
     public function hasProvidersForType(string $serviceType): bool
     {
         return !empty($this->providersByType[$serviceType] ?? []);