Skip to content

Plugin-first cleanup: migrate plugin off legacy hook shelling#178

Merged
G9Pedro merged 3 commits intomasterfrom
cursor/clawvault-plugin-architecture-4e68
Mar 16, 2026
Merged

Plugin-first cleanup: migrate plugin off legacy hook shelling#178
G9Pedro merged 3 commits intomasterfrom
cursor/clawvault-plugin-architecture-4e68

Conversation

@G9Pedro
Copy link
Copy Markdown
Contributor

@G9Pedro G9Pedro commented Mar 16, 2026

Summary

  • migrated plugin context injection (vault-context-injector.ts) from CLI subprocess calls to in-process library usage (ClawVault.find() + buildSessionRecap()), removing pre-prompt shell execution
  • migrated session lifecycle recover/checkpoint handling to in-process command-library calls in session-lifecycle.ts
  • kept observer as CLI-based execution but switched plugin observer trigger path to non-blocking spawn instead of synchronous execFileSync
  • removed legacy hooks/clawvault/ handler tree and dead packages/plugin/ scripts package
  • added .npmignore exclusions for non-publish directories and updated release metadata (package.json, openclaw.plugin.json, CHANGELOG.md) for 3.5.0
  • added targeted plugin tests for the new in-process context/lifecycle paths

Validation

  • npm run build
  • npx vitest run src/plugin/vault-context-injector.test.ts src/plugin/hooks/session-lifecycle.test.ts
  • npm test ✅ (120 files, 745 tests passing)
  • npm publish --dry-run ✅ (clawvault@3.5.0)

Notes

  • no changes were made to memory_search / memory_get tool registration
  • plugin config schema and communication protocol behavior were preserved
Open in Web Open in Cursor 

cursoragent and others added 3 commits March 16, 2026 01:50
Co-authored-by: G9Pedro <G9Pedro@users.noreply.github.com>
Co-authored-by: G9Pedro <G9Pedro@users.noreply.github.com>
Co-authored-by: G9Pedro <G9Pedro@users.noreply.github.com>
@G9Pedro G9Pedro marked this pull request as ready for review March 16, 2026 02:16
@G9Pedro G9Pedro merged commit 35eed50 into master Mar 16, 2026
2 checks passed
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.

2 participants