[FEATURE] Add AI-based /ask command (complementing #54) #56

2025-05-08T03:06:10Z

lucmsilva651 commented

2025-05-08 03:06:10 +00:00

(Migrated from github.com)

No description provided.

GiovaniFZ (Migrated from github.com) approved these changes 2025-06-06 17:04:42 +00:00

GiovaniFZ (Migrated from github.com) left a comment

Alr

src/commands/ai.ts

					
				@ -0,0 +92,4 @@

				      }

				    )

				    let fullResponse = ""

GiovaniFZ (Migrated from github.com) commented

2025-06-06 17:02:36 +00:00

I'm not sure about what it does, but always prefer to use const whenever it's possible

ihatenodejs commented

2025-06-27 07:22:42 +00:00

(Migrated from github.com)

@lucmsilva651 Everything is good now, if you want to test and/or merge.

GiovaniFZ (Migrated from github.com) requested changes 2025-06-28 13:58:44 +00:00

src/commands/ai.ts

GiovaniFZ (Migrated from github.com) commented

2025-06-28 13:52:15 +00:00

Don't use axios like this, check https://axios-http.com/docs/instance

GiovaniFZ (Migrated from github.com) commented

2025-06-28 13:53:07 +00:00

Why are you printing p1?

GiovaniFZ (Migrated from github.com) commented

2025-06-28 13:56:48 +00:00

There are too many ifs and elses, not only in this file but others too. Maybe we could separate them soon. let's try to do this in the future ;)

src/utils/rate-limiter.ts

GiovaniFZ (Migrated from github.com) commented

2025-06-28 13:58:13 +00:00

pls, avoid to set any as a type

ihatenodejs commented

2025-06-28 16:06:39 +00:00

(Migrated from github.com)

Tysm for the review @GiovaniFZ !! I'll take a look

ihatenodejs (Migrated from github.com) reviewed 2025-06-28 19:20:51 +00:00

ihatenodejs (Migrated from github.com) left a comment

All suggestions implemented, and the files are cleaned up now

src/commands/ai.ts

ihatenodejs (Migrated from github.com) commented

2025-06-28 16:13:06 +00:00

I like this idea! I will look into cleaning it up, and maybe break some things into functions so the code is easier to read

src/commands/ai.ts

					
				@ -0,0 +92,4 @@

				      }

				    )

				    let fullResponse = ""

ihatenodejs (Migrated from github.com) commented

2025-06-28 16:11:54 +00:00

LLMs essentially stream their output, which is fetched through API (done above with the axios.post()).

I used let because as the response comes in, fullResponse contains the latest complete response (it's live updating) which gets edited on Telegram. Hope this clears it up!

LLMs essentially stream their output, which is fetched through API (done above with the `axios.post()`). I used let because as the response comes in, `fullResponse` contains the latest complete response (it's live updating) which gets edited on Telegram. Hope this clears it up!

Sign in to join this conversation.

No reviewers

GiovaniFZ