Files
My-Brain-Is-Full-Crew/tests/adapters/lib.test.sh
Giacomo Nunziati 53b605379e feat: multi-platform adapter architecture (Claude Code, Gemini CLI, OpenCode) (#32)
* Fix istall/update scripts

* test: capture pre-refactor install snapshot for regression

Adds take-snapshot.sh script and the resulting snapshot/ directory,
capturing the exact vault state produced by launchme.sh before the
framework-agnosticity refactor begins.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Summary: Refactor agents/skills/hooks/mcp in agentic-platform-agnostic templates.

refactor: rename source CLAUDE.md → DISPATCHER.md (framework-neutral)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

refactor: convert agent frontmatter from tools: to neutral capabilities:

Replace Claude Code-specific `tools:` frontmatter with framework-agnostic
`mode: subagent` and `capabilities: [...]` in all 8 agent files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

refactor: add neutral hook trigger manifests (.hook.yaml)

refactor: hooks read neutral JSON schema (args.* instead of tool_input.*)

refactor: convert .mcp.json to neutral mcp/servers.yaml

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Implement agentic-platform adapters skeleton.

build: add adapters/lib.sh skeleton with vocabulary constants

test: bash test runner for adapter helpers

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): parse_frontmatter helper with tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): parse_capabilities helper with tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): should_include helper with tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): parse_hook_yaml helper with tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): agent_body helper with tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(adapters): enumerate_agents and enumerate_hooks helpers

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Implement agentic-platform adapter for Claude Code.

build(adapters): claude-code adapter skeleton with capability/event tables

build(claude-code): adapter_translate_dispatcher with test

build(claude-code): adapter_translate_references with test

build(claude-code): adapter_translate_skills with tests

build(claude-code): adapter_translate_agents with capability→tools mapping

build(claude-code): hook wrapper template (CC native → neutral schema)

build(claude-code): adapter_translate_hooks with wrapper generation

build(claude-code): adapter_translate_mcp with hand-rolled YAML parser

build(claude-code): adapter_finalize and complete adapter_build wiring

build: scripts/build.sh dispatches to per-framework adapter

Also fix adapter_translate_hooks and adapter_translate_agents to use
while-read loops (avoiding word-splitting on paths with spaces) and
guard grep calls with || true to survive set -eo pipefail when hooks
have no match-tool field. Remove scripts/build.sh from .gitignore.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Refactor install/update scripts to support agentic-platform agnosticity.

refactor(lib.sh): generalize install_claude_md → install_dispatcher

New signature takes the full destination path instead of just the vault
dir, allowing callers to install CLAUDE.md, AGENTS.md, or any dispatcher
file to an explicit location.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(launchme): support --framework flag, build dist/ before install

Add --framework and --target arg parsing. Run build.sh before installing
to populate dist/<framework>/. All install_* calls now read from
dist/<framework>/ instead of the raw source dirs. MCP is now handled
automatically by the adapter (no interactive prompt). Replaced
install_claude_md with install_dispatcher.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(updateme): support --framework flag, build dist/ before update

Add --framework and --target arg parsing. Run build.sh before installing
to populate dist/<framework>/. All install_* calls now read from
dist/<framework>/ instead of raw source dirs. Replaced install_claude_md
with install_dispatcher.

Also fix set -e compatibility in lib.sh: add || true to all conditional
[[ ... ]] && info "..." logging lines so they don't abort the script
when VERBOSE_COPY=0.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test: regression runner diffs dist/claude-code against pre-refactor snapshot

- Add tests/regression/run.sh that builds dist/claude-code and compares
  against snapshot, excluding runtime-only artifacts (.mbifc-manifest,
  .mcp.json, .claude-plugin/plugin.json)
- Fix adapters/lib.sh agent_body: preserve '---' section dividers in body
  (awk now only skips '---' while still inside frontmatter, fm < 2)
- Fix adapters/claude-code/adapter.sh: change 'read' capability to expand
  to only 'Read', appending 'Glob, Grep' at end of tools list to match
  snapshot ordering
- Update snapshot to reflect intentional refactor changes: hook JSON schema
  (.args.* instead of .tool_input.*), wrapper scripts, settings.json with
  wrapper paths, and consistent tool ordering for postman/sorter

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Implement opencode adapter.

Co-Authored-By: win0na <winnie@winneon.moe>

