chore(opencode): fix drifted agent/skill docs and repo hygiene #283

Merged
TapTap merged 2 commits from chore/fix-agent-skill-docs into dev 2026-09-13 10:44:57 +02:00
Owner

What

The .opencode/ agent and skill definitions had drifted far from the current codebase and tooling — the same class of bug as the benchmark tool (references to nonexistent scripts and invented flags).

Agent/skill fixes

  • Replaced the removed python3 test.py with the real integration command (python3 -m pytest tests/integration/ -n 4 --dist=load -m "not setpriv").
  • feature-scout.md: replaced an almost entirely fabricated flag list (--host, --server-mode, --use-compression -c, --use-sendfile -s, …) with flags derived from src/client/usage.c.
  • perf-analyst.md: -m -c → -j -z, point at benchmark/bench.py, drop stale numbers.
  • code-explainer.md: no getopt_long (custom OPTION_TABLE parser); --sendfile not -f; TCP server needs --allow-unauthenticated.
  • release/SKILL.md: version 1.1.0 → 2.20.0; no direct push to main (dev→main PR).
  • Replaced GitHub/gh flows with Gitea/tea across all agents ("Branch Strategy" x16, issue-creator).
  • Sanitizer blocks now use -DSANITIZER=address|thread instead of hand-rolled -fsanitize flags.
  • Plain-TCP server examples gained -p 8080 --allow-unauthenticated.
  • Consolidated redundant security-screener into security-auditor (16 agents remain); AGENTS.md count updated.

Repo hygiene

  • .gitignore: added root/ and test_partial_install_tmp/.
  • Removed empty leftover trees root/ and test_partial_install_tmp/.
  • Deleted tracked scratch scripts tmux.sh and to_one_file.py.
  • README note for the compile_commands.json symlink.

Verification

  • Greps: gh pr create, gh issue, python3 test.py, security-screener, -m -c, -fsanitize in skills, tests/pytest/, 17 specialized → all zero matches.
  • 16 agent + 7 skill files: valid frontmatter, balanced code fences.
  • Docs/config only; no C source or test changes.
## What The `.opencode/` agent and skill definitions had drifted far from the current codebase and tooling — the same class of bug as the benchmark tool (references to nonexistent scripts and invented flags). ### Agent/skill fixes - Replaced the removed `python3 test.py` with the real integration command (`python3 -m pytest tests/integration/ -n 4 --dist=load -m "not setpriv"`). - `feature-scout.md`: replaced an almost entirely fabricated flag list (`--host`, `--server-mode`, `--use-compression -c`, `--use-sendfile -s`, …) with flags derived from `src/client/usage.c`. - `perf-analyst.md`: `-m -c` → `-j -z`, point at `benchmark/bench.py`, drop stale numbers. - `code-explainer.md`: no `getopt_long` (custom `OPTION_TABLE` parser); `--sendfile` not `-f`; TCP server needs `--allow-unauthenticated`. - `release/SKILL.md`: version `1.1.0` → `2.20.0`; no direct push to `main` (dev→main PR). - Replaced GitHub/`gh` flows with Gitea/`tea` across all agents ("Branch Strategy" x16, issue-creator). - Sanitizer blocks now use `-DSANITIZER=address|thread` instead of hand-rolled `-fsanitize` flags. - Plain-TCP server examples gained `-p 8080 --allow-unauthenticated`. - Consolidated redundant `security-screener` into `security-auditor` (16 agents remain); `AGENTS.md` count updated. ### Repo hygiene - `.gitignore`: added `root/` and `test_partial_install_tmp/`. - Removed empty leftover trees `root/` and `test_partial_install_tmp/`. - Deleted tracked scratch scripts `tmux.sh` and `to_one_file.py`. - README note for the `compile_commands.json` symlink. ## Verification - Greps: `gh pr create`, `gh issue`, `python3 test.py`, `security-screener`, `-m -c`, `-fsanitize` in skills, `tests/pytest/`, `17 specialized` → all zero matches. - 16 agent + 7 skill files: valid frontmatter, balanced code fences. - Docs/config only; no C source or test changes.
TapTap added 1 commit 2026-09-13 10:35:09 +02:00
chore(opencode): fix drifted agent/skill docs and repo hygiene
CI / lint (pull_request) Successful in 1m29s
CI / sanitizers (address) (pull_request) Skipped
CI / sanitizers (undefined) (pull_request) Skipped
CI / fuzz-build (pull_request) Skipped
CI / coverage (pull_request) Skipped
CI / valgrind (pull_request) Skipped
CI / build-and-test (pull_request) Successful in 1m45s
5c8970c64f
The agent and skill definitions had drifted badly from the current
codebase and tooling, repeating the same class of bug as the benchmark
tool (references to nonexistent scripts and invented flags):

- Replace the removed `python3 test.py` with the real integration
  command (`python3 -m pytest tests/integration/ -n 4 --dist=load
  -m "not setpriv"`) across agents and skills.
- Fix `feature-scout`'s fabricated CLI flag list (--host, --server-mode,
  --use-* etc.) using the authoritative src/client/usage.c flags.
- Fix `perf-analyst` benchmark flags (-m -c -> -j -z) and point at
  benchmark/bench.py instead of stale numbers.
- Correct `code-explainer` (no getopt_long; --sendfile not -f) and
  version drift in the release skill (1.1.0 -> 2.20.0).
- Replace GitHub/`gh` workflows with Gitea/`tea` (PRs target dev; issues
  via tea; branch strategy updated in all agents).
- Use the built-in `-DSANITIZER=address|thread` CMake option instead of
  hand-rolled -fsanitize flags.
- Add `-p 8080 --allow-unauthenticated` to plain-TCP server examples.
- Merge the redundant security-screener into security-auditor; drop the
  duplicate (16 agents remain).

Repo hygiene: gitignore `root/` and `test_partial_install_tmp/`, remove
the empty leftover trees, delete the tracked scratch scripts tmux.sh and
to_one_file.py, and note the compile_commands.json symlink in README.
TapTap added 1 commit 2026-09-13 10:41:28 +02:00
fix(opencode): correct remaining agent/skill doc drift
CI / lint (pull_request) Successful in 1m31s
CI / sanitizers (address) (pull_request) Skipped
CI / sanitizers (undefined) (pull_request) Skipped
CI / fuzz-build (pull_request) Skipped
CI / coverage (pull_request) Skipped
CI / valgrind (pull_request) Skipped
CI / build-and-test (pull_request) Successful in 1m44s
0f95f48899
- pr-review: replace invalid 'tea pr comment' with 'tea comment' (the
  former is not a tea subcommand)
- integrator: drop stray '-M' from client examples (-M is now
  --remote-option and requires an argument), use the canonical pytest
  integration command, and bump the CI image tag to v10
- test-writer: build fuzz targets via -DENABLE_FUZZ=ON instead of
  hand-rolled -fsanitize flags; fix the fuzz binary path
- cmake-expert: document -DSANITIZER=undefined, which is now live in
  CMakeLists.txt
- README: add --allow-unauthenticated to the plain-TCP server example,
  use --preserve for metadata (not -M), and use the canonical
  integration command
- AGENTS.md: use the canonical integration command
TapTap merged commit 264964411c into dev 2026-09-13 10:44:57 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: TapTap/FastSync#283