LGT-167 Linguist Connector Package Update by Aldr0n · Pull Request #2 · hyperlinkgroup/linguist-package

Aldr0n · 2026-03-12T11:27:00Z

Beschreibung:

Dieser PR aktualisiert das Linguist Connector Package auf ein moderneres Level.

Was wurde gemacht?

Das Packet an sich wurde aktualisiert (Dependencies). Zudem wurde die Projektstruktur aufgeräumt, sauber in Services und Actions unterteilt und anschließend erweitert.

Das aufsetzen des Paketes ist nun via einem einzigen Command möglich (linguist:setup). Dabei werden alle nötigen Informationen abgefragt (API Key, zugehöriges Projekt, usw.). Zudem wurde ein Sprachdatei-Import Feature hinzugefügt. Bei Ersteinrichtungen kann das Package jetzt die bestehenden Sprachdateien Scannen und die Übersetzungs-Keys automatisch in Linguist anlegen. Man muss also nicht mehr manuell eine Datei nach der anderen hochladen.

Der sync Command wurde ebenfalls erweitert. Entwickler müssen sich jetzt nicht mehr darum kümmern einen eigenen Command anzulegen sondern können einfach mit linguist:sync die Sprachdateien aktualisieren. Hier gibt es nun 3 Modi.

sync: Es werden sowohl neue Sprachdateien hochgeladen als auch aktuelle Übersetzungen runtergeladen
pull: Es werden aktuelle Übersetzungen runtergeladen
push: Es werden sowohl neue Sprachdateien hochgeladen

Warum wurde es gemacht?

Bisher war die Integration von Linguist in Projekten sehr unregelmäßig. Entwickler mussten viele Schritte selbst machen wie einen Sync-Command anlegen. Das hat dazu geführt das bspw. jedes Projekt eine eigene Sync-Command-Signatur hat, was immer wieder zu Verwirrung führt.

Umgebung & Voraussetzungen:

Eine eingerichtete Projekt Entwicklungsumgebung.

Ablauf der Nachprüfung:

Das LInguist Repo und das Linguist Connector Package Repo local auf dem Ticket branch ausschecken
Per Composer Symlink das Connector Package lokal installieren ( Ich hab das Package mit Linguist selbst getestet)
Sicherstellen das man einen aktuellen API key hat
In einem Projekt welchem das Package installiert ist folgende Commands ausführen:

php artisan linguist:setup

php artisan linguist:sync

[Optional] Screenshots/Videos

[Optional] Weitere Hinweise:

[Optional] Sind nach dem Deployment Tasks auszuführen?

- Replace project and token checks with direct property comparisons - Remove redundant methods for getting project and token - Update command signature and handle method for improved error handling - Enhance tests to reflect changes in method calls and command execution

…on handling

…on files

…ration during setup

… with Linguist API

…ization actions

…tive setup and configuration management

…Provider

…endencies

Copilot

Pull request overview

Dieses PR modernisiert das Linguist-Connector-Package durch eine neue Action-/DTO-basierte Architektur und erweitert die DX um einen interaktiven Setup-Flow sowie ein überarbeitetes linguist:sync-Command.

Changes:

Einführung eines LinguistApiClient plus Actions für pull, push und sync inkl. Orchestrierung (SetupOrchestrator) und Setup-Wizard (linguist:setup).
Neue DTOs (SetupInput, SyncResult) und Anpassungen an Service Provider/Facade/Legacy-API.
Dokumentation und umfangreiche Test-Suite-Erweiterungen rund um die neuen Flows.

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
tests/Unit/Services/LinguistApiClientTest.php	Unit-Tests für Key-Counting/Fallback-Logik des API-Clients
tests/Unit/DTO/SyncResultTest.php	Unit-Tests für Ergebnis-/Summary-Logik des SyncResult DTO
tests/Unit/DTO/SetupInputTest.php	Unit-Tests für SetupInput DTO (Helper/Defaults/Serialisierung)
tests/Unit/Actions/PushTranslationsTest.php	Unit-Tests für Push-Action (Batching/Progress/Mapping)
tests/Unit/Actions/PullTranslationsTest.php	Unit-Tests für Pull-Action (Key-Counting/Files)
tests/Unit/Actions/PersistLinguistConfigTest.php	Unit-Tests für .env-Persistierung/Masking
tests/Unit/Actions/CollectLocalTranslationsTest.php	Unit-Tests für lokale Translation-Discovery/Flattening/Writing
tests/Pest.php	Pest-Setup + Test-Helper für HTTP/Filesystem
tests/LinguistTest.php	Anpassung Legacy-Tests auf neue Facade-Nutzung + Artisan-Tests
src/Services/SetupOrchestrator.php	Orchestriert Setup: persist config, resolve project, run sync, auto-translate
src/Services/LinguistApiClient.php	Neuer HTTP-Client für Linguist API inkl. Helper/Count/Mapping
src/LinguistServiceProvider.php	Registriert neue Commands/Services/Actions im Container
src/Linguist.php	Legacy-Klasse refaktoriert (Pooling, vereinfachte Config-Nutzung)
src/Facades/Linguist.php	Facade-Docblock an neue/entfernte Methoden angepasst
src/DTO/SyncResult.php	Neuer Ergebnis-DTO für Sync-/Pull-/Push-Flows
src/DTO/SetupInput.php	Neuer Eingabe-DTO für Setup-Wizard/Orchestrierung
src/Commands/LinguistSetupCommand.php	Neues interaktives Setup-Command (Prompts, Setup-Ausführung, Reporting)
src/Commands/LinguistCommand.php	Neues `linguist:sync` Command mit Modi/Optionen/Progress
src/Actions/SyncTranslations.php	Sync-Action (remote exports laden + local merge + write)
src/Actions/PushTranslations.php	Push-Action (batch store + fallback per key, Progress-Callback)
src/Actions/PullTranslations.php	Pull-Action (download exports, write linguist.json, key counts)
src/Actions/PersistLinguistConfig.php	Persistiert Token/URL/Project in .env + runtime config
src/Actions/CollectLocalTranslations.php	Discover/Parse/Flatten/Write lokaler Translation-Files
composer.json	Dependencies erweitert (laravel-actions)
README.md	Dokumentation für Setup-Wizard, Commands, Modes und Usage erweitert/aktualisiert

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

…checks

…nfiguration

…pCommand

… SetupOrchestrator

…rocessing

…and utility methods

…tus for Linguist commands

…rror reporting for missing search roots

…ushTranslationsTest

…rmal laravel standards

…cumentation and tests

…d PushTranslations actions to reflect accurate counts

… selection to use lang keys instead of ids

…eys in sync mode

… sync mode

…ranslation pull

kaiserkiwi

Neben den angesprochenen Änderungen beim Pullen, noch zwei weitere Dinge als Code-Kommentare

kaiserkiwi · 2026-04-21T14:22:53Z

+	private function updateEnvValue(string $key, string $value): bool
+	{
+		$envPath = base_path('.env');
+
+		if (! File::exists($envPath)) {
+			$examplePath = base_path('.env.example');
+			if (File::exists($examplePath)) {
+				File::copy($examplePath, $envPath);
+			} else {
+				File::put($envPath, '');
+			}
+		}
+
+		$content = File::get($envPath);
+		$escapedValue = $this->escapeEnvValue($value);
+		$pattern = '/^' . preg_quote($key, '/') . '=.*/m';
+
+		if (preg_match($pattern, $content)) {
+			$content = preg_replace($pattern, "{$key}={$escapedValue}", $content);
+		} else {
+			$content .= "\n{$key}={$escapedValue}\n";
+		}
+
+		return File::put($envPath, $content) !== false;
+	}


suggestion: Hier sollte vielleicht auch direkt noch geprüft werden, ob eine .env.example existiert, um die Daten auch dort hinzuzufügen.

kaiserkiwi · 2026-04-21T14:24:01Z

+		if (preg_match($pattern, $content)) {
+			$content = preg_replace($pattern, "{$key}={$escapedValue}", $content);
+		} else {
+			$content .= "\n{$key}={$escapedValue}\n";


nitpick: Durch das \n wird auch zwischen den einzelnen Zeilen der Linguist-Keys immer eine Leerzeile eingefügt

…push operations

kaiserkiwi · 2026-05-05T13:19:38Z

Da mein Review nicht mehr requested wurde und auch kein Kommentar mehr kam: Wird hier noch dran gearbeitet oder kann ich schon wieder draufschauen?

Aldr0n · 2026-05-07T08:18:42Z

Ah my bad, ja bitte @kaiserkiwi

Aldr0n added 14 commits March 11, 2026 09:56

feat(LGT-167): DTOs for project setup and sync results handling

2cc2109

feat(LGT-167): LinguistApiClient for project management and translati…

b288fbb

…on handling

feat(LGT-167): CollectLocalTranslations action for managing translati…

b7df8ef

…on files

feat(LGT-167): PersistLinguistConfig action for managing .env configu…

d72126e

…ration during setup

feat(LGT-167): Actions for pulling, pushing, and syncing translations…

75d011b

… with Linguist API

feat(LGT-167): Enhanced LinguistCommand with new translation synchron…

805d247

…ization actions

feat(LGT-167): LinguistSetupCommand and SetupOrchestrator for interac…

b169ab7

…tive setup and configuration management

feat(LGT-167): Register new actions and API client in LinguistService…

828dfb9

…Provider

chore(LGT-167): Add lorisleiva/laravel-actions package and update dep…

8e1edb1

…endencies

chore: Pint formatting

27df782

docs(LGT-167): Update README

df140d0

test(LGT-167): Update test suite after rework

2d9bc23

fix(LGT-167): Pre sync summary not displaying remote key count

977d264

Copilot AI review requested due to automatic review settings March 12, 2026 11:27

Copilot started reviewing on behalf of Aldr0n March 12, 2026 11:27 View session

Copilot AI reviewed Mar 12, 2026

View reviewed changes

Aldr0n self-assigned this Mar 12, 2026

Aldr0n added the enhancement An existing feature that just got better label Mar 12, 2026

Aldr0n requested a review from kaiserkiwi March 12, 2026 11:43

Aldr0n changed the title ~~Lgt 167 linguist connector package update~~ LGT-167 Linguist Connector Package Update Mar 12, 2026

Aldr0n added 9 commits March 12, 2026 12:48

refactor(LGT-167): Update translation status messages

6b51156

chore: Add CI workflow for testing, PHPStan analysis, and code style …

e7bf6f7

…checks

chore: Remove checkMissingIterableValueType parameter from PHPStan co…

a44835c

…nfiguration

refactor(LGT-167): Simplify token and name validation in LinguistSetu…

8d41689

…pCommand

chore: Add Laravel preset to Pint configuration

46d3490

refactor(LGT-167): Remove debug logging from LinguistSetupCommand and…

86d3c42

… SetupOrchestrator

refactor(LGT-167): Harden sync command flow and ensure proper batch p…

9b47e92

…rocessing

chore: Pint reformat

8e5317b

docs: Update README

59f1fb4

Aldr0n force-pushed the lgt-167-linguist-connector-package-update branch 2 times, most recently from 3cc0825 to e86fdcf Compare March 26, 2026 10:19

Aldr0n and others added 10 commits March 30, 2026 10:05

feat(LGT-167): Enhance language export handling with prefix fallback …

3517084

…and utility methods

feat(LGT-167): Add local JSON search roots display in translation sta…

1e8ca6d

…tus for Linguist commands

refactor(LGT-167): Streamline translation file handling and improve e…

791eed4

…rror reporting for missing search roots

test(LGT-167): Ajust tests for latest changes

31391af

chore: pint reformat

0be77a5

chore(Pint): Code formatting

bb0e871

refactor(LGT-167): Simplify configured path assignment

a4037c9

test(LGT-167): Add setup and teardown for isolated language path in P…

a8c2598

…ushTranslationsTest

fix(LGT-167): Update api key check error message type

4246c6d

feat(LGT-167): The file format for downloaded files no follows the no…

3c08372

…rmal laravel standards

Aldr0n force-pushed the lgt-167-linguist-connector-package-update branch from e86fdcf to 3c08372 Compare March 30, 2026 08:05

Aldr0n and others added 9 commits March 30, 2026 10:24

fix(LGT-167): Update package for API versioning and adjust related do…

799fc8d

…cumentation and tests

refactor(LGT-167): Update key processing logic in PullTranslations an…

cb75bb0

…d PushTranslations actions to reflect accurate counts

chore: pint reformat

b07b196

feat(LGT-167): Allow team selection for new projects and enhance lang…

6328c29

… selection to use lang keys instead of ids

feat(LGT-167): Add event dispatching for Pull, Push, and Sync operations

bee35d8

chore(Pint): Code formatting

22cb871

feat(LGT-167): Add api link to the linguist api token prompt

9bcf5b2

feat(LGT-167): Update pruneRemoteKeys logic to always remove remote k…

2786794

…eys in sync mode

feat(LGT-167): Option to confirm removal of non-local keys in regular…

00dea60

… sync mode

Aldr0n force-pushed the lgt-167-linguist-connector-package-update branch from 28a883a to 00dea60 Compare April 21, 2026 14:26

feat(LGT-167): Include source language in the list of languages for t…

329ffb9

…ranslation pull

kaiserkiwi reviewed Apr 22, 2026

View reviewed changes

Aldr0n and others added 2 commits April 24, 2026 23:46

feat(LGT-167): Implement sync source option for translation sync and …

e8db75f

…push operations

chore(Pint): Code formatting

5772a2e

Aldr0n requested a review from kaiserkiwi May 7, 2026 08:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

LGT-167 Linguist Connector Package Update#2

LGT-167 Linguist Connector Package Update#2
Aldr0n wants to merge 54 commits intomainfrom
lgt-167-linguist-connector-package-update

Aldr0n commented Mar 12, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kaiserkiwi left a comment

Uh oh!

kaiserkiwi Apr 21, 2026

Uh oh!

kaiserkiwi Apr 21, 2026

Uh oh!

kaiserkiwi commented May 5, 2026

Uh oh!

Aldr0n commented May 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Aldr0n commented Mar 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Beschreibung:

Was wurde gemacht?

Warum wurde es gemacht?

Umgebung & Voraussetzungen:

Ablauf der Nachprüfung:

[Optional] Screenshots/Videos

[Optional] Weitere Hinweise:

[Optional] Sind nach dem Deployment Tasks auszuführen?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kaiserkiwi left a comment

Choose a reason for hiding this comment

Uh oh!

kaiserkiwi Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

kaiserkiwi Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

kaiserkiwi commented May 5, 2026

Uh oh!

Aldr0n commented May 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Aldr0n commented Mar 12, 2026 •

edited

Loading