Skip to content

[codex] Fix leaderboard posting#322

Merged
redreceipt merged 1 commit into
mainfrom
codex/fix-leaderboard-posting
May 9, 2026
Merged

[codex] Fix leaderboard posting#322
redreceipt merged 1 commit into
mainfrom
codex/fix-leaderboard-posting

Conversation

@redreceipt
Copy link
Copy Markdown
Member

@redreceipt redreceipt commented May 8, 2026

Summary

Fixes the weekly leaderboard worker path so it can post successfully and run at the intended 4pm Eastern wall-clock time.

Root cause

The completed-project contributor query declared $project_id as String!, but Linear's project.id.eq filter expects an ID. That caused a GraphQL validation error before the worker reached the Slack post.

The leaderboard schedule also used friday.at("20:00") without a timezone, unlike the other wall-clock Slack jobs. On UTC workers that resolved to 20:00 UTC instead of the intended Eastern-time schedule.

Changes

  • Change the completed-project contributor query variable to ID!.
  • Schedule post_leaderboard at 16:00 in America/New_York.
  • Update tests to assert the query type and timezone registration.
  • Stub post_inactive_engineers in the debug-job test so it stays focused on debug job orchestration.

Validation

  • .venv-ci/bin/python -m unittest tests.test_jobs tests.test_linear_projects tests.test_leaderboard
  • .venv-ci/bin/ruff check jobs.py tests/test_jobs.py linear/projects.py tests/test_linear_projects.py
  • .venv-ci/bin/ruff format --check jobs.py tests/test_jobs.py linear/projects.py tests/test_linear_projects.py
  • Verified under TZ=UTC that 16:00 America/New_York resolves to 20:00 UTC / 16:00 Eastern.

@redreceipt redreceipt force-pushed the codex/fix-leaderboard-posting branch from 23b3ee9 to 2fce29d Compare May 8, 2026 22:18
@redreceipt redreceipt marked this pull request as ready for review May 8, 2026 22:59
@redreceipt redreceipt merged commit b925734 into main May 9, 2026
8 checks passed
@redreceipt redreceipt deleted the codex/fix-leaderboard-posting branch May 9, 2026 00:47
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