Skip to content

fix(tui): persist reverted exercise text - #162

Open
suganthan2005 wants to merge 2 commits into
abhiksark:devfrom
suganthan2005:fix/issue-74-persist-reverted-text
Open

suganthan2005 wants to merge 2 commits into
abhiksark:devfrom
suganthan2005:fix/issue-74-persist-reverted-text

Conversation

@suganthan2005

Copy link
Copy Markdown

Summary

  • Track the text actually persisted to disk so reverting to the initially loaded text schedules a save after an earlier autosave.
  • Flush the active TrackScreen for app-level Ctrl+Q, including when a docs screen is open.
  • Add Textual pilot coverage for autosave, rapid edits, initial loading, F4/Escape/Ctrl+Q, and quitting from docs.

Closes #74

Tests

  • python -m pytest tests/tui/test_app_pilot.py -q — 30 passed
  • python -m pytest -q — 221 passed
  • pythonlings --root tests/fixtures/passing_curriculum verify — passing1, passing2

Python 3.9 was not installed in the local environment; the change uses existing Python 3.9-compatible syntax and the repository CI matrix covers Python 3.9.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0665bbb2-ab06-4bc0-99f5-d562cd79adc1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch has not been deployed

No deployments
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.

1 participant