Skip to content

fix: update reviewer DM test info for AI question and 90 min limit - #579

Merged
kylehoehns merged 1 commit into
mainfrom
fix/reviewer-dm-test-info-ai-question
Sep 9, 2026
Merged

fix: update reviewer DM test info for AI question and 90 min limit#579
kylehoehns merged 1 commit into
mainfrom
fix/reviewer-dm-test-info-ai-question

Conversation

@kylehoehns

Copy link
Copy Markdown
Contributor

What this does

When a reviewer clicks "Accept" on a HackerRank review request, the bot sends them a DM with a one-line summary of the test the candidate took. That line was out of date. This updates it.

Before:

Test Info: 4 questions (2 easy, 2 medium) · 70 min · candidates answer 1 per section (2 total)

After:

Test Info: 5 questions (2 easy, 2 medium, 1 AI-assisted) · 90 min · candidates answer 1 per section (2 total) plus the AI question

Why

We added a fifth question that lets the candidate use an AI coding assistant, and the total time went from 70 to 90 minutes. Candidates still pick one question from each of the two normal sections, then do the AI question at the end. The old text would have told reviewers the wrong thing about what they are grading.

Files changed

  • src/bot/acceptReviewRequest.ts — the message itself
  • src/bot/__tests__/acceptReviewRequest.test.ts — the test checks the exact string, so it changes too

Testing

pnpm verify passes: lint, format check, type-check, build, and 265 tests across 34 suites.

🤖 Generated with Claude Code

https://claude.ai/code/session_019dUo24dvyCizU9GbU9dTww

The assessment now includes a fifth question that allows an AI coding
assistant, and the total time went from 70 to 90 minutes. Candidates still
answer one question per section (2 total) and then do the AI question.

The old text told reviewers "4 questions ... 70 min", which no longer matches
what candidates actually took.
@kylehoehns
kylehoehns force-pushed the fix/reviewer-dm-test-info-ai-question branch from 222e99c to 098085e Compare September 9, 2026 13:32
@kylehoehns
kylehoehns marked this pull request as ready for review September 9, 2026 13:33
@kylehoehns
kylehoehns merged commit f9dc90a into main Sep 9, 2026
1 check passed
@kylehoehns
kylehoehns deleted the fix/reviewer-dm-test-info-ai-question branch September 9, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant