2 Commits

Author	SHA1	Message	Date
jakevin	15c6d0d508	refactor: deduplicate code, improve type safety, simplify error classes (#480) ... - Extract shared parseYamlArgs() to yaml-schema.ts, eliminating duplicate YAML args parsing in discovery.ts and build-manifest.ts - Unify BROWSER_ONLY_STEPS: export from capabilityRouting.ts, reuse in pipeline executor (fixes missing intercept/tap in retry set) - Remove dead normalizeArgValue from commanderAdapter; bool coercion now handled solely by coerceAndValidateArgs in execution.ts - Add closeWindow?() to IPage interface, replacing unsafe casts in executor - BrowserBridge/CDPBridge implement IBrowserFactory, removing double cast in getBrowserFactory() - Simplify CliError subclasses with new.target.name (9 redundant this.name assignments removed) - Add hook dedup in addHook() to prevent duplicate registrations - Fix normalizeRows to safely handle primitive values - Unify CommandArgs type: execution.ts now imports from registry.ts - Cache strategyLabel() call in cli.ts list command	2026-03-27 02:45:42 +08:00
AlexYue	1512016967	feat(plugin): add lifecycle hooks API (onStartup, onBeforeExecute, onAfterExecute) (#376) ... * feat(plugin): add lifecycle hooks API (onStartup, onBeforeExecute, onAfterExecute) Introduce a hooks system that allows plugins to tap into opencli's execution lifecycle without modifying core code. New files: - src/hooks.ts: hook registration, emission, and globalThis singleton - src/hooks.test.ts: 10 unit tests covering registration, ordering, error isolation, async support, and globalThis sharing Modified files: - src/execution.ts: emit onBeforeExecute/onAfterExecute around command execution - src/main.ts: emit onStartup after discoverPlugins() - src/registry-api.ts: export hooks API for plugin consumption Example plugin: https://github.com/ByteYue/opencli-plugin-audit-log * fix(discovery): load plugin files that register lifecycle hooks The isCliModule() check only matched files containing 'cli(' calls, silently skipping hook-only files like audit-hooks.ts that register onBeforeExecute/onAfterExecute without any cli() command registration. Renamed CLI_MODULE_PATTERN → PLUGIN_MODULE_PATTERN and extended the regex to also match onStartup(, onBeforeExecute(, onAfterExecute(. * fix(plugin): tighten lifecycle hook semantics --------- Co-authored-by: jackwener <jakevingoo@gmail.com>	2026-03-25 00:09:43 +08:00