Replace 27 debug print() statements with proper logging #3

New Issue

llabeyrie · 2026-03-19T17:29:52Z

llabeyrie commented

2026-03-19 17:29:52 +00:00

Description

prompt_to_card_pipeline.py contains 27 unconditional print() statements with informal debug messages scattered throughout the file:

"module successfully charged"
"model charging 1" / "model charged 1"
"model charge 2" / "model charged 2"
"getting module" / "module got"
"building pipeline" / "pipeline build"
"will start result" / "result is done"
"json parsed with success"
"main get clean text" / "main got clean text"
...

Affected lines: 40, 45, 47, 57, 78, 91, 107, 116, 121, 167, 169, 174, 178, 188, 190, 208, 210, 299, 302, 305, 308, 316, 317, 326, 334

Problems

Pollutes stdout, which is also used to parse structured output (JSON, image paths)
Breaks --print-json and --json-only output when piped or parsed by app.py
Informal messages (mix of English/French) look unprofessional
No way to enable/disable them (no log level control)

Fix

Replace all print() calls with Python's logging module:

import logging
logger = logging.getLogger(__name__)

logger.debug("Module loaded from %s", module_path)

## Description `prompt_to_card_pipeline.py` contains **27 unconditional `print()` statements** with informal debug messages scattered throughout the file: ``` "module successfully charged" "model charging 1" / "model charged 1" "model charge 2" / "model charged 2" "getting module" / "module got" "building pipeline" / "pipeline build" "will start result" / "result is done" "json parsed with success" "main get clean text" / "main got clean text" ... ``` **Affected lines:** 40, 45, 47, 57, 78, 91, 107, 116, 121, 167, 169, 174, 178, 188, 190, 208, 210, 299, 302, 305, 308, 316, 317, 326, 334 ### Problems - Pollutes stdout, which is also used to parse structured output (JSON, image paths) - Breaks `--print-json` and `--json-only` output when piped or parsed by `app.py` - Informal messages (mix of English/French) look unprofessional - No way to enable/disable them (no log level control) ### Fix Replace all `print()` calls with Python's `logging` module: ```python import logging logger = logging.getLogger(__name__) logger.debug("Module loaded from %s", module_path) ```

llabeyrie referenced this issue

2026-03-19 17:30:28 +00:00

Fragile JSON extraction with silent fallback in prompt_to_card_pipeline.py #9

llabeyrie referenced this issue

2026-03-19 17:31:11 +00:00

Unsafe path traversal in app.py image extraction #15

llabeyrie added the code-quality priority: high labels 2026-03-19 17:31:38 +00:00

llabeyrie referenced this issue from a commit

2026-03-19 18:45:22 +00:00

Replace debug print() statements with proper logging

llabeyrie referenced a pull request that will close this issue

2026-03-19 18:45:39 +00:00