stefano
251148c3ff
## Summary - add repository-wide quality tooling and verification scaffolding, including CI workflows, pnpm workspace setup, ESLint/Prettier/markdown checks, and generated-output verification helpers - reorganize skill sources and generation flow by introducing canonical `_source` variants, generator/manifests, reusable helper abstractions, and shared web-automation/browser utilities - clean up and expand documentation so the root README flows into docs and skill docs, with clearer development, reviewer, installer, and workflow guidance ## Notable changes - docs flow and consistency cleanup across `README.md`, `docs/README.md`, and related docs - new scripts for `check`, docs verification, generated-file verification, shell portability, and safe directory replacement - refactors in Atlassian and web-automation skill runtimes to reduce duplication and centralize reusable code - changelog, development documentation, and CI surface updates ## Test Plan - [ ] `pnpm run check` - [ ] review generated/manifests and skill sync outputs - [ ] smoke-check docs flow from `README.md` to `docs/README.md` to skill docs ## Notes - this branch currently includes tracked `skills/web-automation/shared/node_modules` content that should be reviewed carefully as potentially noisy/accidental committed artifacts Co-authored-by: Stefano Fiorini <stefano.fiorini@firsthorizon.com> Reviewed-on: #1
55 lines
2.1 KiB
Bash
Executable File
55 lines
2.1 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
set -euo pipefail
|
|
|
|
WORKFLOW_FILES=(
|
|
"skills/create-plan/pi/SKILL.md"
|
|
"skills/do-task/pi/SKILL.md"
|
|
"skills/implement-plan/pi/SKILL.md"
|
|
)
|
|
|
|
# These terms cover the Codex-only constructs explicitly called out by the plan.
|
|
# Expand this deny-list if later pi workflow ports introduce new harness-specific tokens.
|
|
|
|
for file in "${WORKFLOW_FILES[@]}"; do
|
|
test -f "$file"
|
|
grep -q 'docs/PI-SUPERPOWERS.md' "$file"
|
|
# shellcheck disable=SC2016
|
|
grep -q 'Reviewer CLI: `codex`, `claude`, `cursor`, `opencode`, `pi`, or `skip`' "$file"
|
|
grep -q 'pi --no-session --no-skills --no-prompt-templates --no-extensions --no-context-files' "$file"
|
|
grep -q -- '--tools read,grep,find,ls -p' "$file"
|
|
grep -q 'pi --list-models \[search\]' "$file"
|
|
done
|
|
|
|
grep -q 'reviewer model is configured independently' docs/PI-COMMON-REVIEWER.md
|
|
grep -q 'provider-qualified model IDs' docs/PI-COMMON-REVIEWER.md
|
|
# shellcheck disable=SC2016
|
|
grep -q 'MUST NOT include `write`, `edit`, or `bash`' docs/PI-COMMON-REVIEWER.md
|
|
|
|
if command -v pi >/dev/null 2>&1; then
|
|
PI_HELP=$(pi --help 2>&1)
|
|
grep -q -- '--model <pattern>' <<<"$PI_HELP"
|
|
grep -q -- '--print, -p' <<<"$PI_HELP"
|
|
grep -q -- '--tools, -t <tools>' <<<"$PI_HELP"
|
|
grep -q -- '--no-session' <<<"$PI_HELP"
|
|
grep -q -- '--no-skills' <<<"$PI_HELP"
|
|
grep -q -- '--no-prompt-templates' <<<"$PI_HELP"
|
|
grep -q -- '--no-extensions' <<<"$PI_HELP"
|
|
grep -q -- '--no-context-files' <<<"$PI_HELP"
|
|
fi
|
|
|
|
test -f skills/create-plan/pi/templates/continuation-runbook.md
|
|
test -f skills/create-plan/pi/templates/milestone-plan.md
|
|
test -f skills/create-plan/pi/templates/story-tracker.md
|
|
test -f skills/do-task/pi/templates/task-plan.md
|
|
grep -q 'Reviewer CLI | codex \\| claude \\| cursor \\| opencode \\| pi' skills/do-task/pi/templates/task-plan.md
|
|
test -x skills/reviewer-runtime/pi/run-review.sh
|
|
test -x skills/reviewer-runtime/pi/notify-telegram.sh
|
|
|
|
# SC2251: restructured to avoid ! outside condition
|
|
if rg -n 'update_plan|plan mode|sub-agent|subagents' "${WORKFLOW_FILES[@]}"; then
|
|
echo "Error: pi workflow SKILL.md files must not contain Codex-specific terms" >&2
|
|
exit 1
|
|
fi
|
|
|
|
echo "pi workflow skill docs verified"
|