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

update .openhands instructions #5821

Merged
merged 13 commits into from
Dec 26, 2024
Merged

update .openhands instructions #5821

merged 13 commits into from
Dec 26, 2024

Conversation

rbren
Copy link
Collaborator

@rbren rbren commented Dec 26, 2024

End-user friendly description of the problem this fixes or functionality that this introduces

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below
    CHANGELOG: Documented the new .openhands directory for customizing prompts for individual repositories

Give a summary of what the PR does, explaining any non-trivial design decisions

  • Makes it easier to write a generic microagent (no YAML metadata at the top of the file == always trigger)
  • Puts the generic instructions into the system prompt (this seems to have been missing?)
  • Documents the new pattern

Link of any specific issues this addresses


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:02072eb-nikolaik   --name openhands-app-02072eb   docker.all-hands.dev/all-hands-ai/openhands:02072eb

.openhands/microagents/repo.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@tofarr tofarr left a comment

Choose a reason for hiding this comment

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

🍰

@rbren rbren enabled auto-merge (squash) December 26, 2024 17:49
<REPOSITORY_INSTRUCTIONS>
{{ repo_instructions }}
</REPOSITORY_INSTRUCTIONS>
{% endif %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

As a side note, i think it probably makes more sense to add repository instructions in the first USER message rather than system message, which is typically used to define the behavior of LLM?

@rbren rbren merged commit e2a25f1 into main Dec 26, 2024
16 checks passed
@rbren rbren deleted the rb/openhands-instructions branch December 26, 2024 18:15
@@ -1,28 +0,0 @@
OpenHands is an automated AI software engineer. It is a repo with a Python backend
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since this was deleted, will the resolver find it? It doesn't seem so

openhands_instructions_path = os.path.join(repo_dir, '.openhands_instructions')

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we should create a new issue for it

Copy link
Collaborator

Choose a reason for hiding this comment

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

Done -ish, here, it's expressed as a quick prompt for oh #5830 , but its fix was to hardcode the new path... That can't be what we want for a microagent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants