Improve guess ordering and add data policy warning#36
Merged
SupraSummus merged 2 commits intomainfrom Apr 12, 2026
Merged
Conversation
- Order guesses with nulls_first so pending guesses (embedding not yet computed) appear at the top of the list immediately after submission - Add "Don't share any private, secret, or classified information!" disclaimer to the guess form, matching the warrior create page https://claude.ai/code/session_01G6KHKtYCqjXQHSgPRPZ8js
Adds a "Last guess" heading with the most recently submitted guess, so users can immediately see how their latest attempt did, separate from the full list ordered by distance. https://claude.ai/code/session_01G6KHKtYCqjXQHSgPRPZ8js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR improves the guess ordering logic and adds a data policy warning to the guess submission form.
Key Changes
_annotated_guesses()to useF('distance').asc(nulls_first=True)instead of simpleorder_by('distance'). This ensures explicit null handling, placing null distances first in the result set.Implementation Details
Fexpressions for more explicit control over null value placement in the query results.<strong>tag within the form, making it visually prominent to users before they submit their guess.reverse()function to reference thedata_policy:rootURL.https://claude.ai/code/session_01G6KHKtYCqjXQHSgPRPZ8js