@mo7yw4ng/openape 1.0.2 → 1.0.3

package/script/src/commands/forums.js

@@ -74,6 +74,7 @@ function registerForumsCommand(program) {
                     cmid: wsForum.cmid.toString(),
                     forum_id: wsForum.id,
                     name: wsForum.name,
+                    timemodified: wsForum.timemodified,
                     // url: `https://ilearning.cycu.edu.tw/mod/forum/view.php?id=${wsForum.cmid}`,
                 });
             }
@@ -117,7 +118,7 @@ function registerForumsCommand(program) {
                     cmid: wsForum.cmid.toString(),
                     forum_id: wsForum.id,
                     name: wsForum.name,
-                    url: `https://ilearning.cycu.edu.tw/mod/forum/view.php?id=${wsForum.cmid}`,
+                    timemodified: wsForum.timemodified,
                 });
             }
         }
@@ -137,7 +138,6 @@ function registerForumsCommand(program) {
         .description("List discussions in a forum (use forum ID)")
         .argument("<forum-id>", "Forum ID")
         .option("--output <format>", "Output format: json|csv|table|silent")
-        .option("--msg", "Include message content in response")
         .action(async (forumId, options, command) => {
         const apiContext = await createApiContext(options, command);
         if (!apiContext) {
@@ -168,20 +168,15 @@ function registerForumsCommand(program) {
             forum_name: targetForum.name,
             course_id: course?.id,
             course_name: course?.fullname,
-            discussions: discussions.map(d => {
-                const discussion = {
-                    id: d.id,
-                    name: d.name,
-                    user_id: d.userId,
-                    time_modified: d.timeModified,
-                    post_count: d.postCount,
-                    unread: d.unread,
-                };
-                if (options.msg) {
-                    discussion.message = (0, utils_js_1.stripHtmlTags)(d.message || "");
-                }
-                return discussion;
-            }),
+            discussions: discussions.map(d => ({
+                id: d.id,
+                name: d.name,
+                user_id: d.userId,
+                time_modified: d.timeModified,
+                post_count: d.postCount,
+                unread: d.unread,
+                message: ((0, utils_js_1.stripHtmlTags)(d.message || "")).substring(0, 250) + "...",
+            })),
             summary: {
                 total_discussions: discussions.length,
             },

package/script/src/commands/materials.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"materials.d.ts","sourceRoot":"","sources":["../../../src/src/commands/materials.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA+BpC,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAka/D"}
+{"version":3,"file":"materials.d.ts","sourceRoot":"","sources":["../../../src/src/commands/materials.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAgCpC,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAqlB/D"}

package/script/src/commands/materials.js

@@ -10,21 +10,17 @@ const logger_js_1 = require("../lib/logger.js");
 const auth_js_1 = require("../lib/auth.js");
 const session_js_1 = require("../lib/session.js");
 const auth_js_2 = require("../lib/auth.js");
+const index_js_1 = require("../index.js");
 const token_js_1 = require("../lib/token.js");
 const node_path_1 = __importDefault(require("node:path"));
 const node_fs_1 = __importDefault(require("node:fs"));
 function registerMaterialsCommand(program) {
     const materialsCmd = program.command("materials");
     materialsCmd.description("Material/resource operations");
-    // Helper to get output format from global or local options
-    function getOutputFormat(command) {
-        const opts = command.optsWithGlobals();
-        return opts.output || "json";
-    }
     // Pure API context - no browser required (fast!)
     async function createApiContext(options, command) {
         const opts = command?.optsWithGlobals ? command.optsWithGlobals() : options;
-        const outputFormat = getOutputFormat(command || { optsWithGlobals: () => ({ output: "json" }) });
+        const outputFormat = (0, utils_js_1.getOutputFormat)(command || { optsWithGlobals: () => ({ output: "json" }) });
         const silent = outputFormat === "json" && !opts.verbose;
         const log = (0, logger_js_1.createLogger)(opts.verbose, silent);
         const baseDir = (0, utils_js_1.getBaseDir)();
@@ -52,7 +48,7 @@ function registerMaterialsCommand(program) {
     // Helper function to create session context (for download commands)
     async function createSessionContext(options, command) {
         const opts = command?.optsWithGlobals ? command.optsWithGlobals() : options;
-        const outputFormat = getOutputFormat(command || { optsWithGlobals: () => ({ output: "json" }) });
+        const outputFormat = (0, utils_js_1.getOutputFormat)(command || { optsWithGlobals: () => ({ output: "json" }) });
         const silent = outputFormat === "json" && !opts.verbose;
         const log = (0, logger_js_1.createLogger)(opts.verbose, silent);
         const baseDir = (0, utils_js_1.getBaseDir)();
@@ -83,10 +79,6 @@ function registerMaterialsCommand(program) {
             throw err;
         }
     }
-    // Helper to sanitize filenames
-    function sanitizeFilename(name) {
-        return name.replace(/[<>:"/\\|?*]/g, "_").replace(/\s+/g, "_");
-    }
     // Helper to download a single resource
     async function downloadResource(page, resource, outputDir, log) {
         try {
@@ -96,7 +88,7 @@ function registerMaterialsCommand(program) {
                 return null;
             }
             // Create course directory
-            const courseDir = node_path_1.default.join(outputDir, sanitizeFilename(resource.course_name));
+            const courseDir = node_path_1.default.join(outputDir, (0, utils_js_1.sanitizeFilename)(resource.course_name));
             if (!node_fs_1.default.existsSync(courseDir)) {
                 node_fs_1.default.mkdirSync(courseDir, { recursive: true });
             }
@@ -114,7 +106,7 @@ function registerMaterialsCommand(program) {
             // Extract filename from URL or use resource name
             const urlObj = new URL(downloadUrl);
             const filenameParam = urlObj.searchParams.get("filename");
-            let filename = filenameParam || sanitizeFilename(resource.name);
+            let filename = filenameParam || (0, utils_js_1.sanitizeFilename)(resource.name);
             // Add extension if missing
             if (resource.mimetype && !node_path_1.default.extname(filename)) {
                 const extMap = {
@@ -365,4 +357,172 @@ function registerMaterialsCommand(program) {
             await (0, auth_js_2.closeBrowserSafely)(browser, browserContext);
         }
     });
+    materialsCmd
+        .command("complete")
+        .description("Mark all incomplete resources (non-video) as complete in a course")
+        .argument("<course-id>", "Course ID")
+        .option("--dry-run", "Show what would be marked complete without doing it")
+        .option("--output <format>", "Output format: json|csv|table|silent")
+        .action(async (courseId, options, command) => {
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        const apiContext = await createApiContext(options, command);
+        if (!apiContext) {
+            process.exitCode = 1;
+            return;
+        }
+        try {
+            const { log, session } = apiContext;
+            // Get user ID
+            const siteInfo = await (0, moodle_js_1.getSiteInfoApi)(session);
+            // Get completion status for all activities in the course
+            const completionData = await (0, moodle_js_1.moodleApiCall)(session, "core_completion_get_activities_completion_status", { courseid: parseInt(courseId, 10), userid: siteInfo.userid });
+            if (!completionData?.statuses) {
+                log.info("No activities found in this course.");
+                return;
+            }
+            // Filter for resources (non-video) that have completion enabled but are not complete
+            const incompleteResources = completionData.statuses.filter((status) => {
+                // Only resources, not supervideo
+                if (status.modname === "supervideo")
+                    return false;
+                // Must have completion enabled
+                if (!status.hascompletion)
+                    return false;
+                // Must be incomplete
+                if (status.isoverallcomplete)
+                    return false;
+                return true;
+            });
+            if (incompleteResources.length === 0) {
+                log.info("All resources are already complete (or no resources with completion tracking).");
+                return;
+            }
+            log.info(`Found ${incompleteResources.length} incomplete resources to complete:`);
+            for (const resource of incompleteResources) {
+                log.info(`  - ${resource.name} (cmid: ${resource.cmid})`);
+            }
+            if (options.dryRun) {
+                log.info("\n[Dry run] No changes made.");
+                return;
+            }
+            // Mark each resource as complete
+            const results = [];
+            for (const resource of incompleteResources) {
+                const success = await (0, moodle_js_1.updateActivityCompletionStatusManually)(session, resource.cmid, true);
+                results.push({
+                    cmid: resource.cmid,
+                    name: resource.name,
+                    success,
+                });
+                if (success) {
+                    log.success(`  ✓ Completed: ${resource.name}`);
+                }
+                else {
+                    log.error(`  ✗ Failed: ${resource.name}`);
+                }
+            }
+            const completed = results.filter(r => r.success).length;
+            const failed = results.filter(r => !r.success).length;
+            log.info(`\n執行結果: ${completed} 成功, ${failed} 失敗`);
+            if (output !== "silent") {
+                (0, index_js_1.formatAndOutput)(results, output, log);
+            }
+        }
+        catch (e) {
+            apiContext.log.error(`Error: ${e instanceof Error ? e.message : String(e)}`);
+            process.exitCode = 1;
+        }
+    });
+    materialsCmd
+        .command("complete-all")
+        .description("Mark all incomplete resources (non-video) as complete across all in-progress courses")
+        .option("--dry-run", "Show what would be marked complete without doing it")
+        .option("--level <type>", "Course level: in_progress (default) | all", "in_progress")
+        .option("--output <format>", "Output format: json|csv|table|silent")
+        .action(async (options, command) => {
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        const apiContext = await createApiContext(options, command);
+        if (!apiContext) {
+            process.exitCode = 1;
+            return;
+        }
+        try {
+            const { log, session } = apiContext;
+            // Get user ID
+            const siteInfo = await (0, moodle_js_1.getSiteInfoApi)(session);
+            // Get all courses
+            const classification = options.level === "all" ? undefined : "inprogress";
+            const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(session, { classification });
+            log.info(`Scanning ${courses.length} courses for incomplete resources...`);
+            const allResults = [];
+            let totalIncomplete = 0;
+            for (const course of courses) {
+                try {
+                    // Get completion status for all activities in the course
+                    const completionData = await (0, moodle_js_1.moodleApiCall)(session, "core_completion_get_activities_completion_status", { courseid: course.id, userid: siteInfo.userid });
+                    if (!completionData?.statuses)
+                        continue;
+                    // Filter for resources (non-video) that have completion enabled but are not complete
+                    const incompleteResources = completionData.statuses.filter((status) => {
+                        if (status.modname === "supervideo")
+                            return false;
+                        if (!status.hascompletion)
+                            return false;
+                        if (status.isoverallcomplete)
+                            return false;
+                        return true;
+                    });
+                    if (incompleteResources.length > 0) {
+                        log.info(`\n${course.fullname}: ${incompleteResources.length} incomplete resources`);
+                        totalIncomplete += incompleteResources.length;
+                        if (options.dryRun) {
+                            for (const resource of incompleteResources) {
+                                log.info(`  - ${resource.name} (cmid: ${resource.cmid})`);
+                            }
+                        }
+                        else {
+                            for (const resource of incompleteResources) {
+                                const success = await (0, moodle_js_1.updateActivityCompletionStatusManually)(session, resource.cmid, true);
+                                allResults.push({
+                                    courseId: course.id,
+                                    courseName: course.fullname,
+                                    cmid: resource.cmid,
+                                    name: resource.name,
+                                    success,
+                                });
+                                if (success) {
+                                    log.success(`  ✓ Completed: ${resource.name}`);
+                                }
+                                else {
+                                    log.error(`  ✗ Failed: ${resource.name}`);
+                                }
+                            }
+                        }
+                    }
+                }
+                catch (e) {
+                    log.warn(`Failed to process course ${course.fullname}: ${e}`);
+                }
+            }
+            if (totalIncomplete === 0) {
+                log.info("\nAll resources are already complete (or no resources with completion tracking).");
+                return;
+            }
+            if (options.dryRun) {
+                log.info(`\n[Dry run] Found ${totalIncomplete} incomplete resources across ${courses.length} courses.`);
+                log.info("Run without --dry-run to mark them as complete.");
+                return;
+            }
+            const completed = allResults.filter(r => r.success).length;
+            const failed = allResults.filter(r => !r.success).length;
+            log.info(`\n執行結果: ${completed} 成功, ${failed} 失敗`);
+            if (output !== "silent") {
+                (0, index_js_1.formatAndOutput)(allResults, output, log);
+            }
+        }
+        catch (e) {
+            apiContext.log.error(`Error: ${e instanceof Error ? e.message : String(e)}`);
+            process.exitCode = 1;
+        }
+    });
 }

package/script/src/commands/videos.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"videos.d.ts","sourceRoot":"","sources":["../../../src/src/commands/videos.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAYpC,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAuV5D"}
+{"version":3,"file":"videos.d.ts","sourceRoot":"","sources":["../../../src/src/commands/videos.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAYpC,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA+X5D"}

package/script/src/commands/videos.js

@@ -17,15 +17,11 @@ const node_fs_1 = __importDefault(require("node:fs"));
 function registerVideosCommand(program) {
     const videosCmd = program.command("videos");
     videosCmd.description("Video progress operations");
-    // Helper to get output format from global or local options
-    function getOutputFormat(command) {
-        const opts = command.optsWithGlobals();
-        return opts.output || "json";
-    }
     // Pure API context - no browser required (fast!)
     async function createApiContext(options, command) {
         const opts = command?.optsWithGlobals ? command.optsWithGlobals() : options;
-        const outputFormat = getOutputFormat(command || { optsWithGlobals: () => ({ output: "json" }) });
+        // Don't silence logs for commands that don't have explicit output format control
+        const outputFormat = command && command.optsWithGlobals ? (0, utils_js_1.getOutputFormat)(command) : "table";
         const silent = outputFormat === "json" && !opts.verbose;
         const log = (0, logger_js_1.createLogger)(opts.verbose, silent);
         const baseDir = (0, utils_js_1.getBaseDir)();
@@ -53,7 +49,7 @@ function registerVideosCommand(program) {
     // Helper function to create session context (for browser-only commands)
     async function createSessionContext(options, command) {
         const opts = command?.optsWithGlobals ? command.optsWithGlobals() : options;
-        const outputFormat = getOutputFormat(command || { optsWithGlobals: () => ({ output: "json" }) });
+        const outputFormat = (0, utils_js_1.getOutputFormat)(command || { optsWithGlobals: () => ({ output: "json" }) });
         const silent = outputFormat === "json" && !opts.verbose;
         const log = (0, logger_js_1.createLogger)(opts.verbose, silent);
         const baseDir = (0, utils_js_1.getBaseDir)();
@@ -91,52 +87,67 @@ function registerVideosCommand(program) {
         .option("--incomplete-only", "Show only incomplete videos")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (courseId, options, command) => {
-        const output = getOutputFormat(command);
+        const output = (0, utils_js_1.getOutputFormat)(command);
         const apiContext = await createApiContext(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
-        const videos = await (0, moodle_js_1.getSupervideosInCourseApi)(apiContext.session, parseInt(courseId, 10));
-        // Note: API doesn't provide completion status, so --incomplete-only shows all
+        let videos = await (0, moodle_js_1.getSupervideosInCourseApi)(apiContext.session, parseInt(courseId, 10));
+        // Filter for incomplete videos if requested
         if (options.incompleteOnly) {
-            apiContext.log.warn("--incomplete-only is not supported in API mode, showing all videos");
+            videos = videos.filter(v => !v.isComplete);
         }
         (0, index_js_1.formatAndOutput)(videos, output, apiContext.log);
     });
     videosCmd
         .command("complete")
-        .description("Complete videos in a course (requires browser)")
+        .description("Complete videos in a course (uses API for list & completion, browser for metadata)")
         .argument("<course-id>", "Course ID")
         .option("--dry-run", "Discover videos but don't complete them")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (courseId, options, command) => {
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        // Get API context for getting incomplete videos and completion
+        const apiContext = await createApiContext(options, command);
+        if (!apiContext) {
+            process.exitCode = 1;
+            return;
+        }
+        // Get incomplete videos via API (fast, no browser needed)
+        const incompleteVideos = await (0, moodle_js_1.getIncompleteVideosApi)(apiContext.session, parseInt(courseId, 10));
+        if (incompleteVideos.length === 0) {
+            apiContext.log.info("所有影片已完成（或無影片）。");
+            return;
+        }
+        apiContext.log.info(`找到 ${incompleteVideos.length} 部未完成影片`);
+        // Dry-run: show videos without needing browser
+        if (options.dryRun) {
+            const results = incompleteVideos.map(v => ({ name: v.name, success: true }));
+            for (const video of incompleteVideos) {
+                apiContext.log.info(`  [試執行] ${video.name}`);
+            }
+            apiContext.log.info(`\n執行結果: ${results.length} 影片將被完成`);
+            if (output !== "silent") {
+                (0, index_js_1.formatAndOutput)(results, output, apiContext.log);
+            }
+            return;
+        }
+        // Need browser only for getting viewId and duration (not needed for dry-run)
         const context = await createSessionContext(options, command);
         if (!context) {
             process.exitCode = 1;
             return;
         }
-        const { log, page, session, browser, context: browserContext } = context;
-        const output = getOutputFormat(command);
+        const { log, page, browser, context: browserContext } = context;
         try {
-            const videos = await (0, moodle_js_1.getSupervideosInCourse)(page, session, parseInt(courseId, 10), log, {
-                incompleteOnly: true, // Only operate on incomplete videos
-            });
-            if (videos.length === 0) {
-                log.info("所有影片已完成（或無影片）。");
-                return;
-            }
             const results = [];
-            for (const sv of videos) {
+            for (const sv of incompleteVideos) {
                 log.info(`處理中: ${sv.name}`);
                 try {
                     const video = await (0, moodle_js_1.getVideoMetadata)(page, sv.url, log);
-                    if (options.dryRun) {
-                        log.info(`  [試執行] viewId=${video.viewId}, duration=${video.duration}s`);
-                        results.push({ name: sv.name, success: true });
-                        continue;
-                    }
-                    const success = await (0, moodle_js_1.completeVideo)(page, session, { ...video, cmid: sv.cmid }, log);
+                    // Use WS API for completion
+                    const success = await (0, moodle_js_1.completeVideoApi)(apiContext.session, { ...video, cmid: sv.cmid.toString() });
                     if (success) {
                         log.success(`  已完成！`);
                         results.push({ name: sv.name, success: true });
@@ -165,65 +176,94 @@ function registerVideosCommand(program) {
     });
     videosCmd
         .command("complete-all")
-        .description("Complete all incomplete videos across all courses (requires browser)")
+        .description("Complete all incomplete videos across all courses (uses API for list & completion, browser for metadata)")
         .option("--dry-run", "Discover videos but don't complete them")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (options, command) => {
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        // Get API context for getting incomplete videos and completion
+        const apiContext = await createApiContext(options, command);
+        if (!apiContext) {
+            process.exitCode = 1;
+            return;
+        }
+        // Get all courses via API
+        const classification = undefined; // all courses
+        const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(apiContext.session, { classification });
+        apiContext.log.info(`掃描 ${courses.length} 個課程...`);
+        // Collect all incomplete videos across all courses using flatMap for cleaner code
+        const allIncompleteVideos = (await Promise.allSettled(courses.map(async (course) => {
+            try {
+                const videos = await (0, moodle_js_1.getIncompleteVideosApi)(apiContext.session, course.id);
+                return videos.map((video) => ({
+                    courseId: course.id,
+                    courseName: course.fullname,
+                    cmid: video.cmid,
+                    name: video.name,
+                    url: video.url,
+                }));
+            }
+            catch (e) {
+                apiContext.log.warn(`無法取得課程 ${course.fullname} 的影片: ${e}`);
+                return [];
+            }
+        })))
+            .filter((result) => result.status === "fulfilled")
+            .flatMap((result) => result.status === "fulfilled" ? result.value : []);
+        if (allIncompleteVideos.length === 0) {
+            apiContext.log.info("所有影片已完成（或無影片）。");
+            return;
+        }
+        apiContext.log.info(`找到 ${allIncompleteVideos.length} 部未完成影片`);
+        // Dry-run: show videos without needing browser
+        if (options.dryRun) {
+            for (const video of allIncompleteVideos) {
+                apiContext.log.info(`  [試執行] [${video.courseName}] ${video.name}`);
+            }
+            apiContext.log.info("\n===== 執行結果 =====");
+            apiContext.log.info(`掃描課程數: ${courses.length}`);
+            apiContext.log.info(`找到未完成影片: ${allIncompleteVideos.length}`);
+            apiContext.log.info(`執行影片數: ${allIncompleteVideos.length} (試執行)`);
+            return;
+        }
+        // Need browser only for getting viewId and duration (not needed for dry-run)
         const context = await createSessionContext(options, command);
         if (!context) {
             process.exitCode = 1;
             return;
         }
-        const { log, page, session, browser, context: browserContext } = context;
-        const output = getOutputFormat(command);
+        const { log, page, browser, context: browserContext } = context;
         try {
-            const courses = await (0, moodle_js_1.getEnrolledCourses)(page, session, log);
             const allResults = [];
-            let totalVideos = 0;
             let totalCompleted = 0;
             let totalFailed = 0;
-            for (const course of courses) {
-                log.info(`\n======================================`);
-                log.info(`課程: ${course.fullname}`);
-                log.info(`======================================`);
-                const videos = await (0, moodle_js_1.getSupervideosInCourse)(page, session, course.id, log);
-                if (videos.length === 0) {
-                    log.info("  所有影片已完成（或無影片）。");
-                    continue;
-                }
-                totalVideos += videos.length;
-                for (const sv of videos) {
-                    log.info(`  處理中: ${sv.name}`);
-                    try {
-                        const video = await (0, moodle_js_1.getVideoMetadata)(page, sv.url, log);
-                        if (options.dryRun) {
-                            log.info(`    [試執行] viewId=${video.viewId}, duration=${video.duration}s`);
-                            allResults.push({ courseName: course.fullname, name: sv.name, success: true });
-                            continue;
-                        }
-                        const success = await (0, moodle_js_1.completeVideo)(page, session, { ...video, cmid: sv.cmid }, log);
-                        if (success) {
-                            log.success(`    已完成！`);
-                            allResults.push({ courseName: course.fullname, name: sv.name, success: true });
-                            totalCompleted++;
-                        }
-                        else {
-                            log.error(`    失敗。`);
-                            allResults.push({ courseName: course.fullname, name: sv.name, success: false, error: "Failed to complete" });
-                            totalFailed++;
-                        }
+            for (const video of allIncompleteVideos) {
+                log.info(`處理中: [${video.courseName}] ${video.name}`);
+                try {
+                    const metadata = await (0, moodle_js_1.getVideoMetadata)(page, video.url, log);
+                    // Use WS API for completion
+                    const success = await (0, moodle_js_1.completeVideoApi)(apiContext.session, { ...metadata, cmid: video.cmid.toString() });
+                    if (success) {
+                        log.success(`  已完成！`);
+                        allResults.push({ courseName: video.courseName, name: video.name, success: true });
+                        totalCompleted++;
                     }
-                    catch (err) {
-                        const msg = err instanceof Error ? err.message : String(err);
-                        log.error(`    錯誤: ${msg}`);
-                        allResults.push({ courseName: course.fullname, name: sv.name, success: false, error: msg });
+                    else {
+                        log.error(`  失敗。`);
+                        allResults.push({ courseName: video.courseName, name: video.name, success: false, error: "Failed to complete" });
                         totalFailed++;
                     }
                 }
+                catch (err) {
+                    const msg = err instanceof Error ? err.message : String(err);
+                    log.error(`  錯誤: ${msg}`);
+                    allResults.push({ courseName: video.courseName, name: video.name, success: false, error: msg });
+                    totalFailed++;
+                }
             }
             log.info("\n===== 執行結果 =====");
             log.info(`掃描課程數: ${courses.length}`);
-            log.info(`掃描影片數: ${totalVideos}`);
+            log.info(`找到未完成影片: ${allIncompleteVideos.length}`);
             log.info(`執行影片數: ${totalCompleted}`);
             if (totalFailed > 0)
                 log.warn(`失敗影片數: ${totalFailed}`);
@@ -235,13 +275,6 @@ function registerVideosCommand(program) {
             await (0, auth_js_2.closeBrowserSafely)(browser, browserContext);
         }
     });
-    // Helper function to sanitize filename
-    function sanitizeFilename(name) {
-        return name
-            .replace(/[<>:"/\\|?*]/g, "_")
-            .replace(/\s+/g, "_")
-            .substring(0, 200);
-    }
     videosCmd
         .command("download")
         .description("Download videos from a course (requires browser)")
@@ -265,7 +298,7 @@ function registerVideosCommand(program) {
             node_fs_1.default.mkdirSync(outputDir, { recursive: true });
             const downloaded = [];
             for (const video of videos) {
-                const filename = sanitizeFilename(video.name) + ".mp4";
+                const filename = (0, utils_js_1.sanitizeFilename)(video.name) + ".mp4";
                 const outputPath = node_path_1.default.join(outputDir, filename);
                 log.info(`處理中: ${video.name}`);
                 try {

package/script/src/index.js

@@ -41,7 +41,7 @@ program
     .option("--silent", "Suppress all log output (JSON only)")
     .option("--headed", "Run browser in visible mode");
 // Register subcommands
-(0, auth_js_2.registerAuthCommand)(program);
+(0, auth_js_2.registerCommand)(program);
 (0, courses_js_1.registerCoursesCommand)(program);
 (0, videos_js_1.registerVideosCommand)(program);
 (0, quizzes_js_1.registerQuizzesCommand)(program);