add TODO comment

Author: avivkeller

lib/internal/test_runner/test.js

@@ -264,8 +264,14 @@ class TestContext {
         };
       });
 
-      // assert.ok needs it's own function, as it checks the stack
-      // trace to provide more descriptive errors.
+      // TODO(RedYetiDev): This is a hack. It should be replaced with better handling in
+      // internal/assert/util.js
+      //
+      // The innerOk function receives a reference to the userland function that invoked it
+      // to correctly generate the stack trace. However, when each assert method is wrapped
+      // (for test planning purposes), the reference passed to innerOk no longer points to
+      // the original userland function. To address this issue, the ok function is implemented
+      // below without using the wrapper, allowing innerOk to receive the correct reference.
       function ok(...args) {
         if (plan !== null) {
           plan.actual++;