Sort computed decorator in Octane gives "this.notifyPropertyChange is not a function" error

[CodingItWrong]

I'm attempting to sort a list of Ember Data records, and add a new record to the list. In stable this works fine, but in Octane I get the following error (error shown is from Firefox):

TypeError: this.notifyPropertyChange is not a function

Minimal reproduction: https://github.com/CodingItWrong/octane-sort-test
Comparable stable app showing the error does not occur: https://github.com/CodingItWrong/ember-sort-test

Here's the class where the sorting happens:

import Component from '@glimmer/component';
import { action } from '@ember/object';
import { sort } from '@ember/object/computed';
import { inject as service } from '@ember/service';

export default class WidgetListComponent extends Component {
  @service store;

  sortProperties = Object.freeze(['id:desc']);

  @sort('args.widgets', 'sortProperties')
  sortedWidgets;

  @action
  addWidget() {
    const widget = this.store.createRecord('widget', { name: 'New Widget' });
    widget.save();
  }
}

[rwjblue]

Can you share the full stack trace?

[CodingItWrong]

@rwjblue Sure thing:

TypeError: this.notifyPropertyChange is not a function - reduce_computed_macros.js:983
    Ember
        propertySort
        sendEvent
        notifyObservers
        notifyPropertyChange
        notify
        chainsDidChange
        notifyPropertyChange
        arrayDidChange
        eachProxyArrayDidChange
        arrayContentDidChange
        arrayContentDidChange
        _arrangedContentArrayDidChange
        sendEvent
        arrayContentDidChange
        replaceInNativeArray
        replace
        pushObjects
        _pushInternalModels
        updateLiveRecordArray
        updateLiveRecordArray
        _flushPendingInternalModelsForModelName
        _flush

    invoke backburner.js:341
    flush backburner.js:233
    flush backburner.js:437
    _end backburner.js:1013
    end backburner.js:699
    _run backburner.js:1068
    _join backburner.js:1042
    join backburner.js:759
    createRecord Ember
    addWidget widget-list.js:29
    addWidget self-hosted:1003

[rwjblue]

The issue is here:

ember.js/packages/@ember/object/lib/computed/reduce_computed_macros.js

Line 971 in 5e4b58f

this.notifyPropertyChange(key);

The reduced macros can absolutely not assume that this.notifyPropertyChange is "a thing".

I've submitted #17841 to fix.

[CodingItWrong]

Thanks, running locally off of that PR's branch confirms that it fixes the issue for me