Skip to content

chore: update dependencies and CLI tools #961

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kzndotsh
Copy link
Contributor

@kzndotsh kzndotsh commented Jul 20, 2025

  • Update poetry.lock and pyproject.toml
  • Enhance CLI test functionality

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. If this change fixes any issues please put "Fixes #XX" in the description. Please also ensure to add the appropriate labels to the PR.

Guidelines

  • My code follows the style guidelines of this project (formatted with Ruff)

  • I have performed a self-review of my own code

  • I have commented my code, particularly in hard-to-understand areas

  • I have made corresponding changes to the documentation if needed

  • My changes generate no new warnings

  • I have tested this change

  • Any dependent changes have been merged and published in downstream modules

  • I have added all appropriate labels to this PR

  • I have followed all of these guidelines.

How Has This Been Tested? (if applicable)

Please describe how you tested your code. e.g describe what commands you ran, what arguments, and any config stuff (if applicable)

Screenshots (if applicable)

Please add screenshots to help explain your changes.

Additional Information

Please add any other information that is important to this PR.

Summary by Sourcery

Update project dependencies, add a new CLI command for task usage analysis, and tweak coverage report logging

New Features:

  • Add 'tux test tasks' CLI subcommand to analyze runtime task usage with various output formats, filtering, and recommendations

Enhancements:

  • Refine coverage command to log success after opening the HTML report
  • Enhance CLI test functionality to support saving analysis output to a file

Build:

  • Bump sentry-sdk to ^2.0.1 with prisma and loguru extras
  • Add vulture static analysis tool and configure it in pyproject.toml

- Update poetry.lock and pyproject.toml
- Enhance CLI test functionality
Copy link
Contributor

sourcery-ai bot commented Jul 20, 2025

Reviewer's Guide

This PR updates project dependencies (including bumping sentry-sdk, adding vulture and its configuration) and enhances the CLI test suite by introducing a new analyze_tasks command with multiple output formats and helpers, plus a minor logging adjustment in the coverage command.

Class diagram for new analyze_tasks CLI command and helpers

classDiagram
    class TestCLI {
        +analyze_tasks(format: str)
        +coverage()
        ...
    }
    class AnalyzeTasksHelper {
        +parse_tasks()
        +format_output(format: str)
    }
    TestCLI --> AnalyzeTasksHelper : uses
Loading

Class diagram for vulture configuration addition

classDiagram
    class VultureConfig {
        +paths: list
        +exclude: list
        +min_confidence: int
        +sort_by_size: bool
        +verbose: bool
        +ignore_decorators: list
        +ignore_names: list
    }
Loading

File-Level Changes

Change Details Files
Add analyze_tasks CLI command for runtime task usage analysis
  • Introduced analyze_tasks decorator and options
  • Implemented logic to fetch and filter mock analysis data
  • Added display helpers for table, summary, JSON formats
  • Added file save helper and error handling
tux/cli/test.py
Update dependency versions and add code analysis tool
  • Bumped sentry-sdk version constraint and reordered extras
  • Added vulture as a dev dependency
  • Configured vulture under [tool.vulture] in pyproject.toml
pyproject.toml
poetry.lock
Reorder logging message in coverage command
  • Moved coverage report opening log inside the success block
  • Adjusted logger.info placement around webbrowser.open
tux/cli/test.py

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link

Deploying tux with  Cloudflare Pages  Cloudflare Pages

Latest commit: 485d326
Status: ✅  Deploy successful!
Preview URL: https://b1c014f6.tux-afh.pages.dev
Branch Preview URL: https://05-dependencies-and-cli-spli.tux-afh.pages.dev

View logs

@kzndotsh
Copy link
Contributor Author

Depends on: #957

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