fix: Interactive mode fails for non-PostgreSQL sources (SQLite, MongoDB, MySQL)

- Detect source type before entering interactive mode
- Skip interactive mode automatically for non-PostgreSQL sources
- Update README-SQLite.md examples to include -y flag
- Fixes #72

Author: taariq

CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to this project will be documented in this file.
 
+## [7.0.5] - 2025-12-09
+
+### Fixed
+
+- **Interactive mode fails for non-PostgreSQL sources**: Fixed bug where `init`, `validate`, and `sync` commands would fail for SQLite, MongoDB, and MySQL sources because interactive mode attempted to connect to the source as PostgreSQL. The tool now detects the source type before entering interactive mode and automatically skips it for non-PostgreSQL sources.
+
+### Changed
+
+- **README-SQLite.md**: Updated all examples to include `-y` flag and added notes explaining that interactive mode only works with PostgreSQL sources.
+
 ## [7.0.4] - 2025-12-09
 
 ### Fixed

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "database-replicator"
-version = "7.0.4"
+version = "7.0.5"
 edition = "2021"
 license = "Apache-2.0"
 description = "Universal database-to-PostgreSQL replication CLI. Supports PostgreSQL, SQLite, MongoDB, and MySQL."

Cargo.toml

@@ -11,12 +11,14 @@ This guide explains how to replicate SQLite databases to PostgreSQL using databa
 **Note:** SQLite replication requires local execution (the `--local` flag) since the source file must be accessible from your machine.
 
 ```bash
-database-replicator init \
+database-replicator init -y \
   --source /path/to/database.db \
   --target "postgresql://user:pass@your-db.serendb.com:5432/db" \
   --local
 ```
 
+**Important:** The `-y` flag (or `--no-interactive`) is required for SQLite sources to skip the interactive database selection prompt, which only works with PostgreSQL sources.
+
 ---
 
 ## Installation
@@ -74,11 +76,13 @@ The tool automatically detects SQLite database files and replicates them to Post
 Replicate an entire SQLite database to PostgreSQL:
 
 ```bash
-database-replicator init \
+database-replicator init -y \
   --source /path/to/database.db \
   --target "postgresql://user:pass@target-host:5432/db"
 ```
 
+> **Note:** The `-y` flag skips interactive mode, which is required for SQLite sources.
+
 The tool automatically:
 1. Detects that the source is SQLite (from `.db` extension)
 2. Validates the file path for security
@@ -249,7 +253,7 @@ Table names are validated before use in SQL:
 wget https://example.com/sample.db
 
 # Migrate to PostgreSQL
-database-replicator init \
+database-replicator init -y \
   --source ./sample.db \
   --target "postgresql://localhost:5432/mydb"
 ```
@@ -454,7 +458,7 @@ FROM users;
    sqlite3 test.db < test_data.sql
 
    # Test replication
-   database-replicator init --source test.db --target "postgresql://localhost/test"
+   database-replicator init -y --source test.db --target "postgresql://localhost/test"
    ```
 
 3. **Verify PostgreSQL Space**

README-SQLite.md

@@ -255,7 +255,12 @@ async fn main() -> anyhow::Result<()> {
                 anyhow::anyhow!("Target database URL not provided and not set in state. Use `--target` or `database-replicator target set`.")
             })?;
 
-            let filter = if !no_interactive {
+            // Detect source type - interactive mode only works with PostgreSQL
+            let source_type = database_replicator::detect_source_type(&source)
+                .context("Failed to detect source database type")?;
+            let is_postgres_source = matches!(source_type, database_replicator::SourceType::PostgreSQL);
+
+            let filter = if !no_interactive && is_postgres_source {
                 // Interactive mode (default) - prompt user to select databases and tables
                 let (filter, rules) =
                     database_replicator::interactive::select_databases_and_tables(&source).await?;
@@ -324,17 +329,23 @@ async fn main() -> anyhow::Result<()> {
                 || include_tables.is_some()
                 || exclude_tables.is_some();
 
+            // Detect source type early to determine if interactive mode is supported
+            let source_type = database_replicator::detect_source_type(&source)
+                .context("Failed to detect source database type")?;
+            let is_postgres_source = matches!(source_type, database_replicator::SourceType::PostgreSQL);
+
             // Interactive mode is default unless:
             // - --no-interactive flag is set
             // - --yes flag is set (implies automation)
             // - CLI filter flags are provided
+            // - Source is not PostgreSQL (interactive mode only works with PostgreSQL sources)
             // Run this BEFORE remote execution check so interactive mode works for both local and remote
             let (
                 final_include_databases,
                 final_exclude_databases,
                 final_include_tables,
                 final_exclude_tables,
-            ) = if !no_interactive && !yes && !has_cli_filters {
+            ) = if !no_interactive && !yes && !has_cli_filters && is_postgres_source {
                 // Interactive mode (default) - prompt user to select databases and tables
                 let (filter, _rules) =
                     database_replicator::interactive::select_databases_and_tables(&source).await?;
@@ -525,7 +536,12 @@ async fn main() -> anyhow::Result<()> {
                 || include_tables.is_some()
                 || exclude_tables.is_some();
 
-            let filter = if !no_interactive && !has_cli_filters {
+            // Detect source type - interactive mode only works with PostgreSQL
+            let source_type = database_replicator::detect_source_type(&source)
+                .context("Failed to detect source database type")?;
+            let is_postgres_source = matches!(source_type, database_replicator::SourceType::PostgreSQL);
+
+            let filter = if !no_interactive && !has_cli_filters && is_postgres_source {
                 // Interactive mode (default) - prompt user to select databases and tables
                 let (filter, rules) =
                     database_replicator::interactive::select_databases_and_tables(&source).await?;