Skip to content
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

Change prompts to improve game directions format and map clarity #125

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mentatai[bot]
Copy link
Contributor

@mentatai mentatai bot commented Mar 14, 2025

Implements the changes requested in #123:

  1. Updated map display in prompts:

    • Removed row/column numbers from the displayed map
    • Added explanation of map symbols
  2. Changed direction formatting:

    • Changed from "7 steps up-left" to "3 left, 4 up" format
    • Applied this format to coins, water tiles, and unit descriptions
  3. Changed border terminology:

    • Changed from "northern" to "top", etc.
    • Made descriptions more intuitive
  4. Specified team dynamics:

    • Added information that units A and B are on the same team
    • Encouraged cooperative coin collection

Closes #123


🤖 See my steps and cost here

  • Wake on any new activity.

Implements the changes requested in # 123:

1. Updated map display in prompts:
   - Removed row/column numbers from the displayed map
   - Added explanation of map symbols

2. Changed direction formatting:
   - Changed from "7 steps up-left" to "3 left, 4 up" format
   - Applied this format to coins, water tiles, and unit descriptions

3. Changed border terminology:
   - Changed from "northern" to "top", etc.
   - Made descriptions more intuitive

4. Specified team dynamics:
   - Added information that units A and B are on the same team
   - Encouraged cooperative coin collection

Closes # 123

Mentat precommits failed. Log: https://qa.mentat.ai/log/bd9be113-74ad-4a8b-bb0c-e3d8840423b6
@mentatai mentatai bot marked this pull request as ready for review March 14, 2025 21:36
@mentatai mentatai bot requested a review from biobootloader March 14, 2025 21:36
The test was failing because it was checking for "Current Game State:" in the description,
but in our changes we updated this string to "Current state of the map:". This commit
updates the test to check for the new string.

- Updated the test assertion to look for "Current state of the map:"
- No changes to actual functionality

Mentat precommits passed. Log: https://qa.mentat.ai/log/b1b0f52c-c181-45b9-aa3b-bdeefbf9080d
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.

change prompts
0 participants