forked from Kilo-Org/kilocode
-
Notifications
You must be signed in to change notification settings - Fork 0
feat(accessibility): Improves screen reader accessibility for indexing status #1
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
Draft
adamhill
wants to merge
35
commits into
main
Choose a base branch
from
adamhill/fix-code-indexing-vis-for-screen-readers
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.
Draft
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
dac1da2
feat(accessibility): Improves screen reader accessibility for indexin…
adamhill 540ba29
Adjust position within context menus to be near Copilot
gerardbalaoro 8605b76
Update famous-crabs-think.md
kevinvandijk a1a78d9
Make the organization selector look better
kevinvandijk d03c07c
Remove unnecessary local state
kevinvandijk 3258252
Tweak organization selector so it can be displayed on chat view as well
kevinvandijk f80e2a2
Add organization selector to kilocode provider section
kevinvandijk 8600535
Small tweaks too visual style
kevinvandijk f98934a
Changed py from 2 to 1.5 for OrganizationSelector
beatlevic 4f1d5fc
Add hover effect
kevinvandijk 580246a
Set organization back to personal if organization is no longer part o…
kevinvandijk f85a423
Add organization header to fetch models call for kilocode provider
kevinvandijk 2c94c14
Fix Qwen Code link
chrarnoldus 4d08ddb
Re-enable cache breakpoints for Gemini 2.5 Pro
chrarnoldus 1407ea1
Update approval test
chrarnoldus ddd065b
feat(ollama): add API key support, enabling usage of Ollama Turbo
mcowgeratsigma 0aaeeb7
Add translations
chrarnoldus 90a11bb
Fix notifications overlap with organization selector (#2014)
kevinvandijk 308a3c0
Changeset version bump (#2002)
kilocode-bot 4c264f3
Add qwenCode.oauthLoadFailed error
chrarnoldus d4c58bf
Check delta defined
chrarnoldus b3ecf22
Expose reasoning for Sonic
chrarnoldus 6ebda10
Set default headers
chrarnoldus d18c081
Kilo markers
chrarnoldus 89b4ad7
Ignore providers that have a context window smaller than the advertis…
chrarnoldus f7f62ca
Add more telemetry
chrarnoldus f55292b
Simplify
chrarnoldus 01e90dd
Create eleven-cougars-tap.md
chrarnoldus ec60458
Update eleven-cougars-tap.md
chrarnoldus 0aa1bca
Update eleven-cougars-tap.md
chrarnoldus 325809c
Add kilocode_change markers
chrarnoldus 40046e6
Create nasty-rats-begin.md
chrarnoldus ba859a8
Skip Turbo Cache on VSIX Publish Workflow (#2011)
catrielmuller b66f058
Make provider list searchable and alphabetically sorted (#2029)
kevinvandijk fc40864
Changeset version bump (#2027)
kilocode-bot 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
Some comments aren't visible on the classic Files Changed page.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "kilo-code": patch | ||
| --- | ||
|
|
||
| Adjust position within context menus to be below default items |
This file was deleted.
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
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
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 |
|---|---|---|
| @@ -1,13 +1,20 @@ | ||
| import * as undici from "undici" | ||
|
|
||
| export function fetchWithTimeout(timeoutMs: number): typeof fetch { | ||
| export function fetchWithTimeout(timeoutMs: number, headers?: Record<string, string>): typeof fetch { | ||
| const agent = new undici.Agent({ headersTimeout: timeoutMs, bodyTimeout: timeoutMs }) | ||
| return (input, init) => | ||
| undici.fetch( | ||
| return async (input, init) => { | ||
| const mergedHeaders = { | ||
| ...headers, // Persistent headers from function parameter | ||
| ...(init?.headers || {}), // Request-specific headers (can override persistent ones) | ||
| } | ||
| const response = (await undici.fetch( | ||
| input as undici.RequestInfo, | ||
| { | ||
| ...init, | ||
| headers: mergedHeaders, | ||
| dispatcher: agent, | ||
| } as undici.RequestInit, | ||
| ) as unknown as Promise<Response> | ||
| )) as unknown as Response | ||
| return response | ||
| } | ||
| } |
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.
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.
🛠️ Refactor suggestion
Models cache can bleed across organizations — key the memory cache by organization.
You’re now varying the response via X-KiloCode-OrganizationId, but the memory cache key is still just provider. If a user switches orgs within the TTL, they’ll see the prior org’s models. That’s the kind of subtle bug that makes people think the cosmos is conspiring.
Recommend deriving a cacheKey that includes org context (and optionally baseUrl), e.g.:
If you want to keep the existing exports unchanged, implement the cacheKey locally in getModels and keep the public getModelsFromCache/flushModels untouched for now.
🤖 Prompt for AI Agents