Skip to content

fix(pstack): renumber swarm's Phase A and fix its dangling table reference - #751

Open
mstephenholl wants to merge 1 commit into
cursor:mainfrom
mstephenholl:fix/swarm-step-numbering
Open

mstephenholl wants to merge 1 commit into
cursor:mainfrom
mstephenholl:fix/swarm-step-numbering

Conversation

@mstephenholl

@mstephenholl mstephenholl commented Oct 21, 2026 •

Copy link
Copy Markdown

Summary

This is a follow-up to #634, which cut Phase A item 3 and the Phase C result-table sentence from swarm/SKILL.md. It fixes the two references those cuts left behind.

  • Phase A numbering. The source list read 1, 2, 4, 5, so that Phase B's "the step 4 model" kept pointing at the model step. Rendered Markdown renumbers the list 1 to 4, so on GitHub the model step shows as 3 while Phase B says 4. The list now reads 1 to 4, and Phase B says "the step 3 model". Source and rendered view now agree.
  • Phase D's "the table". The Phase C sentence that introduced the result table is gone, so "the table" had nothing to refer to. Phase D now asks for "a result table".

Four lines change, in this one file.

Notes

  • No other file refers to swarm's step numbers. poteto-mode/playbooks/autopilot-full.md mentions a "Step 4", but that's its own step.
  • feat(pstack): support Claude Code as a harness #629 also edits the Phase A model step and keeps its number 4. Whichever PR merges second will need a one-line rebase.
  • No version change.

Test plan

  • git diff --check is clean.
  • scripts/validate-plugins.mjs checks manifests only, and this PR changes none.

Note

Low Risk
Documentation-only edits to the swarm skill; no runtime, security, or data-handling impact.

Overview
Follow-up doc fix in swarm/SKILL.md after Phase A item 3 was removed elsewhere: Phase A steps are renumbered 1–4 (was 1, 2, 4, 5) so source numbering matches rendered lists on GitHub.

Phase B now refers to “the step 3 model” instead of step 4, aligned with the model-pick step. Phase D asks for “a result table” instead of “the table”, since the Phase C intro to that table was removed.

Reviewed by Cursor Bugbot for commit b029880. Bugbot is set up for automated code reviews on this repo. Configure here.

…rence

cursor#634 cut Phase A item 3 and left the list numbered 1, 2, 4, 5 so that
Phase B's "the step 4 model" kept pointing at the model step. Rendered
Markdown renumbers the list, so readers see the model step as 3 while
Phase B says 4. Number the list 1 to 4 and point Phase B at step 3.

cursor#634 also cut the Phase C sentence that introduced the result table,
which left Phase D's "the table" with nothing to refer to. Phase D now
asks for "a result table".
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.

1 participant