P3-04: Plugin requires_env lists DECIDER_URL but it has a default — not actually required #22

New issue

Closed

opened 2026-06-16 13:57:02 +00:00 by Artur · 0 comments

Artur commented

2026-06-16 13:57:02 +00:00

Owner

Severity: P3 (Low)
File: ~/.hermes/plugins/decider/plugin.yaml line 9

Problem

requires_env:
  - DECIDER_URL

The code has a hardcoded default: _DEFAULT_DECIDER_URL = "http://localhost:8655". So DECIDER_URL is NOT required — it falls back gracefully.

Fix

Either:

Remove from requires_env (it's not required)
Or remove the default so it IS required (for non-standard installations)

**Severity**: P3 (Low) **File**: `~/.hermes/plugins/decider/plugin.yaml` line 9 ## Problem ```yaml requires_env: - DECIDER_URL ``` The code has a hardcoded default: `_DEFAULT_DECIDER_URL = "http://localhost:8655"`. So `DECIDER_URL` is NOT required — it falls back gracefully. ## Fix Either: 1. Remove from `requires_env` (it's not required) 2. Or remove the default so it IS required (for non-standard installations)

Artur added the

enhancement

P3-Low

labels

2026-06-16 13:57:02 +00:00

Artur referenced this issue from a commit

2026-06-16 13:59:15 +00:00

fix: P2-01, P2-04, P3-01, P3-02, P3-03, P3-04, P3-05

Artur closed this issue

2026-06-16 14:24:07 +00:00