Skip to content

LOW findings (10) #23

Description

@slowernet

Grouped because each is a one-line fix. Full reasoning in docs/notes/2026-08-20-plan-3a-review-triage.md under "LOW". Split out any one that turns out to be bigger.

  • The "waiting" notice is suppressed for the life of the process — supervisor.rb:249
  • The cursor stores created_at but since filters updated_atpoller.rb:154
  • An item with no repository or number is dropped with no trace — poller.rb:196
  • no_route is a fourth event state the schema does not document — poller.rb:126
  • want writes a decision an unconfigured board can never act on — board.rb:37
  • MILL_BIND= binds nowhere — config/puma.rb:4
  • The rate-limit block message is off by one and names the wrong stage — runner.rb:163
  • require './app' creates ~/.mill and opens the database — app.rb:20
  • A trailing slash on origin makes a clone invisible — repo.rb:31 (tracked with the Repo.slug issue)
  • rate_limit_pause calls .to_i on an unvalidated field — runner.rb:183

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions