feat(knb): auto-compile (Issue #43) - #50
Conversation
…ebounced auto-compile runner, wire server write handlers to schedule auto-compiles, add unit & integration tests, and fix dynamic import in integration test.
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment Thanks for integrating Codecov - We've got you covered ☂️ |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| tags: { name, category } ORDER BY .name | ||
| } | ||
| ORDER BY .dates.` + "`start` DESC" | ||
| ); |
There was a problem hiding this comment.
EdgeDB query missing fields used in output generation
Medium Severity
The EdgeDB query does not select verification_status or last_verified fields, but the code at lines 105-106 attempts to use e.verification_status and e.last_verified. Since these fields are not in the query result, they will always be undefined, causing the nullish coalescing to always use fallback values ('verified' and today's date). The actual database values for these fields are silently discarded.
Additional Locations (1)
|
Proposed checklist and smoke test (follow-ups recommended):\n\n- [ ] Run the integration suite locally with EdgeDB: then
DEV v1.6.1 /home/michelek/Documents/github/cv_system stdout | tests/build_context.test.ts > compileKnowledgeBase generates expected markdown structure for experiences ✓ tests/build_context.test.ts (1 test) 9ms stdout | mcp-servers/knowledge-base/src/services/tests/experience.test.ts > Experience CRUD stdout | mcp-servers/knowledge-base/src/services/tests/integration-auto-compile.test.ts > adding experience in EdgeDB triggers compile via auto-runner (integration) stdout | mcp-servers/knowledge-base/src/services/tests/tag.test.ts > Tag/Classifier CRUD stdout | mcp-servers/knowledge-base/src/services/tests/achievement.test.ts > Achievement CRUD stdout | mcp-servers/knowledge-base/src/services/tests/experience.test.ts > Experience Search ✓ mcp-servers/knowledge-base/src/services/tests/experience.test.ts (10 tests) 387ms stdout | mcp-servers/knowledge-base/src/services/tests/integration-auto-compile.test.ts > adding experience in EdgeDB triggers compile via auto-runner (integration) ✓ mcp-servers/knowledge-base/src/services/tests/compileHook.test.ts (1 test) 505ms stdout | mcp-servers/knowledge-base/src/services/tests/tag.test.ts > Tag Usage Statistics stdout | mcp-servers/knowledge-base/src/services/tests/tag.test.ts > Fuzzy Tag Matching ✓ mcp-servers/knowledge-base/src/services/tests/integration-auto-compile.test.ts (1 test) 739ms stdout | mcp-servers/knowledge-base/src/services/tests/achievement.test.ts > Achievement Search ✓ mcp-servers/knowledge-base/src/services/tests/achievement.test.ts (9 tests) 1136ms Test Files 7 passed (7) PASS Waiting for file changes...
RUN v1.6.1 /home/michelek/Documents/github/cv_system/mcp-servers/knowledge-base stdout | src/services/tests/skill.test.ts > Skill CRUD stdout | src/services/tests/skill.test.ts > Skill Search ✓ src/services/tests/skill.test.ts (10 tests) 400ms stdout | src/services/tests/experience.test.ts > Experience Search ✓ src/services/tests/experience.test.ts (10 tests) 279ms stdout | src/services/tests/tag.test.ts > Tag Usage Statistics stdout | src/services/tests/tag.test.ts > Fuzzy Tag Matching ✓ src/services/tests/tag.test.ts (13 tests) 333ms stdout | src/services/tests/achievement.test.ts > Achievement Search ✓ src/services/tests/achievement.test.ts (9 tests) 233ms stdout | src/services/tests/integration-auto-compile.test.ts > adding experience in EdgeDB triggers compile via auto-runner (integration) stdout | src/services/tests/integration-auto-compile.test.ts > adding experience in EdgeDB triggers compile via auto-runner (integration) ✓ src/services/tests/integration-auto-compile.test.ts (1 test) 661ms Test Files 6 passed (6) |
1 participant
Export compiler function compileKnowledgeBase and auto-compile runner; tests pass locally (45 tests). See Issue #43.
Note
Introduces EdgeDB-backed compilation and validates auto-compile flow end-to-end.
compileKnowledgeBaseto acceptsource(filesystem|edgedb) and implement EdgeDB compilation: queriesExperience, emits YAML + article content, and writes_compiled_context.md.integration-auto-compile.test.tsthat creates anExperience, triggers the auto-compile runner, and asserts compiled output.Written by Cursor Bugbot for commit 41e3fe6. This will update automatically on new commits. Configure here.