Skip to content

Conversation

@vsumner
Copy link
Contributor

@vsumner vsumner commented Jan 19, 2026

Summary

  • extract Sisyphus prompt constants into a dedicated module
  • wire sisyphus.ts to consume the new constants
  • reduce prompt source size while preserving behavior

Summary by cubic

Extracted all Sisyphus prompt strings into a dedicated constants module and updated the agent to import them, reducing noise in sisyphus.ts and centralizing prompt content without changing behavior.

  • Refactors
    • Moved prompt literals to src/agents/sisyphus/constants.ts and re-exports via src/agents/sisyphus/index.ts.
    • Updated buildDynamicSisyphusPrompt to consume constants; removed inline definitions.
    • Fixed delegation examples to use run_in_background and replaced dynamic constraint assembly with SISYPHUS_CONSTRAINTS.

Written for commit 7a88703. Summary will update on new commits.

vsumner and others added 2 commits January 18, 2026 20:36
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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.

1 issue found across 3 files

Confidence score: 3/5

  • The prompt example in src/agents/sisyphus/constants.ts uses an unsupported background flag, which can cause delegate_task behavior to fail unless run_in_background is used instead.
  • This is a concrete, user-facing mismatch in a documented example, so there is some risk if users follow it as-is.
  • Pay close attention to src/agents/sisyphus/constants.ts - prompt example flag naming mismatch could break expected background execution.
Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="src/agents/sisyphus/constants.ts">

<violation number="1" location="src/agents/sisyphus/constants.ts:222">
P2: Prompt example uses unsupported `background` flag; delegate_task requires `run_in_background`.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@kdcokenny
Copy link
Collaborator

I like the idea but please resolve the conflicts.

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.

2 participants