Skip to content

✨ Set up Copilot instructions#16

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-7ba77367-3230-4f53-9d9a-023c3ce3d5c8
Closed

✨ Set up Copilot instructions#16
Copilot wants to merge 2 commits intomainfrom
copilot/fix-7ba77367-3230-4f53-9d9a-023c3ce3d5c8

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 26, 2025

Configures GitHub Copilot instructions for the docker-mcp repository following GitHub's best practices for Copilot coding agent.

What This PR Does

Creates .github/copilot-instructions.md with repository-specific guidance to help Copilot understand and work with this codebase's unique patterns and standards.

Key Features

Architecture Patterns: Documents the Consolidated Action-Parameter Pattern used throughout this repository instead of simple @mcp.tool() decorators. This pattern is critical for Docker infrastructure management operations that require orchestration and state management.

Modern Python Standards: Enforces Python 3.11+ syntax including union types (str | None), type aliases, and modern async exception handling patterns that are used consistently across the codebase.

Security Guidelines: Includes Docker command validation, SSH security requirements, and migration safety patterns that are essential for this infrastructure management tool.

Testing & Development: Documents the FastMCP in-memory testing patterns, pytest markers, and uv-based development workflow specific to this project.

Implementation Approach

The instructions were extracted and consolidated from existing documentation (CLAUDE.md, AGENTS.md, pyproject.toml) to maintain consistency while providing Copilot with actionable guidance. This ensures minimal changes while leveraging the extensive documentation already present in the repository.

The file structure mirrors the repository's modular architecture, explaining the service → tool → model layering and hybrid connection model (Docker contexts vs. SSH) that makes this codebase unique.

Fixes #[issue_number]

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #15

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] ✨ Set up Copilot instructions ✨ Set up Copilot instructions Sep 26, 2025
Copilot AI requested a review from jmagar September 26, 2025 05:54
@jmagar jmagar closed this Sep 27, 2025
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.

✨ Set up Copilot instructions

2 participants