Deal with encoded characters better

Author: matt-aitken

apps/webapp/app/components/runs/v3/RunFilters.tsx

@@ -280,7 +280,7 @@ export function getRunFiltersFromSearchParams(
     bulkId: searchParams.get("bulkId") ?? undefined,
     tags:
       searchParams.getAll("tags").filter((v) => v.length > 0).length > 0
-        ? searchParams.getAll("tags").map((t) => decodeURIComponent(t))
+        ? searchParams.getAll("tags")
         : undefined,
     from: searchParams.get("from") ?? undefined,
     to: searchParams.get("to") ?? undefined,
@@ -836,7 +836,7 @@ function TagsDropdown({
   useEffect(() => {
     const searchParams = new URLSearchParams();
     if (searchValue) {
-      searchParams.set("name", encodeURIComponent(searchValue));
+      searchParams.set("name", searchValue);
     }
     if (period) {
       searchParams.set("period", period);
@@ -973,7 +973,7 @@ function QueuesDropdown({
       const searchParams = new URLSearchParams();
       searchParams.set("per_page", "25");
       if (searchValue) {
-        searchParams.set("query", encodeURIComponent(s));
+        searchParams.set("query", s);
       }
       fetcher.load(
         `/resources/orgs/${organization.slug}/projects/${project.slug}/env/${
@@ -1235,7 +1235,7 @@ function VersionsDropdown({
     (s) => {
       const searchParams = new URLSearchParams();
       if (searchValue) {
-        searchParams.set("query", encodeURIComponent(s));
+        searchParams.set("query", s);
       }
       fetcher.load(
         `/resources/orgs/${organization.slug}/projects/${project.slug}/env/${

apps/webapp/app/routes/resources.environments.$envId.runs.tags.tsx

@@ -34,10 +34,9 @@ export async function loader({ request, params }: LoaderFunctionArgs) {
   }
 
   const search = new URL(request.url).searchParams;
-  const name = search.get("name");
 
   const parsedSearchParams = SearchParams.safeParse({
-    name: name ? decodeURIComponent(name) : undefined,
+    name: search.get("name") ?? undefined,
     period: search.get("period") ?? undefined,
     from: search.get("from") ?? undefined,
     to: search.get("to") ?? undefined,