Merge branch 'main' into feat-changeCapture-Updates

Author: AubaidAhmedSaiyed

docs/docs/core/cli-commands.md

@@ -108,7 +108,7 @@ cocoindex server [OPTIONS] APP_TARGET
 | `-ci, --cors-cocoindex` | Allow `https://cocoindex.io` to access the server. |
 | `-cl, --cors-local INTEGER` | Allow `http://localhost:<port>` to access the server. |
 | `-L, --live-update` | Continuously watch changes from data sources and apply to the target index. |
-| `--setup` | Automatically setup backends for the flow if it's not setup yet. |
+| `--setup` | (DEPRECATED) Automatically setup backends for the flow if it's not setup yet. This is now the default behavior.  [default: True] |
 | `--reset` | Drop existing setup before starting server (equivalent to running 'cocoindex drop' first). `--reset` implies `--setup`. |
 | `--reexport` | Reexport to targets even if there's no change. |
 | `-f, --force` | Force setup without confirmation prompts. |
@@ -194,7 +194,7 @@ cocoindex update [OPTIONS] APP_FLOW_SPECIFIER
 |--------|-------------|
 | `-L, --live` | Continuously watch changes from data sources and apply to the target index. |
 | `--reexport` | Reexport to targets even if there's no change. |
-| `--setup` | Automatically setup backends for the flow if it's not setup yet. |
+| `--setup` | (DEPRECATED) Automatically setup backends for the flow if it's not setup yet. This is now the default behavior.  [default: True] |
 | `--reset` | Drop existing setup before updating (equivalent to running 'cocoindex drop' first). `--reset` implies `--setup`. |
 | `-f, --force` | Force setup without confirmation prompts. |
 | `-q, --quiet` | Avoid printing anything to the standard output, e.g. statistics. |

python/cocoindex/cli.py

@@ -182,8 +182,7 @@ def ls(app_target: str | None) -> None:
     "--color/--no-color", default=True, help="Enable or disable colored output."
 )
 @click.option("--verbose", is_flag=True, help="Show verbose output with full details.")
-@click.option("--live-status", is_flag=True, help="Show live update status for each data source.")
-def show(app_flow_specifier: str, color: bool, verbose: bool, live_status: bool) -> None:
+def show(app_flow_specifier: str, color: bool, verbose: bool) -> None:
     """
     Show the flow spec and schema.
 
@@ -204,15 +203,6 @@ def show(app_flow_specifier: str, color: bool, verbose: bool, live_status: bool)
     console = Console(no_color=not color)
     console.print(fl._render_spec(verbose=verbose))
     console.print()
-    
-    if live_status:
-        # Show live update status
-        console.print("\n[bold cyan]Live Update Status:[/bold cyan]")
-        options = flow.FlowLiveUpdaterOptions(live_mode=False, reexport_targets=False, print_stats=False)
-        with flow.FlowLiveUpdater(fl, options) as updater:
-            updater.print_cli_status()
-        console.print()
-    
     table = Table(
         title=f"Schema for Flow: {fl.name}",
         title_style="cyan",
@@ -261,6 +251,23 @@ def _drop_flows(flows: Iterable[flow.Flow], app_ref: str, force: bool = False) -
     setup_bundle.apply(report_to_stdout=True)
 
 
+def _deprecate_setup_flag(
+    ctx: click.Context, param: click.Parameter, value: bool
+) -> bool:
+    """Callback to warn users that --setup flag is deprecated."""
+    # Check if the parameter was explicitly provided by the user
+    if param.name is not None:
+        param_source = ctx.get_parameter_source(param.name)
+        if param_source == click.core.ParameterSource.COMMANDLINE:
+            click.secho(
+                "Warning: The --setup flag is deprecated and will be removed in a future version. "
+                "Setup is now always enabled by default.",
+                fg="yellow",
+                err=True,
+            )
+    return value
+
+
 def _setup_flows(
     flow_iter: Iterable[flow.Flow],
     *,
@@ -285,22 +292,6 @@ def _setup_flows(
     setup_bundle.apply(report_to_stdout=not quiet)
 
 
-def _deprecate_setup_flag(
-    ctx: click.Context, param: click.Parameter, value: bool
-) -> bool:
-    """Callback to warn users that --setup flag is deprecated."""
-    if param.name is not None:
-        param_source = ctx.get_parameter_source(param.name)
-        if param_source == click.core.ParameterSource.COMMANDLINE:
-            click.secho(
-                "Warning: The --setup flag is deprecated and will be removed in a future version. "
-                "Setup is now always enabled by default.",
-                fg="yellow",
-                err=True,
-            )
-    return value
-
-
 def _show_no_live_update_hint() -> None:
     click.secho(
         "NOTE: No change capture mechanism exists. See https://cocoindex.io/docs/core/flow_methods#live-update for more details.\n",
@@ -488,10 +479,6 @@ def update(
     else:
         assert len(flow_list) == 1
         with flow.FlowLiveUpdater(flow_list[0], options) as updater:
-            if options.live_mode:
-                # Show initial status
-                updater.print_cli_status()
-                click.echo()
             updater.wait()
             if options.live_mode:
                 _show_no_live_update_hint()