🐛 [Bug] WebpackError: ReferenceError: window is not defined

[tuliooassis]

User Story (if applicable)

As a
I want to
So that

Description

Describe the bug with a clear and concise description of what it is:

• Error;
• Impact;
• Consequence;
• How to fix;
• Effort to fix.

Assumptions

Optionally, add assumptions here.

Steps to reproduce the behavior:

1. Go/navigate to '...';
2. Click/inspect on '....';
3. Scroll down to '....';
4. See error.

Expected behavior

(A clear and concise description of what you expected to happen)

Screenshots

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Mobile (please complete the following information):

• Device: [e.g. iPhone6]
• OS: [e.g. iOS8.1]
• Browser [e.g. stock browser, safari]
• Version [e.g. 22]

Acceptance criteria

• 1. User should...
  Given
  When
  Then

Test scenarios

Add test scenarios.

E.g. try to reproduce this bug in devices like X and browsers like Y, feature toggle on/off...

Additional context

(Add any other context about the problem here, like screenshots)

Checklist do kickoff

• A análise da história está concluída?
• QA (ou pessoa que assumiu o papel de QA) revisou a análise?
• A história está completa com todos os detalhes e as informações relevantes?
  (Ex.: Contexto, Escopo, Fora de Escopo, Critérios de aceite, etc);
• O valor de negócio da história está bem entendido? (se aplicável)
• Esta história não tem dependência de outras histórias futuras?
• Não existem dívidas técnicas (débitos técnicos) relacionadas a história;
• Há protótipos de telas para essa história (se aplicável)
• Há detalhes na história de mensagens de erro e outros feedbacks para o usuário (se aplicável)
• Há mensagens de ajuda ou outros textos definidos na história (se aplicável)
• A história está em um tamanho adequado
• Existe feature toggle para a feature (se aplicável)

Checklist do deskcheck

• Há testes unitários que validem a implementação? (se aplicável)
• As alterações de código passaram em todas as pipelines
• A história foi executada manualmente
• A história não trouxe regressões ou impacto negativo a outras implementações
• Todos os critérios de aceite foram cobertos
• A história não precisa de feedback ou correção
• Foi testada uma jornada completa do usuário através da história (se aplicável)

[jimmyandrade]

@tuliooassis Veja isso: gatsbyjs/gatsby#309. Talvez possa ajudar.

[tuliooassis]

@jimmyandrade
Verifiquei e vi que a causa foi a atualização da recoil neste merge.

Encontrei também uma issue no repo da lib.

Acredito que voltar para a versão 0.0.7 resolverá. O que acha?

[jimmyandrade]

@tuliooassis acho excelente! Parabéns e obrigado por ter achado esse problema! <3

Acabei de subir o commit a2c69d2, que volta para a versão 0.0.7 até o pessoal resolver isso. Nesse commit, também pedi ao Renovate para passar a ignorar updates do Recoil, para não corrermos o risco de ter esse problema de novo.

Já até fizeram uma PR em facebookexperimental/Recoil#220 (e uma duplicata em facebookexperimental/Recoil#233) para resolver isso. Vou acompanhar de perto e, quando lançarem uma correção (provavelmente na 0.0.9), a gente atualiza manualmente.