feat(lib.sh): add install_plugins helper for opencode JS plugins

build(adapters): opencode adapter skeleton with capability/event tables

build(opencode): adapter_translate_dispatcher (DISPATCHER.md → AGENTS.md)

build(opencode): adapter_translate_references and adapter_translate_skills

Implements Task 4 and Task 5:
- adapter_translate_references: Copies reference markdown files to .opencode/references/
- adapter_translate_skills: Copies skill SKILL.md files to .opencode/skills/<name>/ with exclude filtering

Both functions respect framework filtering via should_include().

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(opencode): adapter_translate_agents with capability→permission mapping

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(opencode): bash-executor template for spawning hook scripts

build(opencode): plugin-stub template for mbifc-hooks.js

build(opencode): adapter_translate_hooks with JS plugin generation

Implements _oc_hook_registry_json and adapter_translate_hooks in the
opencode adapter. Copies hook scripts to .opencode/hooks/, generates a
single .opencode/plugins/mbifc-hooks.js by inlining bash-executor.js and
synthesising a hook registry from *.hook.yaml files. Uses python3 for
template substitution to safely handle multi-line JS content. Adds 3
unit tests (copies scripts, registry entries, noop when no hooks dir).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(opencode): adapter_translate_mcp with local/remote handling

build(opencode): adapter_finalize and complete adapter_build wiring

Add adapter_finalize placeholder and wire adapter_translate_mcp into
adapter_build; add end-to-end integration test (14/14 pass).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(launchme): branch on --framework for opencode install layout

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(updateme): branch on --framework for opencode install layout

Mirror the same case "$FRAMEWORK" block from launchme.sh: framework-specific
DIST_COMPONENTS_DIR, VAULT_COMPONENTS_DIR, DISPATCHER_SRC/DST, MCP_SRC/DST,
HAS_PLUGINS; conditional install_plugins; conditional install_settings;
framework-aware vault-setup check; framework-neutral summary messages.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Fix adapters to follow the same template.

fix: restore adapter_build() contract, revert function renames

Both adapters now export adapter_build() and adapter_translate_*() as
the uniform public contract. scripts/build.sh sources one adapter and
calls adapter_build uniformly. Private helpers (_oc_*) and vocabulary
tables (cc_capability_to_tools, oc_capability_to_permission, etc.)
retain their prefixes. CC regression and OC unit tests all pass.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(tests): restore test_oc_ prefix on adapter_build end-to-end test

* Fix agent format in opencode adapter

* refactor: rename --framework to --platform across all scripts and tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Modify generic name for model tiers

Co-Authored-By: win0na <winnie@winneon.moe>

refactor: neutral model vocabulary (low/mid/high) in source agents

feat(claude-code): cc_model_to_native() maps low/mid/high to haiku/sonnet/opus

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(opencode): update oc_model_to_provider() for low/mid/high vocabulary

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add gemini-cli adapter

Co-Authored-By: win0na <winnie@winneon.moe>

build(gemini-cli): adapter skeleton with capability/event/model tables

build(gemini-cli): adapter_translate_dispatcher (DISPATCHER.md → GEMINI.md)

build(gemini-cli): adapter_translate_references and adapter_translate_skills

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(gemini-cli): adapter_translate_agents with capability→tools mapping

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(gemini-cli): adapter_translate_hooks with wrapper scripts

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

feat(install): add gemini-cli platform to launchme.sh and updateme.sh

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Implement preserving config merge for opencode.

Co-Authored-By: win0na <winnie@winneon.moe>

feat(opencode): config-merge.sh with formatting-preserving JSON merge

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

build(opencode): source config-merge.sh from adapter

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(install): use oc_config_merge for opencode.json instead of overwrite

Source config-merge.sh from install scripts for opencode platform so
user keys in opencode.json are preserved on reinstall and update.
Fix in-place merge by writing to a temp file before moving to output.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add mcp files to gitignore.

* Fix claude-specific references in agents, skills and references

* Fix: remove claude-specific reference from hooks.

build: add platform_dir and dispatcher_name to all hook wrapper/plugin templates

feat(hooks): platform-aware path checks using platform_dir and dispatcher_name from JSON input

test: update regression snapshot for platform-aware hook wrappers and scripts

* Added interactive platform choice in launchme, and platform auto-detection in updateme.

* Fix: remove claude-specific references from documentation

* Update documentation to reflect the new platform-agnostic architecture

* fix: address Copilot review feedback on PR #32

- tests/run.sh: check source return code, report failures
- tests/regression/run.sh: use mktemp + trap cleanup instead of fixed /tmp paths
- tests/regression/run.sh: include .mcp.json in regression comparison
- tests/regression/take-snapshot.sh: use --platform flag instead of stale scripted input
- adapters/opencode/templates/plugin-stub.js.tmpl: include stdout in hook block error message

* fix: address Copilot review round 2

- config-merge.sh: reword comment to only promise indentation preservation (not full formatting)
- take-snapshot.sh: copy required artifacts explicitly, optional ones with existence check
- adapters/lib.sh: document parse_hook_yaml single-trigger limitation

* fix: address Copilot review round 3

- adapters/opencode/adapter.sh: replace python3 template substitution with
  pure bash (while-read loop with case matching), removing python3 dependency
- adapters/lib.sh: should_include now falls back to plain YAML key read for
  files without frontmatter delimiters (fixes hook .yaml exclude: support)

* fix: address Copilot review round 4

- scripts/launchme.sh: fix double-dot in FW_DIR_NAME display (basename
  already includes the dot, e.g. ".claude")
- scripts/launchme.sh: replace undefined MCP_ANSWER with check on MCP_DST
  existence for summary banner

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-04-10 23:28:28 +02:00

248 lines
6.9 KiB
Bash
Executable File

