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.

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

feat: copilot controller #6272

Merged
merged 1 commit into from
Apr 10, 2024
Merged

feat: copilot controller #6272

merged 1 commit into from
Apr 10, 2024

Conversation

darkskygit
Copy link
Member

@darkskygit darkskygit commented Mar 22, 2024

fix CLOUD-27

@toeverything toeverything deleted a comment from graphite-app bot Mar 22, 2024
Copy link

codecov bot commented Mar 22, 2024

Codecov Report

Attention: Patch coverage is 42.25122% with 354 lines in your changes are missing coverage. Please review.

Project coverage is 61.90%. Comparing base (e6a5765) to head (7c38a54).

Files Patch % Lines
...ages/backend/server/src/plugins/copilot/session.ts 8.19% 112 Missing ⚠️
...s/backend/server/src/plugins/copilot/controller.ts 31.12% 104 Missing ⚠️
...ges/backend/server/src/plugins/copilot/resolver.ts 60.00% 103 Missing and 1 partial ⚠️
...s/backend/server/src/core/workspaces/permission.ts 46.15% 28 Missing ⚠️
...end/server/src/plugins/copilot/providers/openai.ts 40.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           canary    #6272      +/-   ##
==========================================
- Coverage   62.30%   61.90%   -0.41%     
==========================================
  Files         522      524       +2     
  Lines       24860    25416     +556     
  Branches     2295     2321      +26     
==========================================
+ Hits        15490    15734     +244     
- Misses       9098     9409     +311     
- Partials      272      273       +1     
Flag Coverage Δ
server-test 72.34% <42.15%> (-0.94%) ⬇️
unittest 38.81% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

perfsee bot commented Mar 28, 2024

affine-toeverything

Bundle main

diff ------------------- Bundle Size Diff -------------------------

@@                        EntryPoint: app                        @@
##                     master …ilot-resolver                 +/- ##
===================================================================
  Bundle                    ?        16.5 MB                       
  Initial JS                ?        6.33 MB                       
  Initial CSS               ?         107 kB                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
  Assets                    ?            110                       
  Chunks                    ?            108                       
  Packages                  ?            280                       
< Duplicates                0             10                 +10   
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Deduplicate versions of libraries                                
! Split assets into smaller pieces                                 
! Avoid large JavaScript libraries with smaller alternatives       
! Separate mixed content assets files                              
! Improve page load speed with `preconnect` or `dns-prefetch`      
! Avoid cache wasting                                              

@darkskygit darkskygit changed the base branch from darksky/copilot-plugins to canary April 10, 2024 11:28
Copy link

graphite-app bot commented Apr 10, 2024

Merge activity

@graphite-app graphite-app bot merged commit 7c38a54 into canary Apr 10, 2024
36 of 40 checks passed
@graphite-app graphite-app bot deleted the darksky/copilot-resolver branch April 10, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:server test Related to test cases
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants