ember-data-model-fragments 7.0.1 → 7.0.3

package/.release-plan.json

@@ -2,17 +2,17 @@
   "solution": {
     "ember-data-model-fragments": {
       "impact": "patch",
-      "oldVersion": "7.0.0",
-      "newVersion": "7.0.1",
+      "oldVersion": "7.0.2",
+      "newVersion": "7.0.3",
       "tagName": "latest",
       "constraints": [
         {
           "impact": "patch",
-          "reason": "Appears in changelog section :house: Internal"
+          "reason": "Appears in changelog section :bug: Bug Fix"
         }
       ],
       "pkgJSONPath": "./package.json"
     }
   },
-  "description": "## Release (2026-02-06)\n\n* ember-data-model-fragments 7.0.1 (patch)\n\n#### :house: Internal\n* `ember-data-model-fragments`\n  * [#509](https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/509) Update release-plan ([@RobbieTheWagner](https://github.com/RobbieTheWagner))\n\n#### Committers: 1\n- Robbie Wagner ([@RobbieTheWagner](https://github.com/RobbieTheWagner))\n"
+  "description": "## Release (2026-02-27)\n\n* ember-data-model-fragments 7.0.3 (patch)\n\n#### :bug: Bug Fix\n* `ember-data-model-fragments`\n  * [#514](https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/514) Guard fragment access on destroyed/unloaded records ([@deanmarano](https://github.com/deanmarano))\n\n#### Committers: 1\n- Dean Marano ([@deanmarano](https://github.com/deanmarano))\n"
 }

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## Release (2026-02-27)
+
+* ember-data-model-fragments 7.0.3 (patch)
+
+#### :bug: Bug Fix
+* `ember-data-model-fragments`
+  * [#514](https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/514) Guard fragment access on destroyed/unloaded records ([@deanmarano](https://github.com/deanmarano))
+
+#### Committers: 1
+- Dean Marano ([@deanmarano](https://github.com/deanmarano))
+
+## Release (2026-02-11)
+
+* ember-data-model-fragments 7.0.2 (patch)
+
+#### :bug: Bug Fix
+* `ember-data-model-fragments`
+  * [#512](https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/512) Prevent newly created fragments from being marked deleted after save ([@deanmarano](https://github.com/deanmarano))
+
+#### Committers: 1
+- Dean Marano ([@deanmarano](https://github.com/deanmarano))
+
 ## Release (2026-02-06)
 
 * ember-data-model-fragments 7.0.1 (patch)

package/addon/attributes/array.js

@@ -55,6 +55,9 @@ export default function array(type, options) {
   // eslint-disable-next-line ember/require-computed-property-dependencies
   return computed({
     get(key) {
+      if (this.isDestroying || this.isDestroyed) {
+        return null;
+      }
       const identifier = recordIdentifierFor(this);
       const cache = this.store.cache;
       if (cache.getFragment(identifier, key) === null) {

package/addon/attributes/fragment-array.js

@@ -64,6 +64,9 @@ export default function fragmentArray(type, options) {
   // eslint-disable-next-line ember/require-computed-property-dependencies
   return computed({
     get(key) {
+      if (this.isDestroying || this.isDestroyed) {
+        return null;
+      }
       const identifier = recordIdentifierFor(this);
       const cache = this.store.cache;
       if (cache.getFragment(identifier, key) === null) {

package/addon/attributes/fragment-owner.js

@@ -1,5 +1,6 @@
 import { assert } from '@ember/debug';
 import { computed } from '@ember/object';
+import { isDestroying, isDestroyed } from '@ember/destroyable';
 import { isFragment } from '../fragment';
 import { recordIdentifierFor } from '@ember-data/store';
 
@@ -27,7 +28,15 @@ import { recordIdentifierFor } from '@ember-data/store';
  @return {Attribute}
  */
 export default function fragmentOwner() {
-  return computed('store.{_instanceCache,cache}', function () {
+  // No dependent keys: the value is invalidated via notifyPropertyChange in
+  // setFragmentOwner(). Omitting dependent keys also avoids Ember's
+  // "Attempted to access the computed ... on a destroyed object" assertion
+  // when a fragment is torn down.
+  // eslint-disable-next-line ember/require-computed-property-dependencies
+  return computed(function () {
+    if (isDestroying(this) || isDestroyed(this)) {
+      return null;
+    }
     assert(
       'Fragment owner properties can only be used on fragments.',
       isFragment(this),

package/addon/attributes/fragment.js

@@ -60,8 +60,12 @@ export default function fragment(type, options) {
     options,
   };
 
+  // eslint-disable-next-line ember/require-computed-property-dependencies -- isDestroying/isDestroyed are guards, not dependencies
   return computed('store.{_instanceCache,cache}', {
     get(key) {
+      if (this.isDestroying || this.isDestroyed) {
+        return null;
+      }
       const identifier = recordIdentifierFor(this);
       const cache = this.store.cache;
       const fragmentIdentifier = cache.getFragment(identifier, key);

package/addon/cache/fragment-state-manager.js

@@ -1161,14 +1161,16 @@ export default class FragmentStateManager {
       }
     }
 
+    // Upsert the committed values as new canonical state BEFORE rollback.
+    // This clears isNew on the inner cache, which prevents rollbackAttrs
+    // from incorrectly setting isDeleted=true for newly created fragments.
+    // (rollbackAttrs treats isNew records as "discard new record" and marks
+    // them deleted, but here we want to transition them to saved state.)
+    innerCache.upsert(identifier, { attributes: commitAttrs }, false);
+
     // Rollback the inner cache to clear dirty/in-flight tracking
     innerCache.rollbackAttrs(identifier);
 
-    // Push the committed values as the new canonical state
-    if (Object.keys(commitAttrs).length > 0) {
-      innerCache.upsert(identifier, { attributes: commitAttrs }, false);
-    }
-
     // Re-apply any new dirty changes made during in-flight
     for (const [key, value] of Object.entries(newDirtyAttrs)) {
       innerCache.setAttr(identifier, key, value);

package/addon/fragment.js

@@ -1,5 +1,6 @@
 import { get, computed } from '@ember/object';
 import Ember from 'ember';
+import { isDestroying, isDestroyed } from '@ember/destroyable';
 // DS.Model gets munged to add fragment support, which must be included first
 import { Model } from './ext';
 import { copy } from './util/copy';
@@ -115,6 +116,9 @@ const Fragment = Model.extend(Ember.Comparable, {
   },
 
   toStringExtension() {
+    if (isDestroying(this) || isDestroyed(this)) {
+      return '';
+    }
     const identifier = recordIdentifierFor(this);
     const owner = this.store.cache.getFragmentOwner(identifier);
     return owner ? `owner(${owner.ownerIdentifier?.id})` : '';
@@ -125,6 +129,9 @@ const Fragment = Model.extend(Ember.Comparable, {
     ember-data 4.12+ doesn't call toStringExtension in Model.toString().
   */
   toString() {
+    if (isDestroying(this) || isDestroyed(this)) {
+      return `<fragment(destroyed)>`;
+    }
     const identifier = recordIdentifierFor(this);
     const extension = this.toStringExtension();
     const extensionStr = extension ? `:${extension}` : '';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-data-model-fragments",
-  "version": "7.0.1",
+  "version": "7.0.3",
   "description": "Ember Data addon to support nested JSON documents",
   "keywords": [
     "ember-addon",