issue-review

name

issue-review

description

Reviews a GitHub issue to determine validity, classify as user error or bug, check if already fixed, and take appropriate action. For user errors, improves error messages and documentation. For real bugs, implements a fix following the full commit workflow including tests and adversarial review. Closes the issue with a clear resolution message. Use when the user says "review issue", "triage issue", "is this a bug", "check issue", "issue review", or provides a GitHub issue URL or number.

GitHub Issue Review and Resolution

Systematically review a GitHub issue to determine its validity, classify it, and take the appropriate action — whether that is improving documentation, fixing a bug, or closing as not-a-bug.

Prerequisites

• gh CLI authenticated to github.com (gh auth login --hostname github.com)

gh auth status --hostname github.com 2>/dev/null || echo "ERROR: run 'gh auth login --hostname github.com'"

Repository

• Owner: mock-server
• Repo: mockserver

Step 1: Parse Input and Fetch Issue

Extract the issue number from the user's input (URL or number).

URL Pattern:

https://github.com/mock-server/mockserver-monorepo/issues/{number}

Fetch the full issue with comments:

gh issue view {number} --repo mock-server/mockserver-monorepo --json number,title,body,author,createdAt,updatedAt,state,labels,comments,assignees

Read the issue carefully. Extract:

• Reported behaviour: what the user says is happening
• Expected behaviour: what the user expected
• Steps to reproduce: how to trigger the issue
• Environment: MockServer version, Java version, OS, deployment method (Docker, JAR, WAR, Maven plugin)
• Configuration: any custom properties, expectations, or settings mentioned
• Error messages: exact error text, stack traces, or log output
• Attachments: links to logs, screenshots, or configuration files

If critical information is missing (no version, no reproduction steps, no error output), note this as a gap but continue investigation with what is available.

Step 2: Validate the Issue — Is This Still Relevant?

Before deep investigation, check if the issue is outdated:

2a: Check MockServer Version

# Current released version
gh release list --repo mock-server/mockserver-monorepo --limit 1 --json tagName,publishedAt

# Version from pom.xml (development version — target the project version, not parent/plugin)
grep -m1 'SNAPSHOT' mockserver/pom.xml | sed 's/.*<version>\(.*\)<\/version>.*/\1/'

If the issue was filed against a version more than 2 major versions old, check whether the affected code has been significantly rewritten. If so, the issue may no longer apply.

2b: Search for Existing Fixes

Search for commits and PRs that may have already addressed this issue:

Extract the issue creation date for use in --since filters:

ISSUE_DATE=$(gh issue view {number} --repo mock-server/mockserver-monorepo --json createdAt --jq '.createdAt[:10]')

# Search commit history for references to this issue
git log --all --oneline --grep="#{number}"
git log --all --oneline --grep="issue {number}"
git log --all --oneline --grep="fix.*{keyword}" --since="$ISSUE_DATE"

# Search closed PRs
gh pr list --repo mock-server/mockserver-monorepo --state closed --search "{keywords from issue title}" --json number,title,mergedAt --limit 10

# Search for related code changes
git log --all --oneline --since="$ISSUE_DATE" -- {files_likely_affected}

2c: Check if Reported Behaviour Still Exists

Locate the relevant code and verify whether the reported behaviour is still present in the current codebase:

# Find relevant source files — use a short, sanitised class name or keyword,
# not the full raw error message (which may contain shell metacharacters)
grep -rl "{sanitised_class_name_or_keyword}" --include="*.java" .

Read the relevant code to determine if the issue still applies.

Classification at this point:

Finding	Classification	Next Step
Fix commit found referencing this issue	ALREADY_FIXED	Go to Step 8 (close)
Code rewritten, behaviour no longer possible	ALREADY_FIXED	Go to Step 8 (close)
Code unchanged, behaviour still present	POTENTIALLY_VALID	Continue to Step 3
Cannot determine from code alone	NEEDS_REPRODUCTION	Continue to Step 3

Step 3: Classify — User Error, Bug, or Feature Request?

Investigate the root cause by examining the code, documentation, and the user's report.

3a: Check Configuration and Usage

Review the user's configuration against the documentation:

# Check if the configuration property exists and what it does
grep -rl "{property_name}" --include="*.java" mockserver/mockserver-core/src/main/java/org/mockserver/configuration/

# Check documentation for the feature
grep -rl "{feature_or_property}" jekyll-www.mock-server.com/ --include="*.html" --include="*.md"

Common user error patterns:

• Incorrect expectation JSON syntax
• Wrong content-type headers
• Misunderstanding of matching semantics (subString vs exact)
• Using HTTP features with HTTPS or vice versa
• Port conflicts or incorrect server lifecycle management
• Classpath issues with the Maven plugin or JUnit integration
• Docker networking issues (localhost vs container networking)
• Configuration property spelling/naming mistakes

3b: Attempt Reproduction (if feasible)

If the issue includes reproduction steps and can be verified via unit test:

1. Write a minimal test case that reproduces the reported behaviour
2. Run it against the current codebase

# Run a specific test
cd mockserver && ./mvnw test -pl {module} -Dtest="{TestClass}#{testMethod}"

3c: Classify

Finding	Classification	Next Step
User misconfigured or misused the API	USER_ERROR	Go to Step 4
Documentation is misleading or incomplete	DOCS_GAP	Go to Step 4
Error message is unhelpful or missing	ERROR_MSG_GAP	Go to Step 4
Behaviour contradicts documentation	BUG	Go to Step 6
Unexpected behaviour with valid configuration	BUG	Go to Step 6
Edge case not handled	BUG	Go to Step 6
Requesting new functionality or API surface	FEATURE_REQUEST	Go to Step 5
Requesting behaviour that already exists	FEATURE_REQUEST + ALREADY_EXISTS	Go to Step 5

A single issue may have multiple classifications (e.g., USER_ERROR + DOCS_GAP + ERROR_MSG_GAP).

Step 4: User Error — Improve Error Messages and Documentation

Even when an issue is user error, it signals a gap in developer experience. Address it.

4a: Improve Error Messages

If the error message could have guided the user to the correct solution:

1. Find the code that produces the error or handles the misconfiguration
2. Add or improve validation with a clear, actionable error message
3. The error message MUST include:
   • What was wrong (the specific invalid value or configuration)
   • What was expected (the correct format, valid values, or constraints)
   • Where to find help (link to documentation page or configuration property name)

Example pattern:

throw new IllegalArgumentException(
    "Invalid value \"" + value + "\" for property \"" + propertyName + "\". "
    + "Expected: " + expectedFormat + ". "
    + "See: https://www.mock-server.com/mock_server/configuration_properties.html"
);

4b: Improve Documentation

If the documentation could have prevented the user's mistake:

1. Identify the relevant documentation page in jekyll-www.mock-server.com/
2. Add or clarify the section that would have helped
3. Consider adding:
   • A common pitfalls or troubleshooting section
   • A concrete example showing the correct usage
   • A note about the specific misconception
4. Keep changes minimal and user-focused — explain WHAT to do, not internal implementation details

4c: Validate Changes

Follow the validation steps from .opencode/rules/commit-workflow.md Step 2 for all changed file categories.

After validation passes, proceed to Step 7.

Step 5: Feature Request — Evaluate Complexity vs Value

Feature requests require careful evaluation. MockServer's value comes from a simple, predictable API. Every new feature adds surface area that users must learn, maintainers must support, and that can interact unexpectedly with existing features.

5a: Check if the Requested Behaviour Already Exists

Before evaluating a new feature, check whether MockServer can already do what the user wants:

# Search documentation for the requested capability
grep -rl "{feature_keyword}" jekyll-www.mock-server.com/ --include="*.html" --include="*.md"

# Search the codebase for existing support
grep -rl "{feature_keyword}" --include="*.java" mockserver/mockserver-core/src/main/java/

Common cases where the feature already exists:

• The user wants a specific matching behaviour that is already supported via a different matcher type
• The user wants configuration that exists under a different property name
• The user wants an API that exists but is not well-documented
• The user wants behaviour achievable through expectation composition (multiple expectations working together)

If the feature already exists, classify as ALREADY_EXISTS and go to Step 4 to improve documentation (so future users can find it).

5b: Evaluate Whether to Add the Feature

Apply these criteria in order:

Reject if any of these are true:

• The feature adds a new way to do something already possible — this adds UX complexity without new capability
• The feature serves a narrow use case that most users would never encounter
• The feature would require users to learn new concepts or API patterns to use existing features
• The feature introduces ambiguity (two ways to configure the same behaviour, unclear which takes precedence)
• The feature would break backward compatibility
• The feature adds significant maintenance burden relative to its value

Accept if all of these are true:

• The feature addresses a genuine gap — something users legitimately cannot do today
• The feature follows existing patterns and conventions (consistent API surface)
• The implementation is minimal and surgical — no new abstractions, no framework changes
• The feature is self-contained and does not create unexpected interactions with existing features
• The benefit clearly outweighs the added complexity

5c: Classify

Finding	Classification	Next Step
Feature already exists, user didn't know	ALREADY_EXISTS	Go to Step 4 (improve docs)
Feature makes sense, passes all criteria	ACCEPTED	Go to Step 6 (implement)
Feature fails complexity/value criteria	DECLINED	Go to Step 8 (close)
Feature needs more thought/discussion	NEEDS_DESIGN	Go to Step 8 (close with recommendation)

Step 6: Real Bug or Accepted Feature — Implement the Fix

6a: Identify the Fix Location

Follow the Fix Placement Policy from AGENTS.md: fix the bug at the architecturally correct layer. If the bug surfaces in mockserver-netty but the root cause is in mockserver-core, fix it in mockserver-core.

Read the project's module overview at docs/code/overview.md to understand module boundaries.

6b: Implement the Fix

1. Read the surrounding code context thoroughly before making changes
2. Follow existing code conventions (see .opencode/rules/coding-principles.md)
3. Make the minimal change that correctly fixes the issue
4. Do not refactor adjacent code — surgical changes only

6c: Write or Update Tests

1. Add a test that reproduces the reported issue (fails before the fix, passes after)
2. Add edge case tests if the fix reveals related boundary conditions
3. Follow existing test conventions in the module (most modules use JUnit 4; use JUnit 5 only in mockserver-junit-jupiter)

6d: Run Tests

# Run tests for affected modules
cd mockserver && ./mvnw test -pl {affected_modules}

If tests fail, fix the code and re-run until all tests pass.

6e: Check for Regression

Verify the fix doesn't break other behaviour:

# Run the full test suite for the affected module
cd mockserver && ./mvnw test -pl {module}

After all tests pass, proceed to Step 7.

Step 7: Commit — Follow the Full Pre-Commit Workflow

Follow the COMPLETE workflow from .opencode/rules/commit-workflow.md:

7a: Classify Changed Files

Run git status --short and classify all changed files by category.

7b: Run Category-Specific Validations

Run ALL validations for ALL categories of changed files (Java tests, doc link checks, etc.).

7c: Stage Files

Stage files individually by explicit path (NEVER git add .):

git add {file1} {file2} ...

7d: Adversarial Review

Launch a review-cheap subagent via the Task tool with subagent_type: "review-cheap":

Provide the staged diff (git diff --cached) and ask the reviewer to check:

• Hallucinated function/method/module names that don't exist
• Plausible-looking but incorrect logic
• Missing error handling or edge cases
• Security issues
• Whether the fix actually addresses the reported issue
• Whether tests adequately cover the fix
• Whether error message improvements are clear and actionable
• Whether documentation changes are accurate and user-friendly

If the review returns BLOCK, fix the issues, re-run validations, and re-run the review.

7e: Re-run Tests After Review Fixes

If the adversarial review caused code changes, re-run all affected tests:

cd mockserver && ./mvnw test -pl {affected_modules}

7f: Commit

Only after all validations pass AND the adversarial review returns PASS, create the commit with a message that references the issue:

Fix #{number}: {concise description of the fix}

{Brief explanation of the root cause and what was changed}

For user-error issues with documentation/error-message improvements:

Improve error handling for {feature} (#{number})

{Brief explanation of what was improved and why}

Step 8: Close the Issue

GATE: Before closing, present the classification, proposed resolution, and intended closing comment to the user. Only execute gh issue close after explicit user approval.

Close the issue with a clear, helpful message that explains the resolution.

For ALREADY_FIXED issues:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
This issue has been resolved. The fix was included in commit {sha} ({brief description}).

If you are still experiencing this issue, please ensure you are using MockServer version {version} or later. If the problem persists on the latest version, please open a new issue with updated reproduction steps.
EOF
)"

For BUG fixes:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Fixed in commit {sha}.

**Root cause:** {one-line explanation of what was wrong}
**Fix:** {one-line explanation of what was changed}

This fix will be included in the next release. In the meantime, you can build from the `master` branch to get the fix immediately.
EOF
)"

For USER_ERROR with improvements:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Thank you for reporting this. After investigation, this turned out to be a configuration issue rather than a bug — {brief explanation of the correct usage}.

