feat(git): openen en opslaan kiezen hun repository #360

Merged

brenno merged 1 commit from feat/git-connection-picker into main 2026-07-19 14:19:43 +00:00

Conversation 0 Commits 1 Files changed 44 +706 -300

brenno commented 2026-07-19 14:18:27 +00:00

Owner

Copy link

Sluit de verbindingenlijst af: na #358 (het model plus de WebDAV-kant) en #359 (wachtrij per repo) kan git nu ook meerdere verbindingen aan.

De regel die dit bruikbaar houdt

Niet elke handeling mag vragen. Er zijn er twee soorten:

• Repo-breed — openen, synchroniseren, zoeken in alle decks, afbeeldingen in de repository. Die gaan over een repository als geheel, dus daar kiest de gebruiker welke.
• Deck-gebonden — historie, versies, uitbrengen ter review, concept mergen, versie vastleggen, en opslaan van een deck dat al ergens vandaan komt. Die vragen nooit: ze volgen de herkomst van het geopende deck. Vragen zou de gebruiker juist de kans geven daarvan af te wijken, en naar de verkeerde opdrachtgever schrijven levert geen foutmelding op — alleen werk op de verkeerde plek.

En zoals bij WebDAV: met precies één git-verbinding verschijnt er helemaal geen dialoog.

Onder de motorkap

Alle forge-providers zijn families geworden op verbindings-id: gitForgeProvider, nativeGitMirrorProvider, draftMirrorProvider, outboxProvider, syncEngineProvider, gitDeckListProvider, gitDeckTagsProvider. Gekeyd op de id en niet op de configuratie, zodat het herstellen van een typefout in de server-URL een geopend deck niet losweekt van zijn bron.

GitOrigin draagt daarvoor een connectionId. Herkomst uit een oudere versie draagt die nog niet; AppSettings.gitConnectionFor valt dan terug op het vergelijken van de configuratie, zodat zo'n deck niet losgeslagen raakt. Is de verbinding echt verwijderd, dan zegt de app dat — in plaats van stilletjes een andere repo te pakken. Dat is het geval dat git_connection_resolution_test.dart bewaakt.

Splitsingen onderweg

Drie dingen liepen over hun grens en zijn gesplitst, niet gebaselined:

• de verbindingskiezers → shell_actions_connections.dart
• het git-menu → _gitMenuItems
• de werkbranch-beslissing uit saveToGit → _workBranchFor (die nu ook de baseSha teruggeeft, waarmee drie midRound ? origin.baseSha : '' verdwijnen)

Controle

make check volledig groen: 3490 tests, dekking 79,2%.

🤖 Generated with Claude Code

Sluit de verbindingenlijst af: na #358 (het model plus de WebDAV-kant) en #359 (wachtrij per repo) kan git nu ook meerdere verbindingen aan. ## De regel die dit bruikbaar houdt Niet elke handeling mag vragen. Er zijn er twee soorten: - **Repo-breed** — openen, synchroniseren, zoeken in alle decks, afbeeldingen in de repository. Die gaan over een repository als geheel, dus daar kiest de gebruiker welke. - **Deck-gebonden** — historie, versies, uitbrengen ter review, concept mergen, versie vastleggen, en opslaan van een deck dat al ergens vandaan komt. Die vragen **nooit**: ze volgen de herkomst van het geopende deck. Vragen zou de gebruiker juist de kans geven daarvan af te wijken, en naar de verkeerde opdrachtgever schrijven levert geen foutmelding op — alleen werk op de verkeerde plek. En zoals bij WebDAV: met precies één git-verbinding verschijnt er helemaal geen dialoog. ## Onder de motorkap Alle forge-providers zijn families geworden op verbindings-id: `gitForgeProvider`, `nativeGitMirrorProvider`, `draftMirrorProvider`, `outboxProvider`, `syncEngineProvider`, `gitDeckListProvider`, `gitDeckTagsProvider`. Gekeyd op de **id** en niet op de configuratie, zodat het herstellen van een typefout in de server-URL een geopend deck niet losweekt van zijn bron. `GitOrigin` draagt daarvoor een `connectionId`. Herkomst uit een oudere versie draagt die nog niet; `AppSettings.gitConnectionFor` valt dan terug op het vergelijken van de configuratie, zodat zo'n deck niet losgeslagen raakt. Is de verbinding echt verwijderd, dan zegt de app dat — in plaats van stilletjes een andere repo te pakken. Dat is het geval dat `git_connection_resolution_test.dart` bewaakt. ## Splitsingen onderweg Drie dingen liepen over hun grens en zijn gesplitst, niet gebaselined: - de verbindingskiezers → `shell_actions_connections.dart` - het git-menu → `_gitMenuItems` - de werkbranch-beslissing uit `saveToGit` → `_workBranchFor` (die nu ook de baseSha teruggeeft, waarmee drie `midRound ? origin.baseSha : ''` verdwijnen) ## Controle `make check` volledig groen: 3490 tests, dekking 79,2%. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit 2026-07-19 14:18:27 +00:00

feat(git): openen en opslaan kiezen hun repository ... a567d5d398

Met meerdere git-verbindingen moet elke handeling weten naar welke repo
hij gaat. Alle forge-providers zijn daarom per verbinding gekeyd — op id
en niet op de configuratie, zodat het herstellen van een typefout in de
server-URL een geopend deck niet losweekt van zijn bron.

Het onderscheid dat de kiezer bruikbaar houdt: repo-brede handelingen
(openen, synchroniseren, zoeken, afbeeldingen) vragen welke repository,
maar deck-gebonden handelingen (historie, versies, review, mergen, taggen)
nooit — die volgen de herkomst van het geopende deck. Vragen zou de
gebruiker daar de kans geven per ongeluk van af te wijken, en naar de
verkeerde opdrachtgever schrijven geeft geen foutmelding. Bij één
verbinding verschijnt er sowieso geen dialoog.

Is de verbinding van een deck verwijderd, dan zegt de app dat in plaats
van een andere repo te kiezen.

Onderweg gesplitst omdat ze over hun grens liepen: de verbindingskiezers
naar shell_actions_connections.dart, het git-menu naar _gitMenuItems, en
de werkbranch-beslissing uit saveToGit naar _workBranchFor.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit e4897bac89 into main 2026-07-19 14:19:43 +00:00

brenno referenced this pull request from a commit 2026-07-19 14:19:43 +00:00

Merge pull request 'feat(git): openen en opslaan kiezen hun repository' (#360) from feat/git-connection-picker into main

brenno deleted branch feat/git-connection-picker 2026-07-19 14:19:43 +00:00

brenno referenced this pull request 2026-10-14 18:41:02 +00:00

OciServe-thema's: publiceer en haal thema's op, opt-in (nu vult niemand het themaregister) #2429

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

release-blocker

Must land before the next release. scripts/release_auto.sh refuses to start while this is open.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

release-blocker

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

LibreKAT/Ocideck!360

No description provided.