Skip to content

Commit 6dea1d7

Browse files
fix: skills catalog + wizard lightrag provider coverage
Two Devin Review findings addressed: 1. skills/README.md catalog listed only 9 skills; PR added 4 new ones (daily-inbox-triage, hermes-weekly, spam-trap, meeting-prep). Appended rows for all 4 so the catalog matches the 13-skill count now quoted in README / CHANGELOG / outreach docs. 2. Wizard generator: when memory=lightrag is selected, the generated config references google/gemini-2.5-flash (LLM) and openai/text-embedding-3-small (embedding), but the provider-emission set only accounted for the default-model provider + routing-rule providers. For 3 of 5 personas this produced broken configs with missing embedding provider credentials. Added a lightrag branch that injects google + openai into providersToEmit. Co-Authored-By: Rob <onerobby@gmail.com>
1 parent 6eef216 commit 6dea1d7

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

docs/wizard/index.html

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -318,6 +318,10 @@ <h1>Hermes Config Wizard</h1>
318318
};
319319
const providersToEmit = new Set([model.split('/')[0]]);
320320
(EXTRA_PROVIDERS[persona] || []).forEach(p => providersToEmit.add(p));
321+
// LightRAG uses google/gemini-2.5-flash (LLM) + openai/text-embedding-3-small
322+
// (embedding) — make sure both providers are configured.
323+
if (memory === 'lightrag') { providersToEmit.add('google'); providersToEmit.add('openai'); }
324+
// mem0 is a cloud service with its own key; no extra LLM provider needed here.
321325
[...providersToEmit].forEach(p => {
322326
const envVar = providerKeys[p] || 'API_KEY';
323327
lines.push(` ${p}:`);

skills/README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,10 @@ hermes /reload
3838
| **ops** | `telegram-triage` | Classifies inbound Telegram DMs, autoreplies low-stakes, escalates high-stakes to you |
3939
| **dev** | `pr-review` | Delegates a PR review to Claude Code with a scoped read-only GitHub PAT |
4040
| **dev** | `release-notes` | Builds a human-readable release note from a range of commits or merged PRs |
41+
| **dev** | `meeting-prep` | Pulls context for an upcoming meeting (calendar + notes + recent threads) into a briefing |
42+
| **ops** | `daily-inbox-triage` | Morning pass across Telegram/email/Slack DMs — classifies, autoreplies, escalates |
43+
| **ops** | `hermes-weekly` | Weekly digest of your Hermes activity (skills run, cost, errors, upcoming crons) |
44+
| **security** | `spam-trap` | Sinks confirmed spam into a quarantine profile with no memory write / no MCP access |
4145

4246
## Contributing
4347

0 commit comments

Comments
 (0)