last call: anyone who never shipped can ship through nov 5, and clippy DMs everyone with an hour or more, once

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: cskartikey

app/models/program.rb

@@ -7,6 +7,9 @@ module Program
   EXTENSION_DAYS = 7
   # Ship stays open this many days past the deadline, quietly: nobody's told, and the site keeps saying the deadline.
   SHIP_GRACE_DAYS = 2
+  # Last call, after it all closed: anyone who never shipped can still ship through this day (Project#ship_cutoff), and
+  # ships:last_call DMs them to.
+  LAST_CALL_THROUGH = Date.new(2026, 11, 5)
   # The first hours pay more (Shop::FIRST_RATE an hour; Shop::RATE after), and the stats count who's past them.
   HOURS_PER_REWARD = 10
   # Hackatime time from this day on counts: only projects worked on since then can be linked.

app/models/project.rb

@@ -355,9 +355,12 @@ def program_dates
     Program::DATES.begin..deadline
   end
 
-  # Ship closes at the end of the deadline where you are, plus Program::SHIP_GRACE_DAYS nobody's told about.
+  # Ship closes at the end of the deadline where you are, plus Program::SHIP_GRACE_DAYS nobody's told about. Never
+  # shipped (sent back counts), it stays open for last call (Program::LAST_CALL_THROUGH).
   def ship_cutoff
-    (deadline + Program::SHIP_GRACE_DAYS).in_time_zone(user.time_zone).end_of_day
+    last_day = deadline + Program::SHIP_GRACE_DAYS
+    last_day = [ last_day, Program::LAST_CALL_THROUGH ].max unless shipped?
+    last_day.in_time_zone(user.time_zone).end_of_day
   end
 
   def may_ship?(now: Time.current)

db/migrate/20261102120000_add_last_call_dmed_at_to_projects.rb

@@ -0,0 +1,6 @@
+class AddLastCallDmedAtToProjects < ActiveRecord::Migration[8.1]
+  def change
+    # When Clippy DMed them that shipping's open again (ships:last_call), so nobody hears it twice.
+    add_column :projects, :last_call_dmed_at, :datetime
+  end
+end

db/schema.rb

@@ -28,4 +28,38 @@ namespace :ships do
     puts
     puts "#{sent} of #{owed.size} #{dry_run ? "would be " : ""}DMed."
   end
+
+  # Last call, after shipping closed: Clippy DMs everyone who never shipped (nothing approved or in review) but logged
+  # an hour or more that shipping's open for them again, through Program::LAST_CALL_THROUGH. Noted on the project
+  # (last_call_dmed_at), so running it twice sends nothing new. DRY_RUN=1 lists who'd hear without sending.
+  #
+  #   bin/rails ships:last_call DRY_RUN=1
+  #   bin/rails ships:last_call
+  desc "DM everyone who never shipped that shipping's open again for last call"
+  task last_call: :environment do
+    dm = "Shipping's open again for anyone who didn't make it. *%{title}* can ship until the end of %{until}, " \
+         "finished or not. If it plays, ship it.\n:clippy-hopeful: clippy propped the door back open for you. :blob-wiggle:"
+    dry_run = ENV["DRY_RUN"].present?
+    owed = Project.where(last_call_dmed_at: nil)
+                  .where.not(id: Ship.where(status: %w[approved in_review]).select(:project_id))
+                  .includes(:user).order(:id)
+                  .select { |project| project.hours_logged >= 1 }
+
+    sent = owed.count do |project|
+      user = project.user
+      reachable = user.slack_id.present? && user.slack_muted_at.nil?
+      puts format("%-20s %-30s %5.1f hrs  %s", user.public_name[0, 20], project.title[0, 30], project.hours_logged,
+                  reachable ? (dry_run ? "would DM" : "DMed") : "skipped (not on Slack, or stopped Clippy)")
+      if reachable && !dry_run
+        user.slack_dmed!
+        project.update_column(:last_call_dmed_at, Time.current)
+        text = format(dm, title: project.title, until: Program::LAST_CALL_THROUGH.strftime("%b %-d"))
+        SlackMessageJob.perform_later(user.slack_id, text, link: [ "Ship it", "#{Rails.configuration.x.app_url}/ship" ])
+      end
+      reachable
+    end
+
+    puts
+    puts "#{sent} of #{owed.size} #{dry_run ? "would be " : ""}DMed."
+  end
 end

lib/tasks/ships.rake

@@ -66,7 +66,7 @@ class ShipsControllerTest < ActionDispatch::IntegrationTest
     assert_select ".ship__error", /has to show the game running/
   end
 
-  test "shipping closes after the deadline where you are, with two quiet days nobody's told about" do
+  test "never shipped, shipping stays open past the quiet days through last call, then closes" do
     link_hackatime(users(:orpheus))
     projects(:orpheus).update!(hackatime_projects: [ "rhythm-game" ], repo_later: true, buddy_skipped: true)
     users(:orpheus).update!(timezone: "America/New_York")
