Adding Leetcode and /eightball responses - #487
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe pull request adds seventeen responses to ChangesMessage Pool Updates
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 7 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (7 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/tk/src/consts/index.ts`:
- Around line 19-23: The LeetCode reminder is still using a separate
DAILY_MESSAGES list instead of the shared constants, so the new strings in the
shared export are never picked up. Update the LeetCode cron logic in leetcode.ts
to import and use the shared daily-message export from consts/index.ts, and
remove the duplicate DAILY_MESSAGES definition so there is a single source of
truth.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro
Run ID: efb1a21b-782b-4920-b378-3de2e1152f32
📒 Files selected for processing (1)
apps/tk/src/consts/index.ts
Preserve the live portion of PR KnightHacks#487 after main removed the unused LeetCode reminder path.
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/tk/src/consts/index.ts`:
- Line 119: In the response text constant containing "That seems potential.",
replace the wording with "That seems possible." while leaving the surrounding
constants and behavior unchanged.
- Line 126: Replace the abusive “Lame question, sybau 💔” entry in the random
response constants with neutral, user-friendly wording while preserving the
existing response collection and eight-ball behavior.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro Plus
Run ID: 6e26bc44-0119-4667-90de-3ac357a39db5
📒 Files selected for processing (1)
apps/tk/src/consts/index.ts
Rebase PR KnightHacks#487 onto current main and place the LeetCode prompts in the live cron-owned message list.
Rebase PR KnightHacks#487 onto current main and place the LeetCode prompts in the live cron-owned message list.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/cron/src/crons/leetcode.ts`:
- Around line 37-43: Keep this PR scoped to the cron application by removing the
unrelated changes under the TK application, including updates to its constants.
Only retain TK modifications if a demonstrated dependency is required by the
cron implementation; otherwise move them to a separate PR.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro Plus
Run ID: 437acfa7-723d-43ed-a56a-2e10db97e9c2
📒 Files selected for processing (1)
apps/cron/src/crons/leetcode.ts
Rebase PR KnightHacks#487 onto current main and place the LeetCode prompts in the live cron-owned message list.
Why
Adding new prompts to TK's daily Leetcode reminders and /eightball responses.
What
Added new prompts inside the 2 arrays
DAILY_MESSAGES(5 new prompts) andEIGHTBALL_RESPONSES(11 new responses) offorge/apps/tk/src/consts/index.ts.Test Plan
Since the prompts of daily Leetcode reminders and /eightball responses are randomized (and I failed to set up the appropriate environment to start TK on my end), I want to have the bot tested in the Knight Hacks Discord server.
Checklist
Summary by CodeRabbit