 Garry Tan
|
ab8509611c
|
test: 100% coverage for host config system
71 new tests in test/host-config.test.ts covering:
- hosts/index.ts: ALL_HOST_CONFIGS, getHostConfig, resolveHostArg (aliases),
getExternalHosts, uniqueness checks
- host-config.ts validateHostConfig: name regex, displayName, cliCommand,
cliAliases, globalRoot, localSkillRoot, hostSubdir, frontmatter.mode,
linkingStrategy, shell injection attempts, paths with $ and ~
- host-config.ts validateAllConfigs: duplicate name/hostSubdir/globalRoot
detection, error prefix format, real configs pass
- HOST_PATHS derivation: env vars for external hosts, literal paths for
Claude, localSkillRoot matches config, every host has entry
- host-config-export.ts CLI: list, get (string/boolean/array), detect,
validate, symlinks, error cases (missing args, unknown field/host)
- Golden-file regression: claude/codex/factory ship SKILL.md vs baselines
- Individual host config correctness: prefixable, linkingStrategy,
usesEnvVars, description limits, metadata, sidecar, tool rewrites,
conditional fields, suppressed resolvers, boundary instruction,
co-author trailers, skip rules, path rewrites, runtime root assets
Combined with the 35 parameterized smoke tests from gen-skill-docs.test.ts,
total new test coverage for multi-host: 106 tests.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
2026-04-03 17:49:41 -07:00 |
|