Prøven, der skal holde udgaverne adskilt, stod grøn gennem to merges, mens
34 opgaveruter blev betjent af noter-udgaven. Den fejlede ikke: den
sammenligner routeren med en nedskreven liste, og listen var skrevet ud
fra de ruter, nogen havde tænkt på. Den er en DRIFT-kontrol, og den kan
ikke være en fuldstændighedskontrol. Det står nu i prøven selv.
Lukket: CalDAV-træet (8 ruter, alle læser eller skriver en VTODO),
.well-known/caldav (en påstand om at der er en CalDAV-server), ICS-feedet
og de to ruter til dets token, de fire opgave-eksporter, Todoist- og
CSV-importen, skabelonerne, sektionerne, de seks AI-opgaveruter og
indbakke-krogen. Plus fem af ni MCP-værktøjer — endepunktet er der
stadig, værktøjslisten er kortere.
Tre blev afgjort ved at læse frem for ved navnet: en skabelon laver
opgaver (CreateProjectFromTemplate), `section_id` står på `tasks` og
findes ikke på `notes`, og add_comment hænger på en opgave.
Men den dyre halvdel var fladen. En vandring gennem hver side, der kan
nås i noter-udgaven, med konsollen og svarene aflæst, fandt SYV brud —
fire af dem fra den første port:
sidebjælken GET /filters 404 ved hver navigation, ædt af .catch()
sidebjælken GET /labels 404 ved hver navigation, ædt af .catch()
notifikationer GET /ai/plan 404, stille
notesbogens side GET /tasks kastede → status = 'failed'
notesbogens side GET /sections kastede → samme fejlskærm
indstillinger GET /templates toast med fejlbesked på skærmen
integrationer GET /feed 404, stille
Den fjerde er den, der har kostet noget: /projekt/{id} er notesbogens
side, og kaldene samles i et Promise.all, så én 404 tog hele siden med.
Den har vist en fejlskærm frem for noter siden porten blev sat. Ingen
prøve åbnede en notesbog i den udgave, og .catch(() => {}) sørgede for,
at resten ikke sagde noget. "Fladen følger udgaven" var sandt om menuen
og falsk om hentningerne.
Vagten for opgaver står i butikken frem for ved kaldestederne: der var
tre, og den fjerde blev glemt — hvilket er præcis sådan, den slap
igennem.
Og rettelsen lavede selv en fejl, som vandringen fangede: da opgavekaldet
holdt op med at fejle, kom en uendelig effekt-løkke frem, som 404'eren
havde holdt låg på. En ubetinget tilskrivning til app.tasks får
projektsidens effekt til at kalde tilbage. 404'eren var låget på en
løkke, ikke bare støj.
Også: -timeout 20m i begge CI-workflows. internal/httpapi tog 664s her og
498s i CI på #95, mod Go's standardgrænse på 600 — og når den krydses, er
beskeden "panic: test timed out after 10m0s" plus et goroutine-dump, som
læser som en deadlock i koden frem for som en suite, der blev for stor.
watchForTrouble udskilt til web/e2e/trouble.js, så udgave-suiten kan bruge
den samme.
Co-Authored-By: Claude Opus 5 <[email protected]>
280 lines
13 KiB
YAML
280 lines
13 KiB
YAML
name: CI
|
|
|
|
# Gitea-tvillingen til .github/workflows/ci.yml. Den prøver at nå samme resultat
|
|
# ad en anden vej, og forskellen er hele pointen med filen:
|
|
#
|
|
# INGEN `uses:`. Gitea Actions kan læse GitHubs format, men `DEFAULT_ACTIONS_URL`
|
|
# står ikke i app.ini på gitea.nolimit.dk, og standarden er github.com — så et
|
|
# `uses: actions/checkout@v4` her ville blive hentet FRA GitHub, hver gang, også
|
|
# selvom både koden, runneren og registret er vores egne. Et workflow, der bruger
|
|
# markedspladsen, er derfor ikke uafhængigt af GitHub; det ser bare sådan ud.
|
|
#
|
|
# Prisen er, at værktøjskæden skal komme et sted fra. Den kommer fra jobbets
|
|
# container, hvilket er billigere end det lyder: `container: golang:1.26` er
|
|
# nøjagtig det, setup-go ellers ville bruge et trin på at installere.
|
|
#
|
|
# Alternativerne, hvis nogen senere vil skrive filen om: sæt DEFAULT_ACTIONS_URL
|
|
# til gitea.com, som spejler de gængse handlinger — men det flytter kun
|
|
# afhængigheden fra én andens server til en andens, og det er en indstilling for
|
|
# HELE instansen, altså også for alt andet, der bygger der. Denne fil rører kun
|
|
# dette repo.
|
|
|
|
on:
|
|
push:
|
|
branches: [main]
|
|
pull_request:
|
|
# Så en kørsel kan udløses uden at skulle opfinde en commit. Det er ikke
|
|
# bekvemmelighed: uden den er den eneste måde at få CI til at køre igen mod
|
|
# uændret kode at skubbe noget tomt, og en tom commit i historikken er en
|
|
# løgn om, at der skete noget.
|
|
workflow_dispatch:
|
|
|
|
# Ingen `concurrency:`-blok. GitHub-udgaven aflyser en kørsel, når en nyere
|
|
# kommer, og det er stadig det rigtige at gøre — men understøttelsen kom sent til
|
|
# Gitea Actions, og et felt, runneren ikke kender, er en fejl ved oversættelsen
|
|
# frem for en ignoreret linje. Den kan sættes på, når instansen er en version,
|
|
# der kan.
|
|
|
|
jobs:
|
|
go:
|
|
name: Go
|
|
runs-on: ubuntu-latest
|
|
# Containeren ER værktøjskæden. `-race` kræver en C-oversætter, og den er med
|
|
# i det officielle golang-image — et slankere ville koste et apt-trin.
|
|
container:
|
|
image: golang:1.26
|
|
steps:
|
|
- name: Checkout
|
|
env:
|
|
TOKEN: ${{ secrets.GITEA_TOKEN }}
|
|
run: |
|
|
set -eu
|
|
# Håndrullet, fordi actions/checkout ville blive hentet fra GitHub.
|
|
# Det er de samme fire greb, den handling tager: init, remote, en
|
|
# fetch af præcis den commit, der udløste kørslen, og et checkout.
|
|
#
|
|
# Legitimationen sættes med -c på selve fetch-kommandoen frem for i
|
|
# remotens adresse. En adresse med token i havner i .git/config, i
|
|
# `git remote -v` og i enhver fejlbesked, der citerer den; en header
|
|
# gør ikke.
|
|
git config --global --add safe.directory '*'
|
|
git init -q .
|
|
git remote add origin "$GITHUB_SERVER_URL/$GITHUB_REPOSITORY.git"
|
|
# To former for basic auth, fordi kun en kørende runner kan svare på,
|
|
# hvilken Gitea tager imod. actions/checkout bruger x-access-token, og
|
|
# det er den, økosystemet beviser virker; $GITHUB_ACTOR er reserven.
|
|
# Løkken skriver, hvilken der blev godkendt — så ÉN kørsel besvarer
|
|
# spørgsmålet, uanset om den ender grøn eller rød. Feedback-sløjfen her
|
|
# går gennem en kø og et andet menneske; at bruge en hel runde på at få
|
|
# ét bit at vide er for dyrt.
|
|
godkendt=
|
|
for bruger in x-access-token "$GITHUB_ACTOR"; do
|
|
header="Authorization: Basic $(printf '%s:%s' "$bruger" "$TOKEN" | base64 | tr -d '\n')"
|
|
if git -c http.extraheader="$header" fetch -q --depth=1 origin "$GITHUB_SHA"; then
|
|
godkendt=$bruger
|
|
break
|
|
fi
|
|
done
|
|
if [ -z "$godkendt" ]; then
|
|
echo "checkout: hverken x-access-token eller $GITHUB_ACTOR blev accepteret"
|
|
exit 1
|
|
fi
|
|
echo "checkout: godkendt som $godkendt"
|
|
git checkout -q FETCH_HEAD
|
|
|
|
- name: Formatting
|
|
run: |
|
|
unformatted=$(gofmt -l .)
|
|
if [ -n "$unformatted" ]; then
|
|
echo "These files are not gofmt'd:"
|
|
echo "$unformatted"
|
|
exit 1
|
|
fi
|
|
|
|
- name: Vet
|
|
run: go vet ./...
|
|
|
|
# -race is the whole point of running tests in CI for a server with a
|
|
# WebSocket fan-out: a data race is exactly the bug that passes locally.
|
|
- name: Test
|
|
# -timeout, fordi internal/httpapi nærmer sig Go's standardgrænse på 600s:
|
|
# 498s i CI på #95, 656s på udviklingsmaskinen. Når den krydser, er
|
|
# beskeden "panic: test timed out after 10m0s" plus et goroutine-dump, og
|
|
# det læser som en deadlock i koden frem for som en suite, der blev for
|
|
# stor. Grænsen er der stadig; den er bare sat et sted, hvor den betyder
|
|
# "noget hænger" i stedet for "maskinen var langsom i dag".
|
|
run: go test -race -count=1 -timeout 20m ./...
|
|
|
|
- name: Build
|
|
run: CGO_ENABLED=0 go build ./...
|
|
|
|
spec:
|
|
name: OpenAPI is valid
|
|
runs-on: ubuntu-latest
|
|
container:
|
|
image: node:22
|
|
steps:
|
|
- name: Checkout
|
|
env:
|
|
TOKEN: ${{ secrets.GITEA_TOKEN }}
|
|
run: |
|
|
set -eu
|
|
git config --global --add safe.directory '*'
|
|
git init -q .
|
|
git remote add origin "$GITHUB_SERVER_URL/$GITHUB_REPOSITORY.git"
|
|
# To former for basic auth, fordi kun en kørende runner kan svare på,
|
|
# hvilken Gitea tager imod. actions/checkout bruger x-access-token, og
|
|
# det er den, økosystemet beviser virker; $GITHUB_ACTOR er reserven.
|
|
# Løkken skriver, hvilken der blev godkendt — så ÉN kørsel besvarer
|
|
# spørgsmålet, uanset om den ender grøn eller rød. Feedback-sløjfen her
|
|
# går gennem en kø og et andet menneske; at bruge en hel runde på at få
|
|
# ét bit at vide er for dyrt.
|
|
godkendt=
|
|
for bruger in x-access-token "$GITHUB_ACTOR"; do
|
|
header="Authorization: Basic $(printf '%s:%s' "$bruger" "$TOKEN" | base64 | tr -d '\n')"
|
|
if git -c http.extraheader="$header" fetch -q --depth=1 origin "$GITHUB_SHA"; then
|
|
godkendt=$bruger
|
|
break
|
|
fi
|
|
done
|
|
if [ -z "$godkendt" ]; then
|
|
echo "checkout: hverken x-access-token eller $GITHUB_ACTOR blev accepteret"
|
|
exit 1
|
|
fi
|
|
echo "checkout: godkendt som $godkendt"
|
|
git checkout -q FETCH_HEAD
|
|
|
|
# Structural validity only. That the spec matches the *router* is checked by
|
|
# a Go test, which is the half that actually goes stale.
|
|
- run: npx --yes @redocly/cli@latest lint docs/openapi.yaml
|
|
|
|
e2e:
|
|
name: End-to-end
|
|
runs-on: ubuntu-latest
|
|
# node frem for golang her: Playwrights browsere trækker et lag
|
|
# Debian-pakker, som npm-imaget er vant til at have, mens Go til gengæld er
|
|
# fire linjer at hælde ind. Den anden vej rundt er den dyre.
|
|
container:
|
|
image: node:22
|
|
steps:
|
|
- name: Checkout
|
|
env:
|
|
TOKEN: ${{ secrets.GITEA_TOKEN }}
|
|
run: |
|
|
set -eu
|
|
git config --global --add safe.directory '*'
|
|
git init -q .
|
|
git remote add origin "$GITHUB_SERVER_URL/$GITHUB_REPOSITORY.git"
|
|
# To former for basic auth, fordi kun en kørende runner kan svare på,
|
|
# hvilken Gitea tager imod. actions/checkout bruger x-access-token, og
|
|
# det er den, økosystemet beviser virker; $GITHUB_ACTOR er reserven.
|
|
# Løkken skriver, hvilken der blev godkendt — så ÉN kørsel besvarer
|
|
# spørgsmålet, uanset om den ender grøn eller rød. Feedback-sløjfen her
|
|
# går gennem en kø og et andet menneske; at bruge en hel runde på at få
|
|
# ét bit at vide er for dyrt.
|
|
godkendt=
|
|
for bruger in x-access-token "$GITHUB_ACTOR"; do
|
|
header="Authorization: Basic $(printf '%s:%s' "$bruger" "$TOKEN" | base64 | tr -d '\n')"
|
|
if git -c http.extraheader="$header" fetch -q --depth=1 origin "$GITHUB_SHA"; then
|
|
godkendt=$bruger
|
|
break
|
|
fi
|
|
done
|
|
if [ -z "$godkendt" ]; then
|
|
echo "checkout: hverken x-access-token eller $GITHUB_ACTOR blev accepteret"
|
|
exit 1
|
|
fi
|
|
echo "checkout: godkendt som $godkendt"
|
|
git checkout -q FETCH_HEAD
|
|
|
|
- name: Go
|
|
run: |
|
|
set -eu
|
|
# Versionen læses ud af go.mod frem for at stå her. To steder at rette
|
|
# er ét for mange, og den slags glemmes lige indtil CI bygger med en
|
|
# anden Go end udviklerne.
|
|
version=$(awk '/^go /{print $2; exit}' go.mod)
|
|
arch=$(dpkg --print-architecture)
|
|
curl -fsSL "https://go.dev/dl/go${version}.linux-${arch}.tar.gz" | tar -xz -C /usr/local
|
|
echo /usr/local/go/bin >> "$GITHUB_PATH"
|
|
|
|
- name: Install
|
|
working-directory: web
|
|
run: npm ci --no-audit --no-fund
|
|
|
|
# Chromium for the suite, WebKit for one test.
|
|
#
|
|
# A smoke test exists to catch a build that does not work at all, and running
|
|
# the whole thing in three engines finds the same breakage three times for
|
|
# three times the cost. Focus is the exception, and it took a real report to
|
|
# learn it: Safari ignores `autofocus` on anything inserted after the page has
|
|
# loaded, so every inline form in the app opened with no cursor in it. WebKit
|
|
# here is the same engine Safari is, and `webkit-forms` in the Playwright
|
|
# config greps a single test out of the suite to run in it.
|
|
- name: Install browsers
|
|
working-directory: web
|
|
run: npx playwright install --with-deps chromium webkit
|
|
|
|
# The suite builds the frontend, embeds it and starts the real binary — see
|
|
# playwright.config.js. There is nothing to build here first.
|
|
- name: Test
|
|
working-directory: web
|
|
run: npm run test:e2e
|
|
|
|
# Ingen rapport at hente ned bagefter. GitHub-udgaven lægger
|
|
# web/playwright-report/ op som et artefakt, og det trin ER en handling fra
|
|
# markedspladsen — der er ikke en `run:`-form af det. Går suiten i stykker
|
|
# her, står grunden i loggen frem for i en zip. Skal rapporten tilbage, er
|
|
# det upload-artifact spejlet ind lokalt, ikke en linje mere i denne fil.
|
|
|
|
docker:
|
|
name: Docker image builds
|
|
runs-on: ubuntu-latest
|
|
# Ingen `container:` her: jobbet skal bruge docker-klienten og buildx, og dem
|
|
# har runnerens eget ubuntu-image. En golang- eller node-container ville have
|
|
# sproget og mangle værktøjet.
|
|
steps:
|
|
- name: Checkout
|
|
env:
|
|
TOKEN: ${{ secrets.GITEA_TOKEN }}
|
|
run: |
|
|
set -eu
|
|
git config --global --add safe.directory '*'
|
|
git init -q .
|
|
git remote add origin "$GITHUB_SERVER_URL/$GITHUB_REPOSITORY.git"
|
|
# To former for basic auth, fordi kun en kørende runner kan svare på,
|
|
# hvilken Gitea tager imod. actions/checkout bruger x-access-token, og
|
|
# det er den, økosystemet beviser virker; $GITHUB_ACTOR er reserven.
|
|
# Løkken skriver, hvilken der blev godkendt — så ÉN kørsel besvarer
|
|
# spørgsmålet, uanset om den ender grøn eller rød. Feedback-sløjfen her
|
|
# går gennem en kø og et andet menneske; at bruge en hel runde på at få
|
|
# ét bit at vide er for dyrt.
|
|
godkendt=
|
|
for bruger in x-access-token "$GITHUB_ACTOR"; do
|
|
header="Authorization: Basic $(printf '%s:%s' "$bruger" "$TOKEN" | base64 | tr -d '\n')"
|
|
if git -c http.extraheader="$header" fetch -q --depth=1 origin "$GITHUB_SHA"; then
|
|
godkendt=$bruger
|
|
break
|
|
fi
|
|
done
|
|
if [ -z "$godkendt" ]; then
|
|
echo "checkout: hverken x-access-token eller $GITHUB_ACTOR blev accepteret"
|
|
exit 1
|
|
fi
|
|
echo "checkout: godkendt som $godkendt"
|
|
git checkout -q FETCH_HEAD
|
|
|
|
# Build only — publishing happens on a tag, in release.yml. This job exists
|
|
# so a Dockerfile that no longer builds fails on the pull request rather
|
|
# than at release time.
|
|
#
|
|
# `type=cacheonly` frem for `--load`: resultatet skal ingen steder hen, og
|
|
# at hente et image ind i daemonen for straks at glemme det er et halvt
|
|
# minut hver kørsel. Én arkitektur er nok til at fange en Dockerfile, der
|
|
# ikke bygger; de to rigtige bygges på et mærkat.
|
|
- name: Build
|
|
run: |
|
|
set -eu
|
|
docker buildx create --use --name ci-verdande 2>/dev/null || docker buildx use ci-verdande
|
|
docker buildx build --platform linux/amd64 --output type=cacheonly .
|