Skip to content

Remove all FUSE/CLI references from hivemind plugin#37

Open
kaghni wants to merge 1 commit intomainfrom
fix/remove-fuse-references
Open

Remove all FUSE/CLI references from hivemind plugin#37
kaghni wants to merge 1 commit intomainfrom
fix/remove-fuse-references

Conversation

@kaghni
Copy link
Copy Markdown
Collaborator

@kaghni kaghni commented Apr 11, 2026

Hivemind plugin should be independent from deeplake-cli. Removed CLI detection, FUSE mount checks, install prompts. Unsafe commands now denied with safe alternatives message.

Hivemind plugin is independent from deeplake-cli. Removed:
- CLI installation detection (which deeplake, path checks)
- FUSE mount active check
- CLI install/mount prompts
- deeplake CLI command passthrough

Unsafe commands now simply denied with suggestion to use safe
builtins (cat, ls, grep, jq, etc.) that work natively.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@claude
Copy link
Copy Markdown

claude bot commented Apr 11, 2026

Claude finished @kaghni's task in 1m 43s —— View job


Review Complete

The changes are correct and well-scoped. No significant issues found.

  • Read changed files
  • Analyze logic and correctness

The simplification is clean: the unsafe-command path in main() now always denies with a "use safe builtins" message, removing the CLI detection / FUSE mount branching. The getShellCommand Bash case correctly returns null for unsafe commands, which triggers the deny block via touchesMemory(cmd), while safe commands get rewritten and routed to the virtual shell as before. The execFileSync import removal is correct. Bundle and source are updated in sync.

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