fastapi-sqlalchemy-pg-catalog: parameterize README + SQL_ECHO env knob

Addresses two Copilot review nits on #102:

* README.md: the local-repro snippet hardcoded
  `--container-name pg-catalog-repro-app`, but the compose file uses
  `${APP_CONTAINER:-pg-catalog-repro-app}` so a user who overrides
  APP_CONTAINER (e.g. to isolate concurrent runs) would have keploy
  point at a non-existent container. Both keploy invocations now
  thread `${APP_CONTAINER:-pg-catalog-repro-app}` so they pick up
  the same env override compose sees. Also switched the deprecated
  camelCase `--apiTimeout`/`--disableMockUpload` to the kebab forms
  the v3 CLI actually registers (`--api-timeout`; mock-upload flag
  dropped — not registered on v3-dev `test`).

* main.py: SQL echo was unconditional. It is INTENTIONALLY on by
  default for this sample (the whole point is to surface SQLAlchemy's
  pg_class probe + create_all behaviour in the app log so a reader
  can correlate it with the keploy agent log), but the noise is real
  for unrelated investigations. Gated behind SQL_ECHO env var with a
  comment explaining why it's on by default.

Signed-off-by: Akash Kumar <meakash7902@gmail.com>

Author: AkashKumar7902

fastapi-sqlalchemy-pg-catalog/README.md

@@ -43,19 +43,23 @@ bash flow.sh
 docker compose down -v
 
 # Record
+# Both keploy invocations below pass `--container-name "${APP_CONTAINER:-pg-catalog-repro-app}"`
+# so they track whatever the compose file is rendering for the app
+# service. If you've overridden APP_CONTAINER (e.g. to isolate
+# concurrent runs), the same export reaches both keploy and compose.
 ( bash flow.sh > flow-record.log 2>&1 ) &
 sudo -E keploy record \
   -c "docker compose -f docker-compose.yml up" \
-  --container-name pg-catalog-repro-app \
+  --container-name "${APP_CONTAINER:-pg-catalog-repro-app}" \
   --cmd-type docker-compose \
   --record-timer 60s
 
 # Replay (pre-fix: FAILS with "no recorded invocation matched" on CREATE TABLE)
 sudo -E keploy test \
   -c "docker compose -f docker-compose.yml up" \
-  --container-name pg-catalog-repro-app \
+  --container-name "${APP_CONTAINER:-pg-catalog-repro-app}" \
   --cmd-type docker-compose \
-  --apiTimeout 120 --delay 15 --disableMockUpload
+  --api-timeout 120 --delay 15
 ```
 
 ## Layout

fastapi-sqlalchemy-pg-catalog/app/main.py

@@ -33,6 +33,15 @@
 log = logging.getLogger("repro")
 
 DATABASE_URL = os.environ["DATABASE_URL"]
+# SQL echo is INTENTIONALLY on by default — this is a sample for
+# demonstrating the dispatcher's simple-Query catalog path, and seeing
+# the actual SQLAlchemy queries (pg_catalog.version, pg_class probe,
+# CREATE TABLE on miss) in the app log is the load-bearing observation
+# that lets a reader correlate the keploy agent log with what the app
+# is doing. The trade-off: SQLAlchemy logs every statement at INFO,
+# which is verbose in normal operation. Override SQL_ECHO=0 to quiet
+# it down for unrelated investigations.
+SQL_ECHO = os.environ.get("SQL_ECHO", "1") != "0"
 
 Base = declarative_base()
 
@@ -44,7 +53,7 @@ class Project(Base):
     name = Column(String(100), nullable=False)
 
 
-engine = create_engine(DATABASE_URL, echo=True, future=True)
+engine = create_engine(DATABASE_URL, echo=SQL_ECHO, future=True)
 
 
 @asynccontextmanager