Repository navigation
fix(coord): COORD_HUMAN accepts a leading @ - #129
Merged
Merged
Conversation
The comment header adds its own @, so COORD_HUMAN="@login" produced "human: @@login". Both scripts now strip one leading @, and SKILL.md says either form works. Closes #128. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 6, 2026
Open
xcjs
approved these changes
Oct 6, 2026
xcjs
left a comment
xcjs
left a comment
Owner
There was a problem hiding this comment.
Approved as non-author. Verified the diff: both scripts strip one leading @ from COORD_HUMAN (TrimStart in coord.ps1, param expansion in coord.sh), the header still adds its own @, SKILL.md documents both forms, and the @@xcjs examples from #120/#5 comments are exactly the symptom this fixes. bash -n + PS parse claims are consistent with the change size.
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
2 participants
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.
COORD_HUMAN(#96) holds a GitHub login, and the comment header adds its own@. When it's set as@login, comments readhuman: @@login. #120's STATUS on 10-04 and the hub SYNC on 10-05 02:49Z both show it. Both scripts now strip one leading@, and SKILL.md says either form works.Closes #128.
Verification
bash -n coord.shis clean. The expansion maps@xcjsandxcjstohuman: @xcjs, and an unset value falls back to the gh account.coord.ps1parses with 0 errors in PowerShell 7.4, andTrimStart('@')gives the same two results.🤖 Generated with Claude Code