Skip to content

Remove rules and mcp_configs input parameters #12

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

Closed
justinxu421 wants to merge 1 commit into from
Closed

Remove rules and mcp_configs input parameters #12

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
4 changes: 1 addition & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,7 @@ Each example includes a complete workflow file that you can copy to your `.githu
| `pull_number` | The number of the pull request being reviewed | Yes | `${{ github.event.pull_request.number }}` |
| `repo_name` | The full name (owner/repo) of the repository | Yes | `${{ github.repository }}` |
| `custom_guidelines` | Custom guidelines to include in PR reviews | No | See [Custom Guidelines](#custom-guidelines) section |
| `model` | Optional model name to use; passed directly to augment agent | No | e.g. `sonnet4`, from `auggie --list-models` |
| `rules` | JSON array of rule file paths forwarded to augment agent as repeated `--rules` flags | No | `'[".augment/rules.md"]'` |
| `mcp_configs` | JSON array of MCP config file paths forwarded as repeated `--mcp-config` flags | No | `'[".augment/mcp.json"]'` |
| `model` | Optional model name to use; passed directly to augment agent | No | e.g. `sonnet4`, from `auggie --list-models` |

## How It Works

Expand Down
8 changes: 0 additions & 8 deletions action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,6 @@ inputs:
model:
description: "Optional model name to use; passed directly to augmentcode/augment-agent."
required: false
rules:
description: "Optional JSON array of rules file paths to forward as --rules flags to augmentcode/augment-agent."
required: false
mcp_configs:
description: "Optional JSON array of MCP config file paths to forward as --mcp-config flags to augmentcode/augment-agent."
required: false

runs:
using: "composite"
Expand Down Expand Up @@ -65,5 +59,3 @@ runs:
repo_name: ${{ inputs.repo_name }}
custom_context: ${{ steps.custom_context.outputs.context }}
model: ${{ inputs.model }}
rules: ${{ inputs.rules }}
mcp_configs: ${{ inputs.mcp_configs }}
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "augment-review-pr",
"version": "0.1.4",
"version": "0.1.3",
Copy link
Contributor

@github-actions github-actions bot Oct 8, 2025

Choose a reason for hiding this comment

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

Semantic Versioning Concern: Downgrading from version 0.1.4 to 0.1.3 violates semantic versioning principles. Version 0.1.4 has already been released (commit 866c166), and downgrading to a previous version number can cause confusion and conflicts for users who may have already adopted 0.1.4.

Recommended approach:

  • Bump to 0.1.5 if this is a patch-level change (bug fix/minor improvement)
  • Bump to 0.2.0 if removing these parameters is considered a breaking change for users who were using them

Since this PR removes functionality that users may depend on, version 0.2.0 would be more appropriate to signal the breaking change.

Current code:

"version": "0.1.3",

Suggested improvement:

"version": "0.2.0",

"description": "AI-powered code assistance for reviewing pull requests using Augment's intelligent development tools",
"main": "action.yml",
"type": "module",
Expand Down
Loading