coursecode 0.1.13 → 0.1.15

package/README.md

@@ -191,6 +191,12 @@ When ready, deploy:
 coursecode deploy
 ```
 
+If the cloud course was deleted but the project still has the old local binding, redeploy with:
+
+```bash
+coursecode deploy --repair-binding
+```
+
 **Without Cloud**: Build a ZIP package and upload it to your LMS manually:
 
 ```bash

package/bin/cli.js

@@ -353,6 +353,7 @@ program
   .option('--preview', 'Deploy as preview-only (production untouched, preview pointer always moved). Combine with --promote or --stage for a full deploy that also moves the preview pointer.')
   .option('--promote', 'Force-promote: always move production pointer regardless of deploy_mode setting. Mutually exclusive with --stage.')
   .option('--stage', 'Force-stage: never move production pointer regardless of deploy_mode setting. Mutually exclusive with --promote.')
+  .option('--repair-binding', 'Clear a stale local Cloud binding if the remote course was deleted, then continue')
   .option('--password', 'Password-protect preview (interactive prompt, requires --preview)')
   .option('-m, --message <message>', 'Deploy reason (e.g. "Fixed accessibility issues")')
   .option('--local', 'Use local Cloud instance (http://localhost:3000)')
@@ -369,6 +370,7 @@ program
   .option('--production', 'Promote to the production pointer')
   .option('--preview', 'Promote to the preview pointer')
   .option('--deployment <id>', 'Deployment ID to promote (skip interactive prompt)')
+  .option('--repair-binding', 'Clear a stale local Cloud binding if the remote course was deleted')
   .option('-m, --message <message>', 'Reason for promotion')
   .option('--local', 'Use local Cloud instance (http://localhost:3000)')
   .option('--json', 'Emit machine-readable JSON result')
@@ -382,24 +384,26 @@ program
 program
   .command('status')
   .description('Show deployment status for current course')
+  .option('--repair-binding', 'Clear a stale local Cloud binding if the remote course was deleted')
   .option('--local', 'Use local Cloud instance (http://localhost:3000)')
   .option('--json', 'Output raw JSON')
   .action(async (options) => {
     const { status, setLocalMode } = await import('../lib/cloud.js');
     if (options.local) setLocalMode();
-    await status({ json: options.json });
+    await status({ json: options.json, repairBinding: options.repairBinding });
   });
 
 program
   .command('delete')
   .description('Remove course from CourseCode Cloud (does not delete local files)')
   .option('--force', 'Skip confirmation prompt')
+  .option('--repair-binding', 'Clear a stale local Cloud binding if the remote course was already deleted')
   .option('--local', 'Use local Cloud instance (http://localhost:3000)')
   .option('--json', 'Emit machine-readable JSON result')
   .action(async (options) => {
     const { deleteCourse, setLocalMode } = await import('../lib/cloud.js');
     if (options.local) setLocalMode();
-    await deleteCourse({ force: options.force, json: options.json });
+    await deleteCourse({ force: options.force, json: options.json, repairBinding: options.repairBinding });
   });
 
 program.parse();

package/framework/docs/FRAMEWORK_GUIDE.md

@@ -123,7 +123,7 @@ The browser only downloads the one chunk matching the meta tag. Unused driver ch
 | `coursecode build` | `dist/` | Universal build + format manifest + meta tag stamped |
 | `coursecode build` (with `PACKAGE=true`) | `dist/` + ZIP | Same + format-specific ZIP for LMS upload |
 | `coursecode preview --export` | `course-preview/` | Copy of `dist/` wrapped in stub player (for Netlify/GitHub Pages) |
-| `coursecode deploy` | Uploads `dist/` | Cloud hosts universal build, assembles format ZIPs on demand. Flags: `--promote` (force live), `--stage` (force staged), `--preview` (preview-only: production untouched, preview always moved). `--promote`/`--stage` are mutually exclusive; `--preview` stacks with either. |
+| `coursecode deploy` | Uploads `dist/` | Cloud hosts universal build, assembles format ZIPs on demand. Flags: `--promote` (force live), `--stage` (force staged), `--preview` (preview-only: production untouched, preview always moved), `--repair-binding` (clear stale local cloud binding first if the remote course was deleted). `--promote`/`--stage` are mutually exclusive; `--preview` stacks with either. |
 
 The ZIP never includes preview/stub player assets. Preview is a separate concern (see below).
 

package/framework/docs/USER_GUIDE.md

@@ -740,6 +740,7 @@ Open the URL in any browser, log in with your CourseCode account, and enter the
 - **Preview pointer** — the version served on the cloud preview link (for stakeholder review).
 - **deploy_mode** — a per-course or org setting in the Cloud dashboard. Default is auto-promote (new uploads immediately go live). Can be set to staged (new uploads require a manual promote step).
 - `--promote` and `--stage` are mutually exclusive.
+- If a cloud deployment was deleted outside the CLI and this project still has the old local binding, rerun with `coursecode deploy --repair-binding`. To clear the stale binding without deploying yet, run `coursecode status --repair-binding`.
 
 **Typical Cloud workflow:**
 1. Run `coursecode login` once, open the URL shown, and enter the code.

package/framework/js/core/event-bus.js

@@ -1,10 +1,38 @@
 import { generateId } from '../utilities/utilities.js';
 import { logger } from '../utilities/logger.js';
 
+/**
+ * Safely serialize any value for logging. Handles circular references,
+ * Error instances, and oversized payloads without throwing.
+ */
+function safeStringify(data, maxLength = 4096) {
+  const seen = new WeakSet();
+  try {
+    const json = JSON.stringify(data, (key, value) => {
+      if (value instanceof Error) {
+        return { name: value.name, message: value.message, stack: value.stack };
+      }
+      if (typeof value === 'object' && value !== null) {
+        if (seen.has(value)) return '[Circular]';
+        seen.add(value);
+      }
+      return value;
+    }, 2);
+    if (json && json.length > maxLength) {
+      return json.slice(0, maxLength) + '...[truncated]';
+    }
+    return json;
+  } catch {
+    return `[Unserializable: ${typeof data}]`;
+  }
+}
+
 class EventBus {
   constructor() {
     // Event listeners registry
     this.events = {};
+    // Re-entrancy guard — prevents infinite :error → log → :error cascade
+    this._emittingError = false;
   }
 
   /**
@@ -86,31 +114,50 @@ class EventBus {
       return false;
     }
 
-    // Automatically log events that follow the ':error' naming convention
-    if (event.endsWith(':error')) {
-      logger.error(`[EventBus Error] ${event}:`, JSON.stringify(data, null, 2));
-    }
+    const isErrorEvent = event.endsWith(':error');
 
-    // Create a copy of listeners to avoid issues if listeners modify the array
-    const listeners = [...this.events[event]];
-    const onceListeners = [];
+    // Re-entrancy guard — if we're already inside an :error emit,
+    // suppress to prevent infinite cascade
+    if (isErrorEvent) {
+      if (this._emittingError) {
+        console.warn(`[EventBus] Suppressed recursive error event: ${event}`);
+        return false;
+      }
+      this._emittingError = true;
+    }
 
-    listeners.forEach(listener => {
-      try {
-        listener.callback(data);
+    try {
+      // Automatically log events that follow the ':error' naming convention
+      if (isErrorEvent) {
+        logger.error(`[EventBus Error] ${event}:`, safeStringify(data));
+      }
 
-        // Track once listeners for removal
-        if (listener.once) {
-          onceListeners.push(listener.id);
+      // Create a copy of listeners to avoid issues if listeners modify the array
+      const listeners = [...this.events[event]];
+      const onceListeners = [];
+
+      listeners.forEach(listener => {
+        try {
+          listener.callback(data);
+
+          // Track once listeners for removal
+          if (listener.once) {
+            onceListeners.push(listener.id);
+          }
+        } catch (error) {
+          // Log the error but don't break other listeners — use safeStringify
+          // to prevent a secondary cascade from unserializable error objects
+          logger.error(`[EventBus] Error in listener for '${event}':`, safeStringify(error));
         }
-      } catch (error) {
-        // Log the error but don't break other listeners
-        logger.error(`[EventBus] Error in listener for '${event}':`, error);
-      }
-    });
+      });
 
-    // Remove once listeners
-    onceListeners.forEach(id => this.off(event, id));
+      // Remove once listeners
+      onceListeners.forEach(id => this.off(event, id));
+    } finally {
+      if (isErrorEvent) {
+        this._emittingError = false;
+      }
+    }
 
     return true;
   }

package/framework/js/drivers/scorm-2004-driver.js

@@ -561,13 +561,27 @@ export class Scorm2004Driver extends ScormDriverBase {
      * Populates the CMI cache at init time. Single LMS read pass.
      */
     _populateCache() {
-        // Read-only scalars
-        this._cmiCache.entry = this._getValue('cmi.entry') || '';
-        this._cmiCache.bookmark = this._getValue('cmi.location') || '';
-        this._cmiCache.completionStatus = this._getValue('cmi.completion_status') || 'unknown';
-        this._cmiCache.successStatus = this._getValue('cmi.success_status') || 'unknown';
-        this._cmiCache.learnerId = this._getValue('cmi.learner_id') || '';
-        this._cmiCache.learnerName = this._getValue('cmi.learner_name') || '';
+        // Helper: read a CMI value via strict _getValue, but tolerate error 403
+        // ("Data Model Element Value Not Initialized") which strict LMSes like
+        // SCORM Cloud return for unset elements on a fresh session.
+        // Any other SCORM error still throws through _getValue's normal path.
+        const getOrDefault = (key, fallback) => {
+            try {
+                return this._getValue(key) || fallback;
+            } catch (e) {
+                const code = this._scorm.debug.getCode();
+                if (code === 403) return fallback;
+                throw e;
+            }
+        };
+
+        // Read-only scalars (may be uninitialized on first launch)
+        this._cmiCache.entry = getOrDefault('cmi.entry', '');
+        this._cmiCache.bookmark = getOrDefault('cmi.location', '');
+        this._cmiCache.completionStatus = getOrDefault('cmi.completion_status', 'unknown');
+        this._cmiCache.successStatus = getOrDefault('cmi.success_status', 'unknown');
+        this._cmiCache.learnerId = getOrDefault('cmi.learner_id', '');
+        this._cmiCache.learnerName = getOrDefault('cmi.learner_name', '');
 
         // Skip array hydration for fresh sessions
         if (this._cmiCache.entry === 'ab-initio') {

package/lib/cloud.js

@@ -144,6 +144,15 @@ function writeProjectConfig(data) {
   );
 }
 
+function updateProjectConfig(mutator) {
+  const dir = path.join(process.cwd(), PROJECT_CONFIG_DIR);
+  fs.mkdirSync(dir, { recursive: true });
+  const fullPath = path.join(process.cwd(), PROJECT_CONFIG_PATH);
+  const existing = readProjectConfig() || {};
+  const next = mutator({ ...existing }) || existing;
+  fs.writeFileSync(fullPath, JSON.stringify(next, null, 2) + '\n');
+}
+
 // =============================================================================
 // COURSE IDENTITY (committed: .coursecoderc.json → cloudId)
 // =============================================================================
@@ -172,6 +181,117 @@ function writeRcConfig(fields) {
   fs.writeFileSync(rcPath, JSON.stringify(existing, null, 2) + '\n');
 }
 
+function updateRcConfig(mutator) {
+  const rcPath = path.join(process.cwd(), '.coursecoderc.json');
+  const existing = readRcConfig() || {};
+  const next = mutator({ ...existing }) || existing;
+  fs.writeFileSync(rcPath, JSON.stringify(next, null, 2) + '\n');
+}
+
+function emitJson(payload) {
+  process.stdout.write(JSON.stringify(payload) + '\n');
+}
+
+function getBindingSnapshot(slug = resolveSlug()) {
+  const rcConfig = readRcConfig() || {};
+  const projectConfig = readProjectConfig() || {};
+  return {
+    slug,
+    cloudId: rcConfig.cloudId || projectConfig.courseId || null,
+    orgId: projectConfig.orgId || rcConfig.orgId || null,
+    hasBinding: Boolean(rcConfig.cloudId || projectConfig.courseId),
+  };
+}
+
+function clearCloudBinding() {
+  updateRcConfig((rc) => {
+    delete rc.cloudId;
+    delete rc.orgId;
+    return rc;
+  });
+  updateProjectConfig((project) => {
+    delete project.courseId;
+    delete project.orgId;
+    return project;
+  });
+}
+
+function buildStaleBindingPayload({
+  slug,
+  operation,
+  binding,
+  bindingCleared = false,
+  success = false,
+  alreadyDeleted = false,
+}) {
+  return {
+    success,
+    error: 'Cloud course was deleted. Local binding is stale.',
+    errorCode: 'stale_cloud_binding',
+    staleBinding: true,
+    bindingCleared,
+    repairable: true,
+    needsRedeploy: true,
+    alreadyDeleted,
+    operation,
+    suggestedAction: 'redeploy',
+    suggestedCommand: 'coursecode deploy --repair-binding',
+    repairFlag: '--repair-binding',
+    binding,
+    slug,
+  };
+}
+
+async function resolveStaleBinding({
+  operation,
+  slug,
+  options = {},
+  promptText,
+  onRepaired,
+  onDeclined,
+  onJson,
+}) {
+  const binding = getBindingSnapshot(slug);
+  if (!binding.hasBinding) return false;
+
+  const payload = buildStaleBindingPayload({ slug, operation, binding });
+
+  if (options.repairBinding) {
+    clearCloudBinding();
+    return onRepaired(buildStaleBindingPayload({
+      slug,
+      operation,
+      binding,
+      bindingCleared: true,
+      success: operation === 'delete',
+      alreadyDeleted: operation === 'delete',
+    }));
+  }
+
+  if (options.json || !process.stdin.isTTY) {
+    if (onJson) return onJson(payload);
+    emitJson(payload);
+    return true;
+  }
+
+  const answer = await prompt(`${promptText} [Y/n] `);
+  if (answer && !['y', 'yes'].includes(answer.toLowerCase())) {
+    if (onDeclined) return onDeclined(payload);
+    console.log('  Cancelled.\n');
+    process.exit(1);
+  }
+
+  clearCloudBinding();
+  return onRepaired(buildStaleBindingPayload({
+    slug,
+    operation,
+    binding,
+    bindingCleared: true,
+    success: operation === 'delete',
+    alreadyDeleted: operation === 'delete',
+  }));
+}
+
 // =============================================================================
 // HTTP HELPERS
 // =============================================================================
@@ -805,6 +925,39 @@ export async function deploy(options = {}) {
   const log = (...args) => { if (!options.json) console.log(...args); };
   const logErr = (...args) => { if (!options.json) console.error(...args); };
 
+  // Preflight a cached binding so deleted cloud courses can be repaired
+  // before we spend time building and uploading.
+  const binding = getBindingSnapshot(slug);
+  if (binding.hasBinding && binding.orgId) {
+    const statusRes = await cloudFetch(
+      `/api/cli/courses/${encodeURIComponent(slug)}/status?orgId=${binding.orgId}`,
+      {},
+      readCredentials()?.token
+    );
+
+    if (statusRes.status === 404) {
+      const handled = await resolveStaleBinding({
+        operation: 'deploy',
+        slug,
+        options,
+        promptText: '\n  This project is still linked locally, but the Cloud course was deleted. Clear the stale binding and rebuild/redeploy?',
+        onRepaired: () => false,
+        onDeclined: () => {
+          logErr('\n❌ Deploy cancelled. Local binding still points to a deleted Cloud course.\n');
+          process.exit(1);
+        },
+        onJson: (payload) => {
+          emitJson(payload);
+          console.error('\n❌ Cloud course was deleted. Re-run deploy with --repair-binding to clear the stale binding first.\n');
+          process.exit(1);
+        },
+      });
+      if (handled) return;
+    } else if (!statusRes.ok) {
+      await handleResponse(statusRes);
+    }
+  }
+
   // Validate mutually exclusive flags
   if (options.promote && options.stage) {
     logErr('\n❌ --promote and --stage are mutually exclusive.\n');
@@ -998,7 +1151,46 @@ export async function promote(options = {}) {
     return handleResponse(res, { retryFn: makeRequest, _isRetry });
   };
 
-  const result = await makeRequest();
+  const token = readCredentials()?.token;
+  const firstRes = await cloudFetch(
+    `/api/cli/courses/${encodeURIComponent(slug)}/promote`,
+    {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ deployment_id: deploymentId, target, reason }),
+    },
+    token
+  );
+
+  if (firstRes.status === 404 && getBindingSnapshot(slug).hasBinding) {
+    const handled = await resolveStaleBinding({
+      operation: 'promote',
+      slug,
+      options,
+      promptText: '\n  This project is still linked locally, but the Cloud course was deleted. Clear the stale binding?',
+      onRepaired: (payload) => {
+        if (options.json) {
+          emitJson(payload);
+        } else {
+          console.log('\n  Cleared stale Cloud binding.');
+          console.log('  The course is no longer deployed. Run `coursecode deploy` before promoting.\n');
+        }
+        return true;
+      },
+      onDeclined: () => {
+        console.error('\n❌ Promote cancelled. Local binding still points to a deleted Cloud course.\n');
+        process.exit(1);
+      },
+      onJson: (payload) => {
+        emitJson(payload);
+        console.error('\n❌ Cloud course was deleted. Deploy again before promoting.\n');
+        process.exit(1);
+      },
+    });
+    if (handled) return;
+  }
+
+  const result = await handleResponse(firstRes, { retryFn: makeRequest, _isRetry: false });
 
   if (options.json) {
     process.stdout.write(JSON.stringify({ success: true, ...result }) + '\n');
@@ -1037,7 +1229,43 @@ export async function status(options = {}) {
     return handleResponse(res, { retryFn: makeRequest, _isRetry });
   };
 
-  const data = await makeRequest();
+  const token = readCredentials()?.token;
+  const firstRes = await cloudFetch(
+    `/api/cli/courses/${encodeURIComponent(slug)}/status${orgQuery}`,
+    {},
+    token
+  );
+
+  if (firstRes.status === 404 && getBindingSnapshot(slug).hasBinding) {
+    const handled = await resolveStaleBinding({
+      operation: 'status',
+      slug,
+      options,
+      promptText: '\n  This project is still linked locally, but the Cloud course was deleted. Clear the stale binding?',
+      onRepaired: (payload) => {
+        const result = {
+          ...payload,
+          success: true,
+          deployed: false,
+          message: 'Local stale Cloud binding cleared. This course is no longer deployed.',
+        };
+        if (options.json) {
+          emitJson(result);
+        } else {
+          console.log('\n  Cleared stale Cloud binding.');
+          console.log('  This course is no longer deployed. Run `coursecode deploy` to create a new Cloud deployment.\n');
+        }
+        return true;
+      },
+      onJson: (payload) => {
+        emitJson(payload);
+        return true;
+      },
+    });
+    if (handled) return;
+  }
+
+  const data = await handleResponse(firstRes, { retryFn: makeRequest, _isRetry: false });
 
   if (options.json) {
     process.stdout.write(JSON.stringify(data) + '\n');
@@ -1122,7 +1350,52 @@ export async function deleteCourse(options = {}) {
     return handleResponse(res, { retryFn: makeRequest, _isRetry });
   };
 
-  const result = await makeRequest();
+  const token = readCredentials()?.token;
+  const firstRes = await cloudFetch(
+    `/api/cli/courses/${encodeURIComponent(slug)}${orgQuery}`,
+    {
+      method: 'DELETE',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ cloudId: rcConfig.cloudId }),
+    },
+    token
+  );
+
+  if (firstRes.status === 404 && getBindingSnapshot(slug).hasBinding) {
+    const handled = await resolveStaleBinding({
+      operation: 'delete',
+      slug,
+      options,
+      promptText: '\n  The Cloud course is already gone, but this project still has a local binding. Clear the stale binding too?',
+      onRepaired: (payload) => {
+        const result = {
+          ...payload,
+          success: true,
+          alreadyDeleted: true,
+          message: 'Cloud course was already deleted. Local stale binding cleared.',
+        };
+        if (options.json) {
+          emitJson(result);
+        } else {
+          console.log('\n✓ Course was already deleted from CourseCode Cloud.');
+          console.log('  Cleared stale local Cloud binding.\n');
+        }
+        return true;
+      },
+      onJson: (payload) => {
+        emitJson({
+          ...payload,
+          success: true,
+          alreadyDeleted: true,
+          message: 'Cloud course was already deleted. Local binding still needs cleanup.',
+        });
+        return true;
+      },
+    });
+    if (handled) return;
+  }
+
+  const result = await handleResponse(firstRes, { retryFn: makeRequest, _isRetry: false });
 
   if (options.json) {
     process.stdout.write(JSON.stringify({ success: true, ...result }) + '\n');

package/lib/manifest/cmi5-manifest.js

@@ -19,22 +19,20 @@
  * @returns {string} The cmi5.xml content
  */
 export function generateCmi5Manifest(config, _files, options = {}) {
-    // cmi5 course identifier - use configured identifier or generate from title
-    const courseId = config.identifier || 
-        `urn:coursecode:${config.title.replace(/[^a-zA-Z0-9]/g, '-').toLowerCase()}`;
-    
-    // Calculate mastery score from passing percentage (convert percentage to 0-1 scale)
-    const masteryScore = config.passingScore ? (config.passingScore / 100).toFixed(2) : '0.8';
-    
-    // AU identifier - derive from course ID
-    const auId = `${courseId}/au/1`;
-
-    // URL: absolute for cmi5-remote, relative for standard cmi5
-    const auUrl = options.externalUrl 
-        ? `${options.externalUrl.replace(/\/$/, '')}/index.html`
-        : 'index.html';
-
-    return `<?xml version="1.0" encoding="UTF-8"?>
+  // cmi5 course identifier - use configured identifier or generate from title
+  const courseId = config.identifier ||
+    `urn:coursecode:${config.title.replace(/[^a-zA-Z0-9]/g, '-').toLowerCase()}`;
+
+  // Calculate mastery score from passing percentage (convert percentage to 0-1 scale)
+  const masteryScore = config.passingScore ? (config.passingScore / 100).toFixed(2) : '0.8';
+
+  // AU identifier - derive from course ID
+  const auId = `${courseId}/au/1`;
+
+  // URL: absolute for cmi5-remote (use as-is), relative for standard cmi5
+  const auUrl = options.externalUrl || 'index.html';
+
+  return `<?xml version="1.0" encoding="UTF-8"?>
 <!-- cmi5 Course Structure - GENERATED FILE - DO NOT EDIT MANUALLY -->
 <courseStructure xmlns="https://w3id.org/xapi/profiles/cmi5/v1/CourseStructure.xsd">
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coursecode",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "description": "Multi-format course authoring framework with CLI tools (SCORM 2004, SCORM 1.2, cmi5, LTI 1.3)",
   "type": "module",
   "bin": {