#!/usr/bin/env bash
# Tests for adapters/lib.sh
# Source the lib under test
ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)"
source "$ROOT/adapters/lib.sh"
# Test functions will be added in subsequent tasks.
# Each function must be named test_* to be auto-discovered by tests/run.sh.
test_parse_frontmatter_scalar() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
description: Text Capture
model: sonnet
---
body content
EOF
local result; result="$(parse_frontmatter "$fixture" name)"
rm "$fixture"
[[ "$result" == "scribe" ]] || { echo "expected 'scribe', got '$result'"; return 1; }
}
test_parse_frontmatter_list() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
capabilities: [read, write, edit]
---
EOF
local result; result="$(parse_frontmatter "$fixture" capabilities)"
rm "$fixture"
[[ "$result" == "[read, write, edit]" ]] || { echo "expected '[read, write, edit]', got '$result'"; return 1; }
}
test_parse_frontmatter_missing_key() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
---
EOF
local result; result="$(parse_frontmatter "$fixture" nonexistent)"
rm "$fixture"
[[ -z "$result" ]] || { echo "expected empty, got '$result'"; return 1; }
}
test_parse_capabilities_normal() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
capabilities: [read, write, edit]
---
EOF
local result; result="$(parse_capabilities "$fixture")"
rm "$fixture"
[[ "$result" == "read write edit" ]] || { echo "expected 'read write edit', got '$result'"; return 1; }
}
test_parse_capabilities_single() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
capabilities: [read]
---
EOF
local result; result="$(parse_capabilities "$fixture")"
rm "$fixture"
[[ "$result" == "read" ]] || { echo "expected 'read', got '$result'"; return 1; }
}
test_parse_capabilities_empty() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
capabilities: []
---
EOF
local result; result="$(parse_capabilities "$fixture")"
rm "$fixture"
[[ -z "$result" ]] || { echo "expected empty, got '$result'"; return 1; }
}
test_should_include_no_exclude() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
---
EOF
if should_include "$fixture" claude-code; then
rm "$fixture"; return 0
else
rm "$fixture"; echo "expected 0, got 1"; return 1
fi
}
test_should_include_empty_exclude() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
exclude: []
---
EOF
if should_include "$fixture" claude-code; then
rm "$fixture"; return 0
else
rm "$fixture"; echo "expected 0, got 1"; return 1
fi
}
test_should_include_excluded() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
exclude: [opencode]
---
EOF
if should_include "$fixture" opencode; then
rm "$fixture"; echo "expected 1, got 0"; return 1
else
rm "$fixture"; return 0
fi
}
test_should_include_excluded_other_fw() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
exclude: [opencode]
---
EOF
if should_include "$fixture" claude-code; then
rm "$fixture"; return 0
else
rm "$fixture"; echo "expected 0, got 1"; return 1
fi
}
test_parse_hook_yaml_simple() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
name: notify
script: notify.sh
triggers:
- event: on-notification
exclude: []
EOF
local result; result="$(parse_hook_yaml "$fixture")"
rm "$fixture"
[[ "$result" == *"name=notify"* ]] || { echo "missing name=notify in: $result"; return 1; }
[[ "$result" == *"script=notify.sh"* ]] || { echo "missing script="; return 1; }
[[ "$result" == *"event=on-notification"* ]] || { echo "missing event="; return 1; }
}
test_parse_hook_yaml_with_match() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
name: protect-system-files
script: protect-system-files.sh
triggers:
- event: before-tool-use
match-tool: [edit, write]
EOF
local result; result="$(parse_hook_yaml "$fixture")"
rm "$fixture"
[[ "$result" == *"event=before-tool-use"* ]] || { echo "missing event"; return 1; }
[[ "$result" == *"match-tool=edit write"* ]] || { echo "missing match-tool: $result"; return 1; }
}
test_agent_body() {
local fixture; fixture="$(mktemp)"
cat > "$fixture" <<'EOF'
---
name: scribe
---
You are the Scribe.
You write notes.
EOF
local result; result="$(agent_body "$fixture")"
rm "$fixture"
[[ "$result" == *"You are the Scribe."* ]] || { echo "missing body line 1"; return 1; }
[[ "$result" == *"You write notes."* ]] || { echo "missing body line 2"; return 1; }
[[ "$result" != *"name: scribe"* ]] || { echo "frontmatter leaked into body"; return 1; }
}
test_enumerate_agents() {
local dir; dir="$(mktemp -d)"
touch "$dir/foo.md" "$dir/bar.md" "$dir/not-an-agent.txt"
local count; count="$(enumerate_agents "$dir" | wc -l | xargs)"
rm -rf "$dir"
[[ "$count" == "2" ]] || { echo "expected 2, got $count"; return 1; }
}
test_enumerate_hooks() {
local dir; dir="$(mktemp -d)"
touch "$dir/foo.hook.yaml" "$dir/bar.hook.yaml" "$dir/foo.sh"
local count; count="$(enumerate_hooks "$dir" | wc -l | xargs)"
rm -rf "$dir"
[[ "$count" == "2" ]] || { echo "expected 2, got $count"; return 1; }
}
test_rewrite_platform_paths_replaces_both() {
local tmp; tmp="$(mktemp)"
cat > "$tmp" <<'HEREDOC'
See .platform/references/agent-orchestration.md for details.
The dispatcher (DISPATCHER.md) handles routing.
Files live in .platform/agents/ directory.
HEREDOC
rewrite_platform_paths "$tmp" "claude" "CLAUDE.md"
local result=0
grep -q '\.claude/references/agent-orchestration.md' "$tmp" || { echo ".platform/ not rewritten"; result=1; }
grep -q 'CLAUDE.md' "$tmp" || { echo "DISPATCHER.md not rewritten"; result=1; }
grep -q '\.claude/agents/' "$tmp" || { echo "second .platform/ not rewritten"; result=1; }
grep -q '\.platform/' "$tmp" && { echo ".platform/ still present"; result=1; }
grep -q 'DISPATCHER\.md' "$tmp" && { echo "DISPATCHER.md still present"; result=1; }
rm -f "$tmp"
return $result
}
test_rewrite_platform_paths_opencode() {
local tmp; tmp="$(mktemp)"
echo 'See .platform/references/agents.md and DISPATCHER.md' > "$tmp"
rewrite_platform_paths "$tmp" "opencode" "AGENTS.md"
local result=0
grep -q '\.opencode/references/agents.md' "$tmp" || { echo "not rewritten to .opencode/"; result=1; }
grep -q 'AGENTS.md' "$tmp" || { echo "not rewritten to AGENTS.md"; result=1; }
rm -f "$tmp"
return $result
}
test_rewrite_platform_paths_gemini() {
local tmp; tmp="$(mktemp)"
echo 'See .platform/agents/scribe.md and DISPATCHER.md' > "$tmp"
rewrite_platform_paths "$tmp" "gemini" "GEMINI.md"
local result=0
grep -q '\.gemini/agents/scribe.md' "$tmp" || { echo "not rewritten to .gemini/"; result=1; }
grep -q 'GEMINI.md' "$tmp" || { echo "not rewritten to GEMINI.md"; result=1; }
rm -f "$tmp"
return $result
}