Compare commits

..
Author SHA1 Message Date
Thore Cimbal 6632ad5a80 docs(fork): der Anschluss ist vollzogen - Abschnitt 4 war der Zustand davor
Abschnitt 4 beschreibt das Repo ohne gemeinsamen Vorfahren. Seit dem Merge am
2026-08-19 (88c4e15) stimmt das nicht mehr. Er bleibt stehen, weil er die
Messungen enthaelt, auf denen ADR-0022 beruht - jetzt aber als historisch
markiert und mit Verweis nach unten.

Neuer Abschnitt: wie ein Upstream-Update heute laeuft (gewoehnlicher Merge), was
beim Aufloesen zu pruefen ist, das Typpruefungs-Tor in der CI, und die drei
Fallstricke, die beim ersten Mal Zeit gekostet haben - shallow-Grenze, .npmrc im
Docker-Kontext, Erstschub auf den Gitea-Spiegel.

Der wichtigste Satz darin ist der ueber den umgekehrten Fall: Der Merge meldet,
wenn Upstream unsere Zeilen wegschiebt, aber nicht, wenn beide Seiten die
Aufloesung ueberleben und nur eine noch Sinn ergibt. Daran starb v0.6.0-rc.2.
2026-08-20 12:00:00 +00:00
Thore Cimbal ac343e9347 Typpruefung auf den ganzen Baum ausweiten
apps/desktop/electron-builder.ts:237 war der letzte Typfehler im eigenen Code:
config.dmg ist moeglicherweise undefined, weil Upstreams Configuration-Interface
zwar linux/win/mac/deb verpflichtend macht, dmg aber nicht. Der Wert steht
literal in config - behoben mit ! statt mit einer Zeile in Upstreams Interface,
weil der Kommentar direkt darueber genau diese Aenderung vermeiden will, um bei
Upstream-Merges keinen Konflikt zu erzeugen. Dasselbe Idiom nutzt die Datei
weiter unten fuer win.signtoolOptions.

Damit ist der Baum typrein und der Job kann von apps/web auf pnpm -r wechseln.
Bis heute brach die Baumpruefung an genau dieser Datei ab und erreichte apps/web
nie - deshalb war das Tor am 19.08. eng gefasst; die Begruendung ist hinfaellig.

desktop_linux und desktop_windows fuehren typecheck jetzt ebenfalls als needs.
Ein Typfehler im Desktop-Code haette sonst weiter ein Paket erzeugt - genau das
Loch, das fuer das Web-Image geschlossen wurde.

Massstab bleibt unveraendert: kein Fehler ausserhalb von node_modules. Upstreams
drei in matrix-js-sdk 42.2.0 bleiben ausgenommen.
2026-08-20 12:00:00 +00:00
3 changed files with 86 additions and 2 deletions
+13 -1
View File
@@ -162,7 +162,11 @@ typecheck:
# Farbcodes muessen raus: nx faerbt auch in der Pipe, und dann steht zwischen "error"
# und "TS" eine Escape-Sequenz - ein grep auf "error TS" liefe ins Leere und das Tor
# waere immer gruen, ohne je zu pruefen.
- pnpm --dir apps/web lint:types 2>&1 | sed -r 's/\x1b\[[0-9;]*[mK]//g' | tee tsc.log || true
# Ganzer Baum, nicht nur apps/web: Bis 2026-08-20 brach `pnpm -r lint:types` an
# apps/desktop/electron-builder.ts ab (config.dmg moeglicherweise undefined) und
# erreichte apps/web nie - deshalb war der Job anfangs eng gefasst. Der Fehler ist
# behoben, damit deckt das Tor auch die Desktop-Pakete ab.
- pnpm -r --workspace-concurrency=1 lint:types 2>&1 | sed -r 's/\x1b\[[0-9;]*[mK]//g' | tee tsc.log || true
- |
if grep -E "error TS[0-9]+" tsc.log | grep -v node_modules; then
echo "FEHLER: Typfehler im eigenen Code (siehe oben)."
@@ -254,6 +258,10 @@ desktop_linux:
image: registry.git.lab/axion1337.chat/threadnet-web/desktop-build:bullseye
needs:
- job: web
# Seit 2026-08-20 prueft typecheck den ganzen Baum, also auch apps/desktop -
# dann darf auch kein Desktop-Paket ohne bestandene Pruefung entstehen.
- job: typecheck
artifacts: false
artifacts: true
rules:
- if: $CI_COMMIT_TAG =~ /^v/
@@ -295,6 +303,10 @@ desktop_windows:
- windows
needs:
- job: web
# Seit 2026-08-20 prueft typecheck den ganzen Baum, also auch apps/desktop -
# dann darf auch kein Desktop-Paket ohne bestandene Pruefung entstehen.
- job: typecheck
artifacts: false
artifacts: true
rules:
- if: $CI_COMMIT_TAG =~ /^v/
+6 -1
View File
@@ -234,7 +234,12 @@ if (variant["mac.icon"]) {
}
if (variant["dmg.badgeIcon"]) {
config.dmg.badgeIcon = variant["dmg.badgeIcon"];
// `!`, weil Upstreams Configuration-Interface oben zwar linux/win/mac/deb
// verpflichtend macht, dmg aber nicht - der Wert steht literal in config.
// Dort eine Zeile zu ergaenzen waere die schoenere Loesung und genau die
// Aenderung an Upstreams Block, die der Kommentar oben vermeiden will.
// Dasselbe Idiom nutzt die Datei weiter unten fuer win.signtoolOptions.
config.dmg!.badgeIcon = variant["dmg.badgeIcon"];
}
/**
+67
View File
@@ -55,6 +55,11 @@ Beide behoben.
## 4. Merge-Reibung: was ein Upstream-Update wirklich kostet
> ⚠️ **Historisch seit 2026-08-19.** Dieser Abschnitt beschreibt den Zustand *ohne*
> gemeinsamen Vorfahren. Der ist behoben — siehe „Upstream-Update seit dem Anschluss"
> weiter unten. Der Abschnitt bleibt stehen, weil er die Messungen enthält, auf denen
> die Entscheidung beruht (management ADR-0022).
*Arbeitspaket 3 aus ThreadNet-Web#7. Gemessen am 2026-08-06, nicht geschätzt.*
### Die unangenehme Grundlage zuerst
@@ -214,3 +219,65 @@ git diff v1.12.17 v1.12.26 -- <datei> # was Upstream seither geaendert ha
⚠️ Ein flacher Fetch liefert **keine Historie**: `git merge-base`, `git log` über
Upstream und ein Graft-Versuch brauchen `git fetch upstream` ohne `--depth`.
## Upstream-Update seit dem Anschluss (seit 2026-08-19)
**Der Fork hängt wieder an der Upstream-Historie.** Am 2026-08-19 wurde `v1.12.26` mit
einem echten Merge-Commit hereingeholt (`88c4e15`, beide Eltern: unser bisheriger
`main` `fa5dcc5` und `c43ef70b` = `v1.12.26`). Entscheidung und Messungen stehen in
management ADR-0022 und #0099.
Ein Update ist damit ein **gewöhnlicher Merge** — kein Auftragen von Patches mehr:
```sh
git fetch upstream # ohne --depth, sonst fehlt die Historie
git merge v1.12.27 # oder der gewuenschte Tag
```
### Worauf beim Auflösen zu achten ist
Der Merge meldet jetzt, wenn Upstream eine Datei verschiebt, die wir angefasst haben —
das war der Zweck der Übung, und es hat beim ersten Mal funktioniert (die Umbenennung
`RoomListItemAccessibilityWrapper``RoomListItemWrapper` kam als Konflikt).
⚠️ **Was er NICHT meldet, ist der umgekehrte Fall:** Beide Seiten überleben die
Auflösung, und nur eine ergibt noch Sinn. Genau das brach `v0.6.0-rc.2` in Produktion —
in `RoomListItemViewModel.ts` blieb eine `getValue`-Zeile auf den von Upstream
entfernten Labs-Schalter `feature_room_list_sections` stehen, direkt über der neuen
Zeile mit `RoomList.showSections`. Niemand las sie, sie warf trotzdem — bei jedem
Raumlisteneintrag.
Nach dem Auflösen deshalb auf **überlebende** Reste prüfen, nicht nur auf verlorene
Zeilen. Der billigste Hebel ist `pnpm -r lint:types`; er fand beide Reste sofort. Wo
Zeichenketten statt Typen im Spiel sind — Einstellungsschlüssel, Feature-Namen,
Übersetzungs-IDs — reicht das nicht, dort braucht es einen Abgleich gegen die
jeweilige Registry.
### Das Tor in der CI
`typecheck` in `.gitlab-ci.yml` prüft `pnpm -r lint:types` über den ganzen Baum;
`docker_web`, `desktop_linux` und `desktop_windows` führen ihn als `needs`. **Kein
Artefakt entsteht ohne bestandene Typprüfung.**
Maßstab ist „kein Fehler außerhalb von `node_modules`", nicht „`tsc` sauber": Upstream
v1.12.26 ist selbst nicht typrein — `matrix-js-sdk@42.2.0` wirft drei Fehler in der
eigenen Quelle (`skipLibCheck` deckt nur `.d.ts` ab), nachgeprüft in einem sauberen
v1.12.26-Checkout. Fällt das bei einem Upstream-Wechsel weg, verschärft sich der Job
von allein.
Der Job existiert, weil `web` **nur baut**: webpack entfernt Typen, ohne sie zu prüfen,
und ein unbekannter Einstellungsschlüssel ist zur Bauzeit bloß ein String. `tsc` meldete
den rc.2-Fehler die ganze Zeit — gefragt hatte ihn niemand.
### Fallstricke, die beim ersten Mal Zeit gekostet haben
- **`--depth=1` geholte Tags** tragen eine `.git/shallow`-Grenze. Jeder Merge-Versuch
scheitert dann mit „refusing to merge unrelated histories", was wie ein
Grundproblem aussieht und keines ist. `git fetch --unshallow upstream` löst es.
- **`.npmrc` muss im Docker-Build-Kontext liegen**, sonst löst pnpm `@sorb/*` gegen
npmjs auf. Unter pnpm 10 fiel das nicht auf, weil `--frozen-lockfile` die gepinnte
Tarball-URL nahm; pnpm 11 prüft mit `minimumReleaseAgeStrict` jeden Eintrag und löst
wieder auf (management #0055).
- **Der Gitea-Spiegel** trägt nach dem Merge 70.000 fremde Commits. Der Erstschub
scheitert, wenn das Ziel-Repo nicht sauber gepackt ist — erst dessen Zustand messen
(`count-objects -vH`, freier Platz), dann übertragen (#0099).