mirror of
https://github.com/jackwener/OpenCLI.git
synced 2026-09-14 18:25:42 +08:00
jakevin
14672ddf9b
- Remove dead code: unused `promises` array in discovery.ts, unused `DEFAULT_BROWSER_SMOKE_TIMEOUT`, unused `checkFfmpeg()`, deprecated `PlaywrightMCP` alias - Extract shared `YamlArgDefinition`/`YamlCliDefinition` into `yaml-schema.ts` (was duplicated in discovery.ts and build-manifest.ts) - Unify `BrowserCookie` type: remove duplicate from download/index.ts, re-export from types.ts - Create `_shared/desktop-commands.ts` with factory functions (makeScreenshotCommand, makeStatusCommand, makeNewCommand, makeDumpCommand), simplifying 11 adapter files from ~20-30 lines each to 3 lines - Fix unnecessary dynamic imports in utils.ts Net: +30 / -361 lines
72 lines
1.9 KiB
Bash
Executable File
72 lines
1.9 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# check-doc-coverage.sh — Verify every adapter in src/clis/ has a doc page.
|
|
#
|
|
# Exit codes:
|
|
# 0 — all adapters have docs
|
|
# 1 — at least one adapter is missing documentation
|
|
#
|
|
# Usage:
|
|
# bash scripts/check-doc-coverage.sh # report only
|
|
# bash scripts/check-doc-coverage.sh --strict # exit 1 on missing docs
|
|
|
|
set -euo pipefail
|
|
|
|
STRICT=false
|
|
if [[ "${1:-}" == "--strict" ]]; then
|
|
STRICT=true
|
|
fi
|
|
|
|
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
|
ROOT_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
|
|
|
|
SRC_DIR="$ROOT_DIR/src/clis"
|
|
DOCS_DIR="$ROOT_DIR/docs/adapters"
|
|
|
|
missing=()
|
|
covered=0
|
|
total=0
|
|
|
|
for adapter_dir in "$SRC_DIR"/*/; do
|
|
adapter_name="$(basename "$adapter_dir")"
|
|
# Skip internal directories (e.g., _shared)
|
|
[[ "$adapter_name" == _* ]] && continue
|
|
total=$((total + 1))
|
|
|
|
# Check if doc exists in browser/ or desktop/ subdirectories
|
|
if [[ -f "$DOCS_DIR/browser/$adapter_name.md" ]] || \
|
|
[[ -f "$DOCS_DIR/desktop/$adapter_name.md" ]]; then
|
|
covered=$((covered + 1))
|
|
else
|
|
# Handle directory name mismatches (e.g., discord-app -> discord)
|
|
alt_name="${adapter_name%-app}"
|
|
if [[ "$alt_name" != "$adapter_name" ]] && \
|
|
{ [[ -f "$DOCS_DIR/browser/$alt_name.md" ]] || \
|
|
[[ -f "$DOCS_DIR/desktop/$alt_name.md" ]]; }; then
|
|
covered=$((covered + 1))
|
|
else
|
|
missing+=("$adapter_name")
|
|
fi
|
|
fi
|
|
done
|
|
|
|
echo "📊 Doc Coverage: $covered/$total adapters documented"
|
|
echo ""
|
|
|
|
if [[ ${#missing[@]} -gt 0 ]]; then
|
|
echo "⚠️ Missing docs for ${#missing[@]} adapter(s):"
|
|
for name in "${missing[@]}"; do
|
|
echo " - $name → create docs/adapters/browser/$name.md or docs/adapters/desktop/$name.md"
|
|
done
|
|
echo ""
|
|
if $STRICT; then
|
|
echo "❌ Doc check failed (--strict mode)."
|
|
exit 1
|
|
else
|
|
echo "💡 Run with --strict to fail CI on missing docs."
|
|
exit 0
|
|
fi
|
|
else
|
|
echo "✅ All adapters have documentation."
|
|
exit 0
|
|
fi
|