fix: load config in project with no dirs

[uinstinct]

Description

Wait for cascadeInit to finish before getting the serialized config. This previously created a race condition where the current profile and config were still null and not loaded.

Also in ParallelListeners, do not update config till configLoadInterrupted returns true.

resolves CON-5299
closes #9587

AI Code Review

• Team members only: AI review runs automatically when PR is opened or marked ready for review
• Team members can also trigger a review by commenting @continue-review

Checklist

• [] I've read the contributing guide
• [] The relevant docs, if any, have been updated or created
• [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]

---

Continue Tasks: ❌ 7 failed — View all

---

Summary by cubic

Fix a config loading race in projects without directories by waiting for initialization before reading the serialized config and skipping updates when the load is interrupted (CON-5299).

• Bug Fixes
  • ConfigHandler: await initialization before returning serialized config to avoid null profile/config.
  • ParallelListeners: do not update config when configLoadInterrupted is true.

^{Written for commit 409a945. Summary will update on new commits.}

[haystack-code-reviewer-pr-hook]

Try Haystack Code Reviewer

Want AI-powered code review for this PR? Get instant analysis, interactive visualizations, and actionable insights.

Review this PR with Haystack

[cubic-dev-ai]

1 issue found across 2 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="core/config/ConfigHandler.ts">

<violation number="1" location="core/config/ConfigHandler.ts:595">
P1: Deadlock: getSerializedConfig awaits isInitialized but when currentProfile is null, reloadConfig returns early without emitting the "init" event, causing isInitialized to never resolve. This will hang callers on projects without profiles/directories.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}