@mo7yw4ng/openape 1.0.4 → 1.0.6

package/script/src/lib/moodle.js

@@ -40,6 +40,7 @@ exports.getEnrolledCourses = getEnrolledCourses;
 exports.getCourseState = getCourseState;
 exports.getSupervideosInCourse = getSupervideosInCourse;
 exports.getForumsApi = getForumsApi;
+exports.resolveForumId = resolveForumId;
 exports.getForumDiscussionsApi = getForumDiscussionsApi;
 exports.getDiscussionPostsApi = getDiscussionPostsApi;
 exports.deleteForumPostApi = deleteForumPostApi;
@@ -58,6 +59,7 @@ exports.getIncompleteVideosApi = getIncompleteVideosApi;
 exports.getSupervideosInCourseApi = getSupervideosInCourseApi;
 exports.getQuizzesByCoursesApi = getQuizzesByCoursesApi;
 exports.startQuizAttemptApi = startQuizAttemptApi;
+exports.getAllQuizAttemptDataApi = getAllQuizAttemptDataApi;
 exports.getQuizAttemptDataApi = getQuizAttemptDataApi;
 exports.processQuizAttemptApi = processQuizAttemptApi;
 exports.getResourcesByCoursesApi = getResourcesByCoursesApi;
@@ -319,6 +321,41 @@ async function getForumsApi(session, courseIds) {
         timemodified: f.timemodified,
     }));
 }
+/**
+ * Resolve a forum ID (cmid or instance ID) to a forum instance ID.
+ * Tries cmid resolution first (via core_course_get_course_module) to get name/course info.
+ * Falls back to treating the ID as a raw forum instance ID.
+ */
+async function resolveForumId(session, id) {
+    const numId = parseInt(id, 10);
+    // Try cmid resolution first (gets name + course info)
+    try {
+        const cm = await moodleApiCall(session, "core_course_get_course_module", { cmid: numId });
+        if (cm?.cm && cm.cm.modname === "forum") {
+            return {
+                forumId: cm.cm.instance,
+                cmid: numId,
+                name: cm.cm.name,
+                courseid: cm.cm.course,
+            };
+        }
+    }
+    catch {
+        // Not a valid cmid, try as forum instance ID
+    }
+    // Fall back: treat as forum instance ID directly
+    try {
+        const data = await moodleApiCall(session, "mod_forum_get_forum_discussions", { forumid: numId, limit: 1 });
+        // If we get discussions back (even empty), the forum exists
+        if (data) {
+            return { forumId: numId };
+        }
+    }
+    catch {
+        // Invalid forum instance ID
+    }
+    return null;
+}
 /**
  * Get discussions in a forum via WS API (no browser required).
  * Uses mod_forum_get_forum_discussions
@@ -410,7 +447,6 @@ async function addForumDiscussionApi(session, forumId, subject, message) {
             forumid: forumId,
             subject,
             message: message.replace(/\n/g, "<br>"),
-            messageformat: 1,
         });
         if (data?.discussionid) {
             return { success: true, discussionId: data.discussionid };
@@ -1043,6 +1079,24 @@ async function startQuizAttemptApi(session, quizId, options = {}) {
 /**
  * Get quiz attempt data including questions via pure WS API.
  */
+async function getAllQuizAttemptDataApi(session, attemptId) {
+    const firstPage = await getQuizAttemptDataApi(session, attemptId, 0);
+    // Moodle re-indexes question keys per page (always starts at 0),
+    // so we must re-key by actual slot number to avoid overwrites.
+    const allQuestions = {};
+    for (const q of Object.values(firstPage.questions)) {
+        allQuestions[q.slot] = q;
+    }
+    let nextPage = firstPage.nextpage;
+    while (nextPage !== undefined && nextPage !== null && nextPage !== -1) {
+        const pageData = await getQuizAttemptDataApi(session, attemptId, nextPage);
+        for (const q of Object.values(pageData.questions)) {
+            allQuestions[q.slot] = q;
+        }
+        nextPage = pageData.nextpage;
+    }
+    return { ...firstPage, questions: allQuestions, nextpage: undefined };
+}
 async function getQuizAttemptDataApi(session, attemptId, page = 0) {
     const data = await moodleApiCall(session, "mod_quiz_get_attempt_data", { attemptid: attemptId, page });
     if (!data?.attempt || !data?.questions) {
@@ -1288,7 +1342,7 @@ async function uploadFileApi(session, filePath, options) {
         // Prepare multipart form data
         const formData = new FormData();
         formData.append("token", session.wsToken);
-        formData.append("file", new Blob([fileContent]), fileName);
+        formData.append("file", new Blob([new Uint8Array(fileContent)]), fileName);
         formData.append("filepath", options?.filepath || "/");
         formData.append("itemid", String(draftItemId)); // Use our generated draft ID
         formData.append("contextid", String(userContextId)); // Use calculated user context

package/script/src/lib/utils.d.ts

@@ -12,7 +12,7 @@ export declare function stripHtmlTags(html: string): string;
 /**
  * Extract clean course name from Moodle fullname.
  * Removes mlang tags, course codes, and instructor info.
- * Example: "{mlang zh-tw}1142爵士樂賞析(遠距)-楊曊恩..." -> "爵士樂賞析"
+ * Example: "{mlang zh-tw}1142Jazz Analysis(Distance)-Instructor..." -> "Jazz Analysis"
  */
 export declare function extractCourseName(fullname: string): string;
 /**
@@ -24,11 +24,6 @@ export declare function getOutputFormat(command: {
         output?: OutputFormat;
     };
 }): OutputFormat;
-/**
- * Determine if logs should be silenced based on output format and verbosity.
- * JSON output without verbose flag silences logs.
- */
-export declare function shouldSilenceLogs(outputFormat: OutputFormat, verbose?: boolean): boolean;
 /**
  * Sanitize filename by removing invalid characters and limiting length.
  * Replaces invalid characters with underscores and limits to maxLength.
@@ -47,11 +42,11 @@ export declare function formatFileSize(bytes: number, decimals?: number): string
  */
 export declare function formatMoodleDate(timestamp?: number): string;
 /**
- * 統一時間戳記轉換 (預設：本地時間字串)
+ * Unified timestamp conversion (default: local time string)
  */
 export declare function formatTimestamp(timestamp: number | undefined | null, format?: "iso" | "local" | "relative"): string;
 /**
- * 相對時間格式 (e.g., "2 hours ago")
+ * Relative time format (e.g., "2 hours ago")
  */
 export declare function formatRelativeTime(timestamp: number): string;
 //# sourceMappingURL=utils.d.ts.map

package/script/src/lib/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../src/src/lib/utils.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAE/C;;;GAGG;AACH,wBAAgB,UAAU,IAAI,MAAM,CAcnC;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAelD;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAO1D;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE;IAAE,eAAe,IAAI;QAAE,MAAM,CAAC,EAAE,YAAY,CAAA;KAAE,CAAA;CAAE,GAAG,YAAY,CAGvG;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,YAAY,EAAE,YAAY,EAAE,OAAO,CAAC,EAAE,OAAO,GAAG,OAAO,CAExF;AAED;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,GAAE,MAAY,GAAG,MAAM,CAK9E;AAED;;GAEG;AACH,wBAAgB,cAAc,IAAI,MAAM,CAGvC;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,KAAK,EAAE,MAAM,EAAE,QAAQ,GAAE,MAAU,GAAG,MAAM,CAE1E;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,SAAS,CAAC,EAAE,MAAM,GAAG,MAAM,CAG3D;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,EAAE,MAAM,GAAE,KAAK,GAAG,OAAO,GAAG,UAAoB,GAAG,MAAM,CAQ5H;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAM5D"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../src/src/lib/utils.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAE/C;;;GAGG;AACH,wBAAgB,UAAU,IAAI,MAAM,CAcnC;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAelD;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAO1D;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE;IAAE,eAAe,IAAI;QAAE,MAAM,CAAC,EAAE,YAAY,CAAA;KAAE,CAAA;CAAE,GAAG,YAAY,CAGvG;AAED;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,GAAE,MAAY,GAAG,MAAM,CAK9E;AAED;;GAEG;AACH,wBAAgB,cAAc,IAAI,MAAM,CAGvC;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,KAAK,EAAE,MAAM,EAAE,QAAQ,GAAE,MAAU,GAAG,MAAM,CAE1E;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,SAAS,CAAC,EAAE,MAAM,GAAG,MAAM,CAG3D;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,EAAE,MAAM,GAAE,KAAK,GAAG,OAAO,GAAG,UAAoB,GAAG,MAAM,CAQ5H;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAM5D"}

package/script/src/lib/utils.js

@@ -37,7 +37,6 @@ exports.getBaseDir = getBaseDir;
 exports.stripHtmlTags = stripHtmlTags;
 exports.extractCourseName = extractCourseName;
 exports.getOutputFormat = getOutputFormat;
-exports.shouldSilenceLogs = shouldSilenceLogs;
 exports.sanitizeFilename = sanitizeFilename;
 exports.getSessionPath = getSessionPath;
 exports.formatFileSize = formatFileSize;
@@ -90,7 +89,7 @@ function stripHtmlTags(html) {
 /**
  * Extract clean course name from Moodle fullname.
  * Removes mlang tags, course codes, and instructor info.
- * Example: "{mlang zh-tw}1142爵士樂賞析(遠距)-楊曊恩..." -> "爵士樂賞析"
+ * Example: "{mlang zh-tw}1142Jazz Analysis(Distance)-Instructor..." -> "Jazz Analysis"
  */
 function extractCourseName(fullname) {
     if (!fullname)
@@ -109,13 +108,6 @@ function getOutputFormat(command) {
     const opts = command.optsWithGlobals();
     return opts.output || "json";
 }
-/**
- * Determine if logs should be silenced based on output format and verbosity.
- * JSON output without verbose flag silences logs.
- */
-function shouldSilenceLogs(outputFormat, verbose) {
-    return outputFormat === "json" && !verbose;
-}
 /**
  * Sanitize filename by removing invalid characters and limiting length.
  * Replaces invalid characters with underscores and limits to maxLength.
@@ -148,7 +140,7 @@ function formatMoodleDate(timestamp) {
     return new Date(timestamp * 1000).toLocaleString("zh-TW");
 }
 /**
- * 統一時間戳記轉換 (預設：本地時間字串)
+ * Unified timestamp conversion (default: local time string)
  */
 function formatTimestamp(timestamp, format = "local") {
     if (!timestamp || timestamp === 0)
@@ -161,7 +153,7 @@ function formatTimestamp(timestamp, format = "local") {
     return date.toLocaleString("zh-TW");
 }
 /**
- * 相對時間格式 (e.g., "2 hours ago")
+ * Relative time format (e.g., "2 hours ago")
  */
 function formatRelativeTime(timestamp) {
     const seconds = Math.floor(Date.now() / 1000) - timestamp;

package/skills/openape/SKILL.md

@@ -41,11 +41,13 @@ openape <command> [subcommand] [args] [flags]
 
 ### quizzes — Quiz operations
 
-  - `list <course-id>` — List incomplete quizzes in a course
+  - `list <course-id>` — List incomplete quizzes in a course. Flags: `--all`
   - `list-all` — List all incomplete quizzes across courses. Flags: `--level in_progress|all`
   - `start <quiz-id>` — Start a new quiz attempt
   - `info <attempt-id>` — Get quiz attempt data and questions. Flags: `--page <number>`
-  - `save <attempt-id> '<answers-json>'` — Save answers for a quiz attempt. JSON format: `[{"slot":1,"answer":"0"}]`. Multichoice: number, multichoices: `"0,2"`, shortanswer: text
+  - `save <attempt-id> '<answers-json>'` — Save answers for a quiz attempt. Flags: `--submit`. JSON format: `[{"slot":1,"answer":"0"}]`. Multichoice: number, multichoices: `"0,2"`, shortanswer: text
+
+> **NEVER SUBMIT WITHOUT USER'S PERMISSION**, you have to make sure answer is saved before submitting.
 
 ### materials — Material/resource operations
 
@@ -69,12 +71,12 @@ openape <command> [subcommand] [args] [flags]
 
 ### forums — Forum operations
 
-  - `list` — List forums from in-progress courses. Flags: `--level in_progress|all`
-  - `list-all` — List all forums across all courses
+  - `list` — List forums from in-progress courses
+  - `list-all` — List all forums across all courses. Flags: `--level in_progress|all`
   - `discussions <forum-id>` — List discussions in a forum
   - `posts <discussion-id>` — Show posts in a discussion
   - `post <forum-id> <subject> <message>` — Post a new discussion. Flags: `--subscribe`, `--pin`
-  - `reply <post-id> <subject> <message>` — Reply to a discussion post. Flags: `--parent-id <id>`
+  - `reply <post-id> <subject> <message>` — Reply to a discussion post. Flags: `--attachment-id <id>`, `--inline-attachment-id <id>`
   - `delete <post-id>` — Delete a forum post or discussion
 
 ### announcements — Announcement operations
@@ -84,8 +86,8 @@ openape <command> [subcommand] [args] [flags]
 
 ### calendar — Calendar operations
 
-  - `events` — List calendar events. Flags: `--course-id <id>`, `--events-after <date>`, `--events-before <date>`
-  - `export` — Export calendar events to file. Flags: `--format json|ics`, `--output <file>`
+  - `events` — List calendar events. Flags: `--course <id>`, `--upcoming`, `--days <n>`
+  - `export` — Export calendar events to file. Flags: `--output <path>`, `--days <n>`
 
 ### upload — File upload
 
@@ -98,7 +100,7 @@ openape <command> [subcommand] [args] [flags]
 
 ## Output Formats
 
-All commands support `--output`: `json` (default), `csv`, `table`, `silent`
+Most data commands support `--output`: `json` (default), `csv`, `table`, `silent`
 
 Global flags: `--verbose`, `--headed`, `--session <path>`