momsse/octant
1
0
Fork 0
Code Issues 9 Pull requests 4 Projects Releases Packages Wiki Activity Actions

PR 06 — Adapters OAuth (arctic, discord, google) #6

New issue
Closed
opened 2026-07-04 12:10:42 +00:00 by momsse · 0 comments
momsse commented 2026-07-04 12:10:42 +00:00
Owner
Copy link

Migré depuis viziertronic/octant#19 — ouvert le 2026-06-11 par @momsse.

Tranche 6/13 du découpage de feat/authentication (plan : docs/pr-split/feat-authentication.md).

Branche : split/06-oauth-adapters — Bloquée par : #5
Taille : 26 fichiers, ~1 712 lignes

Périmètre

  • packages/infrastructure/arctic-oauth : cœur d'adapter partagé (fetch de profil en erreur typée, retry des timeouts)
  • packages/infrastructure/discord-oauth, packages/infrastructure/google-oauth (offline access pour obtenir les refresh tokens Google)
  • patches/arctic.patch + l'entrée patchedDependencies dans pnpm-workspace.yaml (le patch saute le header Content-Length explicite)

Focus review

Politique de retry (timeouts uniquement, jamais les 4xx), surface d'erreurs typées, justification du patch arctic.

Commits d'origine

892d5ee, f7d874e, 19fb731, 2d421fc (partie adapter), 9950e60, 76efac3, e31bbed, 890e4f5

> _Migré depuis [viziertronic/octant#19](https://github.com/viziertronic/octant/issues/19) — ouvert le 2026-06-11 par @momsse._ Tranche 6/13 du découpage de `feat/authentication` (plan : `docs/pr-split/feat-authentication.md`). **Branche** : `split/06-oauth-adapters` — **Bloquée par** : #5 **Taille** : 26 fichiers, ~1 712 lignes ## Périmètre - `packages/infrastructure/arctic-oauth` : cœur d'adapter partagé (fetch de profil en erreur typée, retry des timeouts) - `packages/infrastructure/discord-oauth`, `packages/infrastructure/google-oauth` (offline access pour obtenir les refresh tokens Google) - `patches/arctic.patch` + l'entrée `patchedDependencies` dans `pnpm-workspace.yaml` (le patch saute le header Content-Length explicite) ## Focus review Politique de retry (timeouts uniquement, jamais les 4xx), surface d'erreurs typées, justification du patch arctic. ## Commits d'origine `892d5ee`, `f7d874e`, `19fb731`, `2d421fc` (partie adapter), `9950e60`, `76efac3`, `e31bbed`, `890e4f5`
momsse 2026-07-04 12:10:42 +00:00
  • closed this issue
  • added the
    pr-split
         label
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
renovate/dotenv-17.x
renovate/effect
renovate/catalog-pnpm.catalog.testing
renovate/catalog-pnpm.catalog.ui
harden/145-email-change-verification
perf/bound-banking-list-rpcs
perf/bound-list-reconciliations
harden/supersede-all-established
perf/targeted-match-queries
fix/exact-match-sign-normalization
refactor/extract-rpc-guarded
fix/bank-account-sync-dedup
fix/reconciliation-document-supersession
feat/transaction-matcher-documents
feat/transaction-matcher-banking
feat/transaction-matcher-presentation
feat/compose-matcher-backend
feat/scheduling-clock-signals
feat/documents-sources-analysis
feat/banking-bank-api-sync
feat/transaction-matching
feat/postgres-reconciliation
feat/postgres-documents
feat/postgres-banking
feat/reconciliation-domain
feat/documents-domain
feat/banking-domain
worktree-glistening-hopping-tiger
split/13-backoffice
chore/treat-issue-skill
split/12-api
refacto/schema-driven-projector-writes
split/11-postgres-authorization
split/10-postgres-authentication
feat/oxlint-no-redundant-return-type
feat/observability-spans
split/09-authorization-permissions
split/08-authorization-relations
split/07-authorization-roles-groups
split/06-oauth-adapters
refactor/46-unrepresentable-errors-auth-use-cases
test/45-standardized-test-service-factories
split/05-authentication-use-cases-rpc
test/38-aggregate-state-fixtures
split/04b-user-status-value-object
crypto/37-envelope-key-id
refactor/event-sourcing-stream-cursor
feat/authentication
feat/event-sourcing-package
feat/auth-domain
auth
No results found.
Labels
Clear labels   
bug
Something isn't working

  
enhancement
New feature or request

  
pr-split
Découpage de feat/authentication en PRs reviewables

  
question
Further information is requested

  
security

   
transaction-matcher
Domaine Transaction Matcher (banking / documents / reconciliation)

No labels
bug
enhancement
pr-split
question
security
transaction-matcher
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
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
momsse/octant#6
No description provided.