fix: resolving the Claude rendering issue #23
Conversation
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
sestinj
requested changes
Jun 28, 2025
| return false | ||
| // If alwaysApply is not explicitly set, default to true | ||
| // This ensures rules without the alwaysApply field are included in single-file formats | ||
| return true |
There was a problem hiding this comment.
Given the ways that this function is used, I think we should allow it to still return false, but should instead change just the rendering logic for CLAUDE.md, etc. files.
I'm happy to do this myself, just lmk, but what I would recommend here is a) change the spec for rendering so that it describes the behavior we want and then b) in Continue, run the @ regenerate command, which will help you update the codebase to match the change in behavior from the spec
I've found this workflow helps keep everything pretty consistent
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.
What is this?
fixes #22
User reported the basic example in the docs is not working when running
render claude. After a quick triage it is due to thealwaysApplynot existing in thestarter/nextjs-rulesrules does not included this frontmatter and the rendering specifically is looking for that.