Compact LLM files: consolidate, remove unused#157
Conversation
Result: 39→27 files (-31%). Bash agent 61%↓, Python agent 79%↓. Changes: - Agents: Reference instructions; rm 4.1-Beast (generic), rust (unused) - Instructions: Enhance Python; rm Windows (ahk/cmd/ps1), unused langs (go/java/rust/node) - Prompts: Rm bash/python wrappers (duplicate instructions) - copilot-instructions.md: Reference bash.instructions.md vs duplicate ∴ DRY principle, minimal Δ, repo-specific focus per CLAUDE.md
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
🤖 Hi @Ven0m0, I've received your request, and I'm working on it now! You can track my progress in the logs for more details. |
Summary of ChangesHello @Ven0m0, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly refactors the repository's LLM-related configurations by applying the DRY principle and focusing on repo-specific needs. The primary goal was to reduce file count and agent complexity by removing redundant or unused files and centralizing detailed instructions. This leads to a more compact and maintainable set of configurations, improving clarity and reducing overhead for agents and instruction sets. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
🤖 I'm sorry @Ven0m0, but I was unable to process your request. Please see the logs for more details. |
There was a problem hiding this comment.
Code Review
This pull request does an excellent job of consolidating and simplifying the LLM agent and instruction files. Removing unused files and applying the DRY principle has significantly cleaned up the repository structure. The new agent definitions are much more concise, and the enhanced Python instructions are a solid improvement. I've found a couple of minor areas for improvement in the new instructions to make them even better.
There was a problem hiding this comment.
Pull request overview
This pull request successfully consolidates the LLM configuration files from 39 to 27 files (-31%), achieving its stated goal of reducing duplication and focusing on repo-specific content. The changes follow the DRY (Don't Repeat Yourself) principle by having agent files reference instruction files rather than duplicating standards.
Changes:
- Removed prompt files (bash.prompt.md, python.prompt.md) that duplicated instruction content
- Removed Windows-specific instructions (PowerShell, CMD, AutoHotkey) for platforms not used in this repository
- Removed instruction files for unused languages (Rust, Go, Java, Node.js)
- Streamlined agent files (bash, python) to reference instruction files instead of embedding full standards
- Removed generic 4.1-Beast agent and unused rust agent
- Enhanced python.instructions.md with modern patterns (Protocol, type hints, security)
- Updated copilot-instructions.md to reference bash.instructions.md instead of duplicating content
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/prompts/bash.prompt.md | Removed - duplicated bash.instructions.md content |
| .github/prompts/python.prompt.md | Removed - duplicated python.instructions.md content |
| .github/instructions/rust.instructions.md | Removed - no Rust code in repository |
| .github/instructions/powershell.instructions.md | Removed - no PowerShell scripts in repository |
| .github/instructions/cmd.instructions.md | Removed - no CMD/Batch files in repository |
| .github/instructions/autohotkey.instructions.md | Removed - no AutoHotkey scripts in repository |
| .github/instructions/nodejs-javascript-vitest.instructions.md | Removed - no Node.js project in repository |
| .github/instructions/java.instructions.md | Removed - no Java code in repository |
| .github/instructions/go.instructions.md | Removed - no Go code in repository |
| .github/instructions/intructions.intructions.md | Removed - meta-instructions no longer needed |
| .github/instructions/python.instructions.md | Enhanced with modern patterns, type safety examples, and security guidelines |
| .github/agents/bash.agent.md | Streamlined to reference bash.instructions.md (61% reduction) |
| .github/agents/python.agent.md | Streamlined to reference python.instructions.md (79% reduction) |
| .github/agents/rust.agent.md | Removed - no Rust code in repository |
| .github/agents/4.1-Beast.agent.md | Removed - generic agent not repo-specific |
| .github/copilot-instructions.md | Updated to reference bash.instructions.md instead of duplicating bash standards |
| applyTo: "**/*.py" | ||
| description: "Python coding conventions and guidelines" | ||
| description: "Production Python: strict typing, security, performance" |
There was a problem hiding this comment.
The frontmatter has inconsistent ordering compared to other instruction files. In bash.instructions.md, the order is applyTo then description, but here it's reversed on line 2-3. For maintainability and consistency, frontmatter fields should follow the same order across all instruction files.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
| def stream_file(path: str) -> Iterator[str]: | ||
| with open(path) as f: | ||
| for line in f: | ||
| yield line.strip() |
There was a problem hiding this comment.
The function signature uses Iterator[str] but the import statement is missing. The code example should include from collections.abc import Iterator to be complete and runnable.

Result: 39→27 files (-31%). Bash agent 61%↓, Python agent 79%↓.
Changes:
∴ DRY principle, minimal Δ, repo-specific focus per CLAUDE.md