However, your report highlighted that {the error message / documentation} could be clearer. We've made the following improvements in commit {sha}:

- {improvement 1}
- {improvement 2}

These improvements will help other users avoid the same issue. {Link to relevant documentation page if applicable}.
EOF
)"

For USER_ERROR without improvements needed:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Thank you for reporting this. After investigation, this appears to be a {configuration/usage} issue rather than a bug.

**What's happening:** {explanation of the observed behaviour}
**Expected usage:** {correct way to achieve what the user wanted}

{Link to relevant documentation: https://www.mock-server.com/...}

If this doesn't resolve your issue, please feel free to reopen with additional details.
EOF
)"

For FEATURE_REQUEST — already exists:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Thank you for the feature request. MockServer already supports this — {brief explanation of how to achieve it}.

{Example configuration or API call showing the existing approach}

See: {link to relevant documentation page}

We've improved the documentation to make this easier to discover. If this doesn't fully address your use case, please feel free to reopen with details about what's missing.
EOF
)"

For FEATURE_REQUEST — declined:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Thank you for the feature request. After careful evaluation, we've decided not to add this feature.

**Reason:** {explanation — e.g., "This would add a second way to achieve {X}, which is already possible via {existing approach}. Adding another mechanism would increase the API surface area without new capability, making MockServer harder to learn and maintain."}

{If applicable: "The existing approach to achieve this is: {description}. See: {link}"}

We try to keep MockServer's API surface minimal and predictable. Every new feature must justify the complexity it adds for all users, not just the requesting use case.

If you believe we've misunderstood your use case, please feel free to reopen with additional context.
EOF
)"

For FEATURE_REQUEST — needs design:

gh issue close {number} --repo mock-server/mockserver-monorepo --comment "$(cat <<'EOF'
Thank you for the feature request. This is an interesting idea that would benefit from further design discussion before implementation.

**What we found:** {summary of investigation — what exists today, what the gap is}
**Complexity concern:** {what makes this non-trivial — interaction with existing features, API surface impact, etc.}

We recommend opening a discussion or creating a more detailed proposal covering:
- Specific use cases this would enable
- Proposed API/configuration surface
- How it interacts with existing features

This will help ensure any implementation adds clear value without unnecessary complexity.
EOF
)"

Decision Flowchart

flowchart TD
    A[Issue received] --> B[Step 1: Fetch and parse issue]
    B --> C[Step 2: Check if still relevant]
    C --> D{Already fixed?}
    D -->|Yes| CLOSE_FIXED[Step 8: Close as already fixed]
    D -->|No| E[Step 3: Classify]
    E --> F{Classification?}
    F -->|USER_ERROR| G{Can we improve DX?}
    G -->|Yes: error msgs / docs| H[Step 4: Improve error messages and docs]
    G -->|No: clear enough| CLOSE_USER[Step 8: Close as user error]
    H --> I[Step 7: Commit workflow]
    I --> CLOSE_IMPROVED[Step 8: Close with improvements noted]
    F -->|BUG| J[Step 6: Implement fix + tests]
    J --> K[Step 7: Commit workflow]
    K --> CLOSE_BUG[Step 8: Close with fix reference]
    F -->|FEATURE_REQUEST| L[Step 5: Evaluate complexity vs value]
    L --> M{Already exists?}
    M -->|Yes| H
    M -->|No| N{Passes criteria?}
    N -->|Yes| J
    N -->|No| CLOSE_DECLINED[Step 8: Close — declined or needs design]

Notes

• Always be respectful in closing comments — the user took time to report the issue
• Even user errors are valuable signals about developer experience gaps
• Check the issue comments for additional context — the original report may have been updated
• If the issue has multiple problems (e.g., a real bug AND a documentation gap), address all of them
• If the fix is too complex or risky for a single session, document findings and recommend next steps instead of making a partial fix
• If the issue requires changes across multiple modules, verify the fix at each layer
• Never close an issue without a clear, actionable resolution message
• Reference the commit SHA in the closing comment so users can track the fix
• For issues filed against old versions, check the CHANGELOG or release notes for relevant fixes
• If reproduction requires infrastructure (Docker, specific OS, etc.) that isn't available, note this limitation and make a best-effort assessment from code review
• For feature requests, default to "no" — the burden of proof is on the feature to justify the complexity it adds, not on the project to justify rejecting it
• A feature request where the capability already exists is a documentation problem, not a missing feature — improve discoverability instead of adding a second path