Rohit Ghumare comments

Results 224 comments of


                                            Rohit Ghumare

Add validated flow helper proof of concept

@abishekgiri hey, how's it going? Is this PR up for review for @sergiofilhowz and @andersonleal

fix(cli): skip onboarding prompts without a tty

Merged + shipping in next release. Thanks @honor2030 — clean fix for non-tty onboarding interruption (CI / pipe / docker run -d). Regression test covers TTY-restoring sandbox.

Fix vector index Buffer serialization

Thanks @NikolaiL — during bug-fix triage we landed an equivalent fix in #683. Closing this as superseded; the detailed write-up in your description was really helpful for confirming the symptom.

fix(hooks): quote plugin script paths

Merged + shipping in next release. Thanks @honor2030 — closes #477 (hooks crashing on Windows usernames with spaces). The path-quoting fix + the regression test in test/codex-plugin.test.ts both look clean.

feat(retention): schedule evict sweeps

Hey @honor2030 — deferring this from v0.9.21 against the minimal-surface rule. The PR is well-built: timer.unref() so it never blocks shutdown, EVICTION_ENABLED env toggle, configurable interval, and tracked issue #480...

fix(mcp): preserve recall format option

Closed as superseded by #516 (merged today), which is the more complete fix: - Routes `memory_recall` through `/agentmemory/search` (this PR still routed through `/smart-search`) - Forwards both `format` and `token_budget`...

buildEmbeddingUrl / buildChatUrl hardcodes /v1/ breaking non-OpenAI providers

Fixed in v0.9.22 — https://github.com/rohitg00/agentmemory/releases/tag/v0.9.22. Thanks for the report.

fix: read Claude Code tool_response in PostToolUse hook

Shipped in v0.9.22 via maintainer PR — https://github.com/rohitg00/agentmemory/releases/tag/v0.9.22. Thanks for the contribution.

fix cross-project memory leakage

Please fix coderabbit issues @abhinav-m22

fix cross-project memory leakage

Very good work @abhinav-m22