mirror of
https://github.com/ChuckBuilds/LEDMatrix.git
synced 2026-04-10 21:03:01 +00:00
* fix(web): wire up "Check & Update All" plugins button window.updateAllPlugins was never assigned, so the button always showed "Bulk update handler unavailable." Wire it to PluginInstallManager.updateAll(), add per-plugin progress feedback in the button text, show a summary notification on completion, and skip redundant plugin list reloads. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: add dev preview server, CLI render script, and visual test display manager Adds local development tools for rapid plugin iteration without deploying to RPi: - VisualTestDisplayManager: renders real pixels via PIL (same fonts/interface as production) - Dev preview server (Flask): interactive web UI with plugin picker, auto-generated config forms, zoom/grid controls, and mock data support for API-dependent plugins - CLI render script: render any plugin to PNG for AI-assisted visual feedback loops - Updated test runner and conftest to auto-detect plugin-repos/ directory Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(dev-preview): address code review issues - Use get_logger() from src.logging_config instead of logging.getLogger() in visual_display_manager.py to match project logging conventions - Eliminate duplicate public/private weather draw methods — public draw_sun/ draw_cloud/draw_rain/draw_snow now delegate to the private _draw_* variants so plugins get consistent pixel output in tests vs production - Default install_deps=False in dev_server.py and render_plugin.py — dev scripts don't need to run pip install; developers are expected to have plugin deps installed in their venv already - Guard plugins_dir fixture against PermissionError during directory iteration - Fix PluginInstallManager.updateAll() to fall back to window.installedPlugins when PluginStateManager.installedPlugins is empty (plugins_manager.js populates window.installedPlugins independently of PluginStateManager) - Remove 5 debug console.log statements from plugins_manager.js button setup and initialization code Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix(scroll): fix scroll completion to prevent multi-pass wrapping Change required_total_distance from total_scroll_width + display_width to total_scroll_width alone. The scrolling image already contains display_width pixels of blank initial padding, so reaching total_scroll_width means all content has scrolled off-screen. The extra display_width term was causing 1-2+ unnecessary wrap-arounds, making the same games appear multiple times and producing a black flicker between passes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix(dev-preview): address PR #264 code review findings - docs/DEV_PREVIEW.md: add bash language tag to fenced code block - scripts/dev_server.py: add MAX/MIN_WIDTH/HEIGHT constants and validate width/height in render endpoint; add structured logger calls to discover_plugins (missing dirs, hidden entries, missing manifest, JSON/OS errors, duplicate ids); add type annotations to all helpers - scripts/render_plugin.py: add MIN/MAX_DIMENSION validation after parse_args; replace prints with get_logger() calls; narrow broad Exception catches to ImportError/OSError/ValueError in plugin load block; add type annotations to all helpers and main(); rename unused module binding to _module - scripts/run_plugin_tests.py: wrap plugins_path.iterdir() in try/except PermissionError with fallback to plugin-repos/ - scripts/templates/dev_preview.html: replace non-focusable div toggles with button role="switch" + aria-checked; add keyboard handlers (Enter/Space); sync aria-checked in toggleGrid/toggleAutoRefresh - src/common/scroll_helper.py: early-guard zero total_scroll_width to keep scroll_position at 0 and skip completion/wrap logic - src/plugin_system/testing/visual_display_manager.py: forward color arg in draw_cloud -> _draw_cloud; add color param to _draw_cloud; restore _scrolling_state in reset(); narrow broad Exception catches in _load_fonts to FileNotFoundError/OSError/ImportError; add explicit type annotations to draw_text - test/plugins/test_visual_rendering.py: use context manager for Image.open in test_save_snapshot - test/plugins/conftest.py: add return type hints to all fixtures Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: add bandit and gitleaks pre-commit hooks Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Chuck <chuck@example.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
128 lines
4.0 KiB
JavaScript
128 lines
4.0 KiB
JavaScript
/**
|
|
* Plugin installation and update management.
|
|
*
|
|
* Handles plugin installation, updates, and uninstallation operations.
|
|
*/
|
|
|
|
const PluginInstallManager = {
|
|
/**
|
|
* Install a plugin.
|
|
*
|
|
* @param {string} pluginId - Plugin identifier
|
|
* @param {string} branch - Optional branch name to install from
|
|
* @returns {Promise<Object>} Installation result
|
|
*/
|
|
async install(pluginId, branch = null) {
|
|
try {
|
|
const result = await window.PluginAPI.installPlugin(pluginId, branch);
|
|
|
|
// Refresh installed plugins list
|
|
if (window.PluginStateManager) {
|
|
await window.PluginStateManager.loadInstalledPlugins();
|
|
}
|
|
|
|
return result;
|
|
} catch (error) {
|
|
if (window.errorHandler) {
|
|
window.errorHandler.displayError(error, `Failed to install plugin ${pluginId}`);
|
|
}
|
|
throw error;
|
|
}
|
|
},
|
|
|
|
/**
|
|
* Update a plugin.
|
|
*
|
|
* @param {string} pluginId - Plugin identifier
|
|
* @returns {Promise<Object>} Update result
|
|
*/
|
|
async update(pluginId) {
|
|
try {
|
|
const result = await window.PluginAPI.updatePlugin(pluginId);
|
|
|
|
// Refresh installed plugins list
|
|
if (window.PluginStateManager) {
|
|
await window.PluginStateManager.loadInstalledPlugins();
|
|
}
|
|
|
|
return result;
|
|
} catch (error) {
|
|
if (window.errorHandler) {
|
|
window.errorHandler.displayError(error, `Failed to update plugin ${pluginId}`);
|
|
}
|
|
throw error;
|
|
}
|
|
},
|
|
|
|
/**
|
|
* Uninstall a plugin.
|
|
*
|
|
* @param {string} pluginId - Plugin identifier
|
|
* @returns {Promise<Object>} Uninstall result
|
|
*/
|
|
async uninstall(pluginId) {
|
|
try {
|
|
const result = await window.PluginAPI.uninstallPlugin(pluginId);
|
|
|
|
// Refresh installed plugins list
|
|
if (window.PluginStateManager) {
|
|
await window.PluginStateManager.loadInstalledPlugins();
|
|
}
|
|
|
|
return result;
|
|
} catch (error) {
|
|
if (window.errorHandler) {
|
|
window.errorHandler.displayError(error, `Failed to uninstall plugin ${pluginId}`);
|
|
}
|
|
throw error;
|
|
}
|
|
},
|
|
|
|
/**
|
|
* Update all plugins.
|
|
*
|
|
* @param {Function} onProgress - Optional callback(index, total, pluginId) for progress updates
|
|
* @returns {Promise<Array>} Update results
|
|
*/
|
|
async updateAll(onProgress) {
|
|
// Prefer PluginStateManager if populated, fall back to window.installedPlugins
|
|
// (plugins_manager.js populates window.installedPlugins independently)
|
|
const stateManagerPlugins = window.PluginStateManager && window.PluginStateManager.installedPlugins;
|
|
const plugins = (stateManagerPlugins && stateManagerPlugins.length > 0)
|
|
? stateManagerPlugins
|
|
: (window.installedPlugins || []);
|
|
|
|
if (!plugins.length) {
|
|
return [];
|
|
}
|
|
const results = [];
|
|
|
|
for (let i = 0; i < plugins.length; i++) {
|
|
const plugin = plugins[i];
|
|
if (onProgress) onProgress(i + 1, plugins.length, plugin.id);
|
|
try {
|
|
const result = await window.PluginAPI.updatePlugin(plugin.id);
|
|
results.push({ pluginId: plugin.id, success: true, result });
|
|
} catch (error) {
|
|
results.push({ pluginId: plugin.id, success: false, error });
|
|
}
|
|
}
|
|
|
|
// Reload plugin list once at the end
|
|
if (window.PluginStateManager) {
|
|
await window.PluginStateManager.loadInstalledPlugins();
|
|
}
|
|
|
|
return results;
|
|
}
|
|
};
|
|
|
|
// Export
|
|
if (typeof module !== 'undefined' && module.exports) {
|
|
module.exports = PluginInstallManager;
|
|
} else {
|
|
window.PluginInstallManager = PluginInstallManager;
|
|
window.updateAllPlugins = (onProgress) => PluginInstallManager.updateAll(onProgress);
|
|
}
|
|
|