fix(gateway): reduce ClawX context merge waits; bundle memory-core chokidar#930
fix(gateway): reduce ClawX context merge waits; bundle memory-core chokidar#930
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fcda3d6562
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| const isMainDefault = | ||
| agent?.default === true || (agent?.id === 'main' && !hasDefaultWorkspace); |
There was a problem hiding this comment.
Treat first agent as default when no explicit default exists
The new startup-wait classification only marks a workspace as retryable when agent.default === true or the id is main, but the rest of the codebase treats the first entry in agents.list as the default when no explicit default is set (see normalizeAgentsConfig). In configs that rely on that positional default behavior (no agents.defaults.workspace, no default: true), this branch misclassifies the real default workspace as non-retryable, so ensureClawXContext can return before Gateway seeding finishes and skip ClawX context merge/cleanup for the active default agent.
Useful? React with 👍 / 👎.
No description provided.