Skip to content

fix(plugin): avoid partial active plugin replacement#500

Merged
FinleyGe merged 1 commit into
labring:mainfrom
FinleyGe:codex/fix-active-plugin-replacement
Jul 8, 2026
Merged

fix(plugin): avoid partial active plugin replacement#500
FinleyGe merged 1 commit into
labring:mainfrom
FinleyGe:codex/fix-active-plugin-replacement

Conversation

@FinleyGe

@FinleyGe FinleyGe commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • defer direct install activation until plugin files save successfully
  • replace same-version active plugins and the system installation relation in one Mongo session
  • add regression coverage for disabled ETag restore and upload failure paths

Tests

  • ./node_modules/.bin/vitest run packages/infrastructure/src/plugin/plugin.repo.test.ts packages/usecase/src/usecase.test.ts
  • ./node_modules/.bin/eslint packages/infrastructure/src/plugin/plugin.repo.ts packages/infrastructure/src/plugin/plugin.repo.test.ts
  • git diff --check upstream/main...HEAD
  • ./node_modules/.bin/tsc --noEmit (fails on existing sdk/factory/src/tool-factory.test.ts stream mock type mismatch)

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Build Successful - Preview Server image for this PR:

ghcr.io/labring/fastgpt-plugin-pr:server_7980869a5572ace1cc73fd5ce9b39459513ce14b

Time: 2026-07-08 15:47:54 (UTC+8)

@FinleyGe
FinleyGe merged commit 6aa56fd into labring:main Jul 8, 2026
3 checks passed
@FinleyGe
FinleyGe deleted the codex/fix-active-plugin-replacement branch July 8, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant