Skip to content

fix: hefestoai-auditor — remove security scanner triggers#131

Closed
artvepa80 wants to merge 1 commit intoopenclaw:mainfrom
artvepa80:main
Closed

fix: hefestoai-auditor — remove security scanner triggers#131
artvepa80 wants to merge 1 commit intoopenclaw:mainfrom
artvepa80:main

Conversation

@artvepa80
Copy link

Summary

  • Remove all home directory references (~, /home/user/, dotfiles)
  • Remove all 2>/dev/null error suppression patterns
  • Remove Multi-Model Architecture section (not part of pip package)
  • Remove Stripe checkout URLs (link to website instead)
  • Remove Wrapper Script section
  • Remove env var export instructions
  • Remove Socratic Constitution details
  • Add explicit privacy statement (all analysis runs locally)
  • Add explicit permissions statement (read-only, no code modification)
  • Fix version inconsistency (title now matches metadata v2.1.0)
  • Simplify Quick Start to: pip install + hefesto analyze

Why

The skill page has 587 views but 0 installs. Security scanners flag the skill as Suspicious due to patterns like home directory access, error suppression, and references to external AI services. This rewrite eliminates all triggers while preserving full functionality.

- Remove all home directory references (~, /home/user/, dotfiles)
- Remove all 2>/dev/null error suppression patterns
- Remove Multi-Model Architecture section (not part of pip package)
- Remove Stripe checkout URLs (link to website instead)
- Remove Wrapper Script section
- Remove env var export instructions
- Remove Socratic Constitution details
- Add explicit privacy statement (all analysis runs locally)
- Add explicit permissions statement (read-only, no code modification)
- Fix version inconsistency (title matches metadata v2.1.0)
- Simplify Quick Start to: pip install + hefesto analyze
@openclaw-barnacle
Copy link

Thanks for the pull request! This repository is read-only and is automatically synced from https://clawhub.ai, so we can’t accept changes here. Please make updates on the website instead.

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