Skip to content

Docs rot: README, TODO.md, stale config comments #8

Description

@miridius

From the /review-pr baseline (comment-analyzer):

  • README: still says 50MB limit (it's 2GB via local bot-api), clone URL is the old video-bot repo name, env-file steps are confused (.env.telegram vs .env.prod), "/verbose" works in DMs only, "no local deps needed" is false for contributors (bun + gitleaks required by pre-commit), no mention of check.sh.
  • TODO.md: "add tests" (146 exist) and "delete files after upload" (implemented) are done; prune/rewrite.
  • yt-dlp.conf: double-commented "TODO fix this ?" block superseded by the app's own info cache.
  • docker-compose.yml: prod mounts ./.prod-video-cache:/app/.video-cache which nothing references anymore.
  • download-video.ts:259 comment says "cached based on url" but the key is filename; line 12 dead-code comment; the @ts-ignore "bug in the telegram bot API" comment should name the actual symptom.
  • test/e2e.test.ts plan comment lists inline-query testing that is describe.todo; simulate-bot-api.ts header's "only part we intercept is getUpdates" is wrong (it intercepts everything).

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