Skip to content

fix: supprimer async inutile sur WhyChallenge#516

Merged
cyrilledaily merged 2 commits intomainfrom
fix/why-challenge-async-client-component
Feb 20, 2026
Merged

fix: supprimer async inutile sur WhyChallenge#516
cyrilledaily merged 2 commits intomainfrom
fix/why-challenge-async-client-component

Conversation

@m4xim1nus
Copy link
Collaborator

Problème

La page /interpeller ('use client') importe WhyChallenge, déclaré async function. Cela provoque deux erreurs Next.js 15 :

  • async Client Component — React/Next.js n'autorise async que sur les Server Components
  • suspended by an uncached promise — React tente d'exécuter la promesse côté client sans cache

Cause

WhyChallenge n'effectue aucune opération asynchrone. Le async était superflu et a été ajouté par erreur.

Fix

Suppression du mot-clé async sur WhyChallenge. Le composant reste fonctionellement identique.

Test plan

  • Ouvrir /interpeller en local → plus d'erreurs console
  • Vérifier que WhyChallenge s'affiche correctement sur la page
  • Vérifier que WhyChallenge s'affiche correctement sur la page d'accueil (/)

🤖 Generated with Claude Code

…nt Component

WhyChallenge n'effectue aucune opération asynchrone. Le mot-clé `async`
provoquait deux erreurs Next.js quand le composant était rendu depuis
interpeller/page.tsx (marqué 'use client') :
- "async Client Component" non supporté
- "suspended by an uncached promise"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

:octocat: Preview is deploying...

Please wait while the preview is deploying on Clever Cloud...

ℹ️ Name 🔗 Infos & links
🌱 Deploying commit ec6fe1f
👁️ Preview Generating preview app...

Copy link
Collaborator

@cyrilledaily cyrilledaily left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — fix correct et nécessaire.

WhyChallenge n'a aucune opération asynchrone, le async était superflu et causait des erreurs Next.js 15 quand le composant est importé depuis un client component (ce qui est le cas sur /interpeller).

Fix minimal, une seule ligne, aucun risque de régression.

✅ Approuvé

@cyrilledaily cyrilledaily merged commit 21b743e into main Feb 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants