masteriin vietäväksi

docs/ci-pipeline-practices.md

@@ -1,5 +1,3 @@
-> **⚠️ STATUS: ALERT DRAFT** — Ei ole validoitu. Voi sisältää virheellisiä tai puutteellisia käytäntöjä.
-
 # CI Pipeline Practices
 
 ## 1. Error Taxonomy
@@ -32,7 +30,24 @@ Kolme toimivaa vaihtoehtoa:
 - `docker volume create -- docker run -v volume:/data -- docker run -v volume:/data`
 - `tar c . | docker run --rm -i -v volume:/data alpine tar x -C /data`
 
-## 4. Pipeline Provides All Dependencies
+## 4. Env variable scope (validated 2026-06-13)
+
+`env:` — oli se workflow- tai job-tasolla — toimii vain **natiiveissa shell-stepeissä** ja `docker run -e VAR` -komennoissa. `docker run` ilman `-e`-lippua **ei peri** `env:`-muuttujia.
+
+Tämä on validioitu POC-ajolla: `tmp/poc-env-scope.yml`
+
+| Sijainti | Native shell | `docker run` ilman `-e` | `docker run -e VAR` |
+|----------|-------------|------------------------|---------------------|
+| workflow `env:` | ✅ perii | ❌ tyhjä | ✅ perii |
+| job `env:` | ✅ perii | ❌ tyhjä | ✅ perii |
+| `GITHUB_ENV` | ✅ perii | ❌ tyhjä | ✅ perii |
+
+Käytäntö:
+- Workflow-tason `env:` sopii arvoille, joita tarvitaan natiivistepeissä (publish, status, reportointi)
+- Jos `docker run` tarvitsee env-arvoja, välitä ne eksplisiittisesti `-e VAR`-lipulla
+- `GITHUB_ENV` on validi tapa välittää arvoja stepien välille samassa jobissa, mutta ei leviä `docker run`-kontteihin ilman `-e`-lippua
+
+## 5. Pipeline Provides All Dependencies
 
 - Ei luottamusta runnerin esiasennettuihin työkaluihin
 - `apk add`, `npm install`, `apt-get install` — kaikki pipelinesta
@@ -40,7 +55,7 @@ Kolme toimivaa vaihtoehtoa:
 - Node-version päivitettävä jos paketti vaatii uudempaa (`node:20` → `node:22`)
 - Jos kontin entrypoint on `sh` (Alpine ash), käytä `--entrypoint bash`
 
-## 5. Rakenne
+## 6. Rakenne
 
 - Rinnakkaiset jobit (bats + cucumber) — tuloksia saa heti kun valmistuu
 - Jokainen testisetti omassa jobissaan