Skip to content

docs(cli): point skill references at absolute raw URLs - #1557

Merged
Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
mainfrom
chore/skill-reference-urls
Sep 2, 2026
Merged

docs(cli): point skill references at absolute raw URLs#1557
Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
mainfrom
chore/skill-reference-urls

Conversation

@theonly1me

Copy link
Copy Markdown
Member

Overview of Changes

tester-skills exposes this skill to Tester by reading skills/qawolf-cli/SKILL.md out of the installed package and passing its contents through createSkillFromSkillMd. Only that one file travels, so the relative links to references/runner.md and references/run-results.md point at paths Tester has no way to open, and the two guides the skill tells it to read first are unreachable.

  • Name the raw GitHub URL beside each relative reference path, so a harness holding only SKILL.md can still fetch the guide.
  • Change the template rather than the generated file, so bun run generate keeps skills/qawolf-cli/SKILL.md in step.

Testing

bun run typecheck && bun run lint && bun run format:check
bun run knip && bun run test

2089 tests pass, and typecheck, lint, format and knip are clean.

  • bun run generate leaves skills/qawolf-cli/SKILL.md byte-identical, so the committed file and its template agree.

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 35dfe750-1871-4c77-bad9-d8e8c44a0ef8

📥 Commits

Reviewing files that changed from the base of the PR and between 960d6fb and c0c1fe5.

📒 Files selected for processing (3)
  • .changeset/skill-reference-urls.md
  • skills/qawolf-cli/SKILL.md
  • src/commands/qawolfCliSkill.template.md

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


Walkthrough

The qawolf-cli skill guidance now provides absolute raw GitHub URLs for the run-results.md and runner.md references when local files are unavailable. The source template includes the same fallback instructions. A patch changeset documents the update for @qawolf/cli.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to c0c1f

This change makes the CLI skill’s referenced guides reachable through absolute raw URLs while keeping the generated file synchronized; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: chajac

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required Conventional Commits format, includes the valid cli scope, uses imperative mood, identifies the URL update, and is 54 characters long.
Description check ✅ Passed The description includes a clear overview and concrete testing results. It omits the required Checklist section, but the main required information is present.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/skill-reference-urls

Comment @coderabbitai help to get the list of available commands.

@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) merged commit 51decf1 into main Sep 2, 2026
7 checks passed
@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) deleted the chore/skill-reference-urls branch September 2, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants