Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

feat: Let the user add their own system prompts - #643

Merged
aponcedeleonch merged 8 commits into
mainfrom
workspace-system-prompt
Jan 20, 2025
Merged

feat: Let the user add their own system prompts#643
aponcedeleonch merged 8 commits into
mainfrom
workspace-system-prompt

Conversation

@aponcedeleonch

@aponcedeleonchaponcedeleonch commented Jan 17, 2025•
edited
Loading

Copy link
Copy Markdown
Member

Related: #454

This PR introduces being able to add system-prompts to workspaces. The workspace system prompts are added in the step that we are using to add CodeGate system prompt. The order of the prompts will be the following assuming all of them are present:

  • CodeGate system prompt
  • Workspace system prompt
  • Request system prompt

The corresponding CLI command is also created and the documentation in the CLI is updated

@aponcedeleonch
aponcedeleonch marked this pull request as draft January 17, 2025 15:29
@aponcedeleonch
aponcedeleonch marked this pull request as ready for review January 20, 2025 10:18
JAORMX
JAORMX suggested changes Jan 20, 2025
View reviewed changes
Comment threadsrc/codegate/api/v1.py
Comment threadsrc/codegate/db/connection.py
Comment threadsrc/codegate/db/connection.py
Comment threadsrc/codegate/db/connection.py
Comment threadsrc/codegate/pipeline/cli/commands.py Outdated
Comment threadsrc/codegate/pipeline/cli/commands.py Outdated
Comment threadsrc/codegate/workspaces/crud.py
Comment threadsrc/codegate/pipeline/cli/commands.py Outdated
Comment threadsrc/codegate/pipeline/cli/commands.py Outdated
@aponcedeleonch
feat: Let the user add their own system prompts
dd9a65e
Related: #454
This PR is not ready yet. For the moment it adds the system prompts
to DB and associates it to a workspace.
It's missing to use the system prompt and actually send it to the
LLM
@aponcedeleonch
Finished functionality to add wrkspace system prompt
a836157
@aponcedeleonch
separated into it's own command system-prompt
d05df39
JAORMX
JAORMX approved these changes Jan 20, 2025
View reviewed changes

@JAORMXJAORMX left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shipit 🚢

@aponcedeleonch
aponcedeleonch merged commit c35d3b1 into mainJan 20, 2025
@aponcedeleonch
aponcedeleonch deleted the workspace-system-prompt branch January 20, 2025 13:46
@lxmcgvrnlxmcgvrn mentioned this pull request Jan 20, 2025
Closed
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@JAORMXJAORMXJAORMX approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aponcedeleonch@JAORMX