Skip to content

new skills#24

Merged
alvinsng merged 2 commits intomasterfrom
alvin-new-skills
Apr 22, 2026
Merged

new skills#24
alvinsng merged 2 commits intomasterfrom
alvin-new-skills

Conversation

@alvinsng
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread plugins/code-review/skills/create-pr/SKILL.md
Comment thread plugins/code-review/skills/follow-up-on-pr/SKILL.md Outdated
Comment thread plugins/debugging/skills/http-toolkit-intercept/SKILL.md Outdated
…framework agnostic

Addresses PR #24 review feedback from @factory-davidgu. Generalize the
three skills away from JS/TS (npm/turbo) and Bun/Node specifics:
- create-pr: describe verification categories and show examples across
  ecosystems (JS/TS, Python, Rust, Go, JVM, Ruby) instead of hard-coded
  npm/turbo commands.
- follow-up-on-pr: same generalization for the CI-check step.
- http-toolkit-intercept: replace Bun-centric guidance with a runtime
  proxy matrix covering Node, Bun, Deno, Python, Go, Ruby, JVM, .NET,
  PHP, Rust, curl/shell, and Docker.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@alvinsng alvinsng merged commit c132162 into master Apr 22, 2026
1 check passed
@alvinsng alvinsng deleted the alvin-new-skills branch April 22, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants