Skip to content

chore: remove superpowers analysis spec#58

Merged
josealekhine merged 1 commit intomainfrom
bug_fix
Apr 6, 2026
Merged

chore: remove superpowers analysis spec#58
josealekhine merged 1 commit intomainfrom
bug_fix

Conversation

@parlakisik
Copy link
Copy Markdown
Contributor

ctx must be self-contained — no external plugin libraries.

Deleted specs/superpowers-analysis.md.

All equivalent capabilities already exist natively in ctx.

ctx must be self-contained — no external plugin libraries.
Deleted specs/superpowers-analysis.md. All equivalent capabilities
already exist natively in ctx.

Signed-off-by: Murat Parlakisik <parlakisik@gmail.com>
@parlakisik parlakisik requested a review from josealekhine as a code owner April 6, 2026 17:55
@josealekhine
Copy link
Copy Markdown
Member

The PR's assessment is partially right but the approach is wrong.

  1. Correct: The superpowers plugin is external, and ctx already has native equivalents for every capability it provides. The spec itself acknowledges this —
    it's a comparison document, not a dependency.

  2. Incorrect framing: The spec isn't a dependency on superpowers — it was deliberately created in e6d8088 as an analysis document when the docs/superpowers/
    directory (which contained actual external skill files) was deleted. The spec captures what patterns were worth noting from the external project. It's areference, not a liability.

  3. The PR body says "All equivalent capabilities already exist natively in ctx" —
    which is true, but the spec already says exactly that. Deleting the analysis doesn't make ctx more self-contained; the external code was already removed.

I'm merging this nonetheless.

Will create a more refined spec later.

Thanks.

@josealekhine josealekhine merged commit 8026c4c into main Apr 6, 2026
12 checks passed
@josealekhine josealekhine deleted the bug_fix branch April 6, 2026 18:09
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