Skip to content

feat(setting): choose which agents to migrate to MongoDB - #496

Merged
iceljc merged 5 commits into
SciSharp:mainfrom
li-zhou-dev:features/migrate-selected-agents
Nov 10, 2026
Merged

iceljc merged 5 commits into
SciSharp:mainfrom
li-zhou-dev:features/migrate-selected-agents

Conversation

@li-zhou-dev

Copy link
Copy Markdown
Contributor

Replace the migrate-all button with a searchable multi-select of the agents in the file repository (GET /refresh-agents/options). Start Migration is disabled until at least one agent is selected, and only the selected agent ids are sent to /refresh-agents.

Replace the migrate-all button with a searchable multi-select of the agents in
the file repository (GET /refresh-agents/options). Start Migration is disabled
until at least one agent is selected, and only the selected agent ids are sent
to /refresh-agents.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

li-zhou-dev and others added 2 commits November 6, 2026 17:35
The settings page mixed the JSON settings editor with the migration state,
handlers and two hand-rolled auto-hiding banners. Move the migration into
agent-migration.svelte and leave the page to the settings tabs.

- one result state and one cancellable timer replace the duplicated
  success/error setTimeouts, so a second message no longer gets hidden early
- log the error in both catch blocks instead of discarding it
- skip agents that have no id instead of offering an option with an empty value
- explain an empty list (no agent files, or the default database is the file
  repository) instead of showing only "Nothing..."

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Follows the rename of the backend method it calls.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Comment thread src/lib/services/api-endpoints.js Resolved
@yileicn

yileicn commented Nov 9, 2026

Copy link
Copy Markdown
Member

✅ Code Review Report

Summary

  • Result: Pass
  • Review ID: 11126
  • Commit: 49bd7c6
  • Full report
  • Status: Completed
  • Reviewed by: Nick Yi
  • Completed at: 2026-11-09 01:04:58

Security issues are evaluated independently of the code quality baseline. High or Critical security issues affect the verdict, except excluded GPL-2.0 license issues.
This comment shows up to 15 security issues and 10 per severity of code quality issues; see the full report for all findings.

🔒 Security Scan

No security issues are recorded; scan completion is not known from this report.

📋 Code Quality Issues

🔵 Low (1 total, 1 counted, 0 new)

• style in .../lib/langs/en.json:8 — Counted
The confirmation message displays "mongoDb", whereas the adjacent migration messages use the official spelling "MongoDB". The current text remains understandable and functional, but consistent branding would improve UI polish.
💡 [Needs Review - Inconclusive] Change "mongoDb" to "MongoDB" in the translated value. Preserve the lookup key unless its callers are updated together.

📊 Test Coverage: Failed


Generated by AegisCode at 2026-11-09 01:04:59 UTC

li-zhou-dev and others added 2 commits November 9, 2026 09:12
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@iceljc
iceljc merged commit 1086501 into SciSharp:main Nov 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants