90ea3cbed7
* fix: remove redundant "None" skip option from module selection The "None - Skip module installation" option was unnecessary since core is always locked/selected, satisfying the required constraint. Users can simply press Enter with only core selected to skip modules. Also removes dead code: selectModules(), getExternalModuleChoices(), and selectExternalModules() methods that were never called. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: support ESM and .cjs module installers in ModuleManager Module installer loading now handles three cases: - .cjs files loaded via require() (always CommonJS regardless of package type) - .js files loaded via dynamic import() (works for both CJS and ESM) - CJS default export unwrapped automatically for consistent API This fixes errors when external modules set "type":"module" in their package.json. Those modules must still rename installer.js to installer.cjs if it uses require() internally. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: address code review findings from PR #1590 - Filter 'core' from CLI --modules in update path for consistency - Update selectAllModules() JSDoc to reflect core exclusion - Fix ESM default-export unwrap to handle function/class exports Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: clarify module post-install script errors as non-fatal warnings Change error display from log.error to log.warn and explain that the module was installed successfully — only the optional post-install script could not run. Prevents users from thinking the module installation itself failed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: suppress non-fatal module post-install script errors Post-install scripts fail due to CJS/ESM incompatibility but module files are already copied successfully. Silently catch the error instead of showing a warning that alarms users into thinking installation failed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: remove redundant modules and tools lines from install summary The checkmark list already shows each installed module and IDE tool. Keep only the install path and file-warning lines in the summary footer. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Brian <bmadcode@gmail.com> |
||
|---|---|---|
| .. | ||
| agent | ||
| activation-builder.js | ||
| agent-analyzer.js | ||
| agent-party-generator.js | ||
| cli-utils.js | ||
| config.js | ||
| file-ops.js | ||
| platform-codes.js | ||
| project-root.js | ||
| prompts.js | ||
| ui.js | ||
| xml-handler.js | ||
| xml-to-markdown.js | ||
| yaml-format.js | ||
| yaml-xml-builder.js | ||