Skip to content

Clarify Python Verification Commands - #813

Merged
ptr727 merged 2 commits into
developfrom
feature/python-verification-skill
Aug 18, 2026
Merged

Clarify Python Verification Commands#813
ptr727 merged 2 commits into
developfrom
feature/python-verification-skill

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Summary

  • Trigger the Python code-style skill when selecting or running Python verification commands.
  • Require repository-specific verification guidance before substituting generic commands.
  • Distinguish build-profile pytest from lint-only unittest execution.
  • Regenerate the GitHub Copilot and Claude Code skill distributions.

Why

The previous trigger covered Python edits and pytest authoring, but not running tests. That gap allowed a generic pytest invocation in this lint-only repository and produced a false missing-dependency diagnosis.

Verification

  • uvx coverage@latest run --source=scripts,spec,host-setup -m unittest discover -s scripts/tests (727 tests passed)
  • python3 scripts/tests/test_build_dist.py (25 tests passed)
  • python3 scripts/build_dist.py --check
  • Diff-scoped prose and sentence-length checks
  • EOL and diff checks
  • markdownlint-cli2 (158 files, 0 issues)

CopilotAI lite review requested due to automatic review settings August 18, 2026 17:41

CopilotAI 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

Updates the fleet’s Python codestyle Skill text to explicitly trigger when selecting/running Python verification commands and to route agents to repository-specific guidance (via OPERATIONS.md) before assuming generic pytest-based workflows. This keeps verification instructions aligned with the repo’s lint-only unittest profile and the generated skill distributions.

Changes:

  • Expand the python-codestyle Skill description to cover choosing/running Python format/lint/type-check/test commands (not just authoring Python code/tests).
  • Clarify the “Two profiles” section to consult OPERATIONS.md before selecting tooling/tests, and to discourage pytest in lint-only repos.
  • Regenerate the distributed skill copies and update the Claude plugin source digest.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

FileDescription
.github/skills/python-codestyle/SKILL.mdUpdates canonical fleet skill guidance for Python verification command selection.
.agents/skills/python-codestyle/SKILL.mdRegenerates the agent-skill distribution copy to match the canonical skill text.
.claude-plugin/fleet-skills/skills/python-codestyle/SKILL.mdRegenerates the Claude plugin skill copy to match the canonical skill text.
.claude-plugin/fleet-skills/.source-digestUpdates the plugin distribution digest for the regenerated skill set.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread.github/skills/python-codestyle/SKILL.md Outdated
Comment thread.agents/skills/python-codestyle/SKILL.md Outdated
Comment thread.claude-plugin/fleet-skills/skills/python-codestyle/SKILL.md Outdated
CopilotAI review requested due to automatic review settings August 18, 2026 17:46

CopilotAI 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

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@ptr727
ptr727 marked this pull request as ready for review August 18, 2026 18:23
@ptr727
ptr727 merged commit 8a545cb into developAug 18, 2026
8 checks passed
Sign up for freeto 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.

2 participants

@ptr727