Skip to content

fix(prompts): add missing opening <Role> tag to Sisyphus system prompt #682

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
Merged
code-yeongyu merged 1 commit into from
Jan 11, 2026
Merged

fix(prompts): add missing opening <Role> tag to Sisyphus system prompt #682

code-yeongyu merged 1 commit into from
Jan 11, 2026
+3 −1

Conversation

@Coaspe
Copy link
Contributor

@Coaspe Coaspe commented Jan 11, 2026 •
edited by cubic-dev-ai bot
Loading

Summary

  • Fixed a structural inconsistency in the Sisyphus orchestrator prompt.
  • Ensured well-formed tags by adding a missing <Role> opening tag.
  • Improved the determinism of the orchestrator agent's identity definition.

Changes

  • Modified src/agents/omo.ts (or the relevant agent file): Added <Role> opening tag to the start of the ORCHESTRATOR_SISYPHUS_SYSTEM_PROMPT constant.
  • Ensured the closing </Role> tag now has a corresponding opening pair.

Screenshots

Before After
You are "Sisyphus" ... </Role> <Role> You are "Sisyphus" ... </Role>

Testing

bun run typecheck
bun run build

## Related Issues

<!-- Link related issues. Use "Closes #123" to auto-close on merge. -->

<!-- Closes # -->


<!-- This is an auto-generated description by cubic. -->
---
## Summary by cubic
Fixed a malformed Sisyphus system prompt by adding the missing opening <Role> tag. This ensures consistent role parsing and a stable orchestrator identity.

- **Bug Fixes**
  - Added the `<Role>` opening tag to ORCHESTRATOR_SISYPHUS_SYSTEM_PROMPT in src/agents/orchestrator-sisyphus.ts.
  - Paired tags are now well-formed, preventing parser issues and prompt ambiguity.

<sup>Written for commit 3bf945db42e45d73ef7476aa6cf7af7ad9514319. Summary will update on new commits.</sup>

<!-- End of auto-generated description by cubic. -->

@github-actions
Copy link
Contributor

github-actions bot commented Jan 11, 2026 •
edited
Loading

All contributors have signed the CLA. Thank you! ✅
Posted by the CLA Assistant Lite bot.

greptile-apps[bot]
greptile-apps bot reviewed Jan 11, 2026
View reviewed changes
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@Coaspe
Copy link
Contributor Author

Coaspe commented Jan 11, 2026

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 11, 2026
@github-actions
@Coaspe has signed the CLA in #682
1132be3
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Jan 11, 2026
View reviewed changes
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@code-yeongyu code-yeongyu merged commit 24bdc7e into code-yeongyu:dev Jan 11, 2026
3 of 4 checks passed
kdcokenny pushed a commit that referenced this pull request Jan 13, 2026
@github-actions @kdcokenny
@Coaspe has signed the CLA in #682
2f6ed8b
kdcokenny pushed a commit that referenced this pull request Jan 13, 2026
@Coaspe @kdcokenny
fix(prompts): add missing opening <Role> tag to Sisyphus system prompt (
9376444 
#682)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

@code-yeongyu code-yeongyu code-yeongyu approved these changes

+1 more reviewer

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

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

@Coaspe @code-yeongyu