Skip to content

feat(skills): add create-github-issue skill#2390

Merged
tusharmath merged 4 commits intomainfrom
create-issue-skill
Feb 12, 2026
Merged

feat(skills): add create-github-issue skill#2390
tusharmath merged 4 commits intomainfrom
create-issue-skill

Conversation

@tusharmath
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Feb 12, 2026
Comment on lines +56 to +80
# Build gh issue command
CMD="gh issue create --title \"$TITLE\""

if [ -n "$BODY" ]; then
CMD="$CMD --body \"$BODY\""
fi

if [ -n "$LABELS" ]; then
CMD="$CMD --label \"$LABELS\""
fi

if [ -n "$ASSIGNEE" ]; then
CMD="$CMD --assignee \"$ASSIGNEE\""
fi

if [ -n "$MILESTONE" ]; then
CMD="$CMD --milestone \"$MILESTONE\""
fi

if [ "$DRAFT" = true ]; then
CMD="$CMD --draft"
fi

# Execute command
eval $CMD No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Critical: Shell injection vulnerability and command execution failure

Building a command string with eval creates multiple critical issues:

  1. Shell injection vulnerability: Variables like $TITLE and $BODY are not properly escaped. Malicious input or special characters will break out of quotes and could execute arbitrary commands.

  2. Will fail with multi-line content: When $BODY contains newlines (from $(cat .forge/FORGE_ISSUE_BODY.md)), the string concatenation and eval approach will fail to properly preserve the content.

  3. Quote handling broken: The nested quoting with \" will not work correctly with eval.

Fix: Call gh directly instead of using eval:

ARGS=("--title" "$TITLE")

[ -n "$BODY" ] && ARGS+=("--body" "$BODY")
[ -n "$LABELS" ] && ARGS+=("--label" "$LABELS")
[ -n "$ASSIGNEE" ] && ARGS+=("--assignee" "$ASSIGNEE")
[ -n "$MILESTONE" ] && ARGS+=("--milestone" "$MILESTONE")
[ "$DRAFT" = true ] && ARGS+=("--draft")

gh issue create "${ARGS[@]}"
Suggested change
# Build gh issue command
CMD="gh issue create --title \"$TITLE\""
if [ -n "$BODY" ]; then
CMD="$CMD --body \"$BODY\""
fi
if [ -n "$LABELS" ]; then
CMD="$CMD --label \"$LABELS\""
fi
if [ -n "$ASSIGNEE" ]; then
CMD="$CMD --assignee \"$ASSIGNEE\""
fi
if [ -n "$MILESTONE" ]; then
CMD="$CMD --milestone \"$MILESTONE\""
fi
if [ "$DRAFT" = true ]; then
CMD="$CMD --draft"
fi
# Execute command
eval $CMD
# Build gh issue command
ARGS=("--title" "$TITLE")
if [ -n "$BODY" ]; then
ARGS+=("--body" "$BODY")
fi
if [ -n "$LABELS" ]; then
ARGS+=("--label" "$LABELS")
fi
if [ -n "$ASSIGNEE" ]; then
ARGS+=("--assignee" "$ASSIGNEE")
fi
if [ -n "$MILESTONE" ]; then
ARGS+=("--milestone" "$MILESTONE")
fi
if [ "$DRAFT" = true ]; then
ARGS+=("--draft")
fi
# Execute command
gh issue create "${ARGS[@]}"

Spotted by Graphite Agent

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

Add support for dark mode theme to improve accessibility and user experience.

## Problem Statement
Users have requested dark mode support through multiple feedback channels. The current light-only并发主题导致长时间使用时眼部疲劳,且不尊重用户的系统偏好设置。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mixed language content: Chinese text is embedded in the English documentation (并发主题导致长时间使用时眼部疲劳,且不尊重用户的系统偏好设置). This appears to be accidental and should be translated to English or removed.

Suggested change
Users have requested dark mode support through multiple feedback channels. The current light-only并发主题导致长时间使用时眼部疲劳,且不尊重用户的系统偏好设置。
Users have requested dark mode support through multiple feedback channels. The current light-only theme causes eye fatigue during prolonged use and does not respect the user's system preference settings.

Spotted by Graphite Agent

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@tusharmath tusharmath merged commit ac26e78 into main Feb 12, 2026
9 checks passed
@tusharmath tusharmath deleted the create-issue-skill branch February 12, 2026 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Brand new functionality, features, pages, workflows, endpoints, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant