[benderl]

Seit Version 2.1.9 werden Commits für Patch-Releases per Cherry-Pick in den jeweils aktuellen Release-Branch geholt und dort mit einem Tag versehen. Mit einem nachfolgenden normalen Release (zuerst mit 2.2.0) wird ein neuer Release-Branch erstellt, da es ansonsten zu Konflikten mit den bereits vorhandenen Commits kommt. Als Ergebnis sind vorherige Patch-Releases (2.1.9-Patch.x ab 2.2.0) nicht mehr in dem Release-Branch enthalten.

Das führ dazu, dass:

• die vorherigen Patch-Releases nicht mehr in der verfügbaren Versionsliste angezeigt werden
• im Backup Infos zum Branch fehlen
• ein Restore einer solchen Patch-Version nicht mehr funktioniert, nachdem ein neues Release veröffentlicht wurde

Dieser PR behebt diese Probleme, indem:

• für Release und Beta ein "virtueller" Branch anhand aller vorhandener Tags zusammen- und dem UI zur Verfügung gestellt wird
• für Release und Beta eine Zusatzinfo in der lokalen git Konfiguration abgelegt wird, um den "virtuellen" Branch zuordnen zu können
• im Backup diese Zusatzinfo mit aufgenommen wird
• beim Restore die Zusatzinfo beachtet wird und der "virtuelle" Branch wieder in der git Konfiguration hinterlegt wird

Zusätzlich wurden

• identische oder ähnliche Codebestandteile von Update, Backup und Restore zentralisiert, um das Verhalten zu vereinheitlichen
• die Archivprüfung direkt beim Hochladen einer Sicherungsdatei korrigiert

ToDo:

• temporär gebaute Einstellungen nach Prüfung entfernen

Settings-UI: openWB/openwb-ui-settings#1124

[Copilot]

🟡 Changes recommended

Production updates are disabled by default, and detached tag checkouts break branch tracking and subsequent updates.

4 open findings

• Preserve virtual train identity after tag checkout · New
• Default test mode off for production updates · New
• Require SELECTED_TAG to reference a valid tag · New
• Accept matching remote branches during validation · New

What changed in this PR

Adds tag-based virtual Release/Beta update channels and restructures update handling.

Changes:

• Builds Release/Beta version options from repository tags.
• Supports tag checkout and fallback selection.
• Adds dry-run wrappers and validation helpers.

File	Description
runs/​update_self.sh	Implements tag-based checkout and update flow.
runs/​update_available_versions.sh	Publishes tag-derived virtual version channels.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🔵 Needs a closer look

Dry-run mode currently disables production updates, and tag fetching/filtering has correctness gaps.

1 open finding

• Default test mode off for production updates

3 resolved since last review

• Preserve virtual train identity after tag checkout
• Accept matching remote branches during validation
• Require SELECTED_TAG to reference a valid tag

Previously missed (2)

In code that hasn't changed since last review

Version filter rejects valid multi-digit major versions

runs/​update_available_versions.sh:102

[2-9]+ is not a numeric “major version >= 2” check: it rejects valid future majors containing 0 or 1, such as 10.0.0 and 20.0.0. The helper still selects those tags, so the advertised train commit can come from a tag that is missing from the corresponding tag list. Match one digit from 2–9 or any multi-digit positive major in both train filters.

Fetch all tags to support standalone release and beta tags

runs/​update_self.sh:77

This fetch does not request all tags, yet the new Release/Beta paths select tags and explicitly support tag commits that may not be reachable from a fetched branch. Git's default auto-follow only obtains tags pointing into fetched histories, so a newly published standalone train tag can remain absent locally; fallback selection then chooses an old tag and an explicitly selected new tag fails validation. Fetch/prune tags here just as the versions script does.

🧠 Review effort: Balanced

[Copilot]

🟡 Changes recommended

Production updates are disabled by default, and virtual trains accept incompatible tags.

2 open findings

• Default test mode off for production updates
• Validate tags against the selected release train · New

🧠 Review effort: Balanced

[Copilot]

🟡 Changes recommended

Production updates are disabled by dry-run mode, and failure paths can leave system control paused indefinitely.

2 open findings

• Clear update_in_progress flag on update script failures · New
• Default test mode off for production updates

1 resolved since last review

• Validate tags against the selected release train

🧠 Review effort: Balanced

[Copilot]

🟡 Changes recommended

Dry-run mode disables real updates, and detached tag checkouts produce unrestorable backups.

2 open findings

• Preserve train ref so detached-HEAD backups can be restored · New
• Default test mode off for production updates

1 resolved since last review

• Clear update_in_progress flag on update script failures

🧠 Review effort: Balanced

[Copilot]

🔵 Needs a closer look

The production update script is permanently configured for dry-run operation.

1 open finding

• Default test mode off for production updates

1 resolved since last review

• Preserve train ref so detached-HEAD backups can be restored

🧠 Review effort: Balanced

[Copilot]

🔵 Needs a closer look

The production update script has dry-run enabled, making updates no-ops and leaving the update flag set.

1 open finding

• Default test mode off for production updates

🧠 Review effort: Balanced

[Copilot]

🔵 Needs a closer look

The extensive generated frontend rebuild and backup/restore state changes require final integration validation.

0 open findings

1 resolved since last review

• Default test mode off for production updates

🧠 Review effort: Balanced

[Copilot]

🟡 Changes recommended

Update compatibility and restoration of backups with deleted upstream branches remain broken.

3 open issues

• Handle commit hashes passed as update tags · New
• Restore backups after upstream branch deletion · New
• Preserve branch identity when installing legacy releases · New

_{Review effort: Balanced}