Skip to content

feat(workflows): add show-me skill and shape-first create-spec - #2540

Merged
lavaman131 merged 5 commits into
mainfrom
feat/show-me-create-spec
Aug 19, 2026
Merged

feat(workflows): add show-me skill and shape-first create-spec#2540
lavaman131 merged 5 commits into
mainfrom
feat/show-me-create-spec

Conversation

@flora131

@flora131 flora131 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

Bundle HumanLayer's show-me skill as a builtin and teach create-spec to use the same visual formats plus Path A / Path B planning from Dillon Mulroy's tech-spec skill.

Changes

  • Added /skill:show-me at packages/workflows/skills/show-me/ with the upstream MIT license and HumanLayer credit. Claude Code Bash(open …) is replaced by Atomic bash plus open / xdg-open / print-the-path.
  • Documented the skill in docs/skills.md, the quickstart top-skills table, /atomic guide, and the root README. Discovery test now expects show-me.
  • Revised create-spec without changing its numbered RFC headings: Path A (convert context) vs Path B (grill first), starting with load-standards-and-local-context; show-me-style pseudocode, call trees, component/file trees, Mermaid, diffs, and optional focused HTML.
  • Moved the show-me docs paragraph out of the prompt-engineer section so prefilling/CoT guidance still refers to /skill:prompt-engineer.

Notes

  • Credits: humanlayer/skills (MIT) and dmmulroy/tech-spec.
  • Worktree: /Users/tonystark/Documents/projects/atomic-show-me.
  • Checks run locally: npm run check, npm run test:unit, npm run test:integration, npm run test:ci-contracts.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Greptile Summary

This change bundles the show-me workflow guidance and extends specification-planning documentation. The reported artifact-path expansion issue was disproved: an executed shell reproduction exercised both open and xdg-open branches with shell-special description values, and each received one unchanged literal pathname.

Confidence Score: 5/5

No blocking failure remains.

The reported artifact-path expansion behavior was disproved by direct execution of both documented platform branches.

T-Rex T-Rex Logs

What T-Rex did

  • Authored reproduction script show-me-pathname-01-before.sh that mocks both openers and runs the documented construction against hostile description values.
  • Ran the reproduction script and reviewed the observed execution output in show-me-pathname-02-after.log, which shows the command, working directory, exit code 0, and argv; the result confirms all eight calls received one unchanged literal pathname.
  • Concluded that the literal {description} is ordinary text in Bash and cannot be substituted or re-evaluated, so no argv manipulation can occur via a separate description value.
  • The validation work confirmed the test directly captures the argv passed by each platform branch across all eight invocations.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (2): Last reviewed commit: "fix(workflows): quote show-me HTML artif..." | Re-trigger Greptile

Keep the prefilling/CoT guidance attached to /skill:prompt-engineer.
Give /skill:show-me its own subsection so the two skills are not one paragraph.

Assistant-model: Grok 4.6
@mintlify

mintlify Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
bastani 🟢 Ready View Preview Aug 19, 2026, 6:44 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Use the Agent Skills license field (and metadata) instead of dangling
credit paragraphs in the skill body.

Assistant-model: Grok 4.6
Comment thread packages/workflows/skills/show-me/SKILL.md Outdated
Keep open, xdg-open, and printf treating a spaced description as one path.

Assistant-model: Grok 4.6
@lavaman131
lavaman131 enabled auto-merge (squash) August 19, 2026 18:56
@lavaman131
lavaman131 disabled auto-merge August 19, 2026 18:56
@lavaman131
lavaman131 merged commit 3646705 into main Aug 19, 2026
16 checks passed
@lavaman131
lavaman131 deleted the feat/show-me-create-spec branch August 19, 2026 18:56
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