Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

This commit addresses three issues:

  1. The cursor was visible in recorded casts. This was fixed by changing the trap handlers in the screensaver scripts to only catch SIGINT, so that the cursor is not restored when the script is terminated by timeout.
  2. The tour-the-gallery.sh script was creating new casts for title cards. This was changed to only use existing casts.
  3. The test suite was broken due to incorrect paths for loading dependencies. The paths in the .bats files have been corrected.

PR created automatically by Jules for task 13500521981430321241

… to only catch SIGINT. This prevents cursor restoration from being triggered by the `timeout` command during recording.

Update `tour-the-gallery.sh` to use existing cast files generated by `smile-for-the-camera.sh` instead of creating new title card casts.

Correct the `load` paths in all `bats` test files to be relative to the project root, fixing the test dependency loading issue.
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll acknowledge your comments with a 👀 emoji and then get to work. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

I will automatically address your feedback. For any comments you don't want me to act on, just include (aside).



For security, I will only act on instructions from the user who triggered this task for this pull request.

Update `tour-the-gallery.sh` to use existing cast files.
Correct `load` paths in `bats` test files.
Update all `.cast` files with new recordings.
@attogram attogram marked this pull request as ready for review September 13, 2025 11:24
@attogram attogram merged commit 544dfca into main Sep 13, 2025
1 check failed
@attogram attogram deleted the fix-casts-tour-and-tests branch September 13, 2025 11:24
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