test(api): stabilize sparse state query coverage#99
Conversation
Co-Authored-By: Duyet Le <me@duyet.net> Co-Authored-By: duyetbot <bot@duyet.net>
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates project documentation and configuration for automation runs and sandboxed environments, including updates to AGENTS.md, CLAUDE.md, and docs/knowledge/core-memory.md. Key changes include modifying the dashboard build command and increasing the timeout for a sparse query test in packages/api/test/state-platform.test.ts. Feedback was provided to optimize the test performance by parallelizing state seeding.
| for (let index = 0; index < 55; index++) { | ||
| await putState(`sparse-capped-miss-${index}`, { | ||
| agent_id: "query-agent-sparse-capped", | ||
| data: { status: "pending", nested: { priority: "low" } }, | ||
| tags: ["sparse-capped-miss"], | ||
| }); | ||
| } |
There was a problem hiding this comment.
Seeding 55 states sequentially using await in a loop is likely the primary cause of the test timeout. Parallelizing these requests with Promise.all will significantly improve test performance and might even allow for a lower timeout value.
await Promise.all(
Array.from({ length: 55 }, (_, index) =>
putState(`sparse-capped-miss-${index}`, {
agent_id: "query-agent-sparse-capped",
data: { status: "pending", nested: { priority: "low" } },
tags: ["sparse-capped-miss"],
}),
),
);
|
Codex Review: Didn't find any major issues. Another round soon, please! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Co-Authored-By: Duyet Le <me@duyet.net> Co-Authored-By: duyetbot <bot@duyet.net>
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Chef's kiss. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
Summary
cd packages/dashboard && bun run buildcheck.Evidence
000d264from PR fix(deps): update all non-major dependencies #92 (dependency/CI update); PR fix(deps): update all non-major dependencies #92 and post-merge main CI were green.packages/api/test/state-platform.test.ts:150timed out in the full API suite at 5s, then passed in isolation.Verification
bun run lintbun run typecheckbun run test(20 files, 307 tests)bun run test:sdk-examplescd packages/dashboard && bun run build