[mbti0n]

Why

1. Since Knight Hacks IX was finished, the corresponding link for the current Knight Hacks hackathon should be temporarily removed.
2. The flowcharts for the 2026/27 academic year have been already posted; however, the newest flowcharts using the /flowchart command only show the ones in the 2025/26 academic year.

What

1. Regarding the /links command, temporarily removed the link for the current Knight Hacks hackathon. However, the link is commented under the list of the LINKS constant.
2. Regarding the /flowchart command, added images and output for the current flowcharts (2026/27) for Computer Science, Information Technology, and Computer Engineering.

Test Plan

Ran the Discord bot on a burner server and tested the new output using the /links and /flowchart commands. Ran all required pnpm commands and passed all tests.

Checklist

• Database: No schema changes, OR I ran pnpm db:generate and committed the generated files in packages/db/drizzle/
• Environment Variables: No environment variables changed, OR I have contacted the Development Lead to modify them on Coolify BEFORE merging.

[coderabbitai]

📝 Walkthrough

The flowchart command adds a 2026–2027 option and URL mappings for Computer Science, Information Technology, and Computer Engineering. Data Science remains available only for 2024–2025. The KnightHacks IX signup link is removed from the active links array and retained as a comment.

Priority: ⬇️ Low

Merge Risk: 🔵 Low · up to e59f4

The new 2026-27 flowchart choices will show broken images until the Blade app serving the added image files is deployed. Merge and deploy Blade first, or confirm the order, so the new options do not go live without images.

Pre-merge checks: 6 · 2

Failed checks: 2 warnings

Check name	Status	Explanation	Resolution
Title check		The title describes the link and flowchart changes but does not start with an issue number in brackets.	Prefix the title with an issue number, such as "[#619] Update links and add 2026/27 flowcharts".
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 6 passed

Check name	Status	Explanation
Description check		The description explains the /links and /flowchart changes, testing, and checklist items.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
No Hardcoded Secrets		No hardcoded secrets were introduced. The changed TypeScript contains only public flowchart URLs, command labels, and link text. Secret-pattern searches found no API keys, passwords, tokens, or secret…
Validated Env Access		No added line in the pull-request diff uses process.env. The two changed TypeScript files contain no direct process.env access.
No Typescript Escape Hatches		The pull request adds no any types, @ts-ignore, @ts-expect-error, or !. non-null assertions. The added TypeScript lines only define flowchart constants, choices, mappings, and comments.

Full details: No Hardcoded Secrets

Explanation

No hardcoded secrets were introduced. The changed TypeScript contains only public flowchart URLs, command labels, and link text. Secret-pattern searches found no API keys, passwords, tokens, or secret literals in the changed source, and the three added PNG files contain no matching printable metadata.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

apps/tk/src/commands/flowchart.ts

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

apps/tk/src/consts/index.ts

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Note

Quiet mode is enabled, so only the most important comments were posted inline. Other review comments are grouped below.

🟡 Other comments: 1

apps/tk/src/consts/index.ts: 1

23-24: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Deploy the Blade assets.

The three files are present in the repository, but their deployed URLs return 404. The new /flowchart embeds will not load until the Blade app serves these files. The constants do not need URL changes.

---

ℹ️ Review info

⚙️ Run configuration

• Configuration used: Repository: KnightHacks/forge/.coderabbit.yml
• Review profile: QUIET
• Plan: Advanced
• Run ID: 6996ed74-7abe-473b-b6b8-a879c9fe673a

📥 Commits

Reviewing files that changed from the base of the PR and between 8097334 and e59f4bc.

⛔ Files ignored due to path filters: 3

• apps/blade/public/flowcharts/bs-cpe-26-27.png is excluded by !**/*.png
• apps/blade/public/flowcharts/bs-cs-26-27.png is excluded by !**/*.png
• apps/blade/public/flowcharts/bs-it-26-27.png is excluded by !**/*.png

📒 Files selected for processing: 2

• apps/tk/src/commands/flowchart.ts
• apps/tk/src/consts/index.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.