Conversation
Include the renamed PARE scenario-generation guide in the PR so the workflow docs match the new CLI and repo naming. Made-with: Cursor
There was a problem hiding this comment.
Why do we need this change?
Why is the language marked as system?
There was a problem hiding this comment.
it's because our project depends on a specific Python version and a uv-managed environment
pre-commit’s default isolated env was not reliably reproducing that and may cause error during the pre-commit check
There was a problem hiding this comment.
so in this case the language: system tells pre-commit not to build a separate env while
uv run mypy ensures the hook runs in the same environment the project actually uses
There was a problem hiding this comment.
The solution is actually simpler. You just need to run uv run git commit -m "<message>".
There was a problem hiding this comment.
nice it's working, a quick note that should we add this cmd to Readme.md?
There was a problem hiding this comment.
no worries, actually I just added it here in previous commit
|
|
||
| [tool.deptry] | ||
| known_first_party = ["pare"] | ||
| known_first_party = ["pare", "scripts"] |
There was a problem hiding this comment.
Why are scripts first party? Scripts is not a package and deptry should not run on them?
There was a problem hiding this comment.
it's due to in the orchestrator code we imported the run_scenarios file in scripts folder here, should we move run_scenarios under pare/ instead? so that we can remove "scripts" from known_first_party.
Description
Adds a new
pare scenariosCLI for listing, inspecting split files, validating scenario ID files, and running scenario generation workflows.Cleans up tooling by moving scenario-runner logic into
pare/and fixing dependency checks for local development.Type of Change
Related Issues
Closes #N/A
Related Design Documents
Component
pare, e.g.pare/environment.py)pare/agents/)pare/scenarios/)pare/scenario_generator/)pare/apps/)docs/,mkdocs.yml,README.md)Other: CLI / developer tooling
Changes Made
pare scenariosCLI module and wired it into the top-levelparecommand.--listbehavior, and split metadata output.pare/scenarios/run_scenarios.pyand kept source code underpare/independent fromscripts/.docs/scenarios/anddocs/index.mdto reflect the new CLI and currentparenaming.deptryand local checks pass with the new source layout.Breaking Changes
Details:
Testing
Test Coverage
make testoruv run pytest)Manual Testing
Test steps:
uv run python -m pare.main scenarios --helpto verify the new CLI registration and command surface.uv run pytest tests/cli/test_scenarios_cli.pyto verify the new CLI behavior.make checkanduv run deptry pareto verify linting, typing, and dependency checks.Test environment:
Code Quality
uv run deptry pare)Pre-commit output: