Skip to content

fix(prompts.ts): edited contradictory assistant output #477

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

Merged
merged 1 commit into from
May 30, 2025

Conversation

jonsguez
Copy link
Contributor

When user wanted a one line commit the system prompt and the user/assistant one-shot example were contradicting each other, confusing the LLM. This fix modifies the assistant output so that prompt and one-shot are consistent.

see Issue #469

Before:

OCO_ONE_LINE_COMMIT=true OCO_DESCRIPTION=false oco
└  Generated commit message:
——————————————————
feat(algo_2.py): replace hardcoded debug_stop value with environment variable for better configurability 
feat(buffers.py): add BufferVal class to encapsulate value and timestamp for improved data handling
OCO_ONE_LINE_COMMIT=true OCO_DESCRIPTION=true oco
└  Generated commit message:
——————————————————
feat(algo_2.py): replace hardcoded debug_stop value with environment variable for flexibility
feat(buffers.py): add BufferVal class to encapsulate value and timestamp
Replacing the hardcoded value of debug_stop with an environment variable allows for greater flexibility in debugging scenarios. The addition of the BufferVal class provides a structured way to manage data values along with their timestamps, enhancing the overall data handling capabilities in the application.

After:

OCO_ONE_LINE_COMMIT=true OCO_DESCRIPTION=false oco
└  Generated commit message:
——————————————————
feat(algo_2.py, buffers.py): add environment variable support for debug stop and implement BufferVal class for enhanced data handling
OCO_ONE_LINE_COMMIT=true OCO_DESCRIPTION=true oco
└  Generated commit message:
——————————————————
feat(algo_2.py, buffers.py): add environment variable support for debug stop and implement BufferVal class for enhanced data handling
The changes introduce the ability to set the debug stop value via an environment variable, allowing for more flexible debugging configurations. Additionally, a new BufferVal class is added to encapsulate value and timestamp, improving data management within the DataProcessingBuffer.

When user wants one line commits the system prompt and the user/assistant one-shot example were contradicting each other, confusing the LLM. This fix modifies the assistant output so that prompt and one-shot are consistent.
@di-sukharev di-sukharev merged commit c1be513 into di-sukharev:master May 30, 2025
4 checks passed
@jonsguez jonsguez deleted the fix/one-line-commit branch May 30, 2025 11:55
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