Skip to content

chore(ai): Add gopls to mcp#16905

Merged
janisz merged 12 commits intomasterfrom
add_gopls_to_mcp
Sep 22, 2025
Merged

chore(ai): Add gopls to mcp#16905
janisz merged 12 commits intomasterfrom
add_gopls_to_mcp

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Sep 18, 2025

Adds AI development tooling to enhance Go development experience with intelligent code assistance and StackRox-specific guidelines.

Changes

Key Features

  • Modern Go development patterns and concurrency best practices
  • StackRox API design standards and coding conventions integration
  • Intelligent code assistance through gopls MCP integration
  • Automated guidance for error handling, testing, and PR standards

Testing performed

  • Verified agent and MCP configuration syntax
  • Confirmed integration with existing Claude Code setu

Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!

Prompt for AI Agents
Please address the comments from this code review:

## Individual Comments

### Comment 1
<location> `.claude/agents/go-expert-architect.md:213-215` </location>
<code_context>
+- "Design a concurrent data processing pipeline with backpressure handling"
+- "Implement a Redis-backed cache with connection pooling"
+- "Set up a modern Go project with proper testing and CI/CD"
+- "Debug and fix race conditions in this concurrent Go code"1~
+
+# The gopls MCP server
</code_context>

<issue_to_address>
**issue (typo):** Stray characters '1~' at the end of the line appear to be a typo.

Please remove the '1~' from the end of the line to improve clarity.

```suggestion
- "Debug and fix race conditions in this concurrent Go code"

# The gopls MCP server
```
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@rhacs-bot
Copy link
Copy Markdown
Contributor

rhacs-bot commented Sep 18, 2025

Images are ready for the commit at cc52b1e.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.9.x-864-gcc52b1e190.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.02%. Comparing base (db21d04) to head (cc52b1e).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #16905      +/-   ##
==========================================
- Coverage   49.02%   49.02%   -0.01%     
==========================================
  Files        2691     2691              
  Lines      201632   201632              
==========================================
- Hits        98846    98845       -1     
- Misses      95123    95124       +1     
  Partials     7663     7663              
Flag Coverage Δ
go-unit-tests 49.02% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
@janisz janisz closed this Sep 18, 2025
@janisz janisz reopened this Sep 18, 2025
Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
@janisz janisz requested a review from mtodor September 19, 2025 09:23
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Sep 22, 2025

/retest

@red-hat-konflux

This comment was marked as off-topic.

@janisz janisz enabled auto-merge (squash) September 22, 2025 14:46
@janisz janisz merged commit fe2e482 into master Sep 22, 2025
87 of 90 checks passed
@janisz janisz deleted the add_gopls_to_mcp branch September 22, 2025 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants