AI disclaimer: one shared component, shown where model output appears
"Freezmo uses AI and can make mistakes. Double-check important info." (plus a natural informal Czech translation), rendered by a shared AiDisclaimer so the copy, style and translations live in one place. Placements — only where the model's words reach the screen: - Chat: under the composer, below the context/model chips — always (chat is pure model output, so it also shows in the empty state). - Make my homework: under the start CTA (desktop) and the mode cards' helper line (phone), plus once under the generated result for all three modes (make / check / guide). - New deck wizard: under the summary helper (phone step 3) and the desktop CTA, next to Freeze my deck. - AI grading (short answers): under the graded feedback; study, review and exam all share it. Stored, already-generated content (deck page, study cards, explanations) deliberately carries no note: the disclaimer belongs to the moment of generation, not to browsing. Style: --text-subtle (documented at 5.6:1 light / 8.6:1 dark), 11.5px, centred, wraps to two lines at 360px, not clickable, no icon. Suites: chat (composer, phone + desktop), wizard (deck step 3 across 360/390 x en/cs x dark/light, homework start cards), phase3 (homework trigger + result, phone + desktop), phase2 (stubbed grading feedback). SW -> freezmo-v37. README: decision 50 + chat composer note. Also fixes the deploy script's test instance: it now execs the server so the cleanup kill reaps the real PID (the wrapper subshell left orphans squatting on the test port — three from earlier sessions were found while verifying this task), and the readiness poll checks the port serves the NEW commit rather than just answering, so a leftover instance can never be tested silently.
This commit is contained in:
parent
05e0fdcc86
commit
37a84df462
14 changed files with 151 additions and 34 deletions
|
|
@ -126,14 +126,25 @@ if [[ "$RUN_TESTS" == "1" ]]; then
|
|||
# shellcheck disable=SC1090
|
||||
source "$PROD/shared/env.local"
|
||||
set +a
|
||||
( cd "$DIR/server" && PORT="$TEST_PORT" HOSTNAME=127.0.0.1 node server.js >"$DIR/test-server.log" 2>&1 ) &
|
||||
# exec so $! is the server itself, not a wrapper subshell — otherwise the
|
||||
# cleanup kill only reaps the subshell and the server survives as an orphan
|
||||
# squatting on the test port.
|
||||
( cd "$DIR/server" && exec env PORT="$TEST_PORT" HOSTNAME=127.0.0.1 node server.js >"$DIR/test-server.log" 2>&1 ) &
|
||||
TEST_PID=$!
|
||||
cleanup_test() { kill "$TEST_PID" >/dev/null 2>&1 || true; wait "$TEST_PID" 2>/dev/null || true; }
|
||||
trap cleanup_test EXIT
|
||||
|
||||
# Ready means the port answers with THIS release's commit: a leftover
|
||||
# instance from an earlier run would otherwise be tested silently.
|
||||
ready=0
|
||||
for _ in $(seq 1 60); do
|
||||
if curl -fsS --max-time 2 "http://127.0.0.1:$TEST_PORT/api/healthz" >/dev/null 2>&1; then ready=1; break; fi
|
||||
body="$(curl -fsS --max-time 2 "http://127.0.0.1:$TEST_PORT/api/healthz" 2>/dev/null || true)"
|
||||
if [[ -n "$body" ]]; then
|
||||
commit="$(printf '%s' "$body" | grep -o '"commit":"[^"]*"' | cut -d'"' -f4 || true)"
|
||||
if [[ -n "$commit" && "${commit:0:7}" == "${TARGET:0:7}" ]]; then ready=1; break; fi
|
||||
cat "$DIR/test-server.log" >&2 || true
|
||||
fail "port $TEST_PORT answers with ${commit:0:7}, not $SHORT — is another instance still running?"
|
||||
fi
|
||||
sleep 1
|
||||
done
|
||||
[[ "$ready" == "1" ]] || { cat "$DIR/test-server.log" >&2; fail "the new release did not start on :$TEST_PORT"; }
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue