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

fix: Use entire file as context in "Optimize Code" prompt #459

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 7 additions & 8 deletions buf.gen.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,15 @@ plugins:
out: src/shared/pyroscope-api
opt: target=ts
inputs:
# Build a subset of the Pyroscope protobuf API
# Build a subset of the Pyroscope protobuf API
- git_repo: https://github.com/grafana/pyroscope.git
# Update this to any commit, which is merged in Pyroscope main
ref: weekly-f105-886b418af
subdir: api
paths:
- adhocprofiles/
- querier/
- settings/
- types/
- vcs/
- google/

- adhocprofiles/
- querier/
- settings/
- types/
- vcs/
- google/
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export function CodeContainer({ dataSourceUid, functionDetails }: CodeContainerP
return (
<>
<Code
lines={data.lines}
lines={data.snippetLines}
unit={data.unit}
githubUrl={data.githubUrl}
isLoadingCode={data.isLoadingCode}
Expand All @@ -39,7 +39,7 @@ export function CodeContainer({ dataSourceUid, functionDetails }: CodeContainerP
<AiSuggestionsPanel
suggestionPromptInputs={{
functionDetails: functionDetails,
lines: data.lines,
lines: data.allLines,
}}
/>
) : null}
Expand Down
Loading
Loading