Repository navigation
Install skills for the rolemodel_rails utilities in core_setup - #222
Merged
Merged
Conversation
Every app with the gem gets resource_for and TaskTools, but nothing tells the agent they exist. In baseline runs, agents asked for rake tasks wrote their own dry-run flags, puts output, and progress lines instead of TaskTools. core_setup now installs polymorphic-parent-resources and rake-tasks alongside the skills the other generators install. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
sanitize_arguments tested each value with match?('_'), which matches an
underscore anywhere, so jane_doe@example.com silently became the default.
A blank argument also overrode its default, contrary to the README. Only a
bare _ or a blank value falls back to the default now.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The docs listed an Editors generator core_setup doesn't run, credited modals to UI Components instead of Turbo, and claimed --js-runner works here when core_setup never passes it to testing:all. They also never mentioned --skip-deployable or the icon library flags. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Modals moved to rolemodel:turbo:modals, but the README still listed them under UI Components with a link to a directory that no longer exists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why?
Re-lands #221, which was reviewed and approved there but merged into
install-agent-skillsinstead ofmaster. #221 was stacked on #208, and its base wasn't retargeted tomasterafter #208 was squash-merged.The first two commits are #221's, cherry-picked unchanged onto
master. See #221 for the full description and review. The rest change docs only.What Changed
From #221:
core_setupinstalls thepolymorphic-parent-resourcesandrake-tasksskillssanitize_argumentskeeps values that contain_. Only a bare_or a blank value falls back to the defaultDocs only:
core_setupREADME and USAGE up to date. They listed an Editors generatorcore_setupdoesn't run, credited modals to UI Components instead of Turbo, and claimed--js-runnerworks here whencore_setupnever passes it totesting:all. Both now document--skip-deployableand the icon library flagsrolemodel:turbo:modalsScreenshots
N/A — no UI changes
🤖 Generated with Claude Code