🎨 refactor: add entity branded type

[matdudek]

No description provided.

[linear]

ARC-12 ID jako branded-type

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
repository-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 9, 2024 7:40pm

[grixu]

Co do budowy nie mam zastrzeżeń - to tak miało wyglądać 👍 Jednak jeśli chodzi o testy - to przegapiłeś mnóstwo punktów w których ID jest używane plus dane testowe i testowa schema w zod nie była przygotowana do użycia z tym typem.

[grixu]

Jako CR wrzucam poprawki i zachęcam do analizy ;)

[github-actions]

Coverage Report for packages/repository

Status	Category	Percentage	Covered / Total
🟢	Lines	96.81% / 80%	152 / 157
🟢	Statements	96.4% / 80%	161 / 167
🟢	Functions	94.2% / 80%	65 / 69
🟢	Branches	92.47% / 80%	86 / 93

File Coverage

File	Stmts	% Branch	% Funcs	% Lines	Uncovered Lines
Changed Files
packages/repository/src/entity/generateId.ts	100%	100%	100%	100%

Generated in workflow #25

[grixu]

@matdudek Czas to mergować! ;)

[grixu]

@coderabbitai please make a deep code review this PR, also analyze why CI doesn't pass. Answer of course in Polish

[coderabbitai]

@grixu, przeprowadziłem wstępną analizę PR oraz konfiguracji CI. Oto kilka punktów, które warto rozważyć:

Przegląd zmian w PR:

Zmiany dotyczą głównie pakietu repository i obejmują testy, generowanie ID oraz interfejsy dla encji. Oto lista zmodyfikowanych plików:

• .changeset/cold-bags-sniff.md
• packages/repository/src/entity/factory.test.ts
• packages/repository/src/entity/generateId.test.ts
• packages/repository/src/entity/generateId.ts
• packages/repository/src/entity/interface/data.test-d.ts
• packages/repository/src/entity/interface/data.ts
• packages/repository/src/entity/interface/entity.test-d.ts
• packages/repository/src/entity/interface/entity.ts
• packages/repository/src/entity/interface/methods.ts
• packages/repository/vitest.setup.ts

Analiza CI:

Konfiguracja CI zawiera dwa główne workflow: Release i CI. Workflow CI jest uruchamiany przy każdym pull requeście do głównej gałęzi i zawiera kroki takie jak:

• Instalacja zależności
• Type-checking
• Linting
• Uruchamianie testów jednostkowych i integracyjnych
• Raportowanie pokrycia kodu testami

Potencjalne przyczyny problemów z CI:

1. Type-checking - Jeśli w kodzie występują błędy typów, krok Type-checking może zakończyć się niepowodzeniem.
2. Linting - Naruszenia zasad lintingu mogą spowodować, że krok Run Biome CI zakończy się błędem.
3. Testy - Niepowodzenie testów jednostkowych lub integracyjnych w kroku Run unit & integration tests może być przyczyną problemów z CI.
4. Pokrycie kodu - Niewystarczające pokrycie kodu testami może być również problemem, chociaż zazwyczaj nie powoduje to bezpośrednio niepowodzenia CI.

Zalecenia:

• Sprawdź logi CI dla konkretnych błędów w wymienionych krokach.
• Upewnij się, że wszystkie testy przechodzą lokalnie przed wysłaniem zmian.
• Przejrzyj kod pod kątem potencjalnych problemów z typami oraz zasad lintingu.

Jeśli potrzebujesz bardziej szczegółowej analizy konkretnych plików lub testów, daj mi znać!

Analysis chain

Scripts executed

The following scripts were executed for the analysis:

---

Script:

#!/bin/bash
# List files changed in the PR
git fetch origin pull/41/head:pr-41
git checkout pr-41
git diff --name-only HEAD $(git merge-base HEAD main)

# Attempt to access CI configuration or logs
# Note: This is a placeholder as accessing CI logs might not be directly possible without specific CI commands or access rights.
# The actual method to access CI logs will depend on the CI system used (e.g., GitHub Actions, Travis CI, Jenkins).
# For demonstration, this script assumes a generic approach.
cat .github/workflows/*.yml

Length of output: 3840

[grixu]

Ping @matdudek