Skip to content

[code-audit] Dwa moduły token bucket: ratelimit (prawda) i rate_limit (shim); README i __init__ się kłócą - #108

Merged
mikolaj92 merged 1 commit into
mainfrom
lokay/94
Oct 17, 2026
Merged

mikolaj92 merged 1 commit into
mainfrom
lokay/94

Conversation

@mikolaj92

@mikolaj92 mikolaj92 commented Oct 17, 2026 •

Copy link
Copy Markdown
Owner

Co zrobiono

Zmieniono dwa pliki:

  • src/datasource_kit/__init__.py importuje klasę bezpośrednio z ratelimit.
  • Usunięto src/datasource_kit/rate_limit.py.

Lokalnie przeszły: 319 testów, uv lock --check, kontrola tożsamości eksportu i git diff --check. Nie dodano komentarzy ani testów.

Commit zablokowany: Git nie może utworzyć index.lock w metadanych worktree poza bieżącym katalogiem. Zmiany pozostają niezacommitowane. Testy wykonano na Pythonie 3.14.3; CI dla 3.12/3.13 nie zostało potwierdzone.

Zmienione pliki

  • src/datasource_kit/__init__.py
  • src/datasource_kit/rate_limit.py

Zmiany

src/datasource_kit/__init__.py   | 2 +-
 src/datasource_kit/rate_limit.py | 7 -------
 2 files changed, 1 insertion(+), 8 deletions(-)
  • lokay build

Plan

Cel

Ujednolicić import TokenBucket wokół datasource_kit.ratelimit, usuwając zbędny moduł pośredni. Zachować eksport datasource_kit.TokenBucket i działające aliasy konstruktora.

Kryteria

  • __init__.py importuje TokenBucket bezpośrednio z ratelimit; moduł rate_limit.py nie istnieje. [źródła: minimal]
  • README, runtime i istniejące testy pozostają zgodne z kanonicznym modułem ratelimit. [źródła: minimal]
  • rate_per_sec, burst i wait() nadal działają; ich zgodność sprawdzają istniejące testy tests/test_rate_limit.py. [źródła: minimal]
  • Wszystkie wymagane kontrole CI na PR przechodzą. Nie powstają komentarze, docstringi ani nowe testy jednostkowe. [źródła: minimal]

Kroki

  • Zmienić import w __init__.py i usunąć rate_limit.py. Usunięcie świadomie kończy obsługę bezpośredniego importu datasource_kit.rate_limit. [źródła: minimal]
  • Pozostawić README, runtime, implementację klasy i istniejące testy bez zmian; już używają właściwego modułu lub publicznego eksportu. [źródła: minimal]
  • Uruchomić istniejące testy oraz sprawdzić tożsamość publicznego eksportu z klasą w ratelimit. [źródła: minimal]
  • Doprowadzić wymagane kontrole PR do powodzenia, uwzględniając konieczne poprawki poza modułem, także zgodność przypiętych wersji. Zachować te poprawki. [źródła: minimal]

Uwagi bez konkretu

  • alignment: lens alignment widzi problem w src/datasource_kit/init.py, src/datasource_kit/rate_limit.py, wskaż konkret

Closes #94

Opened by lokay.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-17T05:23:58.157369Z 62ed027 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@mikolaj92

Copy link
Copy Markdown
Owner Author

Lokay Check 1

SHA: 62ed02781ce48eb501b084426e6fb7a134897a04

Wynik: pending · CI: pending · tests: pending · tested=false

Uwagi i CI:

{
  "uwagi": [],
  "notes": [],
  "failed": [],
  "pending": [
    "test (3.12)",
    "test (3.13)"
  ]
}

@mikolaj92

Copy link
Copy Markdown
Owner Author

Lokay Check 1

SHA: 62ed02781ce48eb501b084426e6fb7a134897a04

Wynik: merge · CI: green · tests: green · tested=true

name status reason
ci_green pass GitHub CI green at 62ed027; expected green or no_ci at 62ed027
local_tests pass local tests green
merge_clean pass stub
plan_files pass stub
lockfile pass stub
secrets pass stub
permissions pass stub
no_new_comments pass no new comments or lint errors
no_unit_tests pass no new unit tests
issue_goal pass stub
scope pass stub
correctness pass stub
tests_cover pass stub
regression pass stub
architecture pass stub
security_review pass stub
docs pass no documentation expansion
style pass stub

Uwagi i CI:

{
  "uwagi": [],
  "notes": [
    {
      "pytanie": "alignment",
      "pliki": [
        "src/datasource_kit/__init__.py",
        "src/datasource_kit/rate_limit.py"
      ],
      "prompt": "lens alignment widzi problem w src/datasource_kit/__init__.py, src/datasource_kit/rate_limit.py, wskaż konkret",
      "note": "lens alignment widzi problem w src/datasource_kit/__init__.py, src/datasource_kit/rate_limit.py, wskaż konkret"
    }
  ],
  "failed": [],
  "pending": []
}

Uwagi bez konkretu

  • alignment: lens alignment widzi problem w src/datasource_kit/init.py, src/datasource_kit/rate_limit.py, wskaż konkret

@mikolaj92
mikolaj92 merged commit 0e5439d into main Oct 17, 2026
2 checks passed
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.

[code-audit] Dwa moduły token bucket: ratelimit (prawda) i rate_limit (shim); README i __init__ się kłócą

1 participant