mirror of
https://github.com/obra/superpowers.git
synced 2026-06-28 21:49:05 +08:00
bfa3e4137a
Prompt: Jesse questioned whether the PR should remove the hooks config from the Codex plugin manifest. Runtime investigation showed Codex accepts a committed plugin manifest with hooks and installs the plugin successfully. Removing the field changes behavior: Codex falls back to the default hooks/hooks.json, which uses the non-Codex session-start hook and CLAUDE_PLUGIN_ROOT path, instead of hooks/hooks-codex.json and the session-start-codex script. Changes: restore .codex-plugin/plugin.json hooks to ./hooks/hooks-codex.json and update the Codex marketplace manifest test to require that Codex-specific hook pointer instead of rejecting hooks. Validation: bash tests/codex/test-marketplace-manifest.sh; scripts/lint-shell.sh tests/codex/test-marketplace-manifest.sh; bash tests/codex-plugin-sync/test-sync-to-codex-plugin.sh; bash tests/kimi/test-plugin-manifest.sh; bash tests/shell-lint/test-lint-shell.sh.
62 lines
1.9 KiB
Bash
Executable File
62 lines
1.9 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
set -euo pipefail
|
|
|
|
SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
|
|
REPO_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)"
|
|
MARKETPLACE="$REPO_ROOT/.agents/plugins/marketplace.json"
|
|
|
|
python3 - "$MARKETPLACE" "$REPO_ROOT" <<'PY'
|
|
import json
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
marketplace_path = Path(sys.argv[1])
|
|
repo_root = Path(sys.argv[2])
|
|
|
|
if not marketplace_path.exists():
|
|
raise AssertionError(".agents/plugins/marketplace.json must exist")
|
|
|
|
marketplace = json.loads(marketplace_path.read_text(encoding="utf-8"))
|
|
|
|
def assert_equal(actual, expected, label):
|
|
if actual != expected:
|
|
raise AssertionError(f"{label}: expected {expected!r}, got {actual!r}")
|
|
|
|
assert_equal(marketplace.get("name"), "superpowers-dev", "marketplace name")
|
|
assert_equal(
|
|
marketplace.get("interface", {}).get("displayName"),
|
|
"Superpowers Dev",
|
|
"marketplace display name",
|
|
)
|
|
|
|
plugins = marketplace.get("plugins")
|
|
if not isinstance(plugins, list):
|
|
raise AssertionError("plugins must be a list")
|
|
|
|
matching_plugins = [plugin for plugin in plugins if plugin.get("name") == "superpowers"]
|
|
assert_equal(len(matching_plugins), 1, "superpowers plugin entry count")
|
|
|
|
plugin = matching_plugins[0]
|
|
assert_equal(plugin.get("source"), {"source": "url", "url": "./"}, "plugin source")
|
|
assert_equal(
|
|
plugin.get("policy"),
|
|
{"installation": "AVAILABLE", "authentication": "ON_INSTALL"},
|
|
"plugin policy",
|
|
)
|
|
assert_equal(plugin.get("category"), "Developer Tools", "plugin category")
|
|
|
|
plugin_manifest = repo_root / ".codex-plugin" / "plugin.json"
|
|
if not plugin_manifest.exists():
|
|
raise AssertionError(".codex-plugin/plugin.json must exist")
|
|
|
|
manifest = json.loads(plugin_manifest.read_text(encoding="utf-8"))
|
|
assert_equal(manifest.get("name"), plugin.get("name"), "plugin manifest name")
|
|
assert_equal(
|
|
manifest.get("hooks"),
|
|
"./hooks/hooks-codex.json",
|
|
"Codex hooks manifest",
|
|
)
|
|
|
|
print("Codex marketplace manifest looks good")
|
|
PY
|