coursecode 0.1.26 → 0.1.27

package/framework/docs/FRAMEWORK_GUIDE.md

@@ -662,9 +662,9 @@ Runs **in Node.js** during build (via `vite.framework-dev.config.js` `closeBundl
 
 **Errors fail the build; warnings print but don't block.**
 
-### MCP `coursecode_lint` — Unified Results
+### MCP `coursecode_lint` — Build-Time Only
 
-The MCP `coursecode_lint` tool always runs the build linter. When the preview server is running and the headless browser is connected, it also merges runtime errors from the preview server's error log into the same response. These are the same errors shown in the debug panel's Errors tab and returned by `coursecode_state` — LMS API misuse, console errors, uncaught exceptions, and data limit warnings. Runtime-sourced items are tagged with `source: 'runtime'` and `rule: 'runtime-error'`. The `runtimeLintIncluded` flag in the response indicates whether runtime errors were included. This gives AI agents a single tool for both static lint and runtime errors without needing a separate `coursecode_state` call.
+The MCP `coursecode_lint` tool runs the build linter (config validation, CSS class verification, structure checks). It does NOT include runtime errors. For runtime errors, contrast warnings, and other dynamic issues, use `coursecode_state` which returns `frameworkLogs` and `errors` from the preview server.
 
 ### Shared Rules (`lib/validation-rules.js`)
 

package/lib/mcp-prompts.js

@@ -308,13 +308,12 @@ Use to discover available interactions before creating assessments.`,
         name: 'coursecode_lint',
         description: `Run the course linter and get structured results.
 
-Always runs build-time lint (config, CSS classes, structure). When the preview server is running and the headless browser is connected, also includes runtime errors from the preview server (same errors shown in coursecode_state and the debug panel Errors tab).
+Always runs build-time lint (config, CSS classes, structure). Does NOT include runtime errors — use coursecode_state for runtime errors and contrast warnings when the preview server is running.
 
 Returns:
-- errors: [{slideId?, rule, message, severity, source?, hint?}]
-- warnings: [{slideId?, rule, message, severity, source?, class?, suggestion?, hint?}]
+- errors: [{slideId?, rule, message, severity, hint?}]
+- warnings: [{slideId?, rule, message, severity, hint?}]
 - passed: boolean
-- runtimeLintIncluded: boolean (true when runtime errors were included)
 
 Build-time rules (always checked):
 - undefined-css-class: hallucinated or stale class names (with fix suggestions)
@@ -325,18 +324,6 @@ Build-time rules (always checked):
 - assessment-id-mismatch: config ID doesn't match assessment ID
 - invalid-gating: bad gating condition configuration
 
-Runtime errors (included when preview is running, source='runtime'):
-- LMS API misuse (GetValue before Initialize, SetValue after Terminate, etc.)
-- Console errors and warnings from the course
-- Uncaught exceptions and unhandled promise rejections
-- Suspend data size warnings
-
-Suppression: Add data-lint-ignore to any HTML element to suppress warnings for it and children.
-  data-lint-ignore           — suppress all warnings
-  data-lint-ignore="spacing" — suppress only spacing warnings
-  data-lint-ignore="spacing,contrast" — suppress multiple categories
-Categories: spacing, contrast, target-size, proximity, overlap, list-style, css-class
-
 Use AFTER making changes to validate the course.`,
         inputSchema: {
             type: 'object',

package/lib/mcp-server.js

@@ -105,14 +105,23 @@ export async function startMcpServer(options = {}) {
                             lmsState: api.getLmsState()
                         };
                     });
-                    // Read last 20 API log entries from parent window (stub player diagnostic)
-                    result.apiLog = await headless.evaluateParent(() => {
-                        return window._stubPlayerState?.apiLog?.slice(0, 20) || [];
-                    });
-                    // Read error log from parent window (stub player diagnostic)
-                    result.errors = await headless.evaluateParent(() => {
-                        return window._stubPlayerState?.errorLog || [];
-                    });
+                    // Read API log and error log from preview server (same data user sees in debug panel)
+                    try {
+                        const [logResp, errResp] = await Promise.all([
+                            fetch(`http://localhost:${port}/__lms/log`),
+                            fetch(`http://localhost:${port}/__lms/errors`)
+                        ]);
+                        result.apiLog = logResp.ok ? (await logResp.json()).entries?.slice(0, 20) || [] : [];
+                        if (errResp.ok) {
+                            const errData = await errResp.json();
+                            result.errors = [...(errData.errors || []), ...(errData.warnings || [])];
+                        } else {
+                            result.errors = [];
+                        }
+                    } catch {
+                        result.apiLog = [];
+                        result.errors = [];
+                    }
                     // Append console errors/warnings captured from the page
                     result.consoleLogs = headless.getConsoleLogs();
                     break;
@@ -247,48 +256,6 @@ export async function startMcpServer(options = {}) {
                     break;
                 case 'coursecode_lint':
                     result = await lintCourse();
-                    // Merge runtime errors if headless browser is already connected
-                    if (headless.isRunning()) {
-                        try {
-                            const runtimeErrors = await headless.evaluateParent(() => {
-                                return window._stubPlayerState?.errorLog || [];
-                            });
-                            result.runtimeLintIncluded = true;
-                            if (runtimeErrors.length > 0) {
-                                const runtimeWarnings = runtimeErrors.filter(e => e.isWarning);
-                                const runtimeErrs = runtimeErrors.filter(e => !e.isWarning);
-                                if (runtimeWarnings.length > 0) {
-                                    result.warnings = (result.warnings || []).concat(
-                                        runtimeWarnings.map(e => ({
-                                            severity: 'warning',
-                                            type: e.type,
-                                            message: e.message,
-                                            hint: e.hint,
-                                            rule: 'runtime-error',
-                                            source: 'runtime'
-                                        }))
-                                    );
-                                    result.warningCount = result.warnings.length;
-                                }
-                                if (runtimeErrs.length > 0) {
-                                    result.errors = (result.errors || []).concat(
-                                        runtimeErrs.map(e => ({
-                                            severity: 'error',
-                                            type: e.type,
-                                            message: e.message,
-                                            hint: e.hint,
-                                            rule: 'runtime-error',
-                                            source: 'runtime'
-                                        }))
-                                    );
-                                    result.errorCount = result.errors.length;
-                                    result.passed = result.errors.length === 0;
-                                }
-                            }
-                        } catch {
-                            // Headless browser may have disconnected — non-fatal, build lint still valid
-                        }
-                    }
                     break;
 
                 case 'coursecode_export_content':

package/package.json

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