fix(api): clean up API routes and remove debug code

- Remove debug echo statement from status API route
- Fix route pattern for /api/i/user/@user_id by removing trailing optional slash
- Improve code formatting by consolidating condition checks
- Add validation condition for username parameter in timeline API route

These changes improve code quality and ensure API routes work correctly.

Author: guanbinrui

src/jsons/status.nim

@@ -55,8 +55,6 @@ proc createJsonApiStatusRouter*(cfg: Config) =
         respJsonError "Invalid tweet ID"
 
       let conv = await getTweet(id, getCursor())
-      if conv == nil:
-        echo "nil conv"
 
       if conv == nil or conv.tweet == nil or conv.tweet.id == 0:
         var error = "Tweet not found"

src/jsons/timeline.nim

@@ -115,7 +115,7 @@ proc formatProfileAsJson*(profile: Profile): JsonNode =
 
 proc createJsonApiTimelineRouter*(cfg: Config) =
   router jsonapi_timeline:
-    get "/api/i/user/@user_id/?":
+    get "/api/i/user/@user_id":
       cond @"user_id".len > 0
       let username = await getCachedUsername(@"user_id")
       if username.len > 0:
@@ -141,8 +141,8 @@ proc createJsonApiTimelineRouter*(cfg: Config) =
 
     get "/api/@name/?@tab?/?":
       cond '.' notin @"name"
-      cond @"name" notin ["pic", "gif", "video", "search", "settings", "login",
-          "intent", "i"]
+      cond @"name" notin ["pic", "gif", "video", "search", "settings", "login", "intent", "i"]
+      cond @"name".allCharsInSet({'a'..'z', 'A'..'Z', '0'..'9', '_', ','})
       cond @"tab" in ["with_replies", "media", "search", ""]
       let
         prefs = cookiePrefs()