blackboard-upc 1.0.7 → 1.0.9

package/CHANGELOG.md

@@ -4,6 +4,29 @@ All notable changes to `blackboard-upc` will be documented here.
 
 ---
 
+## [1.0.9] — 2026-04-24
+
+### Fixed
+- **`whoami` y `status` mostraban `unknown`** — el extractor leía `userData.userName`, campo que la API de UPC no devuelve. Ahora se arma desde `name.given + name.family` con fallback a `studentId`
+- **`logout` no permitía cambiar de cuenta** — solo borraba `session.json` pero no el browser profile de Playwright (`~/.blackboard-cli/browser-profile`), donde viven las cookies de Microsoft SSO. El siguiente `login` se auto-autenticaba con la misma cuenta. Ahora `logout` borra también el profile
+
+### Added
+- `blackboard logout --keep-profile` — conserva las cookies SSO (útil para renovar sesión de la misma cuenta sin re-ingresar credenciales)
+- **Self-heal de sesiones viejas** — `whoami` y `status` detectan `userName: null` en sesiones guardadas con versiones previas y rellenan el nombre llamando a `/users/me` una sola vez (sin necesidad de re-login)
+- `resolveDisplayName()` helper en `src/auth/login.ts` — centraliza la lógica de nombre desde la respuesta de `/users/me`
+
+---
+
+## [1.0.8] — 2026-04-19
+
+### Added
+- **Soporte para preguntas `fimb`** (fill-in-multiple-blanks) en `get_quiz_questions`, `save_quiz_answer` y el tipo `QuizQuestion`:
+  - `QuizQuestion.blanks` — array con los nombres de los blanks (ej. `["BLANK-1", "BLANK-2"]`)
+  - `QuizQuestion.currentAnswer` — para fimb, devuelve `Record<string, string|null>` con el valor actual de cada blank
+  - `save_quiz_answer` ahora acepta un JSON string con el mapa `{blankName: value}` (ej. `'{"BLANK-1":"1438.62","BLANK-2":"140.62"}'`)
+
+---
+
 ## [1.0.7] — 2026-04-12
 
 ### Added

package/CLAUDE.md

@@ -33,6 +33,14 @@ If you get `Not authenticated`, ask the user to run `blackboard login`.
 4. submit_quiz (confirm first!)          → finalize and submit the attempt
 ```
 
+Supported question types in `save_quiz_answer`:
+
+| `question.type`   | `answer` format                                                                |
+|-------------------|--------------------------------------------------------------------------------|
+| `eitherOr`        | boolean (`true` = Verdadero, `false` = Falso)                                  |
+| `multipleanswer`  | number — 0-based index of the chosen option                                    |
+| `fimb`            | JSON string `'{"BLANK-1":"value1","BLANK-2":"value2"}'` — read names from `question.blanks` |
+
 ### Feedback workflow
 
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "blackboard-upc",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "CLI no oficial para UPC Aula Virtual (Blackboard Learn) — acceso desde la terminal y MCP para Claude",
   "main": "run.js",
   "bin": {

package/src/api/quiz.ts

@@ -25,7 +25,7 @@ export interface QuizOption {
   index: number;
 }
 
-export type QuizQuestionType = 'eitherOr' | 'multipleanswer' | 'presentation' | string;
+export type QuizQuestionType = 'eitherOr' | 'multipleanswer' | 'fimb' | 'presentation' | string;
 
 export interface QuizQuestion {
   /** Question attempt ID — used as the URL segment for saving answers */
@@ -34,18 +34,21 @@ export interface QuizQuestion {
   questionId: string;
   /** Position within the quiz (1-based visible number) */
   position: number;
-  /** 'eitherOr' = true/false, 'multipleanswer' = MC, 'presentation' = text only */
+  /** 'eitherOr' = true/false, 'multipleanswer' = MC, 'fimb' = fill-in-multiple-blanks, 'presentation' = text only */
   type: QuizQuestionType;
   /** Plain text of the question (HTML stripped) */
   text: string;
   points: number;
   /** Answer options (present for eitherOr and multipleanswer) */
   options?: QuizOption[];
+  /** Blank names in order (present for fimb), e.g. ['BLANK-1', 'BLANK-2'] */
+  blanks?: string[];
   /** Currently saved answer:
    *   - eitherOr:       true | false | null
    *   - multipleanswer: boolean[] (one per option, in options order)
+   *   - fimb:           Record<string, string|null> (one per blank)
    */
-  currentAnswer?: boolean | boolean[] | null;
+  currentAnswer?: boolean | boolean[] | Record<string, string | null> | null;
   /** Raw question object from API (needed when saving eitherOr answers) */
   _raw: any;
 }
@@ -76,6 +79,8 @@ function parseQuestionAttempt(qa: any, idx: number): QuizQuestion {
   const text = stripHtml(q.questionText?.rawText || q.questionText?.displayText || '');
 
   let options: QuizOption[] | undefined;
+  let blanks: string[] | undefined;
+  let currentAnswer: QuizQuestion['currentAnswer'] = qa.givenAnswer ?? null;
 
   if (qa.questionType === 'eitherOr') {
     // True/False — always two options
@@ -89,6 +94,10 @@ function parseQuestionAttempt(qa: any, idx: number): QuizQuestion {
       text: stripHtml(a.answerText?.rawText || a.answerText?.displayText || ''),
       index: i,
     }));
+  } else if (qa.questionType === 'fimb' && qa.givenAnswers && typeof qa.givenAnswers === 'object') {
+    // Fill-in-multiple-blanks — blank names come from givenAnswers keys (e.g. BLANK-1, BLANK-2)
+    blanks = Object.keys(qa.givenAnswers);
+    currentAnswer = qa.givenAnswers as Record<string, string | null>;
   }
 
   return {
@@ -99,7 +108,8 @@ function parseQuestionAttempt(qa: any, idx: number): QuizQuestion {
     text,
     points: q.points ?? 0,
     options,
-    currentAnswer: qa.givenAnswer ?? null,
+    blanks,
+    currentAnswer,
     _raw: qa,
   };
 }
@@ -165,13 +175,14 @@ export async function getQuizQuestions(
  *   - eitherOr:       boolean  (true = Verdadero, false = Falso)
  *   - multipleanswer: number   (0-based index of the selected option)
  *                    OR boolean[] (one per option)
+ *   - fimb:           Record<string, string>  (one value per blank name, e.g. { "BLANK-1": "1438.62", "BLANK-2": "140.62" })
  */
 export async function saveQuizAnswer(
   client: AxiosInstance,
   courseId: string,
   attemptId: string,
   question: QuizQuestion,
-  answer: boolean | number | boolean[]
+  answer: boolean | number | boolean[] | Record<string, string>
 ): Promise<any> {
   const url = `/learn/api/v1/courses/${courseId}/gradebook/attempts/${attemptId}/assessment/answers/${question.questionAttemptId}`;
 
@@ -203,6 +214,28 @@ export async function saveQuizAnswer(
       order: question._raw.order || [],
       question: question._raw.question,
     };
+  } else if (question.type === 'fimb') {
+    if (typeof answer !== 'object' || Array.isArray(answer) || answer === null) {
+      throw new Error(
+        `fimb answers must be a Record<string, string> mapping blank name to value. ` +
+        `Expected blanks: ${question.blanks?.join(', ') || '(unknown)'}`
+      );
+    }
+
+    // Build givenAnswers using the question's known blank names — preserves order and
+    // ensures any blank not provided ends up as null (consistent with Blackboard behavior).
+    const blanks = question.blanks ?? Object.keys(answer);
+    const givenAnswers: Record<string, string | null> = {};
+    for (const name of blanks) {
+      const val = (answer as Record<string, string>)[name];
+      givenAnswers[name] = val !== undefined ? String(val) : null;
+    }
+
+    body = {
+      questionType: 'fimb',
+      givenAnswers,
+      question: question._raw.question,
+    };
   } else {
     throw new Error(`Unsupported question type: ${question.type}`);
   }

package/src/auth/login.ts

@@ -43,6 +43,20 @@ function extractBbRouterExpiry(cookies: Cookie[]): number {
   return Date.now() + SESSION_TTL_FALLBACK_MS;
 }
 
+// UPC's /users/me omits `userName` at the root. The display name must be
+// assembled from `name.given` + `name.family`, falling back to `studentId`.
+export function resolveDisplayName(userData: any): string | undefined {
+  if (!userData) return undefined;
+  if (typeof userData.userName === 'string' && userData.userName.trim()) return userData.userName.trim();
+  const full = [userData?.name?.given, userData?.name?.family].filter(Boolean).join(' ').trim();
+  if (full) return full;
+  if (userData?.name?.preferredDisplayName && typeof userData.name.preferredDisplayName === 'string') {
+    const pdn = userData.name.preferredDisplayName.trim();
+    if (pdn && pdn !== 'GivenName' && pdn !== 'FamilyName') return pdn;
+  }
+  return userData.studentId || undefined;
+}
+
 function extractXsrf(cookies: Cookie[]): string {
   const bb = cookies.find(c => c.name === 'BbRouter');
   if (bb) {
@@ -158,16 +172,18 @@ export async function login(opts: LoginOptions = {}): Promise<Session> {
       if (resp.ok()) userData = await resp.json();
     } catch {}
 
+    const displayName = resolveDisplayName(userData);
+
     const session: Session = {
       cookies,
       xsrfToken: nonce,
       userId: userData?.id,
-      userName: userData?.userName,
+      userName: displayName,
       expiresAt: extractBbRouterExpiry(cookies),
     };
 
     saveSession(session);
-    console.log(`✓ Logged in as ${userData?.userName || 'unknown'}`);
+    console.log(`✓ Logged in as ${displayName || 'unknown'}`);
 
     return session;
   } finally {
@@ -215,11 +231,33 @@ export async function silentRelogin(previousSession?: Session | null): Promise<S
       throw new SilentLoginFailed('Redirect succeeded but session cookies are missing');
     }
 
+    let userId = previousSession?.userId;
+    let userName = previousSession?.userName;
+
+    // Self-heal: older sessions were stored with userName=null because the
+    // old extractor looked at the wrong field. Refetch once if it's missing.
+    if (!userName || !userId) {
+      try {
+        const cookieStrForApi = cookies
+          .filter(c => c.domain.includes('aulavirtual.upc.edu.pe'))
+          .map(c => `${c.name}=${c.value}`)
+          .join('; ');
+        const resp = await page.request.get(`${BASE_URL}/learn/api/public/v1/users/me`, {
+          headers: { Accept: 'application/json', Cookie: cookieStrForApi },
+        });
+        if (resp.ok()) {
+          const me = await resp.json();
+          userId = userId ?? me?.id;
+          userName = userName ?? resolveDisplayName(me);
+        }
+      } catch {}
+    }
+
     const session: Session = {
       cookies,
       xsrfToken: extractXsrf(cookies),
-      userId: previousSession?.userId,
-      userName: previousSession?.userName,
+      userId,
+      userName,
       expiresAt: extractBbRouterExpiry(cookies),
     };
 

package/src/auth/session.ts

@@ -5,6 +5,7 @@ import type { Session } from '../types/index.js';
 
 const SESSION_DIR = path.join(os.homedir(), '.blackboard-cli');
 const SESSION_FILE = path.join(SESSION_DIR, 'session.json');
+const PROFILE_DIR = path.join(SESSION_DIR, 'browser-profile');
 
 export function saveSession(session: Session): void {
   if (!fs.existsSync(SESSION_DIR)) {
@@ -27,10 +28,20 @@ export function loadSession(): Session | null {
   }
 }
 
-export function clearSession(): void {
+export function clearSession(opts: { keepProfile?: boolean } = {}): void {
   try {
     if (fs.existsSync(SESSION_FILE)) fs.unlinkSync(SESSION_FILE);
   } catch {}
+  if (!opts.keepProfile) clearBrowserProfile();
+}
+
+// The browser profile holds Microsoft SSO cookies. Without clearing it,
+// the next `login` silently re-authenticates with the same account and
+// switching users becomes impossible.
+export function clearBrowserProfile(): void {
+  try {
+    if (fs.existsSync(PROFILE_DIR)) fs.rmSync(PROFILE_DIR, { recursive: true, force: true });
+  } catch {}
 }
 
 export function isSessionValid(session: Session | null): boolean {

package/src/commands/login.ts

@@ -2,8 +2,10 @@ import { Command } from 'commander';
 import inquirer from 'inquirer';
 import chalk from 'chalk';
 import ora from 'ora';
-import { login } from '../auth/login.js';
-import { loadSession, loadOrRefreshSession, clearSession, isSessionValid } from '../auth/session.js';
+import { login, resolveDisplayName } from '../auth/login.js';
+import { loadSession, loadOrRefreshSession, saveSession, clearSession, isSessionValid } from '../auth/session.js';
+import { createClient } from '../api/client.js';
+import { getMe } from '../api/courses.js';
 import { ok, fail, warn, whatNext } from '../ui/theme.js';
 
 export function loginCommand(program: Command) {
@@ -48,21 +50,40 @@ export function loginCommand(program: Command) {
 
   program
     .command('logout')
-    .description('Clear the saved session')
-    .action(() => {
-      clearSession();
-      console.log(chalk.green('✓ Session cleared'));
+    .description('Borra la sesión y las cookies SSO para poder cambiar de cuenta')
+    .option('--keep-profile', 'Conservar las cookies SSO del navegador (no permite cambiar de cuenta)')
+    .action((opts) => {
+      clearSession({ keepProfile: !!opts.keepProfile });
+      if (opts.keepProfile) {
+        console.log(chalk.green('✓ Sesión borrada') + chalk.gray(' (profile SSO conservado)'));
+      } else {
+        console.log(chalk.green('✓ Sesión y profile SSO borrados'));
+        console.log(chalk.gray('  El próximo `blackboard login` te pedirá credenciales de nuevo.'));
+      }
     });
 
   program
     .command('whoami')
     .description('Show current logged-in user')
     .action(async () => {
-      const session = await loadOrRefreshSession();
+      let session = await loadOrRefreshSession();
       if (!isSessionValid(session)) {
         console.log(chalk.red('Not logged in. Run: blackboard login'));
         process.exit(1);
       }
+
+      // Self-heal old sessions that were stored with userName=null.
+      if (!session!.userName) {
+        try {
+          const me = await getMe(createClient(session!));
+          const name = resolveDisplayName(me);
+          if (name) {
+            session = { ...session!, userId: session!.userId ?? me?.id, userName: name };
+            saveSession(session!);
+          }
+        } catch {}
+      }
+
       console.log(chalk.green(`Logged in as: ${chalk.bold(session!.userName || session!.userId || 'unknown')}`));
       const remaining = Math.round((session!.expiresAt - Date.now()) / 60_000);
       console.log(chalk.gray(`Session expires in: ${remaining} minutes`));

package/src/index.ts

@@ -6,9 +6,10 @@ import { coursesCommand } from './commands/courses.js';
 import { apiDocsCommand } from './commands/api-docs.js';
 import { downloadCommand } from './commands/download.js';
 import { assignmentsCommand } from './commands/assignments.js';
-import { loadSession, loadOrRefreshSession, isSessionValid } from './auth/session.js';
+import { loadSession, loadOrRefreshSession, saveSession, isSessionValid } from './auth/session.js';
 import { createClient } from './api/client.js';
 import { getMe, getSystemVersion } from './api/courses.js';
+import { resolveDisplayName } from './auth/login.js';
 import { BANNER, ok, fail, hint } from './ui/theme.js';
 
 const program = new Command();
@@ -16,7 +17,7 @@ const program = new Command();
 program
   .name('blackboard')
   .description('CLI no oficial para UPC Aula Virtual (Blackboard Learn)')
-  .version('1.0.7')
+  .version('1.0.9')
   .addHelpText('beforeAll', BANNER);
 
 // Auth commands
@@ -31,12 +32,23 @@ program
   .description('Estado de sesión y versión del servidor')
   .option('--json', 'Output raw JSON')
   .action(async (opts) => {
-    const session = await loadOrRefreshSession();
+    let session = await loadOrRefreshSession();
     const valid = isSessionValid(session);
 
-    const sysVersion = await getSystemVersion(
-      createClient(session ?? { cookies: [], xsrfToken: '', expiresAt: 0 })
-    ).catch(() => null);
+    const client = createClient(session ?? { cookies: [], xsrfToken: '', expiresAt: 0 });
+    const sysVersion = await getSystemVersion(client).catch(() => null);
+
+    // Self-heal old sessions with userName=null.
+    if (valid && !session!.userName) {
+      try {
+        const me = await getMe(client);
+        const name = resolveDisplayName(me);
+        if (name) {
+          session = { ...session!, userId: session!.userId ?? me?.id, userName: name };
+          saveSession(session!);
+        }
+      } catch {}
+    }
 
     const result = {
       loggedIn: valid,

package/src/mcp/server.ts

@@ -400,20 +400,45 @@ export async function startMcpServer() {
       description:
         'Save a single answer for a quiz question (does NOT submit — use submit_quiz to finalize). ' +
         'question is the full question object from get_quiz_questions. ' +
-        'answer: boolean for true/false, or 0-based index number for multiple-choice.',
+        'answer format depends on question.type:\n' +
+        '  - eitherOr (true/false):       boolean (true = Verdadero)\n' +
+        '  - multipleanswer (MC):         number (0-based index of the chosen option)\n' +
+        '  - fimb (fill-in-multi-blanks): JSON string of an object mapping blank names to values, ' +
+        'e.g. \'{"BLANK-1":"1438.62","BLANK-2":"140.62"}\' (read blank names from question.blanks)',
       inputSchema: {
         courseId: z.string().describe('Course ID'),
         attemptId: z.string().describe('Quiz attempt ID (e.g. _94898825_1)'),
         question: z.string().describe('JSON string of the question object from get_quiz_questions'),
-        answer: z.union([z.boolean(), z.number()]).describe(
-          'For true/false: true or false. For multiple-choice: 0-based index of selected option.'
+        answer: z.union([z.boolean(), z.number(), z.string()]).describe(
+          'eitherOr: true/false. multipleanswer: 0-based index. ' +
+          'fimb: JSON string of {blankName: value} (e.g. \'{"BLANK-1":"1438.62"}\').'
         ),
       },
     },
     async ({ courseId, attemptId, question: questionJson, answer }) => {
       const { client } = await getClient();
       const question: QuizQuestion = JSON.parse(questionJson);
-      const result = await saveQuizAnswer(client, courseId, attemptId, question, answer as boolean | number);
+
+      // For fimb, the MCP transport gives us a JSON string — parse it into the Record<string, string>
+      // saveQuizAnswer expects. boolean / number pass through as-is.
+      let parsedAnswer: boolean | number | Record<string, string>;
+      if (typeof answer === 'string') {
+        try {
+          const obj = JSON.parse(answer);
+          if (obj === null || typeof obj !== 'object' || Array.isArray(obj)) {
+            throw new Error('fimb answer string must parse to a JSON object');
+          }
+          parsedAnswer = obj as Record<string, string>;
+        } catch (e: any) {
+          throw new Error(
+            `Invalid fimb answer: expected a JSON object string like '{"BLANK-1":"value"}'. ${e.message}`
+          );
+        }
+      } else {
+        parsedAnswer = answer;
+      }
+
+      const result = await saveQuizAnswer(client, courseId, attemptId, question, parsedAnswer);
       return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     }
   );

package/.mcp.json

@@ -1,8 +0,0 @@
-{
-    "mcpServers": {
-      "blackboard": {
-        "command": "npx",
-        "args": ["blackboard-upc", "mcp"]
-      }
-    }
-  }