Fix turbo redirect

The previous fix doesn't quite work as turbo just ends up doing a
browser native visit and breaks on form action due to 200 status.

app/helpers.php

@@ -942,7 +942,15 @@ function page_title()
 function ujs_redirect($url, $status = 200)
 {
     $request = Request::instance();
-    if ($request->headers->get('x-turbo-request-id') !== null || ($request->ajax() && !$request->isMethod('get'))) {
+    if ($request->headers->get('x-turbo-request-id') !== null) {
+        if ($status === 200 && $request->getMethod() !== 'GET') {
+            // Turbo doesn't like 200 response on non-GET requests.
+            // Reference: https://github.com/hotwired/turbo/issues/22
+            $status = 201;
+        }
+
+        return response($url, $status, ['content-type' => 'text/osu-turbo-redirect']);
+    } elseif ($request->ajax() && $request->getMethod() !== 'GET') {
         return ext_view('layout.ujs-redirect', compact('url'), 'js', $status);
     } else {
         // because non-3xx redirects make no sense.

resources/js/setup-turbo.ts

@@ -24,6 +24,24 @@ document.addEventListener('turbo:submit-end', (e) => {
   }
 });
 
+document.addEventListener('turbo:before-fetch-response', (e) => {
+  if (!e.detail.fetchResponse.contentType.match(/^text\/osu-turbo-redirect[ ;]*/)) {
+    return;
+  }
+
+  e.preventDefault();
+  e.detail.fetchResponse.responseText.then((url) => {
+    const [currentUrlBase, urlBase] = [document.location.href, url].map((u) => u.replace(/#.*/, ''));
+
+    if (currentUrlBase === urlBase) {
+      // a normal/advance visit to same url won't reload the page
+      Turbo.visit(url, { action: 'replace' });
+    } else {
+      Turbo.visit(url);
+    }
+  });
+});
+
 // disable turbo navigation for old webs
 document.addEventListener('turbo:click', (event) => {
   const url = new URL(event.detail.url);