Skip to content

docs: clarify spec skill bootstrap commands#11974

Merged
captainsafia merged 1 commit into
masterfrom
oz-agent/clarify-spec-skill-bootstrap
Jun 1, 2026
Merged

docs: clarify spec skill bootstrap commands#11974
captainsafia merged 1 commit into
masterfrom
oz-agent/clarify-spec-skill-bootstrap

Conversation

@captainsafia
Copy link
Copy Markdown
Contributor

Description

Clarifies the spec PR guidance in CONTRIBUTING.md so contributors know:

  • /write-product-spec and /write-tech-spec are sourced from warpdotdev/common-skills, not authored directly in warpdotdev/warp
  • skills-lock.json pins the expected common-skill versions for this checkout
  • ./script/bootstrap can restore the pinned common skills, with project-local and global install options

I confirmed the documented commands against script/bootstrap, script/windows/bootstrap.ps1, script/resolve_common_skills, skills-lock.json, WARP.md, and the relevant common-skills installation history.

Linked Issue

N/A — requested from Slack.

  • The linked issue is labeled ready-to-spec or ready-to-implement.
  • Where appropriate, screenshots or a short video of the implementation are included below (especially for user-visible or UI changes).

Testing

  • git --no-pager diff --check

  • I have manually tested my changes locally with ./script/run

Screenshots / Videos

N/A — documentation-only change.

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

CHANGELOG-NONE

Co-Authored-By: Oz oz-agent@warp.dev

Conversation: https://staging.warp.dev/conversation/a6331e13-9a95-4e90-ae7b-94a1f48b4b08
Run: https://oz.staging.warp.dev/runs/019e7f63-cc19-77de-a6af-db85694b4888
This PR was generated with Oz.

Document that product and tech spec skills are sourced from common-skills and list the bootstrap commands for installing the pinned skills.

Co-Authored-By: Oz <oz-agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label May 31, 2026
@captainsafia captainsafia marked this pull request as ready for review May 31, 2026 19:18
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 31, 2026

@captainsafia

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This documentation-only PR clarifies where the spec-writing skills come from, how skills-lock.json pins the expected common skills, and which ./script/bootstrap commands restore them. I checked the changed guidance against the local bootstrap scripts and lockfile context.

Concerns

  • No blocking concerns found.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@captainsafia
Copy link
Copy Markdown
Contributor Author

@vkodithala Mind giving this one a quick stamp? We need to update our contributing docs now that we've moved things to common-skills.

Copy link
Copy Markdown
Contributor

@vkodithala vkodithala left a comment

Choose a reason for hiding this comment

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

(Disregard this comment I was being an idiot)

Copy link
Copy Markdown
Contributor

@vkodithala vkodithala left a comment

Choose a reason for hiding this comment

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

🚀 THANK YOU

@captainsafia captainsafia merged commit 51ce7a4 into master Jun 1, 2026
44 checks passed
@captainsafia captainsafia deleted the oz-agent/clarify-spec-skill-bootstrap branch June 1, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants