fix: OpenClaw integration

[abhisek]

---

[safedep]

SafeDep Report Summary

No dependency changes detected. Nothing to scan.

_{This report is generated by SafeDep Github App}

[codecov-commenter]

Codecov Report

❌ Patch coverage is 8.69565% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 10.80%. Comparing base (d4dda92) to head (38286f2).
⚠️ Report is 25 commits behind head on main.

Files with missing lines	Patch %	Lines
agent/openclaw/hooks.go	0.00%	20 Missing ⚠️
cli/root.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   10.80%   10.80%   -0.01%     
==========================================
  Files         194      194              
  Lines       22813    22836      +23     
==========================================
+ Hits         2466     2468       +2     
- Misses      20035    20056      +21     
  Partials      312      312              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

[devin-ai-integration]

Devin Review found 2 new potential issues.

View 5 additional findings in Devin Review.

[devin-ai-integration]

🟡 "Already installed" check returns early without writing new required manifest and package.json files

When InstallHooks detects that the plugin file (index.ts) already exists and contains "invokeGryph", it returns early at line 80 with a success warning, skipping the manifest and package.json writes at lines 127-137. This means that users upgrading from a previous gryph version (which didn't deploy these files) will get a misleading "already installed" success, while GetHookStatus (agent/openclaw/hooks.go:234-244) will report the installation as invalid due to the missing openclaw.plugin.json and package.json. The only workaround is gryph install --force, but users won't know this unless they run gryph status.

Suggested fix approach

The "already installed" check should also verify that the manifest and package.json exist. If they are missing, the install should proceed to write them (or at least the missing files) rather than returning early.

---

Was this helpful? React with 👍 or 👎 to provide feedback.

[devin-ai-integration]

🟡 UninstallHooks with RestoreBackup leaves manifest file behind

When UninstallHooks is called with RestoreBackup, it only restores the plugin file (index.ts) and returns early at line 160 without removing the gryph-specific manifest file (openclaw.plugin.json). This leaves a gryph artifact in the plugin directory even though the intent of restore-uninstall is to return to the pre-gryph state. The manifest was never backed up during install (only the plugin file was), so there's nothing to restore for it — it should simply be removed.

---

Was this helpful? React with 👍 or 👎 to provide feedback.