Skip to content

Commit 6cb4a9d

Browse files
Gustav Louvclaude
andcommitted
fix(spaerre): "der var fejl" er ikke det samme som "kunne ikke koere"
Spaerren jeg lige har lagt ind blokerede sin foerste rigtige udgivelse. Flow-testen returnerer en fejlkode naar der ER fejl — ogsaa naar det kun er den ene KENDTE. Mit `if ! npm run flow` laeste enhver fejlkode som "testen kunne slet ikke koere" og stoppede med "Er Chrome aaben med udvidelsen indlaest?", mens outputtet lige over sagde 50 OK / 1 FEJL. DAEKNING-linjen er beviset paa at testen naaede hele vejen igennem. Den skelner nu. Bevist i praksis bagefter: samme koersel gav ◦ kendt fejl, accepteret: browser_handle_dialog ✅ ingen nye fejl — spaerren giver groent lys og udgivelsen gik videre til Chrome Web Store. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 778654e commit 6cb4a9d

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

scripts/publish-cws.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,11 @@ if [[ "${SPRING_FLOW_OVER:-}" == "1" ]]; then
6161
else
6262
echo "→ Flow-test mod en aegte Chrome (spaerre foer udgivelse)"
6363
FLOW_UD="$(mktemp)"
64-
if ! npm --prefix mcp-server run flow > "$FLOW_UD" 2>&1; then
64+
# Flow-testen returnerer en fejlkode naar der ER fejl. Det er IKKE det samme som at
65+
# den ikke kunne koere — den skelnen kostede en blokeret udgivelse 30/8. DAEKNING-
66+
# linjen er beviset paa at den naaede hele vejen igennem.
67+
npm --prefix mcp-server run flow > "$FLOW_UD" 2>&1 || true
68+
if ! grep -q "^DAEKNING:" "$FLOW_UD"; then
6569
echo " Flow-testen kunne slet ikke koere:"; tail -20 "$FLOW_UD"
6670
echo " Er Chrome aaben med udvidelsen indlaest?"; exit 1
6771
fi

0 commit comments

Comments
 (0)