@@ -79,7 +79,12 @@ class ShipsControllerTest < ActionDispatch::IntegrationTest
       assert_select ".project__ship", /Ship/
     end
 
-    travel_to Time.utc(2026, 10, 30, 5) do
+    travel_to Time.utc(2026, 11, 6, 4) do # 11pm on Nov 5 in New York: last call, still open
+      get project_ship_path
+      assert_select "form.ship__form"
+    end
+
+    travel_to Time.utc(2026, 11, 6, 6) do
       get project_ship_path
       assert_response :success
       assert_select ".ship__done[data-state=closed] .ship__done-title", "Shipping's closed."

test/controllers/ships_controller_test.rb

@@ -187,15 +187,24 @@ class ProjectDeadlineTest < ActiveSupport::TestCase
     assert_equal Date.new(2026, 10, 6)..Date.new(2026, 10, 27), project.program_dates
   end
 
-  test "ship closes at the end of the deadline where you are, plus two quiet days" do
+  test "ship closes at the end of the deadline where you are, plus two quiet days, once you've shipped" do
     project = projects(:orpheus)
+    def project.shipped? = true
     assert_equal Time.utc(2026, 10, 29, 23, 59, 59), project.ship_cutoff.change(usec: 0), "UTC until they've said where they are"
 
     users(:orpheus).update!(timezone: "America/New_York")
-    assert project.reload.may_ship?(now: Time.utc(2026, 10, 30, 2))
+    project.user.reload
+    assert project.may_ship?(now: Time.utc(2026, 10, 30, 2))
     assert_not project.may_ship?(now: Time.utc(2026, 10, 30, 4))
   end
 
+  test "never shipped, ship stays open through last call" do
+    project = projects(:orpheus)
+    assert_equal Time.utc(2026, 11, 5, 23, 59, 59), project.ship_cutoff.change(usec: 0)
+    assert project.may_ship?(now: Time.utc(2026, 11, 2, 12))
+    assert_not project.may_ship?(now: Time.utc(2026, 11, 6, 1))
+  end
+
   test "nobody can ask for more time any more" do
     project = projects(:orpheus)
     link_hackatime(users(:orpheus))

test/models/project_test.rb

@@ -102,6 +102,43 @@ class ShipTest < ActiveSupport::TestCase
     assert_equal [ "Ship it", "https://wrong.hackclub.com/ship" ], dms.first[:args].third["link"]
   end
 
+  test "last call DMs everyone who never shipped with an hour or more, once, and a dry run sends nothing" do
+    require "rake"
+    Rails.application.load_tasks unless Rake::Task.task_defined?("ships:last_call")
+    ship_now(hours: 2) # in review: not owed
+    ana = projects(:ana)
+    link_hackatime(ana.user, token: "token-heidi")
+    ana.update!(hackatime_projects: [ "heidis-game" ])
+    ana.ships.create!(title: "Pong in Figma", description: "Two paddles, one ball, and every frame is a Figma component.",
+                      repo_url: "https://example.com/repo", demo_url: "https://example.com/demo", hours: 1,
+                      screenshot: fixture_upload, screenshot_shows_game: true).update!(status: "rejected")
+    ana.user.streak_activities.create!(activity_date: Program::DATES.begin, coded_seconds: 2 * 3600)
+
+    with_env("DRY_RUN" => "1") { capture_io { Rake::Task["ships:last_call"].execute } }
+    assert_empty enqueued_jobs.select { |job| job[:job] == SlackMessageJob }
+
+    capture_io { Rake::Task["ships:last_call"].execute }
+    dms = enqueued_jobs.select { |job| job[:job] == SlackMessageJob }
+    assert_equal [ "U0ANA" ], dms.map { |job| job[:args].first }
+    assert_equal "Shipping's open again for anyone who didn't make it. *Pong in Figma* can ship until the end of Nov 5, " \
+                 "finished or not. If it plays, ship it.\n:clippy-hopeful: clippy propped the door back open for you. :blob-wiggle:",
+                 dms.first[:args].second
+    assert_not_nil ana.reload.last_call_dmed_at
+
+    clear_enqueued_jobs
+    capture_io { Rake::Task["ships:last_call"].execute }
+    assert_empty enqueued_jobs.select { |job| job[:job] == SlackMessageJob }, "once"
+  end
+
+  test "last call skips anyone under an hour" do
+    require "rake"
+    Rails.application.load_tasks unless Rake::Task.task_defined?("ships:last_call")
+    @project.user.streak_activities.create!(activity_date: Program::DATES.begin, coded_seconds: 30 * 60)
+
+    capture_io { Rake::Task["ships:last_call"].execute }
+    assert_empty enqueued_jobs.select { |job| job[:job] == SlackMessageJob }
+  end
+
   private
     def with_env(vars)
       originals = vars.keys.index_with { |key| ENV[key] }