Feature: Docker Containers API #210

alashchev17 · 2024-11-26T17:14:22Z

Featur: Docker Containers API

Description

Part of the global configuration UI feature, it provides support for docker containers fetching by filter_image coming from the configuration, also contains a lot of enhancements to the UI of integration setup page

Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Refactoring (no functional changes, only code improvements)
Documentation update

Checklist

My code follows the code style of this project.
I have performed a self-review of my code.
I have commented my code, particularly in hard-to-understand areas.
I have added tests that prove my fix is effective or that my feature works.
New and existing unit tests pass locally with my changes.
Any dependent changes have been merged and published in downstream modules.
I have updated the documentation where necessary.

Linked Issues

Fixes Fix: Edit privacy.yaml & Slight refactoring #123
Closes #456

Additional Notes

Should run with self_configure branch of the LSP

…st instead of raw flexboxes

…nkClick extracted to separate files and scopes

…ional rendering of docker section within integration form

…etter & making integrationsview setup better

MarcMcIntosh · 2024-12-03T12:13:41Z

src/components/ChatContent/ToolsContent.tsx

-            </Flex>
-            <Chevron open={open} />
-          </Flex>
+        <Collapsible.Trigger>


does the onClick handler work without the asChild prop being used?

Yeah, because I'm specifying onClick directly on the element, not relying on Trigger click

src/services/refact/docker.ts

MarcMcIntosh · 2024-12-03T12:20:14Z

src/components/Text/TruncateRight.tsx

<Text> has a truncate property, it might not well with in a flex-box.

https://www.radix-ui.com/themes/docs/components/text
potential fix for truncate in a flex-box
https://css-tricks.com/flexbox-truncated-text/

I'm also handling flex box with width adjustments, but thanks, will check that out

MarcMcIntosh

Good work :)

I've made some notes for things to when I run this branch

src/components/Text/Text.module.css

src/components/SmartLink/SmartLink.tsx

src/components/Reveal/reveal.module.css

src/features/Integrations/renderIntegrationFormField.tsx

MarcMcIntosh · 2024-12-03T13:43:41Z

src/services/refact/docker.ts

+  }),
+  endpoints: (builder) => ({
+    getAllDockerContainers: builder.query<DockerContainersResponse, undefined>({
+      providesTags: ["DOCKER"],


note: tags maybe causing cache issues.

src/components/IntegrationsView/IntegrationsView.tsx

MarcMcIntosh · 2024-12-03T13:47:38Z

src/components/IntegrationsView/IntegrationsView.tsx

    dispatch(integrationsApi.util.resetApiState());
+    dispatch(dockerApi.util.resetApiState());


to check: posable cache issues

src/components/IntegrationsView/IntegrationsView.tsx

MarcMcIntosh · 2024-12-03T14:03:34Z

Show more / hide doesn't seem to change anything.

alashchev17 · 2024-12-03T14:34:54Z

Show more / hide doesn't seem to change anything.

Try to do that in postgres configuration
Already made it conditionally appearing

… launch

* wip(integrations): add service for the api. * fix(integrations): add _ to unused function argument. * wip(integrations): add enabled feild. * chore(types): update integration fixture * wip: integrations page layout + data retrieval * wip: conditional rendering of form and cards & JSON schema parsing into form * wip: customized form & updating data * wip: icon in title & overall enhancements * wip: renamed buttons * wip(integration api): update response types. * wip: ajv & rjsf libraries removal * wip(intergrations api): add save handler. * fix(integrations): typo in types * wip: fixing endpoints & new hooks for data fetching * fix(integrations): typos. * fix: type guards conditions * wip(integrations UI): grouping integrations on global, project specific * wip(integrations): schema parsing & refactoring - [x] Implemented an `IntegrationForm` with reusable fields for each integration field type - [x] Schema check in `handleSubmit` to transform formData to corresponding field types - [x] Added `toPascalCase` utility function * wip: rendering values & LSP responses handling Slight refactoring of apply and return buttons, and some other minor changes. * Wip smart links button (#206) * wip(configuration chat): send first messages. * wip(configuration chat): tool use in configuration chat. * wip: smartlinks within fields & return to integraitons page from chat * wip(integrations): availability * wip: return button alignment * wip: form disabled if values didn't change * wip: better UI & cancel values of form * wip: header for integrations & better layout & availability * wip: field.f_type -> [f_type, f_size] * wip: form cancel removed & smartlinks on the right side of form buttons * WIP: Refactor/integration chat (#209) * wip(configuration chat): handle going back. * fix(integration chat): change tools setting to explore. * chore(missing deps in hook): `renderField` function was missing deps. * feat(integration data cache): add slice for integrations cache. * fix(integration cache): remove item on save. * fix(config chat): chat being sent multiple times. * wip(tool confirmation): skip confirming tool use when running a configuration. * feat: add apply all button to configuration chat. * feat: reload old configuration chat and go back to the form * feat(configuration chat): add error message handler. * fix(configuration chat): use agentic mode and don't set the model. --------- Co-authored-by: Andrii Lashchov <84778697+alashchev17@users.noreply.github.com> * fix(config chat): reuse last model and system prompt * Feature: Docker Containers API (#210) * wip: docker containers api handlers * wip: docker actions API & action buttons * wip: better action handling * wip: simplification of dockerApi & action buttons improvements * chore: fixtures update * chore: removal of unnecessary esling rule bypass * wip: docker container card & refactoring of integration form * feat(configuration): adjusted size of inputs based on f_size & DataList instead of raw flexboxes * wip: form appearance & responsive layout * wip: updated docker section * wip(refactoring): smartlink, availability, renderField, handleSmartlinkClick extracted to separate files and scopes * wip(form): bool f_type support * wip(refactoring): making docker within integr_schema optional, conditional rendering of docker section within integration form * wip(docker): error handling * wip(refactoring): toolscontent usage summary refactoring * wip(docker containers): collapsible container details & reveal if large * wip(refactoring): project name handling for macOS & Windows * wip(refactoring): docker error card & debounced loaded state * wip(docker containers): better loading state management * wip: separation & refactoring of UI, making docker containers cards better & making integrationsview setup better * fix: error types for docker error cards * fix: unchecked switch values included to formData * chore: removal no-console * chore: removal of unexisting prop * fix: extra fields showup button conditional rendering * chore: TODO insertion * fix: rule bypass removal * feat: debug logging library usage for integrations logs * feat: implementing root debug instance, managing debuggable instances launch * chore: better logging message * chore: TODO for tags invalidation docker * fix: replaced <label/> to <Text as='label' /> * chore: endpoints extraction to consts & TODO insertion for cache invalidation logic * fix: padding shift issue & middleware for detail error on integration-get * fix: simplification of UI * feat: hardcoded icons for integrations * feat: smartlinks for each docker container * fix: values[fieldKey] could be falsy * chore: added TODOs and removed legacy code * chore: titles for docker container smartlinks * fix: adjusted color for 'off' badge * fix: preventing click on disabled dropdown item if disabled --------- Co-authored-by: Marc McIntosh <marcmcintosh1987@gmail.com>

* wip(integrations): add service for the api. * fix(integrations): add _ to unused function argument. * wip(integrations): add enabled feild. * chore(types): update integration fixture * wip: integrations page layout + data retrieval * wip: conditional rendering of form and cards & JSON schema parsing into form * wip: customized form & updating data * wip: icon in title & overall enhancements * wip: renamed buttons * wip(integration api): update response types. * wip: ajv & rjsf libraries removal * wip(intergrations api): add save handler. * fix(integrations): typo in types * wip: fixing endpoints & new hooks for data fetching * fix(integrations): typos. * fix: type guards conditions * wip(integrations UI): grouping integrations on global, project specific * wip(integrations): schema parsing & refactoring - [x] Implemented an `IntegrationForm` with reusable fields for each integration field type - [x] Schema check in `handleSubmit` to transform formData to corresponding field types - [x] Added `toPascalCase` utility function * wip: rendering values & LSP responses handling Slight refactoring of apply and return buttons, and some other minor changes. * Wip smart links button (#206) * wip(configuration chat): send first messages. * wip(configuration chat): tool use in configuration chat. * wip: smartlinks within fields & return to integraitons page from chat * wip(integrations): availability * wip: return button alignment * wip: form disabled if values didn't change * wip: better UI & cancel values of form * wip: header for integrations & better layout & availability * wip: field.f_type -> [f_type, f_size] * wip: docker containers api handlers * wip: docker actions API & action buttons * wip: better action handling * wip: simplification of dockerApi & action buttons improvements * chore: fixtures update * chore: removal of unnecessary esling rule bypass * wip: docker container card & refactoring of integration form * wip: form cancel removed & smartlinks on the right side of form buttons * WIP: Refactor/integration chat (#209) * wip(configuration chat): handle going back. * fix(integration chat): change tools setting to explore. * chore(missing deps in hook): `renderField` function was missing deps. * feat(integration data cache): add slice for integrations cache. * fix(integration cache): remove item on save. * fix(config chat): chat being sent multiple times. * wip(tool confirmation): skip confirming tool use when running a configuration. * feat: add apply all button to configuration chat. * feat: reload old configuration chat and go back to the form * feat(configuration chat): add error message handler. * fix(configuration chat): use agentic mode and don't set the model. --------- Co-authored-by: Andrii Lashchov <84778697+alashchev17@users.noreply.github.com> * feat(configuration): adjusted size of inputs based on f_size & DataList instead of raw flexboxes * fix(config chat): reuse last model and system prompt * wip: form appearance & responsive layout * wip: updated docker section * wip(refactoring): smartlink, availability, renderField, handleSmartlinkClick extracted to separate files and scopes * wip(chat links): add stub api for chat links. * wip(chat-links): setup story book. * wip(chat links): add a quick ui to run in sotrybook * chore(chat links): add todos. * wip(chat links): add arguments to request handler and use msw in storybook. * chore(chat links): update lsp url. * ui(chat link title): add a information about the link in the title. * wip(chat links): add chat links component to chat content. * refactor(chat action types): enum couldn't be used for comparison :/ * fix(chat config cations): go back to integration. * fix(chat config): go to action should open a configuration. * chore(storybook): add config story to chat content. * ui(configuration chat): adjust size and spacing of the available actions. * wip(chat actions): comment out unhandled if statments. * chore(links api): add model name to the links request. * fix(links request): model name must be defined. * wip(config chat): add notes about action handlers. * Feature: Docker Containers API (#210) * wip: docker containers api handlers * wip: docker actions API & action buttons * wip: better action handling * wip: simplification of dockerApi & action buttons improvements * chore: fixtures update * chore: removal of unnecessary esling rule bypass * wip: docker container card & refactoring of integration form * feat(configuration): adjusted size of inputs based on f_size & DataList instead of raw flexboxes * wip: form appearance & responsive layout * wip: updated docker section * wip(refactoring): smartlink, availability, renderField, handleSmartlinkClick extracted to separate files and scopes * wip(form): bool f_type support * wip(refactoring): making docker within integr_schema optional, conditional rendering of docker section within integration form * wip(docker): error handling * wip(refactoring): toolscontent usage summary refactoring * wip(docker containers): collapsible container details & reveal if large * wip(refactoring): project name handling for macOS & Windows * wip(refactoring): docker error card & debounced loaded state * wip(docker containers): better loading state management * wip: separation & refactoring of UI, making docker containers cards better & making integrationsview setup better * fix: error types for docker error cards * fix: unchecked switch values included to formData * chore: removal no-console * chore: removal of unexisting prop * fix: extra fields showup button conditional rendering * chore: TODO insertion * fix: rule bypass removal * feat: debug logging library usage for integrations logs * feat: implementing root debug instance, managing debuggable instances launch * chore: better logging message * chore: TODO for tags invalidation docker * fix: replaced <label/> to <Text as='label' /> * chore: endpoints extraction to consts & TODO insertion for cache invalidation logic * fix: padding shift issue & middleware for detail error on integration-get * refactor(chat meta): add meta info to chat request. * fix(fixtures): update configuration chat fixture. * feat: show links for all chats. * chore(links meta): add meta to links. * fix: linter error. * feat(chat links): handle follow up action. * hard code links mode to explore. * feat(system message): request the system message with the chat stream. * WIP(chat links): pass mode through follow-up and summarise project links. * wip(chat links): infer the mode in none is added. * fix: don't set deterministic messages when there isn't a system message. * wip(system message): don't user deterministic messages because it stops the stream * fix(chat meta links): set mode to the correct string. * fix(chat links): show on new chat. --------- Co-authored-by: alashchev17 <andrew.lashchev15@gmail.com> Co-authored-by: Andrii Lashchov <84778697+alashchev17@users.noreply.github.com>

alashchev17 added 23 commits November 26, 2024 18:13

wip: docker containers api handlers

bd870a3

wip: docker actions API & action buttons

514397f

wip: better action handling

201f29b

wip: simplification of dockerApi & action buttons improvements

8eea732

chore: fixtures update

a2b8e42

chore: removal of unnecessary esling rule bypass

00874fd

wip: docker container card & refactoring of integration form

f09ef28

Merge branch 'feat/setup-integrations' into feat/docker-api

edc071f

feat(configuration): adjusted size of inputs based on f_size & DataLi…

9a785c6

…st instead of raw flexboxes

Merge branch 'feat/setup-integrations' into feat/docker-api

7f9e791

wip: form appearance & responsive layout

d46e9fd

wip: updated docker section

2549fc1

wip(refactoring): smartlink, availability, renderField, handleSmartli…

9104122

…nkClick extracted to separate files and scopes

wip(form): bool f_type support

f941e8e

wip(refactoring): making docker within integr_schema optional, condit…

4d21638

…ional rendering of docker section within integration form

wip(docker): error handling

2b1a314

wip(refactoring): toolscontent usage summary refactoring

ecaa68f

wip(docker containers): collapsible container details & reveal if large

4fa35f9

wip(refactoring): project name handling for macOS & Windows

1b2fd88

wip(refactoring): docker error card & debounced loaded state

0dbb9f0

wip(docker containers): better loading state management

c738f8b

wip: separation & refactoring of UI, making docker containers cards b…

f6951d0

…etter & making integrationsview setup better

fix: error types for docker error cards

5a11403

MarcMcIntosh reviewed Dec 3, 2024

View reviewed changes

src/services/refact/docker.ts Outdated Show resolved Hide resolved

MarcMcIntosh reviewed Dec 3, 2024

View reviewed changes

alashchev17 marked this pull request as ready for review December 3, 2024 12:33

MarcMcIntosh requested changes Dec 3, 2024

View reviewed changes

fix: unchecked switch values included to formData

e45bff9

alashchev17 added 2 commits December 3, 2024 15:31

chore: removal no-console

94118f0

chore: removal of unexisting prop

de132ba

alashchev17 added 3 commits December 3, 2024 15:36

fix: extra fields showup button conditional rendering

eab73b0

chore: TODO insertion

0ade262

fix: rule bypass removal

6bbd026

alashchev17 changed the title ~~WIP: Docker Containers API~~ Feature: Docker Containers API Dec 3, 2024

alashchev17 requested a review from MarcMcIntosh December 3, 2024 14:48

alashchev17 added 5 commits December 3, 2024 16:04

feat: debug logging library usage for integrations logs

cd50907

feat: implementing root debug instance, managing debuggable instances…

9d29882

… launch

chore: better logging message

67de24e

chore: TODO for tags invalidation docker

5e2797c

fix: replaced <label/> to <Text as='label' />

3232243

MarcMcIntosh approved these changes Dec 3, 2024

View reviewed changes

alashchev17 merged commit d718260 into feat/setup-integrations Dec 3, 2024
2 checks passed

alashchev17 deleted the feat/docker-api branch December 3, 2024 16:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature: Docker Containers API #210

Feature: Docker Containers API #210

alashchev17 commented Nov 26, 2024 •

edited

Loading

MarcMcIntosh Dec 3, 2024

alashchev17 Dec 3, 2024

MarcMcIntosh Dec 3, 2024

alashchev17 Dec 3, 2024

MarcMcIntosh left a comment •

edited

Loading

MarcMcIntosh Dec 3, 2024

MarcMcIntosh Dec 3, 2024

MarcMcIntosh commented Dec 3, 2024

alashchev17 commented Dec 3, 2024 •

edited

Loading

		dispatch(integrationsApi.util.resetApiState());
		dispatch(dockerApi.util.resetApiState());

Feature: Docker Containers API #210

Feature: Docker Containers API #210

Conversation

alashchev17 commented Nov 26, 2024 • edited Loading

Featur: Docker Containers API

Description

Type of change

Checklist

Linked Issues

Additional Notes

MarcMcIntosh Dec 3, 2024

Choose a reason for hiding this comment

alashchev17 Dec 3, 2024

Choose a reason for hiding this comment

MarcMcIntosh Dec 3, 2024

Choose a reason for hiding this comment

alashchev17 Dec 3, 2024

Choose a reason for hiding this comment

MarcMcIntosh left a comment • edited Loading

Choose a reason for hiding this comment

MarcMcIntosh Dec 3, 2024

Choose a reason for hiding this comment

MarcMcIntosh Dec 3, 2024

Choose a reason for hiding this comment

MarcMcIntosh commented Dec 3, 2024

alashchev17 commented Dec 3, 2024 • edited Loading

alashchev17 commented Nov 26, 2024 •

edited

Loading

MarcMcIntosh left a comment •

edited

Loading

alashchev17 commented Dec 3, 2024 •

edited

Loading