tboudreaux/fourdst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(validation): update validation handler for new JSON architecture

Browse Source 
Fix handleValidateBundle to work with refactored core functions that
return validation data directly instead of nested under result.data.
Improve modal message to show actual error/warning counts.
This commit is contained in:
Emily Boudreaux
2025-08-09 18:55:31 -04:00
parent 205fb586e7
commit 15c9755611
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
electron/renderer.js
View File

@@ -336,8 +336,10 @@ async function handleValidateBundle() {
hideSpinner();
if (result.success) {
const validation = result.data;
const validationIssues = validation.errors.concat(validation.warnings);
// With the new JSON architecture, validation data is directly in result
const errors = result.errors || [];
const warnings = result.warnings || [];
const validationIssues = errors.concat(warnings);
if (validationIssues.length > 0) {
validationResults.textContent = validationIssues.join('\n');
@@ -346,9 +348,14 @@ async function handleValidateBundle() {
validationResults.textContent = 'Bundle is valid.';
validationTabLink.classList.add('hidden');
}
// Switch to the validation tab to show the results.
// Switch to the validation tab to show the results
switchTab('validation-tab');
showModal('Validation Complete', 'Validation check has finished.');
// Show summary in modal
const summary = result.summary || { errors: errors.length, warnings: warnings.length };
const message = `Validation finished with ${summary.errors} errors and ${summary.warnings} warnings.`;
showModal('Validation Complete', message);
} else {
showModal('Validation Error', `Failed to validate bundle: ${result.error}`);