Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix: CodeLens workflow #147

Merged
merged 2 commits into from
Oct 14, 2024
Merged

Fix: CodeLens workflow #147

merged 2 commits into from
Oct 14, 2024

Conversation

alashchev17
Copy link
Collaborator

@alashchev17 alashchev17 commented Oct 11, 2024 •
edited
Loading

Fix: CodeLens workflow

Description

  • Problem: CodeLens in VSCode were lacking of actual chat state, chat window was not opening as expected
  • Solution: Implemented a handler to remember current chat page in VSCode, also fixed types for refact-vscode ability to build up.
  • Any background context or related links?

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

How to Test

  • Edit your customization.yaml, set in every code_lens instruction of auto_submit to false
  • Try to click "Explain" within the code editor's fragment of code
  • Chat should open up a new window and paste the snippet from customization.yaml and place it in a text area.

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

@alashchev17 alashchev17 self-assigned this Oct 11, 2024
@alashchev17 alashchev17 mentioned this pull request Oct 11, 2024
Merged
12 tasks
@alashchev17 alashchev17 marked this pull request as ready for review October 11, 2024 15:04
@alashchev17 alashchev17 merged commit 0374321 into alpha Oct 14, 2024
2 checks passed
@alashchev17 alashchev17 deleted the fix/codelens-workflow branch October 14, 2024 12:53
MarcMcIntosh added a commit that referenced this pull request Oct 29, 2024
@MarcMcIntosh @alashchev17
@MarcoMeijer @olegklimov @reymondzzzz
Alpha (#117)
d35319f 
* wip: chat title generation & saveChat logic afterall in middleware

* fix: chat title generation thunk fix

* chore: added title for a tab link of chat

* refactor(caht content): decrease prop-drilling and process messages so diffs can be grouped.

* wip: group diffs and send apply and unapply.

* wip(diffs): refactor apply and unapply buttons

* fix: show the correct action for individual chunks.

* wip: warning toast render on diff apply failure

* wip: apply/unapply all after applying one.

* wip: partial apply before apply all

* fix: update onPReview function in code block

* fix: better grouping when diffs are separated by tool calls an empty assistant messages.

* fix: apply / unapply label. TODO: try using diff-preview to see if it's better.

* lint: remove console.log

* wip: created warning bubble, updated warning dispatching logic

* wip: placed warning box absolutely for tests

* wip: diff warning callout implemented

* wip: change of bg color for callout warning box

* wip: refactoring of DiffWarningCallout

* alpha branch creation

* fix: calling `useSendChatRequest` in two places causes `useEffect` to be called twice

* fix: added a catch case when connection is lost for diff-apply query executed

* fix: empty lines are getting no longer eaten

* fix: waiting for first assistant response containing content to be truthy

* wip: add 💿 message about diff usage.

* fix: checking for empty title before asking for a title to be generated

* chore: space remove

* lint(chat submit): fix warning about awaiting for promises.

* ui(diff content): wrap apply buttons and content in a `Container`

* refactor(diff usage): prepend diff usage to the user message

* wip: middleware string formatting removal

* fix: filtering messages within async thunk to prevent 500 error from lsp

* wip: commented out and fixed previously edited toDiff function

* fix: system prompt saves after opening a new chat

* fix: adjusted historySlice middleware & chatThread reducer & refactoring

* fix: complexity removal from toDiff function

* fix: improve performance by memoizing chat history

* performance: only render one page at a time

* performance: disable redux immutability and serializability checks

* fix: showing a agentic mode within a chat window

* fix: synchronize api calls when creating new chat

* fix: added tool_use for thread to store data & type guard

* performance: seperate tour and tip of the day from localstorage so the history doesn't need to be rewritten

* performance: avoid rerendering toolbar to prevent unnecessary repaints when switching between chat and home

* chore: storybook fixtures adjustment

* chore: type issues fix inside of tests

* fix: missing dependency

* fix: center tour begin and end

* fix: put back tour inside root reducer so users don't have to restart tour after update

* chore(chat/Thread/getSelectedToolUse): move from actions to selectors file.

* fix(events): unused imports being include in /dist/events

* Revert "fix: synchronize api calls when creating new chat"

This reverts commit 49ba90c.

* fix: check if running in production environment

* fix: remove eslint

* fix: checking for production

* fix: scrolling chat correctly

* Fix: Edit privacy.yaml & Slight refactoring (#123)

* wip: refactoring of path.ts & opening of privacy.yaml

* fix: made a type workaround for fetchPath() & console.log removal

* fix: BYOK now has the same opening logic as other yamls

* chore: console.log removal

* BYOK conditional rendering restoring

* fix: made ChatForm re-render on chatId change (#125)

* Fix cannot update component error (#124)

* fix: Cannot update a component while rendering a different component

* fix: missing dependencies

* remove MemoryContent (#121)

* refactor(production): use `import.meta.env.MODE` to detect if running/building in production

* Pin button (#122)

* wip(patch show apply): fetch the patch data from the lsp

* fix(patchResult type guard): add proper type checks for patchResult.

* feat(pin open): open an unsaved file in the ide.

* refactor(pin): use a stricter regexp for detecting the end of the markdown.

* feat(pins): add apply button and share chunks cache with diffs.

* feat(pins): allow the ide to reset the diff cache when applying / saving diffs

* ui(markdown): limit pin messages to only assistant messages.

* ui(pin buttons): wrap buttons above long 📍 message

* chore(pins): remove todos and hide show button in web host

* refactor(pin): lazily fetch chunks and show warning / error message if result doesn't work.

* ui(pin): move callout to bellow the button

* chore(dev max age for dev tools): decrease to 500

* refactor(pin warning): reuse diff warning callout

* refactor(pin callout): add click handler and timeout props.

* fix(stop chat after cache restore, and ensure tool_use is defined) (#128)

* fix(stop chat after cache restore): Stop button should work after restoring from cache.

* test: wrap test renderer with AbourtControllerProvider.

* Fix(tool_uss): save tool use in the thread, default to explore if not found.

* 2.0.0-alpha.0

* 2.0.0-alpha.1

* 2.0.0

* 2.0.1

* 2.0.2-alpha.0

* Revert "2.0.2-alpha.0"

This reverts commit 021b43b.

* Revert "2.0.1"

This reverts commit 3438550.

* 2.0.1-alpha.0

* Fix: Auto-scroll adjustment (#127)

* fix: saving last scroll height in state & disabling autoscroll on condition

* fix: refactored useAutoScroll & created scroll-bottom button

* chore: adding comments

* fix: fix of resubmit scroll height calculation

* fix: first only resubmit hides scroll to bottom button

* 2.0.1-alpha.1

* Entertain user about subchat (#126)

* feat: entertain user about subchat

*  

* fix: replace   with margin left

* fix: remove context memory

* refactor(handleSubchatResponse): use recursion to process the messages

---------

Co-authored-by: Oleg Klimov <[email protected]>
Co-authored-by: Marc McIntosh <[email protected]>

* chore: set overflow from scroll to auto (#129)

* ui(pin button): change `open` button to a link (#132)

* fix(diff pin): add better error messages (#133)

* fix: adjusted conditional rendering for diff button (#130)

* telemetry_corrected_snippets_dest is outdated

* fix(pin messages): add all the message from chat to the request to /patch-from-single-file (#134)

* Pin animations (#135)

* fix(pin messages): add all the message from chat to the request to /patch-from-single-file

* feat(📍 animations): add event for the ide to animate files being updated by pin messages

* fix(history): limit history size to 100 chats (#136)

* fix: adding special check for current active file and attached file (#137)

* chore: adjusted size of scroll-to-bottom button (#138)

* 2.0.1-alpha.2

* fix(patch error): fix empty error messages and increase open time

* 2.0.1-alpha.3

* ui(diff-title): break very long +++++++++ or ------- lines.

* 2.0.1-alpha.4

* refactor(patch): add show button, turn file name into a plain link.

* feat(pin): add an ide event for writing results to file.

* ui(patch): place show / apply buttons on the same row as the link text.

* ui(patches): make the patch look distinct.

* refactor(openFilesSlice): remove the slice and events.

* refactor(patch) (#139)

* refactor(patch): add show button, turn file name into a plain link.

* feat(pin): add an ide event for writing results to file.

* ui(patch): place show / apply buttons on the same row as the link text.

* ui(patches): make the patch look distinct.

* refactor(openFilesSlice): remove the slice and events.

* 2.0.1-alpha.5

* fix: adjusted layout styles for ScrollArea (#140)

* feat: remove diff api (#142)

* wip(removing diffs): start removing diff actions

* chore(linter): fix lint error about void statement.

* refactor(diff content): remove most interactivity from  diff content.

* chore(diff removal): remove unused code. diff endpoints, warningSlice, diff hooks

* feat(pin other): prevent `other` patches from rendering (#143)

* 2.0.1-alpha.6

* Feat: Addition of a separate edit message button (#141)

* feat: separate edit message button

* chore: maximum width adjustment of UserInput

* fix: position and appearance improvements

* Feature: Chat History export to JSON (#116)

* wip: export chat history utility function

* wip: export of a specific chat thread history as a JSON

* chore: unnecessary fields removal

* chore: rollback

* wip: implemented 'view raw' & 'copy to clipboard'

* wip: blobs opening in external link

* wip: including default_cap if model within redux is empty

* fix: ChatRawJSON component creation & pre margin reset

* feat: separate page for chat history

* chore: renamed ChatHistory into ThreadHistory for semantics

* fix: added a tiny button and made layout fixes for new page

* fix: added extra condition for button to appear

* fix: made button invisible & cleaned out files from unused code

* fix: moved pre styles for codeblocks & adjusted callouts

* chore: linting issue with void operator

* Fix: Lines selection checkbox behavior (#144)

* wip: working with interacted state and rerendering

* fix: removed unnecessary dep on useEffect & interacted state fix

* chore: added comments

* fix: interacted state of checkboxes on combobox changing

* feat(textarea): allow ide's to add and or replace input values (#119)

* fix(setInputValue): open chat page if not already open.

* 2.0.1-alpha.7

* feat(next diff): receive a `showPatchTicket` event from the ide (#145)

* Fix: CodeLens workflow (#147)

* wip: saving current page for IDE & ToolUse type export

* chore: console.log removal

* fix: better condition for title generation (#150)

* Fix: Double scroll on history page (#149)

* fix: adjusted PageWrapper height styles

* fix: sidebar adjustment

* chore: PageWrapper

* Fix: Files Preview (#148)

* fix: interacted state & checkbox rerendering

* fix: test adjustment

* chore: only removal

* fix: preventing re-render if user has interacted with checkbox

* fix: auto-submit support (#152)

* Shift enter release shift then enter (#146)

* test(shift+enter): add failing test for when shift is released first.

* fix(shift+enter): move detecting shift or enter to onKeyDown.

* Pin buttons on same line (#151)

* wip(patch buttons): move the diff and copy buttons out of the markdown block.

* ui(📍buttons): add wrap-reverse to the button and title container.

* wip(📍messages): remove old buttons

* chore(📍buttons): remove old buttons from ui.

* ui(📍messages): don't conditionally render the buttons. It looks weird when streaming.

* chore(📍): move copy button back into the code block.

* ui(📍): remove apply button.

* ui(📍): rename diff to replace.

* ui(📍): don't wrap the buttons and file path.

* labels

* ui(📍): fix buttons rendering off screen.

---------

Co-authored-by: Oleg Klimov <[email protected]>

* 2.0.1-alpha.8

* fix: edit button out of boundaries (#153)

* 2.0.1-alpha.9

* wip: sending streaming state to vscode (#155)

* Fix: Checkboxes set as interacted on chat form change, if it's not empty (#156)

* fix: remembering user's line selection if textarea contains a message

* chore: updating useCallback deps

* 2.0.1-alpha.10

* fix(done streaming): thread maybe in the cache when generating the title (#157)

* fix(done streaming): thread maybe in the cache when generating the title.

* fix: set prevent send to true when restoring a thread.

* chore: update pull request template (#154)

* fix: prevent checkboxes to be checked when codelens insert value (#158)

* 2.0.1-alpha.11

* Fix/pin replace (#159)

* fix(📍): Message containing the pin maybe a array of strings and components.

* fix(pin): update the selectors for the markdown.

* fix(📍): remove href from <Link> in pin message.

* Fix: Opening of files on [Auto Apply] on Windows (#160)

* wip: opening file on auto-apply & starting rainbow animation fix

* fix: timeouting diff showing

* feat: listening for esc pressing and sending event to IDE (#162)

* ui(patch title): only set the top margin to 0

* fix: relying on thread, not on state.chat.thread (#163)

* fix: set right error message on customization & refactoring of prop drilling

* fix: using type guards & resetting pingApi state on new chat action

* chore: adding TODO comment

* fix: setError() not called twice

* 2.0.1-alpha.12

* fix: JetBrains copying to clipboard

* refactoring: fallbackCopying as separated utility function

* fix: titles generation in background after chat resume

* remove sendCorrectedCodeSnippets

* wip(tool loop): set `onlyDeterministicMessages` if a duplicate tool call and result is found. (#168)

* wip(tool loop): set `onlyDeterministicMessages` if a duplicate tool call and result is found.

* fix(types): odd typescript inference when using `filter`

* test(takeFromEndWhile): ensure correct order.

* fix: diff paste back adding new line in vscode. (#169)

* add codelens actions

* chore: semantic rename

* fix: removal of multiply exposed actions

* 2.0.1-alpha.13

* fix: better errors for integration.yaml

* 2.0.1-alpha.14

---------

Co-authored-by: alashchev17 <[email protected]>
Co-authored-by: Andrii Lashchov <[email protected]>
Co-authored-by: marcomeijer <[email protected]>
Co-authored-by: Marco Meijer <[email protected]>
Co-authored-by: Oleg Klimov <[email protected]>
Co-authored-by: Kirill Starkov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarcMcIntosh MarcMcIntosh MarcMcIntosh approved these changes

Assignees

@alashchev17 alashchev17

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alashchev17 @MarcMcIntosh