Ember.js: 3.22에서 "더 공격적인" 주장 역추적

흠, 나는 당신이 여기서 의미하는 바를 완전히 이해하지 못합니다. 던져진 오류는 주장(지원 중단이 아님)이며 해결하기 위한 것이 아닙니다...

지금은 닫으려고 하는데 여기에서 뭔가 잘못 이해한 부분이 있으면 다시 열게 되어 기쁩니다...

https://github.com/emberjs/ember.js/blob/c5bbf863af7cd8f6f92a97f2b777ff857bf9a9f4/packages/%40ember/-internals/glimmer/lib/environment.ts#L65

이것은 실행되어야 하는 사용 중단으로 간주되지만 경고를 표시하는 대신 어설션이 오류를 일으키는 이유는 무엇입니까? 오류로 인해 앱을 리팩터링하라는 경고를 표시하는 대신 앱이 중단됩니다.

링크한 지점은 Ember가 렌더링 엔진을 구성하는 곳으로 어설션 및 사용 중단 모두에 대한 후크를 제공하며 렌더링 엔진은 시나리오에 따라 사용할 것을 선택합니다.

이러한 구성된 옵션은 glimmer-vm에서 다음 중 하나를 실행합니다.

• runInTrackingTransaction

• deprecateMutationsInTrackingTransaction

이 특정 사례가 주장이 아니라 사용 중단되어야 하는 이유를 더 자세히 설명할 수 있습니까? twiddle에서 수행되는 특정 작업은 항상 방지하기 위한 것이었습니다(오류 메시지가 설명하는 대로).

export default class ApplicationController extends Controller {
  <strong i="18">@tracked</strong> appName = 'Ember Twiddle';

  get name() {
    this.appName = "Test"

    if (this.appName === "Test") {
      this.appName = "test"
    }

    return this.appName
  }
}

특히 이 경우 추적된 값 을 읽은

@rwjblue 여기서 문제는 Ember의 부 릴리스로 인해 이전에 무의식적으로 이 주장을 위반했던 앱이 이제 역추적 업데이트가 해결될 때까지 업그레이드가 차단된다는 것입니다. 제 경우에는 사용하는 애드온이 이 어설션을 위반하기 때문에 유지 관리하는 앱이 3.22로 업그레이드되지 않도록 차단되었습니다.

이러한 역추적 어설션이 오류 대신 경고로 도입된 경우 이러한 문제를 추적할 시간이 주어지기 때문에 나와 같은 앱이 최신 상태를 유지하는 더 쉬운 경로를 제공할 것입니다. 현재 상태에서 내 앱은 이 새로운 주장을 위반하는 위치/이유를 추적할 대역폭이 확보될 때까지 3.21을 유지해야 합니다.

내가 버그라고 생각하는 이유는 ember-gestures가 Ember 3.22에서 작동을 멈췄기 때문입니다.
그들은 분해에 이 기능을 사용하고 있으며 이전 버전에서 작동했습니다.
__instance 변수를 읽은 다음 업데이트하므로 더 이상 작동하지 않고 이제 주요 버전 변경이 아니더라도 앱이 중단됩니다.

  __teardownRecognizers: on('willDestroyElement', function() {
    let instance = this.get('__instance');
    if (instance) {
      //instance.off();
      instance.destroy();
      this.set('__instance', null);
    }
  }),

여기에서 문제는 Ember의 부 릴리스로 인해 이전에 무의식적으로 이 주장을 위반했던 앱이 이제 역추적 업데이트가 해결될 때까지 업그레이드가 차단된다는 것입니다. 제 경우에는 사용하는 애드온이 이 어설션을 위반하기 때문에 유지 관리하는 앱이 3.22로 업그레이드되지 않도록 차단되었습니다.

이것은 훌륭합니다! 원래 보고서에는 twiddle의 코드가 실제로 작동한 적이 있는지(또는 오류가 발생한 이유에 대한 추론) 표시되지 않았습니다.

@kpfefferle - 어떤 버전에서 업그레이드합니까? 귀하의 경우 주장을 일으키는 돌연변이의 원인은 무엇입니까?

명확히 하자면, 저는 주로 여기에 보고된 내용(범위, 사용 패턴 등)을 이해하려고 노력하고 있으며 특히 SemVer에 대해 합리화하거나 논쟁하려는 것이 아닙니다. 😸

