Skip to content

feat(config): replace askToContinue with onSessionEnd for session-end#14

Merged
Bahaaio merged 3 commits intoBahaaio:mainfrom
timrekelj:feature/onsessionend
Feb 9, 2026
Merged

feat(config): replace askToContinue with onSessionEnd for session-end#14
Bahaaio merged 3 commits intoBahaaio:mainfrom
timrekelj:feature/onsessionend

Conversation

@timrekelj
Copy link
Contributor

  • Replace the boolean askToContinue setting with a string onSessionEnd config field and wire it through runner and UI model initialisation.
  • Support "ask", "skip", and "quit" actions when a session ends so users can control whether the program prompts, automatically starts the next session, or exits immediately.
  • Update README examples, default config values, schema enum/defaults, sample pomo.yaml, and config tests to reflect the new option.

This PR closes issue #10

… behavior (Bahaaio#10)

available options:
- "ask" (same functionality as previous `askToContinue = true`)
- "skip" (skips to the next session)
- "quit" (quits the therminal app)
@timrekelj timrekelj force-pushed the feature/onsessionend branch from 938b9cc to 9ee0b85 Compare February 9, 2026 07:19
@timrekelj
Copy link
Contributor Author

Force push was a typo fix and nicer commit message (with reference to the issue)

@Bahaaio
Copy link
Owner

Bahaaio commented Feb 9, 2026

Good changes!

I just changed the config value from skip to start as I think it's clearer.

Thank you!

@Bahaaio Bahaaio merged commit 3641edb into Bahaaio:main Feb 9, 2026
1 check passed
@Bahaaio Bahaaio added the enhancement New feature or request label Feb 11, 2026
@Bahaaio Bahaaio linked an issue Feb 11, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auto confirm session change

2 participants