Repository navigation
fix(code-index): port fork reliability fixes (#34, #67, #102, #110) #2437
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
Open
simurg79
wants to merge
42
commits into
Zoo-Code-Org:main
Choose a base branch
from
simurg79:fix/code-index-reliability-port
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+6,621
−3,433
Open
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
3aa633b
fix(code-index): port fork reliability fixes (#34, #67, #102, #110)
bf17260
fix(code-index): address first CI run (format, knip entries, Vitest 4…
947cea7
fix(code-index): address second CI run (orchestrator class mocks, wat…
a6186ae
chore(lint): prune code-index ESLint suppressions made stale by the port
b8fd8e1
chore(lint): prune stale CodebaseSearchTool suppression removed by th…
c13334a
fix(code-index): resolve type errors in orchestrator and test doubles
d510201
build: drop unused p-limit; patch @qdrant/js-client-rest Retry-After …
9309d5e
chore(code-index): remove production-dead code left by the port (knip…
b2fc597
build: skip non-numeric Retry-After in the Qdrant patch so HTTP dates…
92c9b71
test(code-index): await both concurrent verify counts in the Retry-Af…
8844575
fix(code-index): load web-tree-sitter's CJS build in the parse worker…
b592702
fix(code-index): address CodeRabbit review feedback
4aca2e6
fix(code-index): address CodeRabbit round 2 feedback
bb181b2
test(code-index): match trimmed embedding input; cover registered rec…
dc8deae
fix(code-index): guard pacer deadline and use literal incomplete defa…
bd8d4f6
test(code-index): cover provider deadline that becomes out-of-range a…
8e04d28
style(code-index): apply prettier to provider pacer test
2c63f85
fix(code-index): treat missing source files as removed (port fork #116)
5393fa7
refactor(code-index): remove parse worker, log module and layout-only…
26721b4
refactor(code-index): single file writer, pause-gate pacer, counter-m…
35e1aa2
fix(code-index): route scanner and watcher writes through one writer;…
850a2cc
fix(code-index): single-attempt providers with rate-limit scopes; Bed…
51dfbbd
test(code-index): visual specs for failed-file popover, provider-wait…
88d62cf
style(code-index): apply prettier
bbcd375
fix(code-index): address review findings and CI failures
fb2e354
fix(code-index): align tests with single-attempt writer flow and keep…
532a8fb
fix(code-index): resolve lint, knip and file-watcher test failures
7ad9a7e
fix(code-index): prune stale lint suppressions and make failure-path …
fb682aa
fix(code-index): type failedFileSummary as optional and complete stat…
e12706a
fix(code-index): keep failedFileSummary optional in status type
338c058
fix(code-index): settle watcher status, classify Qdrant 429s, clear r…
413ba2f
fix(i18n): translate code index cache and quota errors
c6ee22c
fix(code-index): isolate Qdrant pacer per test and fix CI test failures
e8a075f
fix(code-index): restore detached start in webview startIndexing handler
5061b15
fix(code-index): start from Error in one step, count pending watcher …
2d2f878
test(code-index): use providerIdentifiers.openai in manager reliabili…
df9f19b
fix(code-index): keep failure-only entries when file listing is trunc…
851b844
fix(code-index): remove confirmed-missing failure files on truncated …
e70cb6d
test(code-index): assert abort reason, cover truncated listing at sta…
83e455a
fix(code-index): drop finish events from batches of a stopped session
198520c
test(code-index): read abort reason after scan settles in scanner abo…
a3ae407
fix(code-index): drop progress from stopped watcher chunks; clean up …
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
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
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
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,56 @@ | ||
| diff --git a/dist/browser/index.cjs b/dist/browser/index.cjs | ||
| --- a/dist/browser/index.cjs | ||
| +++ b/dist/browser/index.cjs | ||
| @@ -991,8 +991,8 @@ | ||
| } | ||
| catch (error) { | ||
| if (error instanceof ApiError && error.status === 429) { | ||
| - const retryAfterHeader = error.headers.get('retry-after')?.[0]; | ||
| - if (retryAfterHeader) { | ||
| + const retryAfterHeader = error.headers.get('retry-after'); | ||
| + if (retryAfterHeader && !isNaN(Number(retryAfterHeader))) { | ||
| throw new QdrantClientResourceExhaustedError(error.message, retryAfterHeader); | ||
| } | ||
| } | ||
| diff --git a/dist/browser/index.js b/dist/browser/index.js | ||
| --- a/dist/browser/index.js | ||
| +++ b/dist/browser/index.js | ||
| @@ -989,8 +989,8 @@ | ||
| } | ||
| catch (error) { | ||
| if (error instanceof ApiError && error.status === 429) { | ||
| - const retryAfterHeader = error.headers.get('retry-after')?.[0]; | ||
| - if (retryAfterHeader) { | ||
| + const retryAfterHeader = error.headers.get('retry-after'); | ||
| + if (retryAfterHeader && !isNaN(Number(retryAfterHeader))) { | ||
| throw new QdrantClientResourceExhaustedError(error.message, retryAfterHeader); | ||
| } | ||
| } | ||
| diff --git a/dist/cjs/api-client.js b/dist/cjs/api-client.js | ||
| --- a/dist/cjs/api-client.js | ||
| +++ b/dist/cjs/api-client.js | ||
| @@ -51,8 +51,8 @@ | ||
| } | ||
| catch (error) { | ||
| if (error instanceof openapi_typescript_fetch_1.ApiError && error.status === 429) { | ||
| - const retryAfterHeader = error.headers.get('retry-after')?.[0]; | ||
| - if (retryAfterHeader) { | ||
| + const retryAfterHeader = error.headers.get('retry-after'); | ||
| + if (retryAfterHeader && !isNaN(Number(retryAfterHeader))) { | ||
| throw new errors_js_1.QdrantClientResourceExhaustedError(error.message, retryAfterHeader); | ||
| } | ||
| } | ||
| diff --git a/dist/esm/api-client.js b/dist/esm/api-client.js | ||
| --- a/dist/esm/api-client.js | ||
| +++ b/dist/esm/api-client.js | ||
| @@ -47,8 +47,8 @@ | ||
| } | ||
| catch (error) { | ||
| if (error instanceof ApiError && error.status === 429) { | ||
| - const retryAfterHeader = error.headers.get('retry-after')?.[0]; | ||
| - if (retryAfterHeader) { | ||
| + const retryAfterHeader = error.headers.get('retry-after'); | ||
| + if (retryAfterHeader && !isNaN(Number(retryAfterHeader))) { | ||
| throw new QdrantClientResourceExhaustedError(error.message, retryAfterHeader); | ||
| } | ||
| } |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.