Skip to content

Comments

Improve louis ai chatbot prompt#1336

Merged
RichDom2185 merged 2 commits intomasterfrom
feature/louis
Feb 20, 2026
Merged

Improve louis ai chatbot prompt#1336
RichDom2185 merged 2 commits intomasterfrom
feature/louis

Conversation

@yiilinzhang
Copy link
Contributor

Description

The currrent Louis bot tends to use console.log and JS specific functions not available on source. Adjusted the prompt passed in to open ai to emphasize not to use methods not available in Source.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Code quality improvements

Checklist

  • I have tested this code
  • I have updated the documentation

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the Louis chatbot prompt to prevent it from generating JavaScript code that is incompatible with the Source language used in the Source Academy platform. The changes add explicit instructions to the AI model about Source language restrictions and proper output functions.

Changes:

  • Updated the chatbot system prompt to emphasize Source language constraints and proper syntax usage
  • Added explicit guidance to use display or display_list instead of console.log
  • Included warnings about unsupported JavaScript features (classes, modules, async/await, etc.)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coveralls
Copy link

Coverage Status

coverage: 89.178%. remained the same
when pulling 3a06d70 on feature/louis
into 9a7da2c on master.

@yiilinzhang yiilinzhang changed the title Feature/louis Improve louis ai chatbot prompt Feb 20, 2026
Copy link
Member

@RichDom2185 RichDom2185 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@RichDom2185 RichDom2185 merged commit ce7e5e0 into master Feb 20, 2026
7 checks passed
@RichDom2185 RichDom2185 deleted the feature/louis branch February 20, 2026 18:35
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.

3 participants