Skip to content

refactor(ruby-rails): version-agnostic skill with mandatory testing#2

Open
asharijuang wants to merge 2 commits intomainfrom
improve/ruby-rails-generic-and-testing
Open

refactor(ruby-rails): version-agnostic skill with mandatory testing#2
asharijuang wants to merge 2 commits intomainfrom
improve/ruby-rails-generic-and-testing

Conversation

@asharijuang
Copy link
Copy Markdown
Member

Summary

  • Remove hardcoded Ruby 3.1.3 / Rails 7.0 version locks — agent now detects versions from Gemfile.lock and .ruby-version
  • Stop assuming specific gems (Avo, kredis, Jbuilder) are present — agent checks Gemfile first
  • Add explicit testing requirements: every new feature, behavior change, or bug fix must include specs
  • Add Run specs step to the "When Invoked" workflow and checklist

Files Changed (7 files, +94 -56)

File What Changed
SKILL.md Version detection step, generic gem references, testing enforcement in constraints/checklist
references/api-docs.md Replaced hardcoded versions with "detect from Gemfile" guidance
references/api-development.md Generic serialization references instead of Jbuilder-only
references/background-jobs.md Support Sidekiq, GoodJob, Solid Queue, Active Job
references/openapi-documentation.md Detect existing doc location instead of hardcoding path
references/rspec-testing.md Added "New feature testing requirements" section with per-type guidance
scripts/helper.py Updated checklist: version detection, generic gems, mandatory testing

Test plan

  • Install the skill and invoke it on a Rails 7 project — verify it detects Rails 7 correctly
  • Invoke on a Rails 8 project — verify no Rails 7-specific advice leaks through
  • Ask it to add a new endpoint — verify it generates request specs automatically
  • Ask it to fix a bug — verify it creates a regression spec

aguunng and others added 2 commits April 10, 2026 05:36
… requirements

Remove hardcoded Ruby 3.1.3 and Rails 7.0 version locks — the skill now
instructs the agent to detect versions from Gemfile.lock and .ruby-version.
Gem references (Avo, kredis, Jbuilder) are no longer assumed present;
the agent checks the Gemfile first.

Add explicit testing requirements: every new feature, behavior change,
or bug fix must include accompanying specs before the task is complete.
Covers model specs, request specs, service specs, job specs, policy specs,
and regression specs for bug fixes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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