@mo7yw4ng/openape 1.0.4 → 1.0.6

package/script/src/commands/forums.js

@@ -1,68 +1,28 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.registerForumsCommand = registerForumsCommand;
 const utils_js_1 = require("../lib/utils.js");
 const moodle_js_1 = require("../lib/moodle.js");
-const logger_js_1 = require("../lib/logger.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"));
+const auth_js_1 = require("../lib/auth.js");
+const index_js_1 = require("../index.js");
 function registerForumsCommand(program) {
     const forumsCmd = program.command("forums");
     forumsCmd.description("Forum operations");
-    // Pure API context - no browser required (fast!)
-    async function createApiContext(options, command) {
-        const opts = command?.optsWithGlobals ? command.optsWithGlobals() : options;
-        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)();
-        const sessionPath = node_path_1.default.resolve(baseDir, ".auth", "storage-state.json");
-        // Check if session exists
-        if (!node_fs_1.default.existsSync(sessionPath)) {
-            log.error("未找到登入 session。請先執行 'openape auth login' 進行登入。");
-            log.info(`Session 預期位置: ${sessionPath}`);
-            return null;
-        }
-        // Try to load WS token
-        const wsToken = (0, token_js_1.loadWsToken)(sessionPath);
-        if (!wsToken) {
-            log.error("未找到 WS token。請先執行 'openape auth login' 進行登入。");
-            return null;
-        }
-        // Try to load sesskey from cache
-        const sesskey = (0, token_js_1.loadSesskey)(sessionPath) || undefined;
-        return {
-            log,
-            session: {
-                wsToken,
-                moodleBaseUrl: "https://ilearning.cycu.edu.tw",
-                sesskey,
-            },
-        };
-    }
-    forumsCmd
-        .command("list")
-        .description("List forums from in-progress courses")
-        .option("--output <format>", "Output format: json|csv|table|silent")
-        .action(async (options, command) => {
-        const apiContext = await createApiContext(options, command);
+    async function listForums(classification) {
+        const apiContext = await (0, auth_js_1.createApiContext)({});
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
         const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(apiContext.session, {
-            classification: "inprogress",
+            classification,
         });
-        // Get forums via WS API (no browser needed!)
         const courseIds = courses.map(c => c.id);
         const wsForums = await (0, moodle_js_1.getForumsApi)(apiContext.session, courseIds);
+        const courseMap = new Map(courses.map(c => [c.id, c]));
         const allForums = [];
         for (const wsForum of wsForums) {
-            const course = courses.find(c => c.id === wsForum.courseid);
+            const course = courseMap.get(wsForum.courseid);
             if (course) {
                 allForums.push({
                     course_id: wsForum.courseid,
@@ -72,62 +32,22 @@ function registerForumsCommand(program) {
                     forum_id: wsForum.id,
                     name: wsForum.name,
                     timemodified: wsForum.timemodified,
-                    // url: `https://ilearning.cycu.edu.tw/mod/forum/view.php?id=${wsForum.cmid}`,
                 });
             }
         }
-        console.log(JSON.stringify({
-            status: "success",
-            timestamp: new Date().toISOString(),
-            total_courses: courses.length,
-            total_forums: allForums.length,
-        }));
-        for (const forum of allForums) {
-            console.log(JSON.stringify(forum));
-        }
-    });
+        (0, index_js_1.formatAndOutput)(allForums, "json", apiContext.log, { status: "success", timestamp: new Date().toISOString(), total_courses: courses.length, total_forums: allForums.length });
+    }
+    forumsCmd
+        .command("list")
+        .description("List forums from in-progress courses")
+        .action(() => listForums("inprogress"));
     forumsCmd
         .command("list-all")
         .description("List all forums across all courses")
         .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 apiContext = await createApiContext(options, command);
-        if (!apiContext) {
-            process.exitCode = 1;
-            return;
-        }
+        .action(async (options) => {
         const classification = options.level === "all" ? undefined : "inprogress";
-        const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(apiContext.session, {
-            classification,
-        });
-        // Get forums via WS API (no browser needed!)
-        const courseIds = courses.map(c => c.id);
-        const wsForums = await (0, moodle_js_1.getForumsApi)(apiContext.session, courseIds);
-        const allForums = [];
-        for (const wsForum of wsForums) {
-            const course = courses.find(c => c.id === wsForum.courseid);
-            if (course) {
-                allForums.push({
-                    course_id: wsForum.courseid,
-                    course_name: course.fullname,
-                    intro: wsForum.intro,
-                    cmid: wsForum.cmid.toString(),
-                    forum_id: wsForum.id,
-                    name: wsForum.name,
-                    timemodified: wsForum.timemodified,
-                });
-            }
-        }
-        console.log(JSON.stringify({
-            status: "success",
-            timestamp: new Date().toISOString(),
-            total_courses: courses.length,
-            total_forums: allForums.length,
-        }));
-        for (const forum of allForums) {
-            console.log(JSON.stringify(forum));
-        }
+        await listForums(classification);
     });
     forumsCmd
         .command("discussions")
@@ -135,51 +55,29 @@ function registerForumsCommand(program) {
         .argument("<forum-id>", "Forum ID")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (forumId, options, command) => {
-        const apiContext = await createApiContext(options, command);
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
-        // Get courses via WS API
-        const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(apiContext.session, {
-            classification: "inprogress",
-        });
-        // Get forums via WS API
-        const courseIds = courses.map(c => c.id);
-        const wsForums = await (0, moodle_js_1.getForumsApi)(apiContext.session, courseIds);
-        // Find forum by cmid or instance ID
-        const targetForum = wsForums.find(f => f.cmid.toString() === forumId || f.id === parseInt(forumId, 10));
-        if (!targetForum) {
-            console.log(JSON.stringify({ status: "error", error: "Forum not found" }));
+        const resolved = await (0, moodle_js_1.resolveForumId)(apiContext.session, forumId);
+        if (!resolved) {
+            apiContext.log.error("Forum not found");
             process.exitCode = 1;
             return;
         }
-        const course = courses.find(c => c.id === targetForum.courseid);
-        // Get discussions via WS API
-        const discussions = await (0, moodle_js_1.getForumDiscussionsApi)(apiContext.session, targetForum.id);
-        // Output NDJSON: one line per discussion entry for stream-friendly parsing
-        const meta = {
-            status: "success",
-            timestamp: new Date().toISOString(),
-            forum_id: targetForum.id,
-            forum_name: targetForum.name,
-            forum_intro: targetForum.intro,
-            course_id: course?.id,
-            course_name: course?.fullname,
-            total_discussions: discussions.length,
-        };
-        console.log(JSON.stringify(meta));
-        for (const d of discussions) {
-            console.log(JSON.stringify({
-                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 || ""),
-            }));
-        }
+        const discussions = await (0, moodle_js_1.getForumDiscussionsApi)(apiContext.session, resolved.forumId);
+        const items = 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 || ""),
+        }));
+        (0, index_js_1.formatAndOutput)(items, output, apiContext.log, { status: "success", timestamp: new Date().toISOString(), forum_id: resolved.forumId, forum_name: resolved.name ?? null, course_id: resolved.courseid ?? null, total_discussions: discussions.length });
     });
     forumsCmd
         .command("posts")
@@ -188,44 +86,23 @@ function registerForumsCommand(program) {
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (discussionId, options, command) => {
         const output = (0, utils_js_1.getOutputFormat)(command);
-        const apiContext = await createApiContext(options, command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
         const posts = await (0, moodle_js_1.getDiscussionPostsApi)(apiContext.session, parseInt(discussionId, 10));
-        if (output === "json") {
-            const result = {
-                status: "success",
-                timestamp: new Date().toISOString(),
-                discussion_id: discussionId,
-                posts: posts.map(p => ({
-                    id: p.id,
-                    subject: p.subject,
-                    author: p.author,
-                    author_id: p.authorId,
-                    created: (0, utils_js_1.formatTimestamp)(p.created),
-                    modified: (0, utils_js_1.formatTimestamp)(p.modified),
-                    message: p.message,
-                    unread: p.unread,
-                })),
-                summary: {
-                    total_posts: posts.length,
-                },
-            };
-            console.log(JSON.stringify(result));
-        }
-        else if (output === "table") {
-            console.log(`Discussion ${discussionId} - ${posts.length} posts`);
-            console.log("Use --output json to see full post content");
-            const tablePosts = posts.map(p => ({
-                id: p.id,
-                subject: p.subject.substring(0, 50) + (p.subject.length > 50 ? "..." : ""),
-                author: p.author,
-                created: new Date(p.created * 1000).toLocaleString(),
-            }));
-            console.table(tablePosts);
-        }
+        const items = posts.map(p => ({
+            id: p.id,
+            subject: p.subject,
+            author: p.author,
+            author_id: p.authorId,
+            created: (0, utils_js_1.formatTimestamp)(p.created),
+            modified: (0, utils_js_1.formatTimestamp)(p.modified),
+            message: p.message,
+            unread: p.unread,
+        }));
+        (0, index_js_1.formatAndOutput)(items, output, apiContext.log, { status: "success", timestamp: new Date().toISOString(), discussion_id: discussionId, total_posts: posts.length });
     });
     forumsCmd
         .command("post")
@@ -236,28 +113,20 @@ function registerForumsCommand(program) {
         .option("--subscribe", "Subscribe to the discussion", false)
         .option("--pin", "Pin the discussion", false)
         .action(async (forumId, subject, message, options, command) => {
-        const apiContext = await createApiContext(options, command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
         const { log, session } = apiContext;
-        // Get courses to find the forum
-        const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(session, {
-            classification: "inprogress",
-        });
-        const courseIds = courses.map(c => c.id);
-        const wsForums = await (0, moodle_js_1.getForumsApi)(session, courseIds);
-        // Find forum by cmid or instance ID
-        const targetForum = wsForums.find(f => f.cmid.toString() === forumId || f.id === parseInt(forumId, 10));
-        if (!targetForum) {
+        const resolved = await (0, moodle_js_1.resolveForumId)(session, forumId);
+        if (!resolved) {
             log.error(`Forum not found: ${forumId}`);
             process.exitCode = 1;
             return;
         }
-        const course = courses.find(c => c.id === targetForum.courseid);
-        log.info(`Posting to forum: ${targetForum.name} (${course?.fullname})`);
-        const result = await (0, moodle_js_1.addForumDiscussionApi)(session, targetForum.id, subject, message);
+        log.info(`Posting to forum: ${resolved.name ?? forumId}`);
+        const result = await (0, moodle_js_1.addForumDiscussionApi)(session, resolved.forumId, subject, message);
         if (result.success) {
             log.success(`✓ Discussion posted successfully!`);
             log.info(`  Discussion ID: ${result.discussionId}`);
@@ -276,7 +145,7 @@ function registerForumsCommand(program) {
         .option("--attachment-id <id>", "Draft file ID for attachment")
         .option("--inline-attachment-id <id>", "Draft file ID for inline attachment")
         .action(async (postId, subject, message, options, command) => {
-        const apiContext = await createApiContext(options, command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
@@ -306,7 +175,7 @@ function registerForumsCommand(program) {
         .description("Delete a forum post or discussion (by post ID)")
         .argument("<post-id>", "Post ID to delete (deletes entire discussion if it's the first post)")
         .action(async (postId, options, command) => {
-        const apiContext = await createApiContext(options, command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;

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

@@ -1 +1 @@
-{"version":3,"file":"grades.d.ts","sourceRoot":"","sources":["../../../src/src/commands/grades.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAkBpC,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAsI5D"}
+{"version":3,"file":"grades.d.ts","sourceRoot":"","sources":["../../../src/src/commands/grades.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAgBpC,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA+F5D"}

package/script/src/commands/grades.js

@@ -1,66 +1,32 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.registerGradesCommand = registerGradesCommand;
-const utils_js_1 = require("../lib/utils.js");
 const moodle_js_1 = require("../lib/moodle.js");
-const logger_js_1 = require("../lib/logger.js");
-const token_js_1 = require("../lib/token.js");
+const auth_js_1 = require("../lib/auth.js");
 const index_js_1 = require("../index.js");
-const node_path_1 = __importDefault(require("node:path"));
-const node_fs_1 = __importDefault(require("node:fs"));
+const utils_js_1 = require("../lib/utils.js");
 function registerGradesCommand(program) {
     const gradesCmd = program.command("grades");
     gradesCmd.description("Grade operations");
-    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 silent = outputFormat === "json" && !opts.verbose;
-        const log = (0, logger_js_1.createLogger)(opts.verbose, silent);
-        const baseDir = (0, utils_js_1.getBaseDir)();
-        const sessionPath = node_path_1.default.resolve(baseDir, ".auth", "storage-state.json");
-        // Check if session exists
-        if (!node_fs_1.default.existsSync(sessionPath)) {
-            log.error("未找到登入 session。請先執行 'openape auth login' 進行登入。");
-            log.info(`Session 預期位置: ${sessionPath}`);
-            return null;
-        }
-        // Try to load WS token
-        const wsToken = (0, token_js_1.loadWsToken)(sessionPath);
-        if (!wsToken) {
-            log.error("未找到 WS token。請先執行 'openape auth login' 進行登入。");
-            return null;
-        }
-        return {
-            log,
-            session: {
-                wsToken,
-                moodleBaseUrl: "https://ilearning.cycu.edu.tw",
-            },
-        };
-    }
     gradesCmd
         .command("summary")
         .description("Show grade summary across all courses")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (options, command) => {
-        const output = getOutputFormat(command);
-        const apiContext = await createApiContext(options, command);
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;
         }
         const courses = await (0, moodle_js_1.getEnrolledCoursesApi)(apiContext.session);
+        const gradeResults = await Promise.allSettled(courses.map(course => (0, moodle_js_1.getCourseGradesApi)(apiContext.session, course.id)
+            .then(grades => ({ course, grades }))));
         const gradeSummaries = [];
-        for (const course of courses) {
-            const grades = await (0, moodle_js_1.getCourseGradesApi)(apiContext.session, course.id);
+        for (const result of gradeResults) {
+            if (result.status !== "fulfilled")
+                continue;
+            const { course, grades } = result.value;
             gradeSummaries.push({
                 courseId: course.id,
                 courseName: course.fullname,
@@ -70,19 +36,13 @@ function registerGradesCommand(program) {
                 totalUsers: grades.totalUsers,
             });
         }
-        // Calculate overall statistics
         const gradedCourses = gradeSummaries.filter(g => g.grade !== undefined && g.grade !== null && g.grade !== "-");
         const averageRank = gradeSummaries
             .filter(g => g.rank !== undefined && g.rank !== null)
             .reduce((sum, g) => sum + (g.rank || 0), 0) /
             (gradeSummaries.filter(g => g.rank !== undefined && g.rank !== null).length || 1);
-        const summaryData = {
-            total_courses: courses.length,
-            graded_courses: gradedCourses.length,
-            average_rank: averageRank.toFixed(1),
-            grades: gradeSummaries,
-        };
-        (0, index_js_1.formatAndOutput)(summaryData, output, apiContext.log);
+        apiContext.log.info(`Total: ${courses.length} courses, ${gradedCourses.length} graded, avg rank: ${averageRank.toFixed(1)}`);
+        (0, index_js_1.formatAndOutput)(gradeSummaries, output, apiContext.log);
     });
     gradesCmd
         .command("course")
@@ -90,8 +50,8 @@ function registerGradesCommand(program) {
         .argument("<course-id>", "Course ID")
         .option("--output <format>", "Output format: json|csv|table|silent")
         .action(async (courseId, options, command) => {
-        const output = getOutputFormat(command);
-        const apiContext = await createApiContext(options, command);
+        const output = (0, utils_js_1.getOutputFormat)(command);
+        const apiContext = await (0, auth_js_1.createApiContext)(options, command);
         if (!apiContext) {
             process.exitCode = 1;
             return;

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;AAgCpC,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA8iB/D"}
+{"version":3,"file":"materials.d.ts","sourceRoot":"","sources":["../../../src/src/commands/materials.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA4BpC,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAwd/D"}