feat(skills): add discovery, security, browser QA, and compound layers - #38
Conversation
Refresh Superpowers, Matt Pocock, and gstack. Point pstack at cursor/plugins instead of the Claude Code port. Add shadcn/improve, Cursor Team Kit, Vercel agent-skills, and Addy Osmani's pack. The installer now copies skill scripts and reuses a clone when two packs share a repo. docs/agent-skill-packs.md records why these packs landed and which first-party trees were skipped. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…ss, and engines Advisor-only output from shadcn/improve against 068fec2. No application code changed. Executors should start at plans/README.md. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b3a0a44da3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| (cd "$dir" && find . -type f \ | ||
| -not -path '*/.git/*' \ | ||
| -not -name '*.zip' \ | ||
| \( -name '*.md' -o -path './scripts/*' -o -path './references/*' \) \ |
There was a problem hiding this comment.
Preserve runtime dependencies alongside skill scripts
When an installed skill invokes its bundled scripts, this allowlist omits required sibling runtime files: vercel-optimize/scripts/collect-signals.mjs imports ../lib/*.mjs, but no lib/ directory is vendored, so it exits immediately with ERR_MODULE_NOT_FOUND. The same filter drops styles.css, renderer.js, and template.html, which the new pr-review-canvas skill explicitly requires. Copy each skill's required runtime tree or add pack-specific assets so these indexed skills are usable.
AGENTS.md reference: AGENTS.md:L31-L32
Useful? React with 👍 / 👎.
|
|
||
| | Skill | Description | | ||
| | --- | --- | | ||
| | `vercel-composition-patterns` | | |
There was a problem hiding this comment.
Parse multiline descriptions before publishing the skill index
The generated index now publishes empty descriptions for vercel-composition-patterns and vercel-react-native-skills because their valid YAML descriptions continue on indented lines; Make Bot UI similarly renders as the literal >-. In those cases the promised trigger catalog cannot tell agents when to select the skill. Parse YAML block scalars/multiline values and regenerate the index.
AGENTS.md reference: AGENTS.md:L18-L19
Useful? React with 👍 / 👎.
Record GitHub Issues for praxstack/ai-visual-code-review (PRs-as-request-surface off), default triage labels, and single-context CONTEXT.md + docs/adr/. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Write always-applied .cursor/rules/pstack-models.mdc using detected Task slugs. Map unavailable skill-example names onto the closest available models and keep a four-model cross-family panel. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
…ted subsets Extend the clone+copy installer with allowlists so Cloud Agents get files in git. Vendor trailofbits (on demand), vercel find-skills, agent-browser, Compound Engineering, an Anthropic engineering subset, and a GitHub issue/PR/gh slice of awesome-copilot. Skip spec-kit and whole-set cloud cartridges. Document the one-methodology pipeline. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
…packs Allow empty allowlists to mean the whole SKILL.md tree, pin spec-kit to v1.0.1, record the Microsoft engineering subset, and document AWS, Cloudflare, and Supabase scan roots. Tests now expect the full toolbox and vendored Spec Kit command skills. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Convert github/spec-kit@v1.0.1 constitution/specify/plan/tasks/implement (and the rest of the command set) into SKILL.md trees without running specify init in this Express app. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Replace the 15-skill GitHub-only subset with the complete slimmed skills/ tree so agents can treat the pack as an on-demand shelf. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
…acks Install the documented Microsoft engineering/docs allowlist plus the AWS skills/ tree, Cloudflare skills, and Supabase agent-skills. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Reject non owner/name clone targets and keep git clone stderr so a failed refresh is diagnosable. Tests lock the spec-kit pin and the full awesome-copilot allowlist. Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
📋 Description
Vendors the missing layers of an agent engineering OS — discover → spec → implement → review → security → browser QA → ship → learn — without dumping 400+ unrelated
SKILL.mdfiles into always-on context.Also scaffolds Matt Pocock’s per-repo config and a Cursor-native pstack model map.
🔗 Related Issue
Follow-up on the skill-pack work in this PR. No separate issue.
🔄 Type of Change
Pipeline (pick one methodology per stage)
Do not run gstack, Superpowers, pstack, and Compound Engineering as four simultaneous methodologies on the same task.
What landed
Repo config
docs/agents/issue-tracker.md— GitHub Issues forpraxstack/ai-visual-code-review; PRs-as-request-surface offdocs/agents/triage-labels.md— default labels (needs-triage,needs-info,ready-for-agent,ready-for-human,wontfix)docs/agents/domain.md— single-context (CONTEXT.md+docs/adr/).cursor/rules/pstack-models.mdc—alwaysApply: true, Cursor Task slugs only (no unconfirmed*-max/grok-4.6-fast-xhighnames)scripts/link-agent-skills.shcopies that map to~/.cursor/rules/on Cloud Agent bootNew vendored packs (slimmed
SKILL.md+ references + scripts)scripts/install-agent-browser.shfrontend-design,webapp-testing,mcp-builder,skill-creator,claude-apighsubset onlyExisting packs remain: Superpowers, Matt Pocock, gstack, official Cursor pstack, improve, Cursor Team Kit, Vercel agent-skills, Addy Osmani.
Catalog: 344 skills in
docs/agent-skills.md.The installer now supports an optional allowlist field (
pack|repo|subdir|license|skill,dir,names).Intentionally not vendored whole-set
SKILL.mdtree for this app. Usespecify init --here --integration cursor-agent(install Specify from the Git tag, never a random PyPIspecify-cli).find-skillsis how agents pull more later.gstack’s Cursor
./setup --host cursorinstaller still disagrees with the README on some releases (issue 2361); this repo keeps pstack native for Cursor and gstack as a portable skill tree.✅ Checklist
📝 Additional Notes
App runtime is unchanged.
plans/from the earlier improve audit is still executor handoff, not implemented here.Verification
npm test: 5 suites, 77 tests (pack presence, allowlists, spec-kit not vendored, pstack slug gate, Matt Pocock docs)npm run lint: 0 errors, 31 pre-existing warningsbash -non the three install/link scriptsagent-browserCLI present; Chrome 152 already installed on this VM