Location
client_discovery/service.py — read_sample_questionnaire() (~line 19-21)
Problem
def read_sample_questionnaire() -> str:
return (BASE_DIR / "tests" / "fixtures" / "complete_questionnaire.md").read_text(encoding="utf-8")
This reads only from tests/fixtures/..., but .dockerignore excludes tests/ from the container image. If this module is used in Cloud Run (or any build that omits tests/), the sample loader will crash with FileNotFoundError.
Fix
Mirror the app's runtime-first sample lookup — ship the sample under samples/sample_questionnaire.md (or similar runtime location) and use tests/fixtures/... only as a fallback during local/test runs.
Source
Originally raised in review of PR #21 (now docs-only): outdated thread PRRT_kwDOSzLAqc6JYBhc. Confirmed still present on main.
/tracked-issue
Location
client_discovery/service.py—read_sample_questionnaire()(~line 19-21)Problem
This reads only from
tests/fixtures/..., but.dockerignoreexcludestests/from the container image. If this module is used in Cloud Run (or any build that omitstests/), the sample loader will crash withFileNotFoundError.Fix
Mirror the app's runtime-first sample lookup — ship the sample under
samples/sample_questionnaire.md(or similar runtime location) and usetests/fixtures/...only as a fallback during local/test runs.Source
Originally raised in review of PR #21 (now docs-only): outdated thread
PRRT_kwDOSzLAqc6JYBhc. Confirmed still present onmain./tracked-issue