-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[OPIK-4987] [FE][BE] feat: integrate Ollie Console sidebar into Opik frontend #5680
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
Nimrod007
wants to merge
75
commits into
main
Choose a base branch
from
nimrodlahav/add-ollie-sidebar-to-opik
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 13 commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
b5e0c0c
[OPIK-4987] [FE][BE] feat: integrate Ollie Console sidebar into Opik …
Nimrod007 aecb8e9
refactor: move OllieSidebar to comet plugin system
Nimrod007 0405caf
fix: add fallback lazy-load for OllieSidebar in dev mode
Nimrod007 cc50b4f
update bridge contract, added events, refactor
awkoy 744951e
[OPIK-4987] [FE] Wire bridge context to Ollie sidebar (#5723)
jverre 5dfa620
feat: manifest-based CDN loading for Ollie sidebar
awkoy 26ffb28
feat: wire live bridge context and navigation to Ollie sidebar
jverre cbf187d
fix: harden Ollie sidebar resilience and code quality
awkoy 970b019
merge: resolve conflict in AssistantSidebar emit handler
awkoy 936285c
Merge remote-tracking branch 'origin/main' into nimrodlahav/add-ollie…
awkoy 1b6ead8
feat: enable assistant sidebar in dev and remove debug logs
jverre 5d49549
fix: update feature-toggles-provider import path
jverre f549da1
feat: add local runner API endpoints and bridge pairing events
jverre d114a3f
feat: load Ollie assets from CDN and fix feature-toggles import
awkoy ead9ede
Update Helm documentation
CometActions 98de59c
[OPIK-4949] [SDK] feat: add LocalRunnerTask for evaluation suite inte…
itamargolan 0fc6cd1
test: add edge case tests and remove example script
itamargolan 1728361
fix: address PR review feedback for LocalRunnerTask
itamargolan c0284ed
style: fix ruff format and add comment for id filtering
itamargolan e1ea353
fix: address second round of PR review feedback
itamargolan 10cc42c
fix: use exact deadline check instead of preemptive timeout
itamargolan 8266d3b
fix: use non-exhaustible time.monotonic mock to prevent flaky test on…
itamargolan 4d52862
[OPIK-4942] [BE] Option E — Separate assertion_results table (#5716)
JetoPistola 00b02df
[OPIK-5017] [BE] refactor: local runner architecture follow-up improv…
BorisTkachenko 6d78b2f
[NA] [E2E] fix: update datasets page-objects for new UI labels (#5747)
NatZol 956f1bb
[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code (#5…
CometActions ceb9af4
Revert "[OPIK-4690] [FE] Add user role update permission to Members t…
anastasiapyzhik 342cc02
[NA] [BE] Update model prices file (#5757)
CometActions 3444f14
[OPIK-4935] [BE] feat(api): add project_name scoping to experiment st…
thiagohora 0e2ad6c
[OPIK-5102][FE]: blueprints ux improvements; (#5743)
aadereiko cbb6883
[OPIK-4449] [FE] Fix eval suite Assertions tab and Go to traces butto…
awkoy f7d36c9
[OPIK-5167] [BE] fix: remove user_role_update permission from Workspa…
LifeXplorer c8dc090
[OPIK-5152] [SDK] feat: promote search_threads to Opik client API (#5…
alexkuzmik b441c6c
Update versions to 1.10.45 and bump base version to 1.10.46
CometActions 7b2bdc4
[OPIK-4865] [SDK] fix: support None values in AgentConfig blueprints …
petrotiurin 2c2f3ef
[OPIK-5102]: fix go to traces button; (#5764)
aadereiko 4eb0881
[NA] [CI] chore: add comment_mode: failures to test reporters (#5762)
petrotiurin 98deb67
[OPIK-4938] [BE] Add project-scoped REST endpoints and integration te…
thiagohora c037e62
fix: address PR review - distributed trace headers, remove unused con…
itamargolan 7bd7703
fix: remove unused import to pass ruff lint
itamargolan 5141250
fix: change poll_interval_seconds default to 0.5s per review feedback
itamargolan a192952
fix: lint and CI failures - formatting, mypy, flaky test
itamargolan 5ea992f
fix: use distributed trace headers directly without conversion
itamargolan e45976c
fix: address baz review - narrow exception catch, improve test names
itamargolan 9ab5a8e
fix: use DistributedTraceHeadersDict return type to satisfy mypy
itamargolan 612339d
fix: remove distributed trace headers from LocalRunnerTask
itamargolan d98e0a1
fix: remove trailing newline in test file
itamargolan 055823e
fix: access task_output via test_case in e2e test
itamargolan c24449f
[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code (#5…
CometActions 8a67537
[OPIK-5154] [BE] Add STRING_EXACT field type for case-sensitive ID fi…
YarivHashaiComet ccf963e
[OPIK-4938] [BE] Add X-Opik-Deprecation header for workspace-wide fal…
thiagohora dcede61
[OPIK-4966] [FE] feat: v2 router, page registration, and URL redirect…
andriidudar 1aa0a9f
[OPIK-5176] [SDK] feat: require @track context when calling get_agent…
petrotiurin 60aa93e
[OPIK-5177] [FE] fix: minor UI fixes and copy changes (#5768)
awkoy 1b01701
[OPIK-5050] [BE] fix: trace threads closing job performance and query…
ldaugusto 789b9fe
[OPIK-4714] [FE] Edit dataset permission (#5722)
anastasiapyzhik c8d9681
[NA] [BE] Update model prices file (#5779)
CometActions f60d913
build(deps-dev): bump com.diffplug.spotless:spotless-maven-plugin (#5…
dependabot[bot] 07828e2
build(deps): bump org.jdbi:jdbi3-stringtemplate4 in /apps/opik-backen…
dependabot[bot] 856ad74
[NA][DOCS] Add section for running online evals retrospectively to do…
LeoRoccoBreedt 305acfb
[OPIK-4966] [FE] refactor: extract shared deps from v1 for v1/v2 coex…
andriidudar aad5077
[OPIK-4966] [FE] feat: clone v1 pages, layout, and pages-shared into …
andriidudar c1c59da
Update versions to 1.10.46 and bump base version to 1.10.47
CometActions 665dcb4
[OPIK-4938] [BE] Add project_name support to dataset, experiment and …
thiagohora bb4b00c
[OPIK-5019] [BE] feat: add LLM model registry service and API endpoin…
AndreiCautisanu f2b913c
[OPIK-4891] [BE] Add data retention policy enforcement (#5647)
ldaugusto 83e99c8
[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code (#5…
CometActions f042fc0
[OPIK-4938] [BE] Minor test and code cleanup from PR review feedback …
thiagohora ea95893
[OPIK-5189] [FE] fix: render ChatPrompt messages in optimizer trial c…
itamargolan 8ca0e4f
[NA] [SDK] [DOCS] Update automatically OpenAPI spec and Fern code (#5…
CometActions d81aa3c
[OPIK-5032] [FE] fix: eval suite experiment export with assertions an…
awkoy 890d6eb
[OPIK-4964] [FE] feat: v2 sidebar navigation with project-scoped rout…
andriidudar 295d3a3
[OPIK-4688] [INFRA] Auto-trigger FERN update on BE merge and notify #…
JetoPistola 4b0cdba
[OPIK-4518] [BE] Restore stable dataset item IDs in versioned API (#5…
andrescrz f7a544a
[OPIK-5219] [BE] Fix pass_rate query reading from wrong table (always…
JetoPistola 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| VITE_BASE_URL=/ | ||
| VITE_BASE_API_URL=/api | ||
| VITE_BASE_OPIK_AI_URL=/opik-ai | ||
| VITE_BASE_OPIK_AI_URL=/opik-ai | ||
| VITE_ASSISTANT_SIDEBAR_URL=http://localhost:3333 |
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
47 changes: 47 additions & 0 deletions
47
apps/opik-frontend/src/api/local-runners/useLocalRunnerById.ts
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,47 @@ | ||
| import { QueryFunctionContext, useQuery } from "@tanstack/react-query"; | ||
| import api, { | ||
| LOCAL_RUNNERS_REST_ENDPOINT, | ||
| LOCAL_RUNNERS_KEY, | ||
| } from "@/api/api"; | ||
|
|
||
| interface LocalRunner { | ||
| id: string; | ||
| name: string; | ||
| project_id: string; | ||
| status: "pairing" | "connected" | "disconnected"; | ||
| connected_at: string | null; | ||
| agents: Array<{ | ||
| name: string; | ||
| description: string; | ||
| }>; | ||
| } | ||
|
|
||
| type UseLocalRunnerByIdParams = { | ||
| runnerId: string; | ||
| }; | ||
|
|
||
| const getLocalRunnerById = async ( | ||
| { signal }: QueryFunctionContext, | ||
| { runnerId }: UseLocalRunnerByIdParams, | ||
| ) => { | ||
| const { data } = await api.get<LocalRunner>( | ||
| LOCAL_RUNNERS_REST_ENDPOINT + runnerId, | ||
| { signal }, | ||
| ); | ||
|
|
||
| return data; | ||
| }; | ||
|
|
||
| export default function useLocalRunnerById( | ||
| params: UseLocalRunnerByIdParams, | ||
| options?: { | ||
| enabled?: boolean; | ||
| refetchInterval?: number | false; | ||
| }, | ||
| ) { | ||
| return useQuery({ | ||
| queryKey: [LOCAL_RUNNERS_KEY, params], | ||
| queryFn: (context) => getLocalRunnerById(context, params), | ||
| ...options, | ||
| }); | ||
| } | ||
27 changes: 27 additions & 0 deletions
27
apps/opik-frontend/src/api/local-runners/useLocalRunnerPairMutation.ts
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,27 @@ | ||
| import { useMutation } from "@tanstack/react-query"; | ||
| import api, { LOCAL_RUNNERS_REST_ENDPOINT } from "@/api/api"; | ||
|
|
||
| interface LocalRunnerPairResponse { | ||
| pairing_code: string; | ||
| runner_id: string; | ||
| expires_in_seconds: number; | ||
| } | ||
|
|
||
| type UseLocalRunnerPairMutationParams = { | ||
| projectId: string; | ||
| }; | ||
|
|
||
| const useLocalRunnerPairMutation = () => { | ||
| return useMutation({ | ||
| mutationFn: async ({ projectId }: UseLocalRunnerPairMutationParams) => { | ||
| const { data } = await api.post<LocalRunnerPairResponse>( | ||
| `${LOCAL_RUNNERS_REST_ENDPOINT}pairs`, | ||
| { project_id: projectId }, | ||
| ); | ||
|
|
||
| return data; | ||
| }, | ||
| }); | ||
| }; | ||
|
|
||
| export default useLocalRunnerPairMutation; |
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.
useLocalRunnerByIdrepeats theuseQuery-by-id pattern, should we extract a sharedcreateUseResourceById({ endpoint, keyPrefix })?Finding type:
Code Dedup and Conventions| Severity: 🟢 LowWant Baz to fix this for you? Activate Fixer