Skip to content
Back to skills

245 Fn 17 Wti1 B841b151

ASecurity

Add `review.backend` to the default config structure in `flowctl.py`.

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

Security analysis

A93/100
  • highPerforms destructive filesystem operations

Pro scans all 2 files and shows the line behind each finding

Scanned October 11, 2026

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

Installs into .claude/skills of the current project.

Are you the author of 245 Fn 17 Wti1 B841b151?

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

Security grade badge for 245 Fn 17 Wti1 B841b151
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/tools-only-245-fn-17-wti1-b841b151/badge)](https://www.skillsdirectory.com/skills/tools-only-245-fn-17-wti1-b841b151)

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

Download with Pro
SKILL.md
# fn-17-wti.1 Add review.backend to flowctl default config

## Description
Add `review.backend` to the default config structure in `flowctl.py`.

## File to modify

`plugins/flow-next/scripts/flowctl.py`

## Changes

1. Find `get_default_config()` function (around line 76-78)
2. Add `review` key with `backend` subkey:

```python
def get_default_config() -> dict:
    """Return default config structure."""
    return {
        "memory": {"enabled": False},
        "planSync": {"enabled": False},
        "review": {"backend": None}
    }
```

3. The `None` value means "not configured yet" - skills will error if this is None and no flag/env provided.

## Why None instead of "auto"

- Explicit is better than implicit
- Forces user to make a choice via setup or flag
- Prevents LLM deviation from "auto-detect" logic

## Test

```bash
# After change, init a fresh .flow/ and check config
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}
```
## Acceptance
- [ ] `get_default_config()` returns dict with `review.backend` key
- [ ] Value is `None` (Python None, becomes JSON null)
- [ ] Existing configs upgraded via `flowctl init` get the new key merged in
- [ ] Smoke tests pass
## Done summary
Added review.backend key with None value to get_default_config() in flowctl.py. Existing configs get the new key via deep_merge on upgrade.
## Evidence
- Commits: 3a3248c906b97a0ef66dd9ed3b3a41b487f73925
- Tests: flowctl init --json (fresh dir), config get review.backend (upgrade test), smoke_test.sh (43 tests pass)
- PRs:

Files in this skill

  • README.md810 B
  • skill.md1.6 KB

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.