Skip to content

chore: isolate ruff dependencies in ci - #7142

Merged
chenghao-mou merged 2 commits into
mainfrom
chenghao/chore/AGT-3449-ruff-ci-dependencies
Sep 6, 2026
Merged

chenghao-mou merged 2 commits into
mainfrom
chenghao/chore/AGT-3449-ruff-ci-dependencies

Conversation

@chenghao-mou

@chenghao-mou chenghao-mou commented Sep 5, 2026 •

Copy link
Copy Markdown
Member

Install only locked Ruff for lint and format checks, retain Ruff in dev installs, and give it a separate cache. Python 3.12 satisfies the workspace requirement.

CI medians from three cache-hit runs per version on ubuntu-latest:

Metric Before After
Ruff job 24 s 10 s (58% less)
Dependency setup 9.726 s 0.047 s
Installed packages 308 1
Restored cache 539.6 MiB 10.2 MiB

Ruff and both type-check jobs pass.

Addresses AGT-3449

CI measurement evidence

Before: d8607e6711bf9770bfd7e5476f79ab63e6a451a6. After: 1b24fcf4f7a295297d75674b65b39c6cbf1801dd.

Version CI attempt Job time Dependency setup
Before 2 24 s 9.726 s
Before 3 24 s 10.010 s
Before 4 26 s 8.612 s
After 2 10 s 0.048 s
After 3 8 s 0.047 s
After 4 11 s 0.043 s

Job times use GitHub's job start/completion timestamps and exclude queue time. Dependency setup uses log timestamps from the uv sync step start to the next step start. Cache sizes are compressed archives reported by setup-uv.

All six runs use uv 0.12.10 and Ruff 0.15.22, pass lint and format checks, and report 992 formatted files. Python is 3.12.3 before and 3.12.14 after. These measurements compare the full workflow configurations; runner timing varies.

The first run with the isolated cache had a cache miss and took 11 seconds, with 0.600 seconds for dependency setup. It is excluded from the medians.

Initial prompt and agent context

Model: GPT-6

Can you take a look at this repo and find optimization opportunities, and rank them by effort and impact? (low effort, high impact first)

let's create linear tickets for these optimization opportunities. And start working on number 3 now.

okay, let's measure this in CI and report the before and after in the PR description.

@chenghao-mou
chenghao-mou marked this pull request as ready for review September 5, 2026 23:34
@chenghao-mou
chenghao-mou requested a review from a team as a code owner September 5, 2026 23:34

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@chenghao-mou
chenghao-mou merged commit c6939a5 into main Sep 6, 2026
35 checks passed
@chenghao-mou
chenghao-mou deleted the chenghao/chore/AGT-3449-ruff-ci-dependencies branch September 6, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants