insyri / bhop-bot-ts

🔁 Rewrite of drumman22's Bhop Bot for the Roblox Bhop / Surf Discord in TypeScript, including legacy functions.

👋 Hey, I'm insyri, a highschool student who does software engineering as a hobby.

I started programming seriously in 2021-2022, creating developer tools and small projects. From then, I've evolved to become more intentive and careful with the way I create programs, investigating optimizations heavily within my study.

I'm currently studying Linear Algebra (leading into computer graphics), models of computation, the history and philosophy of mathematics and efficient memory management practices for system-critical programs. I apply these bits of knowledge into some projects of mine, but eventually, I plan to create programs specific to these topics to prove my own mastery to myself.

I'm currently the president of computer science club at my school, providing an introduction to programming architecture and software tooling by developing mini-programs (like Michele Pratusevich's Terminus) and encouraging students to go out and make something they've always wanted to make— you wanna make a game on Roblox? Go for it, I'll help you find resources.

Before the summer, I showed just how approachable computer science is with the right mindset; from building a simple snow program (with written assessments and proposals of the code!) to hosting an (incomplete) course on learning software development. Over the summer, I'm teaching students the Kotlin programming language. When we come back, we'll be building an app to save students from missing their buses at school.

🌌 software socials

🖥 development information

🏗 projects

rsource

rsource is a project aimed to create services around the Roblox StrafesNET ecosystem, including map development support, game unifying creations, and constructing transcendent experiences. View the full introductory showcase at the rsource github organization and at the rsource website.

Note

This project has been postponed and temporarily discontuned as the StrafesNET API, the services that this project is built on, has been out of commision for a long time, and has no anticipated reinstitution date.

tpy

A strongly typed Pylon API client. https://pylon.bot/

pfs

📂 Server for pasting, uploading, and downloading files.

🏫 learning soon

bhop-bot-ts's Issues

[Tracking] Optimizing readability/performance for the insardia command.

Issue Information

This issue will be held for optimization notes for the insardia command.

Goal: Less "wtfs" on the code; code clarity, readability, and removing unnecessary parts.

Tracking

• #9
• #10
• #11
• #12

P1: alternatives map

bhop-bot-ts/src/commands/fun/insardia.ts

Lines 9 to 15 in 50e59f9

	let alternatives = new Map();
	alternatives.set("hi", ["hi", "hizies"]),
	alternatives.set("hey", ["hey", "heya"]),
	alternatives.set("and", ["and", "n"]),
	alternatives.set("you", ["yu", "you", "u"]),
	alternatives.set("are", ["are", "r"]),
	alternatives.set("got", ["got", "gto", "tog"]);

I forgot why exactly I used a map here, I think this can be shortened into an object for less memory movement. Implementation would be rather easy as it would basically transform the Map to an Object using Object.fromEntries, demo:

let alternatives = new Map();
alternatives.set("hi", ["hi", "hizies"]),
  alternatives.set("hey", ["hey", "heya"]),
  alternatives.set("and", ["and", "n"]),
  alternatives.set("you", ["yu", "you", "u"]),
  alternatives.set("are", ["are", "r"]),
  alternatives.set("got", ["got", "gto", "tog"]);

let alternativesobj = Object.fromEntries(alternatives);
console.log(alternativesobj)

And then overwrite the variable via coding not executing.

P2: Lines

bhop-bot-ts/src/commands/fun/insardia.ts

Lines 28 to 30 in 50e59f9

	let lines = args.insardia.split("\n");
	let formatted_string = "";
	for (let line of lines) {

wtf.

@drumman22 If you remember, could you explain why you chose to split the entire string by the newlines or at least remember if this was a relevant string of code to implement? AFAIK this wouldn't change anything in output.

P3: Recursive ifs

bhop-bot-ts/src/commands/fun/insardia.ts

Lines 48 to 60 in 50e59f9

	if (word.length - 1 > 3) {
	if (index > 0 && index < word.length - 1) {
	let alpha = /[a-zA-Z]/;
	if (word[index + 1]?.match(alpha)) {
	if (word[index + 1] == letter) {
	continue;
	}
	if (vowels.includes(letter)) {
	continue;
	}
	}
	}
	}

All these checks lead up to a continue, I'm not sure what would change if it is removed, I would need to do some more research into what exactly continue does.

MDN reference for continue.

P4: Merging duplicate conditions

bhop-bot-ts/src/commands/fun/insardia.ts

Lines 62 to 74 in 50e59f9

	if (index === 0) {
	rng = getRng(50);
	if (rng == 1) {
	new_word += letter + letter;
	}
	}
	if (index === word.length - 1) {
	rng = getRng(50);
	if (rng == 1) {
	new_word += letter + letter;
	}
	}

Literally just:

if (index === 0 || index === word.length - 1) {
  if (getRng(50) === 1) new_word += letter + letter; // Some way to shorten letter + letter?
}

P5: Merge P3 and P4

In short, put P4 inside of P3's condition checking, fundamentally replacing where continue is with P4.

Cleanup

As of now, these are all the specific/critical issues to be resolved. Any more input or follow up is welcome to be commented.

P1: `alternatives` map

TODO

• #15
• #16
• #17
• #19
• #23

P3: Recursive ifs

Make insardia not break on, @ mention, #channel, url link, etc.

fban and votemute

Dynamic help cmd

poll

P2: Lines

P4: Merging duplicate conditions

Add disable boolean for each command.

starboard