fix(update): skip disabled plugins during post-update sync#73970
Conversation
Greptile SummaryThis PR adds a Confidence Score: 5/5PR is safe to merge — narrowly scoped change with full test coverage across all three install sources. The change is small and well-contained: a single optional flag added to one function, called from one site with a hardcoded value. The underlying resolveEffectiveEnableState logic is pre-existing and well-tested. New tests cover all three source types (npm, ClawHub, marketplace) and verify that enabled-plugin failures remain fatal, preserving the fail-closed contract for non-disabled plugins. No files require special attention. Reviews (1): Last reviewed commit: "fix(update): skip disabled plugins durin..." | Re-trigger Greptile |
Co-authored-by: openclaw-clownfish[bot] <280122609+openclaw-clownfish[bot]@users.noreply.github.com>
…73970) Co-authored-by: openclaw-clownfish[bot] <280122609+openclaw-clownfish[bot]@users.noreply.github.com>
…73970) Co-authored-by: openclaw-clownfish[bot] <280122609+openclaw-clownfish[bot]@users.noreply.github.com>
…73970) Co-authored-by: openclaw-clownfish[bot] <280122609+openclaw-clownfish[bot]@users.noreply.github.com>
…73970) Co-authored-by: openclaw-clownfish[bot] <280122609+openclaw-clownfish[bot]@users.noreply.github.com>
Summary
Fixes #73880.
Supersedes #73902.
Validation
tbx_01kqbgen9xmqs1ma6y7ra2h656:env OPENCLAW_TESTBOX=1 pnpm test:serial src/plugins/update.test.ts src/cli/update-cli.test.tstbx_01kqbgen9xmqs1ma6y7ra2h656:env OPENCLAW_TESTBOX=1 pnpm check:changed