From 302e8fbb8f327f14835dd31e15f7f9b34dd2ee91 Mon Sep 17 00:00:00 2001
From: ReaJason <reajason1225@gmail.com>
Date: Mon, 12 Jan 2026 18:19:12 +0800
Subject: [PATCH] refactor(ui): change serverVersion field localtion

---
 .../memshell/results/basic-info.tsx           |   4 +-
 .../components/memshell/tabs/antsword-tab.tsx |   6 +-
 .../components/memshell/tabs/behinder-tab.tsx |   6 +-
 .../components/memshell/tabs/command-tab.tsx  |   6 +-
 .../components/memshell/tabs/custom-tab.tsx   |  51 +++---
 .../components/memshell/tabs/godzilla-tab.tsx |   6 +-
 .../components/memshell/tabs/neoreg-tab.tsx   |   6 +-
 .../memshell/tabs/shelltype-field.tsx         | 154 ++++++++++++++----
 web/app/components/memshell/tabs/suo5-tab.tsx |   6 +-
 .../memshell/tabs/urlpattern-field.tsx        |  29 ----
 web/app/lib/utils.ts                          |   2 +-
 11 files changed, 154 insertions(+), 122 deletions(-)
 delete mode 100644 web/app/components/memshell/tabs/urlpattern-field.tsx

diff --git a/web/app/components/memshell/results/basic-info.tsx b/web/app/components/memshell/results/basic-info.tsx
index 09661c2d..7ed9180b 100644
--- a/web/app/components/memshell/results/basic-info.tsx
+++ b/web/app/components/memshell/results/basic-info.tsx
@@ -2,7 +2,7 @@ import { FileTextIcon } from "lucide-react";
 import { useTranslation } from "react-i18next";
 import { Card, CardContent, CardHeader, CardTitle } from "@/components/ui/card";
 import { Separator } from "@/components/ui/separator";
