Skip to content

Add Midas Skill β€” self-learning wealth extraction engine#399

Closed
hermesnest wants to merge 1 commit intoVoltAgent:mainfrom
hermesnest:add-midas-skill
Closed

Add Midas Skill β€” self-learning wealth extraction engine#399
hermesnest wants to merge 1 commit intoVoltAgent:mainfrom
hermesnest:add-midas-skill

Conversation

@hermesnest
Copy link
Copy Markdown

@hermesnest hermesnest commented Apr 9, 2026

Midas Skill extracts wealth operating systems from any public figure or personal data across 6 dimensions: Money Engine, Deal Architecture, Investment Thesis, Risk Profile, Leverage Model, Exit Pattern. Self-learning β€” gets sharper with every input. Open source, MIT licensed.

ClawHub: https://clawhub.ai/realteamprinz/midas
GitHub: https://github.com/realteamprinz/midas-skill

Summary by CodeRabbit

  • Documentation
    • Added a new entry to the Personal Development section in README documentation.

Added to Personal Development category.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

πŸ“ Walkthrough

Walkthrough

README.md was updated to include a new "midas" entry in the Personal Development category list with its ClawHub link and descriptive text. No other modifications to structure or existing items were made.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added a new "midas" entry to the Personal Development category list.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

weekly-merge-queue

Poem

🐰 Midas shines bright in the list, so grand,
A personal tool, now close at hand,
The README grows, one entry more,
Development paths we all adore! ✨

πŸš₯ Pre-merge checks | βœ… 3
βœ… Passed checks (3 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check βœ… Passed The pull request title 'Add Midas Skill β€” self-learning wealth extraction engine' directly and specifically summarizes the main change: adding a new entry (Midas Skill) to the README with its key characteristic (self-learning wealth extraction engine). The title is concise, clear, and accurately reflects the changeset.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Warning

⚠️ This pull request might be slop. It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

188-188: ⚠️ Potential issue | 🟑 Minor

Update the category count.

The Personal Development category count should be updated to reflect the addition of the new skill. The "View all" text at line 823 shows 51 skills, but the Table of Contents still shows (50).

πŸ”’ Proposed fix to update the count
-| [CLI Utilities](`#cli-utilities`) (180) | [Personal Development](`#personal-development`) (50) | [Gaming](`#gaming`) (35) |
+| [CLI Utilities](`#cli-utilities`) (180) | [Personal Development](`#personal-development`) (51) | [Gaming](`#gaming`) (35) |
πŸ€– Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 188, Update the Table of Contents entry for Personal
Development to reflect the new total of 51 items: find the string "[Personal
Development](`#personal-development`) (50)" and change the numeric count to "(51)"
so it matches the "View all" count shown elsewhere.
πŸ€– Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 821: The README entry for "midas" violates guidelines: replace the
ClawHub link with the canonical GitHub URL for the skill (reference the "midas"
entry text), confirm the repository exists in the openclaw/skills repo before
keeping it, and if it is finance-related remove or reframe it to a non-finance
use-case; also shorten the description to 10 words or fewer (or drop the entry
if it cannot be reframed), and update the Personal Development TOC count from
(50) to (51) accordingly.

---

Outside diff comments:
In `@README.md`:
- Line 188: Update the Table of Contents entry for Personal Development to
reflect the new total of 51 items: find the string "[Personal
Development](`#personal-development`) (50)" and change the numeric count to "(51)"
so it matches the "View all" count shown elsewhere.
πŸͺ„ Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
βš™οΈ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6f3f08a5-b1b3-40a2-9045-31f92096d1f2

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 400ce91 and 1c3736d.

πŸ“’ Files selected for processing (1)
  • README.md

Comment thread README.md
- [ezbookkeeping](https://clawskills.sh/skills/mayswind-ezbookkeeping) - ezBookkeeping is a lightweight, self-hosted personal finance app.
- [fix-life-in-1-day](https://clawskills.sh/skills/evgyur-fix-life-in-1-day) - Fix your entire life in 1 day.
- [founder-coach](https://clawskills.sh/skills/goforu-founder-coach) - AI-powered startup mindset coach that helps founders upgrade.
- [midas](https://clawhub.ai/realteamprinz/midas) - Self-learning wealth extraction engine. Feed it anyone β€” it extracts their wealth operating system across 6 dimensions. Open source, self-learning.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | πŸ”΄ Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Verify link domain consistency and check if skill exists in official repo

# Check the link domain pattern used by other skills in Personal Development section
echo "=== Checking link domains in Personal Development section ==="
rg -A 1 "Personal Development" README.md | rg -oP 'https://[^)]+' | head -20

# Search for any existing clawhub.ai links in the file
echo -e "\n=== Searching for clawhub.ai links ==="
rg 'clawhub\.ai' README.md

# Search for clawskills.sh links (expected pattern)
echo -e "\n=== Count of clawskills.sh links ==="
rg -c 'clawskills\.sh' README.md

Repository: VoltAgent/awesome-openclaw-skills

Length of output: 764


🏁 Script executed:

#!/bin/bash
# Verify key claims from the original review

# 1. Check the actual contributing guidelines (line 93)
echo "=== Contributing guidelines around line 93 ==="
sed -n '85,100p' README.md

# 2. Check the TOC count at line 188
echo -e "\n=== Table of Contents line 188 ==="
sed -n '185,190p' README.md

# 3. Count personal development entries
echo -e "\n=== Count of Personal Development entries ==="
sed -n '790,830p' README.md | rg '^\- \[' | wc -l

# 4. Check if there's a GitHub link mentioned in context
echo -e "\n=== Check context around midas entry ==="
sed -n '815,825p' README.md

Repository: VoltAgent/awesome-openclaw-skills

Length of output: 2768


🏁 Script executed:

#!/bin/bash
# Check if there's a way to verify midas is in the official OpenClaw skills repo

# Search for any references to the official skills repo structure
echo "=== Checking CONTRIBUTING.md for repo structure ==="
head -50 CONTRIBUTING.md

# Look for any automation or PR verification that might show if skill was added to official repo
echo -e "\n=== Search for any GitHub workflow or CI config ==="
fd -e "yml\|yaml" | grep -i "workflow\|action" | head -5

Repository: VoltAgent/awesome-openclaw-skills

Length of output: 2728


Multiple guideline violations: link format, word limit, and potentially finance-restricted category.

The midas entry violates several requirements per CONTRIBUTING.md:

  1. Incorrect link format: The entry uses only the ClawHub link, but per contributing guidelines, the GitHub link should be primary: https://github.com/openclaw/skills/tree/main/skills/realteamprinz/midas. The ClawHub link is for the PR description only.

  2. Description exceeds word limit: The description is ~25 words ("Self-learning wealth extraction engine. Feed it anyone β€” it extracts their wealth operating system across 6 dimensions. Open source, self-learning."), but guidelines require concise descriptions of 10 words or fewer.

  3. Finance-related skill in excluded category: The skill's description focuses on "wealth extraction," which appears finance-related. Contributing guidelines explicitly state: "No crypto, blockchain, DeFi, or finance-related skills for now."

  4. Unverified repository: Guidelines require the skill to already be published in the official OpenClaw skills repo. Confirm the midas skill exists there before merging.

  5. TOC count not updated: Table of Contents shows (50) for Personal Development, but should show (51) after adding this entry.

πŸ€– Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 821, The README entry for "midas" violates guidelines:
replace the ClawHub link with the canonical GitHub URL for the skill (reference
the "midas" entry text), confirm the repository exists in the openclaw/skills
repo before keeping it, and if it is finance-related remove or reframe it to a
non-finance use-case; also shorten the description to 10 words or fewer (or drop
the entry if it cannot be reframed), and update the Personal Development TOC
count from (50) to (51) accordingly.

@hermesnest hermesnest closed this by deleting the head repository Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant