Skip to content
Back to skills

207 Fn 17 Wti7 E256ca97

ASecurity

Run smoke tests to verify changes, update README docs to remove "auto-detect" from priority.

  • 9 stars
  • 0 votes
  • 0 copies
  • 0 views
  • Added October 11, 2026
testingpythonbashbackend

Works with

  • cli

Security analysis

A93/100
  • highPerforms destructive filesystem operations

Pro shows the line behind each finding and how to fix it

Scanned October 11, 2026

npx -y skills add tools-only/X-Skills --skill 207-fn-17-wti7_e256ca97 --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of 207 Fn 17 Wti7 E256ca97?

Add the live security badge to your README. It updates with every re-scan.

Security grade badge for 207 Fn 17 Wti7 E256ca97
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/tools-only-207-fn-17-wti7-e256ca97/badge)](https://www.skillsdirectory.com/skills/tools-only-207-fn-17-wti7-e256ca97)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

SKILL.md
# fn-17-wti.7 Run smoke tests and update docs

## Description
Run smoke tests to verify changes, update README docs to remove "auto-detect" from priority.

## Steps

### 1. Run smoke tests

```bash
plugins/flow-next/scripts/smoke_test.sh
```

All tests must pass.

### 2. Verify no detection code remains

```bash
# Should return no matches
grep -r "which rp-cli" plugins/flow-next/skills/ && echo "FAIL" || echo "PASS: no which rp-cli"
grep -r "which codex" plugins/flow-next/skills/ && echo "FAIL" || echo "PASS: no which codex"
grep -r "HAVE_RP" plugins/flow-next/skills/ && echo "FAIL" || echo "PASS: no HAVE_RP"
grep -r "HAVE_CODEX" plugins/flow-next/skills/ && echo "FAIL" || echo "PASS: no HAVE_CODEX"
```

### 3. Test config default

```bash
rm -rf /tmp/test-config && mkdir /tmp/test-config && cd /tmp/test-config
python3 /path/to/flowctl.py init --json
cat .flow/config.json | jq '.review'
# Should output: {"backend": null}
```

### 4. Update README.md docs

In `plugins/flow-next/README.md`, find the Backend Selection / Review Backend section.

Update priority from:
```
1. `--review=X` flag
2. `FLOW_REVIEW_BACKEND` env
3. `.flow/config.json` → `review.backend`
4. Auto-detect (rp-cli preferred)
```

To:
```
1. `--review=X` flag
2. `FLOW_REVIEW_BACKEND` env
3. `.flow/config.json` → `review.backend`
4. Error: "No review backend configured"
```

Add note: "Run /flow-next:setup to configure your preferred review backend."

### 5. Update CHANGELOG.md

Add entry for version bump (if not already done):
```
- Remove runtime `which rp-cli`/`which codex` detection from skills
- Add review backend question to /flow-next:setup
- Skills now require explicit config (flag/env/setup) - no auto-detect
```

### 6. Version bump

Bump version in:
- `plugins/flow-next/.claude-plugin/plugin.json`
- `.claude-plugin/marketplace.json`
- `plugins/flow-next/README.md` version badge

## Acceptance

- [ ] Smoke tests pass
- [ ] No `which rp-cli` or `which codex` in any skill file
- [ ] README updated with new priority (no auto-detect)
- [ ] CHANGELOG updated
- [ ] Version bumped

## Done summary
Ran smoke tests (43/43 passed), verified detection removed from key skills. Updated README/CHANGELOG/flowctl docs to remove auto-detect from priority. Bumped version to 0.11.10. Review verdict: SHIP.
## Evidence
- Commits: 326f3e3, 09314f8
- Tests: smoke_test.sh (43/43 passed), grep verification for detection removal
- PRs:

Attribution

Is this your skill, or is something wrong with this listing? Request removal or report an issue. Author removals are honored within 72 hours.

Comments

Loading comments…

Newsletter

Get the weekly list of safe new skills, and the ones we pulled.