-
Notifications
You must be signed in to change notification settings - Fork 0
feat(multi-hop): implement 3-stage query decomposition and parallel retrieval #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
439e4ee
test: add embedder config file for quality gate test
nehraa e650ae1
feat: multi-language tree-sitter support for Go, Python, C, C++, Rust
nehraa df3e2be
fix: global CLI binary works correctly with ESM imports
nehraa 51c3f74
perf: optimize ONNX embedding hot loops, parallelize batches, use nat…
nehraa b78bbce
chore: include all 1.0.1 changes
nehraa bc7f45a
- ONNX hot loops: Removed unnecessary ?? 0 null checks from tight lo…
nehraa 6773f60
chore: remove .qwen and .serena from tracking, clean up gitignore
nehraa 1b698a5
feat: memory-efficient chunked embedding pipeline, ONNX stability, po…
nehraa 772745f
chore: apply same changes lost during hook cycle
nehraa a447542
feat(multi-hop): implement 3-stage query decomposition and parallel r…
nehraa 9b303f9
chore: remove .qwen directory from git tracking
nehraa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,3 +3,6 @@ node_modules | |
| .coderag* | ||
| coverage | ||
| .env | ||
| .qwen/ | ||
| .serena/ | ||
| *.tgz | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
58 changes: 58 additions & 0 deletions
58
...-code/reasoning/quality-gates/post-commit-20260406-152438/POST_COMMIT_REPORT.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| # 🎯 Post-Commit Quality Gate Report | ||
|
|
||
| **Commit:** 971d68d feat: add Gemini and ONNX embedding providers | ||
| **Date:** 2026-04-06T15:24:54+05:30 | ||
| **Author:** Abhinav Nehra <nehraa@student.wpunj.edu> | ||
| **Branch:** feat/gemini-onnx-embedding-providers | ||
|
|
||
| --- | ||
|
|
||
| ## 📊 Summary | ||
|
|
||
| | Metric | Value | | ||
| |--------|-------| | ||
| | Changed Files | 0 | | ||
| | Source Files | 0 | | ||
| | Test Files | 0 | | ||
| | Doc Files | 0 | | ||
|
|
||
| --- | ||
|
|
||
| ## 🎯 Quality Gate Results | ||
|
|
||
| | Stage | Status | Details | | ||
| |-------|--------|---------| | ||
|
|
||
| | /7 | Linting & Code Quality | PASS | Checked 1 files | | ||
| | /7 | Security Analysis | FAIL | Scanned for secrets, injections, dependencies | | ||
| | /7 | Fix Security Issues | PASS | Fixed 0 issues | | ||
| | /7 | Run Existing Tests | FAIL | Ran test suite | | ||
| | /7 | Add/Update Tests | PASS | Identified 0 files | | ||
| | /7 | Update Documentation | PASS | Checked README, CHANGELOG, inline docs | | ||
| | /7 | Context Compaction | PASS | Compacted from 40K to 40K | | ||
|
|
||
| --- | ||
|
|
||
| ## 📁 Detailed Reports | ||
|
|
||
| - [Stage 1: Linting](stage-01-linting.md) | ||
| - [Stage 2: Security](stage-02-security.md) | ||
| - [Stage 3: Fix Security](stage-03-fix-security.md) | ||
| - [Stage 4: Run Tests](stage-04-run-tests.md) | ||
| - [Stage 5: Add Tests](stage-05-add-tests.md) | ||
| - [Stage 6: Documentation](stage-06-documentation.md) | ||
| - [Stage 7: Context](stage-07-context.md) | ||
|
|
||
| --- | ||
|
|
||
| ## ✅ Next Steps | ||
|
|
||
| 1. **Fix any FAIL statuses** above | ||
| 2. **Review security issues** and apply fixes | ||
| 3. **Add tests** for new functionality | ||
| 4. **Update documentation** for changed APIs | ||
| 5. **Commit fixes** to trigger another quality gate | ||
|
|
||
| --- | ||
|
|
||
| *Generated by post-commit quality gate hook* |
23 changes: 23 additions & 0 deletions
23
claude-code/reasoning/quality-gates/post-commit-20260406-152438/context-summary.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # Post-Commit Quality Gate Summary | ||
|
|
||
| **Commit:** 971d68d feat: add Gemini and ONNX embedding providers | ||
| **Date:** 2026-04-06T15:24:54+05:30 | ||
| **Changed Files:** 0 | ||
|
|
||
| ## Quality Gate Results | ||
|
|
||
| | Stage | Status | Details | | ||
| |-------|--------|---------| | ||
|
|
||
| | /7 | Linting & Code Quality | PASS | Checked 1 files | | ||
| | /7 | Security Analysis | FAIL | Scanned for secrets, injections, dependencies | | ||
| | /7 | Fix Security Issues | PASS | Fixed 0 issues | | ||
| | /7 | Run Existing Tests | FAIL | Ran test suite | | ||
| | /7 | Add/Update Tests | PASS | Identified 0 files | | ||
| | /7 | Update Documentation | PASS | Checked README, CHANGELOG, inline docs | | ||
|
|
||
| ## Key Takeaways | ||
| - Review any FAIL statuses above | ||
| - Fix security issues before next commit | ||
| - Add tests for new functionality | ||
| - Update documentation as needed |
6 changes: 6 additions & 0 deletions
6
...de-code/reasoning/quality-gates/post-commit-20260406-152438/stage-01-linting.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| # Stage 1: Linting & Code Quality | ||
|
|
||
| **Status:** PASS | ||
| **Files Checked:** 1 | ||
|
|
||
| ✅ No linting issues found |
18 changes: 18 additions & 0 deletions
18
...e-code/reasoning/quality-gates/post-commit-20260406-152438/stage-02-security.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| # Stage 2: Security Analysis | ||
|
|
||
| **Status:** FAIL | ||
|
|
||
|
|
||
| ### npm Audit Vulnerabilities | ||
| ``` | ||
| npm warn config production Use `--omit=dev` instead. | ||
| found 0 vulnerabilities | ||
| ``` | ||
|
|
||
| ## Security Checks Performed | ||
| - ✅ Hardcoded secrets scan | ||
| - ✅ SQL injection risks | ||
| - ✅ eval/exec usage | ||
| - ✅ Dependency vulnerabilities | ||
| - ✅ XSS patterns | ||
| - ✅ Path traversal risks |
11 changes: 11 additions & 0 deletions
11
...de/reasoning/quality-gates/post-commit-20260406-152438/stage-03-fix-security.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| # Stage 3: Fix Security Issues | ||
|
|
||
| **Status:** PASS | ||
| **Issues Fixed:** 0 | ||
|
|
||
| ✅ No security issues required fixing | ||
|
|
||
| ## Auto-Fixes Applied | ||
| - Hardcoded secrets → Environment variables | ||
| - SQL injection → Parameterized queries (manual review needed) | ||
| - eval/exec → Safer alternatives (manual review needed) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README says the ONNX embedding provider uses
Xenova/gte-small, but the implementation defaults toXenova/all-MiniLM-L6-v2(and tests assert that). Please align the README model name + download snippet with the actual default (or change the provider default) so users download the correct model.