Problem
ReplaceImportedDatabase moves the imported temp database into place and then deletes SQLite sidecars. If sidecar cleanup throws after the move, the outer import flow can report the import as failed even though the main database has already been replaced.
Evidence
src/CodeIndex/Cli/ExportImportCommandRunner.cs around ReplaceImportedDatabase
Suggested direction
Separate post-replacement sidecar cleanup failures from the database replacement outcome. Either make sidecar cleanup best-effort after a successful move, or report a partial-success state that does not imply the original DB is still intact.
Problem
ReplaceImportedDatabasemoves the imported temp database into place and then deletes SQLite sidecars. If sidecar cleanup throws after the move, the outer import flow can report the import as failed even though the main database has already been replaced.Evidence
src/CodeIndex/Cli/ExportImportCommandRunner.csaroundReplaceImportedDatabaseSuggested direction
Separate post-replacement sidecar cleanup failures from the database replacement outcome. Either make sidecar cleanup best-effort after a successful move, or report a partial-success state that does not imply the original DB is still intact.