Skip to content

fix: Make CLAUDE.md update fire-and-forget to prevent HTTP timeouts#413

Merged
rjroy merged 1 commit intomainfrom
fix/vault-setup
Jan 25, 2026
Merged

fix: Make CLAUDE.md update fire-and-forget to prevent HTTP timeouts#413
rjroy merged 1 commit intomainfrom
fix/vault-setup

Conversation

@rjroy
Copy link
Copy Markdown
Owner

@rjroy rjroy commented Jan 25, 2026

Summary

  • CLAUDE.md update via SDK now runs in background without blocking setup completion
  • Prevents "Failed fetch" errors caused by 30-60+ second SDK calls exceeding browser timeouts
  • Success/failure is logged asynchronously; marker shows claudeMdUpdated: false since result is unknown at write time

Test plan

  • Unit tests updated to reflect new fire-and-forget behavior
  • Pre-commit checks pass
  • Manual test: trigger vault setup via UI, verify no timeout error

🤖 Generated with Claude Code

The SDK call in updateClaudeMd() can take 30-60+ seconds, which exceeds
browser fetch timeouts and causes "Failed fetch" errors on the frontend.
Now runs in background with success/failure logged asynchronously.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 25, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/src/vault-setup.ts 83.33% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rjroy rjroy merged commit 6ebde95 into main Jan 25, 2026
2 checks passed
@rjroy rjroy deleted the fix/vault-setup branch January 25, 2026 04:58
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