Stop moving out of KJ_IF_SOME

Author: npaun

src/workerd/api/http.c++

@@ -1051,6 +1051,7 @@ jsg::Ref<Request> Request::constructor(
           // explicitly say `signal: null`, they must want to drop the signal that was on the
           // original request.
           signal = kj::mv(s);
+          initDict.signal = kj::none;
         }
 
         KJ_IF_SOME(newCf, initDict.cf) {

src/workerd/api/http.h

@@ -787,14 +787,14 @@ class Request final: public Body {
       // that the cancel machinery is not used but the request.signal accessor will still
       // do the right thing.
       if (s->getNeverAborts()) {
-        this->thisSignal = kj::mv(s);
+        this->thisSignal = s.addRef();
       } else {
-        this->signal = kj::mv(s);
+        this->signal = s.addRef();
       }
     }
 
     KJ_IF_SOME(s, thisSignal) {
-      this->thisSignal = kj::mv(s);
+      this->thisSignal = s.addRef();
     }
   }
   // TODO(conform): Technically, the request's URL should be parsed immediately upon Request

src/workerd/io/worker-entrypoint.c++

@@ -380,7 +380,7 @@ kj::Promise<void> WorkerEntrypoint::request(kj::HttpMethod method,
       // listener.
       KJ_IF_SOME(ctrl, abortController) {
         context.addWaitUntil(context.run(
-            [ctrl = kj::mv(ctrl)](Worker::Lock& lock) mutable { ctrl->abort(lock, kj::none); }));
+            [ctrl = ctrl.addRef()](Worker::Lock& lock) mutable { ctrl->abort(lock, kj::none); }));
       }
     }