Skip to content

[HOLD] Add an automated rubocop agent and workflow to capture new cop noise - #140

Open
aaron-collier wants to merge 2 commits into
mainfrom
add-rubocop-updater-agent
Open

[HOLD] Add an automated rubocop agent and workflow to capture new cop noise#140
aaron-collier wants to merge 2 commits into
mainfrom
add-rubocop-updater-agent

Conversation

@aaron-collier

Copy link
Copy Markdown
Contributor

In an effort to continue to improve our lives, particularly for noisy things like "new cops" installed output when running rubocop, this adds:

  • An agent that reads a list of new cops from a ticket that is assigned to copilot and enables the cop, tries to address complaints, and disables the cop if the corrections are problematic
  • An automated workflow that acts on each merge to main to detect if the version of rubocop was updated, what changed, and then open a ticket with any new cops, assigns the ticket to copilot to automate the agent above.

For now, the workflow_dispatch block of the workflow is so that this run without an update to robocop for testing so that we don't have to jump through hoops to test a version change.

Comment thread .github/agents/rubocop-updater.agent.md Outdated

### Step 4 — Run the test suite
After autocorrect and any manual fixes, run:
bundle exec rspec --format progress

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Needs a DB.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh yeah, good catch. Removed the rpsec instructions for now.

@aaron-collier aaron-collier changed the title Add an automated rubocop agent and workflow to capture new cop noise [HOLD] Add an automated rubocop agent and workflow to capture new cop noise Apr 8, 2026
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.

2 participants