Skip to content

Conversation

@zejjnt
Copy link

@zejjnt zejjnt commented Oct 25, 2025

User description

The "primer" theme does not correctly reset the bash prompt to normal mode, the creator probably missed the variable and no one noticed.


PR Type

Bug fix


Description

  • Add missing ${_omb_prompt_normal} variable to prompt reset

  • Ensures bash prompt correctly returns to normal mode

  • Fixes incomplete color reset in primer theme


Diagram Walkthrough

flowchart LR
  A["Primer Theme Prompt"] -->|Missing normal reset| B["Incomplete color reset"]
  A -->|Add _omb_prompt_normal| C["Correct prompt behavior"]
Loading

File Walkthrough

Relevant files
Bug fix
primer.theme.sh
Add missing prompt normal reset variable                                 

themes/primer/primer.theme.sh

  • Added missing ${_omb_prompt_normal} variable to the PS1 prompt string
  • Placed after ${_omb_prompt_reset_color} to properly reset prompt to
    normal mode
  • Fixes incomplete color reset that was preventing proper prompt display
+1/-1     

@qodo-free-for-open-source-projects

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🎫 No ticket provided
- [ ] Create ticket/issue <!-- /create_ticket --create_ticket=true -->

</details></td></tr>
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
No custom compliance provided

Follow the guide to enable custom compliance check.

Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-free-for-open-source-projects

PR Code Suggestions ✨

No code suggestions found for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant