mirror of
https://github.com/obra/superpowers.git
synced 2026-06-12 13:49:05 +08:00
Compare commits
6 Commits
harness-an
...
f/cross-pl
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
2f5c75d91c | ||
|
|
211618a254 | ||
|
|
72b5f19b60 | ||
|
|
f1db631021 | ||
|
|
2d4447b1e0 | ||
|
|
72b59619ad |
@@ -21,7 +21,6 @@
|
|||||||
"workflow"
|
"workflow"
|
||||||
],
|
],
|
||||||
"skills": "./skills/",
|
"skills": "./skills/",
|
||||||
"hooks": "./hooks/hooks-codex.json",
|
|
||||||
"interface": {
|
"interface": {
|
||||||
"displayName": "Superpowers",
|
"displayName": "Superpowers",
|
||||||
"shortDescription": "Planning, TDD, debugging, and delivery workflows for coding agents",
|
"shortDescription": "Planning, TDD, debugging, and delivery workflows for coding agents",
|
||||||
|
|||||||
@@ -1,121 +0,0 @@
|
|||||||
import { readFileSync } from "node:fs";
|
|
||||||
import { dirname, resolve } from "node:path";
|
|
||||||
import { fileURLToPath } from "node:url";
|
|
||||||
import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
|
|
||||||
|
|
||||||
const EXTREMELY_IMPORTANT_MARKER = "<EXTREMELY_IMPORTANT>";
|
|
||||||
const BOOTSTRAP_MARKER = "superpowers:using-superpowers bootstrap for pi";
|
|
||||||
|
|
||||||
const extensionDir = dirname(fileURLToPath(import.meta.url));
|
|
||||||
const packageRoot = resolve(extensionDir, "../..");
|
|
||||||
const skillsDir = resolve(packageRoot, "skills");
|
|
||||||
const bootstrapSkillPath = resolve(skillsDir, "using-superpowers", "SKILL.md");
|
|
||||||
|
|
||||||
let cachedBootstrap: string | null | undefined;
|
|
||||||
|
|
||||||
export default function superpowersPiExtension(pi: ExtensionAPI) {
|
|
||||||
let injectBootstrap = true;
|
|
||||||
|
|
||||||
pi.on("resources_discover", async () => ({
|
|
||||||
skillPaths: [skillsDir],
|
|
||||||
}));
|
|
||||||
|
|
||||||
pi.on("session_start", async () => {
|
|
||||||
injectBootstrap = true;
|
|
||||||
});
|
|
||||||
|
|
||||||
pi.on("session_compact", async () => {
|
|
||||||
injectBootstrap = true;
|
|
||||||
});
|
|
||||||
|
|
||||||
pi.on("agent_end", async () => {
|
|
||||||
injectBootstrap = false;
|
|
||||||
});
|
|
||||||
|
|
||||||
pi.on("context", async (event) => {
|
|
||||||
if (!injectBootstrap) return;
|
|
||||||
if (event.messages.some(messageContainsBootstrap)) return;
|
|
||||||
|
|
||||||
const bootstrap = getBootstrapContent();
|
|
||||||
if (!bootstrap) return;
|
|
||||||
|
|
||||||
const bootstrapMessage = {
|
|
||||||
role: "user" as const,
|
|
||||||
content: [{ type: "text" as const, text: bootstrap }],
|
|
||||||
timestamp: Date.now(),
|
|
||||||
};
|
|
||||||
|
|
||||||
const insertAt = firstNonCompactionSummaryIndex(event.messages);
|
|
||||||
return {
|
|
||||||
messages: [
|
|
||||||
...event.messages.slice(0, insertAt),
|
|
||||||
bootstrapMessage,
|
|
||||||
...event.messages.slice(insertAt),
|
|
||||||
],
|
|
||||||
};
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
function getBootstrapContent(): string | null {
|
|
||||||
if (cachedBootstrap !== undefined) return cachedBootstrap;
|
|
||||||
|
|
||||||
try {
|
|
||||||
const skillContent = readFileSync(bootstrapSkillPath, "utf8");
|
|
||||||
const body = stripFrontmatter(skillContent);
|
|
||||||
cachedBootstrap = `${EXTREMELY_IMPORTANT_MARKER}
|
|
||||||
${BOOTSTRAP_MARKER}
|
|
||||||
|
|
||||||
You have superpowers.
|
|
||||||
|
|
||||||
The using-superpowers skill content is included below and is already loaded for this Pi session. Follow it now. Do not try to load using-superpowers again.
|
|
||||||
|
|
||||||
${body}
|
|
||||||
|
|
||||||
${piToolMapping()}
|
|
||||||
</EXTREMELY_IMPORTANT>`;
|
|
||||||
return cachedBootstrap;
|
|
||||||
} catch {
|
|
||||||
cachedBootstrap = null;
|
|
||||||
return null;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
function stripFrontmatter(content: string): string {
|
|
||||||
const match = content.match(/^---\n[\s\S]*?\n---\n([\s\S]*)$/);
|
|
||||||
return (match ? match[1] : content).trim();
|
|
||||||
}
|
|
||||||
|
|
||||||
function piToolMapping(): string {
|
|
||||||
return `## Pi tool mapping
|
|
||||||
|
|
||||||
Pi has native skills but does not expose Claude Code's \`Skill\` tool. When a Superpowers instruction says to invoke a skill, use Pi's native skill system instead: load the relevant \`SKILL.md\` with \`read\` when the skill applies, or let a human invoke \`/skill:name\` explicitly.
|
|
||||||
|
|
||||||
Pi's built-in coding tools are lowercase: \`read\`, \`write\`, \`edit\`, \`bash\`, plus optional \`grep\`, \`find\`, and \`ls\`. Use those for the corresponding actions: read a file, create or edit files, run shell commands, search file contents, find files by name, and list directories.
|
|
||||||
|
|
||||||
Pi does not ship a standard subagent tool. If a subagent tool such as \`subagent\` from \`pi-subagents\` is available, use it for Superpowers subagent workflows. If no subagent tool is available, do the work in this session or explain the missing capability instead of inventing \`Task\` calls.
|
|
||||||
|
|
||||||
Pi does not ship a standard task-list tool. If an installed todo/task tool is available, use it. Otherwise track work in plan files or a repo-local \`TODO.md\` when task tracking is needed. Treat older \`TodoWrite\` references as this task-tracking action.`;
|
|
||||||
}
|
|
||||||
|
|
||||||
function messageContainsBootstrap(message: unknown): boolean {
|
|
||||||
const content = (message as { content?: unknown }).content;
|
|
||||||
if (typeof content === "string") return content.includes(BOOTSTRAP_MARKER);
|
|
||||||
if (!Array.isArray(content)) return false;
|
|
||||||
return content.some((part) => {
|
|
||||||
return (
|
|
||||||
part &&
|
|
||||||
typeof part === "object" &&
|
|
||||||
(part as { type?: unknown }).type === "text" &&
|
|
||||||
typeof (part as { text?: unknown }).text === "string" &&
|
|
||||||
(part as { text: string }).text.includes(BOOTSTRAP_MARKER)
|
|
||||||
);
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
function firstNonCompactionSummaryIndex(messages: unknown[]): number {
|
|
||||||
let index = 0;
|
|
||||||
while ((messages[index] as { role?: unknown } | undefined)?.role === "compactionSummary") {
|
|
||||||
index += 1;
|
|
||||||
}
|
|
||||||
return index;
|
|
||||||
}
|
|
||||||
29
README.md
29
README.md
@@ -4,7 +4,7 @@ Superpowers is a complete software development methodology for your coding agent
|
|||||||
|
|
||||||
## Quickstart
|
## Quickstart
|
||||||
|
|
||||||
Give your agent Superpowers: [Claude Code](#claude-code), [Antigravity](#antigravity), [Codex App](#codex-app), [Codex CLI](#codex-cli), [Cursor](#cursor), [Factory Droid](#factory-droid), [Gemini CLI](#gemini-cli), [GitHub Copilot CLI](#github-copilot-cli), [OpenCode](#opencode), [Pi](#pi).
|
Give your agent Superpowers: [Claude Code](#claude-code), [Codex App](#codex-app), [Codex CLI](#codex-cli), [Cursor](#cursor), [Factory Droid](#factory-droid), [Gemini CLI](#gemini-cli), [GitHub Copilot CLI](#github-copilot-cli), [OpenCode](#opencode).
|
||||||
|
|
||||||
## How it works
|
## How it works
|
||||||
|
|
||||||
@@ -60,17 +60,6 @@ The Superpowers marketplace provides Superpowers and some other related plugins
|
|||||||
/plugin install superpowers@superpowers-marketplace
|
/plugin install superpowers@superpowers-marketplace
|
||||||
```
|
```
|
||||||
|
|
||||||
### Antigravity
|
|
||||||
|
|
||||||
Install Superpowers as a plugin from this repository:
|
|
||||||
|
|
||||||
```bash
|
|
||||||
agy plugin install https://github.com/obra/superpowers
|
|
||||||
```
|
|
||||||
|
|
||||||
Antigravity runs the plugin's session-start hook, so Superpowers is active from
|
|
||||||
the first message. Reinstall with the same command to update.
|
|
||||||
|
|
||||||
### Codex App
|
### Codex App
|
||||||
|
|
||||||
Superpowers is available via the [official Codex plugin marketplace](https://github.com/openai/plugins).
|
Superpowers is available via the [official Codex plugin marketplace](https://github.com/openai/plugins).
|
||||||
@@ -162,22 +151,6 @@ already use it in another harness.
|
|||||||
|
|
||||||
- Detailed docs: [docs/README.opencode.md](docs/README.opencode.md)
|
- Detailed docs: [docs/README.opencode.md](docs/README.opencode.md)
|
||||||
|
|
||||||
### Pi
|
|
||||||
|
|
||||||
Install Superpowers as a Pi package from this repository:
|
|
||||||
|
|
||||||
```bash
|
|
||||||
pi install git:github.com/obra/superpowers
|
|
||||||
```
|
|
||||||
|
|
||||||
For local development, run Pi with this checkout loaded as a temporary package:
|
|
||||||
|
|
||||||
```bash
|
|
||||||
pi -e /path/to/superpowers
|
|
||||||
```
|
|
||||||
|
|
||||||
The Pi package loads the Superpowers skills and a small extension that injects the `using-superpowers` bootstrap at session startup and again after compaction. Pi has native skills, so no compatibility `Skill` tool is required. Subagent and task-list tools remain optional Pi companion packages.
|
|
||||||
|
|
||||||
## The Basic Workflow
|
## The Basic Workflow
|
||||||
|
|
||||||
1. **brainstorming** - Activates before writing code. Refines rough ideas through questions, explores alternatives, presents design in sections for validation. Saves design document.
|
1. **brainstorming** - Activates before writing code. Refines rough ideas through questions, explores alternatives, presents design in sections for validation. Saves design document.
|
||||||
|
|||||||
@@ -1,143 +0,0 @@
|
|||||||
# Pi Extension and Evals Implementation Plan
|
|
||||||
|
|
||||||
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
|
||||||
|
|
||||||
**Goal:** Add first-class Pi package support for Superpowers and add Pi as a Drill eval backend.
|
|
||||||
|
|
||||||
**Architecture:** The Pi package is declared in the root `package.json` and loads existing `skills/` plus a small Pi extension. The extension injects the `using-superpowers` bootstrap into provider context as a user-role message on session startup and after compaction, with Pi-specific tool mapping. Drill gains a `pi` backend, Pi session-log normalization, and tests.
|
|
||||||
|
|
||||||
**Tech Stack:** Pi TypeScript extension API, Node built-in test runner, Drill Python eval harness, pytest.
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
### Task 1: Pi package manifest and extension tests
|
|
||||||
|
|
||||||
**Files:**
|
|
||||||
- Modify: `package.json`
|
|
||||||
- Create: `tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
- [ ] **Step 1: Write failing package/extension tests**
|
|
||||||
|
|
||||||
Create `tests/pi/test-pi-extension.mjs` with tests that import `extensions/superpowers.ts`, register fake Pi handlers, and assert:
|
|
||||||
- root `package.json` has `keywords` containing `pi-package`
|
|
||||||
- root `package.json` has `pi.skills: ["./skills"]`
|
|
||||||
- root `package.json` has `pi.extensions: ["./extensions/superpowers.ts"]`
|
|
||||||
- the extension registers `resources_discover`, `session_start`, `session_compact`, `context`, and `agent_end`
|
|
||||||
- startup `context` injects exactly one user-role bootstrap message
|
|
||||||
- `agent_end` clears startup injection
|
|
||||||
- `session_compact` re-enables injection
|
|
||||||
- the extension does not register `session_before_compact`
|
|
||||||
|
|
||||||
- [ ] **Step 2: Run tests and verify RED**
|
|
||||||
|
|
||||||
Run: `node --experimental-strip-types --test tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
Expected: FAIL because `extensions/superpowers.ts` does not exist and `package.json` lacks the `pi` manifest.
|
|
||||||
|
|
||||||
- [ ] **Step 3: Implement manifest fields**
|
|
||||||
|
|
||||||
Update `package.json` with `description`, `keywords`, `pi.extensions`, and `pi.skills` while preserving existing `name`, `version`, `type`, and `main`.
|
|
||||||
|
|
||||||
- [ ] **Step 4: Implement `extensions/superpowers.ts`**
|
|
||||||
|
|
||||||
Create a zero-runtime-dependency extension that:
|
|
||||||
- locates the package root from `import.meta.url`
|
|
||||||
- reads `skills/using-superpowers/SKILL.md`
|
|
||||||
- strips YAML frontmatter
|
|
||||||
- appends Pi-specific tool mapping
|
|
||||||
- exposes `resources_discover` with the skills path
|
|
||||||
- marks bootstrap pending on `session_start` and `session_compact`
|
|
||||||
- injects a user-role bootstrap message in `context`
|
|
||||||
- inserts post-compact bootstrap after leading `compactionSummary` messages
|
|
||||||
- clears pending bootstrap on `agent_end`
|
|
||||||
|
|
||||||
- [ ] **Step 5: Run tests and verify GREEN**
|
|
||||||
|
|
||||||
Run: `node --experimental-strip-types --test tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
Expected: PASS.
|
|
||||||
|
|
||||||
### Task 2: Pi tool mapping reference
|
|
||||||
|
|
||||||
**Files:**
|
|
||||||
- Create: `skills/using-superpowers/references/pi-tools.md`
|
|
||||||
- Modify: `tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
- [ ] **Step 1: Write failing test for Pi reference doc**
|
|
||||||
|
|
||||||
Add assertions that `skills/using-superpowers/references/pi-tools.md` exists and documents mappings for `Skill`, `Task`, `TodoWrite`, and built-in tool names.
|
|
||||||
|
|
||||||
- [ ] **Step 2: Run tests and verify RED**
|
|
||||||
|
|
||||||
Run: `node --experimental-strip-types --test tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
Expected: FAIL because `pi-tools.md` does not exist.
|
|
||||||
|
|
||||||
- [ ] **Step 3: Add Pi reference doc**
|
|
||||||
|
|
||||||
Create `skills/using-superpowers/references/pi-tools.md` explaining Pi-native skills, optional `pi-subagents`, no canonical todo/tasklist plugin, and built-in lowercase tools.
|
|
||||||
|
|
||||||
- [ ] **Step 4: Run tests and verify GREEN**
|
|
||||||
|
|
||||||
Run: `node --experimental-strip-types --test tests/pi/test-pi-extension.mjs`
|
|
||||||
|
|
||||||
Expected: PASS.
|
|
||||||
|
|
||||||
### Task 3: Drill Pi backend and session log normalization
|
|
||||||
|
|
||||||
**Files:**
|
|
||||||
- Create: `evals/backends/pi.yaml`
|
|
||||||
- Modify: `evals/drill/backend.py`
|
|
||||||
- Modify: `evals/drill/engine.py`
|
|
||||||
- Modify: `evals/drill/normalizer.py`
|
|
||||||
- Modify: `evals/tests/test_backend.py`
|
|
||||||
- Modify: `evals/tests/test_normalizer.py`
|
|
||||||
|
|
||||||
- [ ] **Step 1: Write failing backend/normalizer tests**
|
|
||||||
|
|
||||||
Add pytest coverage for:
|
|
||||||
- `load_backend("pi")` returns `family == "pi"`
|
|
||||||
- Pi backend command starts with `pi` and includes `-e ${SUPERPOWERS_ROOT}`
|
|
||||||
- `_resolve_log_dir()` for Pi points under `~/.pi/agent/sessions`
|
|
||||||
- `filter_pi_logs_by_cwd()` keeps only session files whose header `cwd` matches the scenario workdir
|
|
||||||
- `normalize_pi_logs()` extracts `toolCall` blocks from Pi assistant session entries and maps built-in lowercase tools to canonical names
|
|
||||||
|
|
||||||
- [ ] **Step 2: Run tests and verify RED**
|
|
||||||
|
|
||||||
Run: `uv run pytest evals/tests/test_backend.py evals/tests/test_normalizer.py -q`
|
|
||||||
|
|
||||||
Expected: FAIL because the Pi backend and normalizer do not exist.
|
|
||||||
|
|
||||||
- [ ] **Step 3: Add `evals/backends/pi.yaml`**
|
|
||||||
|
|
||||||
Configure the backend to run `pi -e ${SUPERPOWERS_ROOT}`, use permissive TUI readiness, `/quit` shutdown, and Pi session log location.
|
|
||||||
|
|
||||||
- [ ] **Step 4: Implement Pi family support**
|
|
||||||
|
|
||||||
Update `Backend.family`, `Engine._resolve_log_dir`, `Engine._collect_tool_calls`, and `normalizer.py` with Pi log filtering and normalizing.
|
|
||||||
|
|
||||||
- [ ] **Step 5: Run tests and verify GREEN**
|
|
||||||
|
|
||||||
Run: `uv run pytest evals/tests/test_backend.py evals/tests/test_normalizer.py -q`
|
|
||||||
|
|
||||||
Expected: PASS.
|
|
||||||
|
|
||||||
### Task 4: Documentation and full verification
|
|
||||||
|
|
||||||
**Files:**
|
|
||||||
- Modify: `README.md`
|
|
||||||
- Modify: `evals/README.md`
|
|
||||||
|
|
||||||
- [ ] **Step 1: Document Pi install and eval backend**
|
|
||||||
|
|
||||||
Add Pi to README quickstart/install list and add backend entry/usage to `evals/README.md`.
|
|
||||||
|
|
||||||
- [ ] **Step 2: Run verification**
|
|
||||||
|
|
||||||
Run:
|
|
||||||
```bash
|
|
||||||
node --experimental-strip-types --test tests/pi/test-pi-extension.mjs
|
|
||||||
uv run pytest evals/tests/test_backend.py evals/tests/test_setup.py evals/tests/test_normalizer.py -q
|
|
||||||
```
|
|
||||||
|
|
||||||
Expected: all tests pass.
|
|
||||||
2
evals
2
evals
Submodule evals updated: e2b37138c8...f7ac1941d5
@@ -1,16 +0,0 @@
|
|||||||
{
|
|
||||||
"hooks": {
|
|
||||||
"SessionStart": [
|
|
||||||
{
|
|
||||||
"matcher": "startup|resume|clear",
|
|
||||||
"hooks": [
|
|
||||||
{
|
|
||||||
"type": "command",
|
|
||||||
"command": "\"${PLUGIN_ROOT}/hooks/run-hook.cmd\" session-start-codex",
|
|
||||||
"async": false
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -7,6 +7,13 @@ set -euo pipefail
|
|||||||
SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
||||||
PLUGIN_ROOT="$(cd "${SCRIPT_DIR}/.." && pwd)"
|
PLUGIN_ROOT="$(cd "${SCRIPT_DIR}/.." && pwd)"
|
||||||
|
|
||||||
|
# Check if legacy skills directory exists and build warning
|
||||||
|
warning_message=""
|
||||||
|
legacy_skills_dir="${HOME}/.config/superpowers/skills"
|
||||||
|
if [ -d "$legacy_skills_dir" ]; then
|
||||||
|
warning_message="\n\n<important-reminder>IN YOUR FIRST REPLY AFTER SEEING THIS MESSAGE YOU MUST TELL THE USER:⚠️ **WARNING:** Superpowers now uses Claude Code's skills system. Custom skills in ~/.config/superpowers/skills will not be read. Move custom skills to ~/.claude/skills instead. To make this message go away, remove ~/.config/superpowers/skills</important-reminder>"
|
||||||
|
fi
|
||||||
|
|
||||||
# Read using-superpowers content
|
# Read using-superpowers content
|
||||||
using_superpowers_content=$(cat "${PLUGIN_ROOT}/skills/using-superpowers/SKILL.md" 2>&1 || echo "Error reading using-superpowers skill")
|
using_superpowers_content=$(cat "${PLUGIN_ROOT}/skills/using-superpowers/SKILL.md" 2>&1 || echo "Error reading using-superpowers skill")
|
||||||
|
|
||||||
@@ -24,7 +31,8 @@ escape_for_json() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
using_superpowers_escaped=$(escape_for_json "$using_superpowers_content")
|
using_superpowers_escaped=$(escape_for_json "$using_superpowers_content")
|
||||||
session_context="<EXTREMELY_IMPORTANT>\nYou have superpowers.\n\n**Below is the full content of your 'superpowers:using-superpowers' skill - your introduction to using skills. For all other skills, use the 'Skill' tool:**\n\n${using_superpowers_escaped}\n</EXTREMELY_IMPORTANT>"
|
warning_escaped=$(escape_for_json "$warning_message")
|
||||||
|
session_context="<EXTREMELY_IMPORTANT>\nYou have superpowers.\n\n**Below is the full content of your 'superpowers:using-superpowers' skill - your introduction to using skills. For all other skills, use the 'Skill' tool:**\n\n${using_superpowers_escaped}\n\n${warning_escaped}\n</EXTREMELY_IMPORTANT>"
|
||||||
|
|
||||||
# Output context injection as JSON.
|
# Output context injection as JSON.
|
||||||
# Cursor hooks expect additional_context (snake_case).
|
# Cursor hooks expect additional_context (snake_case).
|
||||||
@@ -37,13 +45,13 @@ session_context="<EXTREMELY_IMPORTANT>\nYou have superpowers.\n\n**Below is the
|
|||||||
# See: https://github.com/obra/superpowers/issues/571
|
# See: https://github.com/obra/superpowers/issues/571
|
||||||
if [ -n "${CURSOR_PLUGIN_ROOT:-}" ]; then
|
if [ -n "${CURSOR_PLUGIN_ROOT:-}" ]; then
|
||||||
# Cursor sets CURSOR_PLUGIN_ROOT (may also set CLAUDE_PLUGIN_ROOT)
|
# Cursor sets CURSOR_PLUGIN_ROOT (may also set CLAUDE_PLUGIN_ROOT)
|
||||||
printf '{\n "additional_context": "%s"\n}\n' "$session_context" | cat
|
printf '{\n "additional_context": "%s"\n}\n' "$session_context"
|
||||||
elif [ -n "${CLAUDE_PLUGIN_ROOT:-}" ] && [ -z "${COPILOT_CLI:-}" ]; then
|
elif [ -n "${CLAUDE_PLUGIN_ROOT:-}" ] && [ -z "${COPILOT_CLI:-}" ]; then
|
||||||
# Claude Code sets CLAUDE_PLUGIN_ROOT without COPILOT_CLI
|
# Claude Code sets CLAUDE_PLUGIN_ROOT without COPILOT_CLI
|
||||||
printf '{\n "hookSpecificOutput": {\n "hookEventName": "SessionStart",\n "additionalContext": "%s"\n }\n}\n' "$session_context" | cat
|
printf '{\n "hookSpecificOutput": {\n "hookEventName": "SessionStart",\n "additionalContext": "%s"\n }\n}\n' "$session_context"
|
||||||
else
|
else
|
||||||
# Copilot CLI (sets COPILOT_CLI=1) or unknown platform — SDK standard format
|
# Copilot CLI (sets COPILOT_CLI=1) or unknown platform — SDK standard format
|
||||||
printf '{\n "additionalContext": "%s"\n}\n' "$session_context" | cat
|
printf '{\n "additionalContext": "%s"\n}\n' "$session_context"
|
||||||
fi
|
fi
|
||||||
|
|
||||||
exit 0
|
exit 0
|
||||||
|
|||||||
@@ -1,26 +0,0 @@
|
|||||||
#!/usr/bin/env bash
|
|
||||||
# Codex SessionStart hook for superpowers plugin
|
|
||||||
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
|
||||||
PLUGIN_ROOT="$(cd "${SCRIPT_DIR}/.." && pwd)"
|
|
||||||
|
|
||||||
using_superpowers_content=$(cat "${PLUGIN_ROOT}/skills/using-superpowers/SKILL.md" 2>&1 || echo "Error reading using-superpowers skill")
|
|
||||||
|
|
||||||
escape_for_json() {
|
|
||||||
local s="$1"
|
|
||||||
s="${s//\\/\\\\}"
|
|
||||||
s="${s//\"/\\\"}"
|
|
||||||
s="${s//$'\n'/\\n}"
|
|
||||||
s="${s//$'\r'/\\r}"
|
|
||||||
s="${s//$'\t'/\\t}"
|
|
||||||
printf '%s' "$s"
|
|
||||||
}
|
|
||||||
|
|
||||||
using_superpowers_escaped=$(escape_for_json "$using_superpowers_content")
|
|
||||||
session_context="<EXTREMELY_IMPORTANT>\nYou have superpowers.\n\n**Below is the full content of your 'superpowers:using-superpowers' skill - your introduction to using skills. For all other skills, follow the Codex skill-loading instructions in that skill:**\n\n${using_superpowers_escaped}\n</EXTREMELY_IMPORTANT>"
|
|
||||||
|
|
||||||
printf '{\n "hookSpecificOutput": {\n "hookEventName": "SessionStart",\n "additionalContext": "%s"\n }\n}\n' "$session_context" | cat
|
|
||||||
|
|
||||||
exit 0
|
|
||||||
19
package.json
19
package.json
@@ -1,23 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "superpowers",
|
"name": "superpowers",
|
||||||
"version": "5.1.0",
|
"version": "5.1.0",
|
||||||
"description": "Superpowers skills and runtime bootstrap for coding agents",
|
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"main": ".opencode/plugins/superpowers.js",
|
"main": ".opencode/plugins/superpowers.js"
|
||||||
"keywords": [
|
|
||||||
"pi-package",
|
|
||||||
"skills",
|
|
||||||
"tdd",
|
|
||||||
"debugging",
|
|
||||||
"collaboration",
|
|
||||||
"workflow"
|
|
||||||
],
|
|
||||||
"pi": {
|
|
||||||
"extensions": [
|
|
||||||
"./.pi/extensions/superpowers.ts"
|
|
||||||
],
|
|
||||||
"skills": [
|
|
||||||
"./skills"
|
|
||||||
]
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -53,7 +53,6 @@ EXCLUDES=(
|
|||||||
"/.github/"
|
"/.github/"
|
||||||
"/.gitignore"
|
"/.gitignore"
|
||||||
"/.opencode/"
|
"/.opencode/"
|
||||||
"/.pi/"
|
|
||||||
"/.version-bump.json"
|
"/.version-bump.json"
|
||||||
"/.worktrees/"
|
"/.worktrees/"
|
||||||
".DS_Store"
|
".DS_Store"
|
||||||
@@ -71,6 +70,7 @@ EXCLUDES=(
|
|||||||
"/commands/"
|
"/commands/"
|
||||||
"/docs/"
|
"/docs/"
|
||||||
"/evals/"
|
"/evals/"
|
||||||
|
"/hooks/"
|
||||||
"/lib/"
|
"/lib/"
|
||||||
"/scripts/"
|
"/scripts/"
|
||||||
"/tests/"
|
"/tests/"
|
||||||
@@ -420,7 +420,7 @@ if [[ $BOOTSTRAP -eq 1 ]]; then
|
|||||||
COMMIT_TITLE="bootstrap superpowers v$UPSTREAM_VERSION from upstream main @ $UPSTREAM_SHORT"
|
COMMIT_TITLE="bootstrap superpowers v$UPSTREAM_VERSION from upstream main @ $UPSTREAM_SHORT"
|
||||||
PR_BODY="Initial bootstrap of the superpowers plugin from upstream \`main\` @ \`$UPSTREAM_SHORT\` (v$UPSTREAM_VERSION).
|
PR_BODY="Initial bootstrap of the superpowers plugin from upstream \`main\` @ \`$UPSTREAM_SHORT\` (v$UPSTREAM_VERSION).
|
||||||
|
|
||||||
Creates \`plugins/superpowers/\` by copying the tracked plugin files from upstream, including \`.codex-plugin/plugin.json\`, \`assets/\`, and \`hooks/\`.
|
Creates \`plugins/superpowers/\` by copying the tracked plugin files from upstream, including \`.codex-plugin/plugin.json\` and \`assets/\`.
|
||||||
|
|
||||||
Run via: \`scripts/sync-to-codex-plugin.sh --bootstrap\`
|
Run via: \`scripts/sync-to-codex-plugin.sh --bootstrap\`
|
||||||
Upstream commit: https://github.com/obra/superpowers/commit/$UPSTREAM_SHA
|
Upstream commit: https://github.com/obra/superpowers/commit/$UPSTREAM_SHA
|
||||||
@@ -430,7 +430,7 @@ else
|
|||||||
COMMIT_TITLE="sync superpowers v$UPSTREAM_VERSION from upstream main @ $UPSTREAM_SHORT"
|
COMMIT_TITLE="sync superpowers v$UPSTREAM_VERSION from upstream main @ $UPSTREAM_SHORT"
|
||||||
PR_BODY="Automated sync from superpowers upstream \`main\` @ \`$UPSTREAM_SHORT\` (v$UPSTREAM_VERSION).
|
PR_BODY="Automated sync from superpowers upstream \`main\` @ \`$UPSTREAM_SHORT\` (v$UPSTREAM_VERSION).
|
||||||
|
|
||||||
Copies the tracked plugin files from upstream, including the committed Codex manifest, assets, and hooks.
|
Copies the tracked plugin files from upstream, including the committed Codex manifest and assets.
|
||||||
|
|
||||||
Run via: \`scripts/sync-to-codex-plugin.sh\`
|
Run via: \`scripts/sync-to-codex-plugin.sh\`
|
||||||
Upstream commit: https://github.com/obra/superpowers/commit/$UPSTREAM_SHA
|
Upstream commit: https://github.com/obra/superpowers/commit/$UPSTREAM_SHA
|
||||||
|
|||||||
@@ -14,26 +14,22 @@ Subagent (general-purpose):
|
|||||||
|
|
||||||
## What Was Implemented
|
## What Was Implemented
|
||||||
|
|
||||||
[DESCRIPTION]
|
{DESCRIPTION}
|
||||||
|
|
||||||
## Requirements / Plan
|
## Requirements / Plan
|
||||||
|
|
||||||
[PLAN_OR_REQUIREMENTS]
|
{PLAN_OR_REQUIREMENTS}
|
||||||
|
|
||||||
## Git Range to Review
|
## Git Range to Review
|
||||||
|
|
||||||
**Base:** [BASE_SHA]
|
**Base:** {BASE_SHA}
|
||||||
**Head:** [HEAD_SHA]
|
**Head:** {HEAD_SHA}
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
git diff --stat [BASE_SHA]..[HEAD_SHA]
|
git diff --stat {BASE_SHA}..{HEAD_SHA}
|
||||||
git diff [BASE_SHA]..[HEAD_SHA]
|
git diff {BASE_SHA}..{HEAD_SHA}
|
||||||
```
|
```
|
||||||
|
|
||||||
## Read-Only Review
|
|
||||||
|
|
||||||
Your review is read-only on this checkout. Do not mutate the working tree, the index, HEAD, or branch state in any way. Use tools like `git show`, `git diff`, and `git log` to inspect history. If you need a working copy of a different revision, check it out into a separate temporary directory (e.g. `git worktree add /tmp/review-[SHA] [SHA]`) — never move HEAD on this checkout.
|
|
||||||
|
|
||||||
## What to Check
|
## What to Check
|
||||||
|
|
||||||
**Plan alignment:**
|
**Plan alignment:**
|
||||||
@@ -126,10 +122,10 @@ Subagent (general-purpose):
|
|||||||
```
|
```
|
||||||
|
|
||||||
**Placeholders:**
|
**Placeholders:**
|
||||||
- `[DESCRIPTION]` — brief summary of what was built
|
- `{DESCRIPTION}` — brief summary of what was built
|
||||||
- `[PLAN_OR_REQUIREMENTS]` — what it should do (plan file path, task text, or requirements)
|
- `{PLAN_OR_REQUIREMENTS}` — what it should do (plan file path, task text, or requirements)
|
||||||
- `[BASE_SHA]` — starting commit
|
- `{BASE_SHA}` — starting commit
|
||||||
- `[HEAD_SHA]` — ending commit
|
- `{HEAD_SHA}` — ending commit
|
||||||
|
|
||||||
**Reviewer returns:** Strengths, Issues (Critical / Important / Minor), Recommendations, Assessment
|
**Reviewer returns:** Strengths, Issues (Critical / Important / Minor), Recommendations, Assessment
|
||||||
|
|
||||||
|
|||||||
@@ -18,22 +18,6 @@ Subagent (general-purpose):
|
|||||||
|
|
||||||
[From implementer's report]
|
[From implementer's report]
|
||||||
|
|
||||||
## Git Range to Review
|
|
||||||
|
|
||||||
**Base:** [BASE_SHA — commit before this task]
|
|
||||||
**Head:** [HEAD_SHA — current commit]
|
|
||||||
|
|
||||||
```bash
|
|
||||||
git diff --stat [BASE_SHA]..[HEAD_SHA]
|
|
||||||
git diff [BASE_SHA]..[HEAD_SHA]
|
|
||||||
```
|
|
||||||
|
|
||||||
Only read files in this diff. Do not crawl the broader codebase.
|
|
||||||
|
|
||||||
## Read-Only Review
|
|
||||||
|
|
||||||
Your review is read-only on this checkout. Do not mutate the working tree, the index, HEAD, or branch state in any way. Use tools like `git show`, `git diff`, and `git log` to inspect history. If you need a working copy of a different revision, check it out into a separate temporary directory (e.g. `git worktree add /tmp/review-[SHA] [SHA]`) — never move HEAD on this checkout.
|
|
||||||
|
|
||||||
## CRITICAL: Do Not Trust the Report
|
## CRITICAL: Do Not Trust the Report
|
||||||
|
|
||||||
The implementer finished suspiciously quickly. Their report may be incomplete,
|
The implementer finished suspiciously quickly. Their report may be incomplete,
|
||||||
|
|||||||
@@ -237,7 +237,7 @@ If you catch yourself thinking:
|
|||||||
- "Is that not happening?" - You assumed without verifying
|
- "Is that not happening?" - You assumed without verifying
|
||||||
- "Will it show us...?" - You should have added evidence gathering
|
- "Will it show us...?" - You should have added evidence gathering
|
||||||
- "Stop guessing" - You're proposing fixes without understanding
|
- "Stop guessing" - You're proposing fixes without understanding
|
||||||
- "Ultra-think this" - Question fundamentals, not just symptoms
|
- "Ultrathink this" - Question fundamentals, not just symptoms
|
||||||
- "We're stuck?" (frustrated) - Your approach isn't working
|
- "We're stuck?" (frustrated) - Your approach isn't working
|
||||||
|
|
||||||
**When you see these:** STOP. Return to Phase 1.
|
**When you see these:** STOP. Return to Phase 1.
|
||||||
|
|||||||
@@ -41,7 +41,7 @@ If CLAUDE.md, GEMINI.md, or AGENTS.md says "don't use TDD" and a skill says "alw
|
|||||||
|
|
||||||
## Platform Adaptation
|
## Platform Adaptation
|
||||||
|
|
||||||
Skills speak in actions ("dispatch a subagent", "create a todo", "read a file") rather than naming any one runtime's tools. For per-platform tool equivalents and instructions-file conventions, see [claude-code-tools.md](references/claude-code-tools.md), [codex-tools.md](references/codex-tools.md), [copilot-tools.md](references/copilot-tools.md), [gemini-tools.md](references/gemini-tools.md), [pi-tools.md](references/pi-tools.md), and [antigravity-tools.md](references/antigravity-tools.md). Gemini CLI users get the tool mapping loaded automatically via GEMINI.md.
|
Skills speak in actions ("dispatch a subagent", "create a todo", "read a file") rather than naming any one runtime's tools. For per-platform tool equivalents and instructions-file conventions, see [claude-code-tools.md](references/claude-code-tools.md), [codex-tools.md](references/codex-tools.md), [copilot-tools.md](references/copilot-tools.md), and [gemini-tools.md](references/gemini-tools.md). Gemini CLI users get the tool mapping loaded automatically via GEMINI.md.
|
||||||
|
|
||||||
# Using Skills
|
# Using Skills
|
||||||
|
|
||||||
@@ -102,15 +102,15 @@ These thoughts mean STOP—you're rationalizing:
|
|||||||
|
|
||||||
When multiple skills could apply, use this order:
|
When multiple skills could apply, use this order:
|
||||||
|
|
||||||
1. **Process skills first** (brainstorming, systematic-debugging) - these determine HOW to approach the task
|
1. **Process skills first** (brainstorming, debugging) - these determine HOW to approach the task
|
||||||
2. **Implementation skills second** (frontend-design, mcp-builder) - these guide execution
|
2. **Implementation skills second** (frontend-design, mcp-builder) - these guide execution
|
||||||
|
|
||||||
"Let's build X" → brainstorming first, then implementation skills.
|
"Let's build X" → brainstorming first, then implementation skills.
|
||||||
"Fix this bug" → systematic-debugging first, then domain-specific skills.
|
"Fix this bug" → debugging first, then domain-specific skills.
|
||||||
|
|
||||||
## Skill Types
|
## Skill Types
|
||||||
|
|
||||||
**Rigid** (TDD, systematic-debugging): Follow exactly. Don't adapt away discipline.
|
**Rigid** (TDD, debugging): Follow exactly. Don't adapt away discipline.
|
||||||
|
|
||||||
**Flexible** (patterns): Adapt principles to context.
|
**Flexible** (patterns): Adapt principles to context.
|
||||||
|
|
||||||
|
|||||||
@@ -1,96 +0,0 @@
|
|||||||
# Antigravity CLI (`agy`) Tool Mapping
|
|
||||||
|
|
||||||
Skills speak in actions ("dispatch a subagent", "create a todo", "read a file"). On the Antigravity CLI (`agy`) these resolve to the tools below.
|
|
||||||
|
|
||||||
| Action skills request | Antigravity CLI equivalent |
|
|
||||||
|----------------------|----------------------|
|
|
||||||
| Read a file | `view_file` |
|
|
||||||
| Create a new file | `write_to_file` |
|
|
||||||
| Edit a file | `replace_file_content` |
|
|
||||||
| Edit a file in several places at once | `multi_replace_file_content` |
|
|
||||||
| Run a shell command | `run_command` |
|
|
||||||
| Search file contents | `grep_search` |
|
|
||||||
| Find files by name / list a directory | `list_dir` (no dedicated glob tool — combine `list_dir` with `grep_search`) |
|
|
||||||
| Fetch a URL | `read_url_content` |
|
|
||||||
| Search the web | `search_web` |
|
|
||||||
| Pose a structured question to your human partner | `ask_question` |
|
|
||||||
| Dispatch a subagent (`Subagent (general-purpose):` template) | `invoke_subagent` with a built-in `TypeName` — `self` for full-capability work, `research` for read-only (see [Subagent support](#subagent-support)) |
|
|
||||||
| Multiple parallel dispatches | Multiple entries in one `invoke_subagent` call's `Subagents` array |
|
|
||||||
| Task tracking ("create a todo", "mark complete") | a **task artifact** — `write_to_file` with `IsArtifact: true` and `ArtifactType: "task"` (see [Task tracking](#task-tracking)). **Not** `manage_task`, which manages background processes. |
|
|
||||||
|
|
||||||
## Invoking a skill — read its `SKILL.md`
|
|
||||||
|
|
||||||
Antigravity surfaces every installed skill's `name` + `description` to you at the
|
|
||||||
start of each session, but it has **no `Skill`/`activate_skill` tool**. To load a
|
|
||||||
skill, **read its `SKILL.md` with `view_file`, setting `IsSkillFile: true`** when
|
|
||||||
the skill applies — e.g. `view_file` on
|
|
||||||
`.../plugins/superpowers/skills/<skill-name>/SKILL.md` with `IsSkillFile: true`.
|
|
||||||
(`IsSkillFile` is agy's own signal that you're reading a file to *execute its
|
|
||||||
instructions*, not to edit or preview it — set it whenever you load a skill.)
|
|
||||||
|
|
||||||
This is the blessed skill-loading mechanism on this harness. The general rule
|
|
||||||
"never read skill files manually" means "don't bypass your platform's
|
|
||||||
skill-loading mechanism" — and on Antigravity, reading `SKILL.md` *is* that
|
|
||||||
mechanism. Reading it honors the rule rather than breaking it.
|
|
||||||
|
|
||||||
You already know which skills exist and what they're for: their names and
|
|
||||||
descriptions are in front of you at session start. When a description matches
|
|
||||||
what you're about to do, read that skill's `SKILL.md` before acting.
|
|
||||||
|
|
||||||
## Subagent support
|
|
||||||
|
|
||||||
Antigravity dispatches subagents with `invoke_subagent`, passing each one a
|
|
||||||
`TypeName` in the `Subagents` array. Two `TypeName`s are **built in** — use them
|
|
||||||
directly, no `define_subagent` needed:
|
|
||||||
|
|
||||||
- **`self`** — a full clone of you, with every tool you have (including
|
|
||||||
`write_to_file`/`replace_file_content`/`run_command`). The safe default for
|
|
||||||
general-purpose work: implementing, fixing, anything that edits files or runs
|
|
||||||
commands.
|
|
||||||
- **`research`** — read-only (file reading, `grep_search`, web/URL fetch; no write
|
|
||||||
or command access). Use it when you specifically want a subagent that can't make
|
|
||||||
changes — investigation and read-only review.
|
|
||||||
|
|
||||||
Call `define_subagent` only for a custom system prompt or capability mix: set
|
|
||||||
`enable_write_tools: true` to grant file edits **and** `run_command`,
|
|
||||||
`enable_subagent_tools` for nested dispatch, `enable_mcp_tools` for MCP. Then
|
|
||||||
invoke it by the name you gave it. (`manage_subagents` lists/kills running
|
|
||||||
subagents.)
|
|
||||||
|
|
||||||
Skills dispatch with `Subagent (general-purpose):` and either reference a
|
|
||||||
prompt-template file (e.g. `superpowers:subagent-driven-development`'s
|
|
||||||
`./implementer-prompt.md`) or supply an inline prompt. On Antigravity:
|
|
||||||
|
|
||||||
| Skill dispatch form | Antigravity equivalent |
|
|
||||||
|---------------------|----------------------|
|
|
||||||
| An implementer-style `*-prompt.md` template (writes code, runs tests) | Fill the template, then `invoke_subagent` with `TypeName: "self"` and the filled prompt |
|
|
||||||
| A read-only reviewer template (`spec-reviewer`, `code-quality-reviewer`, `code-reviewer`, `requesting-code-review`'s `./code-reviewer.md`) | `invoke_subagent` with `TypeName: "research"` and the filled review template |
|
|
||||||
| Inline prompt (no template referenced) | `invoke_subagent` with `TypeName: "self"` (or `"research"` if the task only reads) and your inline prompt |
|
|
||||||
|
|
||||||
### Prompt filling
|
|
||||||
|
|
||||||
Skills provide prompt templates with placeholders like `{WHAT_WAS_IMPLEMENTED}` or
|
|
||||||
`[FULL TEXT of task]`. Fill all placeholders before passing the complete prompt to
|
|
||||||
`invoke_subagent`. The prompt template itself contains the agent's role, review
|
|
||||||
criteria, and expected output format — the subagent will follow it.
|
|
||||||
|
|
||||||
### Parallel dispatch
|
|
||||||
|
|
||||||
Put multiple entries in a single `invoke_subagent` call's `Subagents` array to run
|
|
||||||
independent subagent work in parallel. Keep dependent tasks sequential, but do not
|
|
||||||
serialize independent subagent tasks just to preserve a simpler history.
|
|
||||||
|
|
||||||
## Task tracking
|
|
||||||
|
|
||||||
Antigravity has **no todo / `TodoWrite` tool** (`manage_task` manages background
|
|
||||||
processes — `list`/`kill`/`status`/`send_input` — it is *not* a checklist). When a
|
|
||||||
skill says to create a todo list or track tasks, maintain a **task artifact**: a
|
|
||||||
markdown checklist saved with `write_to_file` (`IsArtifact: true`,
|
|
||||||
`ArtifactMetadata.ArtifactType: "task"`), edited with `replace_file_content` /
|
|
||||||
`multi_replace_file_content` as you go.
|
|
||||||
|
|
||||||
At the start of any multi-step task, create the task artifact listing every step of
|
|
||||||
your plan. As you complete each step, edit the artifact to mark it done (`- [x]`).
|
|
||||||
If the plan changes, update the checklist. Keep it current — it is your source of
|
|
||||||
truth for what remains; once the conversation gets long, re-read it before starting
|
|
||||||
each step.
|
|
||||||
@@ -1,28 +0,0 @@
|
|||||||
# Pi Tool Mapping
|
|
||||||
|
|
||||||
Skills speak in actions ("dispatch a subagent", "create a todo", "read a file"). On Pi these resolve to the tools below.
|
|
||||||
|
|
||||||
| Action skills request | Pi equivalent |
|
|
||||||
| --- | --- |
|
|
||||||
| Invoke a skill | Pi native skills: load the relevant `SKILL.md` with `read`, or let the human use `/skill:name` |
|
|
||||||
| Read a file | `read` |
|
|
||||||
| Create a file | `write` |
|
|
||||||
| Edit a file | `edit` |
|
|
||||||
| Run a shell command | `bash` |
|
|
||||||
| Search file contents | `grep` when active; otherwise `bash` with `rg`/`grep` |
|
|
||||||
| Find files by name | `find` or `bash` with shell globs |
|
|
||||||
| List files and subdirectories | `ls` when active; otherwise `bash` with `ls` |
|
|
||||||
| Dispatch a subagent (`Subagent (general-purpose):` template) | Use an installed subagent tool such as `subagent` from `pi-subagents` if available |
|
|
||||||
| Task tracking ("create a todo", "mark complete") | Use an installed todo/task tool if available, otherwise track tasks in the plan or `TODO.md` |
|
|
||||||
|
|
||||||
## Skills
|
|
||||||
|
|
||||||
Pi discovers skills from configured skill directories and installed Pi packages. A Superpowers Pi package should expose `skills/` through its `pi.skills` manifest entry. Pi does not expose Claude Code's `Skill` tool, but the agent should still follow the Superpowers rule: when a skill applies, load and follow it before responding.
|
|
||||||
|
|
||||||
## Subagents
|
|
||||||
|
|
||||||
Pi core does not ship a standard subagent tool. The `pi-subagents` package is a strong optional companion and provides a `subagent` tool with single-agent, chain, parallel, async, forked-context, and resume/status workflows. If no subagent tool is available, do not fabricate `Task` calls; execute sequentially in the current session or explain that the optional subagent capability is not installed.
|
|
||||||
|
|
||||||
## Task lists
|
|
||||||
|
|
||||||
Pi core does not ship a standard task-list tool. If a todo/task extension is installed, use its documented tool. Otherwise use Superpowers plan files, checklists in Markdown, or a repo-local `TODO.md` for task tracking. Older Superpowers docs may refer to `TodoWrite`; treat that as the task-tracking action above.
|
|
||||||
@@ -1,16 +0,0 @@
|
|||||||
#!/usr/bin/env bash
|
|
||||||
# Run all Antigravity (agy) integration tests.
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
|
||||||
|
|
||||||
echo "=== Antigravity integration tests ==="
|
|
||||||
|
|
||||||
for t in "$SCRIPT_DIR"/test-*.sh; do
|
|
||||||
echo
|
|
||||||
echo ">>> $t"
|
|
||||||
bash "$t"
|
|
||||||
done
|
|
||||||
|
|
||||||
echo
|
|
||||||
echo "=== All Antigravity tests passed ==="
|
|
||||||
@@ -1,53 +0,0 @@
|
|||||||
#!/usr/bin/env bash
|
|
||||||
# Validate the Antigravity (agy) integration. agy installs the existing plugin
|
|
||||||
# directly (`agy plugin install <repo-url>`): it loads the bundled skills and
|
|
||||||
# runs the SessionStart hook for bootstrap, so there is no agy-specific scaffold
|
|
||||||
# to test. What IS agy-specific is the tool mapping — agy has no `Skill` tool and
|
|
||||||
# loads skills by reading SKILL.md with view_file — and SKILL.md pointing at it.
|
|
||||||
#
|
|
||||||
# Mirrors tests/pi/test-pi-extension.mjs's "tools reference documents
|
|
||||||
# harness-specific mappings" check. CI-safe: does not require `agy` installed.
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
|
||||||
REPO_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)"
|
|
||||||
|
|
||||||
MAPPING="$REPO_ROOT/skills/using-superpowers/references/antigravity-tools.md"
|
|
||||||
SKILL="$REPO_ROOT/skills/using-superpowers/SKILL.md"
|
|
||||||
|
|
||||||
fail() { echo "FAIL: $*" >&2; exit 1; }
|
|
||||||
|
|
||||||
echo "test-antigravity-tools: checking Antigravity tool mapping"
|
|
||||||
|
|
||||||
# --- Mapping exists ---------------------------------------------------------
|
|
||||||
[ -f "$MAPPING" ] || fail "tool mapping missing at $MAPPING"
|
|
||||||
|
|
||||||
# --- Skill-load mechanism: view_file on SKILL.md (IsSkillFile), no Skill tool -
|
|
||||||
grep -qiE "view_file" "$MAPPING" \
|
|
||||||
|| fail "mapping does not document view_file as the file/skill-read tool"
|
|
||||||
grep -qiE "SKILL\.md" "$MAPPING" \
|
|
||||||
|| fail "mapping does not document reading SKILL.md as the skill-load path"
|
|
||||||
grep -q "IsSkillFile" "$MAPPING" \
|
|
||||||
|| fail "mapping does not document setting IsSkillFile when loading a skill"
|
|
||||||
|
|
||||||
# --- Core action→tool mappings are documented -------------------------------
|
|
||||||
for tool in write_to_file replace_file_content run_command grep_search invoke_subagent; do
|
|
||||||
grep -q "$tool" "$MAPPING" \
|
|
||||||
|| fail "mapping does not document the '$tool' tool"
|
|
||||||
done
|
|
||||||
|
|
||||||
# --- Subagents use the built-in self/research types -------------------------
|
|
||||||
grep -q '`self`' "$MAPPING" \
|
|
||||||
|| fail "mapping does not document the built-in 'self' subagent type"
|
|
||||||
grep -q '`research`' "$MAPPING" \
|
|
||||||
|| fail "mapping does not document the built-in 'research' subagent type"
|
|
||||||
|
|
||||||
# --- Task tracking documents the 'task' artifact mechanism ------------------
|
|
||||||
grep -qE 'ArtifactType.*task|task. artifact' "$MAPPING" \
|
|
||||||
|| fail "mapping does not document task tracking as a 'task' artifact"
|
|
||||||
|
|
||||||
# --- SKILL.md Platform Adaptation links the mapping -------------------------
|
|
||||||
grep -q "antigravity-tools.md" "$SKILL" \
|
|
||||||
|| fail "SKILL.md Platform Adaptation does not reference antigravity-tools.md"
|
|
||||||
|
|
||||||
echo "PASS: Antigravity tool mapping valid (view_file skill-load, agy tools, SKILL.md link)"
|
|
||||||
@@ -1,11 +1,9 @@
|
|||||||
#!/usr/bin/env bash
|
#!/usr/bin/env bash
|
||||||
# Windows lifecycle tests for the brainstorm server.
|
# Windows lifecycle tests for the brainstorm server.
|
||||||
#
|
#
|
||||||
# Verifies brainstorm server lifecycle behavior, including:
|
# Verifies that the brainstorm server survives the 60-second lifecycle
|
||||||
# - Windows/MSYS2 foreground mode and empty OWNER_PID handling
|
# check on Windows, where OWNER_PID monitoring is disabled because the
|
||||||
# - Server survival past the 60-second lifecycle check window
|
# MSYS2 PID namespace is invisible to Node.js.
|
||||||
# - Dead-at-startup OWNER_PID validation (logged, monitoring disabled)
|
|
||||||
# - Clean stop-server.sh shutdown
|
|
||||||
#
|
#
|
||||||
# Requirements:
|
# Requirements:
|
||||||
# - Node.js in PATH
|
# - Node.js in PATH
|
||||||
@@ -22,7 +20,7 @@ SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
|||||||
REPO_ROOT="${SUPERPOWERS_ROOT:-$(cd "$SCRIPT_DIR/../.." && pwd)}"
|
REPO_ROOT="${SUPERPOWERS_ROOT:-$(cd "$SCRIPT_DIR/../.." && pwd)}"
|
||||||
START_SCRIPT="$REPO_ROOT/skills/brainstorming/scripts/start-server.sh"
|
START_SCRIPT="$REPO_ROOT/skills/brainstorming/scripts/start-server.sh"
|
||||||
STOP_SCRIPT="$REPO_ROOT/skills/brainstorming/scripts/stop-server.sh"
|
STOP_SCRIPT="$REPO_ROOT/skills/brainstorming/scripts/stop-server.sh"
|
||||||
SERVER_SCRIPT="$REPO_ROOT/skills/brainstorming/scripts/server.cjs"
|
SERVER_JS="$REPO_ROOT/skills/brainstorming/scripts/server.js"
|
||||||
|
|
||||||
TEST_DIR="${TMPDIR:-/tmp}/brainstorm-win-test-$$"
|
TEST_DIR="${TMPDIR:-/tmp}/brainstorm-win-test-$$"
|
||||||
|
|
||||||
@@ -66,7 +64,7 @@ skip() {
|
|||||||
wait_for_server_info() {
|
wait_for_server_info() {
|
||||||
local dir="$1"
|
local dir="$1"
|
||||||
for _ in $(seq 1 50); do
|
for _ in $(seq 1 50); do
|
||||||
if [[ -f "$dir/state/server-info" ]]; then
|
if [[ -f "$dir/.server-info" ]]; then
|
||||||
return 0
|
return 0
|
||||||
fi
|
fi
|
||||||
sleep 0.1
|
sleep 0.1
|
||||||
@@ -75,9 +73,9 @@ wait_for_server_info() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
get_port_from_info() {
|
get_port_from_info() {
|
||||||
# Read the port from state/server-info. Use grep/sed instead of Node.js
|
# Read the port from .server-info. Use grep/sed instead of Node.js
|
||||||
# to avoid MSYS2-to-Windows path translation issues.
|
# to avoid MSYS2-to-Windows path translation issues.
|
||||||
grep -o '"port":[0-9]*' "$1/state/server-info" | head -1 | sed 's/"port"://'
|
grep -o '"port":[0-9]*' "$1/.server-info" | head -1 | sed 's/"port"://'
|
||||||
}
|
}
|
||||||
|
|
||||||
http_check() {
|
http_check() {
|
||||||
@@ -216,11 +214,11 @@ BRAINSTORM_HOST="127.0.0.1" \
|
|||||||
BRAINSTORM_URL_HOST="localhost" \
|
BRAINSTORM_URL_HOST="localhost" \
|
||||||
BRAINSTORM_OWNER_PID="" \
|
BRAINSTORM_OWNER_PID="" \
|
||||||
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
||||||
node "$SERVER_SCRIPT" > "$TEST_DIR/survival/.server.log" 2>&1 &
|
node "$SERVER_JS" > "$TEST_DIR/survival/.server.log" 2>&1 &
|
||||||
SERVER_PID=$!
|
SERVER_PID=$!
|
||||||
|
|
||||||
if ! wait_for_server_info "$TEST_DIR/survival"; then
|
if ! wait_for_server_info "$TEST_DIR/survival"; then
|
||||||
fail "Server starts successfully" "Server did not write state/server-info within 5 seconds"
|
fail "Server starts successfully" "Server did not write .server-info within 5 seconds"
|
||||||
kill "$SERVER_PID" 2>/dev/null || true
|
kill "$SERVER_PID" 2>/dev/null || true
|
||||||
SERVER_PID=""
|
SERVER_PID=""
|
||||||
else
|
else
|
||||||
@@ -256,15 +254,10 @@ else
|
|||||||
SERVER_PID=""
|
SERVER_PID=""
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# ========== Test 5: Dead-at-startup OWNER_PID is logged but does not kill the server ==========
|
# ========== Test 5: Bad OWNER_PID causes shutdown (control) ==========
|
||||||
#
|
|
||||||
# The server validates BRAINSTORM_OWNER_PID at startup. If it's already dead,
|
|
||||||
# the PID resolution was wrong (common on WSL, Tailscale SSH, cross-user
|
|
||||||
# scenarios). The server logs 'owner-pid-invalid', disables owner monitoring,
|
|
||||||
# and continues running. The idle timeout becomes the only shutdown trigger.
|
|
||||||
|
|
||||||
echo ""
|
echo ""
|
||||||
echo "--- Dead-at-startup OWNER_PID: server survives, logs owner-pid-invalid ---"
|
echo "--- Control: Bad OWNER_PID causes shutdown ---"
|
||||||
|
|
||||||
mkdir -p "$TEST_DIR/control"
|
mkdir -p "$TEST_DIR/control"
|
||||||
|
|
||||||
@@ -279,41 +272,33 @@ BRAINSTORM_HOST="127.0.0.1" \
|
|||||||
BRAINSTORM_URL_HOST="localhost" \
|
BRAINSTORM_URL_HOST="localhost" \
|
||||||
BRAINSTORM_OWNER_PID="$BAD_PID" \
|
BRAINSTORM_OWNER_PID="$BAD_PID" \
|
||||||
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
||||||
node "$SERVER_SCRIPT" > "$TEST_DIR/control/.server.log" 2>&1 &
|
node "$SERVER_JS" > "$TEST_DIR/control/.server.log" 2>&1 &
|
||||||
CONTROL_PID=$!
|
CONTROL_PID=$!
|
||||||
|
|
||||||
if ! wait_for_server_info "$TEST_DIR/control"; then
|
if ! wait_for_server_info "$TEST_DIR/control"; then
|
||||||
fail "Control server starts" "Server did not write state/server-info within 5 seconds"
|
fail "Control server starts" "Server did not write .server-info within 5 seconds"
|
||||||
kill "$CONTROL_PID" 2>/dev/null || true
|
kill "$CONTROL_PID" 2>/dev/null || true
|
||||||
CONTROL_PID=""
|
CONTROL_PID=""
|
||||||
else
|
else
|
||||||
pass "Control server starts with dead-at-startup OWNER_PID=$BAD_PID"
|
pass "Control server starts with bad OWNER_PID=$BAD_PID"
|
||||||
|
|
||||||
echo " Waiting ~75s to verify server survives past lifecycle check..."
|
echo " Waiting ~75s for lifecycle check to kill server..."
|
||||||
sleep 75
|
sleep 75
|
||||||
|
|
||||||
if kill -0 "$CONTROL_PID" 2>/dev/null; then
|
if kill -0 "$CONTROL_PID" 2>/dev/null; then
|
||||||
pass "Server survives with dead-at-startup OWNER_PID (owner monitoring disabled)"
|
fail "Control server self-terminates with bad OWNER_PID" \
|
||||||
|
"Server is still alive (expected it to die)"
|
||||||
|
kill "$CONTROL_PID" 2>/dev/null || true
|
||||||
else
|
else
|
||||||
fail "Server survives with dead-at-startup OWNER_PID" \
|
pass "Control server self-terminates with bad OWNER_PID"
|
||||||
"Server died unexpectedly. Log tail: $(tail -5 "$TEST_DIR/control/.server.log" 2>/dev/null)"
|
|
||||||
fi
|
|
||||||
|
|
||||||
if grep -q "owner-pid-invalid" "$TEST_DIR/control/.server.log" 2>/dev/null; then
|
|
||||||
pass "Server logs 'owner-pid-invalid' for dead-at-startup PID"
|
|
||||||
else
|
|
||||||
fail "Server logs 'owner-pid-invalid' for dead-at-startup PID" \
|
|
||||||
"Log tail: $(tail -5 "$TEST_DIR/control/.server.log" 2>/dev/null)"
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if grep -q "owner process exited" "$TEST_DIR/control/.server.log" 2>/dev/null; then
|
if grep -q "owner process exited" "$TEST_DIR/control/.server.log" 2>/dev/null; then
|
||||||
fail "No spurious 'owner process exited' log" \
|
pass "Control server logs 'owner process exited'"
|
||||||
"Found 'owner process exited' but owner monitoring should be disabled"
|
|
||||||
else
|
else
|
||||||
pass "No spurious 'owner process exited' log"
|
fail "Control server logs 'owner process exited'" \
|
||||||
|
"Log tail: $(tail -5 "$TEST_DIR/control/.server.log" 2>/dev/null)"
|
||||||
fi
|
fi
|
||||||
|
|
||||||
kill "$CONTROL_PID" 2>/dev/null || true
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
wait "$CONTROL_PID" 2>/dev/null || true
|
wait "$CONTROL_PID" 2>/dev/null || true
|
||||||
@@ -324,16 +309,16 @@ CONTROL_PID=""
|
|||||||
echo ""
|
echo ""
|
||||||
echo "--- Clean Shutdown ---"
|
echo "--- Clean Shutdown ---"
|
||||||
|
|
||||||
mkdir -p "$TEST_DIR/stop-test/state"
|
mkdir -p "$TEST_DIR/stop-test"
|
||||||
|
|
||||||
BRAINSTORM_DIR="$TEST_DIR/stop-test" \
|
BRAINSTORM_DIR="$TEST_DIR/stop-test" \
|
||||||
BRAINSTORM_HOST="127.0.0.1" \
|
BRAINSTORM_HOST="127.0.0.1" \
|
||||||
BRAINSTORM_URL_HOST="localhost" \
|
BRAINSTORM_URL_HOST="localhost" \
|
||||||
BRAINSTORM_OWNER_PID="" \
|
BRAINSTORM_OWNER_PID="" \
|
||||||
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
BRAINSTORM_PORT=$((49152 + RANDOM % 16383)) \
|
||||||
node "$SERVER_SCRIPT" > "$TEST_DIR/stop-test/.server.log" 2>&1 &
|
node "$SERVER_JS" > "$TEST_DIR/stop-test/.server.log" 2>&1 &
|
||||||
STOP_TEST_PID=$!
|
STOP_TEST_PID=$!
|
||||||
echo "$STOP_TEST_PID" > "$TEST_DIR/stop-test/state/server.pid"
|
echo "$STOP_TEST_PID" > "$TEST_DIR/stop-test/.server.pid"
|
||||||
|
|
||||||
if ! wait_for_server_info "$TEST_DIR/stop-test"; then
|
if ! wait_for_server_info "$TEST_DIR/stop-test"; then
|
||||||
fail "Stop-test server starts" "Server did not start"
|
fail "Stop-test server starts" "Server did not start"
|
||||||
|
|||||||
@@ -178,7 +178,6 @@ write_upstream_fixture() {
|
|||||||
"$repo/.private-journal" \
|
"$repo/.private-journal" \
|
||||||
"$repo/assets" \
|
"$repo/assets" \
|
||||||
"$repo/evals/drill" \
|
"$repo/evals/drill" \
|
||||||
"$repo/hooks" \
|
|
||||||
"$repo/scripts" \
|
"$repo/scripts" \
|
||||||
"$repo/skills/example"
|
"$repo/skills/example"
|
||||||
|
|
||||||
@@ -219,40 +218,6 @@ EOF
|
|||||||
printf 'png fixture\n' > "$repo/assets/app-icon.png"
|
printf 'png fixture\n' > "$repo/assets/app-icon.png"
|
||||||
printf 'eval harness fixture\n' > "$repo/evals/drill/README.md"
|
printf 'eval harness fixture\n' > "$repo/evals/drill/README.md"
|
||||||
|
|
||||||
cat > "$repo/hooks/hooks-codex.json" <<'EOF'
|
|
||||||
{
|
|
||||||
"hooks": {
|
|
||||||
"SessionStart": [
|
|
||||||
{
|
|
||||||
"matcher": "startup|resume|clear",
|
|
||||||
"hooks": [
|
|
||||||
{
|
|
||||||
"type": "command",
|
|
||||||
"command": "\"${PLUGIN_ROOT}/hooks/run-hook.cmd\" session-start-codex",
|
|
||||||
"async": false
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
}
|
|
||||||
EOF
|
|
||||||
|
|
||||||
cat > "$repo/hooks/session-start" <<'EOF'
|
|
||||||
#!/usr/bin/env sh
|
|
||||||
echo "session-start fixture"
|
|
||||||
EOF
|
|
||||||
cat > "$repo/hooks/session-start-codex" <<'EOF'
|
|
||||||
#!/usr/bin/env sh
|
|
||||||
echo "session-start-codex fixture"
|
|
||||||
EOF
|
|
||||||
|
|
||||||
cat > "$repo/hooks/run-hook.cmd" <<'EOF'
|
|
||||||
@echo off
|
|
||||||
echo run-hook fixture
|
|
||||||
EOF
|
|
||||||
chmod +x "$repo/hooks/session-start" "$repo/hooks/session-start-codex" "$repo/hooks/run-hook.cmd"
|
|
||||||
|
|
||||||
cat > "$repo/skills/example/SKILL.md" <<'EOF'
|
cat > "$repo/skills/example/SKILL.md" <<'EOF'
|
||||||
# Example Skill
|
# Example Skill
|
||||||
|
|
||||||
@@ -271,10 +236,6 @@ EOF
|
|||||||
assets/app-icon.png \
|
assets/app-icon.png \
|
||||||
assets/superpowers-small.svg \
|
assets/superpowers-small.svg \
|
||||||
evals/drill/README.md \
|
evals/drill/README.md \
|
||||||
hooks/hooks-codex.json \
|
|
||||||
hooks/run-hook.cmd \
|
|
||||||
hooks/session-start \
|
|
||||||
hooks/session-start-codex \
|
|
||||||
package.json \
|
package.json \
|
||||||
scripts/sync-to-codex-plugin.sh \
|
scripts/sync-to-codex-plugin.sh \
|
||||||
skills/example/SKILL.md
|
skills/example/SKILL.md
|
||||||
@@ -332,7 +293,6 @@ write_synced_destination_fixture() {
|
|||||||
"$repo/plugins/superpowers/.codex-plugin" \
|
"$repo/plugins/superpowers/.codex-plugin" \
|
||||||
"$repo/plugins/superpowers/.private-journal" \
|
"$repo/plugins/superpowers/.private-journal" \
|
||||||
"$repo/plugins/superpowers/assets" \
|
"$repo/plugins/superpowers/assets" \
|
||||||
"$repo/plugins/superpowers/hooks" \
|
|
||||||
"$repo/plugins/superpowers/skills/example/agents" \
|
"$repo/plugins/superpowers/skills/example/agents" \
|
||||||
"$repo/plugins/superpowers/skills/example"
|
"$repo/plugins/superpowers/skills/example"
|
||||||
|
|
||||||
@@ -349,40 +309,6 @@ EOF
|
|||||||
|
|
||||||
printf 'png fixture\n' > "$repo/plugins/superpowers/assets/app-icon.png"
|
printf 'png fixture\n' > "$repo/plugins/superpowers/assets/app-icon.png"
|
||||||
|
|
||||||
cat > "$repo/plugins/superpowers/hooks/hooks-codex.json" <<'EOF'
|
|
||||||
{
|
|
||||||
"hooks": {
|
|
||||||
"SessionStart": [
|
|
||||||
{
|
|
||||||
"matcher": "startup|resume|clear",
|
|
||||||
"hooks": [
|
|
||||||
{
|
|
||||||
"type": "command",
|
|
||||||
"command": "\"${PLUGIN_ROOT}/hooks/run-hook.cmd\" session-start-codex",
|
|
||||||
"async": false
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
}
|
|
||||||
EOF
|
|
||||||
|
|
||||||
cat > "$repo/plugins/superpowers/hooks/session-start" <<'EOF'
|
|
||||||
#!/usr/bin/env sh
|
|
||||||
echo "session-start fixture"
|
|
||||||
EOF
|
|
||||||
cat > "$repo/plugins/superpowers/hooks/session-start-codex" <<'EOF'
|
|
||||||
#!/usr/bin/env sh
|
|
||||||
echo "session-start-codex fixture"
|
|
||||||
EOF
|
|
||||||
|
|
||||||
cat > "$repo/plugins/superpowers/hooks/run-hook.cmd" <<'EOF'
|
|
||||||
@echo off
|
|
||||||
echo run-hook fixture
|
|
||||||
EOF
|
|
||||||
chmod +x "$repo/plugins/superpowers/hooks/session-start" "$repo/plugins/superpowers/hooks/session-start-codex" "$repo/plugins/superpowers/hooks/run-hook.cmd"
|
|
||||||
|
|
||||||
cat > "$repo/plugins/superpowers/skills/example/SKILL.md" <<'EOF'
|
cat > "$repo/plugins/superpowers/skills/example/SKILL.md" <<'EOF'
|
||||||
# Example Skill
|
# Example Skill
|
||||||
|
|
||||||
@@ -401,10 +327,6 @@ EOF
|
|||||||
plugins/superpowers/.codex-plugin/plugin.json \
|
plugins/superpowers/.codex-plugin/plugin.json \
|
||||||
plugins/superpowers/assets/app-icon.png \
|
plugins/superpowers/assets/app-icon.png \
|
||||||
plugins/superpowers/assets/superpowers-small.svg \
|
plugins/superpowers/assets/superpowers-small.svg \
|
||||||
plugins/superpowers/hooks/hooks-codex.json \
|
|
||||||
plugins/superpowers/hooks/run-hook.cmd \
|
|
||||||
plugins/superpowers/hooks/session-start \
|
|
||||||
plugins/superpowers/hooks/session-start-codex \
|
|
||||||
plugins/superpowers/skills/example/agents/openai.yaml \
|
plugins/superpowers/skills/example/agents/openai.yaml \
|
||||||
plugins/superpowers/skills/example/SKILL.md \
|
plugins/superpowers/skills/example/SKILL.md \
|
||||||
plugins/superpowers/.private-journal/keep.txt
|
plugins/superpowers/.private-journal/keep.txt
|
||||||
@@ -620,10 +542,6 @@ main() {
|
|||||||
assert_contains "$preview_section" ".codex-plugin/plugin.json" "Preview includes manifest path"
|
assert_contains "$preview_section" ".codex-plugin/plugin.json" "Preview includes manifest path"
|
||||||
assert_contains "$preview_section" "assets/superpowers-small.svg" "Preview includes SVG asset"
|
assert_contains "$preview_section" "assets/superpowers-small.svg" "Preview includes SVG asset"
|
||||||
assert_contains "$preview_section" "assets/app-icon.png" "Preview includes PNG asset"
|
assert_contains "$preview_section" "assets/app-icon.png" "Preview includes PNG asset"
|
||||||
assert_contains "$preview_section" "hooks/hooks-codex.json" "Preview includes Codex hook manifest"
|
|
||||||
assert_contains "$preview_section" "hooks/session-start" "Preview includes session-start hook"
|
|
||||||
assert_contains "$preview_section" "hooks/session-start-codex" "Preview includes Codex session-start hook"
|
|
||||||
assert_contains "$preview_section" "hooks/run-hook.cmd" "Preview includes hook command wrapper"
|
|
||||||
assert_contains "$preview_section" ".private-journal/keep.txt" "Preview includes tracked ignored file"
|
assert_contains "$preview_section" ".private-journal/keep.txt" "Preview includes tracked ignored file"
|
||||||
assert_not_contains "$preview_section" ".private-journal/leak.txt" "Preview excludes ignored untracked file"
|
assert_not_contains "$preview_section" ".private-journal/leak.txt" "Preview excludes ignored untracked file"
|
||||||
assert_not_contains "$preview_section" "ignored-cache/" "Preview excludes pure ignored directories"
|
assert_not_contains "$preview_section" "ignored-cache/" "Preview excludes pure ignored directories"
|
||||||
|
|||||||
@@ -1,240 +0,0 @@
|
|||||||
#!/usr/bin/env bash
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
|
||||||
REPO_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)"
|
|
||||||
HOOK_UNDER_TEST="$REPO_ROOT/hooks/session-start"
|
|
||||||
CODEX_HOOK_UNDER_TEST="$REPO_ROOT/hooks/session-start-codex"
|
|
||||||
WRAPPER_UNDER_TEST="$REPO_ROOT/hooks/run-hook.cmd"
|
|
||||||
|
|
||||||
FAILURES=0
|
|
||||||
TEST_ROOT="$(mktemp -d)"
|
|
||||||
|
|
||||||
cleanup() {
|
|
||||||
rm -rf "$TEST_ROOT"
|
|
||||||
}
|
|
||||||
trap cleanup EXIT
|
|
||||||
|
|
||||||
pass() {
|
|
||||||
echo " [PASS] $1"
|
|
||||||
}
|
|
||||||
|
|
||||||
fail() {
|
|
||||||
echo " [FAIL] $1"
|
|
||||||
FAILURES=$((FAILURES + 1))
|
|
||||||
}
|
|
||||||
|
|
||||||
make_home() {
|
|
||||||
local name="$1"
|
|
||||||
local home="$TEST_ROOT/$name/home"
|
|
||||||
mkdir -p "$home"
|
|
||||||
printf '%s\n' "$home"
|
|
||||||
}
|
|
||||||
|
|
||||||
assert_command_output() {
|
|
||||||
local description="$1"
|
|
||||||
local shape="$2"
|
|
||||||
local contains="$3"
|
|
||||||
local not_contains="$4"
|
|
||||||
local home="$5"
|
|
||||||
shift 5
|
|
||||||
|
|
||||||
local output
|
|
||||||
if ! output="$(env -i PATH="${PATH:-}" HOME="$home" "$@" 2>&1)"; then
|
|
||||||
fail "$description"
|
|
||||||
echo " hook exited non-zero"
|
|
||||||
echo "$output" | sed 's/^/ /'
|
|
||||||
return
|
|
||||||
fi
|
|
||||||
|
|
||||||
if printf '%s' "$output" | \
|
|
||||||
EXPECT_SHAPE="$shape" \
|
|
||||||
EXPECT_CONTAINS="$contains" \
|
|
||||||
EXPECT_NOT_CONTAINS="$not_contains" \
|
|
||||||
node -e '
|
|
||||||
const fs = require("fs");
|
|
||||||
|
|
||||||
const input = fs.readFileSync(0, "utf8");
|
|
||||||
let payload;
|
|
||||||
try {
|
|
||||||
payload = JSON.parse(input);
|
|
||||||
} catch (error) {
|
|
||||||
console.error(`invalid JSON: ${error.message}`);
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
|
|
||||||
function hasOwn(object, key) {
|
|
||||||
return Object.prototype.hasOwnProperty.call(object, key);
|
|
||||||
}
|
|
||||||
|
|
||||||
function fail(message) {
|
|
||||||
console.error(message);
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
|
|
||||||
const shape = process.env.EXPECT_SHAPE;
|
|
||||||
let context;
|
|
||||||
|
|
||||||
if (shape === "nested") {
|
|
||||||
if (!hasOwn(payload, "hookSpecificOutput")) {
|
|
||||||
fail("missing hookSpecificOutput");
|
|
||||||
}
|
|
||||||
if (hasOwn(payload, "additional_context") || hasOwn(payload, "additionalContext")) {
|
|
||||||
fail("nested output also included a top-level context field");
|
|
||||||
}
|
|
||||||
const hookOutput = payload.hookSpecificOutput;
|
|
||||||
if (!hookOutput || typeof hookOutput !== "object" || Array.isArray(hookOutput)) {
|
|
||||||
fail("hookSpecificOutput is not an object");
|
|
||||||
}
|
|
||||||
if (hookOutput.hookEventName !== "SessionStart") {
|
|
||||||
fail(`unexpected hookEventName: ${hookOutput.hookEventName}`);
|
|
||||||
}
|
|
||||||
context = hookOutput.additionalContext;
|
|
||||||
} else if (shape === "cursor") {
|
|
||||||
if (hasOwn(payload, "hookSpecificOutput")) {
|
|
||||||
fail("cursor output included hookSpecificOutput");
|
|
||||||
}
|
|
||||||
if (!hasOwn(payload, "additional_context")) {
|
|
||||||
fail("cursor output missing additional_context");
|
|
||||||
}
|
|
||||||
if (hasOwn(payload, "additionalContext")) {
|
|
||||||
fail("cursor output included additionalContext");
|
|
||||||
}
|
|
||||||
context = payload.additional_context;
|
|
||||||
} else if (shape === "sdk") {
|
|
||||||
if (hasOwn(payload, "hookSpecificOutput")) {
|
|
||||||
fail("sdk output included hookSpecificOutput");
|
|
||||||
}
|
|
||||||
if (!hasOwn(payload, "additionalContext")) {
|
|
||||||
fail("sdk output missing additionalContext");
|
|
||||||
}
|
|
||||||
if (hasOwn(payload, "additional_context")) {
|
|
||||||
fail("sdk output included additional_context");
|
|
||||||
}
|
|
||||||
context = payload.additionalContext;
|
|
||||||
} else {
|
|
||||||
fail(`unknown expected shape: ${shape}`);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (typeof context !== "string" || context.trim() === "") {
|
|
||||||
fail("injected context was empty");
|
|
||||||
}
|
|
||||||
|
|
||||||
const expectedText = process.env.EXPECT_CONTAINS || "";
|
|
||||||
if (expectedText && !context.includes(expectedText)) {
|
|
||||||
fail(`context did not contain expected text: ${expectedText}`);
|
|
||||||
}
|
|
||||||
|
|
||||||
const forbiddenTexts = (process.env.EXPECT_NOT_CONTAINS || "")
|
|
||||||
.split("\u001f")
|
|
||||||
.filter(Boolean);
|
|
||||||
for (const forbiddenText of forbiddenTexts) {
|
|
||||||
if (context.includes(forbiddenText)) {
|
|
||||||
fail(`context contained forbidden text: ${forbiddenText}`);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
'; then
|
|
||||||
pass "$description"
|
|
||||||
else
|
|
||||||
fail "$description"
|
|
||||||
echo " output:"
|
|
||||||
echo "$output" | sed 's/^/ /'
|
|
||||||
fi
|
|
||||||
}
|
|
||||||
|
|
||||||
echo "SessionStart hook output tests"
|
|
||||||
|
|
||||||
claude_home="$(make_home claude-code)"
|
|
||||||
assert_command_output \
|
|
||||||
"Claude Code emits nested SessionStart additionalContext" \
|
|
||||||
"nested" \
|
|
||||||
"" \
|
|
||||||
"" \
|
|
||||||
"$claude_home" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
codex_home="$(make_home codex-plugin-hooks)"
|
|
||||||
codex_data="$TEST_ROOT/codex-plugin-hooks/data"
|
|
||||||
mkdir -p "$codex_data"
|
|
||||||
assert_command_output \
|
|
||||||
"Codex plugin hooks use dedicated script and emit nested SessionStart additionalContext" \
|
|
||||||
"nested" \
|
|
||||||
"" \
|
|
||||||
"" \
|
|
||||||
"$codex_home" \
|
|
||||||
PLUGIN_DATA="$codex_data" \
|
|
||||||
CLAUDE_PLUGIN_DATA="$codex_data" \
|
|
||||||
PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$CODEX_HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
codex_wrapper_home="$(make_home codex-wrapper)"
|
|
||||||
codex_wrapper_data="$TEST_ROOT/codex-wrapper/data"
|
|
||||||
mkdir -p "$codex_wrapper_data"
|
|
||||||
assert_command_output \
|
|
||||||
"Codex wrapper path dispatches to dedicated script" \
|
|
||||||
"nested" \
|
|
||||||
"" \
|
|
||||||
"" \
|
|
||||||
"$codex_wrapper_home" \
|
|
||||||
PLUGIN_DATA="$codex_wrapper_data" \
|
|
||||||
CLAUDE_PLUGIN_DATA="$codex_wrapper_data" \
|
|
||||||
PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$WRAPPER_UNDER_TEST" session-start-codex
|
|
||||||
|
|
||||||
cursor_home="$(make_home cursor)"
|
|
||||||
assert_command_output \
|
|
||||||
"Cursor emits top-level additional_context only" \
|
|
||||||
"cursor" \
|
|
||||||
"" \
|
|
||||||
"" \
|
|
||||||
"$cursor_home" \
|
|
||||||
CURSOR_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
copilot_home="$(make_home copilot-cli)"
|
|
||||||
assert_command_output \
|
|
||||||
"Copilot CLI emits top-level additionalContext only" \
|
|
||||||
"sdk" \
|
|
||||||
"" \
|
|
||||||
"" \
|
|
||||||
"$copilot_home" \
|
|
||||||
COPILOT_CLI=1 \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
legacy_home="$(make_home legacy-warning-removed)"
|
|
||||||
mkdir -p "$legacy_home/.config/superpowers/skills"
|
|
||||||
assert_command_output \
|
|
||||||
"SessionStart omits obsolete legacy custom-skill warning" \
|
|
||||||
"nested" \
|
|
||||||
"" \
|
|
||||||
"Superpowers now uses"$'\037'"~/.config/superpowers/skills"$'\037'"~/.claude/skills"$'\037'"legacy" \
|
|
||||||
"$legacy_home" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
codex_legacy_home="$(make_home codex-legacy-warning-removed)"
|
|
||||||
codex_legacy_data="$TEST_ROOT/codex-legacy-warning-removed/data"
|
|
||||||
mkdir -p "$codex_legacy_home/.config/superpowers/skills" "$codex_legacy_data"
|
|
||||||
assert_command_output \
|
|
||||||
"Codex SessionStart omits obsolete legacy custom-skill warning" \
|
|
||||||
"nested" \
|
|
||||||
"" \
|
|
||||||
"Superpowers now uses"$'\037'"~/.config/superpowers/skills"$'\037'"~/.claude/skills"$'\037'"legacy" \
|
|
||||||
"$codex_legacy_home" \
|
|
||||||
PLUGIN_DATA="$codex_legacy_data" \
|
|
||||||
CLAUDE_PLUGIN_DATA="$codex_legacy_data" \
|
|
||||||
PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
CLAUDE_PLUGIN_ROOT="$REPO_ROOT" \
|
|
||||||
bash "$CODEX_HOOK_UNDER_TEST"
|
|
||||||
|
|
||||||
if [[ "$FAILURES" -gt 0 ]]; then
|
|
||||||
echo "STATUS: FAILED ($FAILURES failure(s))"
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
|
|
||||||
echo "STATUS: PASSED"
|
|
||||||
@@ -1,128 +0,0 @@
|
|||||||
import assert from 'node:assert/strict';
|
|
||||||
import { readFile } from 'node:fs/promises';
|
|
||||||
import { existsSync } from 'node:fs';
|
|
||||||
import { dirname, resolve } from 'node:path';
|
|
||||||
import { fileURLToPath, pathToFileURL } from 'node:url';
|
|
||||||
import test from 'node:test';
|
|
||||||
|
|
||||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
|
||||||
const repoRoot = resolve(__dirname, '../..');
|
|
||||||
const packageJsonPath = resolve(repoRoot, 'package.json');
|
|
||||||
const extensionPath = resolve(repoRoot, '.pi/extensions/superpowers.ts');
|
|
||||||
const piToolsPath = resolve(repoRoot, 'skills/using-superpowers/references/pi-tools.md');
|
|
||||||
|
|
||||||
async function readPackageJson() {
|
|
||||||
return JSON.parse(await readFile(packageJsonPath, 'utf8'));
|
|
||||||
}
|
|
||||||
|
|
||||||
async function loadExtension() {
|
|
||||||
const handlers = new Map();
|
|
||||||
const pi = {
|
|
||||||
on(event, handler) {
|
|
||||||
if (!handlers.has(event)) handlers.set(event, []);
|
|
||||||
handlers.get(event).push(handler);
|
|
||||||
},
|
|
||||||
};
|
|
||||||
const mod = await import(pathToFileURL(extensionPath).href + `?cachebust=${Date.now()}-${Math.random()}`);
|
|
||||||
mod.default(pi);
|
|
||||||
return { handlers };
|
|
||||||
}
|
|
||||||
|
|
||||||
function firstHandler(handlers, event) {
|
|
||||||
const eventHandlers = handlers.get(event) ?? [];
|
|
||||||
assert.equal(eventHandlers.length, 1, `expected one ${event} handler`);
|
|
||||||
return eventHandlers[0];
|
|
||||||
}
|
|
||||||
|
|
||||||
function textOf(message) {
|
|
||||||
if (typeof message.content === 'string') return message.content;
|
|
||||||
return message.content
|
|
||||||
.filter((part) => part.type === 'text')
|
|
||||||
.map((part) => part.text)
|
|
||||||
.join('\n');
|
|
||||||
}
|
|
||||||
|
|
||||||
test('package.json declares a pi package with skills and extension resources', async () => {
|
|
||||||
const pkg = await readPackageJson();
|
|
||||||
|
|
||||||
assert.equal(pkg.name, 'superpowers');
|
|
||||||
assert.ok(pkg.keywords.includes('pi-package'));
|
|
||||||
assert.deepEqual(pkg.pi.skills, ['./skills']);
|
|
||||||
assert.deepEqual(pkg.pi.extensions, ['./.pi/extensions/superpowers.ts']);
|
|
||||||
});
|
|
||||||
|
|
||||||
test('extension registers lifecycle hooks without pre-compaction injection', async () => {
|
|
||||||
const { handlers } = await loadExtension();
|
|
||||||
|
|
||||||
for (const event of ['resources_discover', 'session_start', 'session_compact', 'context', 'agent_end']) {
|
|
||||||
assert.equal((handlers.get(event) ?? []).length, 1, `missing ${event} handler`);
|
|
||||||
}
|
|
||||||
assert.equal((handlers.get('session_before_compact') ?? []).length, 0);
|
|
||||||
});
|
|
||||||
|
|
||||||
test('resources_discover contributes the bundled skills directory', async () => {
|
|
||||||
const { handlers } = await loadExtension();
|
|
||||||
const discover = firstHandler(handlers, 'resources_discover');
|
|
||||||
|
|
||||||
const result = await discover({ type: 'resources_discover', cwd: repoRoot, reason: 'startup' }, {});
|
|
||||||
|
|
||||||
assert.deepEqual(result.skillPaths, [resolve(repoRoot, 'skills')]);
|
|
||||||
});
|
|
||||||
|
|
||||||
test('startup context injects the bootstrap as one user message until agent_end', async () => {
|
|
||||||
const { handlers } = await loadExtension();
|
|
||||||
const sessionStart = firstHandler(handlers, 'session_start');
|
|
||||||
const context = firstHandler(handlers, 'context');
|
|
||||||
const agentEnd = firstHandler(handlers, 'agent_end');
|
|
||||||
|
|
||||||
await sessionStart({ type: 'session_start', reason: 'startup' }, {});
|
|
||||||
|
|
||||||
const originalMessages = [
|
|
||||||
{ role: 'user', content: [{ type: 'text', text: 'Let us make a react todo list' }], timestamp: 1 },
|
|
||||||
];
|
|
||||||
const result = await context({ type: 'context', messages: originalMessages }, {});
|
|
||||||
|
|
||||||
assert.equal(result.messages.length, 2);
|
|
||||||
assert.equal(result.messages[0].role, 'user');
|
|
||||||
assert.match(textOf(result.messages[0]), /You have superpowers/);
|
|
||||||
assert.match(textOf(result.messages[0]), /Pi tool mapping/);
|
|
||||||
assert.equal(result.messages[1], originalMessages[0]);
|
|
||||||
|
|
||||||
const repeatedProviderRequest = await context({ type: 'context', messages: originalMessages }, {});
|
|
||||||
assert.equal(repeatedProviderRequest.messages.length, 2);
|
|
||||||
assert.match(textOf(repeatedProviderRequest.messages[0]), /You have superpowers/);
|
|
||||||
|
|
||||||
const alreadyInjected = await context({ type: 'context', messages: result.messages }, {});
|
|
||||||
assert.equal(alreadyInjected, undefined, 'bootstrap should not duplicate when already present');
|
|
||||||
|
|
||||||
await agentEnd({ type: 'agent_end', messages: [] }, {});
|
|
||||||
const afterEnd = await context({ type: 'context', messages: originalMessages }, {});
|
|
||||||
assert.equal(afterEnd, undefined, 'startup bootstrap should clear after agent_end');
|
|
||||||
});
|
|
||||||
|
|
||||||
test('session_compact injects bootstrap after compaction summaries, not before compaction', async () => {
|
|
||||||
const { handlers } = await loadExtension();
|
|
||||||
const sessionCompact = firstHandler(handlers, 'session_compact');
|
|
||||||
const context = firstHandler(handlers, 'context');
|
|
||||||
|
|
||||||
await sessionCompact({ type: 'session_compact', compactionEntry: {}, fromExtension: false }, {});
|
|
||||||
|
|
||||||
const summary = { role: 'compactionSummary', summary: 'Prior work summary', tokensBefore: 123, timestamp: 1 };
|
|
||||||
const user = { role: 'user', content: [{ type: 'text', text: 'Continue' }], timestamp: 2 };
|
|
||||||
const result = await context({ type: 'context', messages: [summary, user] }, {});
|
|
||||||
|
|
||||||
assert.equal(result.messages.length, 3);
|
|
||||||
assert.equal(result.messages[0], summary);
|
|
||||||
assert.equal(result.messages[1].role, 'user');
|
|
||||||
assert.match(textOf(result.messages[1]), /You have superpowers/);
|
|
||||||
assert.equal(result.messages[2], user);
|
|
||||||
});
|
|
||||||
|
|
||||||
test('pi tools reference documents pi-specific mappings', async () => {
|
|
||||||
assert.equal(existsSync(piToolsPath), true, 'pi-tools.md should exist');
|
|
||||||
const text = await readFile(piToolsPath, 'utf8');
|
|
||||||
|
|
||||||
for (const expected of ['Skill', 'Task', 'TodoWrite', 'read', 'write', 'edit', 'bash']) {
|
|
||||||
assert.match(text, new RegExp(expected));
|
|
||||||
}
|
|
||||||
});
|
|
||||||
Reference in New Issue
Block a user