Skip to content

Add @-commands to tools/query.py, rewrite TODO.md, pytest config improvement - #70

Merged
gvanrossum-ms merged 11 commits into
mainfrom
guido
Nov 6, 2025
Merged

Add @-commands to tools/query.py, rewrite TODO.md, pytest config improvement#70
gvanrossum-ms merged 11 commits into
mainfrom
guido

Conversation

@gvanrossum-ms

@gvanrossum-ms gvanrossum-ms commented Nov 6, 2025

Copy link
Copy Markdown
Contributor
  • Add @-commands to tools/query.py (hardly the "small" task Copilot thought it would be)
  • Move 'test' argument from pytest args to pyproject.toml
  • Completely redid TODO.md using grouping by topic, priorities, and sizes (estimated by various LLMs)

Copilot AI 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.

Pull Request Overview

This PR adds interactive @-command support to the query tool (tools/query.py), enabling users to dynamically control debug settings, run partial pipeline stages, view statistics, and access help documentation. The changes also include test configuration updates to use pytest's standard test discovery mechanism.

  • Adds four @-commands: @help, @debug, @stage, and @stats with argument parsing and error handling
  • Implements command routing in the interactive loop to detect and handle lines starting with @
  • Updates test configuration to use pytest's testpaths setting for more consistent test discovery

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tools/query.py Adds shlex import, implements @-command infrastructure including command handlers, argument parsing, and interactive loop integration
test/test_mcp_server.py Updates test query from "Who is the host of this podcast?" to "Who is Kevin Scott?"
pyproject.toml Adds testpaths = ["test"] configuration for pytest test discovery
make.bat Removes explicit test directory argument from pytest command (relies on testpaths config)
TODO.md Major reorganization: adds priority levels (P0-P3) and size estimates, groups tasks by component/feature area
Makefile Removes explicit test directory argument from pytest commands (relies on testpaths config)

Comment thread tools/query.py Outdated
Comment thread tools/query.py Outdated
@gvanrossum-ms gvanrossum-ms changed the title Update TODO and do a few "small" tasks Add @-commands to tools/query.py, rewrite TODO.md, pytest config improvement Nov 6, 2025
@gvanrossum-ms
gvanrossum-ms merged commit c94dd7b into main Nov 6, 2025
21 checks passed
@gvanrossum-ms
gvanrossum-ms deleted the guido branch November 6, 2025 02:59
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.

3 participants