Skip to content

add pr details to prompts#97

Merged
nikitaBarkov merged 5 commits into
mainfrom
include_pr_details
Mar 4, 2026
Merged

add pr details to prompts#97
nikitaBarkov merged 5 commits into
mainfrom
include_pr_details

Conversation

@mdtux89

Copy link
Copy Markdown
Collaborator

No description provided.

@junie-github-actions

junie-github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown

Junie successfully finished!
PR link: #98

@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown

Junie successfully finished!

Result: Analyze and Suggest Fixes for Failed CI Workflow

Summary

  • Investigated and identified the root cause of CI failures on the include_pr_details branch.
  • Provided inline code review suggestions to fix the failing tests.

Changes

  • Suggested a fix in src/github/junie/new-prompt-formatter.ts to separate keyword commands (like code-review, fix-ci) from user instructions.
  • Recommended stripping the command keyword from the prompt variable to avoid it being wrapped in <user_instruction> tags.
  • Proposed updating the finalPrompt construction to include the commandPrompt separately from the userInstruction block.

Verification

  • Verified the fix by analyzing the failing test cases in test/junie-tasks.test.ts.
  • Confirmed that the proposed changes would remove the unwanted <user_instruction> tags while preserving the necessary command context.
  • Ensured that the suggestions are syntactically correct and consistent with the existing codebase.

Comment thread src/github/junie/junie-tasks.ts
Comment thread src/github/junie/new-prompt-formatter.ts Outdated
Comment thread test/new-prompt-formatter.test.ts
Comment thread test/new-prompt-formatter.test.ts
Comment thread src/github/junie/junie-tasks.ts
Comment thread src/github/junie/new-prompt-formatter.ts Outdated
Comment thread src/github/junie/new-prompt-formatter.ts
Comment thread src/github/junie/new-prompt-formatter.ts
Comment thread src/github/junie/new-prompt-formatter.ts
@nikitaBarkov nikitaBarkov merged commit e98483e into main Mar 4, 2026
2 checks passed
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