[Fix] AsyncDisposableStack: continue disposal chain after a throwing defer

Fixes #9

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: 2 people

aos/DisposeResources.js

@@ -11,7 +11,6 @@ var $then = callBound('Promise.prototype.then', true);
 
 var CompletionRecord = require('es-abstract/2025/CompletionRecord');
 var Dispose = require('./Dispose');
-var NormalCompletion = require('es-abstract/2025/NormalCompletion');
 var PromiseResolve = require('es-abstract/2025/PromiseResolve');
 var ThrowCompletion = require('es-abstract/2025/ThrowCompletion');
 
@@ -56,9 +55,8 @@ module.exports = function DisposeResources(disposeCapability, completion) {
 	};
 
 	var getPromise = actualHint === 'ASYNC-DISPOSE' && function getPromise(resource) {
-		return $then(
-			promise,
-			function () {
+		var runDispose = function () {
+			try {
 				var result = Dispose( // step 2.a
 					resource['[[ResourceValue]]'],
 					resource['[[Hint]]'],
@@ -67,9 +65,19 @@ module.exports = function DisposeResources(disposeCapability, completion) {
 				if (!result) {
 					throw new $SyntaxError('Assertion failed: non-`~ASYNC-DISPOSE~` resource returned a promise from Dispose');
 				}
-				return $then(result, NormalCompletion);
-			},
-			rejecter
+				return $then(result, void undefined, rejecter);
+			} catch (e) {
+				rejecter(e);
+			}
+			return void undefined;
+		};
+		return $then(
+			promise,
+			runDispose,
+			function (e) {
+				rejecter(e);
+				return runDispose();
+			}
 		);
 	};
 
@@ -96,5 +104,10 @@ module.exports = function DisposeResources(disposeCapability, completion) {
 	// eslint-disable-next-line no-param-reassign
 	disposeCapability['[[DisposableResourceStack]]'] = null; // step 3
 
-	return actualHint === 'ASYNC-DISPOSE' ? promise : completion; // step 4
+	if (actualHint === 'ASYNC-DISPOSE') { // step 4
+		return $then(promise, function () {
+			return completion;
+		});
+	}
+	return completion;
 };

test/tests.js

@@ -621,7 +621,35 @@ module.exports = {
 					function (e) {
 						st.equal(e, throwSentinel, 'throws `throwSentinel`');
 					}
-				);
+				).then(function () {
+					// https://github.com/es-shims/DisposableStack/issues/9
+					var instance3 = new AsyncDisposableStack();
+					var ran = 0;
+					instance3.defer(function () { ran += 1; });
+					instance3.defer(throwsSentinel);
+
+					return instance3.disposeAsync().then(
+						function () {
+							st.fail('dispose with a throwing later disposable failed to throw');
+						},
+						function (e) {
+							st.equal(e, throwSentinel, 'throws `throwSentinel` when an earlier-pushed defer throws');
+							st.equal(ran, 1, 'later-pushed non-throwing defer still ran');
+						}
+					);
+				}).then(function () {
+					var instance4 = new AsyncDisposableStack();
+					instance4.defer(function () { return Promise.reject(throwSentinel); });
+
+					return instance4.disposeAsync().then(
+						function () {
+							st.fail('dispose with a rejecting disposable failed to throw');
+						},
+						function (e) {
+							st.equal(e, throwSentinel, 'rejecting defer surfaces `throwSentinel`');
+						}
+					);
+				});
 			});
 		});