@rwjblue Dependabot PR이 3.21.3에서 3.22.0으로 충돌하는 데 실패한 두 개의 앱이 있습니다. 둘 다 실패한 모든 테스트에 대해 다음과 같은 유사한 오류 메시지를 표시합니다.

        actual: >
            [object Object]
        stack: >
            Error: Assertion Failed: You attempted to update `_errors` on `changeset:[object Object]`, but it had already been used previously in the same computation.  Attempting to update a value after using it in a computation can cause logical errors, infinite revalidation bugs, and performance issues, and is not supported.

            `_errors` was first used:

            - While rendering:
              application
                forms
                  forms.form
                    form-workflow
                      form-workflow/form
                        form-for

            Stack trace for the update:
                at setter ([...]/assets/vendor.js:56179:9)
                at EmberChangeset.set [as _errors] ([...]/assets/vendor.js:17775:7)
                at EmberChangeset.BufferedChangeset.set (webpack://__ember_auto_import__/./node_modules/validated-changeset/dist/validated-changeset.es5.js?:2674:17)
                at Object.set ([...]/assets/vendor.js:104899:22)
                at Proxy.BufferedChangeset._handleValidation (webpack://__ember_auto_import__/./node_modules/validated-changeset/dist/validated-changeset.es5.js?:2447:18)
                at Proxy.BufferedChangeset._validateKey (webpack://__ember_auto_import__/./node_modules/validated-changeset/dist/validated-changeset.es5.js?:2425:23)
                at eval (webpack://__ember_auto_import__/./node_modules/validated-changeset/dist/validated-changeset.es5.js?:2211:24)
        message: >
            Assertion Failed: You attempted to update `_errors` on `changeset:[object Object]`, but it had already been used previously in the same computation.  Attempting to update a value after using it in a computation can cause logical errors, infinite revalidation bugs, and performance issues, and is not supported.

            `_errors` was first used:

            - While rendering:
              application
                forms
                  forms.form
                    form-workflow
                      form-workflow/form
                        form-for

이것은 일부 오류 재계산을 수행하는 ember-changeset-validations의 내부인 것 같지만 아직 파헤칠 대역폭이 없었습니다. 어설션이 정확하고 유효한 것처럼 보이지만 내 테스트 스위트를 폭파시키는 것은 무슨 일이 일어나고 있는지 알아낼 수 있을 때까지 3.22로 업그레이드할 수 없다는 것을 의미합니다.

잡았다. FWIW 오류 메시지는 디버깅에 도움이 되는 최소한 "유용한" 것으로 보입니다.

@Yelinz - 보고된 원래 문제( twiddle에서 )가 @kpfefferle 이 보고하는 것과 동일한 영역(최소한 3.16 이후의

내 이전 의견에서 말했듯이 3.21에서 3.22까지이며 상황은 다른 애드온을 사용하는 @kpfefferle 와 비슷합니다. 내가 만든 비틀기는 그 오류를 불러일으키기 위한 것이었습니다. 원인이 되는 실제 코드는 이전에 게시한 코드입니다.

2020-10-12T09:38:40.7310067Z         actual: >
2020-10-12T09:38:40.7310575Z             [object Object]
2020-10-12T09:38:40.7311080Z         stack: >
2020-10-12T09:38:40.7314024Z             Error: Assertion Failed: You attempted to update `__instance` on `<ember-caluma<strong i="7">@component</strong>:x-toggle-switch::ember726>`, but it had already been used previously in the same computation.  Attempting to update a value after using it in a computation can cause logical errors, infinite revalidation bugs, and performance issues, and is not supported.
2020-10-12T09:38:40.7315638Z             
2020-10-12T09:38:40.7316183Z             `__instance` was first used:
2020-10-12T09:38:40.7316698Z             
2020-10-12T09:38:40.7317709Z             - While rendering:
2020-10-12T09:38:40.7318285Z               application
2020-10-12T09:38:40.7318787Z                 demo
2020-10-12T09:38:40.7319293Z                   application
2020-10-12T09:38:40.7319772Z             
2020-10-12T09:38:40.7320295Z             Stack trace for the update:
2020-10-12T09:38:40.7321118Z                 at dirtyTagFor (http://localhost:7357/assets/vendor.js:74216:9)
2020-10-12T09:38:40.7322185Z                 at markObjectAsDirty (http://localhost:7357/assets/vendor.js:31365:32)
2020-10-12T09:38:40.7323337Z                 at notifyPropertyChange (http://localhost:7357/assets/vendor.js:31403:5)
2020-10-12T09:38:40.7324352Z                 at _set2 (http://localhost:7357/assets/vendor.js:32379:9)
2020-10-12T09:38:40.7325263Z                 at Class.set (http://localhost:7357/assets/vendor.js:45923:29)
2020-10-12T09:38:40.7326501Z                 at Class.__teardownRecognizers (http://localhost:7357/assets/vendor.js:165238:14)
2020-10-12T09:38:40.7327580Z                 at sendEvent (http://localhost:7357/assets/vendor.js:30998:14)

스택 추적은 또한 그의

내가 만든 twiddle은 그 오류를 불러 일으키기위한 것입니다.

예, 하지만 해당 오류가 발생 해야 하는 시나리오가 많이 있습니다. 우리가 파악해야 할 것은 이전 에 추적되지

사용 패턴이 무엇인지 알게 되면 주장 대신 비추천을 내도록 변경할 수 있는지 알아낼 수 있지만 현재로서는 여러분이 부딪치는 실제 문제를 나타내는 재생산이 아직 없습니다. 그래서 우리가 주장을 비추천으로 조정할 수 있는지 정말로 말할 수 없습니다...

여기에 오는 다른 사람들을 위해 여기에서 보고 싶은 것은 전달되는 재생산(일부 리포지토리에서 테스트 실패의 형태로 이상적으로는, 애드온이든 이 목적으로 만들어진 데모 앱이든 상관없음)입니다. 3.21 및 3.22에서 실패합니다.

3월 21일에 통과한 테스트입니다.
https://github.com/projectcaluma/ember-caluma/pull/1065/checks?check_run_id=1241360614

한편 동일한 코드는 3.22에서 실패합니다.
https://github.com/projectcaluma/ember-caluma/pull/1065/checks?check_run_id=1241408429

문제는 엠버 제스처와 이전에 게시한 코드 조각으로 끝나는 긴 종속성 목록에서 발생합니다.

여러분의 모든 도움에 감사드립니다.

곧 패치 릴리스에서 이를 제공하기 위해 노력할 것입니다.

@rwjblue 문제를 _ 추적 _해 @pzuraq 에게 감사드립니다!

:wave: @rwjblue , 이 수정 사항이 포함된 3.22.1 패치를 릴리스할

ember-changeset-validations 주위에 설정된 패턴으로 인해 많은 앱에서 이 문제가 발생했습니다. 다음은 우리가 사용하는 패턴의 예입니다.

import Component from '@glimmer/component';
import Changeset from 'ember-changeset';
import lookupValidator from 'ember-changeset-validations';
import OrderValidations from '../validations/order';

export default class OrderFormComponent extends Component {
  constructor() {
    super(...arguments);

    let model = this.args.model;
    this.changeset = new Changeset(
      model,
      lookupValidator(OrderValidations),
      OrderValidations
    );
    this.changeset.validate();
  }
}

this.changeset.validate() 는 3.22.0에서 어설션 실패를 트리거합니다.

방금 v3.22.1(https://github.com/emberjs/ember.js/releases/tag/v3.22.1)의 릴리스를 트리거했으며, 곧 NPM에 게시되어야 합니다(CI가 완료되면).

이 수정 사항이 실제로 v3.22.1 릴리스에 포함되었습니까? 아니면 관련 PR이 이 문제를 완전히 수정하지 못했습니까? 릴리스 노트에서 https://github.com/emberjs/ember.js/pull/19193 을 볼 수 있지만 내 프로젝트는 여전히 이전과 동일한 테스트 실패를 보여주고 있습니다 🤔

확인됨, 3.22.1에서도 여전히 표시됨

재조사를 위해 재개합니다. 단순화된/최소한의 데모(3.20에 통과하고 3.22에 실패)는 정말 도움이 될 것입니다.

다음은 3.22.1 에서 오류를 생성하지만 3.21 에서 작동하는 간단한 예 입니다.

export default class ExampleComponent extends Component {
  <strong i="9">@tracked</strong> count = 0;

  constructor() {
    super(...arguments);

    this.count++; // this fails in 3.22.1
  }
}

3.21 :

3.22.1 :

생성자에서 추적 속성을 읽고 업데이트하는 데 문제가 있습니까? 내 정신 모델은 속성이 구성 요소에 로컬이고 아직 DOM에 아무 것도 렌더링하지 않았기 때문에 이것이 문제가 없어야 한다는 것입니다.

위의 예는 또한 최근에 태그가 지정된 3.23 에 대한 주장을 제기합니다.

@GavinJoyce 가 Glimmer 컴포넌트와 함께 제공한 예시인가요? 우리는 이전 버전과의 호환성을 위해 Classic 구성 요소에서만 이 버그를 수정하려고 했습니다. Glimmer 구성 요소에서는 일반적으로 앞으로 _do_ 렌더링 중에 이와 같은 돌연변이가 실패할 것으로 예상합니다. 일반적으로 단일 추적 계산 중에 추적 상태를 안전하게 읽은 다음 추적 상태에 쓰는 것은 불가능합니다.

어설션이 원래 3.15에서 강화되었기 때문에 이러한 유형의 돌연변이를 모두 다루려고 했지만 VM이 내부적으로 자동 추적을 사용하지 않았기 때문에 생성자가 누락되었습니다.

deprecateMutationsInAutotrackingFrame 도우미로 이것을 래핑하여 앱이 회귀하는 것을 방지하기 위해 당분간 어설션 대신 사용 중단을 던질 수 있습니다.

deprecateMutationsInAutotrackingFrame 도우미로 이것을 래핑하여 앱이 회귀하는 것을 방지하기 위해 당분간 어설션 대신 사용 중단을 던질 수 있습니다.

예, 이것은 의미가 있습니다. 지금은 차단이 해제되지만 근본적인 문제를 해결하려면 여전히 애플리케이션/애드온/등이 필요합니다(추적 컨텍스트 내에서 "읽기 후 쓰기"를 지원할 수 없음).

Glimmer 구성 요소와 함께 제공한 예제입니까?

예, Glimmer 구성 요소입니다.

다음은 현재 업그레이드를 차단하고 있는 실제 문제의 예입니다. https://github.com/GavinJoyce/backdrifts/pull/5

생성자에서 모델 데이터를 변경(이 경우 addFragment 호출)하는 것은 확실히 반패턴입니다. 경로 또는 작업 단계에서 모델을 생성할 때 이러한 종류의 모델 돌연변이를 수행할 가능성이 매우 높습니다. . 어설션 대신 지원 중단을 내보내면 이와 같은 문제가 발생한 앱에 이 리팩토링을 수행할 시간이 주어집니다.

어설션 대신 지원 중단을 내보내면 이와 같은 문제가 발생한 앱에 이 리팩토링을 수행할 시간이 주어집니다.

절대적으로 동의합니다! 오늘 아침에 작업해보려고 합니다.

어떤 식 으로든 도울 수 있다면 LMK 감사합니다.

이를 위해서는 https://github.com/glimmerjs/glimmer-vm/pull/1205 에 제출한 glimmer-vm 의 일부 변경이 필요하지만 Ember 통합(지원 중단을 위해)은 https://github에 있습니다. com/emberjs/ember.js/pull/19282.

필요한 Glimmer 변경 사항을 방금 착륙했으며 내일 릴리스를 얻으려고 노력할 것입니다.

모든 개체 유형에 사용 중단(vs assertion) 동작이 있는지 확인해야 하기 때문에 원래 생각했던 것보다 조금 더 복잡했습니다. https://github.com/emberjs/ember.js/pull/19282 가 마침내 검토할 준비가 되었습니다.

다음 단계는 다양한 버전에서 glimmer-vm의 변경 사항을 백포트하고 테스트하는 것입니다(각각 내부적으로 주요 변경 사항이 있음).

• glimmer-vm의 백포트
  
  
  
  • [email protected] - 현재 master 지점 대상 [email protected]
  
  
  • [email protected] - 현재 beta 지점 대상 [email protected]
  
  
  • [email protected] - 현재 release 대한 지점 [email protected]
  
  
• #19282의 변경 사항 백포트
  
  
  
  • [email protected]에 대한 beta 분기
  
  
  • [email protected]에 대한 release 분기
  
  
• 릴리스 변경 사항
  
  
  
  • [email protected]
  
  

아직 100% 확신할 수는 없지만 3.23.1+에서 릴리스할 가능성이 있습니다(3.22는 더 이상 현재 릴리스가 아니기 때문에). 3.22에서 이에 대한 특정 요구 사항이 있고 대신 3.23으로 업데이트할 수 없는 사람이 있습니까?

3.23으로 업데이트하게 되어 기쁩니다.

다른 커피가 필요할 수도 있지만, 우리가 피해야 할 패턴을 완전히 이해하지 못한다는 점을 겸손하게 인정합니다. getter에서 값을 설정한 예를 이해하지만 Gavin의 예가 왜 나쁜지 잘 모르겠습니다(조금 고안된 것일지라도). 누군가가 문제를 교육하는 데 도움을 줄 수 있습니까?

앱을 3.22 또는 3.23으로 업그레이드하기 위해 해야 할 일이 더 많은지 확실하지 않습니다. 우리는 3.22에서 수백 개의 실패한 테스트를 가지고 있으며, 그 중 압도적 다수는 주장을 역추적하는 것으로 보입니다. 이 주장은 귀하의 변경으로 수정되기를 바랍니다. 바라건대 우리는 3.21에서 3.23.1로 바로 이동할 수 있습니다.

추가 부담이 크지 않다면 3.22.2도 출시하는 것이 좋지 않을까요? 한 번에 하나의 버전을 업그레이드하는 사람들은 향후 골칫거리를 줄일 수 있습니다.

참고로 저는 엔진에서 ember-beta 테스트를 실행할 때 이것을 보고 있습니다.

위반 코드:

<LinkToExternal @route="your-coach">{{@coachUser.firstName}}</LinkToExternal>

@jherdman 이 최신 ember-engines 코드를 사용 중인지 확인하십시오. link-to-external 는 이 주장을 피하기 위해 최근에 업데이트되었습니다(https://github.com/ember-engines/ember-engines/pull/735).

나는 실제로 엔진의 v0.8.8에 있다고 말하는 것을 유감스럽게 생각합니다. 이 문제를 탈선시키고 싶지 않습니다. 거기에서 문제를 열어야 합니까?

나는 실제로 엔진의 v0.8.8에 있다고 말하는 것을 유감스럽게 생각합니다. 이 문제를 탈선시키고 싶지 않습니다. 거기에서 문제를 열어야 합니까?

예, 부탁합니다!

다른 커피가 필요할 수도 있지만, 우리가 피해야 할 패턴을 완전히 이해하지 못한다는 점을 겸손하게 인정합니다. getter에서 값을 설정한 예를 이해하지만 Gavin의 예가 왜 나쁜지 잘 모르겠습니다(조금 고안된 것일지라도). 누군가가 문제를 교육하는 데 도움을 줄 수 있습니까?

여기서 개념적 문제는 현재 "추적 프레임"에서 이미 읽거나 소비된 추적된 값을 변경할 때 최종 렌더링된 값이 유효하다는 것을 보장할 수 없다는 것입니다. 예를 들어 다음 설정을 상상해 보십시오.

• 애플리케이션 경로의 무언가가 {{this.someService.randomTrackedProperty}} 액세스합니다.
• 그런 다음 나중에 중첩 경로의 템플릿 어딘가에서 <FooBar /> 와 같은 구성 요소를 호출합니다.
• FooBar 생성자에서 this.someService.randomTrackedProperty = "hello!"

이전 읽기 추적 값을 변경하는 해당 구성 요소는 해당 값에 대한 다른 참조가 이제 유효하지 않음을 의미합니다.

위의 Gavins 예에서 그는 카운터를 증가시켰습니다. 이것은 반드시 기존 값을 읽은 다음 새 값으로 업데이트한다는 것을 의미합니다. 값이 구성 요소에 대해 로컬이고 이 예제에서 다른 코드가 "잘못된" 버전(변경하기 전의 초기 버전)에 액세스할 수 없었기 때문에 그것이 무해해 보인다는 것을 알고 있지만 실제로 는 생성자에 대해 특별한 것이

여기에서 빠른 업데이트, 3.23.1이 수정 사항과 함께 릴리스됩니다. 나는 내일 3.22.x를 완성하려고 노력할 것입니다(체리 따기 문제에 부딪쳤습니다).

알겠습니다. 3.22.2에 태그가 지정되었으며 곧 NPM에 게시되어야 합니다.

3.22.2에 태그를 지정해 주셔서 감사합니다. 먼저 해당 항목으로 이동하는 것이 도움이 될 것입니다.

참고로, 10시간 후에 NPM에 게시되지 않는 것 같습니다. https://www.npmjs.com/package/ember-source/v/3.22.2

방금 수정했습니다. 죄송합니다.

좋아요, 감사합니다!