-import { shouldHidden } from "@/lib/utils";
+import { notNeedUrlPattern } from "@/lib/utils";
 import {
   type AntSwordShellToolConfig,
   type BehinderShellToolConfig,
@@ -45,7 +45,7 @@ export function BasicInfo({
             label={t("mainConfig.shellMountType")}
             text={generateResult?.shellConfig.shellType}
           />
-          {!shouldHidden(generateResult?.shellConfig?.shellType) && (
+          {!notNeedUrlPattern(generateResult?.shellConfig?.shellType) && (
             <CopyableField
               label={t("mainConfig.urlPattern")}
               text={generateResult?.injectorConfig.urlPattern}
diff --git a/web/app/components/memshell/tabs/antsword-tab.tsx b/web/app/components/memshell/tabs/antsword-tab.tsx
index 54224cff..264b44c2 100644
--- a/web/app/components/memshell/tabs/antsword-tab.tsx
+++ b/web/app/components/memshell/tabs/antsword-tab.tsx
@@ -7,7 +7,6 @@ import { TabsContent } from "@/components/ui/tabs";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function AntSwordTabContent({
   form,
@@ -21,10 +20,7 @@ export function AntSwordTabContent({
     <TabsContent value="AntSword">
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <Controller
             control={form.control}
             name="antSwordPass"
diff --git a/web/app/components/memshell/tabs/behinder-tab.tsx b/web/app/components/memshell/tabs/behinder-tab.tsx
index bb662ad1..b5997780 100644
--- a/web/app/components/memshell/tabs/behinder-tab.tsx
+++ b/web/app/components/memshell/tabs/behinder-tab.tsx
@@ -7,7 +7,6 @@ import { TabsContent } from "@/components/ui/tabs";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function BehinderTabContent({
   form,
@@ -21,10 +20,7 @@ export function BehinderTabContent({
     <TabsContent value="Behinder">
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <Controller
             control={form.control}
             name="behinderPass"
diff --git a/web/app/components/memshell/tabs/command-tab.tsx b/web/app/components/memshell/tabs/command-tab.tsx
index bff17206..a2bab9da 100644
--- a/web/app/components/memshell/tabs/command-tab.tsx
+++ b/web/app/components/memshell/tabs/command-tab.tsx
@@ -28,7 +28,6 @@ import { env } from "@/config";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function CommandTabContent({
   form,
@@ -54,10 +53,7 @@ export function CommandTabContent({
     <TabsContent value="Command">
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <Controller
             control={form.control}
             name="commandParamName"
diff --git a/web/app/components/memshell/tabs/custom-tab.tsx b/web/app/components/memshell/tabs/custom-tab.tsx
index 21fe7f14..a3a67639 100644
--- a/web/app/components/memshell/tabs/custom-tab.tsx
+++ b/web/app/components/memshell/tabs/custom-tab.tsx
@@ -13,7 +13,6 @@ import { env } from "@/config";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export default function CustomTabContent({
   form,
@@ -87,15 +86,12 @@ export default function CustomTabContent({
     <TabsContent value="Custom">
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <Controller
             control={form.control}
             name="shellClassBase64"
             render={({ field, fieldState }) => (
-              <Field className="gap-1">
+              <Field className="gap-2">
                 <FieldLabel>{t("shellClass")}</FieldLabel>
                 <RadioGroup
                   value={isFile ? "file" : "base64"}
@@ -105,35 +101,38 @@ export default function CustomTabContent({
                   }}
                   className="flex items-center"
                 >
-                  <div className="flex items-center">
+                  <div className="flex items-center gap-1">
                     <RadioGroupItem value="base64" id="optionOne" />
                     <Label htmlFor="optionOne">Base64</Label>
                   </div>
-                  <div className="flex items-center">
+                  <div className="flex items-center gap-1">
                     <RadioGroupItem value="file" id="optionTwo" />
                     <Label htmlFor="optionTwo">File</Label>
                   </div>
                 </RadioGroup>
                 <div className="mt-2">
                   {isFile ? (
-                    <Input
-                      onChange={(e) => {
-                        const file = e.target.files?.[0];
-                        if (file) {
-                          const reader = new FileReader();
-                          reader.onload = (event) => {
-                            const base64String =
-                              (event.target?.result as string)?.split(",")[1] ||
-                              "";
-                            field.onChange(base64String);
-                          };
-                          reader.readAsDataURL(file);
-                        }
-                      }}
-                      accept=".class"
-                      placeholder={t("common:placeholders.input")}
-                      type="file"
-                    />
+                    <div className="grid w-full max-w-sm items-center gap-3">
+                      <Input
+                        onChange={(e) => {
+                          const file = e.target.files?.[0];
+                          if (file) {
+                            const reader = new FileReader();
+                            reader.onload = (event) => {
+                              const base64String =
+                                (event.target?.result as string)?.split(
+                                  ",",
+                                )[1] || "";
+                              field.onChange(base64String);
+                            };
+                            reader.readAsDataURL(file);
+                          }
+                        }}
+                        accept=".class"
+                        placeholder={t("common:placeholders.input")}
+                        type="file"
+                      />
+                    </div>
                   ) : (
                     <Textarea
                       {...field}
diff --git a/web/app/components/memshell/tabs/godzilla-tab.tsx b/web/app/components/memshell/tabs/godzilla-tab.tsx
index f26c182d..e0f57dae 100644
--- a/web/app/components/memshell/tabs/godzilla-tab.tsx
+++ b/web/app/components/memshell/tabs/godzilla-tab.tsx
@@ -7,7 +7,6 @@ import { TabsContent } from "@/components/ui/tabs";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function GodzillaTabContent({
   form,
@@ -21,10 +20,7 @@ export function GodzillaTabContent({
     <TabsContent value="Godzilla">
       <Card>
         <CardContent className="flex flex-col gap-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
             <Controller
               control={form.control}
diff --git a/web/app/components/memshell/tabs/neoreg-tab.tsx b/web/app/components/memshell/tabs/neoreg-tab.tsx
index 0579927a..74a82440 100644
--- a/web/app/components/memshell/tabs/neoreg-tab.tsx
+++ b/web/app/components/memshell/tabs/neoreg-tab.tsx
@@ -7,7 +7,6 @@ import { TabsContent } from "@/components/ui/tabs";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function NeoRegTabContent({
   form,
@@ -21,10 +20,7 @@ export function NeoRegTabContent({
     <TabsContent value="NeoreGeorg">
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
             <Controller
               control={form.control}
diff --git a/web/app/components/memshell/tabs/shelltype-field.tsx b/web/app/components/memshell/tabs/shelltype-field.tsx
index fe7a5cc6..8773d4a0 100644
--- a/web/app/components/memshell/tabs/shelltype-field.tsx
+++ b/web/app/components/memshell/tabs/shelltype-field.tsx
@@ -1,6 +1,12 @@
-import { Controller, type UseFormReturn } from "react-hook-form";
+import { Controller, type UseFormReturn, useWatch } from "react-hook-form";
 import { useTranslation } from "react-i18next";
-import { Field, FieldLabel } from "@/components/ui/field";
+import {
+  Field,
+  FieldContent,
+  FieldError,
+  FieldLabel,
+} from "@/components/ui/field";
+import { Input } from "@/components/ui/input";
 import {
   Select,
   SelectContent,
@@ -8,6 +14,7 @@ import {
   SelectTrigger,
   SelectValue,
 } from "@/components/ui/select";
+import { cn, notNeedUrlPattern } from "@/lib/utils";
 import type { MemShellFormSchema } from "@/types/schema";
 
 export function ShellTypeFormField({
@@ -18,39 +25,118 @@ export function ShellTypeFormField({
   shellTypes: Array<string>;
 }>) {
   const { t } = useTranslation(["memshell", "common"]);
+  const shellType = useWatch({ control: form.control, name: "shellType" });
+  const server = useWatch({ control: form.control, name: "server" });
+  const needSelectServerVersion =
+    (server === "TongWeb" && shellType.endsWith("Valve")) ||
+    (server === "Jetty" &&
+      (shellType === "Handler" || shellType === "JakartaHandler"));
+  const needUrlPattern = !notNeedUrlPattern(shellType);
+  const serverVersionOptions = getServerVersionOptions(server);
   return (
-    <Controller
-      control={form.control}
-      name="shellType"
-      render={({ field }) => (
-        <Field className="gap-1">
-          <FieldLabel>{t("mainConfig.shellMountType")}</FieldLabel>
-          <Select
-            onValueChange={(e) => {
-              form.resetField("urlPattern");
-              field.onChange(e);
-            }}
-            value={field.value}
-          >
-            <SelectTrigger>
-              <SelectValue data-placeholder={t("common:placeholders.select")} />
-            </SelectTrigger>
-            <SelectContent key={shellTypes?.join(",")}>
-              {shellTypes?.length ? (
-                shellTypes.map((v) => (
-                  <SelectItem key={v} value={v}>
-                    {v}
+    <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
+      <Controller
+        control={form.control}
+        name="shellType"
+        render={({ field }) => (
+          <Field className="gap-1">
+            <FieldLabel>{t("mainConfig.shellMountType")}</FieldLabel>
+            <Select
+              onValueChange={(e) => {
+                form.resetField("urlPattern");
+                field.onChange(e);
+              }}
+              value={field.value}
+            >
+              <SelectTrigger>
+                <SelectValue
+                  data-placeholder={t("common:placeholders.select")}
+                />
+              </SelectTrigger>
+              <SelectContent key={shellTypes?.join(",")}>
+                {shellTypes?.length ? (
+                  shellTypes.map((v) => (
+                    <SelectItem key={v} value={v}>
+                      {v}
+                    </SelectItem>
+                  ))
+                ) : (
+                  <SelectItem value=" ">
+                    {t("tips.shellToolNotSelected")}
                   </SelectItem>
-                ))
-              ) : (
-                <SelectItem value=" ">
-                  {t("tips.shellToolNotSelected")}
-                </SelectItem>
-              )}
-            </SelectContent>
-          </Select>
-        </Field>
-      )}
-    />
+                )}
+              </SelectContent>
+            </Select>
+          </Field>
+        )}
+      />
+      <Controller
+        control={form.control}
+        name="urlPattern"
+        render={({ field, fieldState }) => (
+          <Field
+            className={cn("gap-1", needUrlPattern ? "grid" : "hidden")}
+            data-invalid={fieldState.invalid}
+          >
+            <FieldLabel>{t("common:urlPattern")}</FieldLabel>
+            <Input {...field} placeholder={t("common:placeholders.input")} />
+            {fieldState.error && <FieldError errors={[fieldState.error]} />}
+          </Field>
+        )}
+      />
+      <Controller
+        control={form.control}
+        name="serverVersion"
+        render={({ field, fieldState }) => (
+          <Field
+            className={cn("gap-1", needSelectServerVersion ? "grid" : "hidden")}
+            orientation="vertical"
+            data-invalid={fieldState.invalid}
+          >
+            <FieldContent>
+              <FieldLabel htmlFor="serverVersion">
+                {t("common:serverVersion")}
+              </FieldLabel>
+              <Select onValueChange={field.onChange} value={field.value}>
+                <SelectTrigger
+                  id="serverVersion"
+                  aria-invalid={fieldState.invalid}
+                >
+                  <SelectValue
+                    data-placeholder={t("common:placeholders.select")}
+                  />
+                </SelectTrigger>
+                <SelectContent>
+                  {serverVersionOptions.map((v) => (
+                    <SelectItem key={v.value} value={v.value}>
+                      {v.name}
+                    </SelectItem>
+                  ))}
+                </SelectContent>
+              </Select>
+              {fieldState.error && <FieldError errors={[fieldState.error]} />}
+            </FieldContent>
+          </Field>
+        )}
+      />
+    </div>
   );
 }
+
+function getServerVersionOptions(server: string) {
+  let serverVersionOptions = [] as { name: string; value: string }[];
+  if (server === "TongWeb") {
+    serverVersionOptions = [
+      { name: "6", value: "6" },
+      { name: "7", value: "7" },
+      { name: "8", value: "8" },
+    ];
+  } else if (server === "Jetty") {
+    serverVersionOptions = [
+      { name: "6", value: "6" },
+      { name: "7+", value: "7+" },
+      { name: "12", value: "12" },
+    ];
+  }
+  return serverVersionOptions;
+}
diff --git a/web/app/components/memshell/tabs/suo5-tab.tsx b/web/app/components/memshell/tabs/suo5-tab.tsx
index 8ffce0d1..cd178ad4 100644
--- a/web/app/components/memshell/tabs/suo5-tab.tsx
+++ b/web/app/components/memshell/tabs/suo5-tab.tsx
@@ -7,7 +7,6 @@ import { TabsContent } from "@/components/ui/tabs";
 import type { MemShellFormSchema } from "@/types/schema";
 import { OptionalClassFormField } from "./classname-field";
 import { ShellTypeFormField } from "./shelltype-field";
-import { UrlPatternFormField } from "./urlpattern-field";
 
 export function Suo5TabContent({
   form,
@@ -23,10 +22,7 @@ export function Suo5TabContent({
     <TabsContent value={tabValue}>
       <Card>
         <CardContent className="space-y-2 mt-4">
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
-            <ShellTypeFormField form={form} shellTypes={shellTypes} />
-            <UrlPatternFormField form={form} />
-          </div>
+          <ShellTypeFormField form={form} shellTypes={shellTypes} />
           <div className="grid grid-cols-1 md:grid-cols-2 gap-2">
             <Controller
               control={form.control}
diff --git a/web/app/components/memshell/tabs/urlpattern-field.tsx b/web/app/components/memshell/tabs/urlpattern-field.tsx
deleted file mode 100644
index 604b8af1..00000000
--- a/web/app/components/memshell/tabs/urlpattern-field.tsx
+++ /dev/null
@@ -1,29 +0,0 @@
-import { Controller, type UseFormReturn } from "react-hook-form";
-import { useTranslation } from "react-i18next";
-import { Field, FieldError, FieldLabel } from "@/components/ui/field";
-import { Input } from "@/components/ui/input";
-import { cn, shouldHidden } from "@/lib/utils";
-import type { MemShellFormSchema } from "@/types/schema";
-
-export function UrlPatternFormField({
-  form,
-}: Readonly<{ form: UseFormReturn<MemShellFormSchema> }>) {
-  const { t } = useTranslation("common");
-  const shellType = form.watch("shellType");
-  return (
-    <Controller
-      control={form.control}
-      name="urlPattern"
-      render={({ field, fieldState }) => (
-        <Field
-          className={cn("gap-1", shouldHidden(shellType) ? "hidden" : "grid")}
-          data-invalid={fieldState.invalid}
-        >
-          <FieldLabel>{t("urlPattern")}</FieldLabel>
-          <Input {...field} placeholder={t("placeholders.input")} />
-          {fieldState.error && <FieldError errors={[fieldState.error]} />}
-        </Field>
-      )}
-    />
-  );
-}
diff --git a/web/app/lib/utils.ts b/web/app/lib/utils.ts
index 41b91379..312c2b0c 100644
--- a/web/app/lib/utils.ts
+++ b/web/app/lib/utils.ts
@@ -69,7 +69,7 @@ export function formatBytes(bytes: number) {
   return `${kbValue} ${sizes[1]}`;
 }
 
-export function shouldHidden(shellType: string | undefined) {
+export function notNeedUrlPattern(shellType: string | undefined) {
   if (shellType === undefined) {
     return false;
   }