ado-sync 0.1.56 → 0.1.57

package/dist/sync/engine.js

@@ -39,6 +39,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.failOnParseErrors = failOnParseErrors;
+exports.buildPushDiff = buildPushDiff;
 exports.push = push;
 exports.pull = pull;
 exports.status = status;
@@ -76,6 +78,9 @@ function matchesTags(test, expression) {
     const tagsWithAt = test.tags.map((t) => (t.startsWith('@') ? t : `@${t}`));
     return node.evaluate(tagsWithAt);
 }
+// ─── Code-type detection ──────────────────────────────────────────────────────
+/** Local types whose test bodies are executable code and may benefit from AI summarisation. */
+const CODE_TYPES = new Set(['javascript', 'playwright', 'puppeteer', 'cypress', 'testcafe', 'detox', 'espresso', 'xcuitest', 'flutter', 'java', 'csharp', 'python']);
 // ─── File discovery ───────────────────────────────────────────────────────────
 async function discoverFiles(include, exclude, configDir) {
     const patterns = Array.isArray(include) ? include : [include];
@@ -98,7 +103,57 @@ async function discoverFiles(include, exclude, configDir) {
     }
     return [...new Set(all)].sort();
 }
-// ─── Local file parsing ───────────────────────────────────────────────────────
+function failOnParseErrors(operation, failures) {
+    if (!failures.length)
+        return;
+    const details = failures
+        .map(({ filePath, message }) => `  - ${filePath}: ${message}`)
+        .join('\n');
+    throw new Error(`Aborting ${operation}: ${failures.length} file(s) could not be parsed.\n` +
+        `Fix the parse errors and rerun to avoid partial sync decisions.\n${details}`);
+}
+function supportsPullWriteback(localType) {
+    return localType === 'gherkin' || localType === 'reqnroll' || localType === 'markdown' || localType === 'csv';
+}
+function buildPushDiff(test, remote, config, cached) {
+    const tagPrefix = config.sync?.tagPrefix ?? 'tc';
+    const desiredAzure = (0, test_cases_1.buildAzureSyncContent)(test, config.sync?.format);
+    const localStepsText = desiredAzure.steps.map((s) => `${s.action}|${s.expected ?? ''}`).join('\n');
+    const remoteStepsText = remote.steps.map((s) => `${s.action}|${s.expected ?? ''}`).join('\n');
+    const titleChanged = remote.title !== desiredAzure.title;
+    const stepsChanged = localStepsText !== remoteStepsText;
+    const localTags = new Set(test.tags.filter((t) => !t.startsWith(tagPrefix + ':')));
+    const remoteTags = new Set(remote.tags);
+    const tagsChanged = [...localTags].some((t) => !remoteTags.has(t));
+    const localDescHash = (0, cache_1.hashString)(test.description);
+    const cachedDescHash = cached?.descriptionHash ?? '';
+    const descriptionChanged = localDescHash !== cachedDescHash;
+    const remoteDescHash = (0, cache_1.hashString)(remote.description);
+    const cachedRemoteDescHash = cached?.remoteDescriptionHash ?? '';
+    const remoteDescriptionChanged = cachedRemoteDescHash !== '' && remoteDescHash !== cachedRemoteDescHash;
+    const changedFields = [];
+    if (titleChanged)
+        changedFields.push('title');
+    if (stepsChanged)
+        changedFields.push('steps');
+    if (tagsChanged)
+        changedFields.push('tags');
+    if (descriptionChanged || remoteDescriptionChanged)
+        changedFields.push('description');
+    const diffDetail = [];
+    if (titleChanged)
+        diffDetail.push({ field: 'title', local: desiredAzure.title, remote: remote.title });
+    if (stepsChanged)
+        diffDetail.push({ field: 'steps', local: localStepsText, remote: remoteStepsText });
+    if (tagsChanged) {
+        const addedTags = [...localTags].filter((t) => !remoteTags.has(t));
+        diffDetail.push({ field: 'tags', local: addedTags.join(', '), remote: remote.tags.join(', ') });
+    }
+    if (descriptionChanged || remoteDescriptionChanged) {
+        diffDetail.push({ field: 'description', local: test.description ?? '', remote: remote.description ?? '' });
+    }
+    return { changedFields, diffDetail };
+}
 async function parseLocalFiles(filePaths, config, tagsFilter) {
     const tagPrefix = config.sync?.tagPrefix ?? 'tc';
     const linkConfigs = config.sync?.links;
@@ -106,6 +161,7 @@ async function parseLocalFiles(filePaths, config, tagsFilter) {
     const attachmentsConfig = config.sync?.attachments;
     const localCondition = config.local.condition;
     const results = [];
+    const failures = [];
     for (const fp of filePaths) {
         try {
             let tests;
@@ -210,9 +266,10 @@ async function parseLocalFiles(filePaths, config, tagsFilter) {
         catch (err) {
             const msg = err instanceof Error ? err.message : String(err);
             console.warn(`  [warn] Failed to parse ${fp}: ${msg}`);
+            failures.push({ filePath: fp, message: msg });
         }
     }
-    return results;
+    return { tests: results, failures };
 }
 // ─── Multi-plan helpers ───────────────────────────────────────────────────────
 /**
@@ -281,14 +338,17 @@ async function push(config, configDir, opts = {}) {
     if (config.testPlans?.length) {
         // Collect all tests across plans and run AI in one pass
         const planTests = [];
+        const parseFailures = [];
         for (const entry of config.testPlans) {
             const entryConfig = configForPlanEntry(config, entry);
             const files = await discoverFiles(entryConfig.local.include, entryConfig.local.exclude, configDir);
-            const tests = await parseLocalFiles(files, entryConfig, opts.tags);
+            const parsed = await parseLocalFiles(files, entryConfig, opts.tags);
+            parseFailures.push(...parsed.failures);
+            const tests = parsed.tests;
             planTests.push({ entryConfig, tests });
         }
+        failOnParseErrors('push', parseFailures);
         if (opts.aiSummary) {
-            const CODE_TYPES = new Set(['javascript', 'playwright', 'puppeteer', 'cypress', 'testcafe', 'detox', 'espresso', 'xcuitest', 'flutter', 'java', 'csharp', 'python']);
             const allTargets = planTests.flatMap(({ entryConfig, tests }) => CODE_TYPES.has(entryConfig.local.type) ? tests.filter(t => t.steps.length === 0 || !t.description) : []);
             let aiDone = 0;
             for (const { entryConfig, tests } of planTests) {
@@ -325,22 +385,25 @@ async function push(config, configDir, opts = {}) {
     return pushSingle(config, configDir, opts);
 }
 async function pushSingle(config, configDir, opts) {
-    const files = opts._preloadedTests
-        ? []
-        : await discoverFiles(config.local.include, config.local.exclude, configDir);
-    const tests = opts._preloadedTests
-        ?? await parseLocalFiles(files, config, opts.tags);
+    let resolvedTests = opts._preloadedTests ?? [];
+    let parseFailures = [];
+    if (!opts._preloadedTests) {
+        const files = await discoverFiles(config.local.include, config.local.exclude, configDir);
+        const parsed = await parseLocalFiles(files, config, opts.tags);
+        resolvedTests = parsed.tests;
+        parseFailures = parsed.failures;
+    }
+    failOnParseErrors('push', parseFailures);
     // AI auto-summary: for code-based local types, default to the local node-llama-cpp
     // provider (with heuristic fallback) when no explicit aiSummary opts are provided.
     // If no GGUF model path is set, the local provider transparently falls back to
     // heuristic mode so the push always succeeds even without a model installed.
-    const CODE_TYPES = new Set(['javascript', 'playwright', 'puppeteer', 'cypress', 'testcafe', 'detox', 'espresso', 'xcuitest', 'flutter', 'java', 'csharp', 'python']);
     const effectiveAiOpts = opts._preloadedTests ? undefined // AI already applied in multi-plan pre-pass
         : opts.aiSummary ?? (CODE_TYPES.has(config.local.type) ? { provider: 'local', heuristicFallback: true } : undefined);
     if (effectiveAiOpts) {
-        const aiTargets = tests.filter(t => t.steps.length === 0 || !t.description);
+        const aiTargets = resolvedTests.filter(t => t.steps.length === 0 || !t.description);
         let aiDone = 0;
-        for (const test of tests) {
+        for (const test of resolvedTests) {
             const needsSteps = test.steps.length === 0;
             const needsDescription = !test.description;
             if (needsSteps || needsDescription) {
@@ -424,7 +487,7 @@ async function pushSingle(config, configDir, opts) {
     const markAutomated = config.sync?.markAutomated ?? false;
     const recoveredIds = new Set(); // "filePath:line" keys
     let preloadedRemoteTcs;
-    const unlinkedWithAtName = tests.filter(t => !t.azureId && t.automatedTestName);
+    const unlinkedWithAtName = resolvedTests.filter(t => !t.azureId && t.automatedTestName);
     if (markAutomated && unlinkedWithAtName.length > 0) {
         try {
             preloadedRemoteTcs = await (0, test_cases_1.getTestCasesInSuite)(client, config);
@@ -445,7 +508,7 @@ async function pushSingle(config, configDir, opts) {
     // For suites with many linked tests, fetching each TC individually is slow.
     // Pre-fetch all linked IDs concurrently (up to 8 in-flight at once) and store
     // them in a Map so the sync loop can look them up without extra round-trips.
-    const linkedTests = tests.filter((t) => t.azureId !== undefined);
+    const linkedTests = resolvedTests.filter((t) => t.azureId !== undefined);
     const remoteTcCache = new Map();
     if (linkedTests.length > 0) {
         const CONCURRENCY = 8;
@@ -471,18 +534,100 @@ async function pushSingle(config, configDir, opts) {
     let done = 0;
     const reportProgress = (result) => {
         results.push(result);
-        opts.onProgress?.(++done, tests.length, result);
+        opts.onProgress?.(++done, resolvedTests.length, result);
     };
-    for (const test of tests) {
-        if (test.azureId) {
-            try {
-                const cached = cache[test.azureId];
-                // Use pre-fetched TC from cache; fall back to live fetch on miss
-                const remote = remoteTcCache.has(test.azureId)
-                    ? remoteTcCache.get(test.azureId)
-                    : await (0, test_cases_1.getTestCase)(client, test.azureId, titleField);
-                if (!remote) {
-                    // TC was deleted from Azure — re-create it and write back the new ID.
+    const PUSH_CONCURRENCY = 4; // Or 8, but we don't want to overwhelm ADO with concurrent creates/updates
+    const pushQueue = [...resolvedTests];
+    const pushWorkers = Array.from({ length: Math.min(PUSH_CONCURRENCY, pushQueue.length) }, async () => {
+        while (pushQueue.length > 0) {
+            const test = pushQueue.shift();
+            if (test.azureId) {
+                try {
+                    const cached = cache[test.azureId];
+                    // Use pre-fetched TC from cache; fall back to live fetch on miss
+                    const remote = remoteTcCache.has(test.azureId)
+                        ? remoteTcCache.get(test.azureId)
+                        : await (0, test_cases_1.getTestCase)(client, test.azureId, titleField);
+                    if (!remote) {
+                        // TC was deleted from Azure — re-create it and write back the new ID.
+                        let newId;
+                        if (!opts.dryRun) {
+                            const suiteIdOverride = byFolder
+                                ? await (0, test_cases_1.getOrCreateSuiteForFile)(client, config, test.filePath, configDir, suiteCache)
+                                : await resolveTargetSuiteFromRouting(client, config, test, suiteCache);
+                            newId = await (0, test_cases_1.createTestCase)(client, test, config, suiteIdOverride, configDir);
+                            createdIds.add(newId);
+                            if (!disableLocal) {
+                                pendingWritebacks.push({ test, newId });
+                            }
+                            await (0, test_cases_1.addTestCaseToConditionSuites)(client, config, newId, test, conditionSuiteCache);
+                            const created = await (0, test_cases_1.getTestCase)(client, newId, titleField);
+                            if (created)
+                                updateCacheEntry(cache, test, created);
+                        }
+                        reportProgress({ action: 'created', filePath: test.filePath, title: test.title, azureId: newId });
+                        continue;
+                    }
+                    // For Scenario Outlines, local steps use <param> but Azure stores @param@.
+                    // Normalise local to @param@ before comparing so outlines don't report
+                    // stepsChanged on every push after the first successful sync.
+                    const { changedFields, diffDetail } = buildPushDiff(test, remote, config, cached);
+                    if (changedFields.length === 0) {
+                        // Update cache entry even on skip (changedDate may differ due to other fields)
+                        updateCacheEntry(cache, test, remote);
+                        reportProgress({ action: 'skipped', filePath: test.filePath, title: test.title, azureId: test.azureId });
+                        continue;
+                    }
+                    // Conflict detection: remote was changed since last push AND local also differs
+                    if (cached && remote.changedDate && remote.changedDate !== cached.changedDate) {
+                        const relFile = path.relative(configDir, test.filePath);
+                        const conflict = {
+                            action: 'conflict',
+                            filePath: test.filePath,
+                            title: test.title,
+                            azureId: test.azureId,
+                            changedFields,
+                            diffDetail,
+                            detail: `${relFile}:${test.line} — changed fields: ${changedFields.join(', ')}`,
+                        };
+                        if (conflictAction === 'skip') {
+                            reportProgress(conflict);
+                            continue;
+                        }
+                        if (conflictAction === 'fail') {
+                            conflicts.push(conflict);
+                            done++;
+                            continue;
+                        }
+                        // 'overwrite' — fall through to update
+                    }
+                    if (!opts.dryRun) {
+                        await (0, test_cases_1.updateTestCase)(client, test.azureId, test, config, configDir);
+                        // Ensure the TC is in the configured suite (it may not be if the suite was
+                        // changed in config, or if the TC was imported with an ID but never pushed before).
+                        const updateSuiteId = byFolder
+                            ? await (0, test_cases_1.getOrCreateSuiteForFile)(client, config, test.filePath, configDir, suiteCache)
+                            : await resolveTargetSuiteFromRouting(client, config, test, suiteCache) ?? config.testPlan.suiteId;
+                        if (updateSuiteId) {
+                            await (0, test_cases_1.addTestCaseToSuite)(client, config, test.azureId, updateSuiteId);
+                        }
+                        else {
+                            await (0, test_cases_1.addTestCaseToRootSuite)(client, config, test.azureId);
+                        }
+                        await (0, test_cases_1.addTestCaseToConditionSuites)(client, config, test.azureId, test, conditionSuiteCache);
+                        const updated = await (0, test_cases_1.getTestCase)(client, test.azureId, titleField);
+                        if (updated)
+                            updateCacheEntry(cache, test, updated);
+                    }
+                    reportProgress({ action: 'updated', filePath: test.filePath, title: test.title, azureId: test.azureId, changedFields, diffDetail });
+                }
+                catch (err) {
+                    const msg = err instanceof Error ? err.message : String(err);
+                    reportProgress({ action: 'error', filePath: test.filePath, title: test.title, azureId: test.azureId, detail: msg });
+                }
+            }
+            else {
+                try {
                     let newId;
                     if (!opts.dryRun) {
                         const suiteIdOverride = byFolder
@@ -494,141 +639,21 @@ async function pushSingle(config, configDir, opts) {
                             pendingWritebacks.push({ test, newId });
                         }
                         await (0, test_cases_1.addTestCaseToConditionSuites)(client, config, newId, test, conditionSuiteCache);
+                        // Fetch back to get changedDate for cache
                         const created = await (0, test_cases_1.getTestCase)(client, newId, titleField);
                         if (created)
                             updateCacheEntry(cache, test, created);
                     }
                     reportProgress({ action: 'created', filePath: test.filePath, title: test.title, azureId: newId });
-                    continue;
-                }
-                // For Scenario Outlines, local steps use <param> but Azure stores @param@.
-                // Normalise local to @param@ before comparing so outlines don't report
-                // stepsChanged on every push after the first successful sync.
-                const isOutline = !!test.outlineParameters?.headers.length;
-                const localStepsText = test.steps
-                    .map((s) => {
-                    const raw = `${s.keyword} ${s.text}`;
-                    return isOutline ? raw.replace(/<([^>]+)>/g, '@$1@') : raw;
-                })
-                    .join('\n');
-                const remoteStepsText = remote.steps.map((s) => s.action).join('\n');
-                const titleChanged = remote.title !== test.title;
-                const stepsChanged = localStepsText !== remoteStepsText;
-                // For tags: push is additive (merges local into Azure), so only flag a change
-                // when the local file has tags that are NOT yet present in Azure.
-                const localTags = new Set(test.tags.filter((t) => !t.startsWith(tagPrefix + ':')));
-                const remoteTags = new Set(remote.tags);
-                const tagsChanged = [...localTags].some((t) => !remoteTags.has(t));
-                // Description: compare local hash against what we last pushed (cache), not what
-                // Azure returns (which may have been reformatted by Azure's rich-text editor).
-                const localDescHash = (0, cache_1.hashString)(test.description);
-                const cachedDescHash = cached?.descriptionHash ?? '';
-                const descriptionChanged = localDescHash !== cachedDescHash;
-                // E: also detect when the remote description changed since we last synced
-                const remoteDescHash = (0, cache_1.hashString)(remote.description);
-                const cachedRemoteDescHash = cached?.remoteDescriptionHash ?? '';
-                const remoteDescriptionChanged = cachedRemoteDescHash !== '' && remoteDescHash !== cachedRemoteDescHash;
-                // Collect which fields changed for richer reporting (D)
-                const changedFields = [];
-                if (titleChanged)
-                    changedFields.push('title');
-                if (stepsChanged)
-                    changedFields.push('steps');
-                if (tagsChanged)
-                    changedFields.push('tags');
-                if (descriptionChanged || remoteDescriptionChanged)
-                    changedFields.push('description');
-                // Build per-field diff details for structured JSON output
-                const diffDetail = [];
-                if (titleChanged)
-                    diffDetail.push({ field: 'title', local: test.title, remote: remote.title });
-                if (stepsChanged)
-                    diffDetail.push({ field: 'steps', local: localStepsText, remote: remoteStepsText });
-                if (tagsChanged) {
-                    const addedTags = [...localTags].filter((t) => !remoteTags.has(t));
-                    diffDetail.push({ field: 'tags', local: addedTags.join(', '), remote: remote.tags.join(', ') });
-                }
-                if (descriptionChanged || remoteDescriptionChanged) {
-                    diffDetail.push({ field: 'description', local: test.description ?? '', remote: remote.description ?? '' });
                 }
-                if (changedFields.length === 0) {
-                    // Update cache entry even on skip (changedDate may differ due to other fields)
-                    updateCacheEntry(cache, test, remote);
-                    reportProgress({ action: 'skipped', filePath: test.filePath, title: test.title, azureId: test.azureId });
-                    continue;
-                }
-                // Conflict detection: remote was changed since last push AND local also differs
-                if (cached && remote.changedDate && remote.changedDate !== cached.changedDate) {
-                    const relFile = path.relative(configDir, test.filePath);
-                    const conflict = {
-                        action: 'conflict',
-                        filePath: test.filePath,
-                        title: test.title,
-                        azureId: test.azureId,
-                        changedFields,
-                        diffDetail,
-                        detail: `${relFile}:${test.line} — changed fields: ${changedFields.join(', ')}`,
-                    };
-                    if (conflictAction === 'skip') {
-                        reportProgress(conflict);
-                        continue;
-                    }
-                    if (conflictAction === 'fail') {
-                        conflicts.push(conflict);
-                        done++;
-                        continue;
-                    }
-                    // 'overwrite' — fall through to update
-                }
-                if (!opts.dryRun) {
-                    await (0, test_cases_1.updateTestCase)(client, test.azureId, test, config, configDir);
-                    // Ensure the TC is in the configured suite (it may not be if the suite was
-                    // changed in config, or if the TC was imported with an ID but never pushed before).
-                    const updateSuiteId = byFolder
-                        ? await (0, test_cases_1.getOrCreateSuiteForFile)(client, config, test.filePath, configDir, suiteCache)
-                        : await resolveTargetSuiteFromRouting(client, config, test, suiteCache) ?? config.testPlan.suiteId;
-                    if (updateSuiteId) {
-                        await (0, test_cases_1.addTestCaseToSuite)(client, config, test.azureId, updateSuiteId);
-                    }
-                    else {
-                        await (0, test_cases_1.addTestCaseToRootSuite)(client, config, test.azureId);
-                    }
-                    await (0, test_cases_1.addTestCaseToConditionSuites)(client, config, test.azureId, test, conditionSuiteCache);
-                    updateCacheEntry(cache, test, remote);
-                }
-                reportProgress({ action: 'updated', filePath: test.filePath, title: test.title, azureId: test.azureId, changedFields, diffDetail });
-            }
-            catch (err) {
-                const msg = err instanceof Error ? err.message : String(err);
-                reportProgress({ action: 'error', filePath: test.filePath, title: test.title, azureId: test.azureId, detail: msg });
-            }
-        }
-        else {
-            try {
-                let newId;
-                if (!opts.dryRun) {
-                    const suiteIdOverride = byFolder
-                        ? await (0, test_cases_1.getOrCreateSuiteForFile)(client, config, test.filePath, configDir, suiteCache)
-                        : await resolveTargetSuiteFromRouting(client, config, test, suiteCache);
-                    newId = await (0, test_cases_1.createTestCase)(client, test, config, suiteIdOverride, configDir);
-                    createdIds.add(newId);
-                    if (!disableLocal) {
-                        pendingWritebacks.push({ test, newId });
-                    }
-                    await (0, test_cases_1.addTestCaseToConditionSuites)(client, config, newId, test, conditionSuiteCache);
-                    // Fetch back to get changedDate for cache
-                    const created = await (0, test_cases_1.getTestCase)(client, newId, titleField);
-                    if (created)
-                        updateCacheEntry(cache, test, created);
+                catch (err) {
+                    const msg = err instanceof Error ? err.message : String(err);
+                    reportProgress({ action: 'error', filePath: test.filePath, title: test.title, detail: msg });
                 }
-                reportProgress({ action: 'created', filePath: test.filePath, title: test.title, azureId: newId });
-            }
-            catch (err) {
-                const msg = err instanceof Error ? err.message : String(err);
-                reportProgress({ action: 'error', filePath: test.filePath, title: test.title, detail: msg });
             }
         }
-    }
+    });
+    await Promise.all(pushWorkers);
     if (conflicts.length) {
         const lines = conflicts.map((c) => {
             const fields = c.changedFields?.length ? `\n          Changed fields: ${c.changedFields.join(', ')}` : '';
@@ -640,7 +665,7 @@ async function pushSingle(config, configDir, opts) {
     // For all other framework types this is a no-op.
     if (!opts.dryRun && !disableLocal && config.local.type === 'playwright') {
         const alreadyQueued = new Set(pendingWritebacks.map((wb) => `${wb.test.filePath}:${wb.test.line}`));
-        for (const test of tests) {
+        for (const test of resolvedTests) {
             if (test.azureId && !alreadyQueued.has(`${test.filePath}:${test.line}`)) {
                 pendingWritebacks.push({ test, newId: test.azureId });
             }
@@ -651,7 +676,7 @@ async function pushSingle(config, configDir, opts) {
     // (no new TC was created). Queue them so the source annotation is restored.
     if (!opts.dryRun && !disableLocal && recoveredIds.size > 0) {
         const alreadyQueued = new Set(pendingWritebacks.map((wb) => `${wb.test.filePath}:${wb.test.line}`));
-        for (const test of tests) {
+        for (const test of resolvedTests) {
             const key = `${test.filePath}:${test.line}`;
             if (test.azureId && recoveredIds.has(key) && !alreadyQueued.has(key)) {
                 pendingWritebacks.push({ test, newId: test.azureId });
@@ -682,7 +707,7 @@ async function pushSingle(config, configDir, opts) {
             // matching, otherwise fetch now. This avoids a redundant round-trip.
             const remoteTcs = preloadedRemoteTcs ?? await (0, test_cases_1.getTestCasesInSuite)(client, config);
             const localIds = new Set([
-                ...tests.map((t) => t.azureId).filter(Boolean),
+                ...resolvedTests.map((t) => t.azureId).filter(Boolean),
                 ...createdIds,
             ]);
             for (const remote of remoteTcs) {
@@ -724,7 +749,9 @@ async function pull(config, configDir, opts = {}) {
 }
 async function pullSingle(config, configDir, opts) {
     const files = await discoverFiles(config.local.include, config.local.exclude, configDir);
-    const tests = await parseLocalFiles(files, config, opts.tags);
+    const parsed = await parseLocalFiles(files, config, opts.tags);
+    failOnParseErrors('pull', parsed.failures);
+    const tests = parsed.tests;
     const client = await client_1.AzureClient.create(config);
     const titleField = config.sync?.titleField ?? 'System.Title';
     const tagPrefix = config.sync?.tagPrefix ?? 'tc';
@@ -737,6 +764,7 @@ async function pullSingle(config, configDir, opts) {
         results.push(result);
         opts.onProgress?.(++done, linked.length, result);
     };
+    const canWriteToLocal = supportsPullWriteback(config.local.type);
     for (const test of linked) {
         try {
             const remote = await (0, test_cases_1.getTestCase)(client, test.azureId, titleField);
@@ -751,9 +779,10 @@ async function pullSingle(config, configDir, opts) {
                 });
                 continue;
             }
-            const titleChanged = remote.title !== test.title;
-            const remoteStepsText = remote.steps.map((s) => s.action + '|' + s.expected).join('\n');
-            const localStepsText = test.steps.map((s) => s.keyword + ' ' + s.text + '|' + (s.expected ?? '')).join('\n');
+            const desiredAzure = (0, test_cases_1.buildAzureSyncContent)(test, config.sync?.format);
+            const titleChanged = remote.title !== desiredAzure.title;
+            const remoteStepsText = remote.steps.map((s) => `${s.action}|${s.expected ?? ''}`).join('\n');
+            const localStepsText = desiredAzure.steps.map((s) => `${s.action}|${s.expected ?? ''}`).join('\n');
             const stepsChanged = remoteStepsText !== localStepsText;
             const descriptionChanged = (remote.description ?? '') !== (test.description ?? '');
             if (!titleChanged && !stepsChanged && !descriptionChanged) {
@@ -761,10 +790,37 @@ async function pullSingle(config, configDir, opts) {
                 continue;
             }
             if (!opts.dryRun) {
-                if (!disableLocal) {
-                    applyRemoteToLocal(test, remote.title, remote.steps.map((s) => ({ keyword: 'Step', text: s.action, expected: s.expected })), remote.description, config.local.type, tagPrefix);
+                if (disableLocal) {
+                    reportProgress({
+                        action: 'skipped',
+                        filePath: test.filePath,
+                        title: remote.title,
+                        azureId: test.azureId,
+                        detail: [
+                            titleChanged && 'title',
+                            stepsChanged && 'steps',
+                            descriptionChanged && 'description',
+                        ].filter(Boolean).join(', ') + ' changed (local changes skipped)',
+                    });
+                    continue;
+                }
+                if (!canWriteToLocal) {
+                    reportProgress({
+                        action: 'error',
+                        filePath: test.filePath,
+                        title: test.title,
+                        azureId: test.azureId,
+                        detail: `Pull is not supported for local.type "${config.local.type}".`,
+                    });
+                    continue;
                 }
-                updateCacheEntry(cache, test, remote);
+                applyRemoteToLocal(test, remote.title, remote.steps.map((s) => ({ keyword: 'Step', text: s.action, expected: s.expected })), remote.description, config.local.type, tagPrefix);
+                updateCacheEntry(cache, {
+                    ...test,
+                    title: remote.title,
+                    description: remote.description,
+                    steps: remote.steps.map((s) => ({ keyword: 'Step', text: s.action, expected: s.expected })),
+                }, remote);
             }
             reportProgress({
                 action: 'pulled',
@@ -875,8 +931,9 @@ function stripHtml(html) {
         .replace(/<\/p>/gi, '\n')
         .replace(/<[^>]+>/g, '')
         .replace(/&amp;/g, '&')
-        .replace(/&lt;/g, '<')
-        .replace(/&gt;/g, '>')
+        // Do not decode &lt; or &gt; to avoid reintroducing HTML tag delimiters such as <script>.
+        // .replace(/&lt;/g, '<')
+        // .replace(/&gt;/g, '>')
         .replace(/&quot;/g, '"')
         .replace(/&nbsp;/g, ' ')
         .replace(/\n{3,}/g, '\n\n')
@@ -1071,7 +1128,9 @@ function buildPullMarkdownContent(tc, tagPrefix) {
  */
 async function detectStaleTestCases(config, configDir, opts = {}) {
     const files = await discoverFiles(config.local.include, config.local.exclude, configDir);
-    const tests = await parseLocalFiles(files, config, opts.tags);
+    const parsed = await parseLocalFiles(files, config, opts.tags);
+    failOnParseErrors('stale test case detection', parsed.failures);
+    const tests = parsed.tests;
     const localIds = new Set(tests.map((t) => t.azureId).filter(Boolean));
     const client = await client_1.AzureClient.create(config);
     const plans = config.testPlans?.length ? config.testPlans : [config.testPlan];
@@ -1100,7 +1159,9 @@ async function detectStaleTestCases(config, configDir, opts = {}) {
  */
 async function coverageReport(config, configDir, opts = {}) {
     const files = await discoverFiles(config.local.include, config.local.exclude, configDir);
-    const tests = await parseLocalFiles(files, config, opts.tags);
+    const parsed = await parseLocalFiles(files, config, opts.tags);
+    failOnParseErrors('coverage report', parsed.failures);
+    const tests = parsed.tests;
     const linked = tests.filter((t) => t.azureId !== undefined);
     const unlinked = tests.filter((t) => t.azureId === undefined);
     // Find story link prefix from sync.links config; default to 'story'