Repository navigation
Make the feedback section read as an instruction - #29
Merged
Merged
Conversation
"Send it yourself" had no noun for "it", "needs no approval" didn't say whose, and the tenses mixed past and present. It now says: if a skill, tool, command or doc turns out wrong, stale or confusing, or works unusually well, send Jam feedback; don't ask the user first.
"Jam's agents read it" gave a reason without the reasoning. It now says the feedback goes only to Jam and, by the rules below it, holds none of the user's code or data.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
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.
The "Send feedback to Jam" section at the end of every Jam skill read oddly: "Send it yourself" had nothing for "it" to point to, "needs no approval" didn't say whose or why, and the sentence switched between past and present tense.
It now reads, with the approval part dropped:
Same change in all 10 Jam skills, so
scripts/check.mjsstill finds the section identical everywhere.jam-proofandjam-annotateare also bundled in the Jam CLI, so the bundle needs a resync after this merges.Stacked on #26.
Created with Claude Opus 5.5 in Claude Code.