Repository navigation
Fix open-PR blockers: save-profile carry-over, Pages cache-bust, profile badge - #92
Merged
Merged
Conversation
Resolve PR #87 conflicts by keeping main's TitleScreen boot try/catch while adding shared-slot detection, carry-over UI, and RPG launch links. Strip default archetype on first RPG entry so casino-only slots stay marked as needing character setup until the creator runs. Co-authored-by: Jack J Burleson // LJM <Exios66@users.noreply.github.com>
Port the remaining unique fixes from PR #23: stamp __ASSET_SHA__ on terminal assets for sync-gh-pages.sh, backfill missing horse spriteIds in the paddock, and set minimum canvas dimensions. Co-authored-by: Jack J Burleson // LJM <Exios66@users.noreply.github.com>
Include the ready-to-copy shields.io snippet from draft PR #61. Co-authored-by: Jack J Burleson // LJM <Exios66@users.noreply.github.com>
Co-authored-by: Jack J Burleson // LJM <Exios66@users.noreply.github.com>
This was referenced Jul 26, 2026
Exios66
marked this pull request as ready for review
July 26, 2026 01:08
Contributor
Author
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
Author
There was a problem hiding this comment.
Risk: medium. Non-blocking approval assessment: bounded web terminal/RPG changes with test evidence and no applicable approval policies; criteria met at medium risk. Cannot post GitHub Approve on this bot-authored PR; human approval was not recorded before merge.
Sent by Cursor Approval Agent: Pull Request Router and Approver
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
Resolves the open-PR backlog left after #89 merged: ports the conflicting save-profile carry-over from #87 onto current
main, restores the still-missing Pages__ASSET_SHA__+ paddock hardening from #23, and includes the draft profile badge from #61. Does not delete.github/workflows/checks.yml.Type
Surfaces touched
mandalay_bay/blackjack/poker)docs/)docs/rpg/)What changed
From #87 (conflicts resolved)
hasRpgProgress,hasCasinoProfileProgress,bootstrapSessionForRpgskipIntro=1)try/catchand touch copychecks.ymlin place (unlike Carry web terminal save profiles into RPG save slots #87)From #23
docs/index.html__ASSET_SHA__placeholders forsync-gh-pages.shspriteIdbackfill + horse canvas min sizeFrom #61
Test plan
uv run pytest -q— 208 passednode scripts/smoke-test-rpg.mjs— 4884 checks passedmainis CLEANNotes
Supersedes the unique remaining work in #87, #23, and #61. #6 is empty against
mainand should be closed. #89 already merged.