Skip to content

Resolves #97: Include in tests into the context window when generating #119

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.

Sign up for GitHub

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Resolves #97: Include in tests into the context window when generating #119

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
015d7f6
Merge pull request #1 from promptdriven/main
rakshanda-cmu Oct 27, 2025
4124e49
draftfix-issue97
rakshanda-cmu Oct 27, 2025
cb75c95
changes for making the feature 'include test cases' optional with def…
rakshanda-cmu Nov 3, 2025
b835484
Merge branch 'dev-issue97' of github.com:rakshanda-cmu/pdd into dev-i…
rakshanda-cmu Nov 3, 2025
f221368
changes for main branch
rakshanda-cmu Nov 3, 2025
ca39466
changes to flag
rakshanda-cmu Nov 3, 2025
43fedee
changes for searching unit test files
rakshanda-cmu Nov 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -483,6 +483,7 @@ These options can be used with any command:
- `--local`: Run commands locally instead of in the cloud.
- `--context CONTEXT_NAME`: Override automatic context detection and use the specified context from `.pddrc`.
- `--list-contexts`: List all available contexts defined in `.pddrc` and exit.
- `--exclude-tests`: Exclude test scenarios from code coverage generation. (WIP)
Copy link

Copilot AI Nov 3, 2025

Choose a reason for hiding this comment

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

The documentation description doesn't match the actual feature. This flag disables automatic inclusion of unit test files during code generation, not code coverage generation. Update to match the actual functionality.

Suggested change
- `--exclude-tests`: Exclude test scenarios from code coverage generation. (WIP)
- `--exclude-tests`: Disable automatic inclusion of unit test files during code generation. (WIP)

Copilot uses AI. Check for mistakes.

### Context Selection Flags

Expand Down
9 changes: 9 additions & 0 deletions pdd/cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,13 @@ def _run_setup_utility() -> None:
default=False,
help="Review and optionally exclude few-shot examples before command execution.",
)
@click.option(
"--exclude-tests",
"exclude_tests",
is_flag=True,
default=False,
help="Exclude test scenarios from code coverage generation. (WIP)",
Copy link

Copilot AI Nov 3, 2025

Choose a reason for hiding this comment

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

The help text is misleading. Based on the implementation and PR description, this flag excludes test files from being auto-included during code generation, not from 'code coverage generation'. Consider: 'Disable automatic inclusion of unit test files during code generation.'

Suggested change
help="Exclude test scenarios from code coverage generation. (WIP)",
help="Disable automatic inclusion of unit test files during code generation.",

Copilot uses AI. Check for mistakes.
)
@click.option(
"--local",
is_flag=True,
Expand Down Expand Up @@ -246,6 +253,7 @@ def cli(
quiet: bool,
output_cost: Optional[str],
review_examples: bool,
exclude_tests: bool,
local: bool,
time: Optional[float], # Type hint is Optional[float]
context_override: Optional[str],
Expand All @@ -265,6 +273,7 @@ def cli(
ctx.obj["quiet"] = quiet
ctx.obj["output_cost"] = output_cost
ctx.obj["review_examples"] = review_examples
ctx.obj["exclude_tests"] = exclude_tests
ctx.obj["local"] = local
# Use DEFAULT_TIME if time is not provided
ctx.obj["time"] = time if time is not None else DEFAULT_TIME
Expand Down
Loading