Add `--no-sync` flag to avoid syncing the virtual environment (declare only) by Copilot · Pull Request #1766 · usethis-python/usethis-python

Copilot · 2026-04-02T02:43:10Z

Adds a --no-sync CLI flag that declares/undeclares dependencies in pyproject.toml without touching the virtual environment. Analogous to uv add --no-sync.

When active, an instruction message tells the user to manually install/uninstall:

$ usethis tool ruff --no-sync
✔ Adding dependency 'ruff' to the 'lint' group in 'pyproject.toml'.
☐ Install the dependency 'ruff'.

Config & CLI

NO_SYNC_DEFAULT, no_sync field on UsethisConfig, wired through set() context manager
no_sync_opt typer option added to all 11 interface files that already support --frozen
--frozen and --no-sync can be used simultaneously; --frozen naturally takes precedence since it is strictly stronger (prevents both lockfile updates and syncing)

Backend subprocess wrappers

uv: passes --no-sync to uv add/uv remove; when both --frozen and --no-sync are set, --frozen takes precedence via the elif chain
poetry: passes --lock to poetry add/poetry remove

Dependency messaging (`_deps.py`)

add_deps_to_group: prints install instruction when no_sync (same pattern as frozen)
remove_deps_from_group: prints uninstall instruction when no_sync

Tests

call_uv_subprocess: --no-sync added for add/remove, --frozen precedence over --no-sync verified
call_poetry_subprocess: --lock added for add/remove, not added for other commands
add_deps_to_group / remove_deps_from_group: output messages verified with no_sync=True
UsethisConfig: TestFrozenAndNoSync validates both flags can be set simultaneously, and success when only one is set

…e only) Add a --no-sync CLI flag that declares/undeclares dependencies in pyproject.toml without syncing the virtual environment. For uv, this passes --no-sync to uv add/remove. For poetry, this passes --lock to poetry add/remove. When active, an instruction message is shown telling the user to manually install/uninstall. Resolves #1807 Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/7348ebb6-400f-4e8e-b5aa-c25ea2f852c2 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

Tests for call_uv_subprocess, call_poetry_subprocess, add_deps_to_group, and remove_deps_from_group with no_sync=True. Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/7348ebb6-400f-4e8e-b5aa-c25ea2f852c2 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

Make frozen precedence over no_sync explicit in the elif condition. Add test for remove command with frozen precedence. Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/7348ebb6-400f-4e8e-b5aa-c25ea2f852c2 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

codecov · 2026-04-02T21:49:05Z

Codecov Report

❌ Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
src/usethis/_deps.py	80.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

codspeed-hq · 2026-04-02T21:50:09Z

Merging this PR will degrade performance by 5.37%

⚠️

Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 1 regressed benchmark
✅ 1 untouched benchmark

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Mode	Benchmark	`BASE`	`HEAD`	Efficiency
❌	Simulation	`test_help_flag`	53 ms	56.1 ms	-5.37%

_{Comparing copilot/add-no-sync-flag-to-poetry (0ac0fbc) with main (f84d89d)}

nathanjmcdougall · 2026-04-02T21:49:33Z

    }:
        new_args = ["uv", args[0], "--frozen", *args[1:]]
+    elif (
+        not usethis_config.frozen


@copilot we should add handle the case of BOTH --frozen and --no-sync...

Addressed in 7a4549a. UsethisConfig.set() now raises UsethisError("Cannot use both --frozen and --no-sync at the same time.") when both flags are True. Tests added in TestFrozenAndNoSync (test_config.py) and test_frozen_and_no_sync_raises (test_call.py).

@copilot sorry, I don't understand why this would raise? Why not just accept both args simultaneously?

You're right — both flags can coexist since --frozen is strictly stronger. Removed the mutual exclusivity check in 816d50c. Now both are accepted simultaneously and --frozen naturally takes precedence via the elif chain in call_uv_subprocess.

Instead of silently having --frozen take precedence, raise a UsethisError when both flags are set simultaneously. These flags are mutually exclusive since --frozen prevents both lockfile updates and syncing, while --no-sync only prevents syncing. Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/bbee573e-012b-4649-8923-7e4e46490fcf Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

…precedence) Remove the mutual exclusivity check that raised UsethisError when both flags were set. Instead, both flags are accepted and --frozen naturally takes precedence via the elif chain in call_uv_subprocess. Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/dd1c1911-7c23-496d-a175-2fac1a83d0e4 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

nathanjmcdougall · 2026-04-04T17:27:22Z

Not planned

Initial plan

8c383e3

Copilot AI assigned Copilot and nathanjmcdougall Apr 2, 2026

Copilot AI linked an issue Apr 2, 2026 that may be closed by this pull request

Add a --no-sync flag to avoid installing/uninstalling the dependency (declare only) #1765

Closed

Copilot started work on behalf of nathanjmcdougall April 2, 2026 02:43 View session

Copilot AI and others added 3 commits April 2, 2026 02:54

Copilot AI changed the title ~~[WIP] Add --no-sync flag to poetry for dependency declaration~~ Add --no-sync flag to avoid syncing the virtual environment (declare only) Apr 2, 2026

Copilot finished work on behalf of nathanjmcdougall April 2, 2026 03:05

Copilot AI requested a review from nathanjmcdougall April 2, 2026 03:05

nathanjmcdougall marked this pull request as ready for review April 2, 2026 21:41

Copilot AI had a problem deploying to codecov April 2, 2026 21:41 Error

Merge branch 'main' into copilot/add-no-sync-flag-to-poetry

a0009ee

nathanjmcdougall had a problem deploying to codecov April 2, 2026 21:42 — with GitHub Actions Error

Update call.py

0ac0fbc

nathanjmcdougall temporarily deployed to codecov April 2, 2026 21:45 — with GitHub Actions Inactive

nathanjmcdougall mentioned this pull request Apr 2, 2026

Create an agent skill for doing a grammar review of the codebase #1769

Closed

Copilot AI mentioned this pull request Apr 2, 2026

Create agent skills for grammar review and LLM-based codebase review #1770

Merged

nathanjmcdougall mentioned this pull request Apr 2, 2026

Test the poetry backend under --frozen #1772

Closed

Copilot AI mentioned this pull request Apr 2, 2026

Handle --frozen flag for Poetry backend via lockfile backup/restore #1773

Merged

nathanjmcdougall requested changes Apr 2, 2026

View reviewed changes

Copilot started work on behalf of nathanjmcdougall April 2, 2026 21:53 View session

Copilot AI requested a review from nathanjmcdougall April 2, 2026 22:00

Copilot finished work on behalf of nathanjmcdougall April 2, 2026 22:00

Copilot started work on behalf of nathanjmcdougall April 3, 2026 19:31 View session

Copilot finished work on behalf of nathanjmcdougall April 3, 2026 19:35

nathanjmcdougall closed this Apr 4, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add `--no-sync` flag to avoid syncing the virtual environment (declare only)#1766

Add `--no-sync` flag to avoid syncing the virtual environment (declare only)#1766
Copilot wants to merge 8 commits into
mainfrom
copilot/add-no-sync-flag-to-poetry

Copilot AI commented Apr 2, 2026 •

edited

Loading

Uh oh!

codecov Bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

codspeed-hq Bot commented Apr 2, 2026

Uh oh!

nathanjmcdougall Apr 2, 2026

Uh oh!

Copilot AI Apr 2, 2026

Uh oh!

nathanjmcdougall Apr 3, 2026

Uh oh!

Copilot AI Apr 3, 2026

Uh oh!

nathanjmcdougall commented Apr 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Config & CLI

Backend subprocess wrappers

Dependency messaging (_deps.py)

Tests

Uh oh!

codecov Bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

codspeed-hq Bot commented Apr 2, 2026

Merging this PR will degrade performance by 5.37%

Performance Changes

Uh oh!

nathanjmcdougall Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 2, 2026

Choose a reason for hiding this comment

Uh oh!

nathanjmcdougall Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

nathanjmcdougall commented Apr 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Apr 2, 2026 •

edited

Loading

Dependency messaging (`_deps.py`)

codecov Bot commented Apr 2, 2026 •

edited

Loading