Skip to content

Guard repeated graph-memory LLM failures - #43

Merged
adoresever merged 1 commit into
adoresever:mainfrom
junyangyan4-svg:codex/llm-failure-guard
Aug 1, 2026
Merged

Guard repeated graph-memory LLM failures#43
adoresever merged 1 commit into
adoresever:mainfrom
junyangyan4-svg:codex/llm-failure-guard

Conversation

@junyangyan4-svg

Copy link
Copy Markdown

Summary

  • add LlmFailureGuard for permanent graph-memory LLM API failures
  • pause repeated calls after permanent 4xx responses such as 403/404/422
  • reset the guard immediately after a successful LLM call

Why

This turns a runaway failure mode into a bounded cooldown when the configured memory LLM is misconfigured or unavailable.

Split from #39.

Verification

  • npm test
  • npx tsc --noEmit

@junyangyan4-svg
junyangyan4-svg marked this pull request as ready for review April 6, 2026 05:35
@adoresever
adoresever merged commit 0d30e33 into adoresever:main Aug 1, 2026

@adoresever adoresever left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

核心方案已采纳并合并:连续配置类错误的冷却保护是必要的。合并时把触发范围收窄为持久配置错误(401/403/404),避免 400/422 等请求级错误错误地停用 LLM,并补充了测试。感谢